Skip to content

Conversation

@timfish
Copy link
Collaborator

@timfish timfish commented Feb 10, 2026

This PR removes unplugin as dependency for the esbuild plugin and instead just exports a regular esbuild plugin.

@github-actions
Copy link

github-actions bot commented Feb 10, 2026

Semver Impact of This PR

🟡 Minor (new features)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • Esbuild no longer uses unplugin by timfish in #871
  • Webpack no longer uses unplugin by timfish in #870

🤖 This preview updates automatically when you update the PR.

@timfish timfish force-pushed the timfish/feat/esbuild-no-unplugin branch from a2f60b7 to 8cafa7b Compare February 10, 2026 21:22
@timfish timfish marked this pull request as ready for review February 10, 2026 21:33
@timfish timfish requested review from Lms24 and chargome February 11, 2026 13:23
Copy link
Member

@chargome chargome left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

// eslint-disable-next-line @typescript-eslint/no-explicit-any
export default sentryUnplugin.esbuild as (options?: Options) => any;

export default sentryEsbuildPlugin as (options?: Options) => EsbuildPlugin;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this export a breaking change?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh yes!

@timfish timfish merged commit 62bfaa5 into getsentry:main Feb 12, 2026
25 checks passed
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