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, well-structured body that sequences the tier/gate lifecycle with real safety checkpoints and concrete commands, held back from excellence only by references that point outside the shipped bundle. Conciseness is exemplary; actionability and workflow clarity are strong; progressive disclosure is undermined by dangling paths.
Suggestions
Ship the referenced files in the bundle: scripts/memory_inspect.py, scripts/memory_promote.py, and the rationale/schema content, so the copy-paste commands actually execute.
Move DESIGN.md inside the skill bundle (e.g., references/DESIGN.md) and link it as a relative path within the skill directory instead of ../../DESIGN.md, which escapes the skill root.
Show the adopt step concretely — what /cs:memory adopt runs, what backup it creates, and what the user reviews before confirming — to close the last workflow gap.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~90-line body is lean and every section carries distinct information: problem framing, a compact tier table, gate thresholds, runnable commands, hook config, and hard rules. It assumes Claude's competence (never explains what CLAUDE.md or hooks are) and includes no padding, matching the "every token earns its place" anchor. The 6-line "Forcing questions" section is the only arguably trimmable material, but it is brief and purposeful. | 5 / 5 |
Actionability | Guidance is concrete: exact commands with flags ("python3 scripts/memory_inspect.py --why \"PR base branch is dev\"", "--contested"), a dry-run promotion pass, named hooks, and per-hook env-var kill switches ("AGENT_MEMORY_SESSIONEND=0"). It falls short of fully executable because the referenced scripts (scripts/memory_inspect.py, scripts/memory_promote.py) and the /cs:memory adopt command are not present in the bundle, so the commands cannot actually run as shipped. | 4 / 5 |
Workflow Clarity | The promotion lifecycle is clearly sequenced by tier (L0→L1→L2→L3) with explicit gate criteria ("≥ 3 distinct sessions spanning ≥ 2 distinct calendar days"), refusal conditions, and safety checkpoints ("dry-run the promotion pass; writes nothing", adopt "backs both up first", hooks "fail open"). The gap keeping it below a 5 is that the operational loop is assembled implicitly across sections — the human's adopt step is referenced ("/cs:memory adopt") but never shown as a runnable step. | 4 / 5 |
Progressive Disclosure | Section structure is good and the single external pointer is clearly signaled with its contents named ("Rationale, open decisions, field schema: DESIGN.md"), keeping the body appropriately short. However, the bundle contains no scripts/, references/, or DESIGN.md — the referenced script paths and the ../../DESIGN.md pointer (which also escapes the skill directory) resolve to nothing, so navigation is broken as shipped rather than merely imperfectly organized. | 3 / 5 |
Total | 16 / 20 Passed |