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.
A well-structured, actionable multi-step workflow with explicit validation checkpoints and clear guardrails. It is concise and executable, with only minor gaps in post-archive verification and in the concreteness of the delegated sync reference.
Suggestions
Add a post-move verification step (e.g., confirm `openspec/changes/archive/YYYY-MM-DD-<name>/` exists and `openspec/changes/<name>` no longer exists) to close the feedback loop on the destructive `mv` and lift workflow clarity.
Replace the abstract 'execute /opsx:sync logic (use the openspec-sync-specs skill)' with a concrete pointer or invocation so the delegation is unambiguous.
Tighten the repeated 'Use the AskUserQuestion tool to confirm user wants to proceed / Proceed if user confirms' phrasing in steps 2 and 3 to a single stated convention.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly lean with concrete commands and no padding about what openspec or archives are; a few phrases like 'Use the AskUserQuestion tool' and repeated confirmation sub-steps across steps 2-4 could be tightened slightly. | 4 / 5 |
Actionability | Concrete, runnable commands are given (`openspec list --json`, `openspec status --change`, `mkdir -p`, `mv ...`) with parsed JSON fields, though the archive `mv` and date formatting are pseudocode-ish templates the agent must instantiate rather than fully copy-paste-ready. | 4 / 5 |
Workflow Clarity | A clear six-step sequence with explicit validation checkpoints (artifact status, task completion, delta-sync assessment) and confirm-before-proceed prompts; it is not a 5 because the final destructive move lacks an explicit post-move verification that the archived directory exists and the source is gone. | 4 / 5 |
Progressive Disclosure | Good sectioned structure (Input, Steps, Output On Success, Guardrails) with one-level-deep delegation ('use the openspec-sync-specs skill') rather than nested references; minor gap is that the delegated sync logic is referenced but not linked to a concrete file path. | 4 / 5 |
Total | 16 / 20 Passed |