ruver-code-reviewlisted
Install: claude install-skill ruverd/skills
# Ruver Code Review
Sequential phases per PR. Depth adapts to review history.
**Multi-PR:** one subagent per PR (see §0).
**CI fork:** [GRAPH.md](GRAPH.md) · [LOOP.md](LOOP.md). Pending required CI
waits 5m and does **not** post. Red CI still DEFERS.
## Invariants — never break
1. **Subagents only when reviewing 2+ PRs in one invocation.** Single-PR stays on
the main thread. Multi-PR: orchestrator must **not** review diffs itself —
spawn one fresh subagent per PR, each running this skill for **that PR only**.
2. **Exactly one artifact per PR** — one review (inline comments in the same
call) or one issue comment. Never both. Never two of either **on the same PR**.
3. **Never APPROVE** while any required check is failed, pending or unknown.
4. **Never invent a finding.** A finding with no concrete trigger is dropped in
silence — not downgraded, not mentioned.
5. **Nits never block, and most stay silent.** A nit reaches the PR only inside the
collapsed Nits block (§9), only with a concrete `path:line`, max 5 per run, and
never changes the verdict. Always silent regardless: praise, generic refactor
advice, speculative perf, pre-existing debt, anything the diff did not touch.
"Could be cleaner" is silent except the one code-judo nit in Phase 8, which must
name a layer, branch, or helper to delete.
6. **Voice.** GitHub text and chat follow **Voice** (just above §9). Teammate
English on the PR. Session language in chat. Never caveman. Never a form.
7