Content
100%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 skill body: lean, entirely project-specific, with executable code patterns, explicit guardrails (no new assertion kinds, no snapshot reuse, prefer TUI over invoke_git), and validation guidance. Nothing to trim and no structural gaps.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every line carries project-specific knowledge Claude could not infer — harness APIs ("test_status_tui", "input_then_render"), snapshot conventions, and exact paths ("crates/but/src/command/legacy/status/tui/tests/"). No padding or explanations of concepts Claude already knows. | 5 / 5 |
Actionability | Fully executable guidance: a complete copy-paste-ready Rust test pattern, concrete input-driving examples for every key type, and exact commands ("cargo test -p but <test-name>", "SNAPSHOTS=overwrite cargo test -p but <test-name>"). | 5 / 5 |
Workflow Clarity | The basic pattern is unambiguous and the skill includes explicit validation ("cargo test -p but tui ... Do this after changing things") plus an error-recovery feedback loop ("If a test fails the output will include the rendered state ... way of inspecting the state"). Not a destructive/batch process, so no validation cap applies. | 5 / 5 |
Progressive Disclosure | No bundle files exist and none are needed; the ~78-line body is organized into tight sections (where tests live, pattern, driving, assertions, running) with clearly signaled one-level pointers to the two harness util files for specialized assertions. | 5 / 5 |
Total | 20 / 20 Passed |