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 content is highly actionable with executable config and code across all providers, but it is overlong for a SKILL.md, inlines material that belongs in reference files, and lacks validation feedback loops around its destructive database-reset operations.
Suggestions
Move the per-provider configuration blocks and troubleshooting recipes into reference files (e.g. references/providers.md, references/troubleshooting.md) and link to them from SKILL.md, including a pointer to the existing references/.env.example rather than re-inlining every env var.
Add explicit validation steps before destructive operations — e.g. back up, then verify the backup exists, before 'rm -rf ~/.auto-claude/memories/embeddings', and confirm a fresh DB is created afterward.
Trim the repeated GRAPHITI_ENABLED=true lines and the illustrative 'Memory in Action' narrative, and relocate time-sensitive model/version names to a clearly dated section so they do not penalize conciseness.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient with concrete config blocks, but includes padding — repeated GRAPHITI_ENABLED=true across every provider block, a narrative 'Memory in Action' example, and time-sensitive model names (gpt-4o-mini, claude-sonnet-4-5-latest, gemini-2.0-flash) outside any deprecated section — so it could be tightened rather than earning 4. | 3 / 5 |
Actionability | Copy-paste-ready env vars, bash install/pull commands, a runnable Python API example, and concrete CLI query commands cover the common cases fully, matching the 'fully executable' top anchor. | 5 / 5 |
Workflow Clarity | Steps are present in 'How Memory Works' and troubleshooting, but validation checkpoints are implicit or missing; the destructive 'rm -rf ~/.auto-claude/memories/embeddings' reset has no validate-before-proceed loop, which caps workflow_clarity at 3 per the destructive-operation rule. | 3 / 5 |
Progressive Disclosure | Section headers give reasonable structure, but ~370 lines of provider configs, troubleshooting, and advanced usage are inlined in SKILL.md instead of split out, and the existing references/.env.example bundle file is never linked from the body — fitting 'some structure but content that should be separate is inline'. | 3 / 5 |
Total | 14 / 20 Passed |