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 well-engineered workflow body: fully executable commands, explicit JSON contract details, validation checkpoints and confirmation gates at every risky boundary, and clear guidance-only next-step routing. Its one weakness is token efficiency — the glob-artifact rules are restated across four sections and step 4's nested subprocedure is dense enough to tighten or split out.
Suggestions
State the glob-artifact rules once (e.g., as a dedicated subsection or a short reference file) and have step 2, step 4, Guardrails, and Output reference that single statement instead of each restating them — this would remove the largest source of repeated tokens.
Tighten step 4's five-item subprocedure: merge items 4 and 5 (both re-verify immediately before creation) into a single re-check list, and compress item 3 ("include in step 5's revisions, create after confirm") since step 5 already mandates confirmation for every write.
Trim overlapping restatements of the ready/blocked-vs-skipped routing: it appears in step 4, step 6, Output, and Guardrails; one canonical statement plus a pointer would preserve clarity at lower token cost.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body assumes Claude's competence — no concept explanations, just procedure — but it is noticeably repetitive: the glob-artifact rules appear in step 2, again as step 4's dense five-item subprocedure, again in Guardrails ("never write to a glob `resolvedOutputPath`… Do not advance the build frontier"), and again in Output. This fits anchor 3 ("mostly efficient but… could be tightened") better than anchor 4: the repetition goes beyond minor over-explanation. | 3 / 5 |
Actionability | Fully executable guidance: concrete commands (`openspec status --change "<name>" --json`, `openspec instructions "<artifact-id>" --change "<name>" --json`, `openspec list --json`, `openspec store list --json`), the exact JSON fields to read (`root`, `existingOutputPaths`, `lastModified`, `isPlanningComplete`), copy-ready announce strings ("Using change: <name>"), and fail-if-exists create semantics. Not below 5: for an instruction-only skill this is copy-paste-ready procedural guidance covering the common cases. | 5 / 5 |
Workflow Clarity | Six clearly sequenced steps with explicit validation checkpoints and feedback loops: a root check before any write, non-zero-exit-as-data guidance ("read the JSON instead of retrying"), user confirmation before every write, refresh-and-recheck immediately before creating a glob file, and stop-and-reconcile behavior when a check fails. This matches the anchor-5 example of a sequence with explicit validation and error-recovery loops. | 5 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/), and the body needs none — it is organized into well-labeled sections (Store selection, Project check, Input, Steps, Output, Guardrails) with cross-skill pointers rather than nested file references. Not 5: the five-item glob-artifact subprocedure inside step 4 inlines dense conditional detail that could be lifted into its own subsection or a reference file, which is a minor organization gap matching anchor 4. | 4 / 5 |
Total | 17 / 20 Passed |