Content
71%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 exceptionally actionable reference: exact commands, a complete configuration table, precise formats, explicit hook chains, and error remedies. Its weaknesses are density — long subordinated prose with restated scope rules — and inlined deep-dive material alongside two doc references that do not resolve in the bundle.
Suggestions
Move the pre-push scope derivation and the generated-inventory error handling into a reference document (or into the already-cited DEVELOPMENT.md/CHECKS.md), keeping a two-to-three line summary plus the link in SKILL.md.
Tighten the pre-push section: state the scope rule once, drop the repeated byte-ceiling/baseline restatements and the duplicated DEVELOPMENT.md scope link within the same passage.
Ship CHECKS.md and README.md inside the bundle (or link their canonical hosted URLs the way DEVELOPMENT.md is linked) so the relative references resolve where the skill is installed.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | There is no beginner-concept padding and every sentence carries real configuration or behavior detail, but the prose is baroque ("Two refusals keep the verdict about the thing being pushed, since every range scope there ends at HEAD...") and restates facts — the byte-ceiling baseline rule appears twice in one paragraph and the index/HEAD scope rule repeats across paragraphs — so noticeable tightening is possible. It is above anchor 2 because nothing is filler or material Claude already knows. | 3 / 5 |
Actionability | The opening block gives copy-paste-ready commands with flags ("commit-guards all --staged", "install-git-hooks --check"), the checks table states exact per-check verdict rules, the configuration table enumerates every key with its default and meaning, and the excludes/baseline line formats are specified precisely — fully executable guidance covering the common cases. | 5 / 5 |
Workflow Clarity | The pre-commit order is given as an explicit chain ("doc-limits --staged when installed -> preflight --staged when installed -> one review-file check -> commit-guards all --staged"), exit codes are defined, "install-git-hooks --check" serves as a read-only verification step, and failure remedies are explicit (run "kendex refresh", install jq). It falls short of anchor 5 because no ordered procedure exists for the skill's own load-time tasks, such as enabling byte-ceiling or adding a local pre-commit entry. | 4 / 5 |
Progressive Disclosure | Section headers are clear, links are well signaled, and every cited script path (scripts/commit-guards, scripts/md-refs, scripts/lib/comment-text.sh, etc.) resolves in the bundle — but the multi-paragraph pre-push scope derivation and the generated-inventory jq error handling are inlined detail that belongs in a reference file, and the relative links to CHECKS.md and README.md do not resolve within this bundle, leaving "Full check shapes and scopes" unreachable locally. Structure and signaling are too good for anchor 2, but the inlining and dangling references block anchor 4. | 3 / 5 |
Total | 15 / 20 Passed |