Content
75%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.
The body is a well-structured, actionable rendering contract with executable code, explicit error-recovery paths, and verified one-level-deep references. Its weaknesses are repeated restatement of the lean-twin invariant, an implicit template-instantiation step, no output validation checkpoints, and two orphaned bundle files.
Suggestions
State the lean-twin invariant once (in § The asymmetry principle) and reference it elsewhere, removing the repeated [HARD] restatements in the tier table and Non-goals.
Add a short consolidated workflow (1. read report.format → 2. resolve tier → 3. fill the mode template → 4. write the pair → 5. validate size budget → 6. deliver) that makes template instantiation and output validation explicit.
Reference references/craft-fundamentals.md and references/design-tokens.md from § References (or remove them from the bundle) so every bundled file is discoverable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with load-bearing contract material — mode structure tables, the :root design-token block, the degradation matrix, the executable opener script — and avoids explaining concepts Claude already knows. The lean-twin invariant is restated roughly four times (§ The asymmetry principle, the [HARD] block, the tier table's 'restated because it is the one that is easy to violate' row, and Non-goals), which is minor over-emphasis that could be trimmed, fitting the 'efficient; minor instances' anchor rather than the 'mostly efficient but includes some unnecessary explanation' anchor at 3. | 4 / 5 |
Actionability | Mostly executable guidance: a copy-paste-ready `case "$(uname -s)"` opener script, a complete mermaid-CDN module with initialize config, concrete output paths and size budgets, and per-mode mustache input-field tables. The gap keeping it from 5 is that how the .mustache templates are actually instantiated (no render script; hand-substitution of {{#metrics}}-style placeholders is never stated) is left implicit. | 4 / 5 |
Workflow Clarity | The pipeline — read report.format, resolve the audience tier from the settings chain, render, write the .html/.md pair, deliver by format — is sequenced across ordered sections with explicit fallbacks ('falling back to html+md automatically when the Artifact tool is absent (no error)', opener failure prints the absolute path, the degradation matrix). It is not 5 because checkpoints are implicit in section order rather than a consolidated sequence, and nothing validates the stated size budgets or confirms the file write. | 4 / 5 |
Progressive Disclosure | All body-referenced paths resolve to real bundle files one level deep (references/artifact-contract.md, references/fonts.md, and all six references/templates/*.mustache), clearly signaled in § References with per-file descriptions. Minor organization gaps: references/craft-fundamentals.md and references/design-tokens.md exist in the bundle but are never referenced from SKILL.md, and some inlined detail (the full mermaid script, per-mode input fields) could live in a reference file. | 4 / 5 |
Total | 16 / 20 Passed |