Content
88%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 excellent orchestration skill: fully executable commands, an explicit dispatch contract, validation before trust at every gate, and recovery procedures for every failure mode it names. The costs are the duplicated TASK-BRANCH attribution rationale across the precondition warning and Step 3, and inline material (the long attribution warning) that would sit better in references/gotchas.md or a dedicated reference.
Suggestions
Explain the TASK-BRANCH attribution mechanism once — either in the precondition warning or in Step 3's bullet — and cross-reference it from the other location; the gitBranch-inheritance rationale and the spend percentages currently appear in both places.
Move the extended attribution warning (gitBranch capture, collector mechanics, backfill verification) into references/gotchas.md or a short references/attribution.md, keeping only the 'TASK-BRANCH: <branch> on every dispatch, every time' rule inline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and nearly every line carries non-obvious operational knowledge (the ORCHESTRATE.md block format, the dispatch contract, the dead-subagent recovery ladder) rather than concepts Claude already knows. It misses a 5 because the TASK-BRANCH attribution rationale is explained twice — once in the precondition warning block (~20 lines on gitBranch inheritance and the collector) and again in Step 3's first bullet ('It is 85.8% of subagent spend today') — and both passages could be trimmed to one. | 4 / 5 |
Actionability | Guidance is copy-paste executable throughout: exact commands with full paths ('git -C /Users/ac/.work/englishstreetventures.com.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 blackboard template, and a dispatch checklist where every placeholder (<prefix>, <dir>) is defined. Specific examples cover the common cases (rebase conflicts, check failures, draft PRs). | 5 / 5 |
Workflow Clarity | Steps 00, 0, and 1–5 are clearly sequenced with explicit validation checkpoints and feedback loops: Step 4 mandates verifying the mechanical half before trusting any report ('the commit exists and the tree is clean... every JSON and TOML the branch touched still parses'), 'Never merge red' plus fix-subagent re-poll loops, terminal-state handling for DIRTY/BEHIND, and a dedicated dead-subagent recovery procedure. For a batch/destructive skill (merges, force-pushes, worktree teardowns) this matches the score-5 anchor including error-recovery loops. | 5 / 5 |
Progressive Disclosure | Good structure: the gotchas table is correctly split out ('The full table, with the reason behind each, is references/gotchas.md. The ones that cost a run:' inlines only the top failures), references are one level deep and clearly signaled, and the referenced file exists in the bundle. It misses a 5 because the ~20-line attribution warning block is inline content that belongs in a reference or a tighter form, and several referenced wiki paths (session-metrics.md, stacked-prs.md, review-findings.md) are repo-dependent rather than part of the skill's own bundle, leaving the skill body carrying detail it could point to. | 4 / 5 |
Total | 18 / 20 Passed |