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.
A dense, high-discipline orchestration body: a clearly gated lifecycle with pervasive verification and concrete commands at the operational steps, delegating mechanics to a well-signaled one-level-deep reference. Its weaknesses are repetition of the proof rule across sections, a merge step that lacks a literal command, and a referenced template file missing from the bundle.
Suggestions
Consolidate the review/CI-proof requirement — currently stated in the Role contract, §4 (completion verification), and §6 (pre-merge re-verification) — into one named checkpoint definition that both sections reference, trimming repeated tokens without losing the double-check semantics.
Make the merge step executable as written: include the exact merge command and method (e.g., the `gh pr merge` invocation and merge/squash/rebase choice, or a pointer to where launching.md defines it).
Ship `templates/orchestration-run.md` in the bundle (or inline its essential run-record and final-handoff fields), since the body calls it "the required durable run format" but the file is not present next to references/launching.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, directive, and assumes competence — it never explains git worktrees, CI, or PR mechanics — but the review/CI-proof requirement is stated three times (Role contract, §4 completion, §6 merge) and the max-parallel rule twice, which is deliberate emphasis yet trimmable. That matches anchor 4 ("minor instances that could be trimmed") better than the every-token-earns-its-place anchor 5. | 4 / 5 |
Actionability | Concrete, executable material is abundant: a complete 12-line bash PRP-store resolver, `timeout 1800 gh pr checks <n> --required --watch --fail-fast`, `gh pr diff <n> --name-only`, exact paths like `$PRP_DIR/orchestration/<run-id>.md` with a `YYYY-MM-DD-<slug>` run-ID format, and an engine-selection mapping. Minor gaps remain — no literal merge command or method, and digest/handoff formats are deferred to a template — matching anchor 4 over the fully copy-paste-ready anchor 5. | 4 / 5 |
Workflow Clarity | A clearly sequenced 7-phase lifecycle (Intake/gate → Initialize → Launch → Monitor/steer → Hold gates → Integrate → Close out) with explicit validation at every phase: review and CI proof re-verified "before acceptance and again before merge", merge-commit reachability from `origin/<base>`, resume verification against live state, and destructive-action gates ("Never delete a branch or worktree with unmerged commits"). Feedback loops (watcher timeout re-arm, two-restart escalation, rebase/conflict handling) and a 7-question design checklist satisfy the anchor-5 pattern; the batch/destructive validation cap does not apply. | 5 / 5 |
Progressive Disclosure | The Resources section clearly signals `references/launching.md` with its contents and read timing ("Read it before the first launch"), and the file exists, is one level deep, and delivers what is promised. However, the body also references `templates/orchestration-run.md` ("the required durable run format") twice, and that file is absent from the bundle — a minor navigation gap matching anchor 4 rather than the fully clean anchor 5. | 4 / 5 |
Total | 17 / 20 Passed |