module: fix specifier resolution algorithm

Fixes: #30520 PR-URL: #30574 Reviewed-By: Guy Bedford <[email protected]> Reviewed-By: Myles Borins <[email protected]>

Author: pd4d10

lib/internal/modules/run_main.js

@@ -24,6 +24,10 @@ function shouldUseESMLoader(mainPath){
 constuserLoader=getOptionValue('--experimental-loader');
 if(userLoader)
 returntrue;
+constesModuleSpecifierResolution=
+getOptionValue('--es-module-specifier-resolution');
+if(esModuleSpecifierResolution==='node')
+returntrue;
 // Determine the module format of the main
 if(mainPath&&mainPath.endsWith('.mjs'))
 returntrue;

test/es-module/test-esm-specifiers.mjs

@@ -1,6 +1,9 @@
 // Flags: --es-module-specifier-resolution=node
 import{mustNotCall}from'../common/index.mjs';
 importassertfrom'assert';
+importpathfrom'path';
+import{spawn}from'child_process';
+import{fileURLToPath}from'url';
 
 // commonJS index.js
 importcommonjsfrom'../fixtures/es-module-specifiers/package-type-commonjs';
@@ -33,3 +36,22 @@ async function main(){
 }
 
 main().catch(mustNotCall);
+
+// Test path from command line arguments
+[
+'package-type-commonjs',
+'package-type-module',
+'/',
+'/index',
+].forEach((item)=>{
+constmodulePath=path.join(
+fileURLToPath(import.meta.url),
+'../../fixtures/es-module-specifiers',
+item,
+);
+spawn(process.execPath,
+['--es-module-specifier-resolution=node',modulePath],
+{stdio: 'inherit'}).on('exit',(code)=>{
+assert.strictEqual(code,0);
+});
+});