Afk codex review

Skill jimmy1992abc/afk/skills/afk-codex-review

Part of the afk pipeline. Runs Codex (OpenAI Codex CLI) as an independent, read-only external review gate on the current PR/branch, then triages and fixes the findings. Interchangeable with afk-kimi-review and afk-glm-review, subject to .afk/config.md gate priority and min-pass. The last check before a PR is handed back: internal review first, external gate last. Triggers include "/afk-codex-review", "run codex review", "codex gate".From its SKILL.md

Install

npx -y skills add jimmy1992abc/afk --skill afk-codex-review

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

One thing to look at

1 stars1 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.

SKILL.md

4.7 KB, as published. Nobody here has run it

afk-codex-review

An independent second-opinion review by Codex (a different model), used as the last check before a PR is handed back for approval. It is interchangeable with afk-kimi-review and afk-glm-review: run the configured external gate set from .afk/config.md, and never use a gate whose model matches the implementer's model. Run afk-internal-review first and resolve it, then run this gate: internal review first, external gate last. Codex reviews the diff read-only; you triage and fix.

The helper codex-gate.mjs ships with this skill and travels with the plugin.

Metering

Codex calls are metered — keep invocations to a minimum. Batch every finding into one fix pass, self-review, then re-run once; defer documentation and minor items to a single final pass. Never spend a round-trip on a small or doc-only edit.

Run it

The bundled helper codex-gate.mjs sits beside this SKILL.md. Locate its directory as ${CLAUDE_PLUGIN_ROOT}/skills/afk-codex-review if the env var is set, else <pluginRoot>/skills/afk-codex-review from .afk/config.md, else this skill's own directory (the helper is its sibling). Resolve .afk/ from the repository's main working tree — the first worktree line of git worktree list --porcelain — never the current directory, or a run from a linked worktree reads a different .afk/ than the one afk-init wrote. If .afk/ is absent, the afk-init bootstrap runs automatically first:

node "<helper-dir>/codex-gate.mjs"

Run it in the background with a generous timeout (the review traces code paths and may run tests); redirect stdout to a file and read it when it completes. Pass through any target flag (--base <branch> / --commit <sha> / --uncommitted; default = current branch vs the default branch). Do not poll in a sleep loop — wait for completion.

Design mode (--design <path>) reviews a design document's reasoning instead of a diff — the opt-in design-stage gate (see ../afk/SKILL.md, "Design-stage external gate"). Codex runs it with exec -s read-only and the brief + doc piped on stdin — never the review subcommand or the sandbox bypass, so it stays read-only on every OS. A missing or unreadable --design path fails loudly (ERROR, non-zero), never a skip.

Read the verdict between the ===== CODEX REVIEW (final message) ===== markers. SKIPPED: … (Codex absent, logged out, or disabled via CODEX_REVIEW_GATE=off) is not a failure — report it and continue. ERROR: … means the review itself failed — read the transcript it names; never report an errored run as clean.

Handle findings (batch — minimise calls)

Sort by kind. Structural (architecture, correctness, security, missed edge cases) — act on these. Minor (naming, cosmetics) — defer to one final pass.
Verify before trusting. Each finding is a hypothesis; the verification standard is below. Push back with evidence on anything you can disprove.
Fix every confirmed structural finding in one batch, and sweep for the same pattern elsewhere; keep specs in sync in the same change.
Self-review once over your fixes.
Re-run the gate once. Repeat until the stop rule holds.
Deferred pass once, at the end for the minor items — do not re-run the gate to confirm doc edits.

A structural finding claims both that the code is as described and that it goes wrong; reading the cited file:line settles only the first. Demonstrate the consequence before fixing, and account for every consumer of what you change that lives outside the diff — ../afk/SKILL.md ("External gate") holds both rules.

Apply any invariant in .afk/config.md as an extra must-check lens.

Stop rule

Stop when the loop-termination rule in ../afk/SKILL.md ("External gate") holds: a round with no new structural finding and every prior structural finding closed by a recorded disposition — a driver-verified fix, a refutation, or an accepted risk.

Report honestly: CLEAN, or OUTSTANDING with what remains. A clean pass is not authority to merge — hand back to the operator.

Setup (per machine, once)

Optional and self-skipping. npm i -g @openai/codex && codex login; needs Node + git on PATH. Disable with CODEX_REVIEW_GATE=off.