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.
An unusually actionable, well-sequenced procedural skill: exact commands, validation checkpoints, and failure fallbacks at every step, with a genuine one-level reference file. The two deductions are mild — some rationale prose could be tighter, and a little more of the signal/lookup detail could move out of SKILL.md into references.
Suggestions
Trim the justificatory asides (e.g. the 'This is the ownership step' paragraph and the repeated explanations of why the card is committed) to pure instruction — the reasoning is already implicit in the rules that follow.
Move the detailed --resolve-issue lookup rationale (the .closingIssuesReferences cross-repository trap) into references/card-signals.md or a second short reference, keeping only the rule 'Let --resolve-issue do the lookup; never hand-roll it' inline.
Consider moving the Step 4 signal-to-fix table into the existing card-signals.md, since that file already maps fields to findings — SKILL.md could then carry only the ranked sources list.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with non-generic, repo-specific knowledge (probe commands, card fields, lookup traps) and never explains concepts Claude already knows, but a few rhetorical passages could be trimmed — e.g. 'This is the ownership step. Everything below is analysis... written first so a run that ends early still leaves it behind'. Efficient with minor instances of over-explanation, matching anchor 4 rather than the every-token-earns-its-place anchor 5. | 4 / 5 |
Actionability | Fully executable throughout: exact probes ('git ls-remote --exit-code origin HEAD'), the card commands ('bun run --cwd tools/pr-metrics card -- --resolve-issue'), a branch-slug-escaping git add, a duplicate-block guard ('grep -c '^<details><summary>Session metrics''), a complete RETRO.md template, and an issue-filing table naming repos, labels and issue types. Not 4 because even the failure paths (no gh, no network, no transcripts) carry concrete instructions. | 5 / 5 |
Workflow Clarity | Steps 1–7 are explicitly sequenced with pre-flight environment probes, validation checkpoints (verify the card is not already on the PR body, match the summary line rather than the tag), and feedback loops for error recovery (record which step failed and carry on; re-run only what is needed). Matches the explicit-validation anchor rather than the minor-gaps anchor 4. | 5 / 5 |
Progressive Disclosure | One clearly-signaled, one-level-deep reference ('references/card-signals.md' — 'Read it before Step 3') that exists in the bundle and matches what the body says it contains, and the body is well-sectioned. Falls short of anchor 5 because the body itself is ~330 lines and some inline material — the Step 2 signal table and the --resolve-issue lookup rationale — overlaps what a second reference file could hold; exceeds anchor 3 because the existing split is clean and correctly timed. | 4 / 5 |
Total | 18 / 20 Passed |