pruny 1.44.2 → 1.44.4

package/dist/index.js

@@ -14964,7 +14964,6 @@ async function scanUnusedExports(config, routes = [], options = {}) {
                 console.log(`[DEBUG USE] ${exp.name} used internally in ${file} at line ${i + 1}: ${line.trim()}`);
               }
               usedInternally = true;
-              isUsed = true;
               break;
             }
           }
@@ -14985,6 +14984,10 @@ async function scanUnusedExports(config, routes = [], options = {}) {
         const absoluteOtherFileFixed = isAbsolute2(otherFile) ? otherFile : join4(scanCwd, otherFile);
         const hasIgnoreRanges = ignoreRanges.has(absoluteOtherFileFixed);
         const isGeneric = GENERIC_METHOD_NAMES.has(exp.name);
+        const hasSelfImport = new RegExp(`import.*\\b${escapeRegExp(exp.name)}\\b.*from`).test(content);
+        const hasSelfDecl = new RegExp(`(?:export\\s+)?(?:abstract\\s+)?(?:interface|class|enum)\\s+${escapeRegExp(exp.name)}\\b|` + `(?:export\\s+)?(?:async\\s+)?(?:function)\\s+${escapeRegExp(exp.name)}\\b|` + `(?:export\\s+)?(?:const|let|var|type)\\s+${escapeRegExp(exp.name)}\\s*[=<]`).test(content);
+        if (hasSelfDecl && !hasSelfImport)
+          continue;
         if (!hasIgnoreRanges && !isGeneric) {
           const jsxPattern = new RegExp(`<${exp.name}[\\s/>]`);
           if (jsxPattern.test(content)) {
@@ -15325,11 +15328,18 @@ function removeExportFromLine(rootDir, exp) {
     const content = readFileSync7(fullPath, "utf-8");
     const lines = content.split(`
 `);
-    const lineIndex = findDeclarationIndex(lines, exp.name, exp.line - 1);
+    let lineIndex = findDeclarationIndex(lines, exp.name, exp.line - 1);
+    if (lineIndex === -1) {
+      lineIndex = findDeclarationIndex(lines, exp.name, 0);
+    }
     if (!exp.usedInternally) {
+      if (lineIndex === -1) {
+        return removeFromBlockExport(lines, fullPath, exp.name, exp.line);
+      }
       const trueStartLine = findDeclarationStart(lines, lineIndex);
       const deletedLines = deleteDeclaration(lines, trueStartLine, exp.name);
       if (deletedLines > 0) {
+        removeFromBlockExport(lines, fullPath, exp.name, exp.line);
         const newContent = lines.join(`
 `);
         if (isFileEmpty(newContent)) {
@@ -15341,7 +15351,12 @@ function removeExportFromLine(rootDir, exp) {
       }
       return false;
     }
+    if (lineIndex === -1) {
+      return removeFromBlockExport(lines, fullPath, exp.name, exp.line);
+    }
     const originalLine = lines[lineIndex];
+    if (!originalLine)
+      return false;
     if (originalLine.trim().startsWith("export ")) {
       const newLine = originalLine.replace(/(\s*)export\s+/, "$1");
       lines[lineIndex] = newLine;
@@ -15349,12 +15364,31 @@ function removeExportFromLine(rootDir, exp) {
 `), "utf-8");
       return true;
     }
-    return false;
+    return removeFromBlockExport(lines, fullPath, exp.name, exp.line);
   } catch (err) {
     console.error(`Error fixing export in ${exp.file}:`, err);
     return false;
   }
 }
+function removeFromBlockExport(lines, fullPath, name, exportLine) {
+  const lineIdx = exportLine - 1;
+  const line = lines[lineIdx];
+  if (!line)
+    return false;
+  if (!/^export\s*\{/.test(line.trim()))
+    return false;
+  const escaped = name.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+  let newLine = line.replace(new RegExp(`\\b${escaped}\\b\\s*,\\s*`), "").replace(new RegExp(`,\\s*\\b${escaped}\\b`), "").replace(new RegExp(`\\b${escaped}\\b`), "");
+  newLine = newLine.replace(/\{\s*\}/, "{}").replace(/\{\s+/, "{ ").replace(/\s+\}/, " }");
+  if (newLine.trim() === "export {}" || newLine.trim() === "export { }") {
+    lines.splice(lineIdx, 1);
+  } else {
+    lines[lineIdx] = newLine;
+  }
+  writeFileSync(fullPath, lines.join(`
+`), "utf-8");
+  return true;
+}
 function findDeclarationIndex(lines, name, startLine = 0) {
   const methodRegex = new RegExp(`(?:public|private|protected|static|async|readonly)?\\s*(?:async)?\\s*${name}\\s*[(<]`);
   const declRegex = new RegExp(`(?:export\\s+)?(?:default\\s+)?(?:(?:abstract|async|static|readonly)\\s+)*(?:class|interface|type|enum|function|const|let|var)\\s+${name}\\b`);
@@ -15446,6 +15480,16 @@ function deleteDeclaration(lines, startLine, name) {
     if (inTemplateLiteral) {
       if (backtickCount % 2 !== 0) {
         inTemplateLiteral = false;
+        const lastTick = line.lastIndexOf("`");
+        const afterTick = sanitizeLine(line.slice(lastTick + 1));
+        const extraOpens = (afterTick.match(/{/g) || []).length;
+        const extraCloses = (afterTick.match(/}/g) || []).length;
+        braceCount += extraOpens - extraCloses;
+        if (foundBodyOpening && braceCount <= 0) {
+          endLine = i;
+          foundClosing = true;
+          break;
+        }
       }
       continue;
     }
@@ -18050,6 +18094,7 @@ Analyzing cascading impact...`));
     }
     const selectedList = options.cleanup ? options.cleanup.split(",").map((s) => s.trim()) : [selected];
     let fixedSomething = false;
+    let fixedCode = false;
     if (selectedList.includes("broken-links")) {
       if (result.brokenLinks && result.brokenLinks.total > 0) {
         console.log(source_default.yellow.bold(`
@@ -18189,6 +18234,7 @@ Analyzing cascading impact...`));
             actuallyDeleted.add(path2);
             console.log(source_default.red(`   Deleted: ${relative5(config.dir, path2)}`));
             fixedSomething = true;
+            fixedCode = true;
           } else {
             console.log(source_default.yellow(`      ⚠ Path not found, skipping: ${relative5(config.dir, path2)}`));
           }
@@ -18205,6 +18251,7 @@ Analyzing cascading impact...`));
               actuallyFixed.add(absPath);
               console.log(source_default.green(`      Fixed: Removed ${rem.name} from ${relative5(config.dir, absPath)} (${rem.label})`));
               fixedSomething = true;
+              fixedCode = true;
             }
           }
         }
@@ -18233,6 +18280,7 @@ Analyzing cascading impact...`));
             rmSync(fullPath, { force: true });
             console.log(source_default.red(`   Deleted: ${file.path}`));
             fixedSomething = true;
+            fixedCode = true;
           } catch (_err) {
             console.log(source_default.yellow(`   Failed to delete: ${file.path}`));
           }
@@ -18246,7 +18294,9 @@ Analyzing cascading impact...`));
     }
     if (selectedList.includes("exports")) {
       if (result.unusedExports && result.unusedExports.exports.length > 0) {
-        fixedSomething = await fixUnusedExports(result, config) || fixedSomething;
+        const exportsFixed = await fixUnusedExports(result, config);
+        fixedSomething = exportsFixed || fixedSomething;
+        fixedCode = exportsFixed || fixedCode;
       } else {
         console.log(source_default.green(`
 ✅ No unused exports found!`));
@@ -18278,6 +18328,7 @@ Analyzing cascading impact...`));
               console.log(source_default.green(`   Fixed: ${method.name} in ${file}`));
               fixedCount++;
               fixedSomething = true;
+              fixedCode = true;
             }
           }
         }
@@ -18293,7 +18344,7 @@ Analyzing cascading impact...`));
 ✅ No unused services found!`));
       }
     }
-    if (fixedSomething) {
+    if (fixedCode) {
       console.log(source_default.cyan.bold(`
 \uD83D\uDD04 Checking for cascading dead code (newly unused implementation)...`));
       const secondPass = await scanUnusedExports(config);
@@ -18365,6 +18416,8 @@ Analyzing cascading impact...`));
       result.used = result.routes.filter((r) => r.used).length;
       result.total = result.routes.length;
     }
+    if (options.cleanup)
+      return "exit";
     console.log("");
     continue;
   }

package/package.json

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