Skip to content

Commit c00117a

Browse files
authored
docs: require codex review in contributing guide (openclaw#42503)
1 parent 5337439 commit c00117a

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

CONTRIBUTING.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,7 @@ Welcome to the lobster tank! 🦞
8686

8787
- Test locally with your OpenClaw instance
8888
- Run tests: `pnpm build && pnpm check && pnpm test`
89+
- If you have access to Codex, run `codex review --base origin/main` locally before opening or updating your PR. Treat this as the current highest standard of AI review, even if GitHub Codex review also runs.
8990
- Ensure CI checks pass
9091
- Keep PRs focused (one thing per PR; do not mix unrelated concerns)
9192
- Describe what & why
@@ -99,6 +100,8 @@ If a review bot leaves review conversations on your PR, you are expected to hand
99100
- Resolve the conversation yourself once the code or explanation fully addresses the bot's concern
100101
- Reply and leave it open only when you need maintainer or reviewer judgment
101102
- Do not leave "fixed" bot review conversations for maintainers to clean up for you
103+
- If Codex leaves comments, address every relevant one or resolve it with a short explanation when it is not applicable to your change
104+
- If GitHub Codex review does not trigger for some reason, run `codex review --base origin/main` locally anyway and treat that output as required review work
102105

103106
This applies to both human-authored and AI-assisted PRs.
104107

@@ -127,6 +130,7 @@ Please include in your PR:
127130
- [ ] Note the degree of testing (untested / lightly tested / fully tested)
128131
- [ ] Include prompts or session logs if possible (super helpful!)
129132
- [ ] Confirm you understand what the code does
133+
- [ ] If you have access to Codex, run `codex review --base origin/main` locally and address the findings before asking for review
130134
- [ ] Resolve or reply to bot review conversations after you address them
131135

132136
AI PRs are first-class citizens here. We just want transparency so reviewers know what to look for. If you are using an LLM coding agent, instruct it to resolve bot review conversations it has addressed instead of leaving them for maintainers.

0 commit comments

Comments
 (0)