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 lean, well-structured, mostly executable skill body with genuine verification checkpoints for two hosts. Its weaknesses are unresolved pointers: TROUBLESHOOTING.md and REFERENCE.md are referenced but missing from the bundle, and the Hermes/OpenClaw setup path defers to unspecified 'matching integration setup instructions'.
Suggestions
Ship TROUBLESHOOTING.md and REFERENCE.md in the bundle (e.g., under references/) or remove/inline their content — currently both links dead-end and the exact registered hook events are documented nowhere accessible.
Replace "Follow the matching integration setup instructions" for Hermes and OpenClaw with a concrete pointer (a file path or the actual verification commands) so that path is actionable.
In 'What the hooks do', name the concrete hook events (or a one-line example observation) so the skill body stands alone if REFERENCE.md stays external.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~47-line body is lean and entirely plugin-specific: install commands, verification steps, a hook-event summary, environment flags, and gotchas such as "Installing shared skills or connecting MCP alone does not enable automatic capture". Nothing explains concepts Claude already knows and every section earns its place, matching the top anchor. It stays at 5 rather than dropping to 4 because there is no padding to trim. | 5 / 5 |
Actionability | Concrete, copy-paste guidance dominates: `/plugin marketplace add rohitg00/agentmemory`, `devin plugins install rohitg00/agentmemory#plugin`, "Run `/hooks` inside Devin to confirm the agentmemory hooks are loaded", named flags (`AGENTMEMORY_AUTO_COMPRESS`, `AGENTMEMORY_INJECT_CONTEXT`), and the viewer URL. It falls short of 5 because "Follow the matching integration setup instructions" for Hermes/OpenClaw points nowhere concrete, and the TROUBLESHOOTING.md/REFERENCE.md pointers resolve to files not present in the bundle. | 4 / 5 |
Workflow Clarity | The setup sequence is clear with real validation checkpoints: install → "Run `/hooks` ... to confirm the agentmemory hooks are loaded" → "verify a real turn creates an observation", plus a missing-observation diagnostic path ("confirm the plugin is enabled and the server is running"). It is not 5 because there is no recovery guidance beyond that one-line check (the deep troubleshooting is deferred to a file that is not in the bundle), and the Hermes/OpenClaw path has no concrete steps. | 4 / 5 |
Progressive Disclosure | Section structure is good (Quick start / What the hooks do / Important / See also / Reference) and references are clearly signaled, but the two referenced files — [TROUBLESHOOTING.md](TROUBLESHOOTING.md) and REFERENCE.md ("generated from `plugin/hooks/hooks.json`") — do not exist in the bundle, so navigation dead-ends and the promised hook-event detail is unavailable. This sits between 'references present but not clearly signaled' (3) and broken/inlined-structure problems (2); the clear signaling keeps it at 3. | 3 / 5 |
Total | 16 / 20 Passed |