Content
71%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.
Highly actionable content with comprehensive executable TypeScript patterns and clear decision guidance, undermined by monolithic length, literal rule duplication across sections, and reference-grade detail (language tables, auto-upgrade internals, event-driven wake flows) inlined where separate reference files should carry it. The skill's own delegation discipline toward sibling skills is excellent; it should apply the same split to its own body.
Suggestions
Deduplicate rules stated verbatim in multiple sections: the discovery-before-call rule appears in both 'Use These Defaults' (line 35) and 'Critical Rules' (line 56), and bubbleErrors guidance is repeated in 'Use These Defaults', 'Decision Guide', 'Critical Rules', and the 'Bubble Errors' section — consolidate each into one canonical location.
Split reference-grade material into files under references/ (e.g. references/learning-citations.md for the language-spelling table and playbook evolve details, references/auto-upgrade.md for threshold internals, references/event-driven.md for wake/webhook flows), keeping one-line pointers in SKILL.md — mirroring the delegation pattern already used for ax-mcp and ax-agent-rlm.
Tighten the dense run-on paragraphs (autoUpgrade at line 57, playbook at line 615, relevanceRanking at line 279) to a single rule plus one short example each, moving edge-case caveats to the new reference files.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense rule-style with no tutorial padding ('Do not write tutorial prose unless the user explicitly asks for explanation') and complete executable examples, but at ~710 lines it contains literal duplication — the discovery rule appears identically in 'Use These Defaults' (line 35) and 'Critical Rules' (line 56), bubbleErrors guidance is repeated across three sections — and several run-on mega-paragraphs (autoUpgrade, playbook, relevanceRanking) that could be tightened. Anchor 3: mostly efficient but could be tightened; not 4 due to the duplicated rules, not 2 since nothing explains concepts Claude already knows. | 3 / 5 |
Actionability | Every pattern ships a complete, copy-paste-ready TypeScript example (canonical agent, voice I/O, child agents, fn() tools, grouped modules, host-side protocol, clarification resume with getState/setState, bubbleErrors), plus exact API shapes like the discovery signatures and structured clarification payload rules. Fully executable and covering the common cases — anchor 5. | 5 / 5 |
Workflow Clarity | A 'Decision Guide' maps user intent to agent shape before any code is written, a 'Do Not Generate' section lists anti-patterns, and the clarification section shows an explicit error-recovery loop (catch AxAgentClarificationError -> error.getState() -> agent.setState -> forward). As a codegen skill no validation checkpoints are required, but rules are scattered across four overlapping sections (Use These Defaults / Decision Guide / Critical Rules / per-section Rules), leaving minor sequencing gaps — anchor 4, not 5. | 4 / 5 |
Progressive Disclosure | No references/, scripts/, or assets/ bundle exists; the entire surface is inlined in one ~710-line SKILL.md. Deep-dive material that clearly belongs in separate reference files is inline (the Learning/Citations language-spelling table, auto-upgrade threshold details, the event-driven/MCP-wake sections), though delegation to sibling skills is well signaled and full examples live behind external links. Anchor 3: some structure and clear delegation, but content that should be separate is inline; not 4 given no actual reference files exist. | 3 / 5 |
Total | 15 / 20 Passed |