Content
77%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is rich with executable, well-organized matplotlib guidance, but it is bloated with foundational concepts Claude already knows and—more seriously—points to four reference files and two scripts that do not exist, breaking its progressive-disclosure structure.
Suggestions
Create the referenced bundle files (references/plot_types.md, styling_guide.md, api_reference.md, common_issues.md and scripts/plot_template.py, style_configurator.py) or remove the dangling references so navigation is not broken.
Trim the Overview/Core Concepts explanation of the Figure/Axes/Artist hierarchy and the rcParams key list, moving that detail into a reference file and keeping only what Claude cannot infer.
Move the per-plot-type one-liners and color-specification catalog out of SKILL.md into references/plot_types.md and references/styling_guide.md to reduce the inline wall of text.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body explains concepts Claude already knows — the Figure/Axes/Artist/Axis hierarchy, one-line plot-type definitions, color-specification methods, and a raw rcParams key list — so it is padded rather than lean; it stops short of fully verbose prose only because the embedded code is genuinely useful. | 2 / 3 |
Actionability | Numerous complete, executable snippets (plt.subplots, subplot_mosaic, GridSpec, 3D surface, savefig with dpi/bbox_inches) are copy-paste ready, matching the fully-executable anchor. | 3 / 3 |
Workflow Clarity | Workflows are clearly sequenced (create figure → plot data → customize → save) and the single-action plotting task is unambiguous; no validation cap applies because plotting is not a destructive or batch operation. | 3 / 3 |
Progressive Disclosure | References are clearly signaled ('see references/plot_types.md', scripts/plot_template.py) but none of the referenced files or directories actually exist in the bundle, and the 200+ lines of plot-type/styling/API material that should be offloaded sit inline — the 'content that should be separate is inline' anchor rather than the clean one-level-deep structure. | 2 / 3 |
Total | 10 / 12 Passed |