agent-gauntlet 0.1.10 → 0.1.11

package/src/commands/review.ts

@@ -1,72 +1,78 @@
-import type { Command } from 'commander';
-import chalk from 'chalk';
-import { loadConfig } from '../config/loader.js';
-import { ChangeDetector } from '../core/change-detector.js';
-import { EntryPointExpander } from '../core/entry-point.js';
-import { JobGenerator } from '../core/job.js';
-import { Runner } from '../core/runner.js';
-import { Logger } from '../output/logger.js';
-import { ConsoleReporter } from '../output/console.js';
-import { rotateLogs } from './shared.js';
+import chalk from "chalk";
+import type { Command } from "commander";
+import { loadConfig } from "../config/loader.js";
+import { ChangeDetector } from "../core/change-detector.js";
+import { EntryPointExpander } from "../core/entry-point.js";
+import { JobGenerator } from "../core/job.js";
+import { Runner } from "../core/runner.js";
+import { ConsoleReporter } from "../output/console.js";
+import { Logger } from "../output/logger.js";
+import { rotateLogs } from "./shared.js";
 
 export function registerReviewCommand(program: Command): void {
-  program
-    .command('review')
-    .description('Run only applicable reviews for detected changes')
-    .option('-g, --gate <name>', 'Run specific review gate only')
-    .option('-c, --commit <sha>', 'Use diff for a specific commit')
-    .option('-u, --uncommitted', 'Use diff for current uncommitted changes (staged and unstaged)')
-    .action(async (options) => {
-      try {
-        const config = await loadConfig();
+	program
+		.command("review")
+		.description("Run only applicable reviews for detected changes")
+		.option("-g, --gate <name>", "Run specific review gate only")
+		.option("-c, --commit <sha>", "Use diff for a specific commit")
+		.option(
+			"-u, --uncommitted",
+			"Use diff for current uncommitted changes (staged and unstaged)",
+		)
+		.action(async (options) => {
+			try {
+				const config = await loadConfig();
 
-        // Rotate logs before starting
-        await rotateLogs(config.project.log_dir);
+				// Rotate logs before starting
+				await rotateLogs(config.project.log_dir);
 
-        const changeDetector = new ChangeDetector(config.project.base_branch, {
-          commit: options.commit,
-          uncommitted: options.uncommitted
-        });
-        const expander = new EntryPointExpander();
-        const jobGen = new JobGenerator(config);
-        
-        console.log(chalk.dim('Detecting changes...'));
-        const changes = await changeDetector.getChangedFiles();
-        
-        if (changes.length === 0) {
-          console.log(chalk.green('No changes detected.'));
-          process.exit(0);
-        }
-        
-        console.log(chalk.dim(`Found ${changes.length} changed files.`));
+				const changeDetector = new ChangeDetector(config.project.base_branch, {
+					commit: options.commit,
+					uncommitted: options.uncommitted,
+				});
+				const expander = new EntryPointExpander();
+				const jobGen = new JobGenerator(config);
 
-        const entryPoints = await expander.expand(config.project.entry_points, changes);
-        let jobs = jobGen.generateJobs(entryPoints);
+				console.log(chalk.dim("Detecting changes..."));
+				const changes = await changeDetector.getChangedFiles();
 
-        // Filter to only reviews
-        jobs = jobs.filter(j => j.type === 'review');
+				if (changes.length === 0) {
+					console.log(chalk.green("No changes detected."));
+					process.exit(0);
+				}
 
-        if (options.gate) {
-          jobs = jobs.filter(j => j.name === options.gate);
-        }
+				console.log(chalk.dim(`Found ${changes.length} changed files.`));
 
-        if (jobs.length === 0) {
-          console.log(chalk.yellow('No applicable reviews for these changes.'));
-          process.exit(0);
-        }
+				const entryPoints = await expander.expand(
+					config.project.entry_points,
+					changes,
+				);
+				let jobs = jobGen.generateJobs(entryPoints);
 
-        console.log(chalk.dim(`Running ${jobs.length} review(s)...`));
+				// Filter to only reviews
+				jobs = jobs.filter((j) => j.type === "review");
 
-        const logger = new Logger(config.project.log_dir);
-        const reporter = new ConsoleReporter();
-        const runner = new Runner(config, logger, reporter);
+				if (options.gate) {
+					jobs = jobs.filter((j) => j.name === options.gate);
+				}
 
-        const success = await runner.run(jobs);
-        process.exit(success ? 0 : 1);
+				if (jobs.length === 0) {
+					console.log(chalk.yellow("No applicable reviews for these changes."));
+					process.exit(0);
+				}
 
-      } catch (error: any) {
-        console.error(chalk.red('Error:'), error.message);
-        process.exit(1);
-      }
-    });
+				console.log(chalk.dim(`Running ${jobs.length} review(s)...`));
+
+				const logger = new Logger(config.project.log_dir);
+				const reporter = new ConsoleReporter();
+				const runner = new Runner(config, logger, reporter);
+
+				const success = await runner.run(jobs);
+				process.exit(success ? 0 : 1);
+			} catch (error: unknown) {
+				const err = error as { message?: string };
+				console.error(chalk.red("Error:"), err.message);
+				process.exit(1);
+			}
+		});
 }

package/src/commands/run.test.ts

@@ -1,25 +1,27 @@
-import { describe, it, expect, beforeEach } from 'bun:test';
-import { Command } from 'commander';
-import { registerRunCommand } from './run.js';
+import { beforeEach, describe, expect, it } from "bun:test";
+import { Command } from "commander";
+import { registerRunCommand } from "./run.js";
 
-describe('Run Command', () => {
-  let program: Command;
+describe("Run Command", () => {
+	let program: Command;
 
-  beforeEach(() => {
-    program = new Command();
-    registerRunCommand(program);
-  });
+	beforeEach(() => {
+		program = new Command();
+		registerRunCommand(program);
+	});
 
-  it('should register the run command', () => {
-    const runCmd = program.commands.find(cmd => cmd.name() === 'run');
-    expect(runCmd).toBeDefined();
-    expect(runCmd?.description()).toBe('Run gates for detected changes');
-  });
+	it("should register the run command", () => {
+		const runCmd = program.commands.find((cmd) => cmd.name() === "run");
+		expect(runCmd).toBeDefined();
+		expect(runCmd?.description()).toBe("Run gates for detected changes");
+	});
 
-  it('should have correct options', () => {
-    const runCmd = program.commands.find(cmd => cmd.name() === 'run');
-    expect(runCmd?.options.some(opt => opt.long === '--gate')).toBe(true);
-    expect(runCmd?.options.some(opt => opt.long === '--commit')).toBe(true);
-    expect(runCmd?.options.some(opt => opt.long === '--uncommitted')).toBe(true);
-  });
+	it("should have correct options", () => {
+		const runCmd = program.commands.find((cmd) => cmd.name() === "run");
+		expect(runCmd?.options.some((opt) => opt.long === "--gate")).toBe(true);
+		expect(runCmd?.options.some((opt) => opt.long === "--commit")).toBe(true);
+		expect(runCmd?.options.some((opt) => opt.long === "--uncommitted")).toBe(
+			true,
+		);
+	});
 });

package/src/commands/run.ts

@@ -1,69 +1,75 @@
-import type { Command } from 'commander';
-import chalk from 'chalk';
-import { loadConfig } from '../config/loader.js';
-import { ChangeDetector } from '../core/change-detector.js';
-import { EntryPointExpander } from '../core/entry-point.js';
-import { JobGenerator } from '../core/job.js';
-import { Runner } from '../core/runner.js';
-import { Logger } from '../output/logger.js';
-import { ConsoleReporter } from '../output/console.js';
-import { rotateLogs } from './shared.js';
+import chalk from "chalk";
+import type { Command } from "commander";
+import { loadConfig } from "../config/loader.js";
+import { ChangeDetector } from "../core/change-detector.js";
+import { EntryPointExpander } from "../core/entry-point.js";
+import { JobGenerator } from "../core/job.js";
+import { Runner } from "../core/runner.js";
+import { ConsoleReporter } from "../output/console.js";
+import { Logger } from "../output/logger.js";
+import { rotateLogs } from "./shared.js";
 
 export function registerRunCommand(program: Command): void {
-  program
-    .command('run')
-    .description('Run gates for detected changes')
-    .option('-g, --gate <name>', 'Run specific gate only')
-    .option('-c, --commit <sha>', 'Use diff for a specific commit')
-    .option('-u, --uncommitted', 'Use diff for current uncommitted changes (staged and unstaged)')
-    .action(async (options) => {
-      try {
-        const config = await loadConfig();
-        
-        // Rotate logs before starting
-        await rotateLogs(config.project.log_dir);
+	program
+		.command("run")
+		.description("Run gates for detected changes")
+		.option("-g, --gate <name>", "Run specific gate only")
+		.option("-c, --commit <sha>", "Use diff for a specific commit")
+		.option(
+			"-u, --uncommitted",
+			"Use diff for current uncommitted changes (staged and unstaged)",
+		)
+		.action(async (options) => {
+			try {
+				const config = await loadConfig();
 
-        const changeDetector = new ChangeDetector(config.project.base_branch, {
-          commit: options.commit,
-          uncommitted: options.uncommitted
-        });
-        const expander = new EntryPointExpander();
-        const jobGen = new JobGenerator(config);
-        
-        console.log(chalk.dim('Detecting changes...'));
-        const changes = await changeDetector.getChangedFiles();
-        
-        if (changes.length === 0) {
-          console.log(chalk.green('No changes detected.'));
-          process.exit(0);
-        }
-        
-        console.log(chalk.dim(`Found ${changes.length} changed files.`));
+				// Rotate logs before starting
+				await rotateLogs(config.project.log_dir);
 
-        const entryPoints = await expander.expand(config.project.entry_points, changes);
-        let jobs = jobGen.generateJobs(entryPoints);
+				const changeDetector = new ChangeDetector(config.project.base_branch, {
+					commit: options.commit,
+					uncommitted: options.uncommitted,
+				});
+				const expander = new EntryPointExpander();
+				const jobGen = new JobGenerator(config);
 
-        if (options.gate) {
-          jobs = jobs.filter(j => j.name === options.gate);
-        }
+				console.log(chalk.dim("Detecting changes..."));
+				const changes = await changeDetector.getChangedFiles();
 
-        if (jobs.length === 0) {
-          console.log(chalk.yellow('No applicable gates for these changes.'));
-          process.exit(0);
-        }
+				if (changes.length === 0) {
+					console.log(chalk.green("No changes detected."));
+					process.exit(0);
+				}
 
-        console.log(chalk.dim(`Running ${jobs.length} gates...`));
+				console.log(chalk.dim(`Found ${changes.length} changed files.`));
 
-        const logger = new Logger(config.project.log_dir);
-        const reporter = new ConsoleReporter();
-        const runner = new Runner(config, logger, reporter);
+				const entryPoints = await expander.expand(
+					config.project.entry_points,
+					changes,
+				);
+				let jobs = jobGen.generateJobs(entryPoints);
 
-        const success = await runner.run(jobs);
-        process.exit(success ? 0 : 1);
+				if (options.gate) {
+					jobs = jobs.filter((j) => j.name === options.gate);
+				}
 
-      } catch (error: any) {
-        console.error(chalk.red('Error:'), error.message);
-        process.exit(1);
-      }
-    });
+				if (jobs.length === 0) {
+					console.log(chalk.yellow("No applicable gates for these changes."));
+					process.exit(0);
+				}
+
+				console.log(chalk.dim(`Running ${jobs.length} gates...`));
+
+				const logger = new Logger(config.project.log_dir);
+				const reporter = new ConsoleReporter();
+				const runner = new Runner(config, logger, reporter);
+
+				const success = await runner.run(jobs);
+				process.exit(success ? 0 : 1);
+			} catch (error: unknown) {
+				const err = error as { message?: string };
+				console.error(chalk.red("Error:"), err.message);
+				process.exit(1);
+			}
+		});
 }

package/src/commands/shared.ts

@@ -1,44 +1,53 @@
-import fs from 'node:fs/promises';
-import path from 'node:path';
+import fs from "node:fs/promises";
+import path from "node:path";
 
 export async function exists(path: string): Promise<boolean> {
-  try {
-    await fs.stat(path);
-    return true;
-  } catch {
-    return false;
-  }
+	try {
+		await fs.stat(path);
+		return true;
+	} catch {
+		return false;
+	}
 }
 
 export async function rotateLogs(logDir: string): Promise<void> {
-  const previousDir = path.join(logDir, 'previous');
+	const previousDir = path.join(logDir, "previous");
 
-  try {
-    // 1. Ensure logDir exists (if not, nothing to rotate, but we should create it for future use if needed, 
-    //    though usually the logger creates it. If it doesn't exist, we can just return).
-    if (!(await exists(logDir))) {
-        return;
-    }
+	try {
+		// 1. Ensure logDir exists (if not, nothing to rotate, but we should create it for future use if needed,
+		//    though usually the logger creates it. If it doesn't exist, we can just return).
+		if (!(await exists(logDir))) {
+			return;
+		}
 
-    // 2. Clear .gauntlet_logs/previous if it exists
-    if (await exists(previousDir)) {
-      const previousFiles = await fs.readdir(previousDir);
-      await Promise.all(
-        previousFiles.map(file => fs.rm(path.join(previousDir, file), { recursive: true, force: true }))
-      );
-    } else {
-      await fs.mkdir(previousDir, { recursive: true });
-    }
+		// 2. Clear .gauntlet_logs/previous if it exists
+		if (await exists(previousDir)) {
+			const previousFiles = await fs.readdir(previousDir);
+			await Promise.all(
+				previousFiles.map((file) =>
+					fs.rm(path.join(previousDir, file), { recursive: true, force: true }),
+				),
+			);
+		} else {
+			await fs.mkdir(previousDir, { recursive: true });
+		}
 
-    // 3. Move all existing files in .gauntlet_logs/ to .gauntlet_logs/previous
-    const files = await fs.readdir(logDir);
-    await Promise.all(
-      files
-        .filter(file => file !== 'previous')
-        .map(file => fs.rename(path.join(logDir, file), path.join(previousDir, file)))
-    );
-  } catch (error) {
-    // Log warning but don't crash the run as log rotation failure isn't critical
-    console.warn(`Failed to rotate logs in ${logDir}:`, error instanceof Error ? error.message : error);
-  }
+		// 3. Move all existing files in .gauntlet_logs/ to .gauntlet_logs/previous
+		const files = await fs.readdir(logDir);
+		await Promise.all(
+			files
+				.filter((file) => file !== "previous")
+				.map((file) =>
+					fs.rename(path.join(logDir, file), path.join(previousDir, file)),
+				),
+		);
+	} catch (error) {
+		// Log warning but don't crash the run as log rotation failure isn't critical
+		console.warn(
+			"Failed to rotate logs in",
+			logDir,
+			":",
+			error instanceof Error ? error.message : error,
+		);
+	}
 }

package/src/config/loader.test.ts

@@ -1,23 +1,25 @@
-import { describe, it, expect, beforeAll, afterAll } from 'bun:test';
-import fs from 'node:fs/promises';
-import path from 'node:path';
-import { loadConfig } from './loader.js';
-
-const TEST_DIR = path.join(process.cwd(), 'test-env-' + Date.now());
-const GAUNTLET_DIR = path.join(TEST_DIR, '.gauntlet');
-const CHECKS_DIR = path.join(GAUNTLET_DIR, 'checks');
-const REVIEWS_DIR = path.join(GAUNTLET_DIR, 'reviews');
-
-describe('Config Loader', () => {
-  beforeAll(async () => {
-    // Setup directory structure
-    await fs.mkdir(TEST_DIR);
-    await fs.mkdir(GAUNTLET_DIR);
-    await fs.mkdir(CHECKS_DIR);
-    await fs.mkdir(REVIEWS_DIR);
-
-    // Write config.yml
-    await fs.writeFile(path.join(GAUNTLET_DIR, 'config.yml'), `
+import { afterAll, beforeAll, describe, expect, it } from "bun:test";
+import fs from "node:fs/promises";
+import path from "node:path";
+import { loadConfig } from "./loader.js";
+
+const TEST_DIR = path.join(process.cwd(), `test-env-${Date.now()}`);
+const GAUNTLET_DIR = path.join(TEST_DIR, ".gauntlet");
+const CHECKS_DIR = path.join(GAUNTLET_DIR, "checks");
+const REVIEWS_DIR = path.join(GAUNTLET_DIR, "reviews");
+
+describe("Config Loader", () => {
+	beforeAll(async () => {
+		// Setup directory structure
+		await fs.mkdir(TEST_DIR);
+		await fs.mkdir(GAUNTLET_DIR);
+		await fs.mkdir(CHECKS_DIR);
+		await fs.mkdir(REVIEWS_DIR);
+
+		// Write config.yml
+		await fs.writeFile(
+			path.join(GAUNTLET_DIR, "config.yml"),
+			`
 base_branch: origin/dev
 log_dir: test_logs
 cli:
@@ -31,99 +33,119 @@ entry_points:
       - lint
     reviews:
       - security
-`);
+`,
+		);
 
-    // Write a check definition
-    await fs.writeFile(path.join(CHECKS_DIR, 'lint.yml'), `
+		// Write a check definition
+		await fs.writeFile(
+			path.join(CHECKS_DIR, "lint.yml"),
+			`
 name: lint
 command: npm run lint
 working_directory: .
-`);
+`,
+		);
 
-    // Write a review definition
-    await fs.writeFile(path.join(REVIEWS_DIR, 'security.md'), `---
+		// Write a review definition
+		await fs.writeFile(
+			path.join(REVIEWS_DIR, "security.md"),
+			`---
 cli_preference:
   - gemini
 ---
 
 # Security Review
 Check for vulnerabilities.
-`);
+`,
+		);
 
-    // Write a review definition without preference
-    await fs.writeFile(path.join(REVIEWS_DIR, 'style.md'), `---
+		// Write a review definition without preference
+		await fs.writeFile(
+			path.join(REVIEWS_DIR, "style.md"),
+			`---
 num_reviews: 1
 ---
 
 # Style Review
 Check style.
-`);
-  });
-
-  afterAll(async () => {
-    // Cleanup
-    await fs.rm(TEST_DIR, { recursive: true, force: true });
-  });
-
-  it('should load project configuration correctly', async () => {
-    const config = await loadConfig(TEST_DIR);
-    
-    expect(config.project.base_branch).toBe('origin/dev');
-    expect(config.project.log_dir).toBe('test_logs');
-    expect(config.project.entry_points).toHaveLength(1);
-    expect(config.project.entry_points[0].path).toBe('src/');
-  });
-
-  it('should load check gates correctly', async () => {
-    const config = await loadConfig(TEST_DIR);
-    
-    expect(Object.keys(config.checks)).toContain('lint');
-    expect(config.checks['lint'].command).toBe('npm run lint');
-  });
-
-  it('should load review gates correctly', async () => {
-    const config = await loadConfig(TEST_DIR);
-    
-    expect(Object.keys(config.reviews)).toContain('security');
-    expect(config.reviews['security'].name).toBe('security');
-    expect(config.reviews['security'].cli_preference).toEqual(['gemini']);
-    expect(config.reviews['security'].promptContent).toContain('Check for vulnerabilities.');
-  });
-
-  it('should merge default cli preference', async () => {
-    const config = await loadConfig(TEST_DIR);
-    
-    expect(Object.keys(config.reviews)).toContain('style');
-    expect(config.reviews['style'].cli_preference).toEqual(['claude', 'gemini']);
-  });
-
-  it('should reject check gate with fail_fast when parallel is true', async () => {
-    await fs.writeFile(path.join(CHECKS_DIR, 'invalid.yml'), `
+`,
+		);
+	});
+
+	afterAll(async () => {
+		// Cleanup
+		await fs.rm(TEST_DIR, { recursive: true, force: true });
+	});
+
+	it("should load project configuration correctly", async () => {
+		const config = await loadConfig(TEST_DIR);
+
+		expect(config.project.base_branch).toBe("origin/dev");
+		expect(config.project.log_dir).toBe("test_logs");
+		expect(config.project.entry_points).toHaveLength(1);
+		expect(config.project.entry_points[0].path).toBe("src/");
+	});
+
+	it("should load check gates correctly", async () => {
+		const config = await loadConfig(TEST_DIR);
+
+		expect(Object.keys(config.checks)).toContain("lint");
+		expect(config.checks.lint.command).toBe("npm run lint");
+	});
+
+	it("should load review gates correctly", async () => {
+		const config = await loadConfig(TEST_DIR);
+
+		expect(Object.keys(config.reviews)).toContain("security");
+		expect(config.reviews.security.name).toBe("security");
+		expect(config.reviews.security.cli_preference).toEqual(["gemini"]);
+		expect(config.reviews.security.promptContent).toContain(
+			"Check for vulnerabilities.",
+		);
+	});
+
+	it("should merge default cli preference", async () => {
+		const config = await loadConfig(TEST_DIR);
+
+		expect(Object.keys(config.reviews)).toContain("style");
+		expect(config.reviews.style.cli_preference).toEqual(["claude", "gemini"]);
+	});
+
+	it("should reject check gate with fail_fast when parallel is true", async () => {
+		await fs.writeFile(
+			path.join(CHECKS_DIR, "invalid.yml"),
+			`
 name: invalid
 command: echo test
 parallel: true
 fail_fast: true
-`);
-
-    await expect(loadConfig(TEST_DIR)).rejects.toThrow(/fail_fast can only be used when parallel is false/);
-  });
-
-  it('should accept check gate with fail_fast when parallel is false', async () => {
-    // Clean up the invalid file first
-    try {
-      await fs.unlink(path.join(CHECKS_DIR, 'invalid.yml'));
-    } catch {}
-
-    await fs.writeFile(path.join(CHECKS_DIR, 'valid.yml'), `
+`,
+		);
+
+		await expect(loadConfig(TEST_DIR)).rejects.toThrow(
+			/fail_fast can only be used when parallel is false/,
+		);
+	});
+
+	it("should accept check gate with fail_fast when parallel is false", async () => {
+		// Clean up the invalid file first
+		try {
+			await fs.unlink(path.join(CHECKS_DIR, "invalid.yml"));
+		} catch {}
+
+		await fs.writeFile(
+			path.join(CHECKS_DIR, "valid.yml"),
+			`
 name: valid
 command: echo test
 parallel: false
 fail_fast: true
-`);
-
-    const config = await loadConfig(TEST_DIR);
-    expect(config.checks['valid']).toBeDefined();
-    expect(config.checks['valid'].fail_fast).toBe(true);
-    expect(config.checks['valid'].parallel).toBe(false);
-  });
+`,
+		);
+
+		const config = await loadConfig(TEST_DIR);
+		expect(config.checks.valid).toBeDefined();
+		expect(config.checks.valid.fail_fast).toBe(true);
+		expect(config.checks.valid.parallel).toBe(false);
+	});
 });