entkapp 5.4.0 → 5.6.0

package/src/ast/ASTAnalyzer.js

@@ -66,6 +66,17 @@ export class ASTAnalyzer {
       fileNode.isPureBarrel = true;
     }
 
+    // ADVANCED: Deep reachability analysis for dynamic imports
+    if (fileNode.dynamicImports.size > 0) {
+      fileNode.dynamicImports.forEach(specifier => {
+        const resolved = this.resolveAbsoluteTargetFile(specifier, filePath);
+        if (resolved) {
+          fileNode.outgoingEdges.add(resolved);
+          this.context.getOrCreateNode(resolved).isDynamicEntry = true;
+        }
+      });
+    }
+
     this.scopeStack = [];
     this.currentScope = null;
   }
@@ -222,6 +233,9 @@ export class ASTAnalyzer {
           case ts.SyntaxKind.ExpressionStatement:
             this.handleAssignmentExpressionPass1(node, fileNode, sourceFile);
             break;
+          case ts.SyntaxKind.ExportDeclaration:
+            this.handleExportDeclaration(node, fileNode, sourceFile);
+            break;
         }
     } else {
         // --- PASS 2: REFERENCE TRACKING ---
@@ -265,16 +279,7 @@ export class ASTAnalyzer {
                 fileNode.explicitImports.add(specifier);
                 fileNode.importedSymbols.add(`${specifier}:*`);
                 if (!specifier.startsWith('.') && !specifier.startsWith('/')) {
-                  // UPGRADE: Apply alias/workspace filtering before adding to externalPackageUsage
-                  const pkgNameReq = this._extractPackageName(specifier);
-                  let isInternalReq = false;
-                  if (this.context.pathMapper && typeof this.context.pathMapper.isTsconfigAlias === 'function') {
-                    if (this.context.pathMapper.isTsconfigAlias(specifier) || this.context.pathMapper.isTsconfigAlias(pkgNameReq)) isInternalReq = true;
-                  }
-                  if (!isInternalReq && this.context.workspaceGraph && typeof this.context.workspaceGraph.isLocalWorkspaceSpecifier === 'function') {
-                    if (this.context.workspaceGraph.isLocalWorkspaceSpecifier(specifier) || this.context.workspaceGraph.isLocalWorkspaceSpecifier(pkgNameReq)) isInternalReq = true;
-                  }
-                  if (!isInternalReq) fileNode.externalPackageUsage.add(pkgNameReq);
+                  fileNode.externalPackageUsage.add(this._extractPackageName(specifier));
                 }
               } else if (arg && ts.isTemplateExpression(arg)) {
                 // Dynamic require with template literal
@@ -360,40 +365,8 @@ export class ASTAnalyzer {
       const specifier = node.moduleSpecifier.text;
       fileNode.explicitImports.add(specifier);
       
-      // UPGRADE 5.4.2: Distinguish between local aliases and workspace packages
       if (!specifier.startsWith('.') && !specifier.startsWith('/')) {
-        let isInternalAlias = false;
-        const pkgName = this._extractPackageName(specifier);
-
-        // UPGRADE: First check via isTsconfigAlias() — works even when the target file doesn't exist yet
-        if (this.context.pathMapper && typeof this.context.pathMapper.isTsconfigAlias === 'function') {
-          if (this.context.pathMapper.isTsconfigAlias(specifier) || this.context.pathMapper.isTsconfigAlias(pkgName)) {
-            isInternalAlias = true;
-          }
-        }
-
-        // UPGRADE: Check workspace packages (pnpm-workspace.yaml / package.json workspaces)
-        if (!isInternalAlias && this.context.workspaceGraph && typeof this.context.workspaceGraph.isLocalWorkspaceSpecifier === 'function') {
-          if (this.context.workspaceGraph.isLocalWorkspaceSpecifier(specifier) || this.context.workspaceGraph.isLocalWorkspaceSpecifier(pkgName)) {
-            isInternalAlias = true;
-          }
-        }
-
-        // Fallback: try resolvePath() — works when the target file exists on disk
-        if (!isInternalAlias && this.context.pathMapper && typeof this.context.pathMapper.resolvePath === 'function') {
-          const resolved = this.context.pathMapper.resolvePath(specifier);
-          if (resolved !== specifier && (resolved.startsWith('/') || resolved.includes(':'))) {
-            // Only mark as internal alias if it's NOT a registered workspace package
-            const isWorkspacePkg = this.context.workspaceGraph && this.context.workspaceGraph.isLocalWorkspaceSpecifier(pkgName);
-            if (!isWorkspacePkg) {
-              isInternalAlias = true;
-            }
-          }
-        }
-        
-        if (!isInternalAlias) {
-          fileNode.externalPackageUsage.add(pkgName);
-        }
+        fileNode.externalPackageUsage.add(this._extractPackageName(specifier));
       }
 
       if (this.context.workspaceGraph && typeof this.context.workspaceGraph.auditImportSpecifier === 'function') {
@@ -414,29 +387,14 @@ export class ASTAnalyzer {
           if (ts.isNamespaceImport(node.importClause.namedBindings)) {
             fileNode.importedSymbols.add(`${specifier}:*`);
             if (targetFile) this.context.importUsageRegistry.add(`${targetFile}:*`);
-            
-            // Track local name for unused import detection
-            if (!fileNode.localImportBindings) fileNode.localImportBindings = new Map();
-            fileNode.localImportBindings.set(node.importClause.namedBindings.name.text, { specifier, originalName: '*' });
           } else if (ts.isNamedImports(node.importClause.namedBindings)) {
             node.importClause.namedBindings.elements.forEach(element => {
               const importedName = element.propertyName ? element.propertyName.text : element.name.text;
-              const localName = element.name.text;
               fileNode.importedSymbols.add(`${specifier}:${importedName}`);
               if (targetFile) this.context.importUsageRegistry.add(`${targetFile}:${importedName}`);
-              
-              // Track local name for unused import detection
-              if (!fileNode.localImportBindings) fileNode.localImportBindings = new Map();
-              fileNode.localImportBindings.set(localName, { specifier, originalName: importedName });
             });
           }
         }
-        
-        // Handle default import binding
-        if (node.importClause.name) {
-          if (!fileNode.localImportBindings) fileNode.localImportBindings = new Map();
-          fileNode.localImportBindings.set(node.importClause.name.text, { specifier, originalName: 'default' });
-        }
       } else {
         // Side-Effect Import (import '...')
         // FIX: Mark the target file as used even if no symbols are imported
@@ -479,22 +437,7 @@ export class ASTAnalyzer {
     if (specifier) {
       fileNode.explicitImports.add(specifier);
       if (!specifier.startsWith('.') && !specifier.startsWith('/')) {
-        // UPGRADE: Apply the same alias/workspace filtering as handleImportDeclaration
-        const pkgNameExport = this._extractPackageName(specifier);
-        let isInternalAliasExport = false;
-        if (this.context.pathMapper && typeof this.context.pathMapper.isTsconfigAlias === 'function') {
-          if (this.context.pathMapper.isTsconfigAlias(specifier) || this.context.pathMapper.isTsconfigAlias(pkgNameExport)) {
-            isInternalAliasExport = true;
-          }
-        }
-        if (!isInternalAliasExport && this.context.workspaceGraph && typeof this.context.workspaceGraph.isLocalWorkspaceSpecifier === 'function') {
-          if (this.context.workspaceGraph.isLocalWorkspaceSpecifier(specifier) || this.context.workspaceGraph.isLocalWorkspaceSpecifier(pkgNameExport)) {
-            isInternalAliasExport = true;
-          }
-        }
-        if (!isInternalAliasExport) {
-          fileNode.externalPackageUsage.add(pkgNameExport);
-        }
+        fileNode.externalPackageUsage.add(this._extractPackageName(specifier));
       }
 
       if (!node.exportClause) {
@@ -658,23 +601,14 @@ export class ASTAnalyzer {
           fileNode.dynamicImports.add(specifier);
           fileNode.importedSymbols.add(`${specifier}:*`);
           if (!specifier.startsWith('.') && !specifier.startsWith('/')) {
-            // UPGRADE: Apply alias/workspace filtering before adding to externalPackageUsage
-            const pkgNameDyn = this._extractPackageName(specifier);
-            let isInternalDyn = false;
-            if (this.context.pathMapper && typeof this.context.pathMapper.isTsconfigAlias === 'function') {
-              if (this.context.pathMapper.isTsconfigAlias(specifier) || this.context.pathMapper.isTsconfigAlias(pkgNameDyn)) isInternalDyn = true;
-            }
-            if (!isInternalDyn && this.context.workspaceGraph && typeof this.context.workspaceGraph.isLocalWorkspaceSpecifier === 'function') {
-              if (this.context.workspaceGraph.isLocalWorkspaceSpecifier(specifier) || this.context.workspaceGraph.isLocalWorkspaceSpecifier(pkgNameDyn)) isInternalDyn = true;
-            }
-            if (!isInternalDyn) fileNode.externalPackageUsage.add(pkgNameDyn);
+            fileNode.externalPackageUsage.add(this._extractPackageName(specifier));
           }
         } else {
           // Non-literal dynamic import
-          if (!Array.isArray(fileNode.calculatedDynamicImports)) {
-            fileNode.calculatedDynamicImports = [];
+          if (!fileNode.calculatedDynamicImports || Array.isArray(fileNode.calculatedDynamicImports)) {
+            fileNode.calculatedDynamicImports = new Set();
           }
-          fileNode.calculatedDynamicImports.push({ kind: ts.SyntaxKind[arg.kind], start: arg.getStart(sourceFile) });
+          fileNode.calculatedDynamicImports.add({ kind: ts.SyntaxKind[arg.kind], start: arg.getStart(sourceFile) });
           
           // UPGRADE 2: Try to perform Glob-Analysis on Template Strings
           this.handlePartialEvaluation(arg, fileNode, sourceFile);
@@ -688,17 +622,9 @@ export class ASTAnalyzer {
         if (this.context.verbose) console.log(`[AST-DEBUG] Added import: ${specifier}`);
         fileNode.explicitImports.add(specifier);
         
-        // UPGRADE: Apply alias/workspace filtering before adding to externalPackageUsage
+        // UPGRADE: Also track package usage for unlisted dependency detection
         if (!specifier.startsWith('.') && !specifier.startsWith('/')) {
-          const pkgNameCE = this._extractPackageName(specifier);
-          let isInternalCE = false;
-          if (this.context.pathMapper && typeof this.context.pathMapper.isTsconfigAlias === 'function') {
-            if (this.context.pathMapper.isTsconfigAlias(specifier) || this.context.pathMapper.isTsconfigAlias(pkgNameCE)) isInternalCE = true;
-          }
-          if (!isInternalCE && this.context.workspaceGraph && typeof this.context.workspaceGraph.isLocalWorkspaceSpecifier === 'function') {
-            if (this.context.workspaceGraph.isLocalWorkspaceSpecifier(specifier) || this.context.workspaceGraph.isLocalWorkspaceSpecifier(pkgNameCE)) isInternalCE = true;
-          }
-          if (!isInternalCE) fileNode.externalPackageUsage.add(pkgNameCE);
+          fileNode.externalPackageUsage.add(this._extractPackageName(specifier));
         }
       }
     }