@mongosh/node-runtime-worker-thread 2.1.0 → 2.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mongosh/node-runtime-worker-thread",
-  "version": "2.1.0",
+  "version": "2.1.1",
   "description": "MongoDB shell runtime that lives in a worker thread",
   "homepage": "https://github.com/mongodb-js/mongosh",
   "license": "Apache-2.0",
@@ -37,11 +37,11 @@
     "@mongodb-js/eslint-config-mongosh": "^1.0.0",
     "@mongodb-js/prettier-config-devtools": "^1.0.1",
     "@mongodb-js/tsconfig-mongosh": "^1.0.0",
-    "@mongosh/browser-runtime-core": "2.1.0",
-    "@mongosh/browser-runtime-electron": "2.1.0",
-    "@mongosh/service-provider-core": "2.1.0",
-    "@mongosh/service-provider-server": "2.1.0",
-    "@mongosh/types": "2.1.0",
+    "@mongosh/browser-runtime-core": "2.1.1",
+    "@mongosh/browser-runtime-electron": "2.1.1",
+    "@mongosh/service-provider-core": "2.1.1",
+    "@mongosh/service-provider-server": "2.1.1",
+    "@mongosh/types": "2.1.1",
     "bson": "^6.2.0",
     "depcheck": "^1.4.3",
     "eslint": "^7.25.0",
@@ -54,5 +54,5 @@
     "interruptor": "^1.0.1",
     "system-ca": "^1.0.2"
   },
-  "gitHead": "963c75bc45834b1bf69e62bf5b72c4ae8ef26cb4"
+  "gitHead": "c7e70b65988d21c9020abf2bbd253a94b1d3316c"
 }

package/src/index.spec.ts

@@ -144,6 +144,50 @@ describe('WorkerRuntime', function () {
           .to.have.property('stack')
           .matches(/SyntaxError: Syntax!/);
       });
+
+      it('COMPASS-5919 - correctly serializes babel parse errors', async function () {
+        /**
+         * babel syntax errors have a `clone()` method, which breaks structured cloning
+         */
+        runtime = new WorkerRuntime('mongodb://nodb/', dummyOptions, {
+          nodb: true,
+        });
+
+        const err: Error = await runtime.evaluate('1 +* 3').catch((e) => e);
+
+        expect(err).to.be.instanceof(Error);
+        expect(err).to.have.property('name', 'SyntaxError');
+      });
+
+      context(
+        'when `evaluate` returns an error that has a function property',
+        function () {
+          it('removes the function property from the error', async function () {
+            runtime = new WorkerRuntime('mongodb://nodb/', dummyOptions, {
+              nodb: true,
+            });
+
+            const script = `
+          class CustomError extends Error {
+            constructor() {
+              super('custom error');
+            }
+            foo() {
+              return 'hello, world';
+            }
+          }
+          throw new CustomError();
+          `;
+
+            const err: Error = await runtime.evaluate(script).catch((e) => e);
+
+            expect(err).to.be.instanceof(Error);
+            expect(err).to.have.property('name', 'Error');
+            expect(err).not.to.have.property('foo');
+            expect(err).to.have.property('message', 'custom error');
+          });
+        }
+      );
     });
   });
 

package/src/serializer.ts

@@ -27,7 +27,10 @@ function getNames<T>(obj: T): (keyof T)[] {
  */
 export function serializeError(err: Error) {
   // Name is the only constructor property we care about
-  const keys = getNames(err).concat('name');
+  const keys = getNames(err)
+    .concat('name')
+    // structured cloning cannot handle functions
+    .filter((key) => typeof err[key] !== 'function');
   return keys.reduce((acc, key) => {
     (acc as any)[key] = err[key];
     return acc;