holistic-review

Holistic Review

Perform a holistic code review by routing to the best available review backend. Prefers codex:rescue when available; falls back to a direct one-shot:holistic-reviewer subagent.

Required Context

You must have two pieces of information before invoking this skill:

• Implementation plan path — absolute path to the plan file
• Base branch — the branch or commitish to diff against (everything between base and HEAD is in scope)

If either is missing, stop and ask.

Detection Gate

Before doing anything else, determine whether codex:rescue is available.

Scan your available skills/commands list (shown in your system context) for codex:rescue. Record the result explicitly in your response:

codex:rescue: AVAILABLE / NOT AVAILABLE

You MUST write this line before proceeding. Do not skip it.

Routing

If AVAILABLE: use codex:rescue

This is the primary path. You may not skip it when codex:rescue is available. Falling through to the fallback path when codex:rescue is detected is a failure.

1. Get the one-shot:holistic-reviewer agent definition file's fully qualified path: ${CLAUDE_PLUGIN_ROOT}/agents/holistic-reviewer.md. There is no reason to read this file, if you can expand this path then the file exists.

2. Invoke SkillTool("/codex:rescue --fresh --foreground "{prompt}"). Pass a prompt (the {prompt} variable) containing all three of:

   • The path of the holistic-reviewer agent definition
   • The absolute path to the implementation plan
   • The base branch to diff against

   You are performing a holistic code review. Follow this agent definition exactly. READ THIS AGENT DEFINITON: {paste the FULLY QUALIFIED holistic-reviewer agent file path here}
   
   Review Parameters:
   
   - Implementation plan: {absolute_plan_path}
   - Base branch: {base_branch}
   

If NOT AVAILABLE: fallback to subagent

Spin up the one-shot:holistic-reviewer agent in a new subagent with a clean context. Provide the implementation plan path and the base branch.

<invoke name="Agent">
<parameter name="subagent_type">one-shot:holistic-reviewer</parameter>
<parameter name="description">Holistic code review</parameter>
<parameter name="prompt">
Perform a holistic review of the implementation.

Implementation plan: {absolute_plan_path}
Base branch: {base_branch}
</parameter>
</invoke>

3. MANDATORY: Address ALL issues from the holistic code review. Do not complete until all issues have been addressed. Loop through steps 2-3 until you get a clean code review.

Common Mistakes

Mistake	Fix
Skipping detection and going straight to the subagent fallback	Always record the detection result first. The gate is mandatory.
Summarizing the holistic-reviewer agent definition instead of inlining it	Codex has no access to your files. Paste the full text.
Forgetting --fresh on codex:rescue	Without it, Codex may resume a stale thread. Always pass --fresh.