Content
85%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 unusually dense, high-signal ruleset: every section is written as an imperative, testable rule, the render-then-verify workflow has real feedback loops, and the code blocks are executable as written. The weaknesses are structural: nothing is offloaded to reference files despite ~400 lines of content, and the §8 anti-pattern list duplicates rules already stated earlier.
Suggestions
Move the §6 chart-family catalog and the §9 render-then-verify code into one-level-deep reference files (e.g., references/chart-families.md and references/qa-checks.md), keeping SKILL.md as a scannable overview with clearly signaled links.
Consolidate §8 with the source rules it restates — e.g., mark the §3.3, §4.4, and §4.5 rules inline as anti-patterns or trim the duplicated bullets — so those tokens are not spent twice.
Trim the overlap between the intro paragraph and §0's scope discussion into a single load-trigger statement.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, imperative, and assumes competence — no basic-concept padding, no library tutorials — but not every token earns its place: the §8 anti-pattern list largely restates rules already stated in §1–§7 ("Filled bars on a log-scaled value axis" repeats §3.3; the diverging-colormap center bullet repeats §4.4), and the §0 scope discussion overlaps the intro paragraph. That is the efficient-with-minor-trimming anchor, not the lean 5. | 4 / 5 |
Actionability | Two complete, executable code blocks (the §9.1 bbox/overlap check and the §9.2 panel-crop loop) plus named helper calls with concrete signatures ("apply_figure_style(sizes=(8,7,6))", "focal_palette(...)", "end_of_line_labels(...)", "panel_crops(fig)"). Rules are phrased as testable criteria ("read it aloud cold — if the listener asks 'fewer genes what?', rewrite"), which is fully actionable guidance rather than vague direction. | 5 / 5 |
Workflow Clarity | The process is explicitly sequenced with checkpoints: scope gate in §0, the apply-style call before plotting, per-section rules while plotting, then "After `fig.savefig(...)`, before `save_artifacts`" for §9, with a genuine feedback loop — "Fix (move, shorten, stagger) and re-save until clean" — and a second perceptual verification pass. This matches the clear-sequence-with-explicit-validation-and-feedback anchor. | 5 / 5 |
Progressive Disclosure | Section structure (§0–§9) is clear and easy to navigate, but all 409 lines live inline in SKILL.md with no bundle files at all — the §6 chart-family catalog and the §9 QA code are substantial bodies that would plausibly belong in one-level-deep reference files. This is the some-structure-but-content-that-should-be-separate-is-inline anchor; it is above 2 because the in-file organization is strong, not minimal. | 3 / 5 |
Total | 17 / 20 Passed |