fix(metro): Add next parameter to metroMiddleware to satisfy HandleFunction type

packages/core/package.json

@@ -83,7 +83,7 @@
     "@sentry-internal/eslint-config-sdk": "10.30.0",
     "@sentry-internal/eslint-plugin-sdk": "10.30.0",
     "@sentry-internal/typescript": "10.30.0",
-    "@sentry/wizard": "6.9.0",
+    "@sentry/wizard": "6.10.0",
     "@testing-library/react-native": "^13.2.2",
     "@types/jest": "^29.5.13",
     "@types/node": "^20.9.3",

packages/core/src/js/tools/metroMiddleware.ts

@@ -14,6 +14,7 @@ const readFileAsync = promisify(readFile);
 export const stackFramesContextMiddleware: Middleware = async (
   request: IncomingMessage,
   response: ServerResponse,
+  _next: () => void,
 ): Promise<void> => {
   debug.log('[@sentry/react-native/metro] Received request for stack frames context.');
   request.setEncoding('utf8');
@@ -89,11 +90,15 @@ const SENTRY_CONTEXT_REQUEST_PATH = `${SENTRY_MIDDLEWARE_PATH}/context`;
  * Creates a middleware that adds source context to the Sentry formatted stack frames.
  */
 export const createSentryMetroMiddleware = (middleware: Middleware): Middleware => {
-  return (request: IncomingMessage, response: ServerResponse, next: unknown) => {
+  return (request: IncomingMessage, response: ServerResponse, next: () => void) => {
     if (request.url?.startsWith(SENTRY_CONTEXT_REQUEST_PATH)) {
-      return stackFramesContextMiddleware(request, response);
+      return stackFramesContextMiddleware(request, response, next);
     }
-    return middleware(request, response, next);
+    return (middleware as (req: IncomingMessage, res: ServerResponse, next: () => void) => void)(
+      request,
+      response,
+      next,
+    );
   };
 };
 

packages/core/test/tools/metroMiddleware.test.ts

@@ -78,7 +78,7 @@ describe('metroMiddleware', () => {
       } as any;
       testedMiddleware(sentryRequest, response, next);
       expect(defaultMiddleware).not.toHaveBeenCalled();
-      expect(spiedStackFramesContextMiddleware).toHaveBeenCalledWith(sentryRequest, response);
+      expect(spiedStackFramesContextMiddleware).toHaveBeenCalledWith(sentryRequest, response, next);
     });
 
     it('should call default middleware for non-sentry requests', () => {