Content
56%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 delivers highly actionable, correct d3 v7 guidance with a clear workflow and good coverage of chart types, interactivity, and troubleshooting. Its main weaknesses are verbosity and duplication: the pattern and scales sections repeat content already in the references/ bundle files, and the Resources section misnames the asset files (.js vs actual .jsx).
Suggestions
Cut the ~250-line "Common visualisation patterns" and "Scales reference" sections down to one short inline example each and point to the existing references/d3-patterns.md and references/scale-reference.md instead of duplicating them in the body.
Fix the asset references in the Resources section to match the actual filenames: chart-template.jsx and interactive-template.jsx (not .js), so Claude can find and read them.
Remove the near-identical boilerplate setup (select/clear/margins/scale creation) repeated across drawChart, drawVisualization, and drawBarChart — show it once in the workflow and have pattern snippets assume it.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~820-line body inlines ~250 lines of pattern code and a ~60-line scales reference that duplicate the existing references/d3-patterns.md and references/scale-reference.md, and the boilerplate setup (select, clear, margins, groups) repeats nearly verbatim three times. This goes beyond "some unnecessary explanation" — several sections are padded or duplicated, matching anchor 2. | 2 / 5 |
Actionability | Concrete, executable d3 v7 code throughout (full copy-paste-ready functions for bar chart, chord diagram, heatmap, force network, with data-format comments), plus actionable interactivity, transition, and troubleshooting sections. Not 5: several snippets (line chart, pie, scatter) reference undefined surrounding scope (xScale, g, width) and are fragments rather than runnable code. | 4 / 5 |
Workflow Clarity | The core workflow is a clearly numbered sequence (setup → integration pattern → structure → responsive sizing) with data guards ("if (!data || data.length === 0) return;"), returned cleanup functions for resize listeners/observers, and a Common issues troubleshooting section. Not 5: no explicit validation checkpoints or feedback loops; the domain is non-destructive so the cap does not apply. | 4 / 5 |
Progressive Disclosure | The Resources section clearly signals one-level-deep references (d3-patterns.md, scale-reference.md, colour-schemes.md) and they exist, but bulk pattern/scale content that should live in those files is inlined and duplicated in the body. Additionally, the body cites "chart-template.js" and "interactive-template.js" while the actual assets are .jsx files — a filename mismatch. Content that should be separate is inline, matching anchor 3; not 4 because the duplication is extensive and asset references are inaccurate. | 3 / 5 |
Total | 13 / 20 Passed |