closes import-js#1293. allows aliases that start with @ to be caught as internal

Author: jeffshaver

src/core/importType.js

@@ -8,8 +8,8 @@ function constant(value) {
   return () => value
 }
 
-function baseModule(name) {
-  if (isScoped(name)) {
+function baseModule(name, path) {
+  if (isScoped(name, path)) {
     const [scope, pkg] = name.split('/')
     return `${scope}/${pkg}`
   }
@@ -21,14 +21,16 @@ export function isAbsolute(name) {
   return name.indexOf('/') === 0
 }
 
-export function isBuiltIn(name, settings) {
-  const base = baseModule(name)
+export function isBuiltIn(name, settings, path) {
+  const base = baseModule(name, path)
   const extras = (settings && settings['import/core-modules']) || []
   return coreModules[base] || extras.indexOf(base) > -1
 }
 
 function isExternalPath(path, name, settings) {
-  const folders = (settings && settings['import/external-module-folders']) || ['node_modules']
+  const folders = (settings && settings['import/external-module-folders']) || [
+    'node_modules'
+  ]
   return !path || folders.some(folder => -1 < path.indexOf(join(folder, name)))
 }
 
@@ -39,21 +41,26 @@ function isExternalModule(name, settings, path) {
 
 const externalModuleMainRegExp = /^[\w]((?!\/).)*$/
 export function isExternalModuleMain(name, settings, path) {
-  return externalModuleMainRegExp.test(name) && isExternalPath(path, name, settings)
+  return (
+    externalModuleMainRegExp.test(name) && isExternalPath(path, name, settings)
+  )
 }
 
 const scopedRegExp = /^@[^/]+\/[^/]+/
-function isScoped(name) {
-  return scopedRegExp.test(name)
+function isScoped(name, settings, path) {
+  return scopedRegExp.test(name) && isExternalPath(path, name, settings)
 }
 
 const scopedMainRegExp = /^@[^/]+\/?[^/]+$/
-export function isScopedMain(name) {
-  return scopedMainRegExp.test(name)
+export function isScopedMain(name, settings, path) {
+  return scopedMainRegExp.test(name) && isExternalPath(path, settings, name)
 }
 
 function isInternalModule(name, settings, path) {
-  return externalModuleRegExp.test(name) && !isExternalPath(path, name, settings)
+  return (
+    (scopedRegExp.test(name) || externalModuleRegExp.test(name)) &&
+    !isExternalPath(path, name, settings)
+  )
 }
 
 function isRelativeToParent(name) {

tests/src/core/importType.js

@@ -41,6 +41,11 @@ describe('importType(name)', function () {
     expect(importType('importType', pathContext)).to.equal('internal')
   })
 
+  it("should return 'internal' for internal modules that are referenced by aliases", function () {
+    const pathContext = testContext({ "import/resolver": { node: { paths: [path.join(__dirname, '..', '..', 'files')] } } })
+    expect(importType('@my-alias/fn', pathContext)).to.equal('internal')
+  })
+
   it("should return 'parent' for internal modules that go through the parent", function() {
     expect(importType('../foo', context)).to.equal('parent')
     expect(importType('../../foo', context)).to.equal('parent')