prob not

Author: BYK

packages/nextjs/src/client/index.ts

@@ -43,7 +43,6 @@ const globalWithInjectedValues = GLOBAL_OBJ as typeof GLOBAL_OBJ & {
   _sentryBasePath?: string;
   _sentryRelease?: string;
   _experimentalThirdPartyOriginStackFrames?: string;
-  _sentrySpotlight?: string;
 };
 
 // Treeshakable guard to remove all code related to tracing
@@ -144,9 +143,11 @@ function getDefaultIntegrations(options: BrowserOptions): Integration[] {
   );
 
   // Auto-enable Spotlight from NEXT_PUBLIC_SENTRY_SPOTLIGHT env var
-  // The value is injected at build time via buildTimeVariables in withSentryConfig
-  // following the same pattern as _sentryRelease
-  const spotlightEnvValue = process.env._sentrySpotlight || globalWithInjectedValues._sentrySpotlight;
+  // We read NEXT_PUBLIC_SENTRY_SPOTLIGHT directly because Next.js guarantees that
+  // NEXT_PUBLIC_* variables are exposed to the browser via webpack DefinePlugin replacement.
+  // Using a custom _sentrySpotlight variable doesn't work reliably because Next.js doesn't
+  // always replace arbitrary env vars in node_modules code.
+  const spotlightEnvValue = process.env.NEXT_PUBLIC_SENTRY_SPOTLIGHT;
   if (spotlightEnvValue !== undefined && options.spotlight === undefined) {
     const boolValue = envToBool(spotlightEnvValue, { strict: true });
     const spotlightConfig = boolValue !== null ? boolValue : spotlightEnvValue;

packages/nextjs/src/config/withSentryConfig.ts

@@ -610,12 +610,9 @@ function setUpBuildTimeVariables(
     buildTimeVariables._sentryRelease = releaseName;
   }
 
-  // Inject Spotlight config from NEXT_PUBLIC_SENTRY_SPOTLIGHT env var
-  // We check both userNextConfig.env (from next.config.js) and process.env (from .env files or shell)
-  const spotlightValue = userNextConfig.env?.NEXT_PUBLIC_SENTRY_SPOTLIGHT ?? process.env.NEXT_PUBLIC_SENTRY_SPOTLIGHT;
-  if (spotlightValue) {
-    buildTimeVariables._sentrySpotlight = spotlightValue;
-  }
+  // Note: NEXT_PUBLIC_SENTRY_SPOTLIGHT is read directly in the client code because
+  // Next.js guarantees that NEXT_PUBLIC_* variables are exposed to the browser.
+  // No need to inject it as a build-time variable here.
 
   if (typeof userNextConfig.env === 'object') {
     userNextConfig.env = { ...buildTimeVariables, ...userNextConfig.env };