fixup: this is better

Author: MylesBorins

lib/internal/modules/esm/default_resolve.js

@@ -4,7 +4,6 @@ const { NativeModule } = require('internal/bootstrap/loaders');
 const { ERR_TYPE_MISMATCH,
         ERR_UNKNOWN_FILE_EXTENSION } = require('internal/errors').codes;
 const { getOptionValue } = require('internal/options');
-const optionType = getOptionValue('--type');
 const experimentalJsonModules = getOptionValue('--experimental-json-modules');
 const { resolve: moduleWrapResolve } = internalBinding('module_wrap');
 const { pathToFileURL, fileURLToPath } = require('internal/url');
@@ -77,8 +76,8 @@ function resolve(specifier, parentURL) {
     }
   }
   if (!format) {
-    if (isMain && type === 0 && optionType)
-      format = optionType;
+    if (isMain && asyncESM.typeFlag)
+      format = asyncESM.typeFlag;
     else if (isMain)
       format = type === 2 ? 'module' : 'commonjs';
     else

test/es-module/test-esm-no-extension.js

@@ -7,7 +7,12 @@ const assert = require('assert');
 
 const entry = fixtures.path('/es-modules/noext-esm');
 
-const child = spawn(process.execPath, ['--experimental-modules', '--type=module', entry]);
+const child = spawn(process.execPath, [
+  '--experimental-modules',
+  '--type=module',
+  entry
+]);
+
 let stderr = '';
 child.stderr.setEncoding('utf8');
 child.stderr.on('data', (data) => {