Content
75%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 well-structured, actionable instruction skill: concrete search commands, a clearly sequenced read-only workflow with explicit empty-result handling, and useful system-specific feedback-loop conventions. The main weaknesses are duplication of the briefing format, unverified assumptions about the memory tree's subfolder structure, and no retry path when initial searches come up empty.
Suggestions
Consolidate the two briefing-format templates (the Step 4 RECALL BRIEFING block and the closing "Format for recall briefing" block) into a single template to remove redundancy.
Verify the assumed subfolder layout: either precede the Step 1 keyword→folder mapping with a command that lists the actual tree structure, or note that the listed folders are conventional and to fall back to a full-tree grep if a folder doesn't exist.
Add a retry step to the workflow: if a keyword search returns nothing, broaden the query (search by industry or technique name instead of client name) before reporting 'No prior memories for this context.'
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence — concrete find/grep commands with no explanation of how they work, and the Feedback Loop section teaches system-specific facts (confidence levels, 60-day staleness threshold) Claude wouldn't already know. It falls short of 5 because of redundancy: the briefing format is specified twice (the full RECALL BRIEFING template in Step 4 and again in the "Format for recall briefing" block at the end), and the "You are a memory retrieval agent. Your job is..." framing could be trimmed. | 4 / 5 |
Actionability | Concrete, runnable bash commands are provided for each search mode (find all files, grep -ril for keywords, -mtime -7 for recent), and Step 1 maps task types to specific subdirectories. Minor gaps keep it at 4 rather than 5: the grep uses a "{keyword}" placeholder rather than worked examples, and the assumed subfolder structure (memory/clients/, memory/calls/, etc.) is never verified against the actual tree with a listing command. | 4 / 5 |
Workflow Clarity | The four steps (identify context → search tree → read and filter → return briefing) are clearly sequenced, each with concrete commands, and edge-case handling is explicit ("If nothing relevant exists, say so: 'No prior memories for this context.'") with staleness flagging in the feedback loop. This is a read-only, non-destructive task so the validation cap does not apply, but there is no error-recovery loop (e.g., retry with broader keywords when a search returns nothing) to justify 5. | 4 / 5 |
Progressive Disclosure | The single file is well organized with clear sections (When to Run, How to Search, Rules, Feedback Loop Integration) and, at ~100 lines with no bundle files, inlining everything is a defensible placement decision. It misses 5 because the body exceeds the under-50-lines threshold where a single well-organized file suffices, the duplicate briefing templates are a small structural redundancy, and the post-build technique-update protocol is a separable concern that could live in its own reference. | 4 / 5 |
Total | 16 / 20 Passed |