entkapp 5.2.0 → 5.2.1

package/package.json

@@ -1,7 +1,7 @@
 {
   "$schema": "http://json-schema.org/draft-07/schema#",
   "name": "entkapp",
-  "version": "5.2.0",
+  "version": "5.2.1",
   "description": "The Ultimate Enterprise Codebase Janitor. Faster than Knip with OXC integration, type-aware analysis, and automated structural healing. Fully standalone - solving what Knip cannot.",
   "type": "module",
   "main": "index.js",

package/src/EngineContext.js

@@ -86,10 +86,16 @@ export class GraphNode {
       }
 
       // Fuzzy / Dynamic usage (Identifiers, Strings, JSX, Decorators)
-      if (parentNode.instantiatedIdentifiers.has(symbolName)) return true;
-      if (parentNode.rawStringReferences.has(symbolName)) return true;
-      if (parentNode.jsxComponents.has(symbolName)) return true;
-      if (parentNode.decorators.has(symbolName)) return true;
+      // UPGRADE: Only check for fuzzy matches if we have a reason to believe it might be used.
+      // We skip global names like 'toLowerCase' unless they are explicitly imported.
+      const isGlobalName = ['toLowerCase', 'toUpperCase', 'toString', 'valueOf', 'hasOwnProperty', 'constructor'].includes(symbolName);
+      
+      if (!isGlobalName) {
+        if (parentNode.instantiatedIdentifiers.has(symbolName)) return true;
+        if (parentNode.rawStringReferences.has(symbolName)) return true;
+        if (parentNode.jsxComponents.has(symbolName)) return true;
+        if (parentNode.decorators.has(symbolName)) return true;
+      }
       
       for (const accessChain of parentNode.propertyAccessChains) {
         if (accessChain.endsWith(`.${symbolName}`) || accessChain.includes(`.${symbolName}.`)) return true;
@@ -390,7 +396,15 @@ export class EngineContext {
           continue;
         }
         
-        if (!usedByReachableFiles.has(dep)) {
+        // UPGRADE: Be more aggressive in detecting unused dependencies.
+        // Check both externalPackageUsage and rawStringReferences.
+        const isUsed = usedByReachableFiles.has(dep) || 
+                       Array.from(reachableFiles).some(f => {
+                         const node = this.projectGraph.get(f);
+                         return node && (node.externalPackageUsage.has(dep) || node.rawStringReferences.has(dep));
+                       });
+
+        if (!isUsed) {
           report.unusedDependencies.push({
             package: dep,
             type: deps.dependencies.includes(dep) ? 'dependency' : 'devDependency',

package/src/index.js

@@ -940,7 +940,9 @@ export class RefactoringEngine {
       if (isProtected || isExplicitlyDeclared) {
         finalIsEntry = true;
       } else if (!hasManifestEntries && hasNoIncoming && hasOutgoing && !isLikelyBarrel) {
-        finalIsEntry = true;
+        // UPGRADE: Orphan files with imports are NOT entry points by default.
+        // They must have side effects (detected in AST pass) to be entries.
+        finalIsEntry = node.isEntry; 
       }
 
       node.isEntry = finalIsEntry;