chore(deps)(deps-dev): bump recharts from 2.15.4 to 3.9.0#1361
chore(deps)(deps-dev): bump recharts from 2.15.4 to 3.9.0#1361dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [recharts](https://github.com/recharts/recharts) from 2.15.4 to 3.9.0. - [Release notes](https://github.com/recharts/recharts/releases) - [Changelog](https://github.com/recharts/recharts/blob/main/CHANGELOG.md) - [Commits](recharts/recharts@v2.15.4...v3.9.0) --- updated-dependencies: - dependency-name: recharts dependency-version: 3.9.0 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
LabelsThe following labels could not be found: Please fix the above issues or remove invalid values from |
dependabot
Bot
requested review from
Jurij89,
branarakic and
zsculac
as code owners
| "react-dom": "^19", | ||
| "react-router-dom": "^7", | ||
| "recharts": "^2", | ||
| "recharts": "^3", |
There was a problem hiding this comment.
🟡 Issue: Recharts v3 needs the multi-axis grid to target an existing axis
What's wrong
The dependency bump changes Recharts’ CartesianGrid axis-selection behavior, but the existing multi-axis chart was not updated. Because both Y axes have non-default ids, the grid can stop rendering after this upgrade, weakening the Operations hardware chart’s readability.
Example
Open the Operations hardware history after two snapshots. With Recharts 2, the Network & RPC chart grid used the first Y axis implicitly. With Recharts 3, there is no default-id Y axis in that chart, so the grid lines can disappear; expected behavior is that the grid remains visible.
Suggested direction
Audit the Recharts v3 migration notes against the existing charts and set the grid’s axis id explicitly for the multi-axis chart so the visual contract survives the dependency bump.
Confidence note
I could not run a local build because pnpm is not installed in this environment; this is based on the official Recharts v3 migration note and the existing chart markup.
For Agents
In packages/node-ui/src/ui/pages/Operations.tsx, update the Network & RPC chart’s CartesianGrid to target an existing axis id, for example yAxisId="left", or add a default axis if that is the intended contract. Add/render a focused check that the multi-axis chart still emits grid lines under Recharts 3.
Major Recharts upgrade is not backed by chart rendering coverage
What's wrong
This changes the charting runtime from Recharts 2 to 3 for user-facing Observability charts, but the diff adds no validation that those charts still render or that the page’s Recharts-specific interactions still work. The existing Operations tests can pass even if the upgraded chart library produces no visible series or breaks legend toggling.
Example
A regression where Recharts v3 renders an empty ResponsiveContainer, drops SVG series paths, or stops Legend onClick from toggling hidden series would still leave the existing Operations e2e tests green because they only require surrounding labels and controls to be visible. A focused Playwright test could seed/mock fetchPerTypeStats with two buckets, open Observability, assert svg.recharts-surface and at least one rendered bar/area path are visible, then click a legend item and assert that series is hidden/dimmed.
Suggested direction
Add or extend an Operations UI/e2e test that forces chart data to be present and verifies the user-visible chart output and the existing legend interaction, rather than only checking that the Observability page shell loads.
For Agents
Look at packages/node-ui/src/ui/pages/Operations.tsx Recharts usages and packages/node-ui/e2e/specs/operations.spec.ts. Add coverage that supplies enough operation and hardware history to render charts, then assert actual chart SVG/series output and legend toggle behavior survive the Recharts v3 upgrade.
1 participant
Bumps recharts from 2.15.4 to 3.9.0.
Release notes
Sourced from recharts's releases.
... (truncated)
Commits
8c1cce7v3.9.0 (#7490)e5fe9bbAdd custom fillOpacity on hover website example (#7489)ad1590fAdd test for Pie and Legend animation (#7488)337edbdAdd animate-by-scroll example and update docs (#7487)0063aa4fix: preserve single-value line dash gaps during animation (#7405)6dbe130Add new example where chart animates by scroll (#7484)a2c4171Manual animations on website (#7483)c88f5f3Simplify and export some of the new animation props (#7481)410cf5cRenaming some of the animation props (#7477)a1b89c6chore(deps-dev): bump undici from 6.26.0 to 6.27.0 (#7475)Maintainer changes
This version was pushed to npm by corkscreewe, a new releaser for recharts since your current version.
Install script changes
This version adds
preparescript that runs during installation. Review the package contents before updating.Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)