Content
67%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 well-structured with a clear layered-test workflow and genuinely actionable heuristics, properly backed by six real one-level-deep reference files. Its main weakness is scope creep: roughly a third of the content is niche scenario checklists for sibling domains (WebGL globes, AR/mobile permissions, scrollytelling, UML, fictional simulations) that should live in references, diluting token efficiency and leaving the core workflow without explicit validate/retry checkpoints.
Suggestions
Move the niche scenario checklists (AR/camera/motion/geolocation, globe/terrain/cutaway, scrollytelling triggers, UML round-tripping, fictional-data invariants) out of 'Working Pattern' step 1 and 'Coverage Heuristics' into the corresponding reference files, keeping the body to the universal risk classes (wrong numbers, broken interactions, layout drift, stale states).
Add an explicit feedback loop to the workflow, e.g. 'run the candidate visual test twice before baselining; if output differs, fix determinism (fonts, animation, viewport) before snapshotting' — this would also raise workflow clarity.
Surface the executable material earlier: inline one short concrete example (or a pointer next to each heuristic) such as the playwright-visual-regression-starter.ts template for the visual-regression steps, so the body's guidance is copy-paste actionable for the most common case.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is checklist-style with no filler or teaching of concepts Claude already knows, but the 'Working Pattern' risk list (lines 25-49) and long stretches of 'Coverage Heuristics' / 'Output Expectations' accumulate ~20 niche scenario bullets (AR/camera/motion/geolocation, globe/terrain/cutaway, Codex image-generated concepts, fictional story simulations, scrollytelling trigger ranges) that belong in the reference files or sibling skills. This is 'could be tightened' at scale — more than the 'minor instances' of anchor 4, but the content is substance rather than the padded explanation of anchor 2, so it sits at 3. | 3 / 5 |
Actionability | As an instruction-only skill the guidance is concrete and specific: 'fixed viewport and container size', 'stable fonts, theme tokens, locale, and timezone', 'prefer mocked network responses, data loaders, or repository adapters', plus concrete coverage mappings (unit tests for 'scales, domains, bin boundaries, sort rules'). It is not the pseudocode/vague anchor 3, but it stops short of anchor 5's copy-paste-ready examples — no runnable snippet or command appears in the body (the Playwright starter lives in a referenced template), so a few common cases lack an executable example. | 4 / 5 |
Workflow Clarity | The 'Working Pattern' is a clear 6-step sequence (identify risks -> choose layer -> make deterministic -> mock at boundary -> non-goals -> live modes) with some implicit checkpoints ('explicit render-ready signals before capture', 'start with the narrowest high-value slice'). It matches anchor 4 ('clear sequence with most checkpoints present; minor validation gaps') rather than 5, because there is no explicit validate/fix/retry feedback loop (e.g., re-run a flaky baseline or confirm determinism before locking a baseline). The destructive/batch cap does not apply — this is a strategy skill. | 4 / 5 |
Progressive Disclosure | Structure is good: overview, when-to-use, working pattern, heuristics, output expectations, a clearly grouped References section, and representative prompts. All six local reference files (test-level-selection.md, unit-and-component-tests.md, visual-regression-and-image-testing.md, data-mocking-and-fixtures.md, e2e-dashboard-and-export-strategies.md, avoiding-brittle-over-testing.md) exist and are substantive, one level deep. Not anchor 5 because some content that belongs in those references (the niche WebGL/mobile/UML/scrollytelling scenario checklists) is inlined in the body, and many referenced paths (../../references/foundations/*, ../../assets/templates/*, sibling skills) resolve outside this bundle and cannot be verified from it. | 4 / 5 |
Total | 15 / 20 Passed |