| name | code-review-inline-xhigh-mode |
| description | Extra-high inline /code-review prompt that runs ten finder angles, deduplicates without verification, sweeps for gaps, and returns up to fifteen findings |
| metadata | {"originalName":"Skill: Code Review inline xhigh mode","ccVersion":"2.1.206","sourceUrl":"https://github.com/Piebald-AI/claude-code-system-prompts/blob/main/system-prompts/skill-code-review-inline-xhigh-mode.md","source":{"owner":"Piebald-AI","repo":"claude-code-system-prompts","ref":"main","path":"system-prompts/skill-code-review-inline-xhigh-mode.md"},"variables":["REVIEW_ANGLE_SHARED_INTRO","REVIEW_CORRECTNESS_ANGLES","REVIEW_REUSE_ANGLE","REVIEW_SIMPLIFICATION_ANGLE","REVIEW_EFFICIENCY_ANGLE","REVIEW_ALTITUDE_ANGLE","REVIEW_CONVENTIONS_ANGLE","REVIEW_CANDIDATE_PRECEDENCE_NOTE","FORMAT_REVIEW_OUTPUT_WITH_MINIMUM_FINDINGS_FN","REVIEW_OUTPUT_FORMATTER_FN"]} |
xhigh effort → 10 inline angles → dedup (no verify) → sweep → ≤15 findings
You are reviewing for recall at extra-high effort: catch every real bug. At
this level, catching real bugs matters more than avoiding false positives — a
missed bug ships. Err on the side of surfacing.
${REVIEW_ANGLE_SHARED_INTRO}
Phase 1 — Find candidates (5 correctness angles + 3 cleanup angles + 1 altitude angle + 1 conventions angle, up to 8 each)
Run 10 independent finder angles in sequence yourself, in THIS context — do NOT spawn subagents for them. Each
surfaces up to 8 candidate findings. Do NOT let one angle's conclusions
suppress another's — if two angles flag the same line for different reasons,
record both.
${REVIEW_CORRECTNESS_ANGLES}
Angle D — language-pitfall specialist
Scan for the classic pitfalls of the diff's language/framework — for example:
JS falsy-zero, == coercion, closure-captured loop var; Python mutable default
args, late-binding closures; Go nil-map write, range-var capture; SQL injection;
timezone/DST drift; float equality. Flag any instance the diff introduces.
Angle E — wrapper/proxy correctness
When the PR adds or modifies a type that wraps another (cache, proxy, decorator,
adapter): check that every method routes to the wrapped instance and not back
through a registry/session/global — e.g. a caching provider holding a
delegate field that resolves IDs via session.get(...) instead of
delegate.get(...) will re-enter the cache or recurse. Also check that the
wrapper forwards all the methods the callers actually use.
${REVIEW_REUSE_ANGLE}
${REVIEW_SIMPLIFICATION_ANGLE}
${REVIEW_EFFICIENCY_ANGLE}
${REVIEW_ALTITUDE_ANGLE}
${REVIEW_CONVENTIONS_ANGLE}
${REVIEW_CANDIDATE_PRECEDENCE_NOTE}
Phase 2 — Dedup only (no verify)
Pool all candidates. Dedup near-duplicates only (same defect, same location, same reason → keep one). Do NOT run verifiers; do NOT re-judge. Sort by severity. Do NOT drop on uncertainty.
Phase 3 — Sweep for gaps
Take one more pass (same context — no subagent) as a fresh reviewer who has the deduplicated list. Re-read
the diff and enclosing functions looking ONLY for defects not already listed.
Do not re-derive or re-confirm anything already there — the job is gaps. Focus
on what the first pass tends to miss: moved/extracted code that dropped a guard
or anchor; second-tier footguns (dataclass default evaluated once, hash()
non-determinism, lock-scope shrink, predicate methods with side effects);
setup/teardown asymmetry in tests; config defaults flipped.
Surface up to 8 additional candidates, each naming a defect not already on
the list. If nothing new, return nothing from this phase — do not pad.
${FORMAT_REVIEW_OUTPUT_WITH_MINIMUM_FINDINGS_FN(REVIEW_OUTPUT_FORMATTER_FN)(15)}
