1---2name: quality3description: Quality — Index4---56# Quality — Index78Persimmon quality = a set of focused review passes plus an orchestrator that runs them before delivery. This mother is a map; follow the child for the actual work.910## Trigger1112- "Review this change" / "pre-delivery QA" / "ship check"13- "Is this fast enough / type-safe / DB-efficient?"14- "Final review before the client sees it"1516## The child skills1718| Skill | When to use | Owns |19|---|---|---|20| `quality-final-review` | Before any client delivery | Orchestrates all review-* dimensions + `security-review`; produces a go/no-go summary |21| `quality-review-performance` | Perf concerns | RSC boundaries, bundle size, DB query plans, Core Web Vitals |22| `quality-review-type-safety` | Type rigor | Strict-mode gaps, `any` leaks, unchecked casts, missing Zod |23| `quality-review-data-layer` | DB review | N+1 queries, unbounded lists, missing indexes, transaction boundaries |24| `quality-review-prompt-output` | AI output review | Prompt hygiene, output validation, citation grounding, eval harness |2526## How to route27281. **Delivering?** → `quality-final-review` (it pulls in the others + `security-review`).292. **Single dimension?** → invoke the matching `quality-review-*` child directly.3031## Persimmon quality defaults — one-screen summary3233- **Final review is mandatory before client delivery** — never skip.34- **Review dimensions**: performance, type-safety, data-layer, prompt-output, security.35- **Type-safety**: no `any` without justification; Zod at every boundary.36- **Data-layer**: no N+1, no unbounded lists, indexes present, transactions where needed.37- **AI output**: every claim grounded/cited where the domain requires it (e.g. legal briefs).38- **Verification is evidence-based**: run `tsc --noEmit`, `eslint`, `prisma validate`, tests — don't assert "looks fine".3940## Anti-patterns banned4142- Shipping to a client without `quality-final-review`43- Declaring a review passed without running the checks44- Ignoring N+1 / unbounded lists "for now"45- Accepting ungrounded AI output where citations are required4647## Relationship to other mothers4849| Mother | Connection |50|---|---|51| `workflow` | `workflow-verify` and `workflow-code-review` invoke these children |52| `security` | `security-review` is folded into `quality-final-review` |53| `stack` / `data` / `ai` | Each review dimension audits the matching domain |