Content
57%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.
An information-dense, highly concrete reference body: it assumes Claude's intelligence, avoids known-concept padding, and specifies exact defaults, error strings, and signatures. Its weaknesses are structural — monolithic inline detail that belongs in reference files, no sequenced workflow with validation checkpoints, and dangling references to bundle files that do not exist.
Suggestions
Move per-provider reference detail (sampling rules per model family, retry/backoff specifics, Vertex embedding rules, session-adapter semantics) into a references/ file or files, keeping only a short decision summary per topic in SKILL.md.
Add a sequenced quickstart workflow — choose a named profile, build the client with `ai(...)`, verify against a scripted no-key example, then switch to provider credentials — with explicit validation checkpoints so the topic-organized rules have an executing order.
Either include the referenced materials (`API.md`, `axir-api.json`, `axir-capabilities.json`, `examples/`) in the skill bundle or remove the dangling path references, since none currently exist alongside SKILL.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every sentence carries package-specific rules with no padding about concepts Claude already knows (nothing like "PDF files store text and images..."), so it is above 2. But roughly 160 lines of dense run-on prose — e.g., the sampling paragraph "temperature 0, or temperature 0.7 and top-p 1 for `openai-responses`... GPT-5.1-5.4 while reasoning is off, their default; GPT-5.5 and 5.6 with effort `none`" — could be dramatically tightened or moved to reference tables, matching the 3 anchor "mostly efficient but could be tightened" rather than the lean 4-5 anchors. | 3 / 5 |
Actionability | Concrete, specific guidance dominates: one executable snippet (`llm = ai('openai', api_key=os.environ['OPENAI_API_KEY'])`), exact defaults ("`retryableStatusCodes`, by default 500, 408, 429, 502, 503, 504 and 529"), exact error messages ("Request timed out after <N>ms"), and exact signatures (`add_child_agent(namespace, name, child)`). It is not a 5 because the bulk of guidance is prose rules with only a single code example, and the "start with examples under `examples/`" instructions point to paths that are not part of the skill bundle. | 4 / 5 |
Workflow Clarity | The body is organized by topic (profiles, caching, timeouts, retries, routing, sessions), never by sequenced steps, and contains no validation checkpoints — only starting-point guidance like "Start from package examples for exact native syntax before inventing a new call shape". This matches the 3 anchor (sequence/checkpoints implicit), above 2 because sections are coherent and ordered, below 4 because no explicit step sequence or verification loop exists anywhere. | 3 / 5 |
Progressive Disclosure | Thirteen well-labeled sections provide real structure, and external materials are named ("`API.md` and `axir-api.json`", "`axir-capabilities.json`", "`examples/`", a URL), matching the 3 anchor's "some structure... content that should be separate is inline". It is not a 4-5 because essentially all detail (per-provider sampling rules, retry/backoff internals, session semantics, the full API surface list) is inlined in SKILL.md, and the referenced files are not present in the bundle, so the claimed split between overview and reference does not actually exist. | 3 / 5 |
Total | 13 / 20 Passed |