ref: remove the middleware, config must be statically analyzable

Author: logaretm

packages/nextjs/src/common/index.ts

@@ -12,4 +12,3 @@ export { wrapPageComponentWithSentry } from './pages-router-instrumentation/wrap
 export { wrapGenerationFunctionWithSentry } from './wrapGenerationFunctionWithSentry';
 export { withServerActionInstrumentation } from './withServerActionInstrumentation';
 export { captureRequestError } from './captureRequestError';
-export { withSentryMiddlewareConfig, withSentryProxyConfig } from './withSentryMiddlewareConfig';

packages/nextjs/src/common/withSentryMiddlewareConfig.ts

@@ -93,7 +93,7 @@ export function withSentryConfig<C>(nextConfig?: C, sentryBuildOptions: SentryBu
 
 /**
  * Checks if the user has a middleware/proxy file with a matcher that might exclude the tunnel route.
- * Warns the user if they have a matcher but are not using withSentryMiddlewareConfig or withSentryProxyConfig.
+ * Warns the user if their matcher might interfere with the tunnel route.
  */
 function checkMiddlewareMatcherForTunnelRoute(tunnelPath: string): void {
   if (showedMiddlewareMatcherWarning) {
@@ -108,29 +108,20 @@ function checkMiddlewareMatcherForTunnelRoute(tunnelPath: string): void {
       return;
     }
 
-    // Check if they're already using Sentry middleware/proxy config helpers
-    if (
-      middlewareFile.contents.includes('withSentryMiddlewareConfig') ||
-      middlewareFile.contents.includes('withSentryProxyConfig')
-    ) {
-      return;
-    }
-
     // Look for config.matcher export
     const isProxy = middlewareFile.path.includes('proxy');
     const hasConfigMatcher = /export\s+const\s+config\s*=\s*{[^}]*matcher\s*:/s.test(middlewareFile.contents);
 
     if (hasConfigMatcher) {
-      const helperName = isProxy ? 'withSentryProxyConfig' : 'withSentryMiddlewareConfig';
       // eslint-disable-next-line no-console
       console.warn(
         `[@sentry/nextjs] WARNING: You have a ${isProxy ? 'proxy' : 'middleware'} file (${path.basename(middlewareFile.path)}) with a \`config.matcher\`. ` +
-          `If your matcher does not include the Sentry tunnel route (${tunnelPath}), tunnel requests may be blocked. ` +
-          `To ensure your matcher doesn't interfere with Sentry event delivery, wrap your config with \`${helperName}\`:\n\n` +
-          `  import { ${helperName} } from '@sentry/nextjs';\n` +
-          `  export const config = ${helperName}({\n` +
-          "    matcher: ['/your/routes'],\n" +
-          '  });\n',
+          `If your matcher runs on the Sentry tunnel route (${tunnelPath}), it may interfere with event delivery. ` +
+          'Please ensure your matcher excludes the tunnel route. For example:\n\n' +
+          '  export const config = {\n' +
+          '    // Use a negative lookahead to exclude the Sentry tunnel route\n' +
+          `    matcher: '/((?!${tunnelPath.replace(/^\//, '')}|_next/static|_next/image|favicon.ico).*)',\n` +
+          '  };\n',
       );
       showedMiddlewareMatcherWarning = true;
     }