Content
75%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, well-structured body with concrete commands, explicit safety gates, and a genuinely used progressive-disclosure layer. The main gaps are the undefined `/cs-memory adopt` invocation, a dangling out-of-bundle DESIGN.md link, and the absence of a single ordered lifecycle walkthrough.
Suggestions
Show the `/cs-memory adopt` invocation concretely (arguments, what the backup looks like) since it is the only step that writes to CLAUDE.md and is currently referenced but never defined.
Fix or inline the `../../DESIGN.md` link — it leaves the skill bundle and does not resolve; consider pointing to `assets/memory_schema.json` directly for the field schema instead.
Add a short ordered walkthrough of one claim's lifecycle (captured by hook → staged at L1 → recurses across sessions → dry-run promote → adopt), so the operator sequence doesn't have to be reconstructed from the tier table and gates sections.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and earns its tokens — the tier table, the gates, and the hard rules carry information Claude cannot infer, with commands shown compactly under "Use it". It is not a 5 because the "The problem" narrative ("The fix is not more storage — it is a promotion ladder") and the four "Forcing questions" are rhetorical framing that could be trimmed without losing operational content. | 4 / 5 |
Actionability | Concrete, executable guidance dominates: `python3 scripts/memory_inspect.py --tier L1`, `--why "PR base branch is dev"`, `--contested`, the dry-run `python3 scripts/memory_promote.py`, and exact env-var toggles (`AGENT_MEMORY_SESSIONSTART=0` etc.). Not a 5 because the pivotal `/cs-memory adopt` command is referenced ("Only an explicit `/cs-memory adopt` touches a `CLAUDE.md`") but never shown or defined, leaving the final, most consequential step without an executable form. | 4 / 5 |
Workflow Clarity | The lifecycle is well-specified with strong safety checkpoints: staged promotions, dry-run that "writes nothing", `--contested` surfacing frozen claims, backup before adopt ("it backs both up first"), and fail-open hooks. Not a 5 because the operator-facing sequence (capture → inspect → resolve contested → promote → adopt) must be assembled from the tier table, gates, and "Use it" sections rather than being laid out as one ordered walkthrough with error-recovery branches. | 4 / 5 |
Progressive Disclosure | The "More in this skill" section clearly signals all three reference files (which exist in the bundle) with one-line descriptions at one level of depth, plus script entry points — good structure. Not a 5 because the rationale pointer `[../../DESIGN.md](../../DESIGN.md)` escapes the skill bundle two levels up and does not resolve, and `assets/memory_schema.json` is only reachable indirectly via that link and a validate script mention rather than being directly signposted. | 4 / 5 |
Total | 16 / 20 Passed |