Content
86%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 well-structured, project-specific skill body: lean, correctly scoped, with concrete file targets and runnable validation commands. The remaining gains are small — showing an example changeset/@deprecated annotation and tightening 'run targeted tests' into an actual command would make the guidance fully copy-paste ready.
Suggestions
Add a one-line example changeset (e.g. a patch-bump .changeset/*.md snippet) so the versioning step is copy-paste ready.
Replace 'Run targeted tests for the affected API when available' with a concrete command or a pointer to how to find the relevant test files.
Include a brief example of an existing @deprecated JSDoc pattern so the 'follow existing patterns nearby' instruction has a floor to start from.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and imperative throughout — no concept explanations, no padding, and every bullet carries project-specific information Claude cannot infer (file globs, schema/build paths, workspace-scoped commands). 'Keep runtime behavior unchanged unless the task explicitly requires more than deprecation signaling' is a terse, high-value constraint. | 5 / 5 |
Actionability | Mostly executable: exact file patterns, the literal metadata value "status": "deprecated", and copy-paste commands like 'npx prettier --write <paths>' and 'npm run build:hooks.json -w @primer/react'. Minor gaps keep it below 5: no example changeset content for the versioning step, no example @deprecated JSDoc annotation, and 'Run targeted tests for the affected API when available' gives no command. | 4 / 5 |
Workflow Clarity | The sequence is clear (source → docs metadata → docs content → changeset → validation) and the Validation section is an explicit checkpoint tiered as 'Prefer targeted validation first, then broader validation if needed', including rebuild commands for schema-affecting changes. It falls short of 5 only because there is no explicit feedback loop (what to do when lint/tests fail) and steps are sectioned rather than sequenced with ordering cues. | 4 / 5 |
Progressive Disclosure | This is a compact (~55-line) single-file skill with no bundle files and no content that warrants splitting; sections are well-organized and the pointers it does give ('contributor-docs/deprecating-components.md', 'contributor-docs/versioning.md') are one level deep, clearly named, and appropriately load-bearing rather than buried. | 5 / 5 |
Total | 18 / 20 Passed |