Content
92%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 unusually disciplined procedure skill: every step is executable, validation is layered around an admin merge that bypasses CI (head-OID matching, live PR re-reads, post-merge run monitoring with honest 'unknown, not green' semantics), and the structure is clean with no filler. The only cost is minor wordiness in the lane-recovery and head-moved passages, including one duplicated recovery instruction.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with specifics and assumes competence (no explanation of what CI or a squash merge is), but has minor trims available: step 5 states the head-moved recovery twice ("If it doesn't, someone pushed: redo steps 2-4" and "If the head moved, someone pushed. Redo steps 2-4"), and step 3's lane-recovery explanation runs long. Fits anchor 4 ('Efficient; minor instances of over-explanation that could be trimmed'), not 5's every-token-earns-its-place. | 4 / 5 |
Actionability | Every step carries copy-paste-ready commands: `gh pr list --head ... --json number,url,author`, `pnpm prep:urgent`, `corepack pnpm ship:push -m ...`, `gh pr merge <number> --squash --admin --match-head-commit <head-oid>`, plus post-merge `gh run list --commit ... --json ...`. The only placeholder is the commit subject, which is inherently task-specific. Matches 'Fully executable; copy-paste ready code or commands'. | 5 / 5 |
Workflow Clarity | The six steps are clearly sequenced with explicit validation checkpoints and feedback loops around a risky admin merge: re-reading the live PR (open, base main, ownership), matching `headRefOid` to `git rev-parse HEAD`, `--match-head-commit` on the merge, redo-steps-2-4 recovery when the head moved, confirming the merge commit in `origin/main`, and post-merge monitoring that treats failed/empty queries as 'unknown, not green'. The destructive-operation cap at 3 does not apply because validation is thorough. | 5 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/ directories), and none are needed: the body is a single cohesive ~87-line procedure with clear sections (## Steps, ## After the merge) and no content that belongs in a separate file. References to sibling skills (`ship`, `babysit-pr`, `/new-branch`, `ship-and-monitor`) are one-level and clearly named, matching anchor 5's easy navigation. | 5 / 5 |
Total | 19 / 20 Passed |