fix: stale onError in memo/forwardRef on React 19.2, getConnectedDataURL() on ECharts 6.1

[chensid]

Summary

This PR comes from re-reading upstream docs and sources (React, ECharts, Vite+/Vitest, TypeScript 7, pnpm). Two of the findings are real bugs: each one was reproduced before fixing and verified after. The PR also corrects docs and config.

1. onError goes stale inside memo() / forwardRef on React 19.2 (patch)

• Cause: React 19.2.x only refreshes useEffectEvent callbacks on plain function-component fibers. ForwardRef and SimpleMemo fibers break without updating them (react-dom 19.2.7, commit phase).
  • React 19.3.0 fixed this in [Fiber] Ensure useEffectEvent reads latest values in forwardRef and memo() Components react/react#34831, but the fix was never backported to 19.2.x.
• Impact: the peer range is react ^19.2.0. So with memo(EChart), or a memoized component that calls useEcharts, effect-time errors kept going to the onError from the first render.
• Fix: effect-time errors now read the latest onError from a ref that is synced in a layout effect.
  • useChartCore reuses latestRef; useResizeObserver gets its own ref.
  • Both report through a shared reportEffectError.
  • The library no longer uses useEffectEvent, so the docs no longer cite it as the reason for the 19.2 floor. The peer range itself is unchanged.
• Tests: new tests render the hook inside memo() and forwardRef, swap onError, then make option sync, auto-resize and cleanup fail.
  • With react/react-dom temporarily at 19.2.7, both tests fail on the old code and pass with the fix.
  • The full unit project (282 tests) also passes on 19.2.7.

2. getConnectedDataURL() without options throws on ECharts 6.1.0 (patch)

• Cause: ECharts 6.1.0 reads opts.type without defaulting opts. getDataURL does have a default.
  • The upstream fix (fix(core): makes the opts argument of echartsInstance.getConnectedDataURL optional apache/echarts#21736) is on the release branch only; it hasn't been published to npm.
• Fix: the hook now passes opts ?? {}.
• Tests: a new browser test calls getDataURL() and getConnectedDataURL() without options on a real chart.
  • Before the fix it fails with Cannot read properties of undefined (reading 'type'); after the fix it passes.

3. Docs and config corrections (no behaviour change)

• CLAUDE.md, pnpm prune: it said minimumReleaseAgeExcludePrune "does not reliably remove matured entries". The pnpm docs define it as dropping only entries the lockfile no longer resolves; it never prunes by age, so removing matured entries by hand is the designed workflow. The pnpm-workspace.yaml comment is corrected the same way (the prune also runs on install and dedupe).
• CLAUDE.md, Node: notes that the pre-commit vp staged needs Node 22.22.1+ on 22.x, matching CONTRIBUTING.md and the Vite+ commit-hooks guide.
• Removed redundant config:
  • .vitest-attachments/ in .gitignore: Vitest 5 writes to .vitest/, which is already ignored.
  • clearMocks: true and both inline extends: true in vite.config.ts: these are Vitest 5 defaults, checked in the bundled sources.
  • DOM.Iterable in tsconfig.app.json: it is an empty stub in TypeScript 7.
• Kept peerDependencyRules.allowedVersions: vp migrate rewrites it on every toolchain upgrade, so removing it would only cause churn.

Validation

• vp check and vp exec tsc -b pass.
• Unit tests with coverage: 282 passed, 100% on all metrics.
• Browser project: 9 passed.
• vp pack: attw and publint are clean; there are still two React Compiler caches.
• vp build and size-limit pass (6.88 KB / 1.51 KB / 227 B).

🤖 Generated with Claude Code

https://claude.ai/code/session_01VAs1BuCUj8VWLQvBSaxqWM

---

Generated by Claude Code

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (52a5667) to head (2eb7a93).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #577   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           19        19           
  Lines          574       574           
  Branches       146       143    -3     
=========================================
  Hits           574       574           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.