What problem does it solve?
Reviewing PRs in the Reader's multi-protocol social territory (atmosphere, mastodon, shared post-card components) repeatedly surfaces the same bugs — cross-tenant cache poisoning, missing default arms, silent no-ops — and reviewers waste effort rediscovering them each slice cycle.
Core Features & Use Cases
- Structured review rubric: Produces severity-tagged findings (blocker/important/nit) with file:line anchors, in-PR vs follow-up classification, and a verdict line, capped at ~700 words.
- Recurring-smell checklist: Ships a catalog of 9+ concrete findings from the CM-625/658/660/662 slice cycle, covering optimistic-mutation scoping,
cancelQueries wrapping, encodeURIComponent on wire IDs, and error-observability gaps.
- Cross-PR fix propagation: Guides checking sibling slice PRs for the same defect after a fix lands on one, since the atmosphere and mastodon architectures mirror each other.
- Use Case: When a PR modifies
packages/api-queries/src/reader-mastodon.ts, walk the smell checklist to verify the optimistic patcher is connection-scoped and cancelQueries is wrapped in try/catch before approving.
Quick Start
Review PR number 1234 in wp-calypso using the reader protocol review rubric and the common-smells checklist.
