Content
77%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 is highly actionable and operationally rigorous: commands, exit codes, triage rules, and feedback loops leave no ambiguity about how to run the gate and respond to failures. Its weaknesses are density and structure — roughly half the file is reference-grade findings and validation history inlined into SKILL.md, which hurts token efficiency and works against progressive disclosure.
Suggestions
Move the "Findings" class catalog and the "What a pass proves" planted-bug history into a one-level-deep reference file (e.g. FINDINGS.md) and keep a short summary with a clearly signaled link, reducing SKILL.md's token footprint.
Tighten the dense prose in "What it checks" — the enumerated token lists and fixed-source explanations could be condensed to the operationally relevant facts, since the script itself enumerates the cases.
Consider moving the coverage-record table and its git re-check procedure into a separate reference so the main body focuses on run, read, and triage workflows.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | There is no padding or explanation of concepts Claude already knows, but the ~17KB body inlines reference-grade material — the 157-finding catalog by class, the planted-bug "What a pass proves" history with commit hashes, and sync-commit provenance — that could be tightened or moved. Not 4: the trimming opportunity is more than minor; large sections read as a verification report rather than operating instructions. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready commands ("pnpm check:markdown-round-trip", "--inline=3 --part=round-trip", "--case='<identity or part of one>'", "--lower", "--jobs=<n>") with exit-code semantics, timing/case-count expectations, real example finding output, and a concrete three-way triage procedure. These cover the common cases. | 5 / 5 |
Workflow Clarity | The multi-step process is clearly sequenced (run the gate → read a failure → triage real bug vs modeling error vs intended change → fix → run --lower) with explicit validation checkpoints (exit 0/1/2 meanings, the GATE_CASES count check, baseline comparison) and feedback loops ("the gate fails until you run pnpm check:markdown-round-trip --lower"; "Never weaken a rule or add a baseline entry to get green"), plus a three-point re-check checklist for the coverage record. As a batch operation it has validation, so the cap at 3 does not apply. | 5 / 5 |
Progressive Disclosure | No bundle files exist, and while the single SKILL.md has well-organized section headers, substantial content that belongs in separate one-level-deep reference files — the Findings class catalog, the planted-bug validation history, and the coverage-record table — is inlined in the monolithic body. Not 4: the inlining is substantial rather than a minor organization gap; not 2: the section structure itself is clear and easy to navigate. | 3 / 5 |
Total | 16 / 20 Passed |