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 exceptionally actionable, well-validated orchestration skill: copy-paste commands, deterministic mode detection, idempotent resume, and bounded feedback loops. Weaknesses are limited to moderate repetition of the no-approval/no-human points and small inaccuracies in the References section's read-triggers.
Suggestions
Fix the References section trigger: the subagent prompt template is needed in Phase 2 (Tier 0) and Phase 3 (Parallel Tiers), not 'Phase 4', and release-strategy.md's 'when the user asks about releasing' should be reworded for a headless context (e.g., 'when preparing the eventual feature/<feature> -> main PR').
Trim repeated statements: the no-approval-gate point, the 'archaeological evidence' branch-retention rationale, and the Guidelines DO/DON'T recaps each appear multiple times and could be stated once.
Consider moving the Signal Processing and TODO Structure sections into a reference file, since they describe subagent-side behavior already covered by the prompt template reference.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense procedural instruction (commands, state-recovery matrices, naming conventions) with little concept explanation, but repetition is noticeable: the no-approval-gate point is restated roughly eight times, the 'archaeological evidence' branch-retention rationale appears three times, and the Guidelines DO/DON'T section largely recaps body content. This matches anchor 4 (minor instances that could be trimmed) rather than 5 (every token earns its place). | 4 / 5 |
Actionability | Fully executable guidance throughout: copy-paste bash for worktree creation and PR merges ('git worktree add ... -b ... feature/<feature>', 'gh pr merge <pr-number> --merge'), deterministic pre-condition checks, exact PR-state lookup commands ('gh pr view slice/<number>-<kebab-name> --json state -q .state'), exact naming conventions, and a concrete script invocation with parsed JSON output. Matches the 5 anchor. | 5 / 5 |
Workflow Clarity | Phases 0-5 are clearly sequenced with explicit validation (pre-conditions, preflight bootstrap check, signal fix-retry loop bounded by max_signal_iterations: 3), idempotent resume detection, merge-conflict recovery, failure-to-exit-to-dispatcher-retry feedback loops, and TODO checklists. Slight phase-numbering overlap between sequential and parallel modes is mitigated by per-section 'Applies only to' labels, so it matches 5 rather than 4. | 5 / 5 |
Progressive Disclosure | Three real, one-level-deep reference files (subagent-prompt-template.md, error-handling.md, release-strategy.md) each with a when-to-read signal, plus a scripts directory; all verified to exist. Minor gaps: the References section says to read the subagent template 'when spawning subagents in Phase 4' (spawning actually happens in Phase 2/Tier 0 and Phase 3), and release-strategy.md is triggered by 'when the user asks about releasing' in a skill that explicitly has no human in the loop. Matches anchor 4 (minor organization gaps) rather than 5. | 4 / 5 |
Total | 18 / 20 Passed |