fix(react-router): preserve sourcemaps.disable override (#20042)

Ensure unstable vite plugin options don't overwrite sourcemaps.disable=true and update test expectations accordingly.

Co-Authored-By: gpt-5.3-codex-high <noreply@anthropic.com>

Author: 2 people

packages/react-router/src/vite/makeCustomSentryVitePlugins.ts

@@ -19,6 +19,7 @@ export async function makeCustomSentryVitePlugins(options: SentryReactRouterBuil
   } = options;
 
   const sentryVitePlugins = sentryVitePlugin({
+    ...unstable_sentryVitePluginOptions,
     authToken: authToken ?? process.env.SENTRY_AUTH_TOKEN,
     bundleSizeOptimizations,
     debug: debug ?? false,
@@ -27,25 +28,25 @@ export async function makeCustomSentryVitePlugins(options: SentryReactRouterBuil
     telemetry: telemetry ?? true,
     _metaOptions: {
       telemetry: {
+        ...unstable_sentryVitePluginOptions?._metaOptions?.telemetry,
         metaFramework: 'react-router',
       },
       ...unstable_sentryVitePluginOptions?._metaOptions,
     },
     reactComponentAnnotation: {
+      ...unstable_sentryVitePluginOptions?.reactComponentAnnotation,
       enabled: reactComponentAnnotation?.enabled ?? undefined,
       ignoredComponents: reactComponentAnnotation?.ignoredComponents ?? undefined,
-      ...unstable_sentryVitePluginOptions?.reactComponentAnnotation,
     },
     release: {
       ...unstable_sentryVitePluginOptions?.release,
       ...release,
     },
     // will be handled in buildEnd hook
     sourcemaps: {
-      disable: true,
       ...unstable_sentryVitePluginOptions?.sourcemaps,
+      disable: true,
     },
-    ...unstable_sentryVitePluginOptions,
   }) as Plugin[];
 
   return sentryVitePlugins;

packages/react-router/test/vite/makeCustomSentryVitePlugins.test.ts

@@ -66,7 +66,7 @@ describe('makeCustomSentryVitePlugins', () => {
     );
   });
 
-  it('should allow overriding sourcemaps via unstable_sentryVitePluginOptions', async () => {
+  it('should merge sourcemaps options from unstable_sentryVitePluginOptions', async () => {
     await makeCustomSentryVitePlugins({
       unstable_sentryVitePluginOptions: {
         sourcemaps: {
@@ -75,12 +75,12 @@ describe('makeCustomSentryVitePlugins', () => {
       },
     });
 
-    // unstable_sentryVitePluginOptions is spread last, so it fully overrides sourcemaps
     expect(sentryVitePlugin).toHaveBeenCalledWith(
       expect.objectContaining({
-        sourcemaps: {
+        sourcemaps: expect.objectContaining({
+          disable: true,
           assets: ['dist/**'],
-        },
+        }),
       }),
     );
   });