ignore normal exit when reporting API process close (#431)

Author: calebdoxsey

.gitignore

@@ -34,6 +34,7 @@ node_modules
 release
 src/main.prod.js
 src/main.prod.js.map
+src/main.prod.js.LICENSE.txt
 src/renderer.prod.js
 src/renderer.prod.js.map
 src/style.css

package.json

@@ -253,7 +253,7 @@
     "webpack": "^5.94.0",
     "webpack-bundle-analyzer": "^4.10.2",
     "webpack-cli": "^4.7.2",
-    "webpack-dev-server": "^5.1.0",
+    "webpack-dev-server": "^4.15.2",
     "webpack-merge": "^5.10.0",
     "yarn-deduplicate": "^6.0.2"
   },

src/cli.ts

@@ -1,5 +1,5 @@
 import { ChildProcessWithoutNullStreams, spawn } from 'child_process';
-import * as Sentry from '@sentry/electron';
+import * as Sentry from '@sentry/electron/main';
 import { ChannelCredentials } from '@grpc/grpc-js';
 import { createServer } from 'net';
 
@@ -39,10 +39,14 @@ export async function start(sentryDSN: string): Promise<CLI> {
   });
   process.on('close', (code, signal) => {
     if (signal != null) return;
-    Sentry.captureEvent({
-      message: 'API process unexpectedly quit',
-      extra: { code },
-    });
+    if ((code || 0) > 0) {
+      // code == -2 when the child process didn't start at all,
+      // code == 0 when we close the child process
+      Sentry.captureEvent({
+        message: 'API process unexpectedly quit',
+        extra: { code },
+      });
+    }
   });
 
   const configClient = new ConfigClient(

src/main.dev.ts

@@ -16,7 +16,7 @@ import {
   MessageBoxOptions,
 } from 'electron';
 import * as grpc from '@grpc/grpc-js';
-import * as Sentry from '@sentry/electron';
+import * as Sentry from '@sentry/electron/main';
 import log from 'electron-log';
 import { autoUpdater } from 'electron-updater';
 import { menubar } from 'menubar';

src/main.prod.js.LICENSE.txt

@@ -1,4 +1,4 @@
-import * as Sentry from '@sentry/electron';
+import * as Sentry from '@sentry/electron/main';
 import { BrowserWindow } from 'electron';
 import { getAssetPath } from '../main/binaries';
 import MenuBuilder from './menu';