Content
82%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 strong, highly actionable reference: executable examples for every command group, tight prose that assumes competence, and good internal navigation. The main gaps are minor — a couple of slightly redundant passages, no explicit validation/recovery step for config changes, and all reference detail inlined in one long file rather than split into a references bundle.
Suggestions
Trim the redundancy between the hook-output field descriptions and the decision-aggregation bullets (e.g., fold 'updated_input: last non-empty wins' into one place) and shorten the stdin JSON sample to the fields hooks actually read.
Add a short 'making a change' workflow with a validation checkpoint, e.g., write the crushrc statements, then confirm they took effect via `model large` (no-arg print) or by checking the startup warning when both crushrc and crush.json are present.
Move the long flag inventories (provider/model/mcp/lsp flags) and the hooks runtime spec into a references/ file (e.g., references/commands.md), keeping SKILL.md as an overview with well-signaled one-level-deep links.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense reference material with no padding — it never explains concepts Claude already knows (Bash, MCP, LSPs) and every section is commands, flags, or behavioral rules. Minor trim opportunities exist (the decision-aggregation bullets partially restate the hook-output field rules, and the full stdin JSON sample), which keeps it just below the lean 'every token earns its place' anchor. | 4 / 5 |
Actionability | Every command family ships copy-paste-ready executable examples ('provider add deepseek --type openai-compat --base-url ... --api-key "${DEEPSEEK_API_KEY:?...}"', 'mcp add github --type http --url ...', 'hook add PreToolUse --matcher "^bash$" ...'), plus a concrete crushrc↔crush.json mapping table covering the common cases. | 5 / 5 |
Workflow Clarity | The body follows a clear sequence (formats → discovery priority → per-entity command reference → hooks runtime → user-invocable skills → env vars → legacy JSON mapping) and offers a verification affordance ('model large with no argument prints the current selection'). It falls short of a 5 because there is no explicit validate-the-config step or error-recovery guidance for a change that fails to load; the destructive/batch cap does not apply since config editing is not a destructive batch operation. | 4 / 5 |
Progressive Disclosure | A single self-contained file with no bundle files present; sections are well organized with working internal anchors ([Hooks runtime](#hooks-runtime), [Legacy JSON format](#legacy-json-format)) and the mapping table gives one-glance navigation. It is not a 5 because all detail — full flag inventories, the hooks runtime spec, and the option catalog — is inlined in a ~380-line file rather than offloaded to well-signaled one-level-deep reference files, and the under-50-line simple-skill exception does not apply. | 4 / 5 |
Total | 17 / 20 Passed |