prunify 0.1.6 → 0.1.7

package/README.md

@@ -89,15 +89,22 @@ npx prunify --ci
 
 Summary
 
-┌─────────────────────────────┬───────┬──────────────────┐
-│ Check                       │ Found │ Output File      │
-├─────────────────────────────┼───────┼──────────────────┤
-│ Dead Code (files + exports) │ 7     │ dead-code.txt    │
-│ Circular Dependencies       │ 2     │ circular.txt     │
-│ Duplicate Clusters          │ 3     │ dupes.md         │
-│ Unused Packages             │ 4     │ deps.md          │
-│ Unused Assets               │ 5     │ assets.md        │
-└─────────────────────────────┴───────┴──────────────────┘
+========================================
+  DEAD CODE REPORT
+  Safe to delete     : 3
+  Transitively dead  : 1
+  Dead exports       : 3
+  Recoverable        : ~12.4 KB
+========================================
+
+┌───────────────────────────┬───────┬──────────────────┐
+│ Check                     │ Found │ Output File      │
+├───────────────────────────┼───────┼──────────────────┤
+│ Circular Dependencies     │ 2     │ circular.txt     │
+│ Duplicate Clusters        │ 3     │ dupes.md         │
+│ Unused Packages           │ 4     │ deps.md          │
+│ Unused Assets             │ 5     │ assets.md        │
+└───────────────────────────┴───────┴──────────────────┘
 ```
 
 ---
@@ -184,7 +191,7 @@ prunify builds a directed import graph (with `tsconfig.json` path aliases) and a
 
 1. **Safe to delete** — No other project file imports this file, and it is not treated as an app entry (e.g. Next.js `pages/` / `app/` routes) or a known framework/config file (`next.config.*`, `middleware.*`, `tailwind.config.*`, etc.).
 2. **Transitively dead** — The file is only imported by files that are already dead; removing the safe-to-delete roots would leave it orphaned.
-3. **Dead exports** — The file is still used, but specific named exports are never imported by name elsewhere (namespace imports count as “all exports used”).
+3. **Dead exports** — The file is still used, but specific named exports are never imported by name elsewhere (namespace imports count as “all exports used”). Files under **`pages/`** / **`src/pages/`**, route trees like **`src/routes/`**, **`src/views/`**, **`src/screens/`**, and Next.js App Router entry files (**`page.tsx`**, **`layout.tsx`**, **`route.ts`**, etc. under **`app/`** / **`src/app/`**) are **excluded** — the framework consumes those exports without normal `import` edges.
 
 **Why not “reachable from entry” only?** A pure reachability scan from entry points can mark heavily shared modules as dead if any link in the chain fails to resolve. The importer-based model matches the usual meaning of “nothing uses this file.”
 

package/dist/cli.cjs

@@ -238,6 +238,83 @@ function findEntryPoints(rootDir, packageJson) {
   ];
   return [...new Set(entries)];
 }
+var APP_SPECIAL_ENTRY_BASENAMES_LOWER = /* @__PURE__ */ new Set([
+  "page.tsx",
+  "page.ts",
+  "page.jsx",
+  "page.js",
+  "layout.tsx",
+  "layout.ts",
+  "layout.jsx",
+  "layout.js",
+  "template.tsx",
+  "template.ts",
+  "template.jsx",
+  "template.js",
+  "loading.tsx",
+  "loading.ts",
+  "loading.jsx",
+  "loading.js",
+  "error.tsx",
+  "error.ts",
+  "error.jsx",
+  "error.js",
+  "global-error.tsx",
+  "global-error.ts",
+  "global-error.jsx",
+  "global-error.js",
+  "not-found.tsx",
+  "not-found.ts",
+  "not-found.jsx",
+  "not-found.js",
+  "forbidden.tsx",
+  "forbidden.ts",
+  "forbidden.jsx",
+  "forbidden.js",
+  "unauthorized.tsx",
+  "unauthorized.ts",
+  "unauthorized.jsx",
+  "unauthorized.js",
+  "default.tsx",
+  "default.ts",
+  "default.jsx",
+  "default.js",
+  "route.ts",
+  "route.js",
+  "opengraph-image.tsx",
+  "opengraph-image.ts",
+  "opengraph-image.js",
+  "twitter-image.tsx",
+  "twitter-image.ts",
+  "twitter-image.js",
+  "icon.tsx",
+  "icon.ts",
+  "icon.jsx",
+  "icon.js",
+  "apple-icon.tsx",
+  "apple-icon.ts",
+  "apple-icon.js",
+  "sitemap.ts",
+  "sitemap.js",
+  "robots.ts",
+  "robots.js",
+  "manifest.ts",
+  "manifest.js"
+]);
+function isConventionRoutedSourceFile(filePath, rootDir) {
+  const rel = import_node_path3.default.relative(rootDir, import_node_path3.default.normalize(filePath)).replaceAll("\\", "/");
+  if (rel.startsWith("..")) return false;
+  if (rel === "pages" || rel.startsWith("pages/")) return true;
+  if (rel.startsWith("src/pages/")) return true;
+  if (rel.startsWith("src/routes/") || rel.startsWith("src/views/") || rel.startsWith("src/screens/")) {
+    return true;
+  }
+  if (rel.startsWith("app/") || rel.startsWith("src/app/")) {
+    const base = import_node_path3.default.basename(rel).toLowerCase();
+    if (APP_SPECIAL_ENTRY_BASENAMES_LOWER.has(base)) return true;
+  }
+  return false;
+}
 function detectCycles(graph) {
   const cycles = [];
   const seenKeys = /* @__PURE__ */ new Set();
@@ -534,7 +611,7 @@ function runDeadCodeModule(project, graph, entryPoints, rootDir) {
     chains.set(root, chain);
   }
   const liveFiles = new Set(allFiles.filter((f) => !deadSet.has(f)));
-  const deadExports = findDeadExports(project, liveFiles);
+  const deadExports = findDeadExports(project, liveFiles, rootDir);
   const report = buildDeadCodeReport(
     safeToDelete,
     transitivelyDead,
@@ -569,10 +646,12 @@ function collectTransitiveChain(root, graph, deadSet) {
   }
   return chain;
 }
-function findDeadExports(project, liveFiles) {
+function findDeadExports(project, liveFiles, rootDir) {
   const importedNames = buildImportedNameMap(project, liveFiles);
   const dead = [];
   for (const filePath of liveFiles) {
+    if (isFrameworkFile(filePath, rootDir)) continue;
+    if (isConventionRoutedSourceFile(filePath, rootDir)) continue;
     collectFileDeadExports(filePath, project, importedNames, dead);
   }
   return dead;
@@ -646,7 +725,7 @@ function buildDeadCodeReport(safeToDelete, transitivelyDead, chains, deadExports
   if (deadExports.length > 0) {
     lines.push(
       "\u2500\u2500 DEAD EXPORTS \u2500\u2500",
-      "(Exported but never imported by any other file)",
+      "(Exported but never imported by any other file; pages/routes omitted)",
       ""
     );
     for (const entry of deadExports) {
@@ -1351,6 +1430,19 @@ function countDeadExportsInReport(content) {
   }
   return n;
 }
+function parseDeadCodeReportSummary(content) {
+  const safe = content.match(/Safe to delete\s*:\s*(\d+)/);
+  const trans = content.match(/Transitively dead\s*:\s*(\d+)/);
+  const exp = content.match(/Dead exports\s*:\s*(\d+)/);
+  const rec = content.match(/Recoverable\s*:\s*(~[\d.]+\s*KB)/i);
+  if (!safe || !trans || !exp) return null;
+  return {
+    safeToDelete: Number(safe[1]),
+    transitivelyDead: Number(trans[1]),
+    deadExports: Number(exp[1]),
+    recoverableDisplay: rec ? rec[1].trim() : "~0.0 KB"
+  };
+}
 function parseAssetsReportFile(reportPath, rootDir) {
   if (!import_node_fs9.default.existsSync(reportPath)) return [];
   const content = import_node_fs9.default.readFileSync(reportPath, "utf-8");
@@ -1479,23 +1571,35 @@ async function main(opts) {
     entryPoints = opts.entry ? [import_node_path10.default.resolve(opts.entry)] : findEntryPoints(rootDir, packageJson);
   }
   console.log();
-  let deadFileCount = 0;
+  let deadSafeCount = 0;
+  let deadTransitiveCount = 0;
+  let deadExportsCount = 0;
+  let deadRecoverDisplay = "~0.0 KB";
   let dupeCount = 0;
   let unusedPkgCount = 0;
   let circularCount = 0;
   let unusedAssetCount = 0;
-  let deadReportFile = "";
   let dupesReportFile = "";
   let depsReportFile = "";
   let circularReportFile = "";
   let assetsReportFile = "";
   if (loadDeadFromCache) {
-    deadReportFile = "dead-code.txt";
     try {
       const raw = import_node_fs10.default.readFileSync(deadReportPath, "utf-8");
-      deadFileCount = deadFilePaths.length + countDeadExportsInReport(raw);
+      const parsed = parseDeadCodeReportSummary(raw);
+      if (parsed) {
+        deadSafeCount = parsed.safeToDelete;
+        deadTransitiveCount = parsed.transitivelyDead;
+        deadExportsCount = parsed.deadExports;
+        deadRecoverDisplay = parsed.recoverableDisplay;
+      } else {
+        deadExportsCount = countDeadExportsInReport(raw);
+        deadSafeCount = deadFilePaths.length;
+        deadTransitiveCount = 0;
+        deadRecoverDisplay = "\u2014";
+      }
     } catch {
-      deadFileCount = deadFilePaths.length;
+      deadSafeCount = deadFilePaths.length;
     }
   }
   if (loadAssetsFromCache) {
@@ -1505,19 +1609,22 @@ async function main(opts) {
   if (runDeadCode2) {
     const spinner = createSpinner(import_chalk7.default.cyan("Analysing dead code\u2026"));
     const result = runDeadCodeModule(project, graph, entryPoints, rootDir);
-    deadFileCount = result.safeToDelete.length + result.transitivelyDead.length + result.deadExports.length;
     deadFilePaths = [...result.safeToDelete, ...result.transitivelyDead];
+    deadSafeCount = result.safeToDelete.length;
+    deadTransitiveCount = result.transitivelyDead.length;
+    deadExportsCount = result.deadExports.length;
+    const recoverBytes = deadFilePaths.reduce((sum, f) => sum + getFileSize(f), 0);
+    deadRecoverDisplay = `~${(recoverBytes / 1024).toFixed(1)} KB`;
     spinner.succeed(
       import_chalk7.default.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";
       const banner = `prunify ${PKG_VERSION} \u2014 ${(/* @__PURE__ */ new Date()).toISOString()}
 
 `;
-      writeReport(reportsDir, deadReportFile, banner + result.report);
+      writeReport(reportsDir, "dead-code.txt", banner + result.report);
     }
   }
   if (modules.includes("dupes")) {
@@ -1582,13 +1689,25 @@ async function main(opts) {
   console.log();
   console.log(import_chalk7.default.bold("Summary"));
   console.log();
+  const fmt = (n) => n > 0 ? import_chalk7.default.yellow(String(n)) : import_chalk7.default.green("0");
+  const showDeadSummary = modules.includes("dead-code") || Boolean(opts.delete) || loadDeadFromCache || runDeadCode2;
+  if (showDeadSummary) {
+    const recoverKb = parseFloat(deadRecoverDisplay.replace(/[^\d.]/g, "")) || 0;
+    const recoverCh = deadRecoverDisplay === "\u2014" ? import_chalk7.default.dim("\u2014") : recoverKb > 0 ? import_chalk7.default.yellow(deadRecoverDisplay) : import_chalk7.default.green(deadRecoverDisplay);
+    console.log(import_chalk7.default.dim("========================================"));
+    console.log(import_chalk7.default.dim("  DEAD CODE REPORT"));
+    console.log(import_chalk7.default.dim("  Safe to delete     : ") + fmt(deadSafeCount));
+    console.log(import_chalk7.default.dim("  Transitively dead  : ") + fmt(deadTransitiveCount));
+    console.log(import_chalk7.default.dim("  Dead exports       : ") + fmt(deadExportsCount));
+    console.log(import_chalk7.default.dim("  Recoverable        : ") + recoverCh);
+    console.log(import_chalk7.default.dim("========================================"));
+    console.log();
+  }
   const table = new import_cli_table36.default({
     head: [import_chalk7.default.bold("Check"), import_chalk7.default.bold("Found"), import_chalk7.default.bold("Output File")],
     style: { head: [], border: [] }
   });
-  const fmt = (n) => n > 0 ? import_chalk7.default.yellow(String(n)) : import_chalk7.default.green("0");
   table.push(
-    ["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"],
@@ -1665,7 +1784,8 @@ async function main(opts) {
     console.log(import_chalk7.default.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;
+    const deadIssueCount = deadSafeCount + deadTransitiveCount + deadExportsCount;
+    const hasIssues = deadIssueCount > 0 || dupeCount > 0 || unusedPkgCount > 0 || circularCount > 0 || unusedAssetCount > 0;
     if (hasIssues) process.exit(1);
   }
 }