Content
77%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 an exceptionally actionable and well-sequenced process document with real validation gates, feedback loops, and failure-mode-driven guidance. Its two weaknesses are the ~75 lines of DOT graphs duplicating the prose workflow, and three referenced prompt-template files missing from the bundle, which breaks progressive disclosure at its most important delegation points.
Suggestions
Cut or drastically compress the two DOT digraphs (the 'when to use' decision fits in three lines of prose; the process graph restates the Setup/Task Loop/Final Review sections verbatim) — this alone removes ~75 lines of duplicated workflow.
Ship the three referenced prompt templates (implementer-prompt.md, task-reviewer-prompt.md, re-review-prompt.md) in the bundle, or inline their key contracts, so the dispatch/review steps are not blocked on missing files.
Tighten the recurring restatements of the spec-as-authority ruling rule (it appears in Rulings, Setup, the fix loop, and the breaker) into one canonical statement plus back-references.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most of the body is dense, non-obvious operational guidance that earns its tokens (model-selection heuristics, ledger recovery, fix-loop policy, the rationalizations table), and it assumes competence rather than explaining basics. But the two DOT digraphs (~75 lines) restate what the following prose sections (Setup, The Task Loop, Final Review, Finish) already specify — the process graph duplicates the per-task loop nearly statement-for-statement — which is a padded section that could be cut entirely. Fits anchor 3 ('mostly efficient but could be tightened') more than 4, which would require only 'minor instances' of trimmable material. | 3 / 5 |
Actionability | Guidance is fully executable: exact script invocations ('bash scripts/sdd-workspace PLAN_FILE', 'bash scripts/review-package PLAN_FILE BASE HEAD'), a concrete BASE-recording rule with the HEAD~1 failure mode named, exact ledger line formats ('Task <N>: fix round <R>/5 (...)'), a four-value implementer status taxonomy with a handling rule per value, an enumerated dispatch contract (1)-(5), and a worked example workflow covering the common path including a fix round. Fits anchor 5 ('copy-paste ready commands; specific examples cover the common cases'). | 5 / 5 |
Workflow Clarity | The multi-step process is explicitly sequenced (Setup → task loop steps 1-5 → final review → finish) with validation checkpoints throughout: every task gates on a two-verdict review, fixes get scoped re-reviews with per-finding ADDRESSED/NOT ADDRESSED verdicts, a 5-round breaker with adjudication rules, and destructive operations (rm -rf workspace, git clean -fdx) are explicitly fenced behind a clean final review. Feedback loops, error recovery routes, and a rationalizations checklist are all present — anchor 5 exactly. | 5 / 5 |
Progressive Disclosure | The structure is conceptually right: SKILL.md is the overview, per-role prompt templates are offloaded to separate files, and the three referenced scripts (scripts/sdd-workspace, scripts/task-brief, scripts/review-package) exist in the bundle. However, three of the most load-bearing referenced files — implementer-prompt.md, task-reviewer-prompt.md, and re-review-prompt.md — are absent from the bundle, so the disclosure chain is broken exactly where the skill delegates its detailed contracts. That drops it below anchor 4's 'minor organization gaps' to anchor 3's 'could be better organized' territory, though the references that do exist are clearly signaled and one level deep. | 3 / 5 |
Total | 16 / 20 Passed |