Skip to content

feat(suggest): improve commit message editor configurability #26

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
artback wants to merge 3 commits into
base: main
Choose a base branch
from
Open

feat(suggest): improve commit message editor configurability #26

artback wants to merge 3 commits into from

Conversation

@artback
Copy link
Contributor

@artback artback commented Dec 22, 2025 •
edited
Loading

To be merged after #25

  • Introduce command variable to allow mocking exec.Command calls for testing.
  • Update git package functions to utilize the mockable command variable.
  • Add comprehensive unit tests for GetStatusForFiles, GetChangedFiles, GetChangesForFiles, Commit, and Push.
  • Add unit tests for GetGitRoot.
  • Refactor GetStatusForFiles by introducing filterStatusLines for clearer logic.
  • Remove unused GetDiff and GetStatus functions.

close #20
@artback
refactor(git): Improve testability of git command execution
7d5dcf3 
- Introduce `command` variable to allow mocking `exec.Command` calls for testing. - Update `git` package functions to utilize the mockable `command` variable. - Add comprehensive unit tests for `GetStatusForFiles`, `GetChangedFiles`, `GetChangesForFiles`, `Commit`, and `Push`. - Add unit tests for `GetGitRoot`. - Refactor `GetStatusForFiles` by introducing `filterStatusLines` for clearer logic. - Remove unused `GetDiff` and `GetStatus` functions.
@artback
feat(suggest): Implement edit and regenerate functionality for AI sug…
4677f6f 
…gestions - Add support for internal and external editors for commit messages. - Introduce `suggest.editor` configuration option and `--editor` flag. - Allow editing (`e`) and regenerating (`r`) AI suggestions within the TUI. - Update README to reflect new features and configuration.
@artback
feat(suggest): improve commit message editor configurability
39332dc 
- Reworked editor option 'external' to 'system' to reflect OS environment variables. - Renamed TUI editor option from 'internal' to 'builtin'. - Enabled direct command input for `--editor` flag and config, allowing any custom editor. - Updated documentation and default values for editor options.
@artback artback changed the title refactor(git): Improve testability of git command execution feat(suggest): improve commit message editor configurability Dec 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@artback