Skip to content

Clarify agent git rules: implement ≠ full workflow, ask before new PR#20

Merged
pgarr merged 2 commits into
masterfrom
clarify-agent-git-rules
May 25, 2026
Merged

Clarify agent git rules: implement ≠ full workflow, ask before new PR#20
pgarr merged 2 commits into
masterfrom
clarify-agent-git-rules

Conversation

@pgarr
Copy link
Copy Markdown
Owner

@pgarr pgarr commented May 25, 2026

Summary

  • Replaces the vague "never commit/push unless explicitly asked" rule with two concrete, unambiguous statements:
    • "Implement X" means write the code only — no branching, committing, bumping, pushing, or PR opening until explicitly told
    • Before opening a PR, ask whether the work belongs in an existing open branch

Why

Two recurring violations in PRs #18/#19:

  • CI workflow addition was committed and pushed immediately after being asked to "implement" it, without waiting for explicit git instructions
  • The CI change was opened as a new PR instead of being added to the already-open PR Add ListForm and WidgetDbHelper tests #18

🤖 Generated with Claude Code

pgarr and others added 2 commits May 25, 2026 18:49
Two recurring mistakes:
- agent commits/pushes after being asked to "implement X", treating the
  request as permission for the full git workflow
- agent opens a new PR when an existing open branch would be the right
  target

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@pgarr pgarr merged commit 8ad9c25 into master May 25, 2026
3 checks passed
@pgarr pgarr deleted the clarify-agent-git-rules branch May 25, 2026 16:59
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