Content
73%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, actionable overview with a clear validated workflow and a properly bundled one-level reference. Its main weakness is token efficiency: the /create-spec issue-mode requirements and coverage lists are duplicated between Constraints, the Workflow, and the reference file.
Suggestions
State the /create-spec issue-mode snapshot rules once (in the reference file) and have both the Constraints bullet and Workflow step 1 cross-reference it, instead of repeating nearly identical multi-clause text in both places.
Trim or merge the 'What is covered in this Skill?' bullet list, which largely restates the Constraints and 'When to use this skill' sections.
Move detailed pagination/count-reconciliation mechanics (e.g. 'cross-check the retrieved comment count when the provider exposes a total') out of SKILL.md into references/042-planning-openspec.md, keeping only the stop-condition principle inline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The /create-spec issue-mode rules are stated nearly verbatim in both the Constraints bullets ('exhaust every accessible comment page... stop before scope assessment or OpenSpec authoring') and Workflow step 1, and the 'What is covered in this Skill?' list restates items already covered by Constraints and 'When to use this skill'. Mostly efficient, but noticeably could be tightened through deduplication. | 3 / 5 |
Actionability | Concrete commands are provided ('openspec --version', 'openspec new change <change-id>', 'openspec init', 'openspec validate --all') with clear conditions for each, and scaffold/placeholder-removal details are explicit. Minor gaps remain in artifact-authoring specifics, which are deferred to the reference. | 4 / 5 |
Workflow Clarity | Six clearly sequenced steps with explicit validation checkpoints (step 5 'Validate authority and alignment', step 6 'Run openspec validate --all'), a stop-before-authoring condition in step 1, and user-approval gates for change maps and archiving, providing feedback loops for error recovery. | 5 / 5 |
Progressive Disclosure | A single, real, well-signaled one-level-deep reference (references/042-planning-openspec.md, linked in step 1 and in a Reference section) keeps the overview navigable. However, the long inline Constraints section duplicates material that also lives in the reference, so some content that could be separate remains inline. | 4 / 5 |
Total | 16 / 20 Passed |