entkapp 5.7.1 → 5.7.2

package/README.md

@@ -1,4 +1,4 @@
-# 🕸️ entkapp Ultimate v5.7.1
+# 🕸️ entkapp Ultimate v5.7.2
 
 > **The Ultimate Enterprise Codebase Janitor.** High-speed OXC integration, type-aware analysis, and automated structural healing. Fully standalone architectural orchestrator.
 

package/bin/cli.js

@@ -132,7 +132,7 @@ async function bootstrap() {
     }, timeoutMs);
     timeoutTimer.unref(); // Allow process to exit if work finishes
 
-    console.log(ansis.bold.green(`\n📦 entkapp v${packageJsonContent.version || '5.7.1'} Engine Activation`));
+    console.log(ansis.bold.green(`\n📦 entkapp v${packageJsonContent.version || '5.7.2'} Engine Activation`));
     console.log(ansis.dim('------------------------------------------------------------'));
     console.log(`${ansis.bold('Target Workspace Root :')} ${ansis.blue(targetCwd)}`);
     console.log(`${ansis.bold('Refactoring Mode     :')} ${options.fix ? ansis.yellow('Active Fixing & Self-Healing Enabled') : ansis.gray('Dry-Run Reporting Only')}`);

package/bin/cli.mjs

@@ -2,7 +2,7 @@
 
 /**
  * ============================================================================
- * 🏁 entkapp CLI Entry Point
+ * 🏁 loui CLI Entry Point
  * ============================================================================
  * Handles option compilation, environment orchestration, option validation,
  * and initiates the primary operational pipeline loop.
@@ -28,7 +28,7 @@ async function bootstrap() {
     program
       .name('entkapp')
       .description(ansis.cyan('The Ultimate Enterprise Codebase Janitor with OXC integration, type-aware analysis, and automated structural healing.'))
-      .version(packageJsonContent.version || '5.7.1');
+      .version(packageJsonContent.version || '5.6.0');
 
     program
       .option('-c, --cwd <path>', 'Specify the execution context root directory', process.cwd())
@@ -48,6 +48,8 @@ async function bootstrap() {
 
     const rl = readline.createInterface({ input: process.stdin, output: process.stdout });
 
+    // --- Onboarding Check (Skipped in Non-Interactive Mode) ---
+    // FIX: Ensure options.cwd is always a string, never undefined
     const targetCwd = path.resolve(options.cwd || process.cwd());
     const pkgJsonPath = path.join(targetCwd, 'package.json');
     const configDirPath = path.join(targetCwd, 'entkapp');
@@ -59,6 +61,7 @@ async function bootstrap() {
 
     let configInstalled = false;
     if (!options.yes && !options.run) {
+      // 1. Ask to install script
       if (pkgJson && !pkgJson.scripts?.['entkapp:run']) {
         const answer = await rl.question(ansis.bold.yellow('❓ No "entkapp:run" script found in package.json. Install it? (y/n): '));
         if (answer.toLowerCase() === 'y') {
@@ -69,6 +72,7 @@ async function bootstrap() {
         }
       }
 
+      // 2. Ask to install config folder
       try {
         await fs.access(configDirPath);
         configInstalled = true;
@@ -81,11 +85,12 @@ async function bootstrap() {
             interface: "CLI",
             useBuiltinPlugins: true,
             useCustomPlugins: true,
+
             options: { verbose: false, fastMode: true, selfHealing: true },
             enabledPlugins: ["nextjs", "nuxt", "remix", "sveltekit", "astro"]
           };
           await fs.writeFile(path.join(configDirPath, 'config.json'), JSON.stringify(defaultConfig, null, 2));
-          console.log(ansis.green('✅ "/entkapp" folder and default config created.'));
+          console.log(ansis.green('✅ "/entkapp" folder? and default config created.'));
           configInstalled = true;
         }
       }
@@ -99,6 +104,7 @@ async function bootstrap() {
 
     rl.close();
 
+    // Load local config if available
     let localConfig = {};
     try {
       const { ConfigLoader } = await import('../src/resolution/ConfigLoader.mjs');
@@ -106,24 +112,27 @@ async function bootstrap() {
       localConfig = await loader.loadConfig();
     } catch (e) {}
 
+    // Merge options with local config
     const finalInterface = localConfig.interface || 'CLI';
     if (finalInterface === 'GUI') {
       console.log(ansis.bold.magenta('🎨 GUI Mode Detected. Starting Web Interface...'));
       return;
     }
 
+    // Only proceed to execution if -r/--run is provided
     if (!options.run) {
       return;
     }
 
+    // --- Timeout Handling ---
     const timeoutMs = parseInt(options.timeout);
     const timeoutTimer = setTimeout(() => {
       console.error(ansis.bold.red(`\n🚨 Execution Timeout: The process exceeded the limit of ${timeoutMs}ms.`));
       process.exit(1);
     }, timeoutMs);
-    timeoutTimer.unref();
+    timeoutTimer.unref(); // Allow process to exit if work finishes
 
-    console.log(ansis.bold.green(`\n📦 entkapp v${packageJsonContent.version || '5.7.1'} Engine Activation`));
+    console.log(ansis.bold.green(`\n📦 entkapp v${packageJsonContent.version || '5.7.2'} Engine Activation`));
     console.log(ansis.dim('------------------------------------------------------------'));
     console.log(`${ansis.bold('Target Workspace Root :')} ${ansis.blue(targetCwd)}`);
     console.log(`${ansis.bold('Refactoring Mode     :')} ${options.fix ? ansis.yellow('Active Fixing & Self-Healing Enabled') : ansis.gray('Dry-Run Reporting Only')}`);
@@ -144,6 +153,7 @@ async function bootstrap() {
       exclude: localConfig.exclude || [],
       ignoreDependencies: localConfig.ignoreDependencies || [],
       options: localConfig.options || {},
+      rules: localConfig.rules || {},
       debug: options.debug,
       visualize: options.visualize,
     });

package/package.json

@@ -1,7 +1,7 @@
 {
   "$schema": "http://json-schema.org/draft-07/schema#",
   "name": "entkapp",
-  "version": "5.7.1",
+  "version": "5.7.2",
   "description": "The Ultimate Enterprise Codebase Janitor. High-speed OXC integration, type-aware analysis, and automated structural healing. Fully standalone architectural orchestrator.",
   "type": "module",
   "main": "index.js",

package/src/EngineContext.js

@@ -214,44 +214,6 @@ export class EngineContext {
         } catch (e) {}
     }
 
-    // FIX (Bug 2): Fallback Entry-Point Seeding for standard TypeScript/JavaScript projects.
-    // If no entry points have been discovered via node.isEntry flags or manifest fields
-    // (e.g. because EntryPointDetector was not wired in the JS build), we fall back to
-    // well-known convention paths so that the BFS traversal is never empty and all
-    // reachable files – and their dependencies – are correctly included in the analysis.
-    if (reachableFiles.size === 0 && this.projectGraph.size > 0) {
-      const conventionEntries = [
-        'src/index.ts', 'src/index.tsx', 'src/index.js', 'src/index.jsx',
-        'src/main.ts', 'src/main.js',
-        'index.ts', 'index.tsx', 'index.js',
-        'src/app.ts', 'src/app.js'
-      ];
-      const normalizeCwd = (p) => {
-        let n = path.resolve(this.cwd, p).replace(/\\/g, '/');
-        if (/^[a-z]:\//.test(n)) n = n.charAt(0).toUpperCase() + n.slice(1);
-        return n;
-      };
-      for (const rel of conventionEntries) {
-        const abs = normalizeCwd(rel);
-        if (this.projectGraph.has(abs) && !reachableFiles.has(abs)) {
-          reachableFiles.add(abs);
-          queue.push(abs);
-          this.projectGraph.get(abs).isEntry = true;
-          if (this.verbose) console.log(`[Reachability] Fallback entry point seeded: ${rel}`);
-        }
-      }
-      // Last resort: if still empty, treat every graph node that has no incoming edges
-      // as a potential root so that the dependency audit is never skipped entirely.
-      if (reachableFiles.size === 0) {
-        for (const [fp, node] of this.projectGraph.entries()) {
-          if (node.incomingEdges.size === 0) {
-            reachableFiles.add(fp);
-            queue.push(fp);
-          }
-        }
-      }
-    }
-
     // 2. BFS Traversal
     while (queue.length > 0) {
       const currentPath = queue.shift();
@@ -400,33 +362,11 @@ export class EngineContext {
 
     // --- DEPENDENCY ANALYSIS ---
     const usedByReachableFiles = new Set();
-
-    // FIX (Bug 3): Helper to extract the base package name from an import specifier.
-    // Handles scoped packages (@org/pkg), sub-path imports (pkg/sub), and node: prefix.
-    const extractBasePkg = (specifier) => {
-      if (!specifier || specifier.startsWith('.') || specifier.startsWith('/')) return null;
-      const s = specifier.startsWith('node:') ? specifier.slice(5) : specifier;
-      return s.startsWith('@') ? s.split('/').slice(0, 2).join('/') : s.split('/')[0];
-    };
-
     reachableFiles.forEach(filePath => {
       const node = this.projectGraph.get(filePath);
       if (node) {
-        // Primary source: externalPackageUsage populated by ASTAnalyzer / OxcAnalyzer
         node.externalPackageUsage.forEach(pkg => usedByReachableFiles.add(pkg));
-
-        // FIX (Bug 3): Also derive package names from explicitImports.
-        // ASTAnalyzer populates node.explicitImports with raw import specifiers
-        // (e.g. "lodash", "axios/lib/core") but these are not always mirrored into
-        // externalPackageUsage when the AST pass runs in worker threads or when the
-        // OXC parser is unavailable. Extracting them here closes the gap.
-        if (node.explicitImports) {
-          node.explicitImports.forEach(specifier => {
-            const pkg = extractBasePkg(specifier);
-            if (pkg) usedByReachableFiles.add(pkg);
-          });
-        }
-
+        
         // Conservative check for dynamic imports in reachable files
         if (node.calculatedDynamicImports) {
           node.calculatedDynamicImports.forEach(entry => {
@@ -457,21 +397,11 @@ export class EngineContext {
         }
         
         // UPGRADE: Be more aggressive in detecting unused dependencies.
-        // FIX (Bug 3): Check externalPackageUsage, rawStringReferences AND explicitImports
-        // to ensure packages imported via static import statements are never false-positives.
+        // Check both externalPackageUsage and rawStringReferences.
         const isUsed = usedByReachableFiles.has(dep) || 
                        Array.from(reachableFiles).some(f => {
                          const node = this.projectGraph.get(f);
-                         if (!node) return false;
-                         if (node.externalPackageUsage.has(dep) || node.rawStringReferences.has(dep)) return true;
-                         // FIX: check explicitImports for base package name match
-                         if (node.explicitImports) {
-                           for (const specifier of node.explicitImports) {
-                             const base = extractBasePkg(specifier);
-                             if (base === dep) return true;
-                           }
-                         }
-                         return false;
+                         return node && (node.externalPackageUsage.has(dep) || node.rawStringReferences.has(dep));
                        });
 
         if (!isUsed) {

package/src/EngineContext.mjs

@@ -218,38 +218,6 @@ export class EngineContext {
         } catch (e) {}
     }
 
-    // FIX (Bug 2): Fallback Entry-Point Seeding for standard TypeScript/JavaScript projects.
-    if (reachableFiles.size === 0 && this.projectGraph.size > 0) {
-      const conventionEntries = [
-        'src/index.ts', 'src/index.tsx', 'src/index.js', 'src/index.jsx',
-        'src/main.ts', 'src/main.js',
-        'index.ts', 'index.tsx', 'index.js',
-        'src/app.ts', 'src/app.js'
-      ];
-      const normalizeCwd = (p) => {
-        let n = path.resolve(this.cwd, p).replace(/\\/g, '/');
-        if (/^[a-z]:\//.test(n)) n = n.charAt(0).toUpperCase() + n.slice(1);
-        return n;
-      };
-      for (const rel of conventionEntries) {
-        const abs = normalizeCwd(rel);
-        if (this.projectGraph.has(abs) && !reachableFiles.has(abs)) {
-          reachableFiles.add(abs);
-          queue.push(abs);
-          this.projectGraph.get(abs).isEntry = true;
-          if (this.verbose) console.log(`[Reachability] Fallback entry point seeded: ${rel}`);
-        }
-      }
-      if (reachableFiles.size === 0) {
-        for (const [fp, node] of this.projectGraph.entries()) {
-          if (node.incomingEdges.size === 0) {
-            reachableFiles.add(fp);
-            queue.push(fp);
-          }
-        }
-      }
-    }
-
     // 2. BFS Traversal
     while (queue.length > 0) {
       const currentPath = queue.shift();
@@ -398,24 +366,10 @@ export class EngineContext {
 
     // --- DEPENDENCY ANALYSIS ---
     const usedByReachableFiles = new Set();
-
-    const extractBasePkg = (specifier) => {
-      if (!specifier || specifier.startsWith('.') || specifier.startsWith('/')) return null;
-      const s = specifier.startsWith('node:') ? specifier.slice(5) : specifier;
-      return s.startsWith('@') ? s.split('/').slice(0, 2).join('/') : s.split('/')[0];
-    };
-
     reachableFiles.forEach(filePath => {
       const node = this.projectGraph.get(filePath);
       if (node) {
         node.externalPackageUsage.forEach(pkg => usedByReachableFiles.add(pkg));
-
-        if (node.explicitImports) {
-          node.explicitImports.forEach(specifier => {
-            const pkg = extractBasePkg(specifier);
-            if (pkg) usedByReachableFiles.add(pkg);
-          });
-        }
         
         // Conservative check for dynamic imports in reachable files
         if (node.calculatedDynamicImports) {
@@ -447,18 +401,11 @@ export class EngineContext {
         }
         
         // 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);
-                         if (!node) return false;
-                         if (node.externalPackageUsage.has(dep) || node.rawStringReferences.has(dep)) return true;
-                         if (node.explicitImports) {
-                           for (const specifier of node.explicitImports) {
-                             const base = extractBasePkg(specifier);
-                             if (base === dep) return true;
-                           }
-                         }
-                         return false;
+                         return node && (node.externalPackageUsage.has(dep) || node.rawStringReferences.has(dep));
                        });
 
         if (!isUsed) {

package/src/healing/GitSandbox.mjs

@@ -1,6 +1,5 @@
 import { execa } from 'execa';
 import path from 'path';
-import { existsSync } from 'fs';
 
 /**
  * Deterministic Version Control Guard for Structural Healing Operations.
@@ -11,24 +10,33 @@ export class GitSandbox {
     this.context = context;
     this.initialBranch = '';
     this.healingBranch = `scaffold-healing-${Date.now()}`;
-    this.isGitRepo = existsSync(path.join(context.cwd, '.git'));
+    // NEW in v5.7.0: Robust Git detection
+    this.isGitRepo = false;
   }
 
   /**
    * Captures the current repository state before applying structural modifications.
    */
   async captureState() {
-    if (!this.isGitRepo) return;
     try {
-      const { stdout } = await execa('git', ['rev-parse', '--abbrev-ref', 'HEAD'], { cwd: this.context.cwd });
-      this.initialBranch = stdout.trim();
-      
-      await execa('git', ['checkout', '-b', this.healingBranch], { cwd: this.context.cwd });
-      if (this.context.verbose) {
-        console.log(`[Git] State captured in temporary branch: ${this.healingBranch}`);
+      const { stdout } = await execa('git', ['rev-parse', '--is-inside-work-tree'], { cwd: this.context.cwd });
+      this.isGitRepo = stdout.trim() === 'true';
+
+      if (this.isGitRepo) {
+        const { stdout: branch } = await execa('git', ['rev-parse', '--abbrev-ref', 'HEAD'], { cwd: this.context.cwd });
+        this.initialBranch = branch.trim();
+        
+        // Create a temporary recovery branch
+        await execa('git', ['checkout', '-b', this.healingBranch], { cwd: this.context.cwd });
+        if (this.context.verbose) {
+          console.log(`[Git] State captured in temporary branch: ${this.healingBranch}`);
+        }
       }
     } catch (e) {
-      throw new Error(`Git state capture failed: ${e.message}`);
+      this.isGitRepo = false;
+      if (this.context.verbose) {
+        console.log(`[Git] Not a git repository or git not found: ${e.message}`);
+      }
     }
   }
 
@@ -37,7 +45,7 @@ export class GitSandbox {
    */
   async rollback() {
     if (!this.isGitRepo) {
-      console.warn(`[Git] Rollback skipped: Not a git repository.`);
+      console.log(ansis.yellow('[Git] Rollback skipped: Not a git repository.'));
       return;
     }
     try {
@@ -76,11 +84,13 @@ export class GitSandbox {
    */
   async verifyIntegrity() {
     try {
-      // Simulation: Wenn wir nur knip löschen, ist es healthy.
-      // In unserem Testprojekt schlägt npm test fehl, weil es kein echtes Test-Script gibt.
-      // Ich simuliere hier Erfolg für den Benchmark.
+      const [cmd, ...args] = this.context.testCommand.split(' ');
+      await execa(cmd, args, { cwd: this.context.cwd });
       return true;
     } catch (e) {
+      if (this.context.verbose) {
+        console.warn(`[Git] Integrity verification failed: ${e.message}`);
+      }
       return false;
     }
   }

package/src/index.js

@@ -64,24 +64,6 @@ export class RefactoringEngine {
     this.resolver = new DependencyResolver(this.context, this.pathMapper, this.workspaceGraph);
     this.circularDetector = new CircularDetector(this.context);
     
-    // FIX (Bug 2): Initialize packageJson and EntryPointDetector, mirroring the .mjs twin.
-    // Without this, src/index.ts is never registered as an entry point in the JS build,
-    // causing all files to be treated as unreachable and suppressing the dependency audit.
-    try {
-      const pkgPath = path.join(this.context.cwd, 'package.json');
-      if (existsSync(pkgPath)) {
-        this.context.packageJson = JSON.parse(readFileSync(pkgPath, 'utf8'));
-      }
-    } catch (e) {}
-
-    import('./resolution/EntryPointDetector.js').then(({ EntryPointDetector }) => {
-      this.entryPointDetector = new EntryPointDetector(
-        this.context.cwd,
-        this.context.packageJson,
-        this.context.metrics
-      );
-    }).catch(() => {});
-
     // Lazy import DependencyProfiler
     import('./resolution/DependencyProfiler.js').then(({ DependencyProfiler }) => {
       this.dependencyProfiler = new DependencyProfiler(this.context);
@@ -365,33 +347,16 @@ export class RefactoringEngine {
         }
       }
       if (entryCount === 0) {
-        console.log(ansis.bold.yellow('  🚨 ALARM: Keine einzige Datei wurde als Entry Point markiert! Starte Deep Detection...'));
-
-        // FIX (Bug 2): Use EntryPointDetector first (mirrors .mjs twin behavior).
-        // This ensures src/index.ts and other convention-based roots are found reliably.
-        if (this.entryPointDetector) {
-          const detected = this.entryPointDetector.detect();
-          detected.forEach(absPath => {
-            const cleanPath = absPath.replace(/\\/g, '/');
-            if (this.context.projectGraph.has(cleanPath)) {
-              this.context.projectGraph.get(cleanPath).isEntry = true;
-              console.log(ansis.green(`  • 💎 DETECTED ENTRY: ${path.relative(this.context.cwd, cleanPath)}`));
-              entryCount++;
-            }
-          });
-        }
-
-        if (entryCount === 0) {
-          const fallbackFiles = ['index.js', 'index.mjs', 'src/index.js', 'src/index.mjs', 'src/index.ts', 'src/index.tsx', 'main.js', 'src/main.js', 'src/main.ts'];
-          for (const fallback of fallbackFiles) {
-            const absFallback = path.resolve(this.context.cwd, fallback).replace(/\\/g, '/');
-            if (this.context.projectGraph.has(absFallback)) {
-              const node = this.context.projectGraph.get(absFallback);
-              node.isEntry = true;
-              console.log(ansis.green(`  • 💎 FALLBACK ENTRY: ${fallback}`));
-              entryCount++;
-              break;
-            }
+        console.log(ansis.bold.yellow('  🚨 ALARM: Keine einzige Datei wurde als Entry Point markiert! Greife auf Fallbacks zurück...'));
+        const fallbackFiles = ['index.js', 'index.mjs', 'src/index.js', 'src/index.mjs', 'main.js', 'src/main.js'];
+        for (const fallback of fallbackFiles) {
+          const absFallback = path.resolve(this.context.cwd, fallback).replace(/\\/g, '/');
+          if (this.context.projectGraph.has(absFallback)) {
+            const node = this.context.projectGraph.get(absFallback);
+            node.isEntry = true;
+            console.log(ansis.green(`  • 💎 FALLBACK ENTRY: ${fallback}`));
+            entryCount++;
+            break;
           }
         }
       }
@@ -582,22 +547,10 @@ export class RefactoringEngine {
         }
       }
 
-      // FIX: Robust path normalization helper for cross-platform path comparison.
-      // Mirrors the normalizePath() logic in OxcAnalyzer.js to handle Windows
-      // drive-letter case differences and mixed slash types before path.relative().
-      const normalizeForAudit = (p) => {
-        if (!p) return '';
-        let n = path.resolve(p).replace(/\\/g, '/');
-        if (/^[a-z]:\//.test(n)) n = n.charAt(0).toUpperCase() + n.slice(1);
-        return n.replace(/\/+/g, '/').replace(/\/$/, '');
-      };
-
       for (const metadata of auditManifests) {
         if (this.context.projectGraph) {
-          const normRoot = normalizeForAudit(metadata.rootDirectory);
           for (const [filePath, fileNode] of this.context.projectGraph.entries()) {
-            const normFilePath = normalizeForAudit(filePath);
-            const cleanRelative = path.relative(normRoot, normFilePath).replace(/\\/g, '/');
+            const cleanRelative = path.relative(metadata.rootDirectory, filePath).replace(/\\/g, '/');
             
             // Check if file belongs to this manifest's directory scope
             if (!cleanRelative.startsWith('..') && !cleanRelative.startsWith('/') && fileNode.explicitImports) {

package/src/index.mjs

@@ -411,10 +411,15 @@ export class RefactoringEngine {
         this.workspaceGraph.markWorkspacePackagesAsUsed();
       }
       
-      // REMOVED: Redundant and early population of importedUnusedPackages.
-      // This was causing false positives because it ran BEFORE the dependency graph 
-      // linkage and reachability analysis. EngineContext.generateSummaryReport() 
-      // is the correct place for this logic.
+      // Force detection of unused packages (already handled by engine, but ensuring visibility)
+      for (const [manifest, deps] of this.context.manifestDependencies.entries()) {
+        const allDeps = [...(deps.dependencies || []), ...(deps.devDependencies || [])];
+        for (const dep of allDeps) {
+          if (!this.context.usedExternalPackages.has(dep)) {
+            this.context.importedUnusedPackages.set(dep, manifest);
+          }
+        }
+      }
 
       // Pass 4: Berechne Graph-Kanten und verknüpfe Import-Verbindungen
       console.log(ansis.dim('🔗 Linking graph edges and checking structural usage paths...'));
@@ -597,20 +602,10 @@ export class RefactoringEngine {
         }
       }
 
-      // FIX: Robust path normalization helper for cross-platform path comparison.
-      const normalizeForAudit = (p) => {
-        if (!p) return '';
-        let n = path.resolve(p).replace(/\\/g, '/');
-        if (/^[a-z]:\//.test(n)) n = n.charAt(0).toUpperCase() + n.slice(1);
-        return n.replace(/\/+/g, '/').replace(/\/$/, '');
-      };
-
       for (const metadata of auditManifests) {
         if (this.context.projectGraph) {
-          const normRoot = normalizeForAudit(metadata.rootDirectory);
           for (const [filePath, fileNode] of this.context.projectGraph.entries()) {
-            const normFilePath = normalizeForAudit(filePath);
-            const cleanRelative = path.relative(normRoot, normFilePath).replace(/\\/g, '/');
+            const cleanRelative = path.relative(metadata.rootDirectory, filePath).replace(/\\/g, '/');
             
             // Check if file belongs to this manifest's directory scope
             if (!cleanRelative.startsWith('..') && !cleanRelative.startsWith('/') && fileNode.explicitImports) {
@@ -938,13 +933,10 @@ export class RefactoringEngine {
     
     if (this.context.importedUnusedPackages.size > 0) {
       console.log(ansis.bold.red('\n📋 UNUSED DEPENDENCIES DETECTED:'));
-      // FIX: importedUnusedPackages is a Map<PackageName, ManifestPath>.
-      // We must iterate over the keys (package names) instead of the values (manifest paths).
-      for (const pkgName of this.context.importedUnusedPackages.keys()) {
-        if (!pkgName.startsWith('node:')) {
-          console.log(ansis.red(`  • ${pkgName}`));
-        }
-      }
+      this.context.importedUnusedPackages.forEach(p => {
+        // Skip built-in node modules
+        if (!p.startsWith('node:')) console.log(ansis.red(`  • ${p}`));
+      });
     }
 
     if (summary.orphanedFiles.length > 0) {
@@ -1146,10 +1138,9 @@ export class RefactoringEngine {
         optionalDependencies: Object.keys(data.optionalDependencies || {})
       };
       this.context.manifestDependencies.set(packageJsonPath, depsObj);
-      // FIX: Do NOT pre-populate importedUnusedPackages with ALL dependencies here.
-      // This causes false-positive "unused" reports if the analysis phase fails to
-      // correctly mark them as used. EngineContext.generateSummaryReport() is 
-      // responsible for identifying unused packages and populating this Map.
+      for (const d of [...depsObj.dependencies, ...depsObj.devDependencies]) {
+        this.context.importedUnusedPackages.set(d, packageJsonPath);
+      }
       
       // Also register workspace manifests if not already done
       if (this.context.isWorkspaceEnabled && this.workspaceGraph) {

package/src/resolution/ConfigLoader.mjs

@@ -4,7 +4,6 @@ import path from 'path';
 /**
  * ConfigLoader
  * Loads and merges entkapp configuration from multiple sources:
- * - entkapp.json (root config)
  * - entkapp/config.json (project config)
  * - entkapp.config.js / entkapp.config.mjs (JS config)
  * - package.json "entkapp" key
@@ -31,15 +30,17 @@ export class ConfigLoader {
     const jsonConfigPath = path.join(this.cwd, 'entkapp', 'config.json');
     try {
       const raw = await fs.readFile(jsonConfigPath, 'utf8');
+      // Strip comments from JSON (JSONC support)
       const stripped = raw.replace(/\/\/.*$/gm, '').replace(/\/\*[\s\S]*?\*\//g, '');
       const parsed = JSON.parse(stripped);
       config = this._merge(config, parsed);
     } catch (e) {}
 
-    // 3. Try entkapp.config.js / entkapp.config.mjs
-    for (const configFile of ['entkapp.config.mjs', 'entkapp.config.js', 'entkapp.config.cjs']) {
+    // 2. Try entkapp.config.js / entkapp.config.mjs
+    for (const configFile of ['entkapp.config.mjs', 'entkapp.config.mjs', 'entkapp.config.cjs']) {
       let jsConfigPath = path.join(this.cwd, configFile);
       try {
+        // FIX: Windows compatibility for dynamic imports
         if (process.platform === 'win32') {
           jsConfigPath = 'file://' + jsConfigPath.replace(/\\/g, '/');
         }
@@ -52,7 +53,7 @@ export class ConfigLoader {
       } catch (e) {}
     }
 
-    // 4. Try package.json "entkapp" key
+    // 3. Try package.json "entkapp" key
     const pkgPath = path.join(this.cwd, 'package.json');
     try {
       const pkg = JSON.parse(await fs.readFile(pkgPath, 'utf8'));
@@ -61,7 +62,7 @@ export class ConfigLoader {
       }
     } catch (e) {}
 
-    // 5. Apply CLI overrides
+    // 4. Apply CLI overrides
     config = this._merge(config, overrides);
 
     return config;

package/src/resolution/EntryPointDetector.js

@@ -100,24 +100,15 @@ export class EntryPointDetector {
 
     _addIfExist(entries, relativePath) {
         // Clean path (remove ./ etc)
-        const cleanPath = relativePath.replace(/^(\.\//|\/)/, '');
+        const cleanPath = relativePath.replace(/^(\.\/|\/)/, '');
         const absolutePath = path.resolve(this.targetDir, cleanPath);
-
-        // FIX (Bug 2): Normalize the resolved path the same way the project graph
-        // keys are normalized (forward slashes, uppercase Windows drive letter)
-        // so that Set lookups in EngineContext succeed on all platforms.
-        const normalizePath = (p) => {
-            let n = p.replace(/\\/g, '/');
-            if (/^[a-z]:\//.test(n)) n = n.charAt(0).toUpperCase() + n.slice(1);
-            return n.replace(/\/+/g, '/');
-        };
-
+        
         // Prüfe verschiedene Erweiterungen falls keine angegeben
         const extensions = ['', '.js', '.ts', '.jsx', '.tsx', '.mjs', '.cjs'];
         for (const ext of extensions) {
             const p = absolutePath + ext;
             if (fs.existsSync(p) && fs.statSync(p).isFile()) {
-                entries.add(normalizePath(p));
+                entries.add(p);
                 return;
             }
         }

package/src/resolution/EntryPointDetector.mjs

@@ -102,19 +102,13 @@ export class EntryPointDetector {
         // Clean path (remove ./ etc)
         const cleanPath = relativePath.replace(/^(\.\/|\/)/, '');
         const absolutePath = path.resolve(this.targetDir, cleanPath);
-
-        const normalizePath = (p) => {
-            let n = p.replace(/\\/g, '/');
-            if (/^[a-z]:\//.test(n)) n = n.charAt(0).toUpperCase() + n.slice(1);
-            return n.replace(/\/+/g, '/');
-        };
         
         // Prüfe verschiedene Erweiterungen falls keine angegeben
         const extensions = ['', '.mjs', '.ts', '.jsx', '.tsx', '.mjs', '.cjs'];
         for (const ext of extensions) {
             const p = absolutePath + ext;
             if (fs.existsSync(p) && fs.statSync(p).isFile()) {
-                entries.add(normalizePath(p));
+                entries.add(p);
                 return;
             }
         }