pr-reviewlisted

Use when reviewing a pull request against the ticket it claims to deliver, before marking it ready for review or merging it. Starts from the acceptance criteria rather than from the diff and requires both satisfying code and a proving test for each one, identifies changes with no criterion behind them, walks the compliance checklist for every layer the diff touches, reads the pull request's recorded decisions first so justified deviations are not reported as findings, reads test bodies instead of counting them, ranks findings blocking / should fix / consider, and ends with an explicit verdict. Every finding cites the principle or guide section behind it; an uncited one is labelled a preference rather than dressed as a standard. Reports findings; never edits the code.
konradcinkusz/architecture-standards · ★ 0 · Code & Development · score 75

Install: claude install-skill konradcinkusz/architecture-standards

# Reviewing a pull request against the ticket The last gate before a pull request is ready. It answers one question — **does this diff deliver the ticket, within the architecture?** — and it answers it against two sources that outrank the reviewer's taste: the acceptance criteria, and the [reference architecture](https://github.com/konradcinkusz/architecture-standards/blob/main/docs/architecture/00-REFERENCE-ARCHITECTURE.md). This review reads and reports. It does not edit: a review that fixes what it finds destroys the evidence of what was wrong, and nobody learns the pattern. Last of three phases, after [`TICKET-ANALYSIS.md`](https://github.com/konradcinkusz/architecture-standards/blob/main/docs/delivery/TICKET-ANALYSIS.md) and [`IMPLEMENTATION-PHASE.md`](https://github.com/konradcinkusz/architecture-standards/blob/main/docs/delivery/IMPLEMENTATION-PHASE.md). [`WORKFLOW.md`](https://github.com/konradcinkusz/architecture-standards/blob/main/docs/delivery/WORKFLOW.md) is how they fit together and how to install them. **Contents** 0. [The standards have to be in front of you](#0-the-standards-have-to-be-in-front-of-you) 1. [Establish what is actually being reviewed](#1-establish-the-diff) 2. [Acceptance criteria first](#2-acceptance-criteria-first) 3. [The architectural pass](#3-the-architectural-pass) 4. [The test pass](#4-the-test-pass) 5. [Severity, and the verdict](#5-severity-and-the-verdict) 6. [Failure modes](#6-failure-modes) 7. [Checklist](#7-checklist) --- ##