Auto-commit at workflow gates#26
Merged
Merged
Conversation
Make CDD sessions commit at their gate so none leaves a dirty tree for the next to inherit, while staying non-disruptive: a gate commits only its own changes, never pushes (except /process-pr), and stops-and-surfaces if it finds pre-existing dirty state it didn't author. - Process doc: new §2.11 "Commit conventions" (five rules), with §3.3, §3.5, §3.7/§4.1 and §4 cross-referencing it. §4 frames an auto-commit as a reversible local commit that adds no checkpoint and removes none. - /pre-pr (both copies): snapshot git status at entry; new step 8 commits the reconciliation edits locally (no push) or stops-and-surfaces a dirty entry tree; Open PR renumbered to step 9. - /next-step (both copies): second standing instruction in the generated handoff prompt so the implementation session reliably commits its own work. - Both CLAUDE.md /pre-pr bullets note the new auto-commit behavior. - Roadmap: add and tick the Phase 8 item. - Whitelist the two new free-form command tokens for the template smoke check. Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
drabaioli
commented
Jun 17, 2026
"Follow the commit conventions in CLAUDE.md" already covers the trailer; spelling out `Co-Authored-By` was redundant. Removed from next-step, pre-pr, and process-pr in both the repo and the template (drift-check clean). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Implements issue #20: sessions auto-commit their own changes at their gate so a session never leaves a dirty tree for the next one to inherit.
What changed
git add -A; stop-and-surface a dirty entry tree), local-only with/process-pras the sole auto-push exception, followCLAUDE.mdmessage conventions, surface a one-line summary, and an auto-commit is not a seventh checkpoint. Updates to §3.3 (implementation session), §3.5 (/pre-pr), §3.7 (/process-pr), and §4 (checkpoint count unchanged)./next-step: the generated handoff prompt now carries a standing instruction for the implementation session to commit its own changes locally./pre-pr: records a clean-tree entry snapshot in step 1; new step 8 commits reconciliation edits locally (old step 8 → step 9); checklist gains a "Reconciliation edits committed" line./pre-prworkflow bullet) and roadmap updated; bothtemplate/command copies kept in sync; two new placeholder tokens whitelisted inscripts/template-smoke-whitelist.txt.Verification
bash -nclean across scripts/tools/template/demoCloses #20