Content
81%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.
A well-engineered workflow skill: rigorous sequencing, abundant validation gates, and deliberate reuse of the sibling review-pr skill to avoid duplicating security-critical blocks. The deductions are for justificatory prose that could be trimmed and for the fact that several critical steps are only executable by consulting the sibling skill.
Suggestions
Trim the multi-line rationalizations (the `|| true` bash-vs-zsh verification story, the trust-model framing prose) to one line each; the instruction survives without the argument.
Inline at least a skeleton of the verification block (or ship it as a bundled reference file) so Step 5 is executable without opening the sibling review-pr skill.
Move the charter template and Agent dispatch prompt into a referenced file if the sibling skill already defines them, keeping SKILL.md to the deltas.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and mostly earns its tokens — every step carries non-obvious, calibrated detail ('Do not "fix" it to `$BASE...HEAD`', 'a dirty tree has no stable identity... can never be deduped'). But there are minor instances of over-explanation that could be trimmed, e.g., the multi-line justification of `|| true` shell behavior and trust-model prose like 'Be honest about that rather than implying a boundary that does not exist'. Fits the 4 anchor; not 5 because not every token earns its place, and not 3 because the padding is minor rather than 'several unnecessary explanations'. | 4 / 5 |
Actionability | Mostly executable guidance: complete bash blocks with variables and error handling, a fill-in charter template, the full Agent dispatch prompt, and the draft frontmatter spec. Not 5 because critical parts are delegated by name to the sibling skill rather than being copy-paste ready — 'Use `review-pr`'s verification block exactly... the single-`verdict` shell gate' and 'the prompt shape from `review-pr` Step 5' require reading another skill to execute. Not 3 because everything actually present is concrete and executable, not pseudocode. | 4 / 5 |
Workflow Clarity | Seven clearly sequenced steps with explicit validation checkpoints and feedback loops: `test -n "$BASE" || FATAL`, the empty-diff `test -s` gate, write-then-verify-then-move, the 'Refuse to run on the default branch' check, Step 5's per-agent evidence verification with a retry-once rule ('re-dispatch once, demanding a line in the far half of the file'), and sandbox cleanup. Matches the 5 anchor — clear sequence, explicit validation, error-recovery loops. | 5 / 5 |
Progressive Disclosure | Good structure: clean step sections, and it deliberately delegates bulk to the sibling skill with clearly signaled references ('Read it, don't reimplement it', 'Same shape as `review-pr` Step 5') instead of duplicating security-critical shell — sound one-level-deep disclosure. Not 5 because no bundle files exist and the references point at another skill's sections rather than bundled files, and the ~215-line body keeps the charter and dispatch templates inline; not 3 because organization is clearly better than 'references present but not clearly signaled'. | 4 / 5 |
Total | 17 / 20 Passed |