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 exemplary procedural workflow skill: fully executable commands and templates, rigorously sequenced phases with validation and error-recovery at every decision point, and sound delegation of policy to external docs. The only real cost is length — several justificatory asides push token cost above what the instructions alone require.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The bulk is lean, procedural instruction with exact commands and templates; however, several rationale passages could be trimmed — "The previous form of this step read the current GDD in full and git show-ed the full committed version..." and "an unbounded Read on a large ADR hits the 25k-token cap and, unrecovered, the only path forward is paging through the entire remainder" explain failure modes at length. Not 3: over-explanation is minor relative to the mostly efficient instruction; not 5: the meta-commentary about a "previous form of this step" and repeated why-not explanations are trimmable. | 4 / 5 |
Actionability | Fully executable throughout: exact commands ("git diff HEAD -- design/gdd/[filename].md"), complete Grep invocations with pattern, glob, and output_mode, copy-ready report and dialog templates, and concrete thresholds (~50KB, 25k-token cap). Specific examples cover the common cases, including the ambiguity-resolution table for zero-match scans. | 5 / 5 |
Workflow Clarity | Ten clearly numbered phases with explicit validation checkpoints at every branch: argument/file validation in step 1, empty-diff disambiguation in step 2 ("An empty diff is not 'no impact'"), the N/M denominator logic and zero-match interpretation table in step 4, and the TD-CHANGE-IMPACT gate with APPROVE/CONCERNS/REJECT/NOT ASSESSED verdicts and recovery loops ("re-analyze the impact before continuing", "supply it and re-run the gate"). | 5 / 5 |
Progressive Disclosure | Policy detail (automation modes, workflow tiers, director gates) is correctly delegated to clearly signaled one-level-deep project docs (".claude/docs/automation-modes.md", ".claude/docs/workflow-modes.md", ".claude/docs/director-gates/td-change-impact.md") rather than inlined. Not 5: the ~330-line body carries substantial inline material (report templates, dialog scripts, interpretation tables) that a bundle reference file could offload; not 3 since the inline/external split is mostly well-judged and navigation is easy. | 4 / 5 |
Total | 18 / 20 Passed |