Content
75%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.
The body is exemplary in token efficiency — a tight trigger list plus five concrete policy rules with no padding. Its main weakness is actionability: the evidence and changelog rules would be far more executable with example gh commands and a sample CHANGELOG entry, and the one external persona reference is broken.
Suggestions
Add 1-2 example gh CLI commands for evidence gathering, e.g. `gh api repos/microsoft/apm/traffic` or `gh pr list --repo microsoft/apm --state open --json author,createdAt`, so the "Ground every claim in gh CLI evidence" rule is directly executable.
Include a one-line CHANGELOG entry + migration-note template (e.g. `## [Unreleased]` with a `**Migration:** ...` line) so the breaking-change rule is copy-paste ready.
Fix or remove the dangling [APM CEO persona](../../agents/apm-ceo.agent.md) link — the referenced file does not exist at that path.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | A lean ~25-line body of two tight sections with zero concept explanation (it never explains what a CHANGELOG or release pipeline is) and no filler — every line is either a trigger or a rule. Matches anchor 5 ("every token earns its place"); anchor 4 is ruled out because there is nothing that could be trimmed. | 5 / 5 |
Actionability | Rules like "Every breaking change ships with a CHANGELOG.md entry and a one-line migration note" are specific and checkable, but no example gh commands (e.g. gh api repos/{owner}/{repo}/traffic) or CHANGELOG entry template are provided — a real gap for a rule that mandates CLI evidence. Falls between anchor 3 ("missing key details") and 4 ("concrete code or commands with minor gaps"); since no commands exist at all, the gap is more than minor. | 3 / 5 |
Workflow Clarity | A simple single-purpose reviewer skill with unambiguous activation triggers (exact paths including .github/workflows/build-release.yml) and individually clear rules; no multi-step process exists, so no validation checkpoints are needed. Scored below 5 because the arbitration path ("Final arbiter when... reviewers disagree") and the order of operations remain implicit. | 4 / 5 |
Progressive Disclosure | Under 50 lines with well-organized sections and one external reference, [APM CEO persona](../../agents/apm-ceo.agent.md), but that path does not resolve (no agents/ directory exists at the referenced location). Good structure with a dangling navigation link is anchor 4 ("minor organization gaps"), not 5. | 4 / 5 |
Total | 16 / 20 Passed |