feat: add cargo-fmt and clippy fixers

Adds Rust toolchain fixers: cargo fmt for formatting and clippy --fix
for auto-fixable lint warnings.

Author: wadefletch

dist/index.js

@@ -30015,11 +30015,64 @@ async function runEngine(ctx, opts) {
   }
 }
 
-// src/fixers/eslint.ts
+// src/fixers/cargo-fmt.ts
 import { exec as exec2 } from "node:child_process";
 import { promisify as promisify2 } from "node:util";
 var run2 = promisify2(exec2);
-var DETECT_PATTERNS = [
+var DETECT_PATTERNS = [/cargo fmt/i, /rustfmt/i, /Diff in \/.*\.rs/i, /should run `cargo fmt`/i];
+var cargoFmtFixer = {
+  name: "cargo-fmt",
+  detect(log) {
+    return DETECT_PATTERNS.some((p) => p.test(log));
+  },
+  async fix(cwd) {
+    try {
+      await run2("cargo fmt --all", { cwd });
+    } catch {}
+    const { stdout: status } = await run2("git diff --name-only", { cwd });
+    const files = status.trim().split(`
+`).filter(Boolean);
+    return {
+      summary: files.length > 0 ? `Formatted ${files.length} file(s) with cargo fmt` : "cargo fmt ran but all files were already formatted",
+      filesChanged: files.length
+    };
+  }
+};
+
+// src/fixers/clippy.ts
+import { exec as exec3 } from "node:child_process";
+import { promisify as promisify3 } from "node:util";
+var run3 = promisify3(exec3);
+var DETECT_PATTERNS2 = [
+  /cargo clippy/i,
+  /clippy::[\w_]+/,
+  /warning:.*\(clippy\)/i,
+  /error\[E\d+\]/
+];
+var clippyFixer = {
+  name: "clippy",
+  detect(log) {
+    return DETECT_PATTERNS2.some((p) => p.test(log));
+  },
+  async fix(cwd) {
+    try {
+      await run3("cargo clippy --fix --allow-dirty --allow-staged -- -W clippy::all", { cwd });
+    } catch {}
+    const { stdout: status } = await run3("git diff --name-only", { cwd });
+    const files = status.trim().split(`
+`).filter(Boolean);
+    return {
+      summary: files.length > 0 ? `Auto-fixed clippy warnings in ${files.length} file(s)` : "clippy ran but produced no auto-fixable changes",
+      filesChanged: files.length
+    };
+  }
+};
+
+// src/fixers/eslint.ts
+import { exec as exec4 } from "node:child_process";
+import { promisify as promisify4 } from "node:util";
+var run4 = promisify4(exec4);
+var DETECT_PATTERNS3 = [
   /eslint.*error/i,
   /\d+ errors?.*found/i,
   /Linting error/i,
@@ -30029,13 +30082,13 @@ var DETECT_PATTERNS = [
 var eslintFixer = {
   name: "eslint",
   detect(log) {
-    return DETECT_PATTERNS.some((p) => p.test(log));
+    return DETECT_PATTERNS3.some((p) => p.test(log));
   },
   async fix(cwd) {
     try {
-      await run2("npx eslint --fix .", { cwd });
+      await run4("npx eslint --fix .", { cwd });
     } catch {}
-    const { stdout: status } = await run2("git diff --name-only", { cwd });
+    const { stdout: status } = await run4("git diff --name-only", { cwd });
     const files = status.trim().split(`
 `).filter(Boolean);
     return {
@@ -30046,12 +30099,12 @@ var eslintFixer = {
 };
 
 // src/fixers/lockfile.ts
-import { exec as exec3 } from "node:child_process";
-import { promisify as promisify3 } from "node:util";
+import { exec as exec5 } from "node:child_process";
+import { promisify as promisify5 } from "node:util";
 import { access } from "node:fs/promises";
 import { join as join2 } from "node:path";
-var run3 = promisify3(exec3);
-var DETECT_PATTERNS2 = [
+var run5 = promisify5(exec5);
+var DETECT_PATTERNS4 = [
   /lockfile.*out of date/i,
   /frozen-lockfile/i,
   /ERR_PNPM_OUTDATED_LOCKFILE/i,
@@ -30084,14 +30137,14 @@ async function detectPackageManager(cwd) {
 var lockfileFixer = {
   name: "lockfile",
   detect(log) {
-    return DETECT_PATTERNS2.some((p) => p.test(log));
+    return DETECT_PATTERNS4.some((p) => p.test(log));
   },
   async fix(cwd) {
     const pm = await detectPackageManager(cwd);
     try {
-      await run3(`${pm} install`, { cwd });
+      await run5(`${pm} install`, { cwd });
     } catch {}
-    const { stdout: status } = await run3("git diff --name-only", { cwd });
+    const { stdout: status } = await run5("git diff --name-only", { cwd });
     const files = status.trim().split(`
 `).filter(Boolean);
     return {
@@ -30102,20 +30155,20 @@ var lockfileFixer = {
 };
 
 // src/fixers/oxfmt.ts
-import { exec as exec4 } from "node:child_process";
-import { promisify as promisify4 } from "node:util";
-var run4 = promisify4(exec4);
-var DETECT_PATTERNS3 = [/oxfmt/i, /oxc.*format/i];
+import { exec as exec6 } from "node:child_process";
+import { promisify as promisify6 } from "node:util";
+var run6 = promisify6(exec6);
+var DETECT_PATTERNS5 = [/oxfmt/i, /oxc.*format/i];
 var oxfmtFixer = {
   name: "oxfmt",
   detect(log) {
-    return DETECT_PATTERNS3.some((p) => p.test(log));
+    return DETECT_PATTERNS5.some((p) => p.test(log));
   },
   async fix(cwd) {
     try {
-      await run4("npx oxfmt --write .", { cwd });
+      await run6("npx oxfmt --write .", { cwd });
     } catch {}
-    const { stdout: status } = await run4("git diff --name-only", { cwd });
+    const { stdout: status } = await run6("git diff --name-only", { cwd });
     const files = status.trim().split(`
 `).filter(Boolean);
     return {
@@ -30126,20 +30179,20 @@ var oxfmtFixer = {
 };
 
 // src/fixers/oxlint.ts
-import { exec as exec5 } from "node:child_process";
-import { promisify as promisify5 } from "node:util";
-var run5 = promisify5(exec5);
-var DETECT_PATTERNS4 = [/oxlint/i, /oxc.*lint/i, /Found \d+ errors?.*oxlint/i];
+import { exec as exec7 } from "node:child_process";
+import { promisify as promisify7 } from "node:util";
+var run7 = promisify7(exec7);
+var DETECT_PATTERNS6 = [/oxlint/i, /oxc.*lint/i, /Found \d+ errors?.*oxlint/i];
 var oxlintFixer = {
   name: "oxlint",
   detect(log) {
-    return DETECT_PATTERNS4.some((p) => p.test(log));
+    return DETECT_PATTERNS6.some((p) => p.test(log));
   },
   async fix(cwd) {
     try {
-      await run5("npx oxlint --fix .", { cwd });
+      await run7("npx oxlint --fix .", { cwd });
     } catch {}
-    const { stdout: status } = await run5("git diff --name-only", { cwd });
+    const { stdout: status } = await run7("git diff --name-only", { cwd });
     const files = status.trim().split(`
 `).filter(Boolean);
     return {
@@ -30150,10 +30203,10 @@ var oxlintFixer = {
 };
 
 // src/fixers/prettier.ts
-import { exec as exec6 } from "node:child_process";
-import { promisify as promisify6 } from "node:util";
-var run6 = promisify6(exec6);
-var DETECT_PATTERNS5 = [
+import { exec as exec8 } from "node:child_process";
+import { promisify as promisify8 } from "node:util";
+var run8 = promisify8(exec8);
+var DETECT_PATTERNS7 = [
   /prettier.*--check/i,
   /Code style issues found/i,
   /\[warn\].*isn't formatted/i,
@@ -30163,13 +30216,13 @@ var DETECT_PATTERNS5 = [
 var prettierFixer = {
   name: "prettier",
   detect(log) {
-    return DETECT_PATTERNS5.some((p) => p.test(log));
+    return DETECT_PATTERNS7.some((p) => p.test(log));
   },
   async fix(cwd) {
     try {
-      await run6("npx prettier --write .", { cwd });
+      await run8("npx prettier --write .", { cwd });
     } catch {}
-    const { stdout: status } = await run6("git diff --name-only", { cwd });
+    const { stdout: status } = await run8("git diff --name-only", { cwd });
     const files = status.trim().split(`
 `).filter(Boolean);
     return {
@@ -30181,6 +30234,8 @@ var prettierFixer = {
 
 // src/fixers/index.ts
 var builtinFixers = [
+  cargoFmtFixer,
+  clippyFixer,
   eslintFixer,
   lockfileFixer,
   oxfmtFixer,

src/fixers/cargo-fmt.ts

@@ -0,0 +1,34 @@
+import { exec } from "node:child_process";
+import { promisify } from "node:util";
+import type { Fixer, FixResult } from "../core/types.js";
+
+const run = promisify(exec);
+
+const DETECT_PATTERNS = [/cargo fmt/i, /rustfmt/i, /Diff in \/.*\.rs/i, /should run `cargo fmt`/i];
+
+export const cargoFmtFixer: Fixer = {
+  name: "cargo-fmt",
+
+  detect(log: string): boolean {
+    return DETECT_PATTERNS.some((p) => p.test(log));
+  },
+
+  async fix(cwd: string): Promise<FixResult> {
+    try {
+      await run("cargo fmt --all", { cwd });
+    } catch {
+      // cargo fmt can fail if rustfmt is not installed
+    }
+
+    const { stdout: status } = await run("git diff --name-only", { cwd });
+    const files = status.trim().split("\n").filter(Boolean);
+
+    return {
+      summary:
+        files.length > 0
+          ? `Formatted ${files.length} file(s) with cargo fmt`
+          : "cargo fmt ran but all files were already formatted",
+      filesChanged: files.length,
+    };
+  },
+};

src/fixers/clippy.ts

@@ -0,0 +1,39 @@
+import { exec } from "node:child_process";
+import { promisify } from "node:util";
+import type { Fixer, FixResult } from "../core/types.js";
+
+const run = promisify(exec);
+
+const DETECT_PATTERNS = [
+  /cargo clippy/i,
+  /clippy::[\w_]+/,
+  /warning:.*\(clippy\)/i,
+  /error\[E\d+\]/,
+];
+
+export const clippyFixer: Fixer = {
+  name: "clippy",
+
+  detect(log: string): boolean {
+    return DETECT_PATTERNS.some((p) => p.test(log));
+  },
+
+  async fix(cwd: string): Promise<FixResult> {
+    try {
+      await run("cargo clippy --fix --allow-dirty --allow-staged -- -W clippy::all", { cwd });
+    } catch {
+      // clippy --fix exits non-zero if unfixable warnings remain
+    }
+
+    const { stdout: status } = await run("git diff --name-only", { cwd });
+    const files = status.trim().split("\n").filter(Boolean);
+
+    return {
+      summary:
+        files.length > 0
+          ? `Auto-fixed clippy warnings in ${files.length} file(s)`
+          : "clippy ran but produced no auto-fixable changes",
+      filesChanged: files.length,
+    };
+  },
+};

src/fixers/index.ts

@@ -1,16 +1,28 @@
 import type { Fixer } from "../core/types.js";
+import { cargoFmtFixer } from "./cargo-fmt.js";
+import { clippyFixer } from "./clippy.js";
 import { eslintFixer } from "./eslint.js";
 import { lockfileFixer } from "./lockfile.js";
 import { oxfmtFixer } from "./oxfmt.js";
 import { oxlintFixer } from "./oxlint.js";
 import { prettierFixer } from "./prettier.js";
 
 export const builtinFixers: Fixer[] = [
+  cargoFmtFixer,
+  clippyFixer,
   eslintFixer,
   lockfileFixer,
   oxfmtFixer,
   oxlintFixer,
   prettierFixer,
 ];
 
-export { eslintFixer, lockfileFixer, oxfmtFixer, oxlintFixer, prettierFixer };
+export {
+  cargoFmtFixer,
+  clippyFixer,
+  eslintFixer,
+  lockfileFixer,
+  oxfmtFixer,
+  oxlintFixer,
+  prettierFixer,
+};