@cencori/scan 0.4.3 → 0.4.5

package/dist/cli.mjs

@@ -4,7 +4,7 @@
 import { program } from "commander";
 import chalk from "chalk";
 import ora from "ora";
-import { confirm, password } from "@inquirer/prompts";
+import { confirm, password, select } from "@inquirer/prompts";
 
 // src/scanner/index.ts
 import * as fs from "fs";
@@ -883,24 +883,6 @@ Generate a secure fix.`
   }
   return results;
 }
-async function applyFixes(fixes, fileContents) {
-  for (const fix of fixes) {
-    if (fix.fixedCode === fix.originalCode) {
-      continue;
-    }
-    const content = fileContents.get(fix.issue.file);
-    if (!content) {
-      continue;
-    }
-    const newContent = content.replace(fix.originalCode, fix.fixedCode);
-    if (newContent !== content) {
-      const filePath = path2.resolve(fix.issue.file);
-      fs2.writeFileSync(filePath, newContent, "utf-8");
-      fix.applied = true;
-    }
-  }
-  return fixes;
-}
 
 // src/telemetry.ts
 var TELEMETRY_URL = "https://api.cencori.com/v1/telemetry/scan";
@@ -1153,7 +1135,7 @@ No commits found in the specified period.
 // src/cli.ts
 import * as fs3 from "fs";
 import * as path3 from "path";
-var VERSION = "0.4.2";
+var VERSION = "0.4.5";
 var scoreStyles = {
   A: { color: chalk.green },
   B: { color: chalk.blue },
@@ -1388,23 +1370,135 @@ async function handleAutoFix(result, targetPath) {
       fileContents
     );
     fixSpinner.succeed(`Generated ${fixes.length} fixes`);
-    const applySpinner = ora({
-      text: "Applying fixes...",
-      color: "cyan"
-    }).start();
-    const appliedFixes = await applyFixes(fixes, fileContents);
-    const appliedCount = appliedFixes.filter((f) => f.applied).length;
-    applySpinner.succeed(`Applied ${appliedCount}/${fixes.length} fixes`);
     console.log();
-    console.log(`  ${chalk.bold("Applied fixes:")}`);
-    for (const fix of appliedFixes.filter((f) => f.applied)) {
-      console.log(chalk.green(`    \u2714 ${fix.issue.file}:${fix.issue.line}`));
-      console.log(chalk.gray(`      ${fix.explanation}`));
+    const acceptedFixes = [];
+    const skippedFixes = [];
+    let applyAll = false;
+    let skipRest = false;
+    for (let i = 0; i < fixes.length; i++) {
+      const fix = fixes[i];
+      if (skipRest) {
+        skippedFixes.push(fix);
+        continue;
+      }
+      if (applyAll) {
+        acceptedFixes.push(fix);
+        continue;
+      }
+      console.log(chalk.cyan(`  \u2500\u2500\u2500 Fix ${i + 1}/${fixes.length}: ${fix.issue.file}:${fix.issue.line} \u2500\u2500\u2500`));
+      console.log(chalk.gray(`  Issue: ${fix.issue.name} (${fix.issue.severity})`));
+      console.log();
+      const origLines = fix.originalCode.split("\n");
+      console.log(chalk.red("  - Original:"));
+      origLines.slice(0, 8).forEach((line) => console.log(chalk.red(`    ${line}`)));
+      if (origLines.length > 8) {
+        console.log(chalk.gray(`    ... (${origLines.length - 8} more lines)`));
+      }
+      console.log();
+      const fixLines = fix.fixedCode.split("\n");
+      console.log(chalk.green("  + Suggested fix:"));
+      fixLines.slice(0, 8).forEach((line) => console.log(chalk.green(`    ${line}`)));
+      if (fixLines.length > 8) {
+        console.log(chalk.gray(`    ... (${fixLines.length - 8} more lines)`));
+      }
+      console.log();
+      console.log(chalk.gray(`  Explanation: ${fix.explanation}`));
+      console.log();
+      const action = await select({
+        message: "Apply this fix?",
+        choices: [
+          { name: "Yes - apply this fix", value: "y" },
+          { name: "No - skip this fix", value: "n" },
+          { name: "All - apply all remaining fixes", value: "a" },
+          { name: "Skip rest - save remaining to file", value: "s" },
+          { name: "Quit - stop reviewing", value: "q" }
+        ]
+      });
+      if (action === "y") {
+        acceptedFixes.push(fix);
+        console.log(chalk.green("  \u2714 Fix accepted"));
+      } else if (action === "n") {
+        skippedFixes.push(fix);
+        console.log(chalk.yellow("  \u2298 Fix skipped"));
+      } else if (action === "a") {
+        applyAll = true;
+        acceptedFixes.push(fix);
+        for (let j = i + 1; j < fixes.length; j++) {
+          acceptedFixes.push(fixes[j]);
+        }
+        console.log(chalk.green(`  \u2714 Applying all ${fixes.length - i} remaining fixes`));
+        break;
+      } else if (action === "s") {
+        skipRest = true;
+        skippedFixes.push(fix);
+        for (let j = i + 1; j < fixes.length; j++) {
+          skippedFixes.push(fixes[j]);
+        }
+        console.log(chalk.yellow(`  \u2298 Skipping ${fixes.length - i} remaining fixes`));
+        break;
+      } else if (action === "q") {
+        skippedFixes.push(fix);
+        for (let j = i + 1; j < fixes.length; j++) {
+          skippedFixes.push(fixes[j]);
+        }
+        console.log(chalk.gray("  Stopped reviewing"));
+        break;
+      }
+      console.log();
+    }
+    if (acceptedFixes.length > 0) {
+      console.log();
+      const applySpinner = ora({
+        text: `Applying ${acceptedFixes.length} fixes...`,
+        color: "cyan"
+      }).start();
+      let appliedCount = 0;
+      for (const fix of acceptedFixes) {
+        const content = fileContents.get(fix.issue.file);
+        if (!content) continue;
+        const newContent = content.replace(fix.originalCode, fix.fixedCode);
+        if (newContent !== content) {
+          const filePath = path3.resolve(targetPath, fix.issue.file);
+          fs3.writeFileSync(filePath, newContent, "utf-8");
+          fileContents.set(fix.issue.file, newContent);
+          appliedCount++;
+        }
+      }
+      applySpinner.succeed(`Applied ${appliedCount}/${acceptedFixes.length} fixes to your codebase`);
+    }
+    if (skippedFixes.length > 0) {
+      const fixesFile = ".cencori-fixes.json";
+      const fixesData = {
+        generated_at: (/* @__PURE__ */ new Date()).toISOString(),
+        total_fixes: skippedFixes.length,
+        fixes: skippedFixes.map((f) => ({
+          file: f.issue.file,
+          line: f.issue.line,
+          issue_type: f.issue.type,
+          issue_name: f.issue.name,
+          severity: f.issue.severity,
+          original_code: f.originalCode,
+          suggested_fix: f.fixedCode,
+          explanation: f.explanation
+        }))
+      };
+      fs3.writeFileSync(fixesFile, JSON.stringify(fixesData, null, 2));
+      console.log();
+      console.log(chalk.yellow(`  ${skippedFixes.length} skipped fixes saved to ${chalk.bold(fixesFile)}`));
+    }
+    console.log();
+    console.log(chalk.gray("  \u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500"));
+    console.log();
+    console.log(`  ${chalk.bold("Summary:")}`);
+    if (acceptedFixes.length > 0) {
+      console.log(chalk.green(`    \u2714 ${acceptedFixes.length} fixes applied`));
+    }
+    if (skippedFixes.length > 0) {
+      console.log(chalk.yellow(`    \u2298 ${skippedFixes.length} fixes skipped (saved to .cencori-fixes.json)`));
     }
-    const notApplied = appliedFixes.filter((f) => !f.applied);
-    if (notApplied.length > 0) {
+    if (acceptedFixes.length > 0) {
       console.log();
-      console.log(`  ${chalk.yellow(`${notApplied.length} issues require manual review`)}`);
+      console.log(chalk.cyan(`  Run ${chalk.bold("npx @cencori/scan")} again to verify your fixes!`));
     }
     console.log();
   } catch (error) {