Content
85%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 instruction-only skill body: lean per-skill entries, a concrete routing procedure, honest maintenance guidance, and clean structure with a single well-signaled external reference. The only improvements are trimming the motivational opener and adding one worked routing example.
Suggestions
Add one worked routing example under 'How to use it' (e.g., job: 'commit these changes' -> matches Quality and review -> smart-commit), which would raise actionability to fully executable coverage of common cases.
Collapse the two-sentence opener into a single statement of purpose, e.g. '41 skills and 23 commands exceed anyone's working memory; this index is the map.'
State the update trigger as a concrete checklist condition (skill added/renamed/removed/fires differently) rather than a paragraph, so maintenance steps are scannable alongside the entries.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | One line per skill with [human]/[auto] tags and no padding or over-explanation, but the two-sentence motivational opener ('You cannot hold 41 skills and 23 commands in your head. This is the index so you do not have to.') could be trimmed to a single statement of purpose. | 4 / 5 |
Actionability | The routing procedure is concrete and executable ('State the job in one line. Match it to a group below. If two skills look close, the when to reach clause is the tiebreaker') plus a clear maintenance rule, but there is no worked example of an actual job resolving to a specific skill. | 4 / 5 |
Workflow Clarity | The routing sequence is stated as an unambiguous three-step procedure with an explicit tiebreaker rule; the operation is read-only, so no validation checkpoints are needed and the simple-skill exception applies. | 5 / 5 |
Progressive Disclosure | Well-sectioned groups with easy navigation, a dedicated command-only section, and exactly one clearly-signaled one-level reference (rules/skill-conventions.mdc); the index itself is correctly inline because the router must load to function. | 5 / 5 |
Total | 18 / 20 Passed |