chore(deps-dev): Bump the react-router group with 3 updates

packages/react-router/package.json

@@ -63,10 +63,10 @@
     "sentry": "^0.44.0"
   },
   "devDependencies": {
-    "@react-router/dev": "^7.17.0",
-    "@react-router/node": "^7.15.0",
+    "@react-router/dev": "^8.3.1",
+    "@react-router/node": "^8.3.1",
     "react": "^18.3.1",
-    "react-router": "^7.18.0",
+    "react-router": "^8.3.1",
     "vite": "^6.4.3"
   },
   "peerDependencies": {

packages/react-router/src/server/createSentryHandleRequest.tsx

@@ -1,10 +1,10 @@
 import type { createReadableStreamFromReadable } from '@react-router/node';
 import type { ReactNode } from 'react';
 import React from 'react';
-import type { AppLoadContext, EntryContext, RouterContextProvider, ServerRouter } from 'react-router';
+import type { EntryContext, RouterContextProvider, ServerRouter } from 'react-router';
 import { PassThrough } from 'stream';
 import { getMetaTagTransformer } from './getMetaTagTransformer';
-import { wrapSentryHandleRequest } from './wrapSentryHandleRequest';
+import { type AppLoadContext, wrapSentryHandleRequest } from './wrapSentryHandleRequest';
 
 type RenderToPipeableStreamOptions = {
   [key: string]: unknown;

packages/react-router/src/server/wrapSentryHandleRequest.ts

@@ -7,15 +7,26 @@ import {
   updateSpanName,
 } from '@sentry/core';
 import { flushIfServerless } from '@sentry/core/server';
-import type { AppLoadContext, EntryContext, RouterContextProvider } from 'react-router';
+import type { EntryContext, RouterContextProvider } from 'react-router';
 import { isInstrumentationApiUsed } from './serverGlobals';
 
-type OriginalHandleRequestWithoutMiddleware = (
+/**
+ * React Router v7's `AppLoadContext`, declared here because v8 removed the export: middleware is
+ * always enabled there, so the load context is always a `RouterContextProvider`. The SDK supports
+ * both majors, so the shape is mirrored instead of imported.
+ */
+export interface AppLoadContext {
+  [key: string]: unknown;
+}
+
+// Generic over the load context so apps that augment `AppLoadContext` via declaration merging keep
+// their own shape instead of being widened to the base index signature.
+type OriginalHandleRequestWithoutMiddleware<LoadContext extends AppLoadContext = AppLoadContext> = (
   request: Request,
   responseStatusCode: number,
   responseHeaders: Headers,
   routerContext: EntryContext,
-  loadContext: AppLoadContext,
+  loadContext: LoadContext,
 ) => Promise<unknown>;
 
 type OriginalHandleRequestWithMiddleware = (
@@ -32,9 +43,9 @@ type OriginalHandleRequestWithMiddleware = (
  * @param originalHandle - The original handleRequest function to wrap
  * @returns A wrapped version of the handle request function with Sentry instrumentation
  */
-export function wrapSentryHandleRequest(
-  originalHandle: OriginalHandleRequestWithoutMiddleware,
-): OriginalHandleRequestWithoutMiddleware;
+export function wrapSentryHandleRequest<LoadContext extends AppLoadContext>(
+  originalHandle: OriginalHandleRequestWithoutMiddleware<LoadContext>,
+): OriginalHandleRequestWithoutMiddleware<LoadContext>;
 /**
  * Wraps the original handleRequest function to add Sentry instrumentation.
  *

packages/react-router/test/server/createSentryHandleRequest.test.ts

@@ -48,11 +48,7 @@ describe('createSentryHandleRequest', () => {
       manifestPath: '/path/to/manifest',
     },
     routeModules: {},
-    future: {
-      v8_middleware: false,
-      v8_passThroughRequests: false,
-      v8_trailingSlashAwareDataRequests: false,
-    },
+    future: {},
     isSpaMode: false,
     branches: [],
     staticHandlerContext: {