← ClaudeAtlas

code-reviewlisted

Review the changes since a fixed point (commit, branch, tag, PR, or merge-base) on two axes — Standards (does it follow this repo's documented rules?) and Spec (does it do what the originating issue asked?). Each axis returns a verdict per enumerated item rather than an open-ended list of findings, so a round terminates. Use when the user wants to review a branch, a PR, working-tree changes, or asks to "review since X"; and again after fixes, to review the fix.
ConnorGriffin/skills · ★ 20 · Code & Development · score 66
Install: claude install-skill ConnorGriffin/skills
# Code review Review a diff on two axes, in parallel sub-agents, and report them side by side. - **Standards** — does the code follow the rules this repo documents? - **Spec** — does the code do what the originating issue or plan asked for? The premise that makes this different from "find problems in this diff": **a review round has to be able to finish.** An open-ended hunt samples a diff, and every fresh round samples it differently, so three rounds turn up three disjoint sets of real defects and nobody can tell whether the fourth would turn up more. Each axis here enumerates a closed list from a source document first, then returns a verdict per item. Findings are the items that came back bad. The reader learns what was checked, not just what was noticed. Two rules follow from that, and they are the ones to keep if everything else is forgotten: 1. **Enumerate before you judge.** The list of things to check comes from a document (the repo's standards, the issue's acceptance criteria), so it is finite, and the same on every run. 2. **A finding carries a failure scenario.** Concrete input or state, traced in the code, and the wrong thing that results. No scenario means it goes in `unverified` or is dropped. A plausible-sounding finding nobody traced is how a fix gets written for a defect that was never there. ## Delegation authority Invoking this skill authorizes every sub-agent dispatch that this procedure marks mandatory, including a mandatory nested re