Content
88%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-constructed process skill: a clearly ordered review workflow with executable commands, decision tables, checklists, and user-confirmation checkpoints, written almost entirely from repo-specific knowledge Claude lacks. The main improvement opportunities are trimming occasional over-explanation and moving long policy tables into reference files.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with repo-specific policy Claude cannot know (tag semantics, break windows, council rules) and uses compact tables, but occasionally explains known concepts, e.g. 'A breaking change removes or modifies an existing API in a way that causes compile errors for consumers upgrading.' Efficient with minor trimmable passages — anchor 4, not 5. | 4 / 5 |
Actionability | Guidance is copy-paste ready throughout: 'git diff "$BASE_COMMIT" -- ':(glob)**/api-report/*.md'', 'pnpm flub changeset add --releaseGroup <releaseGroup> --empty', the exact 'test/breaks/client/#.#0/' branch pattern, and a complete @deprecated TSDoc template. | 5 / 5 |
Workflow Clarity | Seven clearly sequenced steps with explicit checkpoints: flagging missing release tags and documentation, verifying export reachability, 'show the content to the user and confirm it looks right before moving on', a deprecation checklist, and a final go/no-go summary. This is a review (non-destructive) workflow, so the validation cap does not apply. | 5 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are absent), and the ~150-line body is well-sectioned with clearly signaled links to deeper repo docs (API-Deprecation.md, Beta-Break-Process.md, .changeset/README.md). Some inline policy tables could live in reference files — good structure with minor organization gaps, an anchor-4 fit. | 4 / 5 |
Total | 18 / 20 Passed |