Skip to content

Commit 286ce7f

Browse files
committed
fix: enforce commit-AND-push in all agent instructions
Commit without push is half the job. Updated CLAUDE.md, all templates (CLAUDE.md, CONVENTIONS.md, .cursorrules, .windsurfrules, copilot- instructions.md) to say 'commit AND push' everywhere.
1 parent 016d39f commit 286ce7f

6 files changed

Lines changed: 21 additions & 20 deletions

File tree

CLAUDE.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,12 +24,12 @@ commit messages, fix failing tests and violations. Same quality bar.
2424
**Every change to this framework MUST include — automatically, without the
2525
user asking:**
2626

27-
### 1. Commit often
27+
### 1. Commit and push often
2828

29-
**Commit after each logical unit of work** — a completed feature, a
30-
passing test, a doc update. Don't accumulate 50 files of uncommitted
31-
changes. Small focused commits with meaningful messages. No AI
32-
attribution trailers.
29+
**Commit AND push after each logical unit of work** — a completed
30+
feature, a passing test, a doc update. Don't accumulate uncommitted
31+
or unpushed changes. Small focused commits with meaningful messages.
32+
No AI attribution trailers. Always `git push` after committing.
3333

3434
### 2. Tests
3535

packages/cli/templates/.cursorrules

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,8 @@ Quality bar stays the same — just no blocking on questions.
3838

3939
## Git rules
4040

41-
- COMMIT OFTEN. After each logical unit of work — don't accumulate changes.
42-
Committing is automatic, the user should never have to ask.
41+
- COMMIT AND PUSH OFTEN. After each logical unit of work — don't accumulate
42+
changes. Always `git push` after committing. This is automatic.
4343
- Write meaningful commit messages: what changed and why, not "update files"
4444
- NEVER add "Co-Authored-By", "Generated by", "AI-assisted" or similar
4545
attribution trailers to commits

packages/cli/templates/.github/copilot-instructions.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,8 @@ If running without interactive approval (sandbox, auto-approve, etc.):
2222
## Mandatory workflow
2323

2424
Every code change must include:
25-
1. Commits — COMMIT OFTEN after each logical unit of work. Don't accumulate
26-
changes. Committing is automatic, the user should never have to ask.
25+
1. Commit and push — COMMIT AND PUSH after each logical unit of work.
26+
Always `git push` after committing. Don't accumulate changes. Automatic.
2727
2. Unit tests in test/unit/*.test.ts (server actions, components, utilities)
2828
3. E2E tests in test/e2e/*.test.ts for user-facing features (Puppeteer)
2929
4. Documentation updates (AGENTS.md for API, docs/ for user guides)

packages/cli/templates/.windsurfrules

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,8 @@ The user should never have to ask for tests or documentation.
3232

3333
## Git rules
3434

35-
- COMMIT OFTEN. After each logical unit of work — don't accumulate changes.
36-
Committing is automatic, the user should never have to ask.
35+
- COMMIT AND PUSH OFTEN. After each logical unit of work — don't accumulate
36+
changes. Always `git push` after committing. This is automatic.
3737
- Meaningful commit messages: what changed and why
3838
- NEVER add Co-Authored-By or AI attribution trailers to commits
3939
- Work on feature branches, never push directly to main

packages/cli/templates/CLAUDE.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -16,13 +16,13 @@ This file instructs AI coding agents on how to work in this project.
1616
**CRITICAL: Every code change MUST include the following — automatically,
1717
without the user having to ask:**
1818

19-
### 1. Commit often (mandatory, never skip)
19+
### 1. Commit and push often (mandatory, never skip)
2020

21-
**Commit after each logical unit of work** — a completed feature, a
22-
passing test, a doc update. Don't accumulate many files of uncommitted
23-
changes. Small focused commits with meaningful messages. No AI
24-
attribution trailers. Committing is automatic — the user should never
25-
have to ask.
21+
**Commit AND push after each logical unit of work** — a completed
22+
feature, a passing test, a doc update. Don't accumulate uncommitted
23+
or unpushed changes. Small focused commits with meaningful messages.
24+
No AI attribution trailers. Always `git push` after committing.
25+
This is automatic — the user should never have to ask.
2626

2727
### 2. Tests (mandatory, never skip)
2828

packages/cli/templates/CONVENTIONS.md

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,10 @@ even if the user doesn't explicitly ask.**
2626

2727
### Every code change must include:
2828

29-
1. **Commits** — Commit after each logical unit of work. Small, focused
30-
commits with meaningful messages. Don't accumulate dozens of changed
31-
files. Committing is automatic — the user should never have to ask.
29+
1. **Commit and push** — Commit AND push after each logical unit of work.
30+
Small, focused commits with meaningful messages. Always `git push`
31+
after committing. Don't accumulate uncommitted or unpushed changes.
32+
This is automatic — the user should never have to ask.
3233

3334
2. **Tests** — Unit test for logic, E2E test for user-facing behavior.
3435
See the "Testing" section below for what type of test each change needs.

0 commit comments

Comments
 (0)