Skip to content

🤖 Generating commit message with claude...#77

Merged
arittr merged 3 commits into8a076f-task-2-generator-agent-integrationfrom
8a076f-task-3-cli-integration
Nov 3, 2025
Merged

🤖 Generating commit message with claude...#77
arittr merged 3 commits into8a076f-task-2-generator-agent-integrationfrom
8a076f-task-3-cli-integration

Conversation

@arittr
Copy link
Owner

@arittr arittr commented Nov 3, 2025

refactor: integrate logger interface throughout CLI layer

  • Replace direct console.log/error calls with logger interface in CLI entry point
  • Pass logger instances to helper functions (displayStagedChanges, displayGenerationStatus, displayCommitMessage, executeCommit)
  • Create logger in main command based on --quiet flag (ConsoleLogger vs SilentLogger)
  • Update init command to always use ConsoleLogger (never quiet)
  • Simplify helper unit tests to use SilentLogger instead of mocking console
  • Update checkGitStatusOrExit to use logger.warn/info instead of console

🤖 Generated with Claude via commitment

refactor: integrate logger interface throughout CLI layer

- Replace direct console.log/error calls with logger interface in CLI entry point
- Pass logger instances to helper functions (displayStagedChanges, displayGenerationStatus, displayCommitMessage, executeCommit)
- Create logger in main command based on --quiet flag (ConsoleLogger vs SilentLogger)
- Update init command to always use ConsoleLogger (never quiet)
- Simplify helper unit tests to use SilentLogger instead of mocking console
- Update checkGitStatusOrExit to use logger.warn/info instead of console

🤖 Generated with Claude via commitment
- Add logger parameter to ChatGPTAgent, MetaEvaluator, and SingleAttemptEvaluator constructors
- Pass logger through EvalRunner and AttemptRunner to dependent components
- Update all test files to inject SilentLogger into evaluator and runner instances
- Update run-eval.ts to instantiate EvalRunner with ConsoleLogger
- Remove direct console.log calls in favor of logger interface pattern

🤖 Generated with Claude via commitment
Implements:
- Updated prepare-commit-msg hook to check {2} parameter
- Hook only runs commitment when {2} is empty (regular commits)
- Preserves messages for git commit -m and merge commits
- Updated hook to use commands instead of jobs for consistency
- Added explanatory comments matching example hooks

Acceptance criteria met:
✅ lefthook.yml checks {2} parameter before running commitment
✅ git commit generates message (hook runs)
✅ git commit -m "test" preserves message (hook skips)
✅ Merge commits preserve messages (hook skips)
✅ commitment dogfooding still works
✅ Examples already correct (no changes needed)
@arittr arittr merged commit 5d929b5 into 8a076f-task-2-generator-agent-integration Nov 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant