Content
96%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 exemplary lean, fully actionable skill body: a worked quick-start example, exact command templates per input type, null-handling recovery guidance, and a validation checklist, with no filler. The single defect is the dangling TROUBLESHOOTING.md reference, which is the only thing holding progressive_disclosure below 5.
Suggestions
Ship the referenced TROUBLESHOOTING.md (e.g., in references/) or remove/inlines its fallback guidance — the current link resolves to a file that does not exist in the bundle, so navigation fails exactly when the skill needs it (memory_commit_lookup unavailable).
Show the memory_recall invocation form in step 3 (or a one-line example) so the 'importance >= 7 observations' path is as copy-paste ready as the blame/lookup steps.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | At roughly 58 lines, every section (Quick start, Why, Workflow, Anti-patterns, Checklist, See also) earns its place: it explains only tool-specific behavior ('When the lookup gives commit: null, the commit predates session linking') and nothing Claude already knows. It matches anchor 5 — lean, assumes competence, no trimmable padding. | 5 / 5 |
Actionability | The Quick start gives a fully worked, copy-paste-ready example with expected output, and the Workflow supplies exact command templates per input type ('git blame -L <start>,<end> <file>', 'git log -L :<function>:<file>', 'memory_commit_lookup { "sha": "<full-sha>" }'). The only gap is that the optional memory_recall invocation syntax is not shown, which does not undermine the common path — matching anchor 5's 'copy-paste ready commands covering common cases'. | 5 / 5 |
Workflow Clarity | A clear 3-step sequence with per-input-type variants, an explicit Checklist serving as validation ('SHA came from git blame/log, not a guess'), and a WRONG/RIGHT anti-pattern pair giving an error-recovery path for the commit: null case. This read-only skill has no destructive/batch operations, so the validation cap does not apply; the checklist plus null-handling loop matches anchor 5. | 5 / 5 |
Progressive Disclosure | The single-file body is well organized into clearly signaled sections, and the one external reference ('See [TROUBLESHOOTING.md](TROUBLESHOOTING.md)') is one level deep and clearly motivated. However, TROUBLESHOOTING.md does not exist in the bundle (no references/ or other files are present), so the link is dangling — a minor organization gap that fits anchor 4 ('references mostly clear; minor organization gaps') rather than anchor 5's 'easy navigation'. | 4 / 5 |
Total | 19 / 20 Passed |