src: disable uncaught exception abortion for ESM syntax detection

Author: anonrig

src/node_contextify.cc

@@ -1448,6 +1448,19 @@ void ContextifyContext::ContainsModuleSyntax(
     code = args[0].As<String>();
   }
 
+  // We need to temporarily disable aborting on uncaught exceptions because
+  // ScriptCompiler::CompileFunction will abort if it encounters an uncaught
+  // exception. This can be reproduced with `--abort-on-uncaught-exception`
+  // flag.
+  bool should_abort_on_uncaught_exception = env->abort_on_uncaught_exception();
+  // Validate that we're restoring the original value.
+  auto on_scope_leave =
+      OnScopeLeave([&env, &should_abort_on_uncaught_exception]() {
+        CHECK_EQ(env->abort_on_uncaught_exception(),
+                 should_abort_on_uncaught_exception);
+      });
+  env->set_abort_on_uncaught_exception(false);
+
   // TODO(geoffreybooth): Centralize this rather than matching the logic in
   // cjs/loader.js and translators.js
   Local<String> script_id = String::Concat(
@@ -1491,6 +1504,10 @@ void ContextifyContext::ContainsModuleSyntax(
       }
     }
   }
+
+  // Revert the abort on uncaught exception setting.
+  env->set_abort_on_uncaught_exception(should_abort_on_uncaught_exception);
+
   args.GetReturnValue().Set(found_error_message_caused_by_module_syntax);
 }
 

test/es-module/test-esm-detect-ambiguous.mjs

@@ -235,3 +235,23 @@ describe('--experimental-detect-module', { concurrency: true }, () => {
     }
   });
 });
+
+// Validate temporarily disabling `--abort-on-uncaught-exception`
+// while running `containsModuleSyntax`.
+// Ref: https://github.com/nodejs/node/issues/50878
+describe('Wrapping a `require` of an ES module while using `--abort-on-uncaught-exception`', () => {
+  it('should work', async () => {
+    const { code, signal, stdout, stderr } = await spawnPromisified(process.execPath, [
+      '--abort-on-uncaught-exception',
+      '--eval',
+      'assert.throws(() => require("./package-type-module/esm.js"), { code: "ERR_REQUIRE_ESM" })',
+    ], {
+      cwd: fixtures.path('es-modules'),
+    });
+
+    strictEqual(stderr, '');
+    strictEqual(stdout, '');
+    strictEqual(code, 0);
+    strictEqual(signal, null);
+  });
+});