Content
81%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 highly actionable and workflow-safe: exact commands, named JSON fields, validation after every write, and careful handling of uninitialized-project and store edge cases. The main cost is token efficiency — guidance is repeated across Step 4, the Artifact Creation Guidelines, and Guardrails sections, which could be consolidated.
Suggestions
Consolidate the duplicated guidance: 're-read dependencies from disk', the context/rules exclusion, and dependency-reading appear in Step 4a, Artifact Creation Guidelines, and Guardrails — state each once.
Break up or externalize the two dense opening prose blocks (Store selection, Project check) into a short summary in SKILL.md with detail in a reference file.
Merge the adjacent bullets in step 4b that both cover skipped/skip_specs artifacts into a single rule to reduce repeated conditions.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most content earns its place (CLI semantics like status being file-existence-only are non-obvious), but there is real duplication: 're-read dependencies from disk' appears in step 4a, the Guardrails, and the Artifact Creation Guidelines; the context/rules exclusion rule is stated twice; and the skipped-artifact handling is covered in adjacent bullets. This places it at 'mostly efficient but could be tightened' rather than the 4 anchor. | 3 / 5 |
Actionability | Fully executable throughout: copy-paste commands (`openspec new change "<name>"`, `openspec status --change "<name>" --json`, `openspec instructions <artifact-id> --change "<name>" --json`) with the exact JSON fields to parse (`applyRequires`, `requires`, `resolvedOutputPath`, `planningHome`) and explicit branch handling for each outcome. | 5 / 5 |
Workflow Clarity | Clear numbered sequence with explicit validation checkpoints and feedback loops: re-run `status --json` after each artifact, verify the artifact exists at `resolvedOutputPath`, explicit stop conditions, and error-path handling (`"root": null`, non-zero exits, store-declaration errors). The batch-operation cap does not apply since verification steps are present. | 5 / 5 |
Progressive Disclosure | A self-contained single file (no bundle files exist to reference) with clear sections and no nested references. Good structure, but the dense opening prose blocks (store selection, project check) run long inline and would benefit from being split or pulled into a reference, so it does not fully meet the 5 anchor. | 4 / 5 |
Total | 17 / 20 Passed |