Content
78%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, well-organized reference body that carries real package-specific semantics and clear pointers to runnable examples. The main weaknesses are the absence of executable syntax for the skill's core memory/skills options, an inconsistent package prefix in the sole code block, and version-migration detail inlined in the lifecycle section rather than isolated.
Suggestions
Add a minimal executable snippet showing constructor `skills` and forward-input `memories` usage with the actual native option-map key names, so the core features do not depend entirely on the external example file.
Reconcile the package prefix in the Core Pattern (`ax.NewAgent`) with the API surface section (`axllm.NewAgent`) and make the snippet self-contained by defining `llm`.
Move version-migration details ('Since 25.0.0...', 'a map saved by a 24.x port...') into a short 'Old formats / migration' note so the lifecycle section stays lean.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and fact-per-line with no over-explanation of concepts Claude already knows, but the contextMap bullet carries version-sensitive migration detail ('Since 25.0.0...', 'a map saved by a 24.x port loads with its items...') outside any old-patterns or deprecated section, which the guidelines penalize. | 4 / 5 |
Actionability | Concrete elements are present — a real Core Pattern code block, a named runnable example file, precise option semantics, and exact API names — but the snippet has an undefined `llm` and a package prefix inconsistent with the API section (`ax.NewAgent` vs `axllm.NewAgent`), and no syntax or option-map key names for the core memory/skills features, deferring entirely to external examples. | 4 / 5 |
Workflow Clarity | There is no numbered sequence (reference-style skill), but the Guardrails encode an implicit ordered procedure — 'Start from package examples for exact native syntax before inventing a new call shape' followed by the verification avenue 'Use `no-key` examples for deterministic local checks' — giving a clear sequence with most checkpoints present and only minor validation gaps. | 4 / 5 |
Progressive Disclosure | A well-sectioned ~60-line overview that appropriately defers bulk detail to one-level-deep, clearly signaled references ('API.md', 'axir-capabilities.json', 'examples/', the runnable example paths) with no nested references and easy navigation. | 5 / 5 |
Total | 17 / 20 Passed |