@@ -23,7 +23,7 @@ Use when:
2323- Prefer small fixes at the right ownership boundary; no refactor unless it clearly improves the bug class.
2424- Keep going until the selected review path returns no accepted/actionable findings.
2525- If a review-triggered fix changes code, rerun focused tests and rerun the review helper.
26-- Default to Codex review. If Codex is unavailable or exits with an error, the helper falls back to the first configured CLI from `claude -p`, `pi -p`, `opencode run`, `droid exec`, or `copilot`. Prefer Codex for final closeout because it uses native review mode; non-Codex reviewers use a Codex-inspired generated diff prompt. The helper runs nested Codex review in yolo/full-access mode by default; use `--no-yolo` only when intentionally testing sandbox behavior.
26+- Default to Codex review with no fallback. Prefer Codex for final closeout because it uses native review mode; non-Codex reviewers use a Codex-inspired generated diff prompt. Use `--fallback-reviewer auto|claude|pi|opencode|droid|copilot` only when a second-model fallback is explicitly wanted and authenticated. The helper runs nested Codex review in yolo/full-access mode by default; use `--no-yolo` only when intentionally testing sandbox behavior.
2727- Stop as soon as the review command/helper exits 0 with no accepted/actionable findings. Do not run an extra direct `codex review` just to get a nicer "clean" line, a second opinion, or clearer closeout wording.
2828- Treat the helper's successful exit plus absence of actionable findings as the clean review result, even if the underlying Codex CLI output is terse.
2929- If rejecting a finding as intentional/not worth fixing, add a brief inline code comment only when it explains a real invariant or ownership decision that future reviewers should know.
@@ -52,11 +52,11 @@ git fetch origin
5252codex review --base origin/main
5353```
545455-Do not pass any prompt with `--base`. Some Codex CLI versions reject both inline
56-and stdin prompt forms, including the helper's `codex review --base <ref> -`,
57-with `--base <BRANCH> cannot be used with [PROMPT]`. If the helper hits this
58-error, run plain `codex review --base <ref>` and report that the helper prompt
59-injection was skipped.
55+Do not pass any prompt with `--base`, `--commit`, or `--uncommitted`. Codex CLI
56+review targets and custom review prompts are mutually exclusive: target modes
57+generate their own review prompt internally. Use plain target review for native
58+Codex closeout, or use custom prompt review (`codex review -`) only when you
59+intentionally want a generated diff prompt instead of native target review.
60606161If an open PR exists, use its actual base:
6262@@ -117,13 +117,13 @@ The helper:
117117- use `--mode commit --commit <ref>` for already-committed work, especially clean `main` after landing
118118- should be left in `--mode auto` or forced to `--mode branch` for PR/branch work; do not force `--mode local` after committing
119119- supports `--reviewer codex|claude|pi|opencode|droid|copilot|auto`; `auto` means Codex first
120-- supports `--fallback-reviewer auto|claude|pi|opencode|droid|copilot|none`; default is configured CLI fallback
120+- supports `--fallback-reviewer auto|claude|pi|opencode|droid|copilot|none`; default is `none`
121121- falls back only when Codex is unavailable or exits nonzero, not when Codex reports findings
122122- writes only to stdout unless `--output` or `AUTOREVIEW_OUTPUT` is set
123123- supports `--dry-run`, `--parallel-tests`, and commit refs
124124- runs nested review with `--dangerously-bypass-approvals-and-sandbox --sandbox danger-full-access` by default
125-- injects maintainer-only OpenClaw validation policy into native Codex review when `OPENCLAW_TESTBOX=1` or `AUTOREVIEW_OPENCLAW_MAINTAINER_VALIDATION=1`, so local memory-heavy Node/Vitest checks are avoided in favor of Crabbox/Testbox proof
126-- branch mode may fail on Codex CLI versions that reject `--base` plus the helper's stdin prompt; on that exact parser error, rerun plain `codex review --base <ref>` instead of falling back to a non-Codex reviewer
125+- with `OPENCLAW_TESTBOX=1` or `AUTOREVIEW_OPENCLAW_MAINTAINER_VALIDATION=1`, disables auto local `pnpm run check` and routes Codex through generated prompt review (`codex review -`) so the no-local-heavy-tests policy is included; native Codex target review cannot accept extra prompt text
126+- non-Codex reviewers receive the generated diff prompt and maintainer validation policy text when maintainer validation is active
127127- keeps accepting `--full-access`; use `--no-yolo` or `AUTOREVIEW_YOLO=0` to opt out
128128- still accepts legacy `CODEX_REVIEW_*` env vars when the matching `AUTOREVIEW_*` var is unset
129129- prints `autoreview clean: no accepted/actionable findings reported` when the selected review command exits 0
此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。 原文来自 — 版权归原作者所有。