module,src: add --experimental-entry-url flag

aduh95 · Sep 30, 2023 · Sep 30, 2023 · Sep 30, 2023 · 5a2a8aa1ce90258715223033ce9be4e398d86bbc · GeoffreyBooth
commit 5a2a8aa1ce90258715223033ce9be4e398d86bbc
diff --git a/doc/api/cli.md b/doc/api/cli.md
@@ -617,6 +617,17 @@ files with no extension will be treated as WebAssembly if they begin with the
 WebAssembly magic number (`\0asm`); otherwise they will be treated as ES module
 JavaScript.
 
+### `--experimental-entry-url`
+
+<!-- YAML
+added:
+  - REPLACEME
+-->
+
+> Stability: 1.0 - Early development
+
+When present, Node.js will interpret the entry point as a URL, rather than a path.
+
 ### `--experimental-import-meta-resolve`
 
 <!-- YAML
@@ -2274,6 +2285,7 @@ Node.js options that are allowed are:
 * `--enable-source-maps`
 * `--experimental-abortcontroller`
 * `--experimental-default-type`
+* `--experimental-entry-url`
 * `--experimental-import-meta-resolve`
 * `--experimental-json-modules`
 * `--experimental-loader`

diff --git a/lib/internal/main/run_main_module.js b/lib/internal/main/run_main_module.js
@@ -6,8 +6,9 @@ const {
   prepareMainThreadExecution,
   markBootstrapComplete,
 } = require('internal/process/pre_execution');
+const { getOptionValue } = require('internal/options');
 
-prepareMainThreadExecution(true);
+prepareMainThreadExecution(!getOptionValue('--experimental-entry-url'));
 
 markBootstrapComplete();
 

diff --git a/lib/internal/modules/run_main.js b/lib/internal/modules/run_main.js
@@ -12,6 +12,9 @@ const path = require('path');
  * @param {string} main Entry point path
  */
 function resolveMainPath(main) {
+  if (getOptionValue('--experimental-default-type') === 'module' || getOptionValue('--experimental-entry-url')) {
+    return;
+  }
   // Note extension resolution for the main entry point can be deprecated in a
   // future major.
   // Module._findPath is monkey-patchable here.
@@ -33,6 +36,7 @@ function resolveMainPath(main) {
  * @param {string} mainPath Absolute path to the main entry point
  */
 function shouldUseESMLoader(mainPath) {
+  if (getOptionValue('--experimental-entry-url')) { return true; }
   /**
    * @type {string[]} userLoaders A list of custom loaders registered by the user
    * (or an empty list when none have been registered).
@@ -64,8 +68,7 @@ function runMainESM(mainPath) {
   const { pathToFileURL } = require('internal/url');
 
   handleMainPromise(loadESM((esmLoader) => {
-    const main = path.isAbsolute(mainPath) ?
-      pathToFileURL(mainPath).href : mainPath;
+    const main = getOptionValue('--experimental-entry-url') ? mainPath : pathToFileURL(mainPath).href;
     return esmLoader.import(main, undefined, { __proto__: null });
   }));
 }

diff --git a/lib/internal/process/pre_execution.js b/lib/internal/process/pre_execution.js
@@ -52,15 +52,15 @@ const {
 } = require('internal/v8/startup_snapshot');
 
 function prepareMainThreadExecution(expandArgv1 = false, initializeModules = true) {
-  prepareExecution({
+  return prepareExecution({
     expandArgv1,
     initializeModules,
     isMainThread: true,
   });
 }
 
 function prepareWorkerThreadExecution() {
-  prepareExecution({
+  return prepareExecution({
     expandArgv1: false,
     initializeModules: false,  // Will need to initialize it after policy setup
     isMainThread: false,

diff --git a/src/node_options.cc b/src/node_options.cc
@@ -657,6 +657,10 @@ EnvironmentOptionsParser::EnvironmentOptionsParser() {
             "set module system to use by default",
             &EnvironmentOptions::type,
             kAllowedInEnvvar);
+  AddOption("--experimental-entry-url",
+            "set module system to use by default",
+            &EnvironmentOptions::entry_is_url,
+            kAllowedInEnvvar);
   AddOption("--extra-info-on-fatal-exception",
             "hide extra information on fatal exception that causes exit",
             &EnvironmentOptions::extra_info_on_fatal_exception,

diff --git a/src/node_options.h b/src/node_options.h
@@ -119,6 +119,7 @@ class EnvironmentOptions : public Options {
   bool experimental_import_meta_resolve = false;
   std::string input_type;  // Value of --input-type
   std::string type;        // Value of --experimental-default-type
+  bool entry_is_url = false;
   std::string experimental_policy;
   std::string experimental_policy_integrity;
   bool has_policy_integrity_string = false;

diff --git a/test/es-module/test-esm-loader-entry-is-url.mjs b/test/es-module/test-esm-loader-entry-is-url.mjs
@@ -0,0 +1,89 @@
+import { spawnPromisified } from '../common/index.mjs';
+import * as fixtures from '../common/fixtures.mjs';
+import assert from 'node:assert';
+import { execPath } from 'node:process';
+import { describe, it } from 'node:test';
+
+describe('--entry-url', { concurrency: true }, () => {
+  it('should reject loading absolute path that contains %', async () => {
+    const { code, signal, stderr, stdout } = await spawnPromisified(
+      execPath,
+      [
+        '--experimental-entry-url',
+        fixtures.path('es-modules/test-esm-double-encoding-native%20.mjs'),
+      ]
+    );
+
+    assert.match(stderr, /ERR_MODULE_NOT_FOUND/);
+    assert.strictEqual(stdout, '');
+    assert.strictEqual(code, 1);
+    assert.strictEqual(signal, null);
+  });
+
+  it('should support loading absolute URLs', async () => {
+    const { code, signal, stderr, stdout } = await spawnPromisified(
+      execPath,
+      [
+        '--experimental-entry-url',
+        fixtures.fileURL('printA.js'),
+      ]
+    );
+
+    assert.strictEqual(stderr, '');
+    assert.match(stdout, /^A\r?\n$/);
+    assert.strictEqual(code, 0);
+    assert.strictEqual(signal, null);
+  });
+
+  it('should support loading relative URLs', async () => {
+    const { code, signal, stderr, stdout } = await spawnPromisified(
+      execPath,
+      [
+        '--experimental-entry-url',
+        './printA.js?key=value',
+      ],
+      {
+        cwd: fixtures.fileURL('./'),
+      }
+    );
+
+    assert.strictEqual(stderr, '');
+    assert.match(stdout, /^A\r?\n$/);
+    assert.strictEqual(code, 0);
+    assert.strictEqual(signal, null);
+  });
+
+  it('should reject loading relative URLs without trailing `./`', async () => {
-  it('should reject loading relative URLs without trailing `./`', async () => {
+  it('should reject loading relative URLs without leading `./`', async () => {
-  it('should reject loading relative URLs without trailing `./`', async () => {
+  it('should reject loading relative URLs without leading `./`', async () => {
+    const { code, signal, stderr, stdout } = await spawnPromisified(
+      execPath,
+      [
+        '--experimental-entry-url',
+        'printA.js?key=value',
+      ],
+      {
+        cwd: fixtures.fileURL('./'),
+      }
+    );
+
+    assert.match(stderr, /ERR_MODULE_NOT_FOUND/);
+    assert.strictEqual(stdout, '');
+    assert.strictEqual(code, 1);
+    assert.strictEqual(signal, null);
+  });
+
+  it('should support loading `data:` URLs', async () => {
+    const { code, signal, stderr, stdout } = await spawnPromisified(
+      execPath,
+      [
+        '--experimental-entry-url',
+        'data:text/javascript,console.log(0)',
+      ],
+    );
+
+    assert.strictEqual(stderr, '');
+    assert.match(stdout, /^0\r?\n$/);
+    assert.strictEqual(code, 0);
+    assert.strictEqual(signal, null);
+  });
+
+});