Content
65%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 highly actionable with executable commands and concrete responses, but it carries metaphorical padding, lacks an explicit verify-then-delete workflow for the destructive forget command, and inlines all detail rather than using reference files despite a sizable bundle.
Suggestions
Trim the philosophical framing (the opening quote, "raw data of the soul", the "two registers"/"filing cabinet" prose) to reduce token padding without losing the consult-vs-surface distinction.
Turn `forget` into an explicit validate-then-delete workflow: instruct running `forget --dry-run` first, reviewing the candidate list, then re-running without --dry-run, so workflow_clarity can exceed the destructive-operation cap.
Offload the Storage Architecture and Configuration detail into a reference file (e.g. references/ARCHITECTURE.md) and link to it one level deep, surfacing the bundled backend directories as navigable references.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient with concrete commands and JSON examples, but the metaphorical framing ("raw data of the soul", "gut feeling that interrupts you mid-thought", the "two registers" prose) adds padding Claude does not need. | 3 / 5 |
Actionability | Every command has copy-paste-ready invocations ("python3 scripts/inland_empire.py remember ..."), realistic options (--depth, --type, --before, --dry-run), a type-inference table, and full JSON response examples covering the common cases. | 5 / 5 |
Workflow Clarity | Each command is clearly explained and "When to Use" is present, but the destructive `forget` is not presented as a verify-then-delete workflow despite a --dry-run flag existing, so the rubric's destructive-operation cap holds at 3. | 3 / 5 |
Progressive Disclosure | Sections are well organized, but at ~288 lines everything (architecture diagram, config table, error handling) is inlined into SKILL.md with no one-level-deep reference files, and the bundled backend directories (mem0, memory_graph, memory_libsql) are not surfaced as navigable references. | 3 / 5 |
Total | 14 / 20 Passed |