-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
feat(astro): Align options with shared build time options type #17396
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Conversation
size-limit report 📦
|
bundleSizeOptimizations, | ||
unstable_sentryVitePluginOptions, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Sentry Vite Plugin Misinterprets Configuration Options
New top-level build-time options (e.g., org
, project
, authToken
, sentryUrl
, headers
, telemetry
, silent
, errorHandler
) are not destructured from the options
object. This causes them to be incorrectly included in otherOptions
, triggering a deprecation warning for 'additional options'. As a result, misleading warnings appear in production builds, despite these options being valid and used for Sentry Vite plugin configuration.
*/ | ||
bundleSizeOptimizations?: BundleSizeOptimizationOptions; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Am I confused or did this disappear with the changes? 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is part of the shared type so it's possible to delete it here :)
bundleSizeOptimizations?: BundleSizeOptimizationsOptions; |
As Astro currently mixes build-time and runtime options, the
release
option was omitted from the type. It's only possible to set this option with theunstable_sentryVitePluginOptions
:(Closes #17067