Content
62%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.
The body defines a thoroughly validated ship workflow with concrete commands, explicit gates, and recovery loops, but it is heavily padded: identical authorization, base-selection, and branch-disposition rules are restated three to four times across sections, and environment-specific procedures that belong in separate reference files are inlined. Actionability is mostly executable with the live-PR gate query left as prose.
Suggestions
Deduplicate the rules repeated across Contract, Flow, the Existing PR backlog section, and sections 5–6 (base selection, merge authorization, branch disposition, preserve-unrelated-work) — state each rule once in the section that owns it and reference it elsewhere.
Move the environment-specific procedures (the Codex get_goal/create_goal block and the verbatim Claude Code /goal message) into a references/ file so SKILL.md stays a navigable overview.
Replace the prose description of the live-PR ownership gate with the actual gh commands (login resolution plus the author-bearing `gh pr view` query) so every pre-push check is copy-paste executable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The same rules — base selection ("fresh origin/main for a new PR, the fetched live PR head for an existing PR update"), branch disposition, merge authorization, and preserve-unrelated-work — are restated three to four times across Contract, Flow, the Existing PR backlog section, and sections 1–6, and long environment-specific goal procedures are inlined. This goes beyond the 'some unnecessary explanation or could be tightened' of a 3; sections are noticeably padded with repetition. | 2 / 5 |
Actionability | Preflight git commands, the remote-aware unpushed-commit check with pathspec excludes, `corepack pnpm ship:push -m "..."`, and `gh pr merge <number> --squash --admin --match-head-commit <verified-head-oid>` are copy-paste ready. However, the live-PR ownership gate ("resolve the active GitHub login with `gh api user --jq .login`, include `author` in the live PR query") and the required-checks/mergeable readiness checks are described in prose without the actual commands, leaving minor gaps. | 4 / 5 |
Workflow Clarity | A numbered 7-step Flow maps to detailed sections 1–6 with explicit validation at every step: ownership classification before any branch movement, a 10-minute soak on the unchanged head, full gate revalidation immediately before merge, and feedback loops (restart the soak on head change or gate failure; conflict recovery only after the three stated preconditions). The guarded merge of a hard-to-reverse operation includes an explicit checklist, so the destructive/batch cap does not apply. | 5 / 5 |
Progressive Disclosure | The body has clear numbered sections and signals companion skills by slash-name (`/new-branch`, `/babysit-pr`, `/review-latest-feedback`), but no bundle files exist and ~400 lines of policy are inlined — including the Codex `get_goal`/`create_goal` procedure, the verbatim Claude Code `/goal` message, and the existing-PR backlog procedure — where separate reference files clearly belong. Structure is present but the split is not made. | 3 / 5 |
Total | 14 / 20 Passed |