prunify 0.1.4 → 0.1.6

package/dist/cli.js

@@ -4,8 +4,8 @@
 import { Command } from "commander";
 import chalk7 from "chalk";
 import Table6 from "cli-table3";
-import fs9 from "fs";
-import path9 from "path";
+import fs10 from "fs";
+import path10 from "path";
 import { fileURLToPath } from "url";
 import readline from "readline";
 
@@ -220,11 +220,11 @@ function detectCycles(graph) {
   const seenKeys = /* @__PURE__ */ new Set();
   const visited = /* @__PURE__ */ new Set();
   const inStack = /* @__PURE__ */ new Set();
-  const path10 = [];
+  const path11 = [];
   const acc = { seenKeys, cycles };
   for (const start of graph.keys()) {
     if (!visited.has(start)) {
-      dfsForCycles(start, graph, visited, inStack, path10, acc);
+      dfsForCycles(start, graph, visited, inStack, path11, acc);
     }
   }
   return cycles;
@@ -282,7 +282,7 @@ function resolveFallbackEntries(rootDir) {
 function mkFrame(node, graph) {
   return { node, neighbors: (graph.get(node) ?? /* @__PURE__ */ new Set()).values(), entered: false };
 }
-function dfsForCycles(start, graph, visited, inStack, path10, acc) {
+function dfsForCycles(start, graph, visited, inStack, path11, acc) {
   const stack = [mkFrame(start, graph)];
   while (stack.length > 0) {
     const frame = stack.at(-1);
@@ -294,30 +294,30 @@ function dfsForCycles(start, graph, visited, inStack, path10, acc) {
       }
       frame.entered = true;
       inStack.add(frame.node);
-      path10.push(frame.node);
+      path11.push(frame.node);
     }
     const { done, value: neighbor } = frame.neighbors.next();
     if (done) {
       stack.pop();
-      path10.pop();
+      path11.pop();
       inStack.delete(frame.node);
       visited.add(frame.node);
     } else {
-      handleCycleNeighbor(neighbor, stack, path10, inStack, visited, acc, graph);
+      handleCycleNeighbor(neighbor, stack, path11, inStack, visited, acc, graph);
     }
   }
 }
-function handleCycleNeighbor(neighbor, stack, path10, inStack, visited, acc, graph) {
+function handleCycleNeighbor(neighbor, stack, path11, inStack, visited, acc, graph) {
   if (inStack.has(neighbor)) {
-    recordCycle(neighbor, path10, acc);
+    recordCycle(neighbor, path11, acc);
   } else if (!visited.has(neighbor)) {
     stack.push(mkFrame(neighbor, graph));
   }
 }
-function recordCycle(cycleStart, path10, acc) {
-  const idx = path10.indexOf(cycleStart);
+function recordCycle(cycleStart, path11, acc) {
+  const idx = path11.indexOf(cycleStart);
   if (idx === -1) return;
-  const cycle = normalizeCycle(path10.slice(idx));
+  const cycle = normalizeCycle(path11.slice(idx));
   const key = cycle.join("\0");
   if (!acc.seenKeys.has(key)) {
     acc.seenKeys.add(key);
@@ -1264,20 +1264,124 @@ function buildAssetReport(unused, totalAssets, rootDir) {
   return lines.join("\n");
 }
 
+// src/core/report-parser.ts
+import fs9 from "fs";
+import path9 from "path";
+function parseDeadCodeReportFile(reportPath, rootDir) {
+  if (!fs9.existsSync(reportPath)) return [];
+  const content = fs9.readFileSync(reportPath, "utf-8");
+  return parseDeadCodeReportContent(content, rootDir);
+}
+function parseDeadCodeReportContent(content, rootDir) {
+  const lines = content.split(/\r?\n/);
+  const paths = [];
+  const legacyRe = /^DEAD FILE —\s+(.+)$/;
+  let legacyMode = content.includes("DEAD FILE \u2014") && content.includes("Dead files :");
+  if (legacyMode) {
+    for (const line of lines) {
+      const m = legacyRe.exec(line.trim());
+      if (m) paths.push(path9.resolve(rootDir, m[1].trim()));
+    }
+    return [...new Set(paths)];
+  }
+  let section = "none";
+  const fileLineRe = /^\s{2}(.+?)\s+\(~[\d.]+\s+KB\)\s*$/;
+  for (const line of lines) {
+    if (line.includes("\u2500\u2500 SAFE TO DELETE \u2500\u2500")) {
+      section = "safe";
+      continue;
+    }
+    if (line.includes("\u2500\u2500 TRANSITIVELY DEAD \u2500\u2500")) {
+      section = "transitive";
+      continue;
+    }
+    if (line.includes("\u2500\u2500 DEAD EXPORTS \u2500\u2500")) {
+      section = "none";
+      continue;
+    }
+    if (line.startsWith("\u2500\u2500 ") && line.includes("\u2500\u2500")) {
+      if (section !== "none" && !line.includes("SAFE TO DELETE") && !line.includes("TRANSITIVELY DEAD")) {
+        section = "none";
+      }
+      continue;
+    }
+    if (section === "safe" || section === "transitive") {
+      if (line.includes("\u2514\u2500") || line.includes("also makes dead")) continue;
+      const m = fileLineRe.exec(line);
+      if (m) {
+        const rel = m[1].trim();
+        if (rel && !rel.startsWith("(")) {
+          paths.push(path9.resolve(rootDir, rel));
+        }
+      }
+    }
+  }
+  return [...new Set(paths)];
+}
+function countDeadExportsInReport(content) {
+  let section = false;
+  let n = 0;
+  for (const line of content.split(/\r?\n/)) {
+    if (line.includes("\u2500\u2500 DEAD EXPORTS \u2500\u2500")) {
+      section = true;
+      continue;
+    }
+    if (section && line.startsWith("\u2500\u2500 ") && line.includes("\u2500\u2500")) break;
+    if (section && /\s→\s.+\[line \d+\]/.test(line)) n++;
+  }
+  return n;
+}
+function parseAssetsReportFile(reportPath, rootDir) {
+  if (!fs9.existsSync(reportPath)) return [];
+  const content = fs9.readFileSync(reportPath, "utf-8");
+  return parseAssetsReportContent(content, rootDir);
+}
+function parseAssetsReportContent(content, rootDir) {
+  const lines = content.split(/\r?\n/);
+  const paths = [];
+  let inTable = false;
+  let headerSeen = false;
+  for (const line of lines) {
+    const trimmed = line.trim();
+    if (trimmed.startsWith("## Unused Assets")) {
+      inTable = true;
+      headerSeen = false;
+      continue;
+    }
+    if (inTable && trimmed.startsWith("## ")) {
+      break;
+    }
+    if (!inTable) continue;
+    if (trimmed.startsWith("|") && trimmed.includes("---")) {
+      headerSeen = true;
+      continue;
+    }
+    if (!headerSeen || !trimmed.startsWith("|")) continue;
+    const cells = trimmed.split("|").map((c) => c.trim()).filter(Boolean);
+    if (cells.length >= 1) {
+      const rel = cells[0];
+      if (rel.toLowerCase() === "asset") continue;
+      if (rel.includes("---")) continue;
+      paths.push(path9.resolve(rootDir, rel));
+    }
+  }
+  return [...new Set(paths)];
+}
+
 // src/cli.ts
 function readPkgVersion() {
   try {
     if (typeof import.meta !== "undefined" && import.meta.url) {
-      const dir = path9.dirname(fileURLToPath(import.meta.url));
-      const pkgPath = path9.resolve(dir, "..", "package.json");
-      return JSON.parse(fs9.readFileSync(pkgPath, "utf-8")).version;
+      const dir = path10.dirname(fileURLToPath(import.meta.url));
+      const pkgPath = path10.resolve(dir, "..", "package.json");
+      return JSON.parse(fs10.readFileSync(pkgPath, "utf-8")).version;
     }
   } catch {
   }
   try {
     const dir = globalThis.__dirname ?? __dirname;
-    const pkgPath = path9.resolve(dir, "..", "package.json");
-    return JSON.parse(fs9.readFileSync(pkgPath, "utf-8")).version;
+    const pkgPath = path10.resolve(dir, "..", "package.json");
+    return JSON.parse(fs10.readFileSync(pkgPath, "utf-8")).version;
   } catch {
     return "0.0.0";
   }
@@ -1290,11 +1394,14 @@ program.name("prunify").description("npm run clean. ship with confidence.").vers
   "Glob pattern to ignore (repeatable)",
   (val, acc) => [...acc, val],
   []
-).option("--out <path>", "Output directory for reports").option("--html", "Also generate code_health.html").option("--delete", "Prompt to delete dead files after analysis").option("--ci", "CI mode: exit 1 if issues found, no interactive prompts").action(main);
+).option("--out <path>", "Output directory for reports").option("--html", "Also generate code_health.html").option("--delete", "Delete dead files (uses prunify-reports/dead-code.txt if present, else runs analysis)").option(
+  "--delete-assets",
+  "Delete unused public assets (uses prunify-reports/assets.md if present, else runs asset scan)"
+).option("--ci", "CI mode: exit 1 if issues found, no interactive prompts").action(main);
 program.parse();
 async function main(opts) {
-  const rootDir = path9.resolve(opts.dir);
-  if (!fs9.existsSync(path9.join(rootDir, "package.json"))) {
+  const rootDir = path10.resolve(opts.dir);
+  if (!fs10.existsSync(path10.join(rootDir, "package.json"))) {
     console.error(chalk7.red(`\u2717  No package.json found in ${rootDir}`));
     console.error(chalk7.dim("   Use --dir <path> to point to your project root."));
     process.exit(1);
@@ -1303,21 +1410,53 @@ async function main(opts) {
   console.log();
   console.log(chalk7.bold.cyan("\u{1F9F9} prunify \u2014 npm run clean. ship with confidence."));
   console.log();
-  const parseSpinner = createSpinner(chalk7.cyan("Parsing codebase\u2026"));
-  const files = discoverFiles(rootDir, opts.ignore);
-  parseSpinner.succeed(chalk7.green(`Parsed codebase \u2014 ${files.length} file(s) found`));
-  const graphSpinner = createSpinner(chalk7.cyan("Building import graph\u2026"));
-  const project = buildProject(files);
-  const graph = buildGraph(files, (f) => {
-    const sf = project.getSourceFile(f);
-    return sf ? getImportsForFile(sf) : [];
-  });
-  const edgeCount = [...graph.values()].reduce((n, s) => n + s.size, 0);
-  graphSpinner.succeed(chalk7.green(`Import graph built \u2014 ${edgeCount} edge(s)`));
-  const packageJson = loadPackageJson2(rootDir);
-  const entryPoints = opts.entry ? [path9.resolve(opts.entry)] : findEntryPoints(rootDir, packageJson);
   const reportsDir = ensureReportsDir(rootDir, opts.out);
   appendToGitignore(rootDir);
+  const deadReportPath = path10.join(reportsDir, "dead-code.txt");
+  const assetsReportPath = path10.join(reportsDir, "assets.md");
+  const loadDeadFromCache = Boolean(opts.delete && fs10.existsSync(deadReportPath));
+  const loadAssetsFromCache = Boolean(opts.deleteAssets && fs10.existsSync(assetsReportPath));
+  let deadFilePaths = [];
+  let unusedAssetPaths = [];
+  if (loadDeadFromCache) {
+    deadFilePaths = parseDeadCodeReportFile(deadReportPath, rootDir);
+    console.log(
+      chalk7.cyan(
+        `  Using existing dead-code report (${deadFilePaths.length} file(s) to delete) \u2192 ${path10.relative(rootDir, deadReportPath)}`
+      )
+    );
+  }
+  if (loadAssetsFromCache) {
+    unusedAssetPaths = parseAssetsReportFile(assetsReportPath, rootDir);
+    console.log(
+      chalk7.cyan(
+        `  Using existing assets report (${unusedAssetPaths.length} asset(s) to delete) \u2192 ${path10.relative(rootDir, assetsReportPath)}`
+      )
+    );
+  }
+  const runDeadCode2 = !loadDeadFromCache && (modules.includes("dead-code") || Boolean(opts.delete));
+  const runCircular2 = modules.includes("circular");
+  const runAssets = !loadAssetsFromCache && (modules.includes("assets") || Boolean(opts.deleteAssets));
+  const needsGraph = runCircular2 || runDeadCode2;
+  let files = [];
+  let project;
+  let graph;
+  let entryPoints = [];
+  if (needsGraph) {
+    const parseSpinner = createSpinner(chalk7.cyan("Parsing codebase\u2026"));
+    files = discoverFiles(rootDir, opts.ignore);
+    parseSpinner.succeed(chalk7.green(`Parsed codebase \u2014 ${files.length} file(s) found`));
+    const graphSpinner = createSpinner(chalk7.cyan("Building import graph\u2026"));
+    project = buildProject(files);
+    graph = buildGraph(files, (f) => {
+      const sf = project.getSourceFile(f);
+      return sf ? getImportsForFile(sf) : [];
+    });
+    const edgeCount = [...graph.values()].reduce((n, s) => n + s.size, 0);
+    graphSpinner.succeed(chalk7.green(`Import graph built \u2014 ${edgeCount} edge(s)`));
+    const packageJson = loadPackageJson2(rootDir);
+    entryPoints = opts.entry ? [path10.resolve(opts.entry)] : findEntryPoints(rootDir, packageJson);
+  }
   console.log();
   let deadFileCount = 0;
   let dupeCount = 0;
@@ -1329,12 +1468,24 @@ async function main(opts) {
   let depsReportFile = "";
   let circularReportFile = "";
   let assetsReportFile = "";
-  const deadFilePaths = [];
-  if (modules.includes("dead-code")) {
+  if (loadDeadFromCache) {
+    deadReportFile = "dead-code.txt";
+    try {
+      const raw = fs10.readFileSync(deadReportPath, "utf-8");
+      deadFileCount = deadFilePaths.length + countDeadExportsInReport(raw);
+    } catch {
+      deadFileCount = deadFilePaths.length;
+    }
+  }
+  if (loadAssetsFromCache) {
+    assetsReportFile = "assets.md";
+    unusedAssetCount = unusedAssetPaths.length;
+  }
+  if (runDeadCode2) {
     const spinner = createSpinner(chalk7.cyan("Analysing dead code\u2026"));
     const result = runDeadCodeModule(project, graph, entryPoints, rootDir);
     deadFileCount = result.safeToDelete.length + result.transitivelyDead.length + result.deadExports.length;
-    deadFilePaths.push(...result.safeToDelete, ...result.transitivelyDead);
+    deadFilePaths = [...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)`
@@ -1342,23 +1493,26 @@ async function main(opts) {
     );
     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")) {
-    const outputPath = path9.join(reportsDir, "dupes.md");
+    const outputPath = path10.join(reportsDir, "dupes.md");
     const dupes = await runDupeFinder(rootDir, { output: outputPath });
     dupeCount = dupes.length;
     if (dupeCount > 0) dupesReportFile = "dupes.md";
   }
-  if (modules.includes("circular")) {
+  if (runCircular2) {
     const spinner = createSpinner(chalk7.cyan("Analysing circular imports\u2026"));
     const cycles = detectCycles(graph);
     circularCount = cycles.length;
     spinner.succeed(chalk7.green(`Circular import analysis complete \u2014 ${circularCount} cycle(s) found`));
     if (circularCount > 0) {
       circularReportFile = "circular.txt";
-      const rel = (p) => path9.relative(rootDir, p).replaceAll("\\", "/");
+      const rel = (p) => path10.relative(rootDir, p).replaceAll("\\", "/");
       const cycleLines = [
         "========================================",
         "  CIRCULAR DEPENDENCIES",
@@ -1376,27 +1530,31 @@ async function main(opts) {
         cycleLines.push(`  \u21BB ${rel(cycle[0])}  (back to start)`);
         cycleLines.push("");
       }
-      writeReport(reportsDir, circularReportFile, cycleLines.join("\n"));
+      const banner = `prunify ${PKG_VERSION} \u2014 ${(/* @__PURE__ */ new Date()).toISOString()}
+
+`;
+      writeReport(reportsDir, circularReportFile, banner + cycleLines.join("\n"));
     }
   }
   if (modules.includes("deps")) {
-    const outputPath = path9.join(reportsDir, "deps.md");
+    const outputPath = path10.join(reportsDir, "deps.md");
     const issues = await runDepCheck({ cwd: rootDir, output: outputPath });
     unusedPkgCount = issues.filter((i) => i.type === "unused").length;
     if (issues.length > 0) depsReportFile = "deps.md";
   }
-  if (modules.includes("assets")) {
-    const outputPath = path9.join(reportsDir, "assets.md");
+  if (runAssets) {
+    const outputPath = path10.join(reportsDir, "assets.md");
     const unusedAssets = await runAssetCheck(rootDir, { output: outputPath });
+    unusedAssetPaths = unusedAssets.map((a) => a.filePath);
     unusedAssetCount = unusedAssets.length;
     if (unusedAssetCount > 0) assetsReportFile = "assets.md";
   }
   if (modules.includes("health")) {
-    const outputPath = path9.join(reportsDir, "health-report.md");
+    const outputPath = path10.join(reportsDir, "health-report.md");
     await runHealthReport(rootDir, { output: outputPath });
   }
   if (opts.html) {
-    const htmlPath = path9.join(reportsDir, "code_health.html");
+    const htmlPath = path10.join(reportsDir, "code_health.html");
     writeHtmlReport(htmlPath, rootDir, deadFilePaths, circularCount, dupeCount, unusedPkgCount);
     console.log(chalk7.cyan(`  HTML report written to ${htmlPath}`));
   }
@@ -1418,22 +1576,72 @@ async function main(opts) {
   console.log(table.toString());
   console.log();
   if (opts.delete && deadFilePaths.length > 0) {
-    console.log(chalk7.yellow(`Dead files (${deadFilePaths.length}):`));
+    console.log(chalk7.yellow(`Dead code files to delete (${deadFilePaths.length}):`));
     for (const f of deadFilePaths) {
-      console.log(chalk7.dim(`  ${path9.relative(rootDir, f)}`));
+      console.log(chalk7.dim(`  ${path10.relative(rootDir, f)}`));
     }
     console.log();
     if (!opts.ci) {
-      const confirmed = await confirmPrompt("Delete these files? (y/N) ");
+      const confirmed = await confirmPrompt("Delete these dead code files? (y/N) ");
       if (confirmed) {
+        let removed = 0;
         for (const f of deadFilePaths) {
-          fs9.rmSync(f, { force: true });
+          try {
+            if (fs10.existsSync(f)) {
+              fs10.rmSync(f, { force: true });
+              removed++;
+            }
+          } catch {
+            console.log(chalk7.red(`  Failed to delete: ${path10.relative(rootDir, f)}`));
+          }
+        }
+        console.log(chalk7.green(`  Deleted ${removed} file(s).`));
+      } else {
+        console.log(chalk7.dim("  Skipped."));
+      }
+    }
+  } else if (opts.delete && deadFilePaths.length === 0) {
+    console.log(chalk7.dim("  --delete: no dead code files listed in report (nothing to delete)."));
+  }
+  if (opts.deleteAssets && unusedAssetPaths.length > 0) {
+    const publicDir = path10.join(rootDir, "public");
+    const safePaths = unusedAssetPaths.filter((p) => {
+      const norm = path10.normalize(p);
+      return norm.startsWith(path10.normalize(publicDir + path10.sep));
+    });
+    if (safePaths.length !== unusedAssetPaths.length) {
+      console.log(
+        chalk7.yellow(
+          `  Warning: skipped ${unusedAssetPaths.length - safePaths.length} path(s) outside public/`
+        )
+      );
+    }
+    console.log(chalk7.yellow(`Unused assets to delete (${safePaths.length}):`));
+    for (const f of safePaths) {
+      console.log(chalk7.dim(`  ${path10.relative(rootDir, f)}`));
+    }
+    console.log();
+    if (!opts.ci) {
+      const confirmed = await confirmPrompt("Delete these unused assets? (y/N) ");
+      if (confirmed) {
+        let removed = 0;
+        for (const f of safePaths) {
+          try {
+            if (fs10.existsSync(f)) {
+              fs10.rmSync(f, { force: true });
+              removed++;
+            }
+          } catch {
+            console.log(chalk7.red(`  Failed to delete: ${path10.relative(rootDir, f)}`));
+          }
         }
-        console.log(chalk7.green(`  Deleted ${deadFilePaths.length} file(s).`));
+        console.log(chalk7.green(`  Deleted ${removed} asset file(s).`));
       } else {
         console.log(chalk7.dim("  Skipped."));
       }
     }
+  } else if (opts.deleteAssets && unusedAssetPaths.length === 0) {
+    console.log(chalk7.dim("  --delete-assets: no unused assets in report (nothing to delete)."));
   }
   if (opts.ci) {
     const hasIssues = deadFileCount > 0 || dupeCount > 0 || unusedPkgCount > 0 || circularCount > 0 || unusedAssetCount > 0;
@@ -1447,7 +1655,7 @@ function resolveModules(only) {
 }
 function loadPackageJson2(dir) {
   try {
-    return JSON.parse(fs9.readFileSync(path9.join(dir, "package.json"), "utf-8"));
+    return JSON.parse(fs10.readFileSync(path10.join(dir, "package.json"), "utf-8"));
   } catch {
     return null;
   }
@@ -1468,7 +1676,7 @@ function writeHtmlReport(outputPath, rootDir, deadFiles, circularCount, dupeCoun
     ["Circular Dependencies", String(circularCount)],
     ["Unused Packages", String(unusedPkgCount)]
   ].map(([label, val]) => `    <tr><td>${label}</td><td>${val}</td></tr>`).join("\n");
-  const deadList = deadFiles.length > 0 ? `<ul>${deadFiles.map((f) => `<li>${path9.relative(rootDir, f)}</li>`).join("")}</ul>` : "<p>None</p>";
+  const deadList = deadFiles.length > 0 ? `<ul>${deadFiles.map((f) => `<li>${path10.relative(rootDir, f)}</li>`).join("")}</ul>` : "<p>None</p>";
   const html = `<!DOCTYPE html>
 <html lang="en">
 <head>
@@ -1495,7 +1703,7 @@ ${rows}
   ${deadList}
 </body>
 </html>`;
-  fs9.mkdirSync(path9.dirname(outputPath), { recursive: true });
-  fs9.writeFileSync(outputPath, html, "utf-8");
+  fs10.mkdirSync(path10.dirname(outputPath), { recursive: true });
+  fs10.writeFileSync(outputPath, html, "utf-8");
 }
 //# sourceMappingURL=cli.js.map