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.
This is a highly actionable, accurate API-contract skill: executable code for every feature, explicit turn-by-turn sequencing, and zero filler or generic explanation. Its weaknesses are repetition and an A/B-statistics digression that cost tokens, and the absence of any bundle structure — the whole 440-line reference is inlined where per-feature reference files would keep SKILL.md an overview.
Suggestions
Split the per-feature detail into one-level-deep reference files (e.g., references/memory-search.md, references/skills-search.md, references/context-map.md, references/types.md) and keep SKILL.md as a defaults/overview with clearly signaled links, so the full API reference is not loaded on every invocation.
State each contract once: the 'returns void' rule and the no-loop/no-Promise.all rule each appear in both a Rules list and the Do Not Generate section — consolidate them into one canonical location.
Trim the relevanceRanking A/B-gate statistics sentence to a one-line outcome ('enabled by default after A/B validation improved lookup precision and answer accuracy') — the per-model run counts and percentages are justification, not guidance.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with non-inferable contract detail (stage visibility, dedup/sort order, callback precedence, catalog vs callback semantics) and explains nothing Claude already knows, but it repeats itself — "both return `void`" appears in roughly four places, the no-loop/no-Promise.all rule appears twice, and the A/B-gate statistics sentence ("49 runs per variant per model: 24%→90% ... 63%→88%") is justification detail that earns no tokens. This matches level 4 (efficient with minor over-explanation that could be trimmed) rather than level 5's every-token-earns-its-place. | 4 / 5 |
Actionability | Every feature ships complete, copy-paste-ready TypeScript: enabling onMemoriesSearch/onSkillsSearch, static catalogs, forward-time preloading, actor-side recall/discover turn usage, carrying memories across calls, relevanceRanking, and used-tracking, plus full type definitions and runnable example URLs. This matches level 5's fully executable coverage of the common cases. | 5 / 5 |
Workflow Clarity | The enable → wire → actor-usage flow is unambiguous: 'Use These Defaults' acts as a decision procedure (catalog vs callback vs contextMap), actor usage is explicitly turn-sequenced ("Turn 1: kick off one batched lookup / Turn 2+: matched entries are now visible"), and a Do-Not-Generate section guards the failure modes. No destructive or batch operations are involved, so the validation cap does not apply, and the sequencing is explicit enough for level 5 rather than level 4's 'most checkpoints'. | 5 / 5 |
Progressive Disclosure | No bundle files exist, and the entire ~440-line API reference — type definitions, per-option rule lists, cross-language persistence notes — is inlined in SKILL.md, matching level 3's 'content that should be separate is inline'. Headers are well organized and the example URLs are clearly signaled one level deep, which keeps it above level 2, but the placement gap is more than the 'minor organization gaps' of level 4 since effectively all detail lives in the main file. | 3 / 5 |
Total | 17 / 20 Passed |