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 lean, high-signal reference skill: it documents non-obvious axllm API semantics with concrete names, a code skeleton, and pointed guardrails, without padding. It falls short of top marks only on minor points — a non-self-contained core snippet, inlined version-migration notes, and references that resolve inside the target package rather than the skill bundle.
Suggestions
Inline one complete, compilable example (e.g., a no-key skills_and_memory_assistant excerpt) so the Core Pattern's undefined `llm` and `inputs` are grounded in copy-paste-ready code.
Move version-migration details ("Since 25.0.0... a map saved by a 24.x port loads with its items") into a short "Compatibility / old formats" section so time-sensitive notes don't dilute the main guidance.
Split the dense Lifecycle And State bullets (e.g., the multi-clause contextMap bullet) into shorter statements or move per-API detail into a referenced API file within the bundle.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and factual with no padding or explanations of known concepts — bullets like "Forward-time `skills` override constructor entries by normalized ID and remain loaded for later calls" are pure non-obvious API semantics. Minor deductions: version-migration detail ("Since 25.0.0 the text and item ids follow TypeScript's format... a map saved by a 24.x port loads with its items") is time-sensitive changelog content outside any deprecation section, and several Lifecycle bullets are packed run-on sentences that could be tightened. | 4 / 5 |
Actionability | Concrete API names (`onSkillsSearch`, `relevanceRanking`, `get_state`, `export_runtime_state`), a runnable-example pointer (`src/examples/rust/long-agents/skills_and_memory_assistant.rs`), and explicit guardrails ("Start from package examples for exact native syntax") give mostly executable guidance. The Core Pattern snippet is not self-contained (`inputs` and `llm` are undefined) and no full working example is inlined, so it stops short of copy-paste ready. | 4 / 5 |
Workflow Clarity | This is a reference guide rather than a sequential workflow, but the entry point is unambiguous ("Start from package examples for exact native syntax before inventing a new call shape") and guardrails act as checkpoints ("Do not interchange the two shapes"). No destructive or batch operations, so no validation-loop cap applies; it lacks explicit verify/feedback steps, keeping it below 5. | 4 / 5 |
Progressive Disclosure | A single well-organized file with clear sections (Package Facts, Runnable Examples, Relevant API Surface, Guardrails) and one-level-deep references clearly signaled (`API.md`, `examples/`, the website gallery). However, the referenced paths live in the target package rather than the skill bundle (no references/ or scripts/ exist), and the dense Lifecycle API semantics are all inlined rather than split out, leaving minor organization gaps. | 4 / 5 |
Total | 16 / 20 Passed |