review-loop · git:20260916.8dd269c · 2026-09-16 · sha256 8bddf8b95b3187cb
review-loop git:20260916.8dd269cA
Immutable. This exact content is served forever at /api/v1/blob/8bddf8b95b3187cb.
--- name: review-loop description: Codex-native Stage 1 review-loop skill. Orchestrates planning and execution with a Codex Executor and Claude/Codex reviewer backends while sharing the .review-loop protocol with Claude Code. --- # review-loop — codex orchestration Read `docs/protocol/loading.md`, then run: ```bash python3 <support-root>/scripts/read_protocol.py --runtime codex --stage entry-review-loop ``` Resolve <support-root> to this plugin/repository, not the task workspace. Keep cwd in the user's workspace. Read the complete emitted text before acting. `docs/protocol/loading.json` is the action/prerequisite map for both runtimes. Detect fresh / plan-exists / code-exists / explicit-resume via the entry procedures. For fresh work run planning → planning-review; after approval continue execution → execution-review → gate → polish → docs → security → delivery. Do not stop after exec alone. Before initialization load `session-init` AND `review-loop-init`; before a resume load `resume` AND `review-loop-init`. Before any work-agent dispatch load the matching `planning` or `execution` bundle; before review load `planning-review` or `execution-review`. On planning approval, only the plan-only entry loads `plan-exit`. The umbrella retains its session/lock and continues directly into execution. Before a stage transition, retry, stop or error exit, load its applicable bundle per `loading.md`. Use `parallel-review` only in the Codex orchestrator for N>1 Claude-CLI reviewer jobs (never in Claude Code or for local Codex Reviewer agents); load `context-persist` only when that optional substep is applicable. The caller owns the session file and lock. Preserve unrelated dirty work; read-only/plan-only scope and user authorization override implementation steps. Independent Reviewer approval, output validation, rubric/triage and evidence guards remain mandatory. Use existing configured backend/model resolution. Never infer a PASS, a skip, or permission from not loading a future stage. Full session schema: `docs/protocol/session-file.md`; active loops: `docs/protocol/planning.md` and `docs/protocol/execution.md`; output contracts: `docs/protocol/executor-output.md` and `docs/protocol/reviewer-output.md`. These are scoped references, not an eager import list. Detailed native entry steps live in `references/entry.md` and are selected by the loading map. Reuse a rule unit only while its exact text remains in this live context, using the fingerprint emitted by the loader. Fresh agents, compaction and new invocations must load their own prerequisites. Do not use persisted session state as proof that instructions are still available.