You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
docs: add explicit git workflow policy to CLAUDE.md
- Never commit/push without explicit user permission
- Permission does not carry over between change sets
- Always wait for user to test before committing
- Especially important for DashSync as it affects downstream projects
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copy file name to clipboardExpand all lines: CLAUDE.md
+37-4Lines changed: 37 additions & 4 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -162,10 +162,43 @@ Supports 15+ languages: en, de, es, ja, zh-Hans, zh-Hant-TW, uk, bg, el, it, cs,
162
162
163
163
## Development Workflow
164
164
165
-
### Commit Policy
166
-
-**DO NOT commit changes until the user has tested them**
167
-
- Wait for explicit approval before creating commits
168
-
- This applies to all code changes, especially logging and behavioral modifications
165
+
### Git Workflow Policy
166
+
167
+
**NEVER commit or push changes without explicit user permission.**
168
+
169
+
When the user asks you to make code changes:
170
+
1. Make the requested changes to the code
171
+
2. Show what was changed (using `git diff` or explanation)
172
+
3.**STOP and WAIT** for explicit permission to commit/push
173
+
4. Only commit/push when the user explicitly says to do so
174
+
175
+
**Example phrases that give permission to commit/push:**
176
+
- "commit these changes"
177
+
- "push to github"
178
+
- "create a commit and push"
179
+
- "commit and push all changes"
180
+
181
+
**Do NOT commit/push** just because the user asked for code changes. They may want to test first.
182
+
183
+
#### Permission Does NOT Carry Over
184
+
185
+
**Each set of changes requires its own explicit permission.** If the user gave permission to commit earlier in the conversation, that permission applies ONLY to those specific changes - NOT to any subsequent changes.
186
+
187
+
**Example scenario:**
188
+
1. User: "Fix bug X, then commit and push" → Permission granted for bug X fix only
189
+
2. User: "Now fix bug Y" → Make the fix, show diff, **STOP AND WAIT** - no permission to commit yet
190
+
3. User: "Looks good, commit it" → NOW permission is granted for bug Y fix
191
+
192
+
**Common mistake to avoid:** After completing a task like "address review comments" or "fix these issues", do NOT automatically commit. The user needs to test the changes first. Always pause after showing the diff and wait for explicit commit instruction.
193
+
194
+
#### Testing Before Commit
195
+
196
+
This is especially important for DashSync because:
197
+
- Changes affect downstream projects (dashwallet-ios, other consumers)
198
+
- Logging and behavioral changes need runtime verification
199
+
- The user must run the code to confirm changes work as expected
200
+
201
+
Always wait for the user to confirm they have tested and verified the changes before committing.
0 commit comments