Content
77%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This is a well-structured executor skill with strong actionability and clear workflow sequencing. The multi-mode persistence logic is handled cleanly, and the design document template provides concrete, copy-paste-ready structure. The main weaknesses are moderate verbosity (the full template inline inflates the token cost) and the inability to verify referenced shared files, though the references themselves are well-signaled.
Suggestions
Consider extracting the full design document template to a separate reference file (e.g., `design-template.md`) to reduce the inline token footprint and improve conciseness.
The persistence contract section repeats mode-specific logic that may already be covered in `sdd-phase-common.md` — consider consolidating to avoid duplication.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The skill is moderately efficient but includes some redundancy — the persistence contract section partially repeats what's in the shared phase-common reference, and the full design document template is quite lengthy. The orchestrator gate preamble and mode-specific branching add necessary but verbose conditional logic. Some tightening is possible (e.g., the template could be more compact). | 2 / 3 |
Actionability | The skill provides highly concrete, actionable guidance: exact file paths, a complete markdown template for the design document, specific artifact keys for persistence, a structured return envelope format, and clear rules. Each step tells the executor precisely what to do with specific outputs expected. | 3 / 3 |
Workflow Clarity | The 5-step workflow is clearly sequenced (Load Skills → Read Codebase → Write Design → Persist Artifact → Return Summary) with explicit mandatory gates ('This step is MANDATORY — do NOT skip it'). The mode-conditional branching (engram/openspec/hybrid/none) is clearly laid out with specific instructions per mode. The rules section adds validation constraints like the 800-word budget and blocking-question escalation. | 3 / 3 |
Progressive Disclosure | The skill references shared files (`skills/_shared/sdd-phase-common.md`, `skills/_shared/openspec-convention.md`, `openspec/config.yaml`) with clear section callouts (Section A, B, C, D), which is good one-level-deep referencing. However, no bundle files were provided to verify these references exist, and the inline design template is quite long — it could arguably be extracted to a separate reference file. The structure is reasonable but the main file carries a lot of inline content. | 2 / 3 |
Total | 10 / 12 Passed |