Content
80%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 tight, repo-specific runbook that gets straight to exact paths and commands with no wasted tokens. It loses points for missing validation on a batch multi-file edit and for leaving the compatibility-table and extraPackages edits underspecified.
Suggestions
Add a verification step after the edits, e.g. grep the repo for stale Mediabunny versions ("grep -rn '<old-version>' packages/ package.json") or re-run "npm view mediabunny version" and diff against every updated file, so the batch edit is validated before finishing.
Specify the shape of the compatibility-table row in packages/docs/docs/mediabunny/version.mdx (what columns/entries to add and which version to pair with the incremented patch version) so that step is unambiguous.
Clarify what the extraPackages entries in packages/studio-shared/src/package-info.ts and packages/cli/src/extra-packages.ts should look like after the edit (a concrete before/after example would close the remaining actionability gap).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Eight lean bullets with zero padding — every line carries non-obvious repo-specific knowledge (catalog vs. explicit versions, the extra-packages.ts and package-info.ts locations, the patch-increment rule). It fully assumes Claude's competence and explains nothing it already knows. | 5 / 5 |
Actionability | Concrete commands ("npm view mediabunny version", "bun i") and exact file paths ("packages/cli/src/extra-packages.ts", "packages/template-*/package.json") make most steps executable. Not a 5 because the compatibility-table step ("Update the compatibility table in packages/docs/docs/mediabunny/version.mdx") and the extraPackages edits leave the exact edit shape unspecified — minor gaps. | 4 / 5 |
Workflow Clarity | The steps are listed in a coherent order, but this is a batch operation editing at least five locations across many template packages with no validation or verification step (no grep to confirm all versions match, no build/test after "bun i"). Per the rubric, a batch workflow without validation is capped at 3, which overrides the simple-skill exception. | 3 / 5 |
Progressive Disclosure | Under 50 lines, single-purpose, no bundle files exist and none are needed; the flat ordered checklist is appropriately organized for a skill of this size, matching the rubric's simple-skill exception for progressive disclosure. | 5 / 5 |
Total | 17 / 20 Passed |