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.
An exemplary operational skill body: highly executable shell workflows with rigorous validation and feedback loops for a destructive filesystem operation, with only moderate verbosity from repeated safety mantras.
Suggestions
Consolidate the repeated mechanical-copy / empty-diff mantra into the single Mechanical Copy Contract section and reference it from the steps and Rules instead of restating it verbatim three to five times.
Tighten the Final-State Authority section by leading with the ranked source list and trimming the philosophical framing about stale claims.
Consider extracting the long bash move/recovery scripts into a referenced script file under scripts/ so the SKILL.md body stays a lean overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most content is concrete and operational, but the mechanical-copy / "NEVER Read→Write" / empty-diff-is-the-only-passing-evidence mantra is repeated verbatim across the Mechanical Copy Contract, each step, and the Rules section, and the Final-State Authority section is somewhat philosophical, so it could be tightened. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready bash with mktemp/EXIT traps, atomic compose-and-mv, git mv with guarded plain-mv fallback, destination-collision checks, and a native `sdd-archive-compose` invocation covering the main-spec-exists and main-spec-absent cases. | 5 / 5 |
Workflow Clarity | A clear six-step sequence (Load → Sync → Move → Verify → Persist → Return) with explicit MANDATORY `diff -r` validation checkpoints, blocked-failure feedback loops (nonzero exit → STOP, no manual retry), and a Step 4 checklist; the destructive-operation validation cap does not apply because validation is present and rigorous. | 5 / 5 |
Progressive Disclosure | Structure is good with well-signaled one-level-deep references to shared docs (sdd-phase-common.md Sections A–D, sdd-status-contract.md, openspec-convention.md); no bundle files exist locally and the operational content is appropriately inline, though nothing is split into this skill's own reference files. | 4 / 5 |
Total | 17 / 20 Passed |