module: experimental modules runMain separation

PR-URL: #21350
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Gus Caplan <[email protected]>
Reviewed-By: Bradley Farias <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>

Author: guybedford

lib/internal/modules/cjs/loader.js

@@ -504,19 +504,6 @@ Module._load = function(request, parent, isMain) {
     debug('Module._load REQUEST %s parent: %s', request, parent.id);
   }
 
-  if (experimentalModules && isMain) {
-    if (asyncESM === undefined) lazyLoadESM();
-    asyncESM.loaderPromise.then((loader) => {
-      return loader.import(getURLFromFilePath(request).pathname);
-    })
-    .catch((e) => {
-      decorateErrorStack(e);
-      console.error(e);
-      process.exit(1);
-    });
-    return;
-  }
-
   var filename = Module._resolveFilename(request, parent, isMain);
 
   var cachedModule = Module._cache[filename];
@@ -741,7 +728,19 @@ if (experimentalModules) {
 // bootstrap main module.
 Module.runMain = function() {
   // Load the main module--the command line argument.
-  Module._load(process.argv[1], null, true);
+  if (experimentalModules) {
+    if (asyncESM === undefined) lazyLoadESM();
+    asyncESM.loaderPromise.then((loader) => {
+      return loader.import(getURLFromFilePath(process.argv[1]).pathname);
+    })
+    .catch((e) => {
+      decorateErrorStack(e);
+      console.error(e);
+      process.exit(1);
+    });
+  } else {
+    Module._load(process.argv[1], null, true);
+  }
   // Handle any nextTicks added in the first tick of the program
   process._tickCallback();
 };

test/es-module/test-esm-cjs-main.js

@@ -1,6 +1,27 @@
-// Flags: --experimental-modules
 'use strict';
-require('../common');
+
+const common = require('../common');
+const fixtures = require('../common/fixtures');
+const { spawn } = require('child_process');
 const assert = require('assert');
-exports.asdf = 'asdf';
-assert.strictEqual(require.main.exports.asdf, 'asdf');
+
+const entry = fixtures.path('/es-modules/cjs.js');
+
+const child = spawn(process.execPath, ['--experimental-modules', entry]);
+let experimentalWarning = false;
+let validatedExecution = false;
+child.stderr.on('data', (data) => {
+  if (!experimentalWarning) {
+    experimentalWarning = true;
+    return;
+  }
+  throw new Error(data.toString());
+});
+child.stdout.on('data', (data) => {
+  assert.strictEqual(data.toString(), 'executed\n');
+  validatedExecution = true;
+});
+child.on('close', common.mustCall((code, stdout) => {
+  assert.strictEqual(validatedExecution, true);
+  assert.strictEqual(code, 0);
+}));

test/es-module/test-esm-error-cache.js

@@ -7,7 +7,7 @@ const assert = require('assert');
 
 common.crashOnUnhandledRejection();
 
-const file = '../../fixtures/syntax/bad_syntax.js';
+const file = '../fixtures/syntax/bad_syntax.js';
 
 let error;
 (async () => {

test/fixtures/es-modules/cjs.js

@@ -0,0 +1,5 @@
+'use strict';
+
+// test we can use commonjs require
+require('path');
+console.log('executed');