Content
96%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.
An exemplary operational skill body: lean, fully executable, with a well-sequenced diagnostic workflow that includes error-recovery branches and safety guardrails. The only weakness is progressive disclosure — the referenced REFERENCE.md and sibling skills are not present in the skill's bundle, leaving the deferred navigation unverified.
Suggestions
Ship REFERENCE.md with the skill bundle (or state where the generated file is produced) so the 'full recognized-variable list' reference resolves at use time.
Verify the 'See also' targets (agentmemory-rest-api, agentmemory-architecture) exist as installed skills, or drop the cross-references.
Consider moving the auth secret fallback chain and HTTPS/loopback rules (step 4) into the agentmemory-rest-api skill, keeping only a pointer here.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with unique operational facts (file paths, env vars, commands, port arithmetic) and contains no padding or explanation of concepts Claude already knows. It assumes competence throughout, matching the lean anchor 5 rather than the 'could be trimmed' anchor 4. | 5 / 5 |
Actionability | Fully executable guidance: concrete commands ('agentmemory status', 'node dist/cli.mjs status', 'agentmemory connect codex --with-hooks', '--port <N>'), exact file paths ('~/.agentmemory/.env', '~/.agentmemory/secret'), and a copy-paste env block covering the common configuration cases. Matches anchor 5. | 5 / 5 |
Workflow Clarity | 'Check the connection first' is a clearly sequenced 6-step diagnostic workflow with an explicit error-recovery branch (step 2: 'If it fails, check Node.js 20+ and agentmemory status' then restart) and validation checkpoints ('A successful handshake alone does not prove the daemon is up', inspect hooks 'before promising automatic capture'). The workflow is diagnostic rather than destructive/batch, so the validation cap does not apply; it matches anchor 5's sequence-with-feedback-loop pattern. | 5 / 5 |
Progressive Disclosure | Sections are well organized and bulk detail is correctly deferred to one-level-deep, clearly signaled references (full variable list to REFERENCE.md, port rationale to agentmemory-architecture, secret usage to agentmemory-rest-api). However, the body exceeds 50 lines and none of the referenced files (REFERENCE.md, the 'See also' skills) exist in this skill's bundle, so navigation cannot be fully verified — anchor 4 rather than 5. | 4 / 5 |
Total | 19 / 20 Passed |