Content
65%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 actionable core is excellent — executable examples and specific parameter coverage for the common seaborn workflows. Weaknesses are the large generic-template boilerplate tail that pads tokens without seaborn-specific value, a generic rather than concrete workflow, and three sizeable bundle files that exist but are never linked from the overview.
Suggestions
Link the bundle files from the body so they are discoverable: e.g., under 'Example Usage' add '**More examples**: See [references/examples.md](references/examples.md)' and under 'Implementation Details' add '**Full API reference**: See [references/function_reference.md](references/function_reference.md) and [references/objects_interface.md](references/objects_interface.md)'.
Remove or drastically condense the generic template sections ('When Not to Use', 'Required Inputs', 'Deterministic Output Rules', 'Output Contract', 'Validation and Safety Rules', 'Failure Handling', 'Completion Checklist', 'Scope Reminder') — they are vague boilerplate that competes for context without adding seaborn-specific guidance.
Replace the generic 'Recommended Workflow' with a concrete seaborn-specific sequence (load/prepare tidy data → pick axes-level vs figure-level → set_theme → map hue/size/style → facet with row/col → annotate via matplotlib → savefig with dpi/bbox_inches), so the workflow reflects the actual plotting procedure.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The seaborn-specific sections (When to Use, Key Features, Example Usage, Implementation Details) are efficient and accurate, but a large tail of generic template sections — 'When Not to Use', 'Required Inputs', 'Deterministic Output Rules', 'Output Contract', 'Validation and Safety Rules', 'Failure Handling', 'Completion Checklist', 'Scope Reminder' — is vague, repetitive padding that adds no seaborn-specific value and could be trimmed substantially, exceeding the 'minor' threshold for a 4. | 3 / 5 |
Actionability | Provides a fully executable, copy-paste-ready example (load_dataset tips, set_theme, scatterplot with hue/style/size, catplot violin facets, correlation heatmap) covering the common relationship/categorical/correlation cases, plus specific parameter references (estimator=, errorbar=, bw_adjust, bins=, stat=, multiple=, sns.set_theme), matching the 5 anchor; the load_dataset internet caveat is flagged inline. | 5 / 5 |
Workflow Clarity | A numbered 'Recommended Workflow' and 'Completion Checklist' exist, but the steps are generic template language ('Select the documented execution path', 'Produce the expected output using the documented file format') rather than a concrete seaborn-specific procedure (load data → choose axes- vs figure-level → set theme → map semantics → facet → export), and the validation checkpoints are boilerplate rather than real checkpoints tied to the actual plotting task. | 3 / 5 |
Progressive Disclosure | The body has clear section structure and bulk content is appropriately split into separate files (references/examples.md, function_reference.md, objects_interface.md), but the SKILL.md body never references, links, or signals any of those bundle files, so a reader would not discover the 27KB API reference or 18KB examples — references are present in the bundle but not clearly signaled, matching the 3 anchor. | 3 / 5 |
Total | 14 / 20 Passed |