Skip to content

feat(skill): add codex-code-review for cross-model code review#211

Merged
notque merged 2 commits intomainfrom
feat/codex-code-review-skill
Mar 30, 2026
Merged

feat(skill): add codex-code-review for cross-model code review#211
notque merged 2 commits intomainfrom
feat/codex-code-review-skill

Conversation

@notque
Copy link
Copy Markdown
Owner

@notque notque commented Mar 30, 2026

Summary

  • New codex-code-review skill that invokes OpenAI Codex CLI (GPT-5.4 xhigh) for independent second-opinion code reviews
  • 4-phase workflow: Scope -> Invoke Codex -> Assess & Filter -> Report
  • Integrated into /do routing table with triggers: "codex review", "second opinion", "cross-model review"
  • Added Phase 2b to pr-pipeline for optional cross-model review after comprehensive review
  • Supports both codex exec review --base (purpose-built) and codex exec (custom prompts) invocation modes
  • Sandbox bypass documented for containerized environments where bwrap fails

Test plan

  • codex CLI available and invocable (codex-cli 0.117.0)
  • codex exec review --base main works with sandbox bypass
  • Skill file follows SKILL.md conventions (frontmatter, phases, gates)
  • Registered in skills/INDEX.json with proper triggers
  • Routing table entry added with disambiguation
  • PR pipeline Phase 2b integration is skip-safe (proceeds if codex unavailable)
  • All CI checks pass locally (ruff, pytest 1449, routing-benchmark 44/44)

notque added 2 commits March 29, 2026 22:04
Invoke OpenAI Codex CLI (GPT-5.4 xhigh) for independent second-opinion
code reviews. Claude orchestrates scoping, prompt construction, and
critically assesses Codex output before presenting to the user.

Key design decisions:
- Read-only sandbox mode (Codex can browse repo, not modify)
- Let Codex access git/filesystem directly (no embedded diffs)
- Structured output: CRITICAL / IMPROVEMENTS / POSITIVE / SUMMARY
- Claude filters and assesses every finding (not pass-through)
- User-invocable as /codex-code-review slash command
…review support

- Add routing table entry for codex-code-review skill with disambiguation
- Add Phase 2b cross-model review to pr-pipeline (optional, skips if codex unavailable)
- Update skill with codex exec review subcommand (--base, --commit flags)
- Add sandbox bypass documentation for containerized environments
@notque notque merged commit a757fb2 into main Mar 30, 2026
4 checks passed
@notque notque deleted the feat/codex-code-review-skill branch March 30, 2026 05:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant