Content
85%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is a strong, actionable, well-sequenced workflow with explicit validation gates and feedback loops, delegating monitoring detail to sibling skills rather than duplicating it. Its main weakness is redundancy: the shipping invariant and production-verification guidance are restated across the invariant section, step 9, and the Important section.
Suggestions
Consolidate the 'commit all non-gitignored local changes including other agents' work' rule so it is stated once authoritatively (e.g., in the invariant) and referenced rather than repeated in steps 2, 4, and Important.
Merge the production deploy/publish verification guidance in the invariant section with step 9 to remove the near-duplicate treatment of the same retrigger-vs-fix-and-ship logic.
Trim the Important bullets that restate babysit/new-branch delegation already covered in steps 7 and 10 to tighten the token budget.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient operational guidance with no explaining of concepts Claude already knows, but the 'Non-Negotiable Shipping Invariant', step 9, and the 'Important' section each restate the 'commit all non-gitignored local changes including other agents' work' and production-verification points, so it could be tightened rather than earning the lean-every-token-earns-its-place anchor at 3. | 2 / 3 |
Actionability | Provides fully executable, specific commands — `git status --short`, `git diff --stat`, `git push -u origin <branch>`, `gh pr merge <number> --squash --admin`, `/babysit-pr <number>`, `/new-branch`, `pnpm run prep` — copy-paste ready, matching the executable anchor rather than the pseudocode/abstract anchors at 1–2. | 3 / 3 |
Workflow Clarity | An explicit 11-step sequence with validation checkpoints (merge gates holding for 10 consecutive minutes, clean tree, green CI, addressed comments) and clear feedback loops (retrigger transient failures vs fix-and-ship code failures), matching the clear-sequence-with-validation anchor rather than the implicit-checkpoint anchors at 1–2. | 3 / 3 |
Progressive Disclosure | No bundle files exist; the skill is self-contained and well-sectioned (Steps, Important) and delegates detail to sibling skills (/babysit-pr, /new-branch) as source-of-truth rather than nesting references, so per the simple-skills scoring note a well-organized single file scores 3 rather than the monolithic or poorly-signaled anchors at 1–2. | 3 / 3 |
Total | 11 / 12 Passed |