Don't overwrite nested default options

Author: angelikatyborska

packages/astro/src/integration/index.ts

@@ -68,24 +68,26 @@ export const sentryAstro = (options: SentryOptions = {}): AstroIntegration => {
                     project: uploadOptions.project ?? env.SENTRY_PROJECT,
                     authToken: uploadOptions.authToken ?? env.SENTRY_AUTH_TOKEN,
                     telemetry: uploadOptions.telemetry ?? true,
+                    _metaOptions: {
+                      telemetry: {
+                        metaFramework: 'astro',
+                      },
+                    },
+                    ...unstable_sentryVitePluginOptions,
+                    debug: options.debug ?? false,
                     sourcemaps: {
                       assets: uploadOptions.assets ?? [getSourcemapsAssetsGlob(config)],
                       filesToDeleteAfterUpload:
                         uploadOptions?.filesToDeleteAfterUpload ?? updatedFilesToDeleteAfterUpload,
+                      ...unstable_sentryVitePluginOptions?.sourcemaps
                     },
                     bundleSizeOptimizations: {
                       ...options.bundleSizeOptimizations,
                       // TODO: with a future version of the vite plugin (probably 2.22.0) this re-mapping is not needed anymore
                       // ref: https://github.com/getsentry/sentry-javascript-bundler-plugins/pull/582
                       excludePerformanceMonitoring: options.bundleSizeOptimizations?.excludeTracing,
+                      ...unstable_sentryVitePluginOptions?.bundleSizeOptimizations
                     },
-                    _metaOptions: {
-                      telemetry: {
-                        metaFramework: 'astro',
-                      },
-                    },
-                    debug: options.debug ?? false,
-                    ...unstable_sentryVitePluginOptions
                   }),
                 ),
               ],

packages/astro/test/integration/index.test.ts

@@ -202,23 +202,27 @@ describe('sentryAstro integration', () => {
     );
   });
 
-  it('prefers user-specified unstable vite plugin options over everything else', async () => {
+  it('prefers user-specified unstable vite plugin options and merges them with default values', async () => {
     const integration = sentryAstro({
+      bundleSizeOptimizations: {
+        excludeReplayShadowDom: true
+      },
       sourceMapsUploadOptions: {
         enabled: true,
         org: 'my-org',
         project: 'my-project',
         assets: ['dist/server/**/*, dist/client/**/*'],
-        debug: true,
         unstable_sentryVitePluginOptions: {
           org: 'my-other-org',
           project: 'my-other-project',
           applicationKey: 'my-application-key',
-          debug: false,
           sourcemaps: {
             assets: ['foo/*.js'],
             ignore: ['bar/*.js'],
           },
+          bundleSizeOptimizations: {
+            excludeReplayIframe: true
+          }
         }
       },
     });
@@ -238,11 +242,15 @@ describe('sentryAstro integration', () => {
         org: 'my-other-org',
         project: 'my-other-project',
         applicationKey: 'my-application-key',
-        debug: false,
         sourcemaps: {
           assets: ['foo/*.js'],
           ignore: ['bar/*.js'],
+          filesToDeleteAfterUpload: ['./dist/**/client/**/*.map', './dist/**/server/**/*.map'],
         },
+        bundleSizeOptimizations: {
+          excludeReplayShadowDom: true,
+          excludeReplayIframe: true
+        }
       }),
     );
   });