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 skill body: dense with non-obvious repo-specific facts, a clear sequenced workflow with explicit validation and error-triage guidance, and appropriate guardrails (never hand-edit generated proto; Renovate boundary; CONTRIBUTING-based PR conventions). The single minor gap is the unspecified exact command for the broad proto-consuming-module compile.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every line carries repo-specific fact Claude could not know: the catalog key location, that models are "not generated in this repo" but come from the Wire-built KMP artifact, snapshot version semantics and ordering, the Renovate boundary, and the exact failure signatures ("NoSuchFieldError/NoSuchMethodError"). There is no padding or explanation of concepts Claude already knows; the one concrete version number ("2.7.26-678281c-SNAPSHOT") appears inline solely as a 'never pin' deprecated-pattern guard, which is load-bearing rather than stale. | 5 / 5 |
Actionability | Mostly executable: set `meshtastic-protobufs = "<version>"`, `git describe --tags --long`, named gradle tasks (`kmpSmokeCompile`, `test`, `allTests`), a named dispatch target (gradle-runner subagent), and `./gradlew :schema-strings:sync`. Kept below 5 only because the 'broad compile of proto-consuming modules (`core:*`, `feature:*`)' names the module pattern but not the exact gradle command line to run for it. | 4 / 5 |
Workflow Clarity | Clear 5-step sequence (read pin, set version, verify, audit, PR) with an explicit validation section ("don't skip test/allTests"), the reason runtime-classpath mismatches surface only in tests, and a feedback loop: "Triage each compile/test failure: adapt this repo's call sites, or, if a break is unexpected, stop and report rather than papering over it." Audit verification is also explicit: "Verify each reference directly: automated gap-lists for this repo are often wrong." | 5 / 5 |
Progressive Disclosure | The body is under 50 lines, needs no external reference files (none exist in the bundle — no references/, scripts/, or assets/ directories), and is cleanly organized into scannable sections (two version forms table, Steps, Verification, Audit, PR + guardrails). Per the rubric's simple-skill guidance, well-organized sections with no external-reference need merit a 5. | 5 / 5 |
Total | 19 / 20 Passed |