← ClaudeAtlas

ruver-code-reviewlisted

Adaptive PR review via gh. One PR: main thread. Two or more PRs: one fresh subagent per PR (full skill each), orchestrator only resolves list and aggregates chat summary. Pending required CI waits 5m (no PR comment) then reviews on green; defers on CI red, draft, or conflict. Deep then light on SHA history. One artifact per PR: REQUEST_CHANGES / APPROVE / DEFER. Use when /ruver-code-review with URL(s), number(s), or current branch.
ruverd/skills · ★ 1 · Code & Development · score 67
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