← ClaudeAtlas

codex-reviewlisted

Run Codex code review as a Symphony closeout check: local dirty changes, PR branch vs main, parallel tests, and accepted-finding cleanup.
PedroAVJ/swe-stack · ★ 0 · Code & Development · score 70
Install: claude install-skill PedroAVJ/swe-stack
# Symphony Codex Review Run Codex's built-in code review as a closeout check. This is code review (`codex review`), not Guardian `auto_review` approval routing. Use when: - user asks for Codex review / autoreview / second-model review - after non-trivial code edits, before final/commit/ship - reviewing a local branch or PR branch after fixes ## Contract - Treat review output as advisory. Never blindly apply it. - Verify every finding by reading the real code path and adjacent files. - Read dependency docs/source/types when the finding depends on external behavior. - Reject unrealistic edge cases, speculative risks, broad rewrites, and fixes that over-complicate the codebase. - Prefer small fixes at the right ownership boundary; no refactor unless it clearly improves the bug class. - Keep going until Codex review returns no accepted/actionable findings. - If a review-triggered fix changes code, rerun focused tests and rerun Codex review. - 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. - 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. - If already running inside a `codex review` session, do not invoke `codex review` again. Perform a manual code-review pass over the provided diff/files instead and report findings