Skip to content

Commit a4c5051

Browse files
Add explicit approval gates for discussion and issue comments
1 parent 104daec commit a4c5051

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

AGENTS.md

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,12 +29,13 @@ It captures practical rules that prevent avoidable CI and PR churn.
2929
2. Create a branch prefixed with `codex/`.
3030
3. Implement scoped changes only.
3131
4. Run required checks: `npm run format`, `npm run lint`, and targeted tests.
32-
5. Verify git diff only contains intended files.
32+
5. Verify git diff only contains intended files. If changes are still being discussed, share the diff and get user approval before committing or pushing.
3333
6. Commit with focused message(s), using `git commit --no-verify`.
34-
7. Push branch.
34+
7. Push branch. Ask for explicit user permission before any force push.
3535
8. Open PR against `main` using a human-readable title (no `feat(...)` / `fix(...)` prefixes).
36-
9. Add labels for both change type and semantic version impact.
37-
10. Ensure PR body includes:
36+
9. Before posting any comment on GitHub issues or PRs, share the proposed message with the user and get explicit approval.
37+
10. Add labels for both change type and semantic version impact.
38+
11. Ensure PR body includes:
3839
- summary of changes
3940
- verification commands run
4041
- test results summary

0 commit comments

Comments
 (0)