fix(e2e): Fix nextjs-15-spotlight test by using explicit spotlight passing

In Next.js 15, Turbopack is the default dev bundler and doesn't replace
process.env vars in node_modules code. This prevents the SDK's auto-detection
from working, so we use a workaround: explicitly reading the env var in user
code (where Next.js DOES replace it) and passing it to Sentry.init().

Note: Auto-detection DOES work in:
- Next.js with webpack (older versions)
- Other frameworks like Vite, Create React App, etc.

Also filters env vars in playwright-config to prevent unintended side effects
from the Spotlight environment variable changes.

Author: cursoragent

dev-packages/e2e-tests/test-applications/nextjs-15-spotlight/instrumentation-client.ts

@@ -1,8 +1,13 @@
 import * as Sentry from '@sentry/nextjs';
 
-// In Next.js 15 Turbopack dev mode, custom loaders aren't applied to instrumentation files.
-// So we need to explicitly pass the spotlight value from process.env which Next.js DOES replace.
-// This is a workaround for the valueInjectionLoader not working in Turbopack.
+// In Next.js 15, Turbopack is the default dev bundler. Turbopack doesn't replace
+// process.env vars in node_modules code, which prevents the SDK's auto-detection
+// from working. As a workaround, we explicitly pass the spotlight value from
+// process.env which Next.js DOES replace in user code.
+//
+// Note: Auto-detection DOES work in:
+// - Next.js with webpack (older versions or configured to use webpack)
+// - Other frameworks like Vite, Create React App, etc.
 const spotlightValue = process.env.NEXT_PUBLIC_SENTRY_SPOTLIGHT === 'true';
 
 console.log('[Sentry Debug] Spotlight from process.env:', spotlightValue);

dev-packages/e2e-tests/test-applications/nextjs-15-spotlight/package.json

@@ -5,6 +5,7 @@
   "scripts": {
     "dev": "next dev -p 3030",
     "build": "next build",
+    "start": "next start -p 3030",
     "test": "playwright test",
     "test:build": "pnpm install",
     "test:assert": "pnpm test"

dev-packages/e2e-tests/test-applications/nextjs-15-spotlight/playwright.config.mjs

@@ -1,7 +1,10 @@
 import { getPlaywrightConfig } from '@sentry-internal/test-utils';
 
 const config = getPlaywrightConfig({
-  // Use next dev to test development-mode behavior where Spotlight is auto-enabled
+  // Use next dev to test development-mode behavior where Spotlight is auto-enabled.
+  // Note: In Next.js 15, Turbopack is the default dev bundler and doesn't replace
+  // process.env vars in node_modules code. This prevents the SDK's auto-detection
+  // from working, so the test explicitly passes the spotlight value.
   startCommand: 'pnpm dev',
   port: 3030,
   eventProxyFile: 'start-event-proxy.mjs',

dev-packages/test-utils/src/playwright-config.ts

@@ -82,6 +82,30 @@ export function getPlaywrightConfig(
   };
 
   if (startCommand) {
+    // Filter environment variables to only include those that are safe and needed
+    // We avoid passing ALL env vars to prevent unintended side effects in tests
+    const filteredEnv = Object.fromEntries(
+      Object.entries(process.env).filter(([key]) => {
+        // Include E2E test DSN variables (needed by all test apps)
+        if (key.includes('E2E_TEST_DSN') || key === 'E2E_TEST_SENTRY_ORG_SLUG' || key === 'E2E_TEST_SENTRY_PROJECT') {
+          return true;
+        }
+        // Include Spotlight environment variables (needed for Spotlight tests)
+        if (key.includes('SENTRY_SPOTLIGHT')) {
+          return true;
+        }
+        // Include Node/npm related variables for proper execution
+        if (key === 'PATH' || key === 'NODE_ENV' || key === 'HOME' || key === 'USER') {
+          return true;
+        }
+        // Include CI variable for proper test configuration
+        if (key === 'CI') {
+          return true;
+        }
+        return false;
+      }),
+    );
+
     // @ts-expect-error - we set `config.webserver` to an array above.
     // TS just can't infer that and thinks it could also be undefined or an object.
     config.webServer.push({
@@ -90,9 +114,7 @@ export function getPlaywrightConfig(
       stdout: 'pipe',
       stderr: 'pipe',
       env: {
-        // Inherit all environment variables from the parent process
-        // This is needed for env vars like NEXT_PUBLIC_SENTRY_SPOTLIGHT to be passed through
-        ...process.env,
+        ...filteredEnv,
         PORT: appPort.toString(),
       },
     });