Content
82%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 lean, high-signal instruction-only skill body: a clearly sequenced maintainer workflow with explicit validation gates, HOLD conditions, and evidence requirements, delegating deeper protocols to clearly signaled one-level-deep shared files. Its only weaknesses are the absence of any concrete commands or examples inline and its reliance on external shared files for several checkpoint definitions.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense, imperative policy text with no concept explanations, no filler, and no padded sections; nearly every clause encodes a non-obvious project rule ("Base-red does not waive a required gate", "A locked fork needs the owner-reviewed credit-preserving fallback"). Every token earns its place by assuming Claude's competence. | 5 / 5 |
Actionability | Concrete, executable process guidance throughout: one isolated worktree per PR, verify the checked-out SHA equals its observed live head before editing, run a RED reproduction then focused and affected suites, and a structured receipt with named fields (base SHA, head SHA, run IDs, artifact references). Not 5 because there are no copy-paste commands/examples and several directives defer entirely to shared files (merge-gates, teardown protocol) without restating key steps. | 4 / 5 |
Workflow Clarity | A clear three-phase sequence (resolve scope and ownership, prepare and validate, evidence and handoff) with explicit pre-flight checkpoints ("Before pushing, verify the remote still equals the observed predecessor; a concurrent change means HOLD") and a full result classification taxonomy. As a batch/destructive-adjacent workflow it has validation, so the cap at 3 does not apply; it stays below 5 because error recovery beyond HOLD lacks explicit fix-and-retry loops and some checkpoint definitions live in external shared files. | 4 / 5 |
Progressive Disclosure | Well-organized sections with the body kept as an overview; shared references are one level deep and clearly signaled with when-to-read conditions ("Read `../_shared/protected-surfaces.md` before any instruction-surface change or merge"). Not 5 because references use bare backtick paths outside the bundle rather than clearly navigable in-bundle files, and no bundle files (references/, scripts/, assets/) exist in this skill directory to verify against. | 4 / 5 |
Total | 17 / 20 Passed |