Content
78%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 tight, executable body with unusually strong safety checkpoints (dry-run, staging, backup, quarantine) around its destructive adopt step. Its one structural weakness is navigation: all deep-dive material sits in unlinked bundle files, and the single outbound link escapes the skill directory.
Suggestions
Link the three references/ files from the sections they deepen — e.g., the gates section should point to references/promotion_gate_design.md and the hook paragraph to references/hook_capture_discipline.md — so the bundled detail is discoverable from SKILL.md.
Fix the final line's link: "../../DESIGN.md" points outside the skill bundle. Either move DESIGN.md into references/ and link it there, or replace it with links to the existing references/ files and assets/memory_schema.json ("field schema" currently points at DESIGN.md while the actual schema sits unlinked in assets/).
Add a short end-to-end sequence (capture → inspect → dry-run promote → review staged → adopt) so the reader does not have to assemble the pipeline from four separate sections.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with material Claude does not already know — the tier/injection table, numeric gates ("≥ 3 distinct sessions spanning ≥ 2 distinct calendar days"), env-var toggles — and explains no basic concepts. Two sections could be trimmed: the "The problem" narrative and especially "Forcing questions", which addresses the human reader rather than instructing the agent. That is more than zero padding but well short of the anchor-3 "some unnecessary explanation" level. | 4 / 5 |
Actionability | Every user-facing command is copy-paste ready with real flags: "python3 scripts/memory_inspect.py --tier L1", "--why \"PR base branch is dev\"", "--contested", and "python3 scripts/memory_promote.py" marked "dry-run the promotion pass; writes nothing"; both referenced scripts exist in the bundle. Hook control via AGENT_MEMORY_SESSIONSTART=0-style env vars and the explicit "/cs:memory adopt" command cover the remaining cases. | 5 / 5 |
Workflow Clarity | The L0→L3 ladder is a clear sequence with explicit gates, and validation checkpoints are present for the destructive step (dry-run promotion, staged proposals in ".memory/staged/", "it backs both up first" on adopt, quarantine of redacted claims, freeze on contradiction) — so the destructive-operation cap does not apply. What keeps it below 5 is that no single end-to-end walkthrough (capture → inspect → promote → adopt) ties the sections together, and hook setup is only described, not shown. | 4 / 5 |
Progressive Disclosure | The body itself is well-sectioned and appropriately lean, but the reference layer is disconnected: the only detail link, "Rationale, open decisions, field schema: ../../DESIGN.md", points outside the bundle rather than to a bundled file, while the three files in references/ (hook_capture_discipline.md, promotion_gate_design.md, tiered_memory_canon.md) and assets/memory_schema.json are never referenced from SKILL.md at all. Deep material exists but is undiscoverable from the entry point — structure is present, signaling is not. | 3 / 5 |
Total | 16 / 20 Passed |