Content
96%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: concrete commands throughout, tightly written with zero concept explanations, and destructive operations (snapshot acceptance, orphan deletion) wrapped in explicit validation loops with a final checklist. The only refinement available is offloading some detailed enforcement rules into bundle reference files.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and imperative throughout — tables mapping change type to command, hard rules ('Never update, add or delete snapshots manually'), and no padding explaining what Biome, insta, or snapshots are. Every line carries instruction; the one version mention (cargo-insta 1.44+) is functional and backed by a changelog reference rather than filler. | 5 / 5 |
Actionability | Nearly every section gives copy-paste-ready commands with concrete placeholders: `just test-lintrule <ruleName>`, `cargo insta pending-snapshots --workspace`, `cargo insta review --workspace --snapshot '<path-from-list>'`, `cargo insta test --workspace --unreferenced delete`, plus the required-codegen command table. The expectation-marker texts are quoted verbatim. | 5 / 5 |
Workflow Clarity | The snapshot workflow is a fully sequenced loop with validation: run focused test → list pending snapshots → inspect full diff → accept/reject only that snapshot by path → repeat → rerun test. Destructive/batch operations (orphan deletion, unfiltered accepts) are explicitly gated on complete unfiltered workspace runs and per-deletion inspection, and a completion checklist closes the loop. | 5 / 5 |
Progressive Disclosure | Well-organized sections with a clearly labeled, one-level-deep References list pointing at real repo artifacts (CONTRIBUTING.md#testing, crates/biome_test_utils/src/lib.rs, justfile) — no nested references and no monolith. At ~130 lines, some detailed rule sets (e.g., expectation-marker enforcement details, fixture-placement rules) could live in a reference file, which is the minor gap keeping it at 4 rather than 5. | 4 / 5 |
Total | 19 / 20 Passed |