Content
86%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 efficient, highly actionable best-practices skill: terse decision tables, exact API names, explicit Never rules, and a clean one-level-deep reference split. The main gaps are the absence of any inline code skeleton for the @capture("graph") pattern the skill itself says most charts need, and an unverifiable cross-skill reference.
Suggestions
Add a minimal @capture("graph") function skeleton (5-8 lines) to the body, since the skill states most bar and line charts need one — a concrete shape would make that instruction copy-paste ready.
Include a one-line Figure model YAML example for kpi_card showing the '_target_: kpi_card → title:' placement rule, which is currently described but never shown.
Soften or localize the ../dashboard-build/references/examples/example_ag_grid.py pointer (e.g., name the pattern in-bundle) so the guidance degrades gracefully when the sibling skill is absent.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and table-driven with zero concept explanations: every line is a decision rule, an exact API pointer, or a reference. The one explanatory sentence — 'Inline px.bar(data_frame="raw"...) on detail-level data stacks individual rows as separate rectangles instead of summing' — encodes non-obvious Vizro-specific failure behavior, not knowledge Claude already has, matching the 'every token earns its place' anchor. | 5 / 5 |
Actionability | Guidance names exact APIs and argument conventions: 'kpi_card / kpi_card_reference from vizro.figures', 'vm.Table with figure=dash_ag_grid(data_frame=...)', '@capture("graph")', 'data_frame as the exact DataFrame argument name (not df, not data)', and 'from vizro.themes import palettes, colors'. It stops short of the score-5 anchor because the body contains no copy-paste-ready code example (e.g., a full @capture("graph") function or a Figure model YAML snippet) — those live in the reference file. | 4 / 5 |
Workflow Clarity | The skill is a decision-rules reference rather than a sequential workflow, and per-scenario routing is unambiguous: a data-question→chart table, explicit 'Use when' criteria for custom charts, and hard Never rules for KPI cards and tables. No destructive or batch operations require validation checkpoints, but the multi-topic structure has no overarching flow connecting selection through implementation, placing it between the clear-sequence anchor and the simple-skill exception. | 4 / 5 |
Progressive Disclosure | The body is a concise overview with one-level-deep, well-signaled references: 'See chart-best-practices.md for palette names and import patterns' plus a Deep Dive section enumerating exactly what the reference covers ('extended chart type decision tree, Plotly Express formatting conventions... accessibility rules, or detailed @capture("graph") guidance'), and the referenced file exists in the bundle. The cross-skill pointer to ../dashboard-build/references/examples/example_ag_grid.py is clearly labeled but not verifiable within this bundle — a negligible organization note against an otherwise exemplary split. | 5 / 5 |
Total | 18 / 20 Passed |