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.
An exceptionally well-engineered operational skill: precise ownership rules, a safety-checked merge gate, and feedback loops throughout. The residual gaps are minor — a missing explicit PR-creation command, slight redundancy with the description, and a few sections that could be split into references if the file grows.
Suggestions
Add the explicit PR-creation command (e.g., `gh pr create --title ... --body ...`) to step 3 so every user-facing action has a copy-paste-ready command.
Trim the opening paragraph, which restates the description's pipeline, and consider moving the Codex goal/heartbeat management details into a runtime-specific reference so the body stays runtime-neutral.
If the file grows further, split the worktree rotation and throwaway-worktree pruning rules (step 6) into a reference file to keep SKILL.md a lean overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and imperative with zero concept explanations ("Permission to push is not permission to merge", "Never enable auto-merge"), assuming Claude's competence. Minor trimmable material keeps it below lean-perfect: the opening paragraph restates the description's scope, and the Codex goal/heartbeat paragraph carries environment-specific detail that not every runtime needs. | 4 / 5 |
Actionability | Mostly executable with copy-paste-ready commands: `gh pr merge <number> --squash --admin --match-head-commit <head-oid>`, `corepack pnpm ship:push -m "<subject>"`, `gh api user --jq .login`, `pnpm worktrees:prune --apply`. Minor gaps: PR creation is described ("Open one ready (not draft) PR") without an explicit command, and the validation step defers to `pnpm guards` selection rather than naming commands. | 4 / 5 |
Workflow Clarity | A clear numbered sequence (Preflight → Validate/push → Open PR → Babysit → Merge gate → After merge) with explicit validation checkpoints: a merge-gate checklist held for 10 minutes on an unchanged head, OID verification recorded before merging, `--match-head-commit` protection, and feedback loops for CI failures and review threads (fix → reply → one batched push → re-check). The semi-irreversible merge operation has extensive validation, so no cap applies. | 5 / 5 |
Progressive Disclosure | No bundle files exist; the single-file body (~130 lines) is well organized into numbered sections with clearly signaled references to sibling skills (/babysit-pr, /new-branch, /review-latest-feedback step 5). It exceeds the under-50-line simple-skill exception, and content like the worktree-rotation and throwaway-worktree pruning rules could arguably live in a reference file, placing it at good-structure-with-minor-gaps rather than fully appropriate splitting. | 4 / 5 |
Total | 17 / 20 Passed |