Content
92%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 well-engineered orchestration body: concrete gate commands, explicit validation checkpoints with stop-and-fix loops, and a clean one-level reference structure fully backed by real bundle files. Its only real cost is density and some restated rules that could be trimmed for token efficiency.
Suggestions
State the two approval stops once (Critical Rules) and have the gate table and Before Execute section reference that rule instead of restating it, cutting ~4 lines.
Move the batch-packing budget detail out of the Before Execute paragraph and keep it solely in Sub-Agent Delegation, leaving a one-line pointer behind.
Replace a few of the longest compound sentences (e.g., Critical Rule 4 and the screens sub-agent paragraph) with short imperative bullets; the rules are the same length but stop requiring re-parsing.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes Claude's competence — "Never invent an API, a pattern or a behaviour" — with no basic-concept explanations, matching the efficient anchor. It falls short of lean because of minor redundancy that could be trimmed: approvals are restated in three places (Critical Rule 1, the gate table's "--execute" row, and the Before Execute section), and the sub-agent batching budget is explained twice (Before Execute and Sub-Agent Delegation). | 4 / 5 |
Actionability | Guidance is fully executable: a gate table gives copy-paste-ready commands per moment ("check_commit.py --message \"<msg>\"", "check_proofs.py <feature> [--mutations <mutations.json>]") with the path convention pinned earlier ("python3 <skill-dir>/scripts/<name>.py"), plus an exact .specs file tree and exact output formats like "**Approved**: <date> - user: \"<their words>\"". Placeholders are parameterized deliberately, and the rule "Skip a script only when no code-execution tool is available; then perform the same checks by reading the artifact" covers the fallback case. | 5 / 5 |
Workflow Clarity | The multi-step process is clearly sequenced (Specify → Design → Tasks → Execute with an auto-sizing matrix) and validation is explicit with feedback loops: "A non-zero exit means STOP and fix", a safety valve ("If that listing reveals >5 steps or complex dependencies, STOP and create a formal tasks.md"), and resume conflict rules ("continue when the files and git agree; stop only on a conflict"). Checkpoints are tied to concrete moments in a When/Command table. | 5 / 5 |
Progressive Disclosure | The body is an overview with all detail pushed one level deep: every one of the 11 unique reference paths cited (specify.md, implement.md, verify-screens.md, etc.) exists in references/, every script named exists in scripts/, and the Commands tables map trigger patterns to their reference file. References are clearly signaled at their decision points, not buried. | 5 / 5 |
Total | 19 / 20 Passed |