Content
77%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 and well-sequenced lifecycle skill: executable commands at every step, explicit gates, feedback loops, and failure handling. Its weaknesses are repetition of the same prohibitions (polling, quota-skip) three-plus times each, and a monolithic ~400-line body that inlines templates and monitor recipes which belong in one-level-deep reference files.
Suggestions
Extract the PR body template and the three monitor subscription recipes (CI completion, Cubic review arrival, merge completion) into a single one-level-deep references file, keeping SKILL.md as the lifecycle overview with well-signaled pointers.
State each repeated rule once: the 'never block a model round-trip on --watch / polling loops' guidance and the 'Cubic SKIPPED only on quota exhaustion' rule each appear 3+ times across phases, the verify loop, Gate B, Phase 4, and the anti-patterns table — consolidate to a single authoritative statement and cross-reference it.
Deduplicate the anti-patterns table against in-body rules (e.g., the 200-vs-2000-line atomicity rationale and the scope-creep warning each appear twice) so each violation is explained in one place.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient — no tutorial filler, no explanation of concepts Claude already knows — but it states the same rules multiple times: 'never block a model round-trip on `gh pr checks --watch`' appears in the Gate A prose, the monitor comment block, and the Cubic section ('never spin a `for _ in $(seq 1 30)` polling loop'), and the Cubic-quota-skip rule is repeated in the Phase 3 intro, the verify-loop pseudocode, Gate B, Phase 4, and the anti-patterns table; the 200-vs-2000-line review rationale is also stated twice. This fits anchor 3 ('mostly efficient… could be tightened') rather than 4, where only minor trimming would be needed. | 3 / 5 |
Actionability | Every phase is backed by copy-paste-ready commands: worktree/branch setup bash, pre-push checks, `gh pr create` with a fully specified PR body template, Cubic review retrieval with a three-branch grep classifier, CI log retrieval, auto-merge commands, and monitor subscription recipes. The only pseudo-code (the `while true` loop and `monitor({...})` blocks) represents harness invocations where that is the native form. This matches anchor 5. | 5 / 5 |
Workflow Clarity | The multi-step process is clearly sequenced (Phase 0–4 with an architecture diagram up front), gates are ordered with stated rationale (CI cheapest, Cubic asynchronous), the failure route back to Phase 1 is explicit, and validation checkpoints abound: evidence-bound manual QA before commit, pre-push local validation, the two-gate verify loop, iteration discipline, failure recovery, and an anti-patterns severity table. This is a textbook match for anchor 5's feedback-loop and checklist profile. | 5 / 5 |
Progressive Disclosure | The body is well-sectioned (phases plus <setup>/<implementation>/<verify_loop> tags), but it is a ~400-line monolithic file with no bundle: the PR body template, the three monitor subscription recipes, and the Cubic parsing/classification logic are content that would sit better in one-level-deep reference files. That fits anchor 3 ('some structure… content that should be separate is inline') rather than 4, which requires most content appropriately placed across files. | 3 / 5 |
Total | 16 / 20 Passed |