Content
75%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 content is a solid, executable CLI workflow with a well-sequenced process and concrete commands throughout. It falls just short of top marks because of light redundancy, template-based (non-code) artifact creation, and weak error-recovery feedback loops.
Suggestions
Add an explicit error-recovery loop after the 'verify the artifact file exists' step — e.g., if the file is missing or instructions JSON parse fails, re-run the command or surface the error before proceeding.
De-duplicate the IMPORTANT note about context/rules being constraints (it appears in both the step body and Guardrails); state it once in Guardrails and reference it.
Consider moving the spec-driven artifact-pattern reference (proposal/specs/design/tasks) into a separate reference file and signaling it from SKILL.md to improve progressive disclosure and trim the main body.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient and assumes Claude's competence (no generic-concept padding), but carries minor redundancy — the IMPORTANT context/rules note appears twice and the artifact-patterns section is long — so it is not fully lean at a 5. | 4 / 5 |
Actionability | Concrete, executable CLI commands (`openspec list/status/instructions`) with documented JSON fields provide mostly copy-paste-ready guidance, but the core artifact-creation step is template-filling guidance rather than executable code, leaving a minor gap. | 4 / 5 |
Workflow Clarity | A clear 4-step branched sequence with STOP checkpoints, a Guardrails checklist, and a 'verify the artifact file exists' validation step is present, but explicit error-recovery (fix→retry) loops are thin, stopping short of a 5. | 4 / 5 |
Progressive Disclosure | No bundle files exist; the skill is a single well-sectioned file with clear headers and no nested references, which is appropriate for a CLI workflow, though the spec-driven artifact-patterns section could conceivably live in a reference file. | 4 / 5 |
Total | 16 / 20 Passed |