Content
70%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 well-structured routing procedure with strong workflow sequencing, explicit validation gates, and a concrete output template. Its weaknesses are repeated restatement of the same safety boundary across many sections and a reference structure where half the cited paths do not resolve and one bundle file is orphaned.
Suggestions
Consolidate the overlapping safety restatements in Philosophy, Execution Boundaries, Failure Mode, Gotchas, and Validation Gates into a single authoritative 'stop boundaries' section to reduce token cost without losing coverage.
Fix the reference structure: either ensure ../../references/stage-arc-boundary-contract.md, ../../references/subagent-call-contract.md, and ../../references/deferred-context-index.md exist at the cited paths, or inline the essential rules they carry; also reference or remove the orphaned references/task-profile.json.
Add one line of inline substance for each named contract invoked by the procedure (phase gate contract, context-disposition policy, stage arc boundary) so the skill remains actionable when the referenced files are unavailable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient and free of padding, but the stop-before-commit/authority rule is restated across Philosophy ("does not authorize implementation, staging, commits, Linear updates, or closure"), Procedure step 6, Execution Boundaries, Failure Mode, Gotchas, and Validation Gates. Consolidating these overlapping safety restatements would tighten the document noticeably. | 3 / 5 |
Actionability | The numbered procedure, exact output field names (slack_policy, stop_rule_status, next_safe_action), and a fully filled Output Template with an exact command ("bash scripts/verify-work.sh --fast") give mostly executable guidance. Minor gaps remain where steps defer to named contracts ("run the phase gate contract", "apply the context-disposition policy") without any inline summary of what those entail. | 4 / 5 |
Workflow Clarity | The 6-step procedure is clearly sequenced, the fail-fast rule ("stop at the first failed gate and do not proceed until the blocker is fixed") provides an explicit feedback loop, and Validation Gates plus the Failure Mode recovery step form a checklist with error recovery. Validation for the risky operations is explicitly present, so the destructive-operation cap does not apply. | 5 / 5 |
Progressive Disclosure | The References section is clearly signaled and one level deep, and references/phase-gate-contract.md, references/contract.yaml, and references/evals.yaml exist in the bundle. However, three referenced paths (../../references/stage-arc-boundary-contract.md, ../../references/subagent-call-contract.md, ../../references/deferred-context-index.md) do not resolve in the bundle, and references/task-profile.json is present but never referenced, leaving organization gaps beyond the minor ones allowed at level 4. | 3 / 5 |
Total | 15 / 20 Passed |