fix(nextjs): Check process.env for Spotlight config as fallback

The valueInjectionLoader may not run in all scenarios (e.g., Next.js dev
mode with Turbopack). Next.js already replaces process.env.NEXT_PUBLIC_*
at build time, so we can use this as a fallback when globalThis is not set.

Also removes debug logging that was added for CI debugging and fixes
prettier formatting issues.

Author: cursoragent

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

@@ -1,16 +1,10 @@
 import * as Sentry from '@sentry/nextjs';
 
-// Debug: Log what the valueInjectionLoader injected (if anything)
-// The loader should inject globalThis["NEXT_PUBLIC_SENTRY_SPOTLIGHT"] = "true" before this code
-console.log('[Sentry Debug] globalThis.NEXT_PUBLIC_SENTRY_SPOTLIGHT:', (globalThis as Record<string, unknown>)['NEXT_PUBLIC_SENTRY_SPOTLIGHT']);
-console.log('[Sentry Debug] process.env.NEXT_PUBLIC_SENTRY_SPOTLIGHT:', process.env.NEXT_PUBLIC_SENTRY_SPOTLIGHT);
-
 Sentry.init({
   environment: 'qa',
   dsn: process.env.NEXT_PUBLIC_E2E_TEST_DSN,
   tunnel: `http://localhost:3031/`,
   tracesSampleRate: 1.0,
   // Note: We don't explicitly set spotlight here - it should be auto-enabled
-  // from NEXT_PUBLIC_SENTRY_SPOTLIGHT env var which is injected by the SDK's
-  // valueInjectionLoader to globalThis
+  // from NEXT_PUBLIC_SENTRY_SPOTLIGHT env var (replaced by Next.js at build time)
 });

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

@@ -2,39 +2,17 @@ import { expect, test } from '@playwright/test';
 
 test.describe('Spotlight auto-enablement in Next.js development mode', () => {
   test('Spotlight is automatically enabled when NEXT_PUBLIC_SENTRY_SPOTLIGHT=true', async ({ page }) => {
-    // Capture console logs for debugging
-    const consoleLogs: string[] = [];
-    page.on('console', msg => {
-      const text = msg.text();
-      consoleLogs.push(`[${msg.type()}] ${text}`);
-      // Print Sentry debug logs immediately
-      if (text.includes('[Sentry Debug]') || text.includes('Spotlight')) {
-        console.log(`BROWSER: ${text}`);
-      }
-    });
-
     await page.goto('/');
 
     // Wait for client-side hydration and Sentry initialization
     await page.waitForTimeout(3000);
 
-    // Print all console logs for debugging
-    console.log('=== Browser Console Logs ===');
-    consoleLogs.forEach(log => console.log(log));
-    console.log('=== End Console Logs ===');
-
-    // Check environment variable is accessible (injected at build time)
+    // Check environment variable is accessible (injected at build time by Next.js)
     const envValue = await page.getByTestId('env-value').textContent();
-    console.log('env-value content:', envValue);
     expect(envValue).toContain('true');
 
-    // Check globalThis value (set by valueInjectionLoader)
-    const globalThisValue = await page.getByTestId('globalthis-value').textContent();
-    console.log('globalthis-value content:', globalThisValue);
-
     // Check Spotlight integration is enabled
     const spotlightStatus = await page.getByTestId('spotlight-enabled').textContent();
-    console.log('spotlight-enabled content:', spotlightStatus);
     expect(spotlightStatus).toBe('ENABLED');
   });
 

packages/browser/test/build-artifacts.test.ts

@@ -70,9 +70,7 @@ describe('Build Artifacts - env.ts', () => {
         const content = fs.readFileSync(esmDevPath, 'utf-8');
 
         // Filter out comments
-        const lines = content
-          .split('\n')
-          .filter(line => !line.trim().startsWith('//') && !line.trim().startsWith('*'));
+        const lines = content.split('\n').filter(line => !line.trim().startsWith('//') && !line.trim().startsWith('*'));
         const codeOnly = lines.join('\n');
 
         // Should NOT contain import.meta in actual code
@@ -82,9 +80,7 @@ describe('Build Artifacts - env.ts', () => {
       if (fs.existsSync(esmProdPath)) {
         const content = fs.readFileSync(esmProdPath, 'utf-8');
 
-        const lines = content
-          .split('\n')
-          .filter(line => !line.trim().startsWith('//') && !line.trim().startsWith('*'));
+        const lines = content.split('\n').filter(line => !line.trim().startsWith('//') && !line.trim().startsWith('*'));
         const codeOnly = lines.join('\n');
 
         expect(codeOnly).not.toContain('import.meta.env');

packages/nextjs/src/client/index.ts

@@ -149,7 +149,12 @@ function getDefaultIntegrations(options: BrowserOptions): Integration[] {
   //
   // We handle this in the Next.js SDK rather than the browser SDK because the browser SDK's
   // auto-detection is development-only (stripped from production builds that users install).
-  const spotlightEnvValue = globalWithInjectedValues.NEXT_PUBLIC_SENTRY_SPOTLIGHT;
+  //
+  // Check both globalThis (valueInjectionLoader) and process.env (Next.js build-time replacement).
+  // globalThis is for Turbopack builds, process.env is replaced by Next.js at build time for
+  // NEXT_PUBLIC_* variables. We need both because valueInjectionLoader may not run in all scenarios.
+  const spotlightEnvValue =
+    globalWithInjectedValues.NEXT_PUBLIC_SENTRY_SPOTLIGHT ?? process.env.NEXT_PUBLIC_SENTRY_SPOTLIGHT;
 
   // Parse the env var value if present (could be 'true', 'false', or a URL)
   let envSpotlight: boolean | string | undefined;