Content
82%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is an effective conventions catalog: nearly every line is non-obvious team judgment delivered as concrete values, tool-specific API options, and worked examples, with clear section organization and a stated decision order. The main flaws are moderate redundancy - three anti-patterns stated twice - and an authoring sequence that is implied by section order rather than explicit.
Suggestions
Deduplicate the anti-patterns: three of the eight rows in 'Common anti-patterns' (Percy/Chromatic mixing, threshold cranking, snapshot-refresh PRs) restate guidance already given in 'Engine selection', 'Masking, threshold, or wait?', and 'When to update a baseline' - either drop the table rows or remove the inline 'Anti-pattern' callouts.
Make the authoring sequence explicit: add a brief numbered flow (pick engine -> select stories/pages -> choose breakpoints -> handle noise -> set severity tier -> name baselines) at the top so the section order reads as a deliberate workflow rather than an implicit one.
Pull the cross-skill pointer to playwright-snapshots' references/responsive-breakpoints.md out of the mid-paragraph prose in 'Engine selection' into a clearly signaled 'Related skills' line so the reference is easy to discover.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, judgment-heavy tables ("the per-prop combination matrix is exponential and catches very few real bugs") with no explanation of concepts Claude already knows, but three anti-patterns appear twice ("running both Percy and Chromatic on the same project" / "Mixing Percy and Chromatic on the same coverage"; "maxDiffPixels: 5000" / "Threshold cranked above 0.3 / maxDiffPixels > 500"; the snapshot-refresh PR in both 'When to update a baseline' and the anti-pattern table) and the intro repeats its how/which distinction. Efficient with minor trimmable redundancy - anchor 4, not 5; not 3 since the padding is incidental duplication, not explanatory filler. | 4 / 5 |
Actionability | Guidance is concrete and copy-paste-ready throughout: exact breakpoint widths (375 / 768 / 1280 / 1920 px), exact tolerances ("maxDiffPixels (50 - 200) or threshold (0.2 default to 0.3 max)"), exact API options ("freezeAnimatedImage (Percy)", "caret: 'hide' (Playwright)", "page.waitForSelector"), and worked naming examples ("Atoms/Button/Primary-Disabled", "/dashboard/billing -> dashboard-billing", "-375, -768, -1280, -1920"). Per the rubric's code-vs-instruction note, an instruction-only skill with guidance this specific earns the top anchor; nothing falls into anchor 4's 'minor gaps'. | 5 / 5 |
Workflow Clarity | The entry point is explicit ("Pick before authoring any baseline"), a decision order is stated ("Order of preference: wait -> mask -> threshold"), and lifecycle rules are sequenced with a promotion checkpoint ("Promote warn-tier baselines to block-tier after they've been stable for ~2 weeks"), but the sections follow the authoring flow implicitly rather than as an explicit numbered procedure - clear sequence with minor gaps, anchor 4. No validation cap applies since the skill is advisory with no destructive or batch operations. | 4 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/ directories), so the ~170-line self-contained catalog is scored on structure: clear section headers, consistent tables, and relevant one-level cross-skill pointers ("playwright-snapshots (references/responsive-breakpoints.md)", "visual-baseline-gate"). It exceeds the under-50-line simple-skill case, and the cross-skill path is embedded mid-paragraph rather than clearly signaled - good structure with minor organization gaps, anchor 4. | 4 / 5 |
Total | 17 / 20 Passed |