feat: add commit command with AI-generated messages by laststylebender14 · Pull Request #1730 · tailcallhq/forgecode

laststylebender14 · 2025-10-14T07:34:19Z

closes: #1728

forge commit --max-diff 10000   // Limit git diff to 10KB 
forge commit  // No limits on git diff

Without --preview: Analyzes your git diff, generates an AI commit message, and commits immediately
With --preview: Shows the proposed commit message for review without committing

Add configurable max_diff_size parameter to generate_commit_message to control git diff truncation. This prevents sending excessively large diffs to AI models, reducing token usage and API costs. Includes comprehensive documentation and truncation notices in the prompt context.

Execute git log, branch name, and diff commands concurrently using tokio::join! to reduce latency. Also parallelize template rendering and provider/model retrieval. Add branch name to commit message prompt context.

Replace default value of 0 with None for unlimited diff size in commit message generation. Update API signatures, documentation, and truncation logic to use Option<usize> for clearer semantics.

Move commit message generation logic from ForgeApp to new GitApp class to improve separation of concerns and modularity. This change isolates git-related functionality and makes the codebase more maintainable.

Allow commit command to work with unstaged changes as fallback when no files are staged. Add parallel fetching of both staged and unstaged diffs, then use staged if available, otherwise unstaged. Update commit execution to use `-a` flag when no files are staged to include all tracked changes.

…es detection Replace `.success()` with explicit exit code comparison to handle the specific behavior where exit code 1 indicates staged files exist, avoiding reliance on the general success check which may not accurately reflect the git diff-index semantics.

Add a new `/commit` CLI command that generates AI-powered commit messages with flexible syntax support. The command accepts optional `preview` flag to display the message without committing, and `max-diff-size` parameter to limit the diff size sent to the AI model.

…gration

Replace heredoc-based commit message passing with direct -m flag for cleaner command execution and improved shell compatibility.

tusharmath · 2025-11-03T05:58:57Z

FIXME

Commits produced don't follow and standards (need evals perhaps)
From ZSH the loader looses colors
forge commit command doesn't hide the loader after completion.
/commit should insert the commit message in the buffer using ! git commit ...

…l command output - Remove `preview` flag from `/commit` command - always preview message - Pre-fill editor buffer with `git commit` command for user to execute - Refactor Console to maintain persistent ForgeEditor instance - Add `set_buffer()` method to ForgeEditor and Console for pre-filling input - Update commit command handling to generate shell command with proper flags - Simplify error handling by using `to_string()` instead of debug formatting - Update tests to reflect removal of preview parameter

- Remove unused GitAppError import - Simplify error display formatting by removing intermediate variable - Remove special case handling for NoChangesToCommit error - Fix line length formatting for git commit command

… for commit messages

feat: add commit command with AI-generated messages

f81de9e

github-actions Bot added the type: feature Brand new functionality, features, pages, workflows, endpoints, etc. label Oct 14, 2025

laststylebender14 and others added 8 commits October 14, 2025 13:05

fix: use action format for commit success message

b41ec6b

feat: add generate_commit_message method to API interface

5a19c97

[autofix.ci] apply automated fixes

790b0a5

refactor: move commit message prompt data to user message with XML tags

23f8074

fix: disable shell output for git commands in commit message generation

cc75877

refactor: improve commit message prompt requirements and clarity

4d51abd

fix: simplify commit handler response to return only message

4981767

laststylebender14 force-pushed the feat/add-commit-cli-commands branch from 4f30aeb to 36c07b4 Compare October 15, 2025 07:38

Merge branch 'main' into feat/add-commit-cli-commands

50203b8

laststylebender14 marked this pull request as ready for review October 15, 2025 07:44

laststylebender14 marked this pull request as draft October 15, 2025 07:45

feat: parallelize git operations in commit message generation

4a8744d

Execute git log, branch name, and diff commands concurrently using tokio::join! to reduce latency. Also parallelize template rendering and provider/model retrieval. Add branch name to commit message prompt context.

laststylebender14 marked this pull request as ready for review October 15, 2025 07:51

laststylebender14 and others added 5 commits October 15, 2025 13:25

refactor: change max_diff_size parameter from usize to Option<usize>

09b6713

Replace default value of 0 with None for unlimited diff size in commit message generation. Update API signatures, documentation, and truncation logic to use Option<usize> for clearer semantics.

[autofix.ci] apply automated fixes

95839e0

feat: add minimum value validation for max-diff-size parameter

a635f6e

refactor: simplify conventional commit format requirement in prompt

9450c80

refactor: simplify commit message prompt to skip preamble

b492c0b

tusharmath reviewed Oct 15, 2025

View reviewed changes

Comment thread crates/forge_api/src/forge_api.rs Outdated

tusharmath reviewed Oct 15, 2025

View reviewed changes

Comment thread crates/forge_app/src/app.rs Outdated

tusharmath reviewed Oct 15, 2025

View reviewed changes

Comment thread templates/forge-commit-message-prompt.md Outdated

laststylebender14 and others added 6 commits October 15, 2025 14:02

refactor: extract git commit message generation into dedicated GitApp

aa1c3f3

Move commit message generation logic from ForgeApp to new GitApp class to improve separation of concerns and modularity. This change isolates git-related functionality and makes the codebase more maintainable.

[autofix.ci] apply automated fixes

dec312b

refactor: rename staged_files variable to has_staged_files for clarity

b01e445

tusharmath added 2 commits November 3, 2025 11:26

chore: update GitApp and ForgeAPI structure for improved service inte…

c03e425

…gration

fix: simplify git commit command by using -m flag instead of heredoc

90618bc

Replace heredoc-based commit message passing with direct -m flag for cleaner command execution and improved shell compatibility.

tusharmath reviewed Nov 3, 2025

View reviewed changes

Comment thread crates/forge_spinner/src/lib.rs Outdated

laststylebender14 added 4 commits November 3, 2025 13:04

- rename to ewrite_ln

dfea6b7

fix: remove trailing whitespace from commit command comment

c67feac

refactor: clean up error handling and formatting in UI module

49a0b00

- Remove unused GitAppError import - Simplify error display formatting by removing intermediate variable - Remove special case handling for NoChangesToCommit error - Fix line length formatting for git commit command

laststylebender14 marked this pull request as draft November 3, 2025 09:15

laststylebender14 and others added 13 commits November 4, 2025 20:03

- retain color for commit spinner message

b1b576b

- improve prompt and add parser for commit message.

1007eca

refactor(commit): improve prompt structure and use XML tag extraction…

bd21722

… for commit messages

feat(commit): add support for piped diff input in commit command

bc65060

Merge branch 'main' into feat/add-commit-cli-commands

4f4ff9d

refactor(commit): remove commit command from list subcommand

d02a8ca

fix(tests): import TopLevelCommand for test module

a77de9c

[autofix.ci] apply automated fixes

1befa1a

- reduce duplication

1826b6a

- tighten the trait bounds

1734869

refactor(services): remove unused AppConfigService import

80eb3e8

refactor(commit): simplify git commit command format to use -m flag

1d78bbc

refactor(commit): move preview output after command completion

bca5688

laststylebender14 marked this pull request as ready for review November 11, 2025 09:45

laststylebender14 and others added 3 commits November 11, 2025 15:16

Merge branch 'main' into feat/add-commit-cli-commands

ff353df

revert: add check back

f0141f7

[autofix.ci] apply automated fixes

8203746

tusharmath merged commit 159a372 into main Nov 12, 2025
9 checks passed

tusharmath deleted the feat/add-commit-cli-commands branch November 12, 2025 05:06

tusharmath added a commit that referenced this pull request Nov 18, 2025

feat(commit): set default max-diff to 100k bytes (#1730)

5778837

tusharmath added a commit that referenced this pull request Nov 18, 2025

feat(commit): set default max-diff to 100k bytes (#1730) (#1918)

ce1bf00

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: add commit command with AI-generated messages#1730

feat: add commit command with AI-generated messages#1730
tusharmath merged 94 commits intomainfrom
feat/add-commit-cli-commands

laststylebender14 commented Oct 14, 2025 •

edited by tusharmath

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

tusharmath commented Nov 3, 2025 •

edited by laststylebender14

Loading

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

laststylebender14 commented Oct 14, 2025 • edited by tusharmath Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

tusharmath commented Nov 3, 2025 • edited by laststylebender14 Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

laststylebender14 commented Oct 14, 2025 •

edited by tusharmath

Loading

tusharmath commented Nov 3, 2025 •

edited by laststylebender14

Loading