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

Set sourcemaps.disable to true (boolean) instead of 'disable-upload' (string)
in makeCustomSentryVitePlugins. The Rollup plugin checks disable !== true, so
the string value was not disabling debug ID injection. This caused double
injection with two different UUIDs per file when sentryOnBuildEnd also ran
sentry-cli sourcemaps inject, breaking source map resolution.

Fixes GH-19874
Co-Authored-By: Claude <noreply@anthropic.com>

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,
         }),
       }),
     );