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.
An exceptionally actionable, well-sequenced procedural skill with strong validation and safety guardrails around destructive spec operations. Its weaknesses are redundancy across steps and a monolithic single-file structure that inlines reference material better kept in separate bundle files.
Suggestions
Deduplicate repeated rules (empty Requirements section, retirement conditions, narrowed-selection handling) into one authoritative statement and reference it from the Guardrails checklist instead of restating steps verbatim.
Move the Delta Spec Format Reference, Main Spec Format Reference, and the six retirement conditions into a references/ file (e.g., references/format.md, references/retirement.md) and link them from SKILL.md to reduce context load for routine syncs.
Trim the store-selection preamble by moving the sticky-flag mechanics into the Guardrails section or a reference file, keeping only the store-detection instruction inline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with non-obvious operational knowledge (CLI flags, JSON fields, archive-matching semantics) and does not pad with concepts Claude already knows, but it repeats itself noticeably: "Never write an empty ## Requirements section" appears twice, retirement conditions are restated across steps 4c and 4d, and the Guardrails section restates earlier steps verbatim. It fits "mostly efficient but could be tightened" rather than the minor-trim anchor. | 3 / 5 |
Actionability | Fully executable throughout: exact commands ("openspec status --change \"<name>\" --json"), precise JSON field names ("artifactPaths.specs.existingOutputPaths", "planningHome.root"), concrete file paths, specific error-message prefixes ("Declared in", "Invalid store declaration in"), and a copy-paste output template. No pseudocode or vague direction. | 5 / 5 |
Workflow Clarity | Six clearly sequenced steps with explicit validation ("openspec validate --specs"), stop-before-write checkpoints on invalid JSON, project-root preflight checks, error-recovery branches per failure mode, and a Guardrails checklist — well above the destructive/batch-operation cap of 3 since validation and feedback loops are present. | 5 / 5 |
Progressive Disclosure | No bundle files exist, so navigation depth is not an issue, but the skill is a ~280-line monolith: reference-like material (the store-selection preamble, Delta/Main Spec Format References, and the six-condition retirement rules) is inlined rather than split into reference files. Structure exists via headers and bold sections, but organization into separate files would serve progressive disclosure better. | 3 / 5 |
Total | 16 / 20 Passed |