educlopez/audit

<!-- HARNESS MIRROR — do not edit here. Canonical source: skills/ or commands/. After editing source, copy into cli/assets/<harness>/ and repo-root harness mirrors. -->

Context: this sub-skill is one lens of the broader ui-craft skill. If the ui-craft skill is also installed, read its SKILL.md first for Discovery + Anti-Slop + Craft Test, then apply the specific lens below.

Run a technical audit of the UI at $ARGUMENTS. Load the ui-craft skill and apply the audit lens.

Step 0: Visual capture (mandatory)

Code-only review is insufficient. Every audit/critique starts with the surface as the user sees it. Try the following in order; use the first one available:

1. Playwright MCP — if playwright MCP server is available, use it. Capture full-page screenshots at three viewports: desktop (1280×800), tablet (768×1024), mobile (375×812). Capture dark mode if the app supports it.
2. Browser DevTools / Chrome MCP — second choice; same viewport set.
3. Other browser automation (agent-browser, cursor-ide-browser) — third choice.
4. Ask the user — last resort. If no automation is available, request screenshots from the user before proceeding. Be specific:
   • "Visual review needs screenshots. Please provide:
     • Full-page at 1280px (desktop)
     • Full-page at 768px (tablet)
     • Full-page at 375px (mobile)
     • Dark mode of each, if supported."

Do not begin the review until visuals are captured or provided. State this explicitly to the user when no automation succeeds — don't silently fall back to code-only review.

If the user declines to provide screenshots, run a code-only pass and clearly mark the report [CODE-ONLY REVIEW — visual issues not assessed] at the top so the limitation is explicit.

Note: audit is knob-agnostic — accessibility and performance are not tunable.

Scope (non-negotiable checks):

1. Accessibility — read references/accessibility.md:
   • Visible :focus-visible on every interactive element
   • Keyboard reachable, no focus traps
   • Touch targets ≥ 44px (mobile)
   • Color not the only signal for state
   • Form labels, error association, required indication
   • prefers-reduced-motion honored for all animations
2. Performance — read references/motion.md Rendering Performance section:
   • Only transform / opacity animated (no width/top/height)
   • No transition: all
   • will-change scoped to active interaction, removed after
   • Images have width/height or aspect-ratio (CLS)
   • No layout thrash in scroll/resize handlers
3. Responsive — read references/responsive.md:
   • Mobile-first breakpoints, no fixed-width components
   • env(safe-area-inset-*) respected on fixed elements
   • Touch zones don't overlap
   • No horizontal scroll at 320px

Output format — the Review Format table from SKILL.md:

| Before | After | Why | | --- | --- | --- |

Group findings by priority: Critical (blocks usability/a11y) → High-impact (immediately noticeable) → Quick wins (polish).

Do NOT rewrite code unless asked. Report findings first; wait for approval before editing.

Close with a Craft Report (references/review.md → Craft Report), wrapping the findings table above — Checked states scope (a11y/perf/responsive, at which viewports), Passed carries anything inspected and found sound, Changed stays empty pre-approval, Verdict names the top priority. Produce it even on a clean audit — a11y/perf/responsive coming back clean is the finding.

Next step: /harden — turn the findings into real production coverage (rung 1).