codex-review · diff

git:20260928.cf9620c to git:20261002.162238f

2 added, 48 removed. Audit A to A.

---
name: codex-review
- description: Get a second opinion from OpenAI Codex on your current code changes. Analyzes staged/unstaged diffs and returns prioritized findings. Use when user asks to "review with Codex", "Codex code review", or "ask Codex to check my code".
+ description: Earlier name of the ask-llm-review skill; use when the user invokes /codex-review.
---
- <!-- PORTABLE-CONTRACT:START -->
- ## Portable contract
-
- Gather the relevant staged, unstaged, and untracked code changes; build a bounded context brief; request a read-only Codex review at high reasoning effort; verify each reported finding against source; and return only prioritized, source-supported findings. Preserve the configured Sol-to-Terra fallback and disclose it.
- <!-- PORTABLE-CONTRACT:END -->
-
- ## Host adapters
-
- ### Pi adapter
-
- Call the native `ask-codex` tool with `reasoningEffort: "high"` and apply only the `Portable contract` section of `../../agents/codex-reviewer.md`. The review runs inline in the current host context; do not claim subagent isolation.
-
- <!-- HOST-ADAPTER:CLAUDE-CODE:START -->
- ### Claude Code adapter
-
- The existing detailed workflow below is the Claude Code adapter. Its Agent, MCP, hook, `CLAUDE_PLUGIN_ROOT`, and `AskUserQuestion` mechanics apply only on Claude Code; they do not override the Pi adapter above.
-
-
-
- # Codex Code Review
-
- Review current code changes by delegating to the `codex-reviewer` agent.
-
- ## When to use this vs codex-pair
-
- This skill is **precision-first**: confidence ≥ 80, "don't flag style/linter-catchable" filter. Optimized for low false-alarm rate on diff review — appropriate default for routine PRs.
-
- For **recall-first** review on hot-path code (money handling, security paths, spec-implementing code), see the [`codex-pair`](../codex-pair/SKILL.md) sibling — a PostToolUse hook that opts in per-project via a `.codex-pair/context.md` marker file. The 4-task benchmark in ADR-077 shows codex-pair catches domain-level "wrong but won't crash" issues (float-money precision, cross-cutting validation gaps, edge clamping) that this skill's confidence filter structurally suppresses. Different classes of bug, not the same class with different completeness.
-
- | Use this skill | Use codex-pair |
- |---|---|
- | Routine PR review (cheap, high precision) | Money/billing/security/spec code (deeper, costlier) |
- | You want one comprehensive report | You want every concern flagged with severity |
- | Cost-sensitive (~$0.04/PR) | Acceptable cost (~$0.20/edit pass) |
-
- ## Instructions
-
- 1. Gather the diff to review:
- - Run `git diff` to get unstaged changes
- - Run `git diff --cached` to get staged changes
- - Combine both into a single diff
-
- 2. If the diff is empty, inform the user there are no changes to review.
-
- 3. Launch the `codex-reviewer` agent with the diff content. The agent handles the Codex prompt structure and output formatting, using GPT-6 Astra at `high` reasoning effort with automatic Terra fallback.
-
- <!-- HOST-ADAPTER:CLAUDE-CODE:END -->
+ `codex-review` is the earlier name of `ask-llm-review`. Load `../ask-llm-review/SKILL.md` and follow it exactly, passing along any arguments the user gave `/codex-review`.