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.
A highly actionable, well-sequenced workflow with concrete CLI commands, explicit state handling, and a genuine per-task validation loop. The main cost is repetition: the context/operationGuidance contract and pause conditions each appear two to three times, which inflates the token budget without adding new guidance.
Suggestions
State the context/operationGuidance contract once (in step 3) and collapse the four related Guardrails bullets (194-197) into a single cross-reference, cutting ~15 lines of duplication.
Trim the Guardrails list to only rules not already stated verbatim in the Steps and Pause-if sections (e.g., drop the duplicates of 185-190).
Move the three full output templates into a short reference section or keep only one canonical example, since the completed/paused templates are close structural variants of each other.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient, workflow-specific instruction with no basic-concept padding, but the context/operationGuidance contract is stated three times (lines 71-84, 93-94, and again as four guardrail bullets at 194-197), and the Guardrails section restates the step-level pause conditions almost verbatim. Anchor 3 ('could be tightened') fits; it does not reach anchor 4 because the redundancy is more than minor. | 3 / 5 |
Actionability | Every step carries copy-paste-ready commands with exact flags ('openspec status --change "<name>" --json', 'openspec instructions apply --change "<name>" --json'), concrete state handling (blocked/all_done), specific checkbox-edit mechanics ('- [ ]' → '- [x]' at the returned sourcePath and line), and filled-in output templates. Fully executable guidance matching anchor 5. | 5 / 5 |
Workflow Clarity | A clearly sequenced 7-step workflow with an explicit feedback loop (rerun apply instructions after each task and confirm progress changed), pre-edit drift check on the checkbox location, explicit blocked/all_done state handling, and enumerated pause conditions. Anchor 5's validation-with-error-recovery pattern is fully present. | 5 / 5 |
Progressive Disclosure | No bundle files exist and all content is inline in SKILL.md, but the ~200-line body is well-sectioned (Store selection, Project check, Steps, Outputs, Guardrails, Fluid Workflow) and everything present is runtime-needed operational guidance rather than reference material that clearly belongs in separate files. Good structure with minor organization gaps (e.g., the output templates could live in a reference file), matching anchor 4 rather than anchor 5's well-signaled external references. | 4 / 5 |
Total | 17 / 20 Passed |