Content
88%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 strong, tightly written instruction skill: the quick-start call, concrete parameters, budget-aware validation steps, and WRONG/RIGHT anti-pattern make it immediately actionable. The only issues are a thrice-repeated fabrication warning and a Troubleshooting pointer to a file that is not present in the bundle.
Suggestions
State the no-fabrication rule once (e.g., keep the Anti-patterns WRONG/RIGHT example) and remove the duplicate statements in the Why section and Checklist, saving several lines.
Ship the referenced TROUBLESHOOTING.md alongside SKILL.md, or drop the pointer and inline the one-line fallback for when memory_smart_search is unavailable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence, but the no-fabrication rule is stated three times ('Only surface what the tool returned. Never fabricate...' in Why, the WRONG/RIGHT fabrication example in Anti-patterns, and 'Every observation shown came from the tool response... not invention' in Checklist) — a minor duplication that could be trimmed, matching anchor 4 rather than 5. | 4 / 5 |
Actionability | Fully executable guidance: a copy-paste-ready 'memory_smart_search { "query": ..., "limit": 10 }' call, an expected-output sample, and exact parameters for scoping ('project', 'agentId', 'targetLayer: "memory"/"observation"', 'expandIds' as comma-separated string). | 5 / 5 |
Workflow Clarity | A clear 6-step sequence with explicit validation checkpoints and error recovery: inspect 'matched_count', 'excluded_by_budget', and 'content_truncated' before treating empty results as absent; report failed expansions rather than guessing; and a no-match fallback with suggested alternative terms. A checklist reinforces the process. | 5 / 5 |
Progressive Disclosure | Sections are well organized (Quick start, Why, Workflow, Anti-patterns, Checklist, See also, Troubleshooting) and the reference is clearly signaled and one level deep. However, the body links to TROUBLESHOOTING.md ('See [TROUBLESHOOTING.md](TROUBLESHOOTING.md) if memory_smart_search is not available') and no such file exists in the bundle, leaving a dangling reference — a minor gap that fits anchor 4, not 5. | 4 / 5 |
Total | 18 / 20 Passed |