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.
A high-quality, highly actionable codegen reference: canonical pattern, copy-ready snippets, explicit guardrails, and an error-fix table. The main weaknesses are inline monolith structure (the mermaid gallery and MCP/event/telemetry sections should be split into reference files), some duplicated rules across sections, and a few snippets that rely on undefined placeholder symbols.
Suggestions
Move the Mermaid dialect + Flow Gallery, Native MCP/UCP, Event-Triggered Flows, and Runtime Hooks sections into references/*.md files and keep one-line pointers plus a single example each in SKILL.md.
Deduplicate rules stated in both 'Use These Defaults', 'Critical Rules', and 'Do Not Generate' (node-before-execute, naming, loop conditions) into one canonical rules section.
Define or annotate placeholder symbols (fetchFromAPI, lookupProgram, checkoutProgram, inventory/merchant, llm, input) so every snippet is self-contained, and add an explicit 'verify generated code' step (typecheck / flow(String(wf)) round-trip).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and code-first with almost no tutorial prose ('Do not write tutorial prose unless the user explicitly asks'), and it explicitly establishes the 'wf is pre-defined' convention to keep excerpts short. However, rules are duplicated across sections — 'define .node() before .execute()' appears in Use These Defaults, Critical Rules, and Do Not Generate, and naming/loop-condition rules each repeat — which keeps it below the lean 'every token earns its place' anchor. | 4 / 5 |
Actionability | Nearly all guidance is concrete, executable TypeScript (canonical pattern, factory options, alias table, planner rules, error-to-cause mapping). Below anchor 5 because several snippets depend on undefined placeholders such as fetchFromAPI, lookupProgram/checkoutProgram, inventory/merchant, and bare llm/input in the mermaid and error-handling sections. | 4 / 5 |
Workflow Clarity | The codegen process is clearly sequenced (defaults → critical rules → canonical pattern → feature recipes → Do-Not-Generate guardrails) and the common-errors section maps each runtime error to its fix, giving a real feedback loop. Not a destructive/batch skill, so no cap applies; it misses anchor 5 only for lack of an explicit validate-the-output checkpoint (e.g. typecheck or 'flow(String(wf)) round-trips' verification of generated code). | 4 / 5 |
Progressive Disclosure | No references/, scripts/, or assets/ bundle exists, so everything — roughly 750 lines — is inlined in SKILL.md. Section headers are clear and cross-skill/example links (ax-signature, ax-typesafe, ax-mcp, ax-gepa, GitHub example URLs) are well signaled, but large self-contained topics (the Mermaid dialect plus its 10-diagram gallery, MCP/UCP coordination, event-triggered flows, telemetry hooks) clearly belong in separate reference files, matching the 'content that should be separate is inline' anchor. | 3 / 5 |
Total | 15 / 20 Passed |