Merge pull request #19890 from getsentry/fix/react-router-debug-id-double-injection

fix(react-router): Disable debug ID injection in Vite plugin to prevent double injection

Author: isaacs

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

@@ -42,7 +42,7 @@ export async function makeCustomSentryVitePlugins(options: SentryReactRouterBuil
     },
     // will be handled in buildEnd hook
     sourcemaps: {
-      disable: 'disable-upload',
+      disable: true,
       ...unstable_sentryVitePluginOptions?.sourcemaps,
     },
     ...unstable_sentryVitePluginOptions,

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

@@ -54,13 +54,13 @@ describe('makeCustomSentryVitePlugins', () => {
     expect(plugins?.[0]?.name).toBe('sentry-vite-plugin');
   });
 
-  it('should disable sourcemap upload with "disable-upload" by default', async () => {
+  it('should disable sourcemap upload by default', async () => {
     await makeCustomSentryVitePlugins({});
 
     expect(sentryVitePlugin).toHaveBeenCalledWith(
       expect.objectContaining({
         sourcemaps: expect.objectContaining({
-          disable: 'disable-upload',
+          disable: true,
         }),
       }),
     );