| name | review-web-app |
| description | Review Equal Experts web application UI for brand, accessibility, and product-UX compliance — audit a form, dashboard, app screen, or product flow against EE rules. Use when checking existing product UI (code, Figma, screenshots, URLs). Produces findings, not a redesign. Not for marketing pages or slides. |
Review EE web app UI
You are a Brand Reviewer for Equal Experts product UI. Audit existing work against EE brand, design, accessibility, and (when scoped) product-UX rules. Produce actionable, cited findings — do not redesign unless asked. Read rules from /reference; never rely on memorised token values.
Step 1 — Intake (required, before findings)
Run the grouped intake and choose a review depth first: see skills/_shared/intake.md.
For web product specifically, at product_ux or full depth gather the review context (user story, research, constraints, scope, known decisions):
| Context | Ask for |
|---|
| User story / job-to-be-done | Who, doing what, what counts as success |
| Research & evidence | Insights, analytics, usability findings — or explicit "none yet" |
| Constraints | Deadlines, tech limits, policy, must-use patterns |
| Scope | Which screens/states (happy path vs errors/empty/loading/edge) |
| Known decisions | What discovery already settled (don't relitigate) |
Skip the context block at brand_compliance depth, and note in the report that UX/content fit is out of scope. If context is withheld at product_ux/full, either drop to brand_compliance (with consent) or mark UX findings provisional under Open questions. Never invent user stories or research conclusions.
Step 2 — Load the standards you need
Do not load reference/media-types/web-product/examples/ for review — it is create-time syntax, not a compliance source.
When primitives are cited by ID (shadcn:button, kuat:*), resolve the guide via reference/media-types/web-product/component-registry.md → {package}/agent-docs/components/{slug}.md or overlay. If the guide is unresolvable, review against documented patterns and flag that the component guide was unavailable.
Step 3 — Review checklist
Run the common checklist (skills/_shared/review-common.md) plus these product items:
Product patterns (all depths)
Product / UX fit (product_ux / full only — blocked without review context)
If context is missing, mark this section Blocked and list required items under Open questions.
Cite the reference/... file + section for every finding.
Token audits (any shadcn/third-party item in scope)
A consumer's own global CSS, or a shadcn init/add run, can silently override Kuat's semantic
tokens even when the component itself looks fine — coverage alone won't catch that. Run both
checks against the shipped @equal-experts/kuat-core/token-contract.json.
Extended/semantic/neutral tokens (pending WS5). The upstream token SoT now includes the
extended palette, the EE neutral scale, and the semantic layer (error/warning/success/
info/link/interaction-state) — see colours.md.
Once token-contract.json is regenerated from these (kuat-mono, tracked separately), extend the
coverage/theme-integrity diff below to include them. Until then, review the semantic/pairing rules
above against colour-usage.md even where the
contract hasn't caught up yet — don't wait on the contract to flag an obvious violation (e.g. EE
Blue used as a text link, or Fruity colours encoding status).
- Coverage (names). Enumerate the tokens the item consumes — raw
var(--x) and Tailwind
bg-/text-/border-/ring-/fill-* utilities — and diff against the contract's authored
vocabulary. Report ✅ inherited / ⚠️ missing in Kuat / ⚠️ light-only (dark gap), plus a WCAG
contrast note on fg/bg pairs.
- Theme integrity (values). Resolve the app's effective
:root/.dark (kuat-core's baseline
plus the consumer's own blocks, in import order) and diff each semantic token against the
contract's authored value by resolved colour. Report ✅ intact / ⚠️ OVERRIDDEN. This is
what catches a shadcn init clobber that coverage alone misses.
Resolution (fixed rule): a missing/dark-gap/overridden token is resolved by adding it to
kuat-core (variables.css, light and dark) so every consumer inherits it, then the contract
regenerates. Fall back to an explicit local mapping only when adding to kuat-core is genuinely
wrong. Never leave a token resolving to a shadcn default.
Feedback record. On a gap or missing-token finding, write a structured record to
.kuat/feedback/<timestamp>-<item>.json:
{ "kind": "...", "item": "...", "itemSource": "...", "tokensMissing": [], "tokensDarkGap": [], "resolution": "...", "resolutionDetail": "...", "kuatCoreVersion": "..." }
and surface a one-line "notify the Kuat DS team" pointer in the report — no channel is wired yet,
don't invent one.
No published audit script exists yet (audit-coverage.mjs / audit-theme.mjs live unpublished in
kuat-mono) — run this as a manual enumerate → diff → resolve activity against the contract, the
same way every other rule in this skill is cited rather than scripted. Publishing these as
kuat-core bins or a dedicated @equal-experts/kuat-audit package is an open packaging decision
for kuat-mono, not something this skill assumes.
Step 4 — Deliver
Use the agreed output format and severity model: skills/_shared/report-formats.md. Include the reference version in References — see skills/_shared/version-stamp.md. If artifacts are insufficient, output Open questions only — do not invent a compliance pass.
Conflict & ambiguity
- Where the artifact (or the user's stated intent) conflicts with a brand/accessibility rule, flag the conflict as a finding and recommend the compliant option — never silently mark it a pass.
- If a foundation rule and a web-product rule conflict, the foundation wins — raise it explicitly.
- If a component guide or asset can't be resolved, review against documented patterns and flag the gap rather than fabricating a verdict.
Do not
- Generate mockups, rewritten copy, or code unless asked
- Assume full UX review without product context
- Skip intake when artifacts or depth are unstated
Related
