Skip to content

Conversation

s1gr1d
Copy link
Member

@s1gr1d s1gr1d commented Sep 26, 2024

Allows to pass other options from the bundler plugins (vite and rollup).

closes #13380

@s1gr1d s1gr1d requested review from Lms24 and lforst and removed request for lforst September 26, 2024 11:12
@darthf1
Copy link

darthf1 commented Sep 26, 2024

Why unstable? :) Do you expect any changes in the near future for this option?

@s1gr1d
Copy link
Member Author

s1gr1d commented Sep 26, 2024

@darthf1 We use this in e.g. Next.js or SolidStart to add the possibility to modify the underlying plugin options. It's an escape hatch and the underlying options could change if we upgrade the version of the bundler plugins in the Nuxt SDK.

*
* Please note that this option is unstable and may change in a breaking way in any release.
*/
unstable_sentryBundlerPluginOptions?: SentryRollupPluginOptions & SentryVitePluginOptions;
Copy link
Member Author

Choose a reason for hiding this comment

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

I used Options from @sentry/bundler-plugin-core first (as this is the type the other plugins inherit from) but this gave me the following build error:

 ERROR  The keyword 'interface' is reserved (4:0) in /[folders...]/sentry-javascript/node_modules/@sentry/bundler-plugin-core/dist/types/index.d.ts

The Options are defined as interface, but Rollup defines interface as reserved name. I think this is related but I couldn't figure out how to fix that.

@s1gr1d s1gr1d self-assigned this Sep 26, 2024
@s1gr1d s1gr1d added the Package: nuxt Issues related to the Sentry Nuxt SDK label Sep 26, 2024
@s1gr1d s1gr1d merged commit 8f3e5b2 into develop Oct 2, 2024
116 checks passed
@s1gr1d s1gr1d deleted the sig/nuxt-plugin-opts branch October 2, 2024 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Package: nuxt Issues related to the Sentry Nuxt SDK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Nuxt] Cannot set release name used by sentry-vite-plugin

3 participants