Content
57%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.
The body demonstrates deep domain expertise with concrete commands, flags, and exemplary fail-closed error semantics, but its extreme compression works against it: a 450-word run-on paragraph, repeated links, a scattered main workflow, and two dangling file references (README.md, templates/ci.yml) make it hard to navigate on first read. The material is strong and mostly needs restructuring, not rewriting.
Suggestions
Split the single ~450-word `scripts/change-class` paragraph into headed subsections (verdict classes, render proof, ownership, queue-only behavior) or move the detail into references/wiring.md, leaving a short summary in SKILL.md.
Add an ordered getting-started sequence near the top (wire the workflow once from wiring.md → run the classifier → aggregate required contexts) so the happy path is explicit rather than scattered.
Fix or remove the dangling references to README.md and templates/ci.yml, and consolidate the four repeated links to the same wiring.md section into one pointer.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | There is no over-explanation of concepts Claude already knows, but the ~450-word single paragraph on `scripts/change-class` is run-on prose, the same wiring.md anchor link is repeated four times, and "Flags and settings: `change-class --help`" deferrals appear twice — clear tightening opportunities. Fits anchor 3 (mostly efficient but could be tightened); not 4 because the repetition and wall-of-text density cost clarity for little token savings. | 3 / 5 |
Actionability | One fully executable command block (`.agents/skills/harness-ci/scripts/harness-only --event pull_request --base "$BASE" --head "$HEAD"`) plus concrete flag documentation (`--mode render-candidate`, `--mode docs`, `--output FILE`, `--lanes-from DIR`, `--queue-only-default true`, `--skippable`, `--waiver`, `--lane JOB=LANE`), with key details legitimately deferred to the bundled scripts' `--help`. Fits anchor 4 (mostly executable guidance with minor gaps); not 5 because the aggregate-needs helper never receives a complete copy-paste example invocation. | 4 / 5 |
Workflow Clarity | Error-path checkpoints are strong (the Fail-closed section, refusal causes carrying remedies such as `cause=source-mirror-missing verb=source operation=refresh`, exit-code semantics, and the Reading a verdict section), but the happy-path sequence (wire the workflow, classify the diff, aggregate required contexts) is scattered across topic sections with no ordered sequence a newcomer can follow. Fits anchor 3 (sequence present but implicit/checkpoints scattered); not 4 because the main workflow steps are never laid out in order. | 3 / 5 |
Progressive Disclosure | `references/wiring.md` exists, is one level deep, and is well signaled with section anchors — but two referenced paths do not resolve in the bundle ([README.md](README.md) and [templates/ci.yml](templates/ci.yml) are missing), and the change-class mega-paragraph inlines detail that belongs in a reference file. Fits anchor 3; not 4 because dangling references and inlined bulk are more than minor organization gaps. | 3 / 5 |
Total | 13 / 20 Passed |