pruny 1.19.0 → 1.20.0

package/dist/index.js

@@ -9369,6 +9369,8 @@ var IGNORED_EXPORT_NAMES = new Set([
   "OPTIONS",
   "default"
 ]);
+var NEST_LIFECYCLE_METHODS = new Set(["constructor", "onModuleInit", "onApplicationBootstrap", "onModuleDestroy", "beforeApplicationShutdown", "onApplicationShutdown"]);
+var classMethodRegex = /^\s*(?:async\s+)?([a-zA-Z0-9_$]+)\s*\([^)]*\)\s*(?::\s*[^\{]*)?\{/gm;
 async function processFilesInParallel(files, cwd, workerCount) {
   const __filename2 = fileURLToPath(import.meta.url);
   const __dirname2 = dirname2(__filename2);
@@ -9475,6 +9477,7 @@ async function scanUnusedExports(config) {
         }
         const content = readFileSync3(join3(cwd, file), "utf-8");
         totalContents.set(file, content);
+        const isService = file.endsWith(".service.ts") || file.endsWith(".service.tsx");
         const lines = content.split(`
 `);
         for (let i = 0;i < lines.length; i++) {
@@ -9498,6 +9501,20 @@ async function scanUnusedExports(config) {
               }
             }
           }
+          if (isService) {
+            classMethodRegex.lastIndex = 0;
+            while ((match2 = classMethodRegex.exec(line)) !== null) {
+              const name = match2[1];
+              if (name && !NEST_LIFECYCLE_METHODS.has(name) && !IGNORED_EXPORT_NAMES.has(name)) {
+                const existing = exportMap.get(file)?.find((e) => e.name === name);
+                if (!existing) {
+                  if (addExport(file, name, i + 1)) {
+                    allExportsCount++;
+                  }
+                }
+              }
+            }
+          }
         }
       } catch {}
     }
@@ -10203,13 +10220,11 @@ program2.action(async (options) => {
       if (result.unusedFiles) {
         result.unusedFiles.files = result.unusedFiles.files.filter((f) => matchesFilter(f.path));
         result.unusedFiles.total = result.unusedFiles.files.length;
-        result.unusedFiles.used = 0;
         result.unusedFiles.unused = result.unusedFiles.files.length;
       }
       if (result.unusedExports) {
         result.unusedExports.exports = result.unusedExports.exports.filter((e) => matchesFilter(e.file));
         result.unusedExports.total = result.unusedExports.exports.length;
-        result.unusedExports.used = 0;
         result.unusedExports.unused = result.unusedExports.exports.length;
       }
     }
@@ -10261,7 +10276,7 @@ program2.action(async (options) => {
       console.log("");
     }
     if (result.unusedExports && result.unusedExports.exports.length > 0) {
-      console.log(source_default.red.bold(`\uD83D\uDD17 Unused Named Exports:
+      console.log(source_default.red.bold(`\uD83D\uDD17 Unused Named Exports/Methods:
 `));
       for (const exp of result.unusedExports.exports) {
         console.log(source_default.red(`   ${exp.name}`));
@@ -10308,7 +10323,6 @@ program2.action(async (options) => {
                 fixedSomething = true;
               } else {
                 console.log(source_default.yellow(`   Skipped File Deletion (internally used): ${filePath}`));
-                console.log(source_default.dim(`      → This controller is imported in another file (e.g. app.module.ts).`));
                 const allMethodsToPrune = [];
                 for (const r of fileRoutes) {
                   for (const m of r.unusedMethods) {
@@ -10386,38 +10400,17 @@ program2.action(async (options) => {
         console.log("");
       }
       if (result.unusedExports && result.unusedExports.exports.length > 0) {
-        console.log(source_default.yellow.bold(`\uD83D\uDD27 Fixing unused exports (removing "export" keyword)...
-`));
-        const exportsByFile = new Map;
-        for (const exp of result.unusedExports.exports) {
-          if (!exportsByFile.has(exp.file)) {
-            exportsByFile.set(exp.file, []);
-          }
-          exportsByFile.get(exp.file).push(exp);
-        }
-        let fixedCount = 0;
-        for (const [file, exports] of exportsByFile.entries()) {
-          const sortedExports = exports.sort((a, b) => b.line - a.line);
-          for (const exp of sortedExports) {
-            const fullPath = join8(config.dir, exp.file);
-            if (!existsSync7(fullPath))
-              continue;
-            if (removeExportFromLine(config.dir, exp)) {
-              console.log(source_default.green(`   Fixed: ${exp.name} in ${exp.file}`));
-              fixedCount++;
-              fixedSomething = true;
-              const expIdx = result.unusedExports.exports.indexOf(exp);
-              if (expIdx !== -1) {
-                result.unusedExports.exports.splice(expIdx, 1);
-                result.unusedExports.unused--;
-              }
-            }
-          }
-        }
-        if (fixedCount > 0) {
-          console.log(source_default.green(`
-✅ Removed "export" from ${fixedCount} item(s).
+        fixedSomething = await fixUnusedExports(result, config) || fixedSomething;
+      }
+      if (fixedSomething) {
+        console.log(source_default.cyan.bold(`
+\uD83D\uDD04 Checking for cascading dead code (newly unused implementation)...`));
+        const secondPass = await scanUnusedExports(config);
+        if (secondPass.unused > 0) {
+          console.log(source_default.yellow(`   Found ${secondPass.unused} newly unused items/methods after pruning.
 `));
+          result.unusedExports = secondPass;
+          await fixUnusedExports(result, config);
         }
       }
       if (fixedSomething) {
@@ -10451,41 +10444,21 @@ program2.action(async (options) => {
     for (const key of sortedKeys) {
       const group = groupedRoutes.get(key);
       const typeLabel = group.type === "nextjs" ? "Next.js" : "NestJS";
-      const label = `${typeLabel} (${group.app})`;
       summary.push({
-        Category: label,
+        Category: `${typeLabel} (${group.app})`,
         Total: group.routes.length,
         Used: group.routes.filter((r) => r.used).length,
         Unused: group.routes.filter((r) => !r.used).length
       });
     }
-    if (summary.length === 0) {
+    if (summary.length === 0)
       summary.push({ Category: "API Routes", Total: result.total, Used: result.used, Unused: result.unused });
-    }
-    if (result.publicAssets) {
-      summary.push({
-        Category: "Public Assets",
-        Total: result.publicAssets.total,
-        Used: result.publicAssets.used,
-        Unused: result.publicAssets.unused
-      });
-    }
-    if (result.unusedFiles) {
-      summary.push({
-        Category: "Source Files",
-        Total: result.unusedFiles.total,
-        Used: result.unusedFiles.used,
-        Unused: result.unusedFiles.unused
-      });
-    }
-    if (result.unusedExports) {
-      summary.push({
-        Category: "Exported Items",
-        Total: result.unusedExports.total,
-        Used: result.unusedExports.used,
-        Unused: result.unusedExports.unused
-      });
-    }
+    if (result.publicAssets)
+      summary.push({ Category: "Public Assets", Total: result.publicAssets.total, Used: result.publicAssets.used, Unused: result.publicAssets.unused });
+    if (result.unusedFiles)
+      summary.push({ Category: "Source Files", Total: result.unusedFiles.total, Used: result.unusedFiles.used, Unused: result.unusedFiles.unused });
+    if (result.unusedExports)
+      summary.push({ Category: "Exported Items", Total: result.unusedExports.total, Used: result.unusedExports.used, Unused: result.unusedExports.unused });
     console.table(summary);
   } catch (_err) {
     console.error(source_default.red("Error scanning:"), _err);
@@ -10495,4 +10468,41 @@ program2.action(async (options) => {
   console.log(source_default.dim(`
 ⏱️  Completed in ${elapsed}s`));
 });
+async function fixUnusedExports(result, config) {
+  if (!result.unusedExports || result.unusedExports.exports.length === 0)
+    return false;
+  console.log(source_default.yellow.bold(`\uD83D\uDD27 Fixing unused exports/methods...
+`));
+  const exportsByFile = new Map;
+  for (const exp of result.unusedExports.exports) {
+    if (!exportsByFile.has(exp.file))
+      exportsByFile.set(exp.file, []);
+    exportsByFile.get(exp.file).push(exp);
+  }
+  let fixedCount = 0;
+  let fixedSomething = false;
+  for (const [file, exports] of exportsByFile.entries()) {
+    const sortedExports = exports.sort((a, b) => b.line - a.line);
+    for (const exp of sortedExports) {
+      const fullPath = join8(config.dir, exp.file);
+      if (!existsSync7(fullPath))
+        continue;
+      if (removeExportFromLine(config.dir, exp)) {
+        console.log(source_default.green(`   Fixed: ${exp.name} in ${exp.file}`));
+        fixedCount++;
+        fixedSomething = true;
+        const expIdx = result.unusedExports.exports.indexOf(exp);
+        if (expIdx !== -1) {
+          result.unusedExports.exports.splice(expIdx, 1);
+          result.unusedExports.unused--;
+        }
+      }
+    }
+  }
+  if (fixedCount > 0)
+    console.log(source_default.green(`
+✅ Cleaned up ${fixedCount} unused item(s).
+`));
+  return fixedSomething;
+}
 program2.parse();

package/dist/workers/file-processor.js

@@ -22,11 +22,11 @@ var IGNORED_EXPORT_NAMES = new Set([
   "POST",
   "PUT",
   "PATCH",
-  "DELETE",
-  "HEAD",
   "OPTIONS",
   "default"
 ]);
+var NEST_LIFECYCLE_METHODS = new Set(["constructor", "onModuleInit", "onApplicationBootstrap", "onModuleDestroy", "beforeApplicationShutdown", "onApplicationShutdown"]);
+var classMethodRegex = /^\s*(?:async\s+)?([a-zA-Z0-9_$]+)\s*\([^)]*\)\s*(?::\s*[^\{]*)?\{/gm;
 var inlineExportRegex = /^export\s+(?:async\s+)?(?:const|let|var|function|type|interface|enum|class)\s+([a-zA-Z0-9_$]+)/gm;
 var blockExportRegex = /^export\s*\{([^}]+)\}/gm;
 if (parentPort && workerData) {
@@ -40,6 +40,7 @@ if (parentPort && workerData) {
       contents.set(file, content);
       const lines = content.split(`
 `);
+      const isService = file.endsWith(".service.ts") || file.endsWith(".service.tsx");
       for (let i = 0;i < lines.length; i++) {
         const line = lines[i];
         inlineExportRegex.lastIndex = 0;
@@ -66,6 +67,20 @@ if (parentPort && workerData) {
             }
           }
         }
+        if (isService) {
+          classMethodRegex.lastIndex = 0;
+          while ((match = classMethodRegex.exec(line)) !== null) {
+            const name = match[1];
+            if (name && !NEST_LIFECYCLE_METHODS.has(name) && !IGNORED_EXPORT_NAMES.has(name)) {
+              const existing = exportMap.get(file)?.find((e) => e.name === name);
+              if (!existing) {
+                if (!exportMap.has(file))
+                  exportMap.set(file, []);
+                exportMap.get(file).push({ name, line: i + 1, file });
+              }
+            }
+          }
+        }
       }
       processedCount++;
       if (processedCount % 10 === 0) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pruny",
-  "version": "1.19.0",
+  "version": "1.20.0",
   "description": "Find and remove unused Next.js API routes & Nest.js Controllers",
   "type": "module",
   "files": [