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.
The body is an exceptionally actionable, well-sequenced orchestration workflow with strong validation checkpoints and error-recovery loops, and it makes sound use of a one-level-deep reference file for the full gotchas table. Its main cost is token efficiency: the branch-attribution rationale is duplicated across two long passages and padded with repository spend statistics.
Suggestions
Explain the TASK-BRANCH attribution mechanism once — collapse the precondition warning block and the first Step 3 dispatch bullet into a single statement of the rule (every dispatch carries 'TASK-BRANCH: <branch>' or its spend is misattributed) plus one pointer to wiki/observability/session-metrics.md for the mechanism.
Move the spend-statistics justification (the 64% HEAD-pool figure, the 85.8% subagent-spend figure, and the gitBranch-capture mechanics) out of the main flow into references/gotchas.md or the observability wiki page, keeping only the operational instruction in SKILL.md.
Trim the rationale prose in Step 3/Step 4 (e.g. why an implementer reliably does 'create the file' and skips 'make something call it', why the shepherd pins the cheapest model) to one clause each; the surrounding imperative already carries the instruction.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient operational instruction, but the TASK-BRANCH attribution rationale is explained at length twice (the warning block and the first Step 3 dispatch bullet), and motivational spend statistics ("64% of this repository's recorded token spend currently sits in that anonymous HEAD pool", "It is 85.8% of subagent spend today") pad otherwise actionable sections. Fits the 3 anchor (mostly efficient, some unnecessary explanation) rather than 4, because the duplication is substantive rather than a minor trim. | 3 / 5 |
Actionability | Fully executable throughout: concrete git commands ("git -C /Users/ac/.work/osn.git worktree add ... -b <prefix>/<dir> origin/main"), GH commands ("gh pr merge <n> --squash --delete-branch", "gh pr view <n> --json mergeStateStatus,statusCheckRollup,state"), verification commands ("git -C <dir> status --porcelain", "bun run --cwd tools/pr-metrics backfill -- --dry-run"), and a concrete dispatch contract with a verbatim "TASK-BRANCH: <branch>" marker. Matches the 5 anchor; nothing is pseudocode or hand-wavy. | 5 / 5 |
Workflow Clarity | Steps 00 → 0 → 1–5 are clearly sequenced with explicit validation checkpoints (mechanical verification before reading a report, "Never merge red", re-verify after fixes, the post-run attribution backfill check) and dedicated feedback loops for error recovery (dead subagent recovery, DIRTY/BEHIND rebase path, NEEDS INPUT handling). Matches the 5 anchor exactly. | 5 / 5 |
Progressive Disclosure | Good structure: the top five gotchas are inlined with the full table clearly signaled one level deep in references/gotchas.md (verified to exist and to carry that table), and sections are well-organized for navigation. Fits the 4 anchor rather than 5 because a large attribution warning block and other long rationale sections are inlined content that could itself live in a reference file. | 4 / 5 |
Total | 17 / 20 Passed |