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 body is an efficient, actionable playbook: concrete CLI commands, explicit confirmation checkpoints before the destructive move, and a clean summary template. Remaining gaps are duplicated guardrail text, undefined delta-spec comparison mechanics, and unhandled edge-failure paths.
Suggestions
Trim Guardrails entries that restate steps 1 and 4, keeping only the new facts (e.g., '.openspec.yaml' preservation and using artifact-graph status).
Make step 4 concrete: specify how to diff delta specs against openspec/specs/<capability>/spec.md (e.g., what to grep or compare) or point to the exact section/command in openspec-sync-specs that performs it.
Add edge-case handling: what to do when 'openspec list --json' shows no active changes, when the change directory doesn't exist before 'mv', and how to verify the archive succeeded after the move.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is command-first and free of concepts Claude already knows, with every step carrying operational detail. The Guardrails section partially reiterates steps 1 and 4 ('Always prompt for change selection if not provided', 'If delta specs exist, always run the sync assessment...'), which could be trimmed. | 4 / 5 |
Actionability | Most steps give copy-ready commands ('openspec list --json', 'openspec status --change "<name>" --json', 'mkdir -p openspec/changes/archive', the dated 'mv'), plus a concrete output template. Step 4's delta-spec comparison and the sync handoff ('use the openspec-sync-specs skill') lack executable mechanics, keeping it below anchor 5. | 4 / 5 |
Workflow Clarity | A clearly sequenced 6-step workflow with real validation checkpoints: artifact status, task counts, sync assessment, a target-exists check before the destructive 'mv', and AskUserQuestion confirmations, so the destructive-operation cap at 3 does not apply. It misses anchor 5 because edge-failure paths (no active changes, change directory missing, failed move) and a fix-and-retry loop are not addressed. | 4 / 5 |
Progressive Disclosure | A single well-organized file with clear section headers and a one-level, clearly-named reference to the openspec-sync-specs skill; no bundle files exist to split further. The delegated sync skill is referenced by name only without signaling what it covers, a minor organization gap versus anchor 5. | 4 / 5 |
Total | 16 / 20 Passed |