sentry-javascript icon indicating copy to clipboard operation
sentry-javascript copied to clipboard

feat(browser): Add environment variable support for Spotlight configuration

Open github-actions[bot] opened this issue 1 month ago • 1 comments

[!NOTE] The pull request "feat(browser): Add environment variable support for Spotlight configuration" was created by @BYK but did not reference an issue. Therefore this issue was created for better visibility in external tools like Linear.

Implements full Spotlight spec with support for multiple framework-specific environment variable prefixes. Adds defensive environment variable access for both process.env and import.meta.env to support various bundlers.

Supported environment variables (in priority order):

  • PUBLIC_SENTRY_SPOTLIGHT (SvelteKit, Astro, Qwik)
  • NEXT_PUBLIC_SENTRY_SPOTLIGHT (Next.js)
  • VITE_SENTRY_SPOTLIGHT (Vite)
  • NUXT_PUBLIC_SENTRY_SPOTLIGHT (Nuxt)
  • REACT_APP_SENTRY_SPOTLIGHT (Create React App)
  • VUE_APP_SENTRY_SPOTLIGHT (Vue CLI)
  • GATSBY_SENTRY_SPOTLIGHT (Gatsby)
  • SENTRY_SPOTLIGHT (base/official)

SENTRY_SPOTLIGHT is last as in environments like Docker Compose, we actually make the front-end env variable different than the base SENTRY_SPOTLIGHT one -- the backends need to reach docker.host.internal whereas front-ends always need localhost as we assume the browser runs on the same host with Spotlight.

Refactors envToBool utility from node-core to core package for shared usage. Adds resolveSpotlightOptions utility to ensure consistent precedence rules across Browser and Node SDKs.

Includes comprehensive test coverage for all new utilities and integration tests for environment variable precedence behavior.

github-actions[bot] avatar Dec 04 '25 12:12 github-actions[bot]

JS-1253

linear[bot] avatar Dec 04 '25 12:12 linear[bot]