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 well-structured orchestration skill: the phase workflow is clearly sequenced with explicit validation, feedback loops, and stop conditions, and delegation/output examples are concrete. Weaknesses are mild redundancy in the stop-condition material, a boilerplate context-disposition paragraph, and a References section that undersignals which file to read when — including paths that escape the bundle.
Suggestions
Consolidate the overlapping stop conditions in Validation, Failure Mode, and Constraints into one authoritative stop-rule section to save tokens.
Annotate each References entry with a one-line 'read when' trigger (as phase-gate-contract.md does internally) so navigation is self-explanatory, and verify the ../../references/ paths actually resolve from the installed skill location.
Drop or rewrite the 'context-disposition policy' paragraph in the Procedure — it reads as generic boilerplate rather than executable guidance for this skill.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes competence — no concept explanations, just commands, contracts, and stop rules — earning above the score-3 anchor. It falls short of score 5 because stop conditions are restated across Validation, Failure Mode, and Constraints, and the 'context-disposition policy' paragraph in the Procedure reads as boilerplate padding that adds tokens without actionable value. | 4 / 5 |
Actionability | The Validation block gives concrete commands (git status --short, test -f, rg, ./bin/ask ... --json --robot), and the Delegation Examples and Output Template are copy-paste ready. Not score 5 because some steps remain abstract ('Refresh dashboards/browser', 'confirm continuation authority') without commands, and the validation commands carry unresolved placeholders like <approved-plan-path>. | 4 / 5 |
Workflow Clarity | The Procedure is a clearly ordered 1–7 sequence with an explicit validation step, a fail-fast rule ('stop at the first failed gate'), and an error-recovery feedback loop (route failures to he-work or he-code-review with the exact command and output). This matches the score-5 anchor: explicit validation steps, feedback loops, and defined stop conditions. | 5 / 5 |
Progressive Disclosure | The SKILL.md stays an overview with detail pushed to one-level-deep bundle files (references/phase-gate-contract.md, references/contract.yaml, references/evals.yaml — all verified present). Not score 5 because the References section is a terse bare list with minimal signaling of when each file should be read, and three referenced paths (../../references/stage-arc-boundary-contract.md, subagent-call-contract.md, deferred-context-index.md) point outside the skill bundle and could not be verified to resolve. | 4 / 5 |
Total | 17 / 20 Passed |