Content
73%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 delivers a well-sequenced, thoroughly validated archive workflow with concrete commands, explicit user-prompt routing, and strong safeguards around the destructive move operation. Its main costs are repetition in the instructions/guard-rules sections and prose-only handling of the spec-sync step, plus dense inline contract text that could live in a reference file.
Suggestions
Deduplicate the instructions-contract rules: state them once in step 1 (or a short 'Runtime inputs' section) and reduce the Guardrails list to items not already covered, which would tighten conciseness without losing behavior.
Move the ~26-line instructions-archive handling contract (context vs operationGuidance semantics) into a one-level-deep reference file and keep a 3-4 line summary inline, improving progressive disclosure and token efficiency.
Make the step 4 sync invocation more concrete: show the exact command or delegation wording used to run the 'openspec-sync-specs' workflow inline, and specify the comparison output format for the combined summary shown before prompting.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense operational guidance with no explanation of concepts Claude already knows, but the instructions-archive contract (lines 35-60) is repetitive — the 'never copy verbatim' / 'prompt-level contracts, not enforceable checks' rules are restated in the section, again mid-step, and again in Guardrails — and the store-selection paragraph explains the same flag rule twice. This fits anchor 3, 'mostly efficient but includes some unnecessary explanation or could be tightened'; it is not anchor 4 because the repetition is more than minor. | 3 / 5 |
Actionability | Concrete, executable commands appear throughout ('openspec status --change "<name>" --json', 'openspec list --json', 'mkdir -p "<planningHome.changesDir>/archive"', 'mv "<changeRoot>" ...') alongside explicit prompt options and answer routing. It stops short of anchor 5 because the delta-spec comparison and the inline 'openspec-sync-specs' workflow invocation are described in prose rather than as runnable commands or a referenced, complete procedure. | 4 / 5 |
Workflow Clarity | The six steps are clearly sequenced with explicit validation checkpoints (artifact status check, task-count check, mandatory post-sync re-verification of every capability against the delta, 'report what differs and stop — do not archive') and error-recovery loops for the destructive mv of changeRoot. The destructive-operation validation cap is satisfied, and this matches anchor 5's 'explicit validation steps; feedback loops for error recovery'. | 5 / 5 |
Progressive Disclosure | No bundle files exist and none are strictly needed for this workflow; the body is well-sectioned (Store selection, Input, numbered Steps, Output template, Guardrails) and easy to navigate. It is anchor 4 rather than 5 because the ~26-line instructions-archive behavioral contract and the dense store-selection paragraph are inline material that could be split into a one-level-deep reference file to slim the always-loaded overview. | 4 / 5 |
Total | 16 / 20 Passed |