| name | design-review |
| description | Designer-who-codes visual audit + fix. Use to tighten a component before shipping — run the four-source critique, then make atomic, on-system fixes with before/after screenshots. Adapted from open-design's design-review (gstack) to emdesign's code-first loop. |
design-review
Adapted from open-design's design-review (Garry Tan / gstack) — but instead of editing shipped HTML, it
audits a emdesign component through the four feedback sources and fixes it on-system.
Steps
- Audit — run
/mds:review <Name>: rule (lint_consistency + graph_where_to_fix), visual
(run_visual_test), vision (vision-critic on the screenshot), LLM (design-reviewer). Collect a
P0-first finding list with file:line.
- Plan — order fixes by severity; group into atomic edits. Each fix references a token role / primitive,
never a raw value.
- Fix —
edit_component per atomic change. After each, re-run the relevant gate (emdesign lint,
visual-test) — keep a before/after screenshot pair from __screenshots__/.
- Re-gate —
critique_score must pass (composite ≥ threshold && mustFix === 0) before "done".
- Evidence —
record_evidence with the final scores + screenshot.
Merge bar
Tightened means: zero P0s, visual baseline stable (or intentionally re-baselined), vision score ≥ threshold,
and the LLM review finds no composition/semantics/intent gaps. Beautiful, consistent, testable.