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, executable quick-start with a genuinely diagnostic verification step and valuable platform notes, wasting no tokens. The main defects are that the adapter table and troubleshooting guide it leans on are not present in the bundle, and the verification procedure leaves 'server running' unchecked.
Suggestions
Ship REFERENCE.md in the skill bundle (e.g., under references/) so the adapter table the workflow depends on actually resolves.
Include the TROUBLESHOOTING.md content path or a brief inline fallback for the 'only 7 tools' failure instead of relying on a ../_shared file outside the skill directory.
Make the verification step concrete: name the command or UI where the tool list and server status should be checked (e.g., the host's MCP panel or a specific agentmemory status command).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence: no explanations of what MCP or REST are, no filler, every line carries operational information ('Windows: use WSL2. Native Windows runs the server but `connect` is not supported there.'). This matches the anchor-5 'every token earns its place' example; there is nothing to trim without losing information. | 5 / 5 |
Actionability | Concrete, copy-paste-ready command ('agentmemory connect claude-code # or cursor, codex, gemini-cli, ...'), a concrete reload example ('/mcp' in Claude Code), and a numeric failure signature ('Only 7 tools means the MCP shim could not reach a server'). It stops short of 5 because the verification step ('show the full tool set with a server running') doesn't specify how to check the server is running, and the adapter-name table the workflow depends on is delegated to REFERENCE.md. | 4 / 5 |
Workflow Clarity | The workflow has a clear detect -> connect -> verify sequence with an explicit validation checkpoint (step 3) and an error-recovery pointer for a named failure mode. It sits at 4 rather than 5 because the validate -> fix -> retry loop is only pointed at ('see ../_shared/TROUBLESHOOTING.md') rather than spelled out, and the check in the verification step is partially implicit; it is above 3 because validation is explicit and tied to a concrete failure signature. | 4 / 5 |
Progressive Disclosure | The body itself is well-sectioned and its references are clearly signaled and one level deep ('The full adapter list ... lives in REFERENCE.md'). However, the bundle contains no references/ directory or REFERENCE.md, and the referenced ../_shared/TROUBLESHOOTING.md is also absent, so the primary references the workflow depends on are dangling relative to the actual bundle structure. Not 4 because a navigation path the reader cannot complete is more than a minor organization gap; not 2 because the in-body structure is good and references are prominent rather than buried. | 3 / 5 |
Total | 16 / 20 Passed |