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.
A well-structured, actionable, and validation-conscious body with executable helper commands, a mandatory queue table, and properly signaled one-level-deep bundle references. It is held just below top marks by recurring constraint redundancy and the lack of fix-retry feedback loops.
Suggestions
Deduplicate the 'never comments/labels/assigns' constraints by stating them once in 'Hard nos' and referencing that section elsewhere, to tighten conciseness toward 5.
Add a short fix-retry feedback loop for helper failures (e.g., 'on fetch_queue failure: verify gh availability / --records-json fallback, then re-run') to lift workflow clarity from 4 to 5.
Move the full activation-card template and queue-table column spec into a referenced asset file so the SKILL.md body stays a lean overview, improving progressive disclosure.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, imperative, and assumes Claude's competence without explaining known concepts; minor redundancy (the 'never comments/labels/assigns' constraints recur in Activation card, Invocation, Selection, and Hard nos) keeps it just below the lean 5 anchor. | 4 / 5 |
Actionability | Provides concrete executable commands ('python <this-skill>/scripts/fetch_queue.py --kind pr --mode sweep --repo microsoft/apm > batch.json', '--mode dispatch --number N') and a fill-in activation-card template; residual placeholders like <this-skill> and <positive integer> leave minor gaps, matching the 4 anchor. | 4 / 5 |
Workflow Clarity | The Procedure is a clear 4-step sequence with explicit fail-closed validation checkpoints ('Missing -> stop', 'Missing table ... -> stop. Do not spawn.') and a mandatory queue-table checklist; because validation is present (and the skill is advisory/non-destructive), the batch-cap of 3 does not apply, but absent fix-retry feedback loops keep it at 4 rather than 5. | 4 / 5 |
Progressive Disclosure | Body is an overview that signals one-level-deep bundle references (assets/fan-out-pool.md, scripts/fetch_queue.py, scripts/triage_state.py, assets/label-contract.json) which exist on disk; the full activation card and queue-table format are inlined rather than split out, a minor organization gap placing it at 4. | 4 / 5 |
Total | 16 / 20 Passed |