Content
65%Weight 40%Scale 1-3Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is highly actionable with exact paths, templates, and concrete per-subcommand workflows, and it is well sectioned. Its weaknesses are repetition of the subcommand list across three sections, lack of a post-operation verification loop on the destructive `close` step, and a monolithic layout with no progressive file references.
Suggestions
Consolidate the subcommand coverage: keep the detailed ## Subcommand logic steps and drop or slim the duplicate summary list and the dispatch table in ## Instructions so each command appears once.
Add a verification step to `close` (e.g., confirm `matters/_archived/<slug>/matter.md` exists after the move and re-read the practice CLAUDE.md to confirm the Active matter line) to close the destructive-operation feedback loop.
Move the full `matter.md` and `history.md` templates into a reference file (e.g. references/matter-template.md) linked from the body, turning SKILL.md into a leaner overview and improving progressive disclosure.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The subcommands are presented three times — a summary list (## Subcommands), a dispatch table (## Instructions step 3), and detailed numbered steps (## Subcommand logic) — and the disabled/off state is explained twice (Instructions step 1 and the post-divider paragraph), so it is mostly efficient but padded with repetition that could be tightened; not 3 because of this redundancy, not 1 because no basic concepts are over-explained. | 2 / 3 |
Actionability | Gives exact storage paths, precise edits ("update the `Active matter:` line"), a full matter.md template, a history.md seed, and a concrete intake field list — fully actionable, copy-ready instruction despite being instruction-only. | 3 / 3 |
Workflow Clarity | Each subcommand is clearly sequenced with numbered steps and pre-checks (confirm slug not present, confirm matter.md exists), but `close` is a destructive directory move with no post-move verification that the archive succeeded or the active-matter line updated, so per the destructive-operation guideline workflow clarity is capped at 2 rather than 3. | 2 / 3 |
Progressive Disclosure | Well-organized with clear section headers and no nested references, but the body exceeds 50 lines as a single monolithic file with the full templates and per-subcommand logic inline and no file-based references to split detail into, fitting the "content that should be separate is inline" anchor rather than the one-level-deep reference structure of a 3. | 2 / 3 |
Total | 9 / 12 Passed |