Skip to content

Comments

perf: upgrade AG Grid v33 → v34#556

Closed
paddymul wants to merge 1 commit intoperf/ag-grid-v33from
perf/ag-grid-v34
Closed

perf: upgrade AG Grid v33 → v34#556
paddymul wants to merge 1 commit intoperf/ag-grid-v33from
perf/ag-grid-v34

Conversation

@paddymul
Copy link
Collaborator

Summary

  • Version bump only — v34 has zero breaking changes
  • ESM bundle: 1,722 KB → 1,773 KB (+3% code growth)

Test plan

  • tsc -b — 0 errors
  • pnpm test — 67/67 pass
  • vite build — success
  • CI checks pass

🤖 Generated with Claude Code

@github-actions
Copy link

github-actions bot commented Feb 24, 2026

📦 TestPyPI package published

pip install --index-strategy unsafe-best-match --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple/ buckaroo==0.12.9.dev22336319086

or with uv:

uv pip install --index-strategy unsafe-best-match --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple/ buckaroo==0.12.9.dev22336319086

MCP server for Claude Code

claude mcp add buckaroo-table -- uvx --from "buckaroo[mcp]==0.12.9.dev22336319086" --index-strategy unsafe-best-match --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple/ buckaroo-table

Zero breaking changes. Version bump only.
ESM bundle: 1,722 KB → 1,773 KB (+3%)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@paddymul
Copy link
Collaborator Author

Closing — v34 not needed

v33 gives us the best bundle size (-31% wheel, -61% widget.js) and v34 adds ~3% back with no features relevant to buckaroo:

  • Cell editor validation, batch/bulk editing — we're read-only
  • Tree data drag & drop, date grouping — not used
  • AI toolkit / MCP server — not applicable
  • React 19.2 support — not needed yet

No performance improvements for infinite scroll, pinned rows, or cell rendering.

Policy: don't bump AG Grid until we need a specific feature. Can always revisit if something in v34+ becomes relevant.

@paddymul paddymul closed this Feb 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant