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.
The body is a tight, highly actionable contract: exact commands, exact tool names, a copy-paste child prompt, a decision-tree routing workflow with explicit failure guards, and a success checklist. Its only weaknesses are mild repetition of the worktree rule and a single-file structure that could split templates into a reference, neither of which meaningfully hurts usability.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean, imperative, and assumes Claude's competence — no concepts are over-explained — with only minor redundancy: the worktree prohibition is stated in the Core Contract ('Do not create or suggest worktrees'), the child prompt template ('Do not create a worktree'), and Success Criteria ('Worktrees are not created'). That repetition could be trimmed but the rest earns its tokens, so it sits above anchor 3 and below the perfectly lean 5. | 4 / 5 |
Actionability | Guidance is copy-paste ready: exact commands ('$orchestrator on/off/status'), seven exact tool names to search for, a ready-to-send child prompt template, a thread title format ('<BRANCH-OR-TICKET> <short task title>'), and a concrete status table schema. These cover the common creation, routing, and status cases the skill will actually face. | 5 / 5 |
Workflow Clarity | Routing Rules is a 9-step sequence with explicit decision points (per-branch vs not, branch key resolution fallbacks, found vs not found) and error-recovery guards: 'If mode state is unclear, ask once before executing locally', 'If these tools are not available, stop and report', and a required child closeout report of branch key, PR URL, blockers, and next owner. The Success Criteria section functions as an explicit checklist. This is not the 4 anchor because checkpoints are not merely 'mostly present' — every risky branch (missing mapping, missing tools, unclear mode, concurrent same-checkout work) has an explicit guard. | 5 / 5 |
Progressive Disclosure | The skill is a single ~150-line file with no bundle directories (references/, scripts/, assets/ do not exist), and the body makes no dangling file references — all sections are well-labeled and navigable. It is not the 5 anchor because that expects clearly signaled one-level-deep references splitting detail out (e.g. the child prompt template or status table could live in a reference file), though at this length a single well-organized file is defensible. | 4 / 5 |
Total | 18 / 20 Passed |