packages/shared-skills/skills/frontend/references/designpowers/lane-c-review.md
Lane C runs after implementation and before final sign-off. It requires objective visual-qa evidence first, then applies designpowers judgment to the same artifact, then hands the reconciled context to review-work. The order matters: measurements and screenshots anchor the review; designpowers adds the human-centered judgment does not fully encode.
| Capability | Materialized designpowers source | owner | Mapping |
|---|---|---|---|
| Review an existing surface without rerunning discovery | design-review | visual-qa plus review-work | Use only for critique context; still captures objective artifacts and final review. |
| Critique against brief, plan, personas, design principles, taste, and craft | designpowers-critique | visual-qa evidence, then review-work | Run after screenshots and objective checks exist, so findings cite the built surface. |
| WCAG, COGA, keyboard, screen reader, motion, content, and adaptive needs | agent accessibility-reviewer | visual-qa evidence, then review-work | Treat the materialized agent file as reviewer-role guidance. Name who is affected and exact fixes. |
| Nielsen heuristics and cognitive walkthroughs | heuristic-evaluation plus agent heuristic-evaluator | visual-qa evidence, then review-work | Walk every key task and classify H1-H10 findings with severity. |
| Persona and task walkthroughs | synthetic-user-testing | visual-qa evidence, then review-work | Validate that inclusive-personas can complete real tasks under their assistive or situational contexts. |
| Human testing plan when needed | usability-testing | review-work context | Produce a test plan or follow-up recommendation when synthetic testing is insufficient. |
| Completion evidence discipline | verification-before-shipping | review-work | Summarize plan completion, accessibility results, persona walkthrough, content status, and debt status. |
Materialized agent references for this lane: design-critic, accessibility-reviewer, and heuristic-evaluator.
Use this sequence for UI review and repair:
Run `visual-qa` first against the actual built surface. Capture objective screenshots, diffs, browser or terminal artifacts, and any required visual QA report.
Then apply Lane C Review & Repair to the same artifact:
- designpowers-critique checks brief, plan, principles, personas, taste, craft, and design-system alignment
- accessibility-reviewer checks WCAG, COGA, keyboard, screen reader, touch, motion, adaptive preferences, and content accessibility
- heuristic-evaluation checks Nielsen H1-H10 and cognitive walkthroughs for key tasks
- synthetic-user-testing walks key tasks as each relevant persona from inclusive-personas
- usability-testing is used for a real-participant test plan when the evidence cannot be resolved synthetically
- verification-before-shipping turns the findings into a single evidence-backed report
Reconcile conflicts by this priority: accessibility over aesthetics, usability over style, brief over opinion, personas to break ties, user escalation for unresolved trade-offs.
Pass the reconciled Lane C report, objective visual-qa artifacts, open findings, and accepted design debt to `review-work` for final implementation review.
Lane C requires all of the following before pass:
visual-qa artifact paths from the actual surface, such as screenshots, image diff JSON, terminal captures, or synthesized visual verdict.heuristic-evaluation results covering relevant Nielsen heuristics and cognitive walkthroughs for key tasks.synthetic-user-testing results with persona, task, steps, outcome, and barrier matrix.review-work, including objective artifacts and designpowers judgments for the same build.visual-qa evidence exists for the surface under review.review-work; Lane C supplies review input, not final approval.PASS when objective visual-qa evidence exists, designpowers review lanes pass or have explicit accepted debt, and the reconciled context is handed to review-work.
FAIL when review runs without real artifacts, skips heuristic-evaluation, skips synthetic-user-testing for persona-critical flows, treats accessibility as optional, leaves Critical or Major issues unrepaired, or sends final context to review-work without the design findings.