Content
81%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.
A thorough, highly actionable workflow skill with clear sequencing, validation checkpoints, and feedback loops. Its main weakness is conciseness — several coaching and general-debugging passages could be trimmed — and minor progressive-disclosure gaps from inlined reference detail and an unreferenced script.
Suggestions
Tighten or relocate the 'Root-Cause, Don't Treat Symptoms' section: it reads as general debugging wisdom Claude already knows; keep only the VS Code-specific lifecycle examples.
Move the detailed flag documentation and 'Seed-copy rules' into a references file (e.g. references/runner-flags.md) and link to it, reducing inlined reference-style content in SKILL.md.
Either document scripts/workbench-css-performance.mts in the 'Checked-in Runners' list or remove it from the bundle so the file listing matches the referenced scripts.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient with high-value specifics (exact flags, import-path depth, summary.json fields), but coaching prose like 'The first version of an automation runner is rarely correct' and the general-debugging 'Root-Cause, Don't Treat Symptoms' section pad beyond what Claude needs. | 3 / 5 |
Actionability | Abundant copy-paste-ready commands (node ... --iterations 3 --no-heap-snapshots, npx @playwright/cli invocations, mkdir/heredoc scratchpad setup, a complete compareSnapshots script) cover the common cases with specific flags and paths. | 5 / 5 |
Workflow Clarity | An 8-step 'Story' sequence with explicit validation checkpoints ('Run a fast smoke', 'Verify the run', 'Validate with --no-heap-snapshots first'), a 'Verify Before Analyzing' checklist, and fix→rerun→compare feedback loops for the destructive snapshot runs. | 5 / 5 |
Progressive Disclosure | Good structure with one-level-deep, clearly signaled links to real scripts in ./scripts/, but a fair amount of reference-style detail (flag docs, seed-copy rules) is inlined and scripts/workbench-css-performance.mts exists yet is never referenced. | 4 / 5 |
Total | 17 / 20 Passed |