technical-debt-radar 1.12.0 → 1.13.0

package/dist/index.js

@@ -161,6 +161,7 @@ var require_credits = __commonJS({
     (function(AiOperation2) {
       AiOperation2["FIX"] = "fix";
       AiOperation2["FIX_GROUPED"] = "fix-grouped";
+      AiOperation2["FIX_MULTI"] = "fix-multi";
       AiOperation2["SCAN_SUMMARY"] = "scan-summary";
       AiOperation2["PR_COMMENT"] = "pr-comment";
       AiOperation2["CROSS_FILE"] = "cross-file";
@@ -168,6 +169,7 @@ var require_credits = __commonJS({
     exports2.CREDIT_COSTS = {
       [AiOperation.FIX]: 1,
       [AiOperation.FIX_GROUPED]: 1,
+      [AiOperation.FIX_MULTI]: 3,
       [AiOperation.SCAN_SUMMARY]: 1,
       [AiOperation.PR_COMMENT]: 3,
       [AiOperation.CROSS_FILE]: 5
@@ -20737,6 +20739,12 @@ var RadarApiClient = class _RadarApiClient {
       body: JSON.stringify(data)
     });
   }
+  async requestMultiFix(data) {
+    return this.fetch("/cli/ai/fix-multi", {
+      method: "POST",
+      body: JSON.stringify(data)
+    });
+  }
 };
 
 // src/commands/scan.ts
@@ -22045,6 +22053,102 @@ function groupBy(items, key) {
   }
   return result;
 }
+async function handleMultiFileFix(client, violation, violationFile, projectRoot, framework, architecture, filesToChange, options) {
+  console.log(import_chalk4.default.magenta.bold("\n  MULTI-FILE FIX"));
+  console.log(import_chalk4.default.dim("  Gathering file contents..."));
+  const files = {};
+  const allPaths = [violationFile, ...filesToChange.map((f) => resolveViolationPath(f, projectRoot))];
+  for (const fp of allPaths) {
+    try {
+      files[fp] = await fs5.readFile(fp, "utf-8");
+    } catch {
+      console.log(import_chalk4.default.yellow(`  Could not read: ${fp} \u2014 skipping`));
+    }
+  }
+  if (Object.keys(files).length < 2) {
+    console.log(import_chalk4.default.yellow("  Not enough files found for multi-file fix."));
+    return;
+  }
+  console.log(import_chalk4.default.dim(`  Sending ${Object.keys(files).length} files to Radar for analysis...`));
+  const result = await client.requestMultiFix({
+    violation: {
+      file: violation.file,
+      line: violation.line,
+      ruleId: violation.ruleId,
+      message: violation.message,
+      severity: violation.severity,
+      category: violation.category
+    },
+    files,
+    framework,
+    architecture
+  });
+  console.log("");
+  if (result.explanation) {
+    console.log(import_chalk4.default.white.bold("  WHY:"));
+    console.log(import_chalk4.default.white(`  ${result.explanation}`));
+  }
+  if (result.risk) {
+    console.log(import_chalk4.default.yellow(`  RISK: ${result.risk}`));
+  }
+  if (result.pattern) {
+    console.log(import_chalk4.default.cyan(`  PATTERN: ${result.pattern}`));
+  }
+  console.log("");
+  for (let i = 0; i < result.changes.length; i++) {
+    const change = result.changes[i];
+    const rel = path5.relative(projectRoot, resolveViolationPath(change.filePath, projectRoot));
+    console.log(import_chalk4.default.bold(`  FILE ${i + 1}: ${rel}`));
+    console.log(import_chalk4.default.dim(`  ${change.description}`));
+    const beforeLines = change.before.split("\n");
+    const afterLines = change.after.split("\n");
+    let diffShown = 0;
+    for (let j = 0; j < Math.max(beforeLines.length, afterLines.length) && diffShown < 10; j++) {
+      if (beforeLines[j] !== afterLines[j]) {
+        if (beforeLines[j]) console.log(import_chalk4.default.red(`  - ${beforeLines[j]}`));
+        if (afterLines[j]) console.log(import_chalk4.default.green(`  + ${afterLines[j]}`));
+        diffShown++;
+      }
+    }
+    if (diffShown >= 10) console.log(import_chalk4.default.dim("  ... (more changes)"));
+    console.log("");
+  }
+  console.log(import_chalk4.default.cyan(`  ${result.creditsUsed} AI credits used (${result.creditsRemaining} remaining)`));
+  if (options.dryRun) {
+    console.log(import_chalk4.default.dim("  [dry-run] Would apply all changes"));
+    return;
+  }
+  const originals = {};
+  for (const change of result.changes) {
+    const fp = resolveViolationPath(change.filePath, projectRoot);
+    try {
+      originals[fp] = await fs5.readFile(fp, "utf-8");
+    } catch {
+      originals[fp] = "";
+    }
+  }
+  for (const change of result.changes) {
+    const fp = resolveViolationPath(change.filePath, projectRoot);
+    fsSync2.writeFileSync(fp, change.after, "utf-8");
+  }
+  console.log(import_chalk4.default.green(`  Applied changes to ${result.changes.length} files`));
+  if (options.safe) {
+    const testCmd = detectTestCommand(projectRoot);
+    if (testCmd) {
+      console.log(import_chalk4.default.dim(`  Running tests: ${testCmd}...`));
+      const passed = runTests(projectRoot, testCmd);
+      if (passed) {
+        console.log(import_chalk4.default.green("  \u2705 Tests passed"));
+      } else {
+        console.log(import_chalk4.default.red("  \u274C Tests failed \u2014 reverting all changes"));
+        for (const [fp, content] of Object.entries(originals)) {
+          fsSync2.writeFileSync(fp, content, "utf-8");
+        }
+        return;
+      }
+    }
+  }
+}
 async function fixCommand(targetPath, options) {
   const client = RadarApiClient.fromConfigOrEnv();
   if (!client) {
@@ -22195,13 +22299,29 @@ ${relativePath} (${fileViolations.length} violations)
         totalCreditsUsed += fix.creditsUsed;
         creditsRemaining = fix.creditsRemaining;
         fixCount++;
+        if (fix.explanation) {
+          console.log(import_chalk4.default.white.bold("  WHY THIS IS A VIOLATION:"));
+          console.log(import_chalk4.default.white(`  ${fix.explanation}`));
+        }
+        if (fix.risk) {
+          console.log(import_chalk4.default.yellow(`  RISK: ${fix.risk}`));
+        }
+        if (fix.pattern) {
+          console.log(import_chalk4.default.cyan(`  PATTERN: ${fix.pattern}`));
+        }
+        if (fix.multiFile && fix.filesToChange && fix.filesToChange.length > 0) {
+          console.log(import_chalk4.default.magenta(`  MULTI-FILE: Also needs changes in ${fix.filesToChange.length} other file(s):`));
+          for (const f of fix.filesToChange) {
+            console.log(import_chalk4.default.magenta(`    \u2192 ${f}`));
+          }
+        }
+        console.log("");
         console.log(import_chalk4.default.red("  BEFORE:"));
         fix.originalCode.split("\n").forEach((l) => console.log(import_chalk4.default.red(`  - ${l}`)));
         console.log("");
         console.log(import_chalk4.default.green("  AFTER:"));
         fix.fixedCode.split("\n").forEach((l) => console.log(import_chalk4.default.green(`  + ${l}`)));
         console.log("");
-        console.log(import_chalk4.default.dim(`  ${fix.explanation}`));
         console.log(import_chalk4.default.dim(`  Confidence: ${fix.confidence}`));
         console.log(import_chalk4.default.cyan(`  ${fix.creditsUsed} AI credit used (${fix.creditsRemaining} remaining)`));
         console.log("");
@@ -22213,17 +22333,19 @@ ${relativePath} (${fileViolations.length} violations)
           shouldApply = true;
           if (!options.auto) console.log(import_chalk4.default.green("  Auto-applying (all)..."));
         } else {
+          const choices = [
+            { name: "Yes - apply this fix", value: "yes" },
+            ...fix.multiFile && fix.filesToChange?.length ? [{ name: `Multi-file fix (${fix.filesToChange.length + 1} files, 3 credits)`, value: "multi" }] : [],
+            { name: "No - skip", value: "no" },
+            { name: "All - apply all remaining fixes", value: "all" },
+            { name: "Quit - stop fixing", value: "quit" }
+          ];
           const { action } = await import_inquirer2.default.prompt([
             {
               type: "list",
               name: "action",
               message: `Apply this fix? (${lineViolations.length} violation${lineViolations.length > 1 ? "s" : ""})`,
-              choices: [
-                { name: "Yes - apply this fix", value: "yes" },
-                { name: "No - skip", value: "no" },
-                { name: "All - apply all remaining fixes", value: "all" },
-                { name: "Quit - stop fixing", value: "quit" }
-              ]
+              choices
             }
           ]);
           if (action === "quit") {
@@ -22239,6 +22361,22 @@ ${relativePath} (${fileViolations.length} violations)
             totalSkipped += lineViolations.length;
             continue;
           }
+          if (action === "multi") {
+            await handleMultiFileFix(
+              client,
+              lineViolations[0],
+              absPath,
+              projectRoot,
+              framework,
+              scanResult.config?.architecture,
+              fix.filesToChange ?? [],
+              options
+            );
+            totalFixed += lineViolations.length;
+            fileContent = await fs5.readFile(absPath, "utf-8");
+            lines = fileContent.split("\n");
+            continue;
+          }
         }
         if (shouldApply) {
           const originalContent = options.safe ? fileContent : null;

package/package.json

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