Properly clean up the "close" handlers to avoid the watch script from spawning a myriad of Electron processes

1 files changed, 4 insertions, 6 deletions

diff --git a/gui/packages/desktop/scripts/serve.js b/gui/packages/desktop/scripts/serve.js
index 10c0cd2086..e2b91813dc 100644
--- a/gui/packages/desktop/scripts/serve.js
+++ b/gui/packages/desktop/scripts/serve.js
@@ -27,15 +27,13 @@ function runElectron(browserSyncUrl) {
     },
     stdio: 'inherit',
   });
-  child.once('close', onCloseElectron);
+  child.once('close', () => {
+    process.exit();
+  });
 
   return child;
 }
 
-function onCloseElectron() {
-  process.exit();
-}
-
 function startBrowserSync() {
   bsync.init(
     {
@@ -62,7 +60,7 @@ function startBrowserSync() {
       bsync
         .watch(['build/src/config.json', 'build/src/main/**/*', 'build/src/shared/**/*'])
         .on('change', () => {
-          child.removeListener('close', onCloseElectron);
+          child.removeAllListeners('close');
           child.once('close', () => {
             child = runElectron(browserSyncUrl);
           });