agentleFS
dev-ikdev-ik/ai-design-rules/skills/visual-qa/SKILL.md

visual-qa

Skill3 starsChanged 37 days agoMITView on GitHub

Use when an implemented screen, screenshot, or UI change needs visual QA for spacing, alignment, wrapping, clipping, overlap, hierarchy, component states, or comparison with a supplied design reference. Skip backend-only changes.

20 other projects use a version of this.

Adopted by

Most recently taken up by

The file

We split this skill into the rules it gives an agent

3 rules. 0 are common, 2 are distinctive, and 1 is very specific to this project.

  • 0

    Common

    Most of the 20 edited versions we read keep these.

  • 2

    Distinctive

    This author's own choices. Worth a look.

  • 1

    Very specific

    Tied to their project's names, files or tools. Probably skip.

  • Common
  • Distinctive
  • Very specific
  1. Distinctive:

    If a capability is unavailable, continue with the available evidence and mark the missing checks unverified.

    line 20
  2. Very specific:

    Check narrow and wide layouts with responsive-check.

    line 25
  3. Distinctive:

    If a design reference exists, compare matching viewport, state, theme, content, and scale.

    line 28

“Common” is measured: kept by at least 6 in 10 of the 20 edited versions of this file we could read. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.

Keep what fits and drop the rest. Your version, ready for your agents.
Files
SKILL.md

Visual QA

Review observed UI against the user's task, existing design system, and retrieved graph context. This execution skill complements design-reviewer, which owns the final traceability review.

Inputs and capabilities

Identify the screen, primary action, changed components, relevant states, target devices, and any supplied reference. Retrieve focused context with npx --no-install ai-design-context context --review "<graph ID or matching phrase>" --intent qa; read the returned research and rules. An arbitrary application path is not automatically a graph match.

Record which tools are actually available:

  • A browser tool or the project's existing Playwright setup for navigation, viewport changes, clicks, keyboard input, scrolling, DOM measurements, and screenshots.
  • An image-viewing tool and an image-capable model for inspecting captured pixels and references.
  • Source and DOM inspection for relating a visible issue to its implementation.

If a capability is unavailable, continue with the available evidence and mark the missing checks unverified. Skills do not install browser binaries, provision model vision, or enable MCP tools. A code-only review cannot establish rendered quality; a screenshot cannot establish keyboard or responsive behavior.

Browser and image loop

  1. Open the actual implementation. Record its URL or fixture path, browser, viewport in CSS pixels, theme, zoom, data, and state. Exercise the primary action and relevant default, loading, empty, error, success, and disabled states; mark inapplicable states with a reason.
  2. Check narrow and wide layouts with responsive-check. Open and close menus, dialogs, and sheets, scroll past sticky regions, and exercise keyboard focus with accessibility-check.
  3. Save viewport screenshots for geometry and separate full-page screenshots for long content. Use the project's evidence convention; otherwise create a dated review folder under output/playwright/. Record the steps that reproduce each capture.
  4. Open the captured images, not just their filenames. Inspect alignment, spacing consistency, typography, line breaks, clipped text, collisions, icon alignment, button/input sizing, content density, primary-action prominence, and state feedback. Confirm suspected geometry problems with DOM measurements where possible.
  5. If a design reference exists, compare matching viewport, state, theme, content, and scale. Record unmatched conditions instead of attributing every pixel difference to a defect. Without a reference, assess task clarity and existing tokens; do not invent a target composition from taste.
  6. For authorized fixes, make a focused patch, repeat the failed interaction, and recapture the same conditions. Run the project's relevant code checks. Preserve product logic and the existing UI stack.

Report contract

Lead with PASS, NEEDS WORK, or PARTIAL, scoped to the surfaces and states actually reviewed. PARTIAL means missing evidence prevents completing that scope. Separate observed defects from coverage gaps; a skipped check is not a confirmed product defect.

For each finding, include:

FieldRequired evidence
ID and severityStable review-local ID; Critical, Major, Minor, or Polish based on impact on the primary task.
Surface and conditionsScreen/component, viewport, data/state, and input method.
ReproductionActions another reviewer can repeat.
Observed / expectedWhat happened; applicable rule, pattern, reference, or explicit knowledge gap.
EvidenceScreenshot path and inspected region; DOM measurement or interaction observation when available.
Impact and fixUser consequence, focused correction, likely file/component when verified.
RetestOutcome and matching after-capture, or pending.

Finish with a coverage table: surface | viewport | state/input | evidence | checked/unverified/not applicable. List missing tools and any reference-comparison gaps. Do not infer a global pass from one desktop screenshot.

Graph anchors and tool references

  • CHECK-00001 / checklists/DESIGN_QA.md: state, responsive, accessibility, and evidence boundaries.
  • RULE-00008 / VIS-001: existing semantic tokens.
  • RULE-00014 / VIS-002: content and primary-action hierarchy.
  • RULE-00011 / PERF-001: loading-state layout stability.
  • Playwright screenshots and emulation describe capture and viewport capabilities, not proof that those tools are installed.

Download now (ZIP)

Does it work?

Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.

No reviews yet

People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.

  • It worked
  • Partly
  • It didn't
What did you use it for? e.g. Reviewing pull requests in a Rails app

Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.

Your trail

Files you open will collect here, so you can find them again.

Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.

Now do this for your own.

Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.