Content
92%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 router-style SKILL.md: lean, tutorial-free, with a routing table that makes page selection mechanical and behavioral rules that constrain scope. The only weakness is that a few Core Behavior directives presuppose project context ('OO-leaning Rust default') without pointing to the guideline page that defines them, leaving minor actionability gaps.
Suggestions
Anchor each abstract Core Behavior bullet to the guideline page that elaborates it (e.g., 'Prefer strong, compiler-backed types' → newtypes/validation page) so every directive resolves to concrete policy in one hop.
Briefly define or link 'OO-leaning Rust default' — it presupposes project context a first-time reader of SKILL.md alone does not have.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~40-line body explains nothing Claude already knows (no Rust tutorials, no library comparisons) — every section is functional: a location note, a file index, a 12-row routing table, and six behavioral rules like "Load only the pages the task needs; guideline pages are the policy, workflow pages are the procedures." Matches the 5 anchor (lean, assumes competence, every token earns its place); there is no padded or over-explained passage that would justify 4. | 5 / 5 |
Actionability | The routing table gives concrete, executable load instructions for 12 common cases ("Create a new Rust project → workflows/new-rust-project.md, guidelines.md"), which is actionable instruction-only guidance. However, Core Behavior bullets such as "Use the project's OO-leaning Rust default without forcing inheritance-shaped designs" and "Prefer strong, compiler-backed types over primitive-heavy APIs" are abstract directives not anchored to the page that elaborates them — minor gaps matching the 4 anchor rather than the fully-resolved 5 anchor. | 4 / 5 |
Workflow Clarity | This is a single-purpose router skill under 50 lines, and its single action is unambiguous: consult the task→files table, load the listed pages, apply the rules, with an explicit fallback ("Ask one focused question only when required project context is missing"). The simple-skill exception applies and there are no destructive or batch operations requiring validation, so 5 is warranted rather than the 4 anchor's 'minor validation gaps'. | 5 / 5 |
Progressive Disclosure | The body is a pure overview: a Supporting Files index of one-level-deep references (guidelines.md plus four workflow files), all clearly signaled with descriptions, and a routing table that names guideline page topics ("library/application errors, error propagation, public API evolution") for direct discovery. The nonstandard file location is explicitly documented ("supporting files live in .fabro/skills/rust-style-guide/ at the repository root... Read them with that prefix") and every link uses that prefix consistently. (Note: the referenced files are not present in this review bundle, but their documented repo-root location and consistent link prefix keep navigation unambiguous.) This matches the 5 anchor (clear overview, well-signaled one-level-deep references, easy navigation); the index→page hop lives inside guidelines.md and does not bury content in SKILL.md itself, so it does not fall to 4. | 5 / 5 |
Total | 19 / 20 Passed |