@cencori/scan 0.4.4 → 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.4";
+var VERSION = "0.4.5";
 var scoreStyles = {
   A: { color: chalk.green },
   B: { color: chalk.blue },
@@ -1388,57 +1370,108 @@ 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();
-    const applied = appliedFixes.filter((f) => f.applied);
-    const notApplied = appliedFixes.filter((f) => !f.applied);
-    if (applied.length > 0) {
-      console.log(`  ${chalk.bold.green("\u2714 Applied fixes:")}`);
-      for (const fix of 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();
-    }
-    if (notApplied.length > 0) {
-      console.log(`  ${chalk.bold.yellow("\u26A0 Suggested fixes (require manual review):")}`);
+      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();
-      for (let i = 0; i < Math.min(notApplied.length, 5); i++) {
-        const fix = notApplied[i];
-        console.log(chalk.cyan(`  \u2500\u2500\u2500 ${fix.issue.file}:${fix.issue.line} \u2500\u2500\u2500`));
-        console.log(chalk.gray(`  Issue: ${fix.issue.name}`));
-        console.log();
-        const origLines = fix.originalCode.split("\n").slice(0, 5);
-        console.log(chalk.red("  - Original:"));
-        origLines.forEach((line) => console.log(chalk.red(`    ${line}`)));
-        if (fix.originalCode.split("\n").length > 5) {
-          console.log(chalk.gray("    ... (truncated)"));
+      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();
-        const fixLines = fix.fixedCode.split("\n").slice(0, 5);
-        console.log(chalk.green("  + Suggested fix:"));
-        fixLines.forEach((line) => console.log(chalk.green(`    ${line}`)));
-        if (fix.fixedCode.split("\n").length > 5) {
-          console.log(chalk.gray("    ... (truncated)"));
+        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.gray(`  Explanation: ${fix.explanation}`));
-        console.log();
+        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;
       }
-      if (notApplied.length > 5) {
-        console.log(chalk.gray(`  ... and ${notApplied.length - 5} more fixes`));
-        console.log();
+      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: notApplied.length,
-        fixes: notApplied.map((f) => ({
+        total_fixes: skippedFixes.length,
+        fixes: skippedFixes.map((f) => ({
           file: f.issue.file,
           line: f.issue.line,
           issue_type: f.issue.type,
@@ -1450,16 +1483,22 @@ async function handleAutoFix(result, targetPath) {
         }))
       };
       fs3.writeFileSync(fixesFile, JSON.stringify(fixesData, null, 2));
-      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("Next steps:")}`);
-      console.log(chalk.cyan(`    1. Review fixes in ${chalk.bold(fixesFile)}`));
-      console.log(chalk.cyan(`    2. Apply fixes manually to your codebase`));
-      console.log(chalk.cyan(`    3. Run ${chalk.bold("npx @cencori/scan")} again to verify`));
       console.log();
+      console.log(chalk.yellow(`  ${skippedFixes.length} skipped fixes saved to ${chalk.bold(fixesFile)}`));
     }
-    if (applied.length === 0 && notApplied.length === 0) {
-      console.log(chalk.green("  No fixes needed!"));
+    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)`));
+    }
+    if (acceptedFixes.length > 0) {
+      console.log();
+      console.log(chalk.cyan(`  Run ${chalk.bold("npx @cencori/scan")} again to verify your fixes!`));
     }
     console.log();
   } catch (error) {