visual-qa-loop
Skill6 starsChanged 4 months agoMITView on GitHub
Run before/after visual QA for local UI changes with screenshots and concise findings.
In this repository27 other files it gives its agents.
- agent-skills-agentsAGENTS.md
- agent-skills-claudeCLAUDE.md
- agent-log-forensicsSkill
- changelog-writerSkill
- chezmoi-driftSkill
- contacts-unifySkill
- decision-logSkill
- dependency-pinningSkill
- django-smoke-alarmSkill
- empathy-auditSkill
- first-contactSkill
- handle-prSkill
- launch-sequenceSkill
- parallel-isolated-app-testingSkill
- post-mortemSkill
- pre-mortemSkill
- profile-meSkill
- ralph-loopSkill
- release-operatorSkill
- remote-host-verifierSkill
- research-personSkill
- scope-hammerSkill
- second-opinionsSkill
- status-copy-trust-auditSkill
- support-stormSkill
- trust-auditSkill
- wide-open-brainstormSkill
The file
We split this skill into the rules it gives an agent
10 rules. About 0 are common, 3 are distinctive, and 7 are very specific to this project.
≈0
Common · estimate
Many files like this have a rule of this kind.
≈3
Distinctive · estimate
This author's own choices. Worth a look.
7
Very specific
Tied to their project's names, files or tools. Probably skip.
- Common
- Distinctive
- Very specific
- Very specific: line 41
Use whatever screenshot path is available, in this order: a project or machine wrapper (for example chrome-shot, if command -v chrome-shot …
- Distinctive: line 49
If no tool can capture the target, stop and say so rather than reviewing from code alone.
- Very specific: line 53
If changes are not applied yet, capture before/.
- Very specific: line 55
Use filenames that encode route/state/viewport, not verbose captions.
- Distinctive: line 73
If an obvious regression is visible, fix it and recapture once (a second failed pass goes in the report, not a third loop).
- Distinctive: line 85
Keep it short. The screenshots are the receipts.
- Very specific: line 91
Use trust-audit when the visual surface touches permissions, privacy, billing, or destructive file changes.
- Very specific: line 92
Use status-copy-trust-audit when UI/CLI wording is the confusing part.
- Very specific: line 93
Use first-contact when the visual change touches onboarding or day-zero mental models.
- Very specific: line 94
Use agent-log-forensics when recurring user complaints point at visual or interaction jank across sessions.
≈ “Common” is an estimate: at least 1 in 5 of the 350,495 skill files we read have a rule of the same kind, not the same words. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.
Files
Visual QA Loop
For UI work, "tests pass" is not the same as "eyes pass."
Third-order stance: static screenshots are receipts, not the whole truth. Look for temporal debt, hidden friction, and trust changes between states: loading, empty, error, hover, focus, permission prompts, and the moment after a user clicks.
When to Use
- The user asks for visual QA, screenshots, design polish, UI review, hover/focus review, or before/after proof
- A frontend/macOS/iOS UI change might alter layout, copy, color, spacing, or trust perception
- The user wants assurance changes are tweaks, not accidental replacement
When NOT to Use
- Pure backend or CLI changes with no visual surface
- The app cannot be rendered locally and no screenshot path exists
- The user explicitly says not to do visual validation
<process>Phase 1: Pick Stable Targets
Identify the exact pages/windows/states to capture. Prefer the smallest set that covers the changed surface. Include transition states when relevant: loading, empty, error, first-run, disabled, hover/focus, and post-action success/failure.
Pick the artifact directory first: the project's existing notes/artifacts location if its instructions name one, otherwise a temp directory. Do not add screenshots to the repo unless asked.
Use whatever screenshot path is available, in this order: a project or machine wrapper (for example chrome-shot, if command -v chrome-shot succeeds), a browser automation tool (Playwright, agent-browser, or an MCP browser), headless Chrome, or screencapture for native macOS windows. Example with chrome-shot:
ART=<artifact-dir>/visual-qa/<run>; mkdir -p "$ART"
chrome-shot -o "$ART/home-1440.png" http://localhost:3000/
chrome-shot --size 390x1200 -o "$ART/home-mobile.png" http://localhost:3000/If no tool can capture the target, stop and say so rather than reviewing from code alone.
Phase 2: Capture Before and After
If changes are not applied yet, capture before/. After edits/build, capture after/ with the same viewport, route, data, and state.
Use filenames that encode route/state/viewport, not verbose captions. If the issue may be motion/jank, capture a short trace, repeated screenshots, or before/after states around the transition rather than only the final settled UI.
Phase 3: Review Like a User
Check:
- layout shifts, overflow, clipping
- copy drift or confusing labels
- hover/focus/selected states too loud or too invisible
- trust fractures: surprise permissions, privacy creep, pricing ambiguity, destructive actions
- mobile density and tap targets
- empty/error/loading states when relevant
- information density: high-signal details should not disappear behind unnecessary hover/clicks
- temporal debt: flicker, layout shift, stale preview, delayed permission prompt, or post-click ambiguity
- downstream friction: a small visual change near a CTA, filename, price, or destructive action can be larger than its pixel area
Phase 4: Iterate Once Before Reporting
If an obvious regression is visible, fix it and recapture once (a second failed pass goes in the report, not a third loop). Only fix regressions in the changed surface. Do not hand back known-bad screenshots unless blocked.
Phase 5: Report Evidence
Return:
- screenshot artifact paths
- what changed visually
- pass/fail verdict
- any remaining concerns
- any "looks fine but may feel wrong over time" temporal/friction concerns
Keep it short. The screenshots are the receipts.
</process><interlocks>- Use
trust-auditwhen the visual surface touches permissions, privacy, billing, or destructive file changes. - Use
status-copy-trust-auditwhen UI/CLI wording is the confusing part. - Use
first-contactwhen the visual change touches onboarding or day-zero mental models. - Use
agent-log-forensicswhen recurring user complaints point at visual or interaction jank across sessions.
</interlocks>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
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.

