Content
77%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.
An actionable, well-sequenced skill body with concrete templates and validation checkpoints, weakened mainly by token redundancy (repeated base paths and duplicated subcommand descriptions) and a monolithic single-file structure that forgoes progressive disclosure for a non-trivial multi-subcommand skill.
Suggestions
Define the base path once (e.g. `$MATTERS = ~/.claude/plugins/config/claude-for-legal/ip-legal/matters/`) and reference it thereafter, eliminating ~15 full-path repetitions and trimming significant tokens.
Consolidate the subcommand summary and the detailed Subcommand logic into one section to remove the duplication between lines 17-21 and 70-112.
Move the matter.md template and the intake interview into a references/ file (e.g. references/matter-template.md) and link to it from the body, giving this multi-faceted skill genuine one-level-deep progressive disclosure.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly specific operational detail rather than conceptual padding, but carries real redundancy: the full path ~/.claude/plugins/config/claude-for-legal/ip-legal/ is repeated ~15 times, the subcommand summary (lines 17-21) restates the detailed Subcommand logic (lines 70-112), and the "default state is off" explanation appears in both the Instructions and Notes sections. | 2 / 3 |
Actionability | Highly concrete instruction-only guidance: exact file paths, exact folder-move operations, an exact markdown table schema, and complete matter.md/history.md templates with literal strings to write — copy-paste ready despite the absence of code. | 3 / 3 |
Workflow Clarity | Each subcommand is a clearly numbered sequence with explicit verification checkpoints ("Confirm slug is not already present", "Confirm matters/<slug>/matter.md exists", "Show the user what changed and confirm before writing"), and the close operation verifies existence before the reversible archive move. | 3 / 3 |
Progressive Disclosure | Well-organized with clear section headers, but it is a single ~184-line monolithic file with no bundle files (references/scripts/assets absent); content that could be split out — the matter.md template, the intake interview, the storage layout — is all inline rather than in one-level-deep reference files. | 2 / 3 |
Total | 10 / 12 Passed |