Content
77%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.
The body is well-organized, dense, and unusually strong on workflow clarity with thorough validation for a destructive-adjacent operation. Its weakness is bundle completeness: every external reference (two scripts and DESIGN.md) points to files that are not shipped, which both breaks progressive-disclosure navigation and leaves the otherwise-concrete commands non-executable.
Suggestions
Bundle the invoked scripts: create scripts/memory_inspect.py and scripts/memory_promote.py (or add a note that they are generated at install time) so the copy-paste commands in 'Use it' actually run.
Ship the referenced ../../DESIGN.md (rationale, open decisions, field schema) or relocate that detail into a bundled reference file so the one-level-deep link resolves instead of dangling.
After restoring the files, re-verify each code block and the DESIGN.md link resolves from the skill directory, which would lift both actionability and progressive_disclosure.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient — a compact tier table, numeric gate thresholds ("≥ 3 distinct sessions spanning ≥ 2 distinct calendar days"), and lean code blocks assume Claude's competence — with only minor discursive framing in "The problem" and "Forcing questions" that could be trimmed, fitting anchor 4 rather than 'every token earns its place' at 5. | 4 / 5 |
Actionability | Concrete, parameterized commands with example arguments are given ("python3 scripts/memory_inspect.py --why \"PR base branch is dev\"", "--contested", "python3 scripts/memory_promote.py"), but they invoke scripts that are not present in the bundle, so they are not fully runnable as-shipped — a minor executability gap that caps at anchor 4 rather than 'fully executable' at 5. | 4 / 5 |
Workflow Clarity | The L0→L1→L2→L3 promotion sequence is clearly gated, with explicit validation ("Redact before writing", staged-only proposals, adopt-with-backup, dry-run "writes nothing", freeze-on-open-contradiction, fail-open hooks) and feedback loops ("--contested" → human resolves → promote); the five "Hard rules" act as a checklist, and because destructive-op validation is present the batch/destructive cap at 3 does not apply — anchor 5. | 5 / 5 |
Progressive Disclosure | In-body structure is good (clear sections) and the delegation to [../../DESIGN.md](../../DESIGN.md) is cleanly signaled one level deep, but the actual bundle is empty — none of scripts/memory_inspect.py, scripts/memory_promote.py, or ../../DESIGN.md exist — so the progressive-disclosure navigation is broken and a reader cannot reach the delegated detail, pulling it below anchor 4. | 3 / 5 |
Total | 16 / 20 Passed |