pruny 1.13.0 → 1.15.0

package/dist/index.js

@@ -9621,13 +9621,22 @@ function extractRoutePath(filePath) {
 }
 function extractExportedMethods(content) {
   const methods = [];
+  const methodLines = {};
+  const lines = content.split(`
+`);
   let match2;
+  EXPORTED_METHOD_PATTERN.lastIndex = 0;
   while ((match2 = EXPORTED_METHOD_PATTERN.exec(content)) !== null) {
     if (match2[1]) {
-      methods.push(match2[1]);
+      const methodName = match2[1];
+      methods.push(methodName);
+      const pos = match2.index;
+      const lineNum = content.substring(0, pos).split(`
+`).length;
+      methodLines[methodName] = lineNum;
     }
   }
-  return methods;
+  return { methods, methodLines };
 }
 function extractNestRoutes(filePath, content, globalPrefix = "api") {
   const controllerMatch = content.match(NEST_CONTROLLER_PATTERN);
@@ -9640,12 +9649,16 @@ function extractNestRoutes(filePath, content, globalPrefix = "api") {
   while ((methodMatch = NEST_METHOD_PATTERN.exec(content)) !== null) {
     const methodType = methodMatch[1].toUpperCase();
     const methodPath = methodMatch[2] || "";
+    const pos = methodMatch.index;
+    const lineNum = content.substring(0, pos).split(`
+`).length;
     const fullPath = `/${globalPrefix}/${controllerPath}/${methodPath}`.replace(/\/+/g, "/").replace(/\/$/, "");
     const existing = routes.find((r) => r.path === fullPath);
     if (existing) {
       if (!existing.methods.includes(methodType)) {
         existing.methods.push(methodType);
         existing.unusedMethods.push(methodType);
+        existing.methodLines[methodType] = lineNum;
       }
     } else {
       routes.push({
@@ -9655,7 +9668,8 @@ function extractNestRoutes(filePath, content, globalPrefix = "api") {
         used: false,
         references: [],
         methods: [methodType],
-        unusedMethods: [methodType]
+        unusedMethods: [methodType],
+        methodLines: { [methodType]: lineNum }
       });
     }
   }
@@ -9733,7 +9747,7 @@ async function scan(config) {
   });
   const nextRoutes = nextFiles.map((file) => {
     const content = readFileSync4(join4(cwd, file), "utf-8");
-    const methods = extractExportedMethods(content);
+    const { methods, methodLines } = extractExportedMethods(content);
     return {
       type: "nextjs",
       path: extractRoutePath(file),
@@ -9741,7 +9755,8 @@ async function scan(config) {
       used: false,
       references: [],
       methods,
-      unusedMethods: [...methods]
+      unusedMethods: [...methods],
+      methodLines
     };
   });
   const nestPatterns = ["**/*.controller.ts"];
@@ -9953,7 +9968,7 @@ function findConfigFile(dir) {
 }
 
 // src/fixer.ts
-import { readFileSync as readFileSync6, writeFileSync, unlinkSync } from "node:fs";
+import { readFileSync as readFileSync6, writeFileSync, unlinkSync, existsSync as existsSync5 } from "node:fs";
 import { join as join6 } from "node:path";
 function removeExportFromLine(rootDir, exp) {
   const fullPath = join6(rootDir, exp.file);
@@ -10039,6 +10054,32 @@ function isFileEmpty(content) {
   }
   return true;
 }
+function removeMethodFromRoute(rootDir, filePath, methodName, lineNum) {
+  const fullPath = join6(rootDir, filePath);
+  if (!existsSync5(fullPath))
+    return false;
+  try {
+    const content = readFileSync6(fullPath, "utf-8");
+    const lines = content.split(`
+`);
+    const lineIndex = lineNum - 1;
+    const deletedLines = deleteDeclaration(lines, lineIndex);
+    if (deletedLines > 0) {
+      const newContent = lines.join(`
+`);
+      if (isFileEmpty(newContent)) {
+        unlinkSync(fullPath);
+      } else {
+        writeFileSync(fullPath, newContent, "utf-8");
+      }
+      return true;
+    }
+    return false;
+  } catch (err) {
+    console.error(`Error removing method ${methodName} in ${filePath}:`, err);
+    return false;
+  }
+}
 
 // src/init.ts
 import { writeFileSync as writeFileSync2, existsSync as existsSync6 } from "node:fs";
@@ -10081,21 +10122,21 @@ program2.action(async (options) => {
 `));
   try {
     let result = await scan(config);
+    const getAppName = (filePath) => {
+      if (filePath.startsWith("apps/"))
+        return filePath.split("/").slice(0, 2).join("/");
+      if (filePath.startsWith("packages/"))
+        return filePath.split("/").slice(0, 2).join("/");
+      return "Root";
+    };
     if (options.filter) {
       const filter2 = options.filter.toLowerCase();
       console.log(source_default.blue(`
 \uD83D\uDD0D Filtering results by "${filter2}"...
 `));
-      const getAppName2 = (filePath) => {
-        if (filePath.startsWith("apps/"))
-          return filePath.split("/").slice(0, 2).join("/");
-        if (filePath.startsWith("packages/"))
-          return filePath.split("/").slice(0, 2).join("/");
-        return "Root";
-      };
       const matchesFilter = (path2) => {
         const lowerPath = path2.toLowerCase();
-        const appName = getAppName2(path2).toLowerCase();
+        const appName = getAppName(path2).toLowerCase();
         if (appName.includes(filter2))
           return true;
         const segments = lowerPath.split("/");
@@ -10186,24 +10227,76 @@ program2.action(async (options) => {
     }
     if (unusedRoutes.length === 0 && partiallyUnusedRoutes.length === 0 && (!result.publicAssets || result.publicAssets.unused === 0)) {
       console.log(source_default.green(`✅ Everything is used! Clean as a whistle.
+`));
+    }
+    if (options.fix) {
+      if (unusedRoutes.length > 0) {
+        console.log(source_default.yellow.bold(`\uD83D\uDDD1️  Deleting unused routes...
+`));
+        for (const route of unusedRoutes) {
+          const routeDir = dirname3(join8(config.dir, route.filePath));
+          try {
+            rmSync(routeDir, { recursive: true, force: true });
+            console.log(source_default.red(`   Deleted: ${route.filePath}`));
+          } catch (_err) {
+            console.log(source_default.yellow(`   Failed to delete: ${route.filePath}`));
+          }
+        }
+      }
+      const partiallyRoutes = result.routes.filter((r) => r.used && r.unusedMethods.length > 0);
+      if (partiallyRoutes.length > 0) {
+        console.log(source_default.yellow.bold(`\uD83D\uDD27 Fixing partially unused routes...
+`));
+        for (const route of partiallyRoutes) {
+          const sortedMethods = [...route.unusedMethods].filter((m) => route.methodLines[m] !== undefined).sort((a, b) => route.methodLines[b] - route.methodLines[a]);
+          for (const method of sortedMethods) {
+            const lineNum = route.methodLines[method];
+            if (removeMethodFromRoute(config.dir, route.filePath, method, lineNum)) {
+              console.log(source_default.green(`   Fixed: Removed ${method} from ${route.path}`));
+            }
+          }
+        }
+        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) {
+            if (removeExportFromLine(config.dir, exp)) {
+              console.log(source_default.green(`   Fixed: ${exp.name} in ${exp.file}`));
+              fixedCount++;
+            }
+          }
+        }
+        if (fixedCount > 0) {
+          console.log(source_default.green(`
+✅ Removed "export" from ${fixedCount} item(s).
+`));
+        }
+      }
+    } else if (unusedRoutes.length > 0 || result.unusedExports && result.unusedExports.exports.length > 0) {
+      console.log(source_default.dim(`\uD83D\uDCA1 Run with --fix to automatically clean up unused routes and exports.
 `));
     }
     console.log(source_default.bold(`\uD83D\uDCCA Summary Report
 `));
     const summary = [];
-    const getAppName = (filePath) => {
-      if (filePath.startsWith("apps/"))
-        return filePath.split("/").slice(0, 2).join("/");
-      if (filePath.startsWith("packages/"))
-        return filePath.split("/").slice(0, 2).join("/");
-      return "Root";
-    };
     const groupedRoutes = new Map;
     for (const route of result.routes) {
-      const appName = getAppName(route.filePath);
-      const key = `${appName}::${route.type}`;
+      const keyAppName = getAppName(route.filePath);
+      const key = `${keyAppName}::${route.type}`;
       if (!groupedRoutes.has(key)) {
-        groupedRoutes.set(key, { type: route.type, app: appName, routes: [] });
+        groupedRoutes.set(key, { type: route.type, app: keyAppName, routes: [] });
       }
       groupedRoutes.get(key).routes.push(route);
     }
@@ -10253,70 +10346,6 @@ program2.action(async (options) => {
       });
     }
     console.table(summary);
-    console.log("");
-    if (options.verbose) {
-      const used = result.routes.filter((r) => r.used);
-      if (used.length > 0) {
-        console.log(source_default.green.bold(`✅ Used routes (References):
-`));
-        for (const route of used) {
-          console.log(source_default.green(`   ${route.path}`));
-          if (route.references.length > 0) {
-            for (const ref of route.references.slice(0, 3)) {
-              console.log(source_default.dim(`      ← ${ref}`));
-            }
-            if (route.references.length > 3) {
-              console.log(source_default.dim(`      ... and ${route.references.length - 3} more`));
-            }
-          }
-        }
-        console.log("");
-      }
-    }
-    if (options.fix) {
-      if (unusedRoutes.length > 0) {
-        console.log(source_default.yellow.bold(`\uD83D\uDDD1️  Deleting unused routes...
-`));
-        for (const route of unusedRoutes) {
-          const routeDir = dirname3(join8(config.dir, route.filePath));
-          try {
-            rmSync(routeDir, { recursive: true, force: true });
-            console.log(source_default.red(`   Deleted: ${route.filePath}`));
-          } catch (_err) {
-            console.log(source_default.yellow(`   Failed to delete: ${route.filePath}`));
-          }
-        }
-      }
-      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) {
-            if (removeExportFromLine(config.dir, exp)) {
-              console.log(source_default.green(`   Fixed: ${exp.name} in ${exp.file}`));
-              fixedCount++;
-            }
-          }
-        }
-        if (fixedCount > 0) {
-          console.log(source_default.green(`
-✅ Removed "export" from ${fixedCount} item(s).
-`));
-        }
-      }
-    } else if (unusedRoutes.length > 0 || result.unusedExports && result.unusedExports.exports.length > 0) {
-      console.log(source_default.dim(`\uD83D\uDCA1 Run with --fix to automatically clean up unused routes and exports.
-`));
-    }
   } catch (_err) {
     console.error(source_default.red("Error scanning:"), _err);
     process.exit(1);

package/package.json

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