Content
92%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 exceptionally well-structured skill body: actionable at every step, clearly sequenced with decision points and failure-recovery guidance, and an exemplary routing/progressive-disclosure architecture that keeps type-specific detail in one-level-deep reference files. The only weakness is mild editorial verbosity in the context-gathering and file-reuse sections.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and largely earns its tokens — routing table, numbered universal constraints with concrete syntax examples, exact tool parameters — but Step 4 and the reuse section carry editorial padding ("One good question beats one wasted diagram", "readers often benefit from seeing the history of attempts", "Regenerating 4 times = 4 draft files to clean up"). These are minor over-explanations that could be trimmed, matching the 4 anchor rather than the 5 ('every token earns its place') or the 3 ('several unnecessary explanations'). | 4 / 5 |
Actionability | Fully concrete, executable guidance throughout: exact supported type list (`flowchart`, `sequenceDiagram`, `stateDiagram` / `stateDiagram-v2`, `gantt`, `erDiagram`), an explicit unsupported list with the correct fallback ("tell them directly… instead of calling the tool and failing"), concrete Mermaid syntax examples (`A["Process (main)"]`, `-->|"O(1) lookup"|`), the exact FigJam error string to avoid, all tool parameters with usage conditions, and a `fileKey` extraction rule. Nothing is pseudocode or vague; an instruction-only skill with copy-ready specificity, matching the 5 anchor. | 5 / 5 |
Workflow Clarity | A clear 7-step sequence (tool fit → type selection → universal constraints → context gathering → hybrid decision → tool call → post-generation) with explicit decision checkpoints ("When in doubt, ask the user", hybrid yes/no signals) and genuine feedback loops for error recovery: stop after 2 failed attempts and ask what's wrong, ask the user's file-reuse preference and remember it. Per-type validation checklists live in the references (e.g. architecture.md's validation-checklist section). This exceeds the 4 anchor and matches 5; the destructive/batch cap does not apply since generation is non-destructive and file proliferation is explicitly managed. | 5 / 5 |
Progressive Disclosure | The body is a textbook overview/routing document: a type-selection table pointing to seven real one-level-deep reference files via markdown links, with type-specific detail (layout codes, per-type rules) correctly deferred to those files. Verified the bundle: all referenced files exist and reference files link only to sibling references or internal anchors — no nested chains. Matches the 5 anchor (clear overview, well-signaled one-level-deep references, easy navigation). | 5 / 5 |
Total | 19 / 20 Passed |