Content
56%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 well-organized with a clear sequenced procedure, explicit constraints, and a fail-fast validation stance, but it under-delivers on specifics: the key skill-template reference is a broken stub, the interview prompts and structure checks referenced are never made concrete, and several sections redundantly restate the same cautions.
Suggestions
Replace the stubbed ./references/skill-template.md (which points to a non-existent fixtures/budget-archive path) with the actual template content, or link to a real file, so the 'Read when: template details are needed' reference resolves one level deep.
Make step 6 concrete by naming the exact structure checks to run (which validation gate, script, or checklist) instead of 'Run the relevant structure checks', and surface the bundled contract.yaml, evals.yaml, and task-profile.json with explicit links since step 5 depends on them.
Consolidate the overlapping Constraints / Anti-patterns / Gotchas / Failure mode sections and remove the duplicated reference lists (the top 'Read when' links repeat the bottom 'Reference materials' block) to tighten token use without losing guidance.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly lean bullet-style guidance with no concept explanations Claude already knows, but it could be tightened: the two 'Read when' links at the top duplicate the bottom 'Reference materials' block, and Constraints, Anti-patterns, Gotchas, and Failure mode restate overlapping ideas (e.g. 'stop and report missing inputs' appears in both Validation and Failure mode; no-transcript guidance appears in multiple sections). This fits the 'mostly efficient but includes some unnecessary explanation or could be tightened' anchor rather than the level-4 anchor's 'minor instances of over-explanation'. | 3 / 5 |
Actionability | The Procedure gives a real sequenced process with some concrete specifics (prefer ~/.agents/session-collector bundle evidence, draft using the template, include procedure/validation/constraints sections), but key details are missing for an instruction skill: the structured interview prompts named in the description never appear, step 6 says 'run the relevant structure checks' without naming any check or command, and the bundle's contract.yaml/evals.yaml/task-profile.json are only named parenthetically. This matches 'some concrete guidance but incomplete; missing key details' rather than 'mostly executable guidance with minor gaps'. | 3 / 5 |
Workflow Clarity | The Procedure is a clear six-step sequence with an explicit validation checkpoint and feedback loop ('Run the relevant structure checks and iterate until clean'), plus a dedicated Validation section and fail-fast handling in Failure mode. Not the level-5 anchor because the validation checkpoints are somewhat abstract — 'relevant structure checks' and 'verify eval cases' are never tied to concrete checks — but this exceeds the level-3 anchor where checkpoints are missing or implicit. | 4 / 5 |
Progressive Disclosure | The 'Read when:' pattern is a well-signaled one-level-deep reference style, but scoring against the actual bundle reveals real navigation failures: references/skill-template.md is a 2-line stub pointing to a fixtures/budget-archive/2026-04-19/... path that does not exist in the bundle, so the body's 'Read when: template details are needed' pointer resolves to nothing; and the bundled contract.yaml, evals.yaml, and task-profile.json are not clearly linked from the body at all. This fits 'some structure but could be better organized; references present but not clearly signaled' better than the level-4 anchor's 'references mostly clear'. | 3 / 5 |
Total | 13 / 20 Passed |