feat: add terraform-fmt and ruff fixers

terraform-fmt: formats .tf files, falls back to tofu if terraform
is not available. ruff: runs both ruff check --fix and ruff format.

Author: wadefletch

dist/index.js

@@ -30232,6 +30232,73 @@ var prettierFixer = {
   }
 };
 
+// src/fixers/ruff.ts
+import { exec as exec9 } from "node:child_process";
+import { promisify as promisify9 } from "node:util";
+var run9 = promisify9(exec9);
+var DETECT_PATTERNS8 = [
+  /ruff check/i,
+  /ruff format/i,
+  /Found \d+ errors?.*\[.*fixable/i,
+  /would reformat/i,
+  /\d+ files? would be reformatted/i
+];
+var ruffFixer = {
+  name: "ruff",
+  detect(log) {
+    return DETECT_PATTERNS8.some((p) => p.test(log));
+  },
+  async fix(cwd) {
+    try {
+      await run9("ruff check --fix .", { cwd });
+    } catch {}
+    try {
+      await run9("ruff format .", { cwd });
+    } catch {}
+    const { stdout: status } = await run9("git diff --name-only", { cwd });
+    const files = status.trim().split(`
+`).filter(Boolean);
+    return {
+      summary: files.length > 0 ? `Auto-fixed ruff issues in ${files.length} file(s)` : "ruff ran but produced no changes",
+      filesChanged: files.length
+    };
+  }
+};
+
+// src/fixers/terraform-fmt.ts
+import { exec as exec10 } from "node:child_process";
+import { promisify as promisify10 } from "node:util";
+var run10 = promisify10(exec10);
+var DETECT_PATTERNS9 = [
+  /terraform fmt/i,
+  /tofu fmt/i,
+  /\.tf.*incorrect/i,
+  /files need formatting/i,
+  /Run terraform fmt/i
+];
+var terraformFmtFixer = {
+  name: "terraform-fmt",
+  detect(log) {
+    return DETECT_PATTERNS9.some((p) => p.test(log));
+  },
+  async fix(cwd) {
+    try {
+      await run10("terraform fmt -recursive", { cwd });
+    } catch {
+      try {
+        await run10("tofu fmt -recursive", { cwd });
+      } catch {}
+    }
+    const { stdout: status } = await run10("git diff --name-only", { cwd });
+    const files = status.trim().split(`
+`).filter(Boolean);
+    return {
+      summary: files.length > 0 ? `Formatted ${files.length} file(s) with terraform fmt` : "terraform fmt ran but all files were already formatted",
+      filesChanged: files.length
+    };
+  }
+};
+
 // src/fixers/index.ts
 var builtinFixers = [
   cargoFmtFixer,
@@ -30240,7 +30307,9 @@ var builtinFixers = [
   lockfileFixer,
   oxfmtFixer,
   oxlintFixer,
-  prettierFixer
+  prettierFixer,
+  ruffFixer,
+  terraformFmtFixer
 ];
 
 // src/index.ts

src/fixers/index.ts

@@ -6,6 +6,8 @@ import { lockfileFixer } from "./lockfile.js";
 import { oxfmtFixer } from "./oxfmt.js";
 import { oxlintFixer } from "./oxlint.js";
 import { prettierFixer } from "./prettier.js";
+import { ruffFixer } from "./ruff.js";
+import { terraformFmtFixer } from "./terraform-fmt.js";
 
 export const builtinFixers: Fixer[] = [
   cargoFmtFixer,
@@ -15,6 +17,8 @@ export const builtinFixers: Fixer[] = [
   oxfmtFixer,
   oxlintFixer,
   prettierFixer,
+  ruffFixer,
+  terraformFmtFixer,
 ];
 
 export {
@@ -25,4 +29,6 @@ export {
   oxfmtFixer,
   oxlintFixer,
   prettierFixer,
+  ruffFixer,
+  terraformFmtFixer,
 };

src/fixers/ruff.ts

@@ -0,0 +1,46 @@
+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 = [
+  /ruff check/i,
+  /ruff format/i,
+  /Found \d+ errors?.*\[.*fixable/i,
+  /would reformat/i,
+  /\d+ files? would be reformatted/i,
+];
+
+export const ruffFixer: Fixer = {
+  name: "ruff",
+
+  detect(log: string): boolean {
+    return DETECT_PATTERNS.some((p) => p.test(log));
+  },
+
+  async fix(cwd: string): Promise<FixResult> {
+    try {
+      await run("ruff check --fix .", { cwd });
+    } catch {
+      // ruff check --fix exits non-zero if unfixable errors remain
+    }
+
+    try {
+      await run("ruff format .", { cwd });
+    } catch {
+      // ruff format can fail if 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
+          ? `Auto-fixed ruff issues in ${files.length} file(s)`
+          : "ruff ran but produced no changes",
+      filesChanged: files.length,
+    };
+  },
+};

src/fixers/terraform-fmt.ts

@@ -0,0 +1,44 @@
+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 = [
+  /terraform fmt/i,
+  /tofu fmt/i,
+  /\.tf.*incorrect/i,
+  /files need formatting/i,
+  /Run terraform fmt/i,
+];
+
+export const terraformFmtFixer: Fixer = {
+  name: "terraform-fmt",
+
+  detect(log: string): boolean {
+    return DETECT_PATTERNS.some((p) => p.test(log));
+  },
+
+  async fix(cwd: string): Promise<FixResult> {
+    try {
+      await run("terraform fmt -recursive", { cwd });
+    } catch {
+      try {
+        await run("tofu fmt -recursive", { cwd });
+      } catch {
+        // neither terraform nor tofu available
+      }
+    }
+
+    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 terraform fmt`
+          : "terraform fmt ran but all files were already formatted",
+      filesChanged: files.length,
+    };
+  },
+};