module: clarify cjs global-like error on ModuleJobSync

PR-URL: #56491 Backport-PR-URL: #59504 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Jacob Smith <jacob@frende.me> Refs: #52697

Author: Ceres6

lib/internal/modules/esm/module_job.js

@@ -58,6 +58,37 @@ const isCommonJSGlobalLikeNotDefinedError = (errorMessage) =>
  (globalLike) => errorMessage === `${globalLike} is not defined`,
  );
 
+
+/**
+ *
+ * @param {Error} e
+ * @param {string} url
+ * @returns {void}
+ */
+const explainCommonJSGlobalLikeNotDefinedError = (e, url) => {
+ if (e?.name === 'ReferenceError' &&
+ isCommonJSGlobalLikeNotDefinedError(e.message)) {
+ e.message += ' in ES module scope';
+
+ if (StringPrototypeStartsWith(e.message, 'require ')) {
+ e.message += ', you can use import instead';
+ }
+
+ const packageConfig =
+ StringPrototypeStartsWith(url, 'file://') &&
+ RegExpPrototypeExec(/\.js(\?[^#]*)?(#.*)?$/, url) !== null &&
+ require('internal/modules/esm/resolve')
+ .getPackageScopeConfig(url);
+ if (packageConfig.type === 'module') {
+ e.message +=
+ '\nThis file is being treated as an ES module because it has a ' +
+ `'.js' file extension and '${packageConfig.pjsonPath}' contains ` +
+ '"type": "module". To treat it as a CommonJS script, rename it ' +
+ 'to use the \'.cjs\' file extension.';
+ }
+ }
+};
+
 class ModuleJobBase {
  constructor(url, importAttributes, isMain, inspectBrk) {
  this.importAttributes = importAttributes;
@@ -285,27 +316,7 @@ class ModuleJob extends ModuleJobBase {
  try {
  await this.module.evaluate(timeout, breakOnSigint);
  } catch (e) {
- if (e?.name === 'ReferenceError' &&
- isCommonJSGlobalLikeNotDefinedError(e.message)) {
- e.message += ' in ES module scope';
-
- if (StringPrototypeStartsWith(e.message, 'require ')) {
- e.message += ', you can use import instead';
- }
-
- const packageConfig =
- StringPrototypeStartsWith(this.module.url, 'file://') &&
- RegExpPrototypeExec(/\.js(\?[^#]*)?(#.*)?$/, this.module.url) !== null &&
- require('internal/modules/esm/resolve')
- .getPackageScopeConfig(this.module.url);
- if (packageConfig.type === 'module') {
- e.message +=
- '\nThis file is being treated as an ES module because it has a ' +
- `'.js' file extension and '${packageConfig.pjsonPath}' contains ` +
- '"type": "module". To treat it as a CommonJS script, rename it ' +
- 'to use the \'.cjs\' file extension.';
- }
- }
+ explainCommonJSGlobalLikeNotDefinedError(e, this.module.url);
  throw e;
  }
  return { __proto__: null, module: this.module };
@@ -409,8 +420,13 @@ class ModuleJobSync extends ModuleJobBase {
  throw new ERR_REQUIRE_ASYNC_MODULE(filename, parentFilename);
  }
  setHasStartedUserESMExecution();
- const namespace = this.module.evaluateSync(filename, parentFilename);
- return { __proto__: null, module: this.module, namespace };
+ try {
+ const namespace = this.module.evaluateSync(filename, parentFilename);
+ return { __proto__: null, module: this.module, namespace };
+ } catch (e) {
+ explainCommonJSGlobalLikeNotDefinedError(e, this.module.url);
+ throw e;
+ }
  }
 }
 

test/es-module/test-require-module-error-catching.js

@@ -17,5 +17,5 @@ assert.throws(() => {
  require('../fixtures/es-modules/reference-error-esm.js');
 }, {
  name: 'ReferenceError',
- message: 'exports is not defined'
+ message: 'exports is not defined in ES module scope'
 });