technical-debt-radar 1.11.0 → 1.12.0

package/dist/index.js

@@ -21949,6 +21949,7 @@ async function validateCommand(options) {
 var fs5 = __toESM(require("fs/promises"));
 var fsSync2 = __toESM(require("fs"));
 var path5 = __toESM(require("path"));
+var import_child_process2 = require("child_process");
 var import_chalk4 = __toESM(require("chalk"));
 var import_inquirer2 = __toESM(require("inquirer"));
 var import_analyzers3 = __toESM(require_dist3());
@@ -22003,6 +22004,38 @@ async function runScan(targetPath, options) {
     policy
   };
 }
+async function quickScanFile(targetPath, filePath, options) {
+  try {
+    const result = await runScan(targetPath, { ...options, file: filePath });
+    return result.violations;
+  } catch {
+    return [];
+  }
+}
+function resolveViolationPath(vFile, projectRoot) {
+  if (path5.isAbsolute(vFile)) return vFile;
+  return path5.resolve(projectRoot, vFile);
+}
+function detectTestCommand(projectRoot) {
+  try {
+    const pkg = JSON.parse(fsSync2.readFileSync(path5.join(projectRoot, "package.json"), "utf-8"));
+    const scripts = pkg.scripts ?? {};
+    if (scripts.test && scripts.test !== 'echo "Error: no test specified" && exit 1') return "npm test";
+    if (scripts["test:unit"]) return "npm run test:unit";
+    if (scripts.jest) return "npm run jest";
+    return null;
+  } catch {
+    return null;
+  }
+}
+function runTests(projectRoot, testCmd) {
+  try {
+    (0, import_child_process2.execSync)(testCmd, { cwd: projectRoot, stdio: "pipe", timeout: 12e4 });
+    return true;
+  } catch {
+    return false;
+  }
+}
 function groupBy(items, key) {
   const result = {};
   for (const item of items) {
@@ -22034,6 +22067,16 @@ async function fixCommand(targetPath, options) {
     console.error(import_chalk4.default.red("\u274C Invalid or expired token. Run: radar login"));
     process.exit(1);
   }
+  const projectRoot = path5.resolve(targetPath);
+  let testCmd = null;
+  if (options.safe) {
+    testCmd = detectTestCommand(projectRoot);
+    if (!testCmd) {
+      console.log(import_chalk4.default.yellow("\u26A0\uFE0F  No test command found in package.json. --safe will skip test verification."));
+    } else {
+      console.log(import_chalk4.default.dim(`  Test command: ${testCmd}`));
+    }
+  }
   console.log(import_chalk4.default.blue("Scanning for violations..."));
   const scanResult = await runScan(targetPath, {
     config: options.config,
@@ -22054,10 +22097,11 @@ async function fixCommand(targetPath, options) {
   }
   console.log(import_chalk4.default.yellow(`Found ${violations.length} violations. Generating AI fixes...
 `));
-  const byFile = groupBy(violations, (v) => v.file);
+  const byFile = groupBy(violations, (v) => resolveViolationPath(v.file, projectRoot));
   let totalFixed = 0;
   let totalSkipped = 0;
   let totalFailed = 0;
+  let totalReverted = 0;
   let totalCreditsUsed = 0;
   let creditsRemaining = -1;
   let fixCount = 0;
@@ -22067,15 +22111,16 @@ async function fixCommand(targetPath, options) {
   const orm = scanResult.config?.stack?.orm || "Prisma";
   for (const [filePath, fileViolations] of Object.entries(byFile)) {
     if (quit) break;
-    const relativePath = path5.relative(path5.resolve(targetPath), filePath);
+    const absPath = path5.resolve(filePath);
+    const relativePath = path5.relative(projectRoot, absPath);
     console.log(import_chalk4.default.bold(`
 ${relativePath} (${fileViolations.length} violations)
 `));
     let fileContent;
     try {
-      fileContent = await fs5.readFile(filePath, "utf-8");
+      fileContent = await fs5.readFile(absPath, "utf-8");
     } catch {
-      console.log(import_chalk4.default.red(`  Could not read file: ${filePath}`));
+      console.log(import_chalk4.default.red(`  Could not read file: ${absPath}`));
       totalFailed += fileViolations.length;
       continue;
     }
@@ -22164,9 +22209,9 @@ ${relativePath} (${fileViolations.length} violations)
         if (options.dryRun) {
           console.log(import_chalk4.default.dim("  [dry-run] Would apply this fix"));
           totalSkipped += lineViolations.length;
-        } else if (applyAll || options.auto && fix.confidence === "high") {
+        } else if (applyAll || options.auto) {
           shouldApply = true;
-          console.log(import_chalk4.default.green("  Auto-applying..."));
+          if (!options.auto) console.log(import_chalk4.default.green("  Auto-applying (all)..."));
         } else {
           const { action } = await import_inquirer2.default.prompt([
             {
@@ -22196,24 +22241,52 @@ ${relativePath} (${fileViolations.length} violations)
           }
         }
         if (shouldApply) {
-          applyFix(filePath, fix);
-          totalFixed += lineViolations.length;
-          console.log(import_chalk4.default.green(`  Fixed! (${lineViolations.length} violation${lineViolations.length > 1 ? "s" : ""})`));
-          fileContent = await fs5.readFile(filePath, "utf-8");
+          const originalContent = options.safe ? fileContent : null;
+          applyFix(absPath, fix);
+          console.log(import_chalk4.default.green(`  Applied fix (${lineViolations.length} violation${lineViolations.length > 1 ? "s" : ""})`));
+          const remaining = await quickScanFile(targetPath, absPath, {
+            config: options.config,
+            rules: options.rules
+          });
+          const stillPresent = remaining.some(
+            (rv) => lineViolations.some(
+              (lv) => rv.ruleId === lv.ruleId && rv.file === lv.file && Math.abs(rv.line - lv.line) <= 3
+            )
+          );
+          if (stillPresent) {
+            console.log(import_chalk4.default.yellow("  \u26A0\uFE0F  Fix applied but violation persists \u2014 manual review needed"));
+          } else {
+            console.log(import_chalk4.default.green("  \u2705 Violation resolved"));
+          }
+          if (options.safe && testCmd) {
+            console.log(import_chalk4.default.dim(`  Running tests: ${testCmd}...`));
+            const testsPassed = runTests(projectRoot, testCmd);
+            if (testsPassed) {
+              console.log(import_chalk4.default.green("  \u2705 Tests passed"));
+              totalFixed += lineViolations.length;
+            } else {
+              console.log(import_chalk4.default.red("  \u274C Tests failed \u2014 reverting fix"));
+              fsSync2.writeFileSync(absPath, originalContent, "utf-8");
+              totalReverted += lineViolations.length;
+            }
+          } else {
+            totalFixed += lineViolations.length;
+          }
+          fileContent = await fs5.readFile(absPath, "utf-8");
           lines = fileContent.split("\n");
         }
       } catch (error) {
         const message = error instanceof Error ? error.message : String(error);
         if (message.includes("403") && message.includes("credits exhausted")) {
-          console.log(import_chalk4.default.red(`  \u274C AI credits exhausted. Upgrade for more: radar upgrade`));
+          console.log(import_chalk4.default.red("  \u274C AI credits exhausted. Upgrade for more: radar upgrade"));
           quit = true;
           break;
         } else if (message.includes("403") && message.includes("Solo")) {
-          console.log(import_chalk4.default.red(`  \u274C radar fix requires Solo plan or higher. Upgrade: radar upgrade`));
+          console.log(import_chalk4.default.red("  \u274C radar fix requires Solo plan or higher. Upgrade: radar upgrade"));
           quit = true;
           break;
         } else if (message.includes("401")) {
-          console.log(import_chalk4.default.red(`  \u274C Authentication failed. Run: radar login`));
+          console.log(import_chalk4.default.red("  \u274C Authentication failed. Run: radar login"));
           quit = true;
           break;
         } else {
@@ -22227,16 +22300,17 @@ ${relativePath} (${fileViolations.length} violations)
   console.log(import_chalk4.default.bold("\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550"));
   console.log(import_chalk4.default.bold("  FIX SUMMARY"));
   console.log(import_chalk4.default.bold("\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550\u2550"));
-  console.log(import_chalk4.default.green(`  Fixed:   ${totalFixed}`));
-  console.log(import_chalk4.default.yellow(`  Skipped: ${totalSkipped}`));
-  if (totalFailed > 0) console.log(import_chalk4.default.red(`  Failed:  ${totalFailed}`));
+  console.log(import_chalk4.default.green(`  Fixed:    ${totalFixed}`));
+  console.log(import_chalk4.default.yellow(`  Skipped:  ${totalSkipped}`));
+  if (totalReverted > 0) console.log(import_chalk4.default.red(`  Reverted: ${totalReverted} (tests failed)`));
+  if (totalFailed > 0) console.log(import_chalk4.default.red(`  Failed:   ${totalFailed}`));
   console.log(import_chalk4.default.cyan(`  AI credits used: ${totalCreditsUsed}`));
   if (creditsRemaining >= 0) {
     console.log(import_chalk4.default.cyan(`  Credits remaining: ${creditsRemaining}`));
   }
   console.log("");
   if (totalFixed > 0 && !options.dryRun) {
-    console.log(import_chalk4.default.blue("Re-scanning to verify fixes...\n"));
+    console.log(import_chalk4.default.blue("Re-scanning to verify all fixes...\n"));
     const newResult = await runScan(targetPath, {
       config: options.config,
       rules: options.rules,
@@ -22627,7 +22701,7 @@ program.command("check <file>").description("Check a single file against policy"
 program.command("init").description("Generate radar.yml + rules.yml from project structure").option("--path <dir>", "Target directory", ".").option("--architecture <pattern>", "Force architecture: ddd, hexagonal, clean, layered, mvc, event-driven").option("--regenerate-rules", "Regenerate rules.yml from existing radar.yml", false).option("--no-ai", "Skip AI refinement (deterministic only)").action(async (options) => {
   await initCommand(options);
 });
-program.command("fix <path>").description("AI-powered fix: generates code diffs, applies with confirmation").option("-c, --config <path>", "Path to radar.yml", "./radar.yml").option("-r, --rules <path>", "Path to rules.yml (default: ./rules.yml)").option("--severity <level>", "Only include: critical, all", "all").option("--dry-run", "Show fixes without applying", false).option("--auto", "Apply all high-confidence fixes without asking", false).option("--file <file>", "Fix single file only").option("--max-fixes <count>", "Max number of AI fixes to generate (0 = unlimited)", "0").action(async (targetPath, options) => {
+program.command("fix <path>").description("AI-powered fix: generates code diffs, applies with confirmation").option("-c, --config <path>", "Path to radar.yml", "./radar.yml").option("-r, --rules <path>", "Path to rules.yml (default: ./rules.yml)").option("--severity <level>", "Only include: critical, all", "all").option("--dry-run", "Show fixes without applying", false).option("--auto", "Apply all fixes automatically without asking", false).option("--safe", "Run tests after each fix, revert on failure", false).option("--file <file>", "Fix single file only").option("--max-fixes <count>", "Max number of AI fixes to generate (0 = unlimited)", "0").action(async (targetPath, options) => {
   await fixCommand(targetPath, {
     ...options,
     maxFixes: parseInt(options.maxFixes, 10)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "technical-debt-radar",
-  "version": "1.11.0",
+  "version": "1.12.0",
   "description": "Stop Node.js production crashes before merge. 47 detection patterns across 5 categories.",
   "bin": {
     "radar": "dist/index.js",