Content
85%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 tight, well-sequenced instruction skill with concrete tool-call syntax, explicit validation, and honest empty-window handling. The main defects are the dangling TROUBLESHOOTING.md reference and a couple of inference-heavy details (recall query construction, cwd matching).
Suggestions
Ship the referenced TROUBLESHOOTING.md in the bundle (or inline a 2-3 line fallback for when memory_sessions/memory_recall is unavailable) so the link at the end of the body resolves.
Spell out the cwd-matching rule concretely — e.g., how to normalize the path (case, trailing slash) and what a 'directory boundary' match means, with a one-line example.
Define what '<top concepts>' means for the memory_recall query — e.g., derive the query from the session title or its first prompt, with an example.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean with zero padding: Quick start gives tool syntax in a few lines, 'Why' states a skill-specific grounding policy ('An empty window is a real answer, not a prompt to invent activity'), and nothing explains concepts Claude already knows. Every section earns its place, matching the 'every token earns its place' anchor. | 5 / 5 |
Actionability | Concrete tool calls with realistic parameter shapes ('memory_sessions { "project": ..., "limit": 30 }') and an expected-output sample make the quick start near-executable, and the 5-step workflow is specific. It falls short of anchor 5 because the recall query ('"query": "<top concepts>"') and the cwd-matching heuristic ('match normalized cwd by directory boundary') leave key details to inference rather than being copy-paste ready. | 4 / 5 |
Workflow Clarity | The 5-step workflow is clearly sequenced with an explicit validation checkpoint ('verify each result belongs to that session before including it'), a paging loop ('Follow nextCursor with the same filters until the requested window is covered or pages end'), empty-window error handling in Anti-patterns, and a closing checklist. This matches anchor 5's 'explicit validation steps; feedback loops for error recovery; checklists' — the operation is read-only, so the destructive/batch cap does not apply. | 5 / 5 |
Progressive Disclosure | Sections are well organized and detail is correctly deferred, but the body's single external reference — 'See [TROUBLESHOOTING.md](TROUBLESHOOTING.md) if memory_sessions or memory_recall is not available' — points to a file that does not exist in the bundle (no references/, scripts/, or assets/ directories, and no TROUBLESHOOTING.md anywhere). Scored against the actual bundle structure, the navigation dead-ends exactly when the user needs recovery guidance, which caps this at anchor 3 rather than 4's 'minor organization gaps'. | 3 / 5 |
Total | 17 / 20 Passed |