Filter out external refs to re-exports

webpro-nl · Nov 10, 2024 · b9ffef4 · b9ffef4
1 parent 9a78e33
commit b9ffef4
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/packages/knip/src/ProjectPrincipal.ts b/packages/knip/src/ProjectPrincipal.ts
@@ -338,7 +338,16 @@ export class ProjectPrincipal {
 
     const externalRefs = referencedSymbols
       .flatMap(refs => refs.references)
-      .filter(ref => !ref.isDefinition && ref.fileName !== filePath);
+      .filter(ref => !ref.isDefinition && ref.fileName !== filePath)
+      .filter(ref => {
+        // Filter out are re-exports
+        const sourceFile = this.backend.program?.getSourceFile(ref.fileName);
+        if (!sourceFile) return true;
+        // @ts-expect-error ts.getTokenAtPosition is internal fn
+        const node = ts.getTokenAtPosition(sourceFile, ref.textSpan.start);
+        if (!node?.parent?.parent?.parent) return true;
+        return !(ts.isExportSpecifier(node.parent) && node.parent.parent.parent.moduleSpecifier);
+      });
 
     return externalRefs.length > 0;
   }