@mongosh/node-runtime-worker-thread 1.6.0 → 1.6.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mongosh/node-runtime-worker-thread",
-  "version": "1.6.0",
+  "version": "1.6.1",
   "description": "MongoDB shell runtime that lives in a worker thread",
   "homepage": "https://github.com/mongodb-js/mongosh",
   "license": "Apache-2.0",
@@ -28,22 +28,22 @@
     "prepublish": "npm run webpack-build"
   },
   "devDependencies": {
-    "@mongosh/browser-runtime-core": "1.6.0",
-    "@mongosh/browser-runtime-electron": "1.6.0",
-    "@mongosh/service-provider-core": "1.6.0",
-    "@mongosh/service-provider-server": "1.6.0",
-    "@mongosh/types": "1.6.0",
+    "@mongosh/browser-runtime-core": "1.6.1",
+    "@mongosh/browser-runtime-electron": "1.6.1",
+    "@mongosh/service-provider-core": "1.6.1",
+    "@mongosh/service-provider-server": "1.6.1",
+    "@mongosh/types": "1.6.1",
     "bson": "^4.7.0",
     "mocha": "^7.1.2",
     "postmsg-rpc": "^2.4.0",
     "terser-webpack-plugin": "^4.2.3",
     "ts-loader": "^8.0.14",
-    "webpack": "^4.44.2",
+    "webpack": "^5.75.0",
     "webpack-cli": "^4.3.1"
   },
   "dependencies": {
     "interruptor": "^1.0.1",
     "system-ca": "^1.0.2"
   },
-  "gitHead": "fbbf54a08fb315ae5fde693a98d439b9aa3285f8"
+  "gitHead": "51c067456d3e0127156ef1722a44659151e01a12"
 }

package/src/child-process-proxy.spec.ts

@@ -1,8 +1,10 @@
 import path from 'path';
-import { ChildProcess, fork } from 'child_process';
+import { ChildProcess, fork, spawn } from 'child_process';
 import { Caller, cancel, createCaller } from './rpc';
 import { expect } from 'chai';
 import { WorkerRuntime } from './worker-runtime';
+import { once } from 'events';
+import { promisify } from 'util';
 
 const childProcessModulePath = path.resolve(
   __dirname,
@@ -22,7 +24,7 @@ describe('child process worker proxy', () => {
     }
 
     if (childProcess) {
-      childProcess.kill('SIGTERM');
+      childProcess.disconnect();
       childProcess = null;
     }
   });
@@ -34,4 +36,38 @@ describe('child process worker proxy', () => {
     const result = await caller.evaluate('1 + 1');
     expect(result.printable).to.equal(2);
   });
+
+  it('should exit on its own when the parent process disconnects', async() => {
+    const intermediateProcess = spawn(process.execPath,
+      ['-e', `require("child_process")
+         .fork(${JSON.stringify(childProcessModulePath)})
+         .on("message", function(m) { console.log("message " + m + " from " + this.pid) })`],
+      { stdio: ['pipe', 'pipe', 'inherit'] });
+
+    // Make sure the outer child process runs and has created the inner child process
+    const [message] = await once(intermediateProcess.stdout.setEncoding('utf8'), 'data');
+    const match = message.trim().match(/^message ready from (?<pid>\d+)$/);
+    expect(match).to.not.equal(null);
+
+    // Make sure the inner child process runs
+    const childPid = +match.groups.pid;
+    process.kill(childPid, 0);
+
+    // Kill the intermediate process and wait for the inner child process to also close
+    intermediateProcess.kill('SIGTERM');
+    let innerChildHasStoppedRunning = false;
+    for (let i = 0; i < 200; i++) {
+      try {
+        process.kill(childPid, 0);
+      } catch (err) {
+        if (err.code === 'ESRCH') {
+          innerChildHasStoppedRunning = true;
+          break;
+        }
+        throw err;
+      }
+      await promisify(setTimeout)(10);
+    }
+    expect(innerChildHasStoppedRunning).to.equal(true);
+  });
 });

package/src/child-process-proxy.ts

@@ -114,6 +114,7 @@ const messageBus = createCaller(['emit', 'on'], process);
 
 exposeAll(messageBus, workerProcess);
 
+process.once('disconnect', () => process.exit());
 process.nextTick(() => {
   // eslint-disable-next-line chai-friendly/no-unused-expressions
   process.send?.('ready');