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 unusually well-engineered orchestration procedure: every step is executable, validation is baked into each stage, and error recovery is specified for every terminal state. The weakness is token efficiency — the ~286-line body carries justification prose and stale-prone statistics ('64% today', '85.8% today') that belong in a reference file or should be trimmed.
Suggestions
Move the precondition's bare-repo-vs-main rationale (gitBranch attribution mechanics, the 64%/85.8% spend figures) into `references/` alongside gotchas.md, keeping only the two expect-style check commands and the stop message in SKILL.md.
Replace the time-sensitive statistics ('currently sits', 'today') with the durable rule they justify, or move them to a metrics reference — they will silently go stale and mislead later runs.
Merge the TASK-BRANCH warning blockquote with Step 3's dispatch-contract bullet on the same marker; they state the same attribution mechanics twice, and one authoritative statement would cut ~20 lines.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with genuinely operational, repo-specific knowledge rather than concepts Claude already knows, but several passages could be tightened: the precondition section spends three paragraphs justifying `main/` over the bare root, and time-sensitive figures ('64% of this repository's recorded token spend currently sits', 'It is 85.8% of subagent spend today') pad the text and will go stale. The long TASK-BRANCH warning blockquote also repeats attribution mechanics already stated in Step 3. | 3 / 5 |
Actionability | Fully executable throughout: exact commands (`git worktree add ... -b <prefix>/<dir> origin/main`, `gh pr view <n> --json mergeStateStatus,statusCheckRollup,state`, `gh pr merge <n> --squash --delete-branch`, `bun run --cwd tools/pr-metrics backfill -- --dry-run`), a concrete dispatch contract with verbatim required lines (`TASK-BRANCH: <branch>`), and an explicit `NEEDS INPUT: <question> + options + recommendation` protocol covering the common cases. | 5 / 5 |
Workflow Clarity | Steps 00, 0, and 1–5 are clearly sequenced with explicit validation checkpoints everywhere: the ORCHESTRATE.md blackboard with 'NOT RUN' as an honest gate value, Step 4's mechanical verification before trusting any report, explicit error-recovery branches (DIRTY/BEHIND → rebase, check failure → fix subagent → re-poll, never merge red), and a dedicated subagent-death recovery procedure. | 5 / 5 |
Progressive Disclosure | Good structure: the full gotcha table is deferred to `references/gotchas.md` (which exists, one level deep, clearly signalled: 'The full table, with the reason behind each, is `references/gotchas.md`. The ones that cost a run') with only the top five inlined, and wiki conventions are referenced by path. Minor gaps: the long precondition rationale and attribution-warning blockquote are inlined operational lore that could live in a reference file like the gotchas do. | 4 / 5 |
Total | 17 / 20 Passed |