Skip to content

test(e2e): Drop unused @sentry/bundler-plugins dependency from bundler apps - #25399

Merged
mydea merged 1 commit into
developfrom
test/e2e-drop-unused-bundler-plugins
Oct 15, 2026
Merged

mydea merged 1 commit into
developfrom
test/e2e-drop-unused-bundler-plugins

Conversation

@mydea

@mydea mydea commented Oct 15, 2026

Copy link
Copy Markdown
Member

node-esbuild, node-rolldown, node-rollup, node-vite and node-webpack declared @sentry/bundler-plugins as a direct dependency, but none of them imports it. They use the plugins re-exported from @sentry/node/<bundler>, and @sentry/node depends on @sentry/bundler-plugins itself. That's also how users install them.

Dropping the direct dependency means the apps get @sentry/bundler-plugins the same way users do. All five pass locally with the change.

🤖 Generated with Claude Code

…r apps

The node bundler apps import the plugins from `@sentry/node/<bundler>`, and
`@sentry/node` depends on `@sentry/bundler-plugins` itself.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@mydea
mydea marked this pull request as ready for review October 15, 2026 12:50
@mydea
mydea requested review from JPeer264, isaacs and timfish October 15, 2026 12:50
@mydea
mydea merged commit a8b06c1 into develop Oct 15, 2026
48 checks passed
@mydea
mydea deleted the test/e2e-drop-unused-bundler-plugins branch October 15, 2026 14:17
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.

2 participants