Content
100%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 lean reference: one fully executable wiring example, an exact adapter contract, concrete per-adapter scope semantics, explicit scope-security validation, and precise failure-mode/devtools behavior — all at ~110 lines with zero padding. Every dimension lands at the top anchor.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean throughout: a three-line orientation, a clear 'do NOT use for within-turn history' disambiguation, and dense package-specific facts on every section (adapter scope mappings like hindsight's '{tenant|_}__{user}__{threadId}' bank format, mem0's user_id/run_id, devtools payload contents). It assumes Claude's competence and never explains generic concepts; nothing is trimmable, so the level 4 'minor instances of over-explanation' anchor does not fit. | 5 / 5 |
Actionability | The wiring example is a complete, copy-paste-ready POST handler with real imports, server-side session resolution, and middleware options ('role', 'onRecall'/'onSave'); the MemoryAdapter interface shows exact signatures and return shapes ('{ systemPrompt, fragments?, tools?, toolGuidance? }'), and custom adapters get a concrete verification path via 'runMemoryAdapterContract from @tanstack/ai-memory/testkit'. No gaps remain for the level 4 anchor. | 5 / 5 |
Workflow Clarity | The single wiring action is unambiguous via the executable example, and the sequence is explicit: resolve session server-side, derive scope, recall before the model runs, defer save after the turn. An explicit validation checkpoint is present ('validate it belongs to the session user BEFORE using it'), error recovery is specified (non-fatal 'memory:error', streaming never blocked), and the testkit provides verification — satisfying the simple-skill exception with no validation gaps. | 5 / 5 |
Progressive Disclosure | The skill is self-contained with no bundle files, and nothing inlined clearly belongs in a separate file: ~110 cohesive lines of reference that would lose value if split. Section headers ('Wire it up', 'The contract', 'Scope security', 'Adapters', 'Failure modes', 'Devtools') make navigation trivial, and the one cross-reference ('see the in-memory skill') is clearly signaled. | 5 / 5 |
Total | 20 / 20 Passed |