fix: return if no matchedPackage (#16)

azz · Oct 28, 2020 · 07f6bdc · 07f6bdc
1 parent 51cac51
commit 07f6bdc
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/rules/no-internal-import.js b/src/rules/no-internal-import.js
@@ -32,6 +32,10 @@ export const create = context => {
   return moduleVisitor(node => {
     const { name, path: internalPath } = tryParse(node.value);
     const matchedPackage = packages.find(pkg => pkg.package.name === name);
+
+    if (!internalPath) return;
+    if (!matchedPackage) return;
+
     const packageRoot = matchedPackage.location;
 
     // Need to take care of "files" field, since they are
@@ -52,8 +56,6 @@ export const create = context => {
       });
     const absoluteInternalPath = path.join(packageRoot, internalPath);
 
-    if (!internalPath) return;
-    if (!matchedPackage) return;
     if (absolutePathsForFiles) {
       const isImportWithinFiles = absolutePathsForFiles.some(maybeGlob => {
         // If import doesn't have an extension, strip it from the file entry