Content
72%Weight 40%Scale 1-3Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A token-efficient, well-structured instruction-only skill with a clean single reference, but it lacks executable gh CLI specifics and any validation checkpoint for its review workflow.
Suggestions
Add the concrete gh commands behind 'Ground every claim in gh CLI evidence' (e.g. `gh api repos/:owner/:repo` for stars/issues, `gh release list`, `gh pr list --state all`) so guidance is copy-paste ready.
Include a short verification step, such as confirming every breaking change has both a CHANGELOG entry and a migration note before approving, to add an explicit feedback loop to the review workflow.
Provide one minimal example of a one-line migration note so the 'ships with a one-line migration note' rule is unambiguous.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean with zero padding — it never explains concepts Claude already knows and every line ('Ground every claim in gh CLI evidence', 'Every breaking change ships with a CHANGELOG.md entry and a one-line migration note') earns its place, matching the 'Lean and efficient; every token earns its place' anchor. | 3 / 3 |
Actionability | Rules are concrete directives, but 'Ground every claim in gh CLI evidence (stars, issues, PRs, releases, traffic, contributors)' names the evidence sources without giving the actual gh commands to fetch them, leaving key executable details missing as in the 'Some concrete guidance but incomplete; missing key details' anchor. | 2 / 3 |
Workflow Clarity | 'When to activate' and 'Key rules' are organized lists but not a sequenced process, and there is no validation/verification checkpoint for a reviewer that can block PRs; this matches 'Steps listed but validation gaps; checkpoints missing or implicit' rather than a clear validated sequence. | 2 / 3 |
Progressive Disclosure | Under 50 lines with well-organized sections ('When to activate', 'Key rules') and a single clearly-signaled one-level reference ([APM CEO persona](../../agents/apm-ceo.agent.md)); per the simple-skill note, this satisfies the 'Clear overview with well-signaled one-level-deep references' anchor. | 3 / 3 |
Total | 10 / 12 Passed |