[claude] Starting commit message generation

lefthook.yml

@@ -14,6 +14,9 @@ prepare-commit-msg:
   skip:
     - merge
     - rebase
-  jobs:
-    - run: ./dist/cli.js --message-only > {1}
+  commands:
+    commitment:
+      # Only run for regular commits (not merge, squash, or when message specified)
+      # {1} is the commit message file, {2} is the commit source
+      run: '[ -z "{2}" ] && ./dist/cli.js --message-only > {1} || true'
       interactive: true

src/agents/__tests__/base-agent.unit.test.ts

@@ -7,6 +7,7 @@ mock.module('../../utils/shell.js', () => ({
   exec: mockExec,
 }));
 
+import { SilentLogger } from '../../utils/logger';
 import { BaseAgent } from '../base-agent.js';
 import type { Agent } from '../types.js';
 
@@ -22,6 +23,10 @@ class TestAgent extends BaseAgent {
   // Track calls to verify execution order
   public callOrder: string[] = [];
 
+  constructor() {
+    super(new SilentLogger());
+  }
+
   // Make executeCommand public for testing (override access modifier)
   public async executeCommand(_prompt: string, _workdir: string): Promise<string> {
     this.callOrder.push('executeCommand');
@@ -52,6 +57,10 @@ class TestAgent extends BaseAgent {
 class CustomCleanAgent extends BaseAgent {
   readonly name = 'CustomCleanAgent';
 
+  constructor() {
+    super(new SilentLogger());
+  }
+
   // Make executeCommand public for testing
   public async executeCommand(_prompt: string, _workdir: string): Promise<string> {
     return '[CUSTOM]feat: test\n\nDescription';
@@ -71,6 +80,10 @@ class CustomCleanAgent extends BaseAgent {
 class CustomValidateAgent extends BaseAgent {
   readonly name = 'CustomValidateAgent';
 
+  constructor() {
+    super(new SilentLogger());
+  }
+
   // Make executeCommand public for testing
   public async executeCommand(_prompt: string, _workdir: string): Promise<string> {
     return 'feat: test';

src/agents/base-agent.ts

@@ -1,3 +1,4 @@
+import type { Logger } from '../utils/logger';
 import { exec } from '../utils/shell.js';
 import { cleanAIResponse, validateConventionalCommit } from './agent-utils';
 import type { Agent } from './types';
@@ -65,6 +66,11 @@ import type { Agent } from './types';
  * ```
  */
 export abstract class BaseAgent implements Agent {
+  /**
+   * Optional logger for debugging and diagnostics
+   */
+  logger?: Logger;
+
   /**
    * Human-readable name of the agent
    *
@@ -74,6 +80,15 @@ export abstract class BaseAgent implements Agent {
    */
   abstract readonly name: string;
 
+  /**
+   * Constructor accepting optional logger
+   *
+   * @param logger - Optional logger for debugging
+   */
+  constructor(logger?: Logger) {
+    this.logger = logger;
+  }
+
   /**
    * Template method for generating commit messages
    *
@@ -92,17 +107,27 @@ export abstract class BaseAgent implements Agent {
    * @throws {Error} If CLI not found, execution fails, or validation fails
    */
   async generate(prompt: string, workdir: string): Promise<string> {
+    this.logger?.debug(`[${this.name}] Starting commit message generation`);
+
     // Step 1: Check CLI availability
+    this.logger?.debug(`[${this.name}] Checking CLI availability`);
     await this.checkAvailability(this.name, workdir);
+    this.logger?.debug(`[${this.name}] CLI is available`);
 
     // Step 2: Execute agent-specific command
+    this.logger?.debug(`[${this.name}] Executing command`);
     const rawOutput = await this.executeCommand(prompt, workdir);
+    this.logger?.debug(`[${this.name}] Command executed, output length: ${rawOutput.length}`);
 
     // Step 3: Clean response (remove artifacts, normalize whitespace)
+    this.logger?.debug(`[${this.name}] Cleaning response`);
     const cleanedOutput = this.cleanResponse(rawOutput);
+    this.logger?.debug(`[${this.name}] Response cleaned, length: ${cleanedOutput.length}`);
 
     // Step 4: Validate response (check conventional commit format)
+    this.logger?.debug(`[${this.name}] Validating response format`);
     this.validateResponse(cleanedOutput);
+    this.logger?.debug(`[${this.name}] Response validated successfully`);
 
     return cleanedOutput;
   }

src/agents/factory.ts

@@ -1,4 +1,5 @@
 import { match } from 'ts-pattern';
+import type { Logger } from '../utils/logger';
 
 import { ClaudeAgent } from './claude';
 import { CodexAgent } from './codex';
@@ -13,6 +14,7 @@ import type { Agent, AgentName } from './types';
  * to the type but not handled here, TypeScript will error.
  *
  * @param name - The agent name ('claude', 'codex', or 'gemini')
+ * @param logger - Optional logger for debugging
  * @returns Agent instance for the specified name
  *
  * @example
@@ -21,10 +23,17 @@ import type { Agent, AgentName } from './types';
  * const message = await agent.generate(prompt, workdir);
  * ```
  */
-export function createAgent(name: AgentName): Agent {
-  return match(name)
+export function createAgent(name: AgentName, logger?: Logger): Agent {
+  const agent = match(name)
     .with('claude', () => new ClaudeAgent())
     .with('codex', () => new CodexAgent())
     .with('gemini', () => new GeminiAgent())
     .exhaustive();
+
+  // Set logger on agent if provided
+  if (logger) {
+    agent.logger = logger;
+  }
+
+  return agent;
 }

src/agents/types.ts

@@ -1,4 +1,5 @@
 import { z } from 'zod';
+import type { Logger } from '../utils/logger';
 
 /**
  * Core Agent interface for AI-powered commit message generation
@@ -49,6 +50,11 @@ export type Agent = {
    */
   generate(prompt: string, workdir: string): Promise<string>;
 
+  /**
+   * Optional logger for debugging and diagnostics
+   */
+  logger?: Logger;
+
   /**
    * Human-readable name of the agent (e.g., "Claude CLI", "Codex CLI")
    */

src/cli.ts

@@ -16,6 +16,7 @@ import {
 import { formatValidationError, validateCliOptions } from './cli/schemas';
 import { CommitMessageGenerator } from './generator';
 import type { GitStatus } from './utils/git-schemas';
+import { ConsoleLogger, SilentLogger } from './utils/logger';
 
 // Read version from package.json
 const Filename = fileURLToPath(import.meta.url);
@@ -37,37 +38,38 @@ async function generateCommitCommand(rawOptions: {
   const agentName = options.agent ?? 'claude';
   const quiet = options.quiet === true;
 
+  // Create logger based on --quiet flag
+  const logger = quiet ? new SilentLogger() : new ConsoleLogger();
+
   try {
-    const gitStatus = await checkGitStatusOrExit(options.cwd);
-    displayStagedChanges(gitStatus, options.messageOnly === true);
-    displayGenerationStatus(agentName, quiet);
+    const gitStatus = await checkGitStatusOrExit(options.cwd, logger);
+    displayStagedChanges(gitStatus, options.messageOnly === true, logger);
+    displayGenerationStatus(agentName, logger);
 
     const task = {
       description: 'Analyze git diff to generate appropriate commit message',
       produces: gitStatus.stagedFiles,
       title: 'Code changes',
     };
 
+    // Pass logger to Generator via config
     const generator = new CommitMessageGenerator({
       agent: agentName,
-      logger: {
-        warn: (warningMessage: string) => {
-          console.error(chalk.yellow(`⚠️  ${warningMessage}`));
-        },
-      },
+      logger,
     });
 
     const message = await generator.generateCommitMessage(task, {
       files: gitStatus.stagedFiles,
       workdir: options.cwd,
     });
 
-    displayCommitMessage(message, options.messageOnly === true);
+    displayCommitMessage(message, options.messageOnly === true, logger);
     await executeCommit(
       message,
       options.cwd,
       options.dryRun === true,
-      options.messageOnly === true
+      options.messageOnly === true,
+      logger
     );
   } catch (error) {
     console.error(chalk.red('❌ Error:'), error instanceof Error ? error.message : String(error));
@@ -99,12 +101,15 @@ function validateOptionsOrExit(
  *
  * Returns a simplified GitStatus-like object with just the fields we need
  */
-async function checkGitStatusOrExit(cwd: string): Promise<GitStatus> {
+async function checkGitStatusOrExit(
+  cwd: string,
+  logger: ConsoleLogger | SilentLogger
+): Promise<GitStatus> {
   const gitStatus = await getGitStatus(cwd);
 
   if (!gitStatus.hasChanges) {
-    console.log(chalk.yellow('No staged changes to commit'));
-    console.log(chalk.gray('Run `git add` to stage changes first'));
+    logger.warn('No staged changes to commit');
+    logger.info('Run `git add` to stage changes first');
     process.exit(1);
   }
 
@@ -132,11 +137,16 @@ prog
       'hook-manager'?: 'husky' | 'simple-git-hooks' | 'plain';
       agent?: 'claude' | 'codex' | 'gemini';
     }) => {
-      await initCommand({
-        agent: options.agent,
-        cwd: options.cwd,
-        hookManager: options['hook-manager'],
-      });
+      // Init command always uses console logger (never quiet)
+      const logger = new ConsoleLogger();
+      await initCommand(
+        {
+          agent: options.agent,
+          cwd: options.cwd,
+          hookManager: options['hook-manager'],
+        },
+        logger
+      );
     }
   );