Content
76%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 token-efficient, well-sectioned overview that points to external docs instead of inlining them, with one concrete code pattern and useful guardrails. Its main weakness is that the central task — deciding between contextMap, contextPolicy, optimization, and recall — is named without any decision procedure or criteria to carry it out, and the code example has an undefined `llm` dependency.
Suggestions
Add 2-3 decision criteria under 'When To Use' for choosing between contextMap, contextPolicy, optimization, and recall (e.g., which to pick for persistent corpus orientation vs. within-run compaction vs. improving prompts offline).
Complete the Core Pattern by showing how the `llm` argument is constructed, ideally using the advertised no-key scripted transport so the example runs without provider credentials.
Trim 'Relevant API Surface' to the four routing-relevant entry points and delegate the full list to the referenced `API.md`/`axir-api.json` to reduce duplication with the external docs.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is a lean, well-sectioned manifest: 'Real network support: yes', 'Treat AxIR as the source of generated package truth' — every line adds package-specific facts or constraints Claude could not know, with no padding or explanation of concepts Claude already knows. The only near-redundant lines ('Language: Python.', 'Package: `axllm`') duplicate the description, but these are terse manifest facts rather than over-explanation. | 5 / 5 |
Actionability | The Core Pattern gives real, executable syntax ('helper = agent("question:string -> answer:string")', 'helper.forward(llm, {...})') and Guardrails give concrete directives ('Use `no-key` examples for deterministic local checks'). It falls short of 5 because `llm` is never constructed or shown (no copy-paste-ready path from import to output), and the no-key transport is named but never demonstrated. | 4 / 5 |
Workflow Clarity | The skill's core task is routing ('Choose between contextMap, contextPolicy, optimization, and recall for a task'), but no decision criteria or sequence tells the agent HOW to choose between them — only that it should. Guardrails ('Start from package examples for exact native syntax before inventing a new call shape') act as loose checkpoints, which keeps this above 2, but the central decision flow has no explicit steps, so it cannot reach 4. | 3 / 5 |
Progressive Disclosure | The body correctly stays an overview and signals one-level-deep references ('Package API docs: `API.md` and `axir-api.json`', 'Runnable examples: `examples/`') rather than inlining bulk API material. It misses 5 because no bundle files exist (no references/, scripts/, or assets/ directories ship with the skill), so the referenced paths cannot be navigated from the skill itself, and the 'Relevant API Surface' section inlines an API index that overlaps the referenced API docs. | 4 / 5 |
Total | 16 / 20 Passed |