prunify 0.1.2 → 0.1.5

package/dist/cli.js

@@ -209,40 +209,12 @@ function buildGraph(files, getImports) {
 function findEntryPoints(rootDir, packageJson) {
   const entries = [
     ...resolveNextJsEntries(rootDir),
+    ...resolveFileBrowserEntries(rootDir),
     ...resolvePkgFieldEntries(rootDir, packageJson),
     ...resolveFallbackEntries(rootDir)
   ];
   return [...new Set(entries)];
 }
-function findRootFiles(graph) {
-  const imported = /* @__PURE__ */ new Set();
-  for (const deps of graph.values()) {
-    for (const dep of deps) imported.add(dep);
-  }
-  const roots = [...graph.keys()].filter((f) => !imported.has(f));
-  return roots.length > 0 ? roots : [...graph.keys()];
-}
-function runDFS(graph, entryPoints) {
-  const visited = /* @__PURE__ */ new Set();
-  const stack = [...entryPoints];
-  let node;
-  while ((node = stack.pop()) !== void 0) {
-    if (visited.has(node)) continue;
-    visited.add(node);
-    for (const neighbor of graph.get(node) ?? []) {
-      if (!visited.has(neighbor)) stack.push(neighbor);
-    }
-  }
-  return visited;
-}
-function findDeadChains(graph, deadFiles) {
-  const reverseGraph = buildReverseGraph(graph);
-  const result = /* @__PURE__ */ new Map();
-  for (const deadRoot of deadFiles) {
-    result.set(deadRoot, dfsDeadChain(deadRoot, graph, deadFiles, reverseGraph));
-  }
-  return result;
-}
 function detectCycles(graph) {
   const cycles = [];
   const seenKeys = /* @__PURE__ */ new Set();
@@ -261,12 +233,21 @@ function resolveNextJsEntries(rootDir) {
   const isNext = fs3.existsSync(path3.join(rootDir, "next.config.js")) || fs3.existsSync(path3.join(rootDir, "next.config.ts")) || fs3.existsSync(path3.join(rootDir, "next.config.mjs"));
   if (!isNext) return [];
   const entries = [];
-  for (const dir of ["pages", "app"]) {
+  for (const dir of ["pages", "app", "src/pages", "src/app"]) {
     const dirPath = path3.join(rootDir, dir);
     if (fs3.existsSync(dirPath)) entries.push(...collectSourceFiles(dirPath));
   }
   return entries;
 }
+function resolveFileBrowserEntries(rootDir) {
+  const PAGE_DIRS = ["src/pages", "src/routes", "src/views", "src/screens"];
+  const entries = [];
+  for (const rel of PAGE_DIRS) {
+    const dirPath = path3.join(rootDir, rel);
+    if (fs3.existsSync(dirPath)) entries.push(...collectSourceFiles(dirPath));
+  }
+  return entries;
+}
 function resolvePkgFieldEntries(rootDir, packageJson) {
   const entries = [];
   for (const field of ["main", "module"]) {
@@ -343,27 +324,6 @@ function recordCycle(cycleStart, path10, acc) {
     acc.cycles.push(cycle);
   }
 }
-function dfsDeadChain(deadRoot, graph, deadFiles, reverseGraph) {
-  const chain = [];
-  const visited = /* @__PURE__ */ new Set();
-  const stack = [...graph.get(deadRoot) ?? []];
-  let node;
-  while ((node = stack.pop()) !== void 0) {
-    if (visited.has(node) || node === deadRoot) continue;
-    visited.add(node);
-    if (deadFiles.has(node) || isOnlyImportedByDead(node, deadFiles, reverseGraph)) {
-      chain.push(node);
-      for (const next of graph.get(node) ?? []) {
-        if (!visited.has(next)) stack.push(next);
-      }
-    }
-  }
-  return chain;
-}
-function isOnlyImportedByDead(file, deadFiles, reverseGraph) {
-  const importers = reverseGraph.get(file) ?? /* @__PURE__ */ new Set();
-  return importers.size === 0 || [...importers].every((imp) => deadFiles.has(imp));
-}
 function buildReverseGraph(graph) {
   const rev = /* @__PURE__ */ new Map();
   for (const [file] of graph) {
@@ -490,16 +450,101 @@ function ensureDir(dir) {
 }
 
 // src/modules/dead-code.ts
+var FRAMEWORK_FILE_PATTERNS = [
+  /^next\.config\.(js|ts|mjs|cjs)$/,
+  /^middleware\.(ts|js)$/,
+  /^instrumentation\.(ts|js)$/,
+  /^tailwind\.config\.(js|ts|mjs|cjs)$/,
+  /^postcss\.config\.(js|ts|mjs|cjs)$/,
+  /^jest\.config\.(js|ts|mjs|cjs)$/,
+  /^vitest\.config\.(js|ts|mjs|cjs)$/,
+  /^vite\.config\.(js|ts|mjs|cjs)$/,
+  /^webpack\.config\.(js|ts|mjs|cjs)$/,
+  /^babel\.config\.(js|ts|cjs|mjs|json)$/,
+  /^\.babelrc\.(js|cjs)$/,
+  /^\.eslintrc\.(js|cjs)$/,
+  /^eslint\.config\.(js|ts|mjs|cjs)$/,
+  /^prettier\.config\.(js|ts|mjs|cjs)$/,
+  /^tsup\.config\.(ts|js)$/,
+  /^rollup\.config\.(js|ts|mjs)$/,
+  /^esbuild\.config\.(js|ts|mjs)$/,
+  /^commitlint\.config\.(js|ts)$/,
+  /^lint-staged\.config\.(js|ts|mjs|cjs)$/,
+  /^sentry\.(client|server|edge)\.config\.(ts|js)$/
+];
+function isFrameworkFile(filePath, rootDir) {
+  const rel = path5.relative(rootDir, filePath);
+  const basename = path5.basename(rel);
+  return FRAMEWORK_FILE_PATTERNS.some((re) => re.test(basename));
+}
 function runDeadCodeModule(project, graph, entryPoints, rootDir) {
   const allFiles = [...graph.keys()];
-  const effectiveEntries = entryPoints.length > 0 ? entryPoints : findRootFiles(graph);
-  const liveFiles = runDFS(graph, effectiveEntries);
-  const deadFiles = allFiles.filter((f) => !liveFiles.has(f));
-  const deadSet = new Set(deadFiles);
-  const chains = findDeadChains(graph, deadSet);
+  const entrySet = new Set(entryPoints);
+  const reverseGraph = buildReverseGraph(graph);
+  const excludedFiles = /* @__PURE__ */ new Set();
+  for (const file of allFiles) {
+    if (isFrameworkFile(file, rootDir)) excludedFiles.add(file);
+  }
+  const safeToDelete = allFiles.filter((f) => {
+    if (entrySet.has(f) || excludedFiles.has(f)) return false;
+    const importers = reverseGraph.get(f);
+    return !importers || importers.size === 0;
+  });
+  const deadSet = new Set(safeToDelete);
+  let changed = true;
+  while (changed) {
+    changed = false;
+    for (const file of allFiles) {
+      if (deadSet.has(file) || entrySet.has(file) || excludedFiles.has(file)) continue;
+      const importers = reverseGraph.get(file);
+      if (!importers || importers.size === 0) continue;
+      if ([...importers].every((imp) => deadSet.has(imp))) {
+        deadSet.add(file);
+        changed = true;
+      }
+    }
+  }
+  const transitivelyDead = [...deadSet].filter((f) => !safeToDelete.includes(f));
+  const chains = /* @__PURE__ */ new Map();
+  for (const root of safeToDelete) {
+    const chain = collectTransitiveChain(root, graph, deadSet);
+    chains.set(root, chain);
+  }
+  const liveFiles = new Set(allFiles.filter((f) => !deadSet.has(f)));
   const deadExports = findDeadExports(project, liveFiles);
-  const report = buildDeadCodeReport(deadFiles, chains, deadExports, rootDir);
-  return { deadFiles, liveFiles, chains, deadExports, report };
+  const report = buildDeadCodeReport(
+    safeToDelete,
+    transitivelyDead,
+    chains,
+    deadExports,
+    rootDir
+  );
+  return {
+    safeToDelete,
+    transitivelyDead,
+    deadFiles: [...deadSet],
+    liveFiles,
+    chains,
+    deadExports,
+    report
+  };
+}
+function collectTransitiveChain(root, graph, deadSet) {
+  const chain = [];
+  const visited = /* @__PURE__ */ new Set();
+  const stack = [...graph.get(root) ?? []];
+  let node;
+  while ((node = stack.pop()) !== void 0) {
+    if (visited.has(node) || node === root) continue;
+    visited.add(node);
+    if (deadSet.has(node)) {
+      chain.push(node);
+      for (const next of graph.get(node) ?? []) {
+        if (!visited.has(next)) stack.push(next);
+      }
+    }
+  }
+  return chain;
 }
 function findDeadExports(project, liveFiles) {
   const importedNames = buildImportedNameMap(project, liveFiles);
@@ -532,51 +577,61 @@ function getFileSize(filePath) {
     return 0;
   }
 }
-function buildDeadCodeReport(deadFiles, chains, deadExports, rootDir) {
+function buildDeadCodeReport(safeToDelete, transitivelyDead, chains, deadExports, rootDir) {
   const rel = (p) => path5.relative(rootDir, p).replaceAll("\\", "/");
-  const totalBytes = deadFiles.reduce((sum, f) => {
-    const chain = chains.get(f) ?? [];
-    return sum + getFileSize(f) + chain.reduce((s, c) => s + getFileSize(c), 0);
-  }, 0);
+  const allDeadFiles = [...safeToDelete, ...transitivelyDead];
+  const totalBytes = allDeadFiles.reduce((sum, f) => sum + getFileSize(f), 0);
   const totalKb = (totalBytes / 1024).toFixed(1);
   const lines = [
     "========================================",
     "  DEAD CODE REPORT",
-    `  Dead files : ${deadFiles.length}`,
-    `  Dead exports: ${deadExports.length}`,
-    `  Recoverable : ~${totalKb} KB`,
+    `  Safe to delete     : ${safeToDelete.length}`,
+    `  Transitively dead  : ${transitivelyDead.length}`,
+    `  Dead exports       : ${deadExports.length}`,
+    `  Recoverable        : ~${totalKb} KB`,
     "========================================",
     ""
   ];
-  if (deadFiles.length > 0) {
-    lines.push("\u2500\u2500 DEAD FILES \u2500\u2500", "");
-    for (const filePath of deadFiles) {
+  if (safeToDelete.length > 0) {
+    lines.push(
+      "\u2500\u2500 SAFE TO DELETE \u2500\u2500",
+      "(These files are not imported by any other file in the codebase)",
+      ""
+    );
+    for (const filePath of safeToDelete) {
       const chain = chains.get(filePath) ?? [];
-      const allFiles = [filePath, ...chain];
-      const sizeBytes = allFiles.reduce((s, f) => s + getFileSize(f), 0);
-      const sizeKb = (sizeBytes / 1024).toFixed(1);
-      const chainStr = chain.length > 0 ? [rel(filePath), ...chain.map(rel)].join(" \u2192 ") : rel(filePath);
-      const plural = allFiles.length === 1 ? "" : "s";
-      lines.push(
-        `DEAD FILE \u2014 ${rel(filePath)}`,
-        `Reason: Not imported anywhere in the codebase`,
-        `Chain: ${chainStr}`,
-        `Size: ~${sizeKb} KB removable across ${allFiles.length} file${plural}`,
-        `Action: Safe to delete all ${allFiles.length} file${plural}`,
-        ""
-      );
+      const sizeKb = (getFileSize(filePath) / 1024).toFixed(1);
+      lines.push(`  ${rel(filePath)}  (~${sizeKb} KB)`);
+      if (chain.length > 0) {
+        lines.push(`    \u2514\u2500 also makes dead: ${chain.map(rel).join(", ")}`);
+      }
+    }
+    lines.push("");
+  }
+  if (transitivelyDead.length > 0) {
+    lines.push(
+      "\u2500\u2500 TRANSITIVELY DEAD \u2500\u2500",
+      "(These files are only imported by dead files \u2014 they become orphaned too)",
+      ""
+    );
+    for (const filePath of transitivelyDead) {
+      const sizeKb = (getFileSize(filePath) / 1024).toFixed(1);
+      lines.push(`  ${rel(filePath)}  (~${sizeKb} KB)`);
     }
+    lines.push("");
   }
   if (deadExports.length > 0) {
-    lines.push("\u2500\u2500 DEAD EXPORTS \u2500\u2500", "");
+    lines.push(
+      "\u2500\u2500 DEAD EXPORTS \u2500\u2500",
+      "(Exported but never imported by any other file)",
+      ""
+    );
     for (const entry of deadExports) {
       lines.push(
-        `DEAD EXPORT \u2014 ${rel(entry.filePath)} \u2192 ${entry.exportName}() [line ${entry.line}]`,
-        `Reason: Exported but never imported`,
-        `Action: Remove the export (file itself is still live)`,
-        ""
+        `  ${rel(entry.filePath)} \u2192 ${entry.exportName} [line ${entry.line}]`
       );
     }
+    lines.push("");
   }
   return lines.join("\n");
 }
@@ -593,7 +648,8 @@ async function runDeadCode(dir, opts) {
     const entries = findEntryPoints(dir, packageJson);
     const result = runDeadCodeModule(project, graph, entries, dir);
     const dead = [
-      ...result.deadFiles.map((f) => ({ file: f, exportName: "(entire file)" })),
+      ...result.safeToDelete.map((f) => ({ file: f, exportName: "(entire file \u2014 safe to delete)" })),
+      ...result.transitivelyDead.map((f) => ({ file: f, exportName: "(entire file \u2014 transitively dead)" })),
       ...result.deadExports.map((e) => ({ file: e.filePath, exportName: e.exportName }))
     ];
     spinner.succeed(chalk.green(`Dead code scan complete \u2014 ${dead.length} item(s) found`));
@@ -669,7 +725,11 @@ function writeDeadOutput(result, opts) {
   if (!opts.output) return;
   if (opts.json) {
     writeJson(
-      { deadFiles: result.deadFiles, deadExports: result.deadExports },
+      {
+        safeToDelete: result.safeToDelete,
+        transitivelyDead: result.transitivelyDead,
+        deadExports: result.deadExports
+      },
       opts.output
     );
     console.log(chalk.cyan(`  JSON written to ${opts.output}`));
@@ -678,16 +738,21 @@ function writeDeadOutput(result, opts) {
   writeMarkdown(
     {
       title: "Dead Code Report",
-      summary: `${result.deadFiles.length} dead file(s), ${result.deadExports.length} dead export(s)`,
+      summary: `${result.safeToDelete.length} safe to delete, ${result.transitivelyDead.length} transitively dead, ${result.deadExports.length} dead export(s)`,
       sections: [
         {
-          title: "Dead Files",
+          title: "Safe to Delete",
           headers: ["File", "Chain"],
-          rows: result.deadFiles.map((f) => [
+          rows: result.safeToDelete.map((f) => [
             f,
             (result.chains.get(f) ?? []).join(" \u2192 ") || "\u2014"
           ])
         },
+        {
+          title: "Transitively Dead",
+          headers: ["File"],
+          rows: result.transitivelyDead.map((f) => [f])
+        },
         {
           title: "Dead Exports",
           headers: ["File", "Export", "Line"],
@@ -716,17 +781,30 @@ import {
 } from "ts-morph";
 
 // src/modules/dupe-finder.ts
+var MIN_BLOCK_CHARS = 120;
 async function runDupeFinder(dir, opts) {
-  const minLines = parseInt(opts.minLines ?? "5", 10);
+  const minLines = parseInt(opts.minLines ?? "15", 10);
   const spinner = ora3(chalk2.cyan(`Scanning for duplicate blocks (\u2265${minLines} lines)\u2026`)).start();
   try {
-    const files = glob(dir, ["**/*.ts", "**/*.tsx", "**/*.js", "**/*.jsx"], ["node_modules", "dist"]);
+    const files = glob(dir, ["**/*.ts", "**/*.tsx", "**/*.js", "**/*.jsx"], [
+      "node_modules",
+      "dist",
+      ".next",
+      "coverage",
+      "**/*.test.ts",
+      "**/*.test.tsx",
+      "**/*.spec.ts",
+      "**/*.spec.tsx",
+      "**/*.d.ts"
+    ]);
     const blockMap = /* @__PURE__ */ new Map();
     for (const filePath of files) {
       const content = fs6.readFileSync(filePath, "utf-8");
       const lines = content.split("\n").map((l) => l.trim()).filter(Boolean);
       for (let i = 0; i <= lines.length - minLines; i++) {
         const block = lines.slice(i, i + minLines).join("\n");
+        const meaningfulChars = block.replace(/[\s{}();,]/g, "").length;
+        if (meaningfulChars < MIN_BLOCK_CHARS) continue;
         if (!blockMap.has(block)) blockMap.set(block, []);
         blockMap.get(block).push({ file: filePath, startLine: i + 1 });
       }
@@ -741,6 +819,8 @@ async function runDupeFinder(dir, opts) {
         });
       }
     }
+    dupes.sort((a, b) => b.occurrences.length - a.occurrences.length);
+    if (dupes.length > 200) dupes.splice(200);
     spinner.succeed(chalk2.green(`Duplicate scan complete \u2014 ${dupes.length} duplicate block(s) found`));
     if (dupes.length === 0) {
       console.log(chalk2.green("  No duplicate blocks detected."));
@@ -1253,12 +1333,19 @@ async function main(opts) {
   if (modules.includes("dead-code")) {
     const spinner = createSpinner(chalk7.cyan("Analysing dead code\u2026"));
     const result = runDeadCodeModule(project, graph, entryPoints, rootDir);
-    deadFileCount = result.deadFiles.length + result.deadExports.length;
-    deadFilePaths.push(...result.deadFiles);
-    spinner.succeed(chalk7.green(`Dead code analysis complete \u2014 ${deadFileCount} item(s) found`));
+    deadFileCount = result.safeToDelete.length + result.transitivelyDead.length + result.deadExports.length;
+    deadFilePaths.push(...result.safeToDelete, ...result.transitivelyDead);
+    spinner.succeed(
+      chalk7.green(
+        `Dead code analysis complete \u2014 ${result.safeToDelete.length} safe to delete, ${result.transitivelyDead.length} transitively dead, ${result.deadExports.length} dead export(s)`
+      )
+    );
     if (result.report) {
       deadReportFile = "dead-code.txt";
-      writeReport(reportsDir, deadReportFile, result.report);
+      const banner = `prunify ${PKG_VERSION} \u2014 ${(/* @__PURE__ */ new Date()).toISOString()}
+
+`;
+      writeReport(reportsDir, deadReportFile, banner + result.report);
     }
   }
   if (modules.includes("dupes")) {
@@ -1274,8 +1361,28 @@ async function main(opts) {
     spinner.succeed(chalk7.green(`Circular import analysis complete \u2014 ${circularCount} cycle(s) found`));
     if (circularCount > 0) {
       circularReportFile = "circular.txt";
-      const cycleText = cycles.map((c, i) => `Cycle ${i + 1}: ${c.join(" \u2192 ")}`).join("\n");
-      writeReport(reportsDir, circularReportFile, cycleText);
+      const rel = (p) => path9.relative(rootDir, p).replaceAll("\\", "/");
+      const cycleLines = [
+        "========================================",
+        "  CIRCULAR DEPENDENCIES",
+        `  Cycles found: ${cycles.length}`,
+        "========================================",
+        ""
+      ];
+      for (let i = 0; i < cycles.length; i++) {
+        const cycle = cycles[i];
+        cycleLines.push(`Cycle ${i + 1} (${cycle.length} files):`);
+        for (let j = 0; j < cycle.length; j++) {
+          const arrow = j < cycle.length - 1 ? "  \u2192 " : "  \u2192 ";
+          cycleLines.push(`  ${rel(cycle[j])}`);
+        }
+        cycleLines.push(`  \u21BB ${rel(cycle[0])}  (back to start)`);
+        cycleLines.push("");
+      }
+      const banner = `prunify ${PKG_VERSION} \u2014 ${(/* @__PURE__ */ new Date()).toISOString()}
+
+`;
+      writeReport(reportsDir, circularReportFile, banner + cycleLines.join("\n"));
     }
   }
   if (modules.includes("deps")) {
@@ -1308,10 +1415,10 @@ async function main(opts) {
   });
   const fmt = (n) => n > 0 ? chalk7.yellow(String(n)) : chalk7.green("0");
   table.push(
-    ["Dead Files / Exports", fmt(deadFileCount), deadReportFile || "\u2014"],
+    ["Dead Code (files + exports)", fmt(deadFileCount), deadReportFile || "\u2014"],
+    ["Circular Dependencies", fmt(circularCount), circularReportFile || "\u2014"],
     ["Duplicate Clusters", fmt(dupeCount), dupesReportFile || "\u2014"],
     ["Unused Packages", fmt(unusedPkgCount), depsReportFile || "\u2014"],
-    ["Circular Deps", fmt(circularCount), circularReportFile || "\u2014"],
     ["Unused Assets", fmt(unusedAssetCount), assetsReportFile || "\u2014"]
   );
   console.log(table.toString());