Content
85%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, well-sequenced command skill with strong validation checkpoints, explicit feedback loops, and guardrails — among the strongest workflow content one could ask for. Its one real defect is bundle integrity: most of the rules/*.md files it delegates to are absent from the bundle, leaving load-bearing steps (especially split mode and the description contract) unnavigable.
Suggestions
Ship the missing rules/ files in the bundle — or inline the load-bearing ones (split-mode procedure, pre-push skip conditions, description contract) — since 7 of 8 referenced local files are currently dangling.
Verify relative paths after any move: links like ../../quality/review-loop/SKILL.md and ../../../agents/shared/rules/github-access.md silently break outside the expected repo layout.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operational — no explanations of git/GitHub/CI concepts Claude already knows, and nearly every line is a non-obvious harness fact ("gh pr checks exits non-zero (8) while checks are merely pending", "Issue each watch call with the Bash tool's timeout parameter explicitly set to 600000"). Minor trimming is possible: skip conditions are stated both in the Modes table and again per-step ("Skip entirely on --no-quality, --no-post-review..."), and the "Record which branch you took, now" motif repeats across steps — that keeps it at 4 rather than 5. | 4 / 5 |
Actionability | Guidance is fully executable: copy-paste-ready commands ("gh pr create --draft --title ... --body ...", "timeout 540 gh pr checks <pr-number> --watch", "Skill("ci-auto-fix", "<pr-url>")"), exact exit-code-to-action tables (0/124/127/other), a precedence-ordered flag table, and enum-valued report slots. It matches the 5 anchor — concrete commands with the common cases specifically covered. | 5 / 5 |
Workflow Clarity | Steps are explicitly sequenced with validation checkpoints at every phase: outcome values recorded before continuing, exit-code classification with literal stderr matches, retry caps with escalation ("up to 4 attempts total... After the fourth... escalate"), ci-auto-fix capped at 2 with re-invoke rules, and a mandatory Step 10 report checklist. Feedback loops and error-recovery paths are present throughout, matching the 5 anchor exactly. | 5 / 5 |
Progressive Disclosure | The structure itself is well designed — one-level-deep references, each with an explicit read-condition ("read when --split is set, before Step 1", "read now") — but scored against the actual bundle, 7 of the 8 referenced local files are missing: rules/description-contract.md, rules/description-examples.md, rules/legacy-flags.md, rules/pre-push-review.md, rules/attach-media-step.md, rules/registration-poll.md, and rules/split-mode.md do not exist, and split mode's entire procedure lives in one of them. Only references/design-rationale.md is real, so the navigation the body promises is broken in practice — better than the disorganized 2 anchor, but the missing load-bearing files preclude 4 or 5. | 3 / 5 |
Total | 17 / 20 Passed |