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

This reverts commit 939e0cd.

Author: BYK

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

@@ -1,13 +1,8 @@
 import * as Sentry from '@sentry/nextjs';
 
-// 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.
+// 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.
 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,7 +5,6 @@
   "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,10 +1,7 @@
 import { getPlaywrightConfig } from '@sentry-internal/test-utils';
 
 const config = getPlaywrightConfig({
-  // 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.
+  // Use next dev to test development-mode behavior where Spotlight is auto-enabled
   startCommand: 'pnpm dev',
   port: 3030,
   eventProxyFile: 'start-event-proxy.mjs',

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

@@ -82,30 +82,6 @@ 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({
@@ -114,7 +90,9 @@ export function getPlaywrightConfig(
       stdout: 'pipe',
       stderr: 'pipe',
       env: {
-        ...filteredEnv,
+        // 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,
         PORT: appPort.toString(),
       },
     });