[claude] Fix empty commits: require git add before commit in Kimi workflow (#1268) #1288

Merged
claude merged 1 commits from claude/issue-1268 into main 2026-03-24 01:48:35 +00:00

1 Commits

Author SHA1 Message Date
Alexander Whitestone
90b3bbe7be fix: require git add before commit in Kimi workflow to prevent empty PRs
Some checks failed
Tests / lint (pull_request) Failing after 34s
Tests / test (pull_request) Has been skipped
Kimi was creating empty commits because the prompt did not explicitly
require `git add` before `git commit`. Empty PRs (0 files changed) waste
review cycles and leave issues appearing worked-on but unresolved.

In-repo fix (.kimi/AGENTS.md):
- Add CRITICAL note requiring `git add .` or `git add <files>` before any
  `git commit`
- Require `git diff --cached --stat` check to confirm non-empty staging
  area before committing
- Explicit warning: NEVER commit without staging files first

Harness fix (kimi-loop.sh, applied to ~/.hermes/bin/ and hermes-config/):
- Updated build_prompt() step 3 to show `git add .` + diff check before
  the commit command
- Added "CRITICAL: ALWAYS COMMIT AND PUSH" rule block to the prompt
- Added PR diff guard in the auto-merge step: checks Gitea /files API
  before merging; if 0 files changed, closes the empty PR, comments on
  the issue, adds to skip list for 2h so the issue retries cleanly

Fixes #1268

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-03-23 21:47:15 -04:00