Content
81%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 CLI examples, a clearly sequenced workflow, and genuine validation feedback loops. Its main weakness is conciseness — the evidence-handling policy prose and domain background pad the file beyond what operating the vault requires.
Suggestions
Trim or move the "Recall is evidence, not certainty" policy prose and the "portable part of Desk-style memory" paragraph into a reference file; keep only the operational rules (bind lookup to workspace, handle ECC_MEMORY_INCOMPLETE, check source freshness) inline.
Split MCP setup (server command, environment variables, exposed tool list) into a references/mcp-setup.md file and keep a one-line pointer in the main body.
Clarify the `mcp-configs/mcp-servers.json` reference — it points outside the skill bundle, so state where that file lives or inline the minimal server entry.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly operational, but sections like "This is the portable part of Desk-style memory: scoped evidence, current-state checks and explicit uncertainty ... Supplier relationship graphs remain an optional domain-specific adapter" and the long "Recall is evidence, not certainty" policy prose explain domain background Claude does not need and could be tightened. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready commands throughout: `ecc memory search "authentication migration" --target-harness codex`, the complete `printf ... | ecc memory save --title ... --stdin` pipeline, `ecc memory handoff --from codex ... --body-file handoff.md`, `ecc memory doctor`, plus named MCP tools and setup commands covering the common cases. | 5 / 5 |
Workflow Clarity | Clear numbered sequence (recall before writing → save → hand off → validate) with explicit validation (`ecc memory doctor`, prerequisite `command -v ecc-memory-mcp`) and a real feedback loop: `ECC_MEMORY_INCOMPLETE` → "Repair the reported vault problem; do not tell the caller the memory does not exist." | 5 / 5 |
Progressive Disclosure | Well-organized sections with clear headers, but the ~190-line body inlines the MCP setup details and a long policy section that could live in reference files; the pointer to `mcp-configs/mcp-servers.json` references a file outside the skill bundle. | 4 / 5 |
Total | 17 / 20 Passed |