Content
63%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 strong, highly actionable API reference with a clear hook-selection guide and guardrails, undermined by token waste: the duplicated actorTurnCallback type block and inline reference-material bulk. Splitting type/event-shape details into a references file and de-duplicating would lift both conciseness and progressive disclosure.
Suggestions
Delete the duplicated actorTurnCallback type block (the second copy marked 'deprecated alias'); if the alias must be documented, one line suffices.
Move the full callback type signatures and event-shape details into a references/ file (e.g., references/types.md), keeping usage examples and selection guidance inline in SKILL.md.
Trim rules that restate the field definitions verbatim (e.g., 'output is not raw stdout' repeats the earlier output definition) and consolidate each hook's 'What it gives you' bullets with its rules list.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Dense rules and code with no explanations of known concepts, but the actorTurnCallback type block is duplicated verbatim (~30 lines, second copy labeled 'deprecated alias') and several rules restate earlier bullet definitions. Mostly efficient but noticeably could be tightened — more than minor trimming, less than several padded sections. | 3 / 5 |
Actionability | Every hook has a concrete TypeScript example plus full type signatures (e.g., the axGlobals.onUsage + usageContext forward snippet and the onFunctionCall payload shape). Examples reference undefined context (usageQueue, runtime, llm, auth), making them illustrative rather than fully copy-paste ready — minor gaps. | 4 / 5 |
Workflow Clarity | 'Choose The Smallest Hook' gives an unambiguous decision sequence, each section opens with 'Use X when...', and 'Do Not Generate' provides guardrails; no validation steps are needed (no destructive/batch operations). Clear selection workflow, but no explicit error-recovery or end-to-end setup ordering beyond the globals snippet. | 4 / 5 |
Progressive Disclosure | No bundle files exist; a ~400-line SKILL.md inlines bulk type-signature dumps and event-shape reference material (part of it duplicated) that belongs in a references/ file. Headers are clear and the one cross-skill reference ('also read ax-ai') is signaled, but reference content that should be separate is inline. | 3 / 5 |
Total | 14 / 20 Passed |