## What changed ## Why ## Sequence / flow diagram (REQUIRED for behavioral changes) ```mermaid sequenceDiagram participant A as Before participant B as After A->>B: (replace with the actual flow) ``` ## Visual evidence - [ ] Visual diff comment reviewed — every 🔴 changed shot is intentional - [ ] Sequence/flow diagram included above (or "n/a — no behavior change") ## Test plan - [ ] Tests / build green locally — paste the command you ran ## Review gate — merge blockers - [ ] **Q1** Enums over magic strings — finite sets are enums + exhaustive switch - [ ] **Q2** Codable on the wire — no hand-built JSON literals; canonical writer if byte-equality matters (Exhibit 13) - [ ] **Q6** Actors / Sendable — shared mutable state isolated; `@unchecked Sendable` justified in writing - [ ] **Q8** Emoji telemetry at every new decision point - [ ] **Q9** Secrets & data posture — no ambient env inheritance, no PII in logs, brokered secrets - [ ] **Q14** Exhaustive proof — CaseIterable tests, sequence diagram, snapshots if UI, receipts from a real run Non-blockers (answer or N/A): Q3 expression · Q4 protocol vs enum · Q5 functional · Q7 lifetime · Q10 previews · Q11 snapshots · Q12 haptics/a11y · Q13 performance · Q15 simpler shape? ## Secrets / data posture ## Blast radius & rollback