Use settings if provided in isExternalModule and isExternalModuleMain

import-js · Dec 22, 2021 · 105545b · 105545b
1 parent 02c1dfb
commit 105545b
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/src/core/importType.js b/src/core/importType.js
@@ -34,11 +34,11 @@ export function isExternalModule(name, settings, path, context) {
   if (arguments.length < 4) {
     throw new TypeError('isExternalModule: name, settings, path, and context are all required');
   }
-  return (isModule(name) || isScoped(name)) && typeTest(name, context, path) === 'external';
+  return (isModule(name) || isScoped(name)) && typeTest(name, context, path, settings) === 'external';
 }
 
 export function isExternalModuleMain(name, settings, path, context) {
-  return isModuleMain(name) && typeTest(name, context, path) === 'external';
+  return isModuleMain(name) && typeTest(name, context, path, settings) === 'external';
 }
 
 const moduleRegExp = /^\w/;
@@ -106,8 +106,7 @@ function isExternalLookingName(name) {
   return isModule(name) || isScoped(name);
 }
 
-function typeTest(name, context, path) {
-  const { settings } = context;
+function typeTest(name, context, path, settings = context.settings) {
   if (isInternalRegexMatch(name, settings)) { return 'internal'; }
   if (isAbsolute(name, settings, path)) { return 'absolute'; }
   if (isBuiltIn(name, settings, path)) { return 'builtin'; }