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 short skill body: dense, executable, and honest about scope, with a clear failure/remediation loop and clean one-level-deep disclosure into a real, well-organized reference file. The only gap is that the configuration-editing use cases (class entries, exclusion rows) have no inline example, though they are one click away in policy.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every sentence carries load: the ~150-line body covers when to run the check ("before review and in CI"), both invocation modes, the measured set ("`AGENTS.md`, `CLAUDE.md`, `GEMINI.md` and `SKILL.md`, at the root and at any depth, and no other file"), failure semantics, and remediation — with zero padding and no explanation of concepts Claude already knows. Matches the anchor 'lean and efficient; assumes Claude's competence; every token earns its place'. | 5 / 5 |
Actionability | The body gives two copy-paste-ready executable commands ("doc-limits" and "doc-limits --staged") plus a concrete pointer for the rest ("Flags, settings and exit codes are in `doc-limits --help`"), matching 'mostly executable guidance; concrete code or commands with minor gaps'. Not 5 because the add/tune use cases promised by the description (a class entry, an excludes row) have no inline example — both are one hop away in references/policy.md; not 3 because the guidance given is concrete and executable, not high-level hints or pseudocode. | 4 / 5 |
Workflow Clarity | This is a simple single-action skill and the action is unambiguous: when to run it ("before review and in CI", and which commit-guards chains use staged mode, including why — "at push the index is held equal to HEAD first, so that mode measures the tree being pushed"), how to run it (both commands), what failure means ("A document over its limit fails the check"), and the remediation loop ("A document that must stay whole gets a row in the configured excludes file with its reason"). The check is itself the validation step, so the batch-operation validation cap does not apply. | 5 / 5 |
Progressive Disclosure | Verified against the actual bundle: references/policy.md exists and contains exactly the promised material (Path classes, exclusion list format), scripts/doc-limits is a real executable, and the body's links ([references/policy.md § Path classes](references/policy.md#path-classes)) are one level deep, well-signaled with a section anchor, with all detail (class format, exclusion rows, flags, exit codes) appropriately split out. Matches 'clear overview with well-signaled one-level-deep references; content appropriately split; easy navigation'. | 5 / 5 |
Total | 19 / 20 Passed |