feat(core,cloudflare): Enable certain fields with env variables

packages/cloudflare/src/handler.ts

@@ -38,7 +38,7 @@ export function withSentry<
     QueueHandlerMessage,
     CfHostMetadata
   >,
->(optionsCallback: (env: Env) => CloudflareOptions, handler: T): T {
+>(optionsCallback: (env: Env) => CloudflareOptions | undefined, handler: T): T {
   setAsyncLocalStorageAsyncContextStrategy();
 
   try {

packages/cloudflare/src/options.ts

@@ -1,3 +1,4 @@
+import { envToBool } from '@sentry/core';
 import type { CloudflareOptions } from './client';
 
 /**
@@ -17,6 +18,12 @@ function isVersionMetadata(value: unknown): value is CfVersionMetadata {
   return typeof value === 'object' && value !== null && 'id' in value && typeof value.id === 'string';
 }
 
+function getEnvVar<T extends Record<string, unknown>>(env: unknown, varName: keyof T): string | undefined {
+  return typeof env === 'object' && env !== null && varName in env && typeof (env as T)[varName] === 'string'
+    ? ((env as T)[varName] as string)
+    : undefined;
+}
+
 /**
  * Merges the options passed in from the user with the options we read from
  * the Cloudflare `env` environment variable object.
@@ -31,7 +38,7 @@ function isVersionMetadata(value: unknown): value is CfVersionMetadata {
  *
  * @returns The final options.
  */
-export function getFinalOptions(userOptions: CloudflareOptions, env: unknown): CloudflareOptions {
+export function getFinalOptions(userOptions: CloudflareOptions = {}, env: unknown): CloudflareOptions {
   if (typeof env !== 'object' || env === null) {
     return userOptions;
   }
@@ -44,5 +51,15 @@ export function getFinalOptions(userOptions: CloudflareOptions, env: unknown): C
         ? env.CF_VERSION_METADATA.id
         : undefined;
 
-  return { release, ...userOptions };
+  const tracesSampleRate = userOptions.tracesSampleRate ?? parseFloat(getEnvVar(env, 'SENTRY_TRACE_SAMPLE_RATE') ?? '');
+
+  return {
+    release,
+    ...userOptions,
+    dsn: userOptions.dsn ?? getEnvVar(env, 'SENTRY_DSN'),
+    environment: userOptions.environment ?? getEnvVar(env, 'SENTRY_ENVIRONMENT'),
+    tracesSampleRate: isFinite(tracesSampleRate) ? tracesSampleRate : undefined,
+    debug: userOptions.debug ?? envToBool(getEnvVar(env, 'SENTRY_DEBUG')),
+    tunnel: userOptions.tunnel ?? getEnvVar(env, 'SENTRY_TUNNEL'),
+  };
 }

packages/cloudflare/test/options.test.ts

@@ -56,6 +56,60 @@ describe('getFinalOptions', () => {
     expect(result).toEqual(userOptions);
   });
 
+  it('uses SENTRY_DSN from env when user dsn is undefined', () => {
+    const userOptions = { release: 'user-release' };
+    const env = { SENTRY_DSN: 'https://key@ingest.sentry.io/1' };
+
+    const result = getFinalOptions(userOptions, env);
+
+    expect(result.dsn).toBe('https://key@ingest.sentry.io/1');
+    expect(result.release).toBe('user-release');
+  });
+
+  it('uses SENTRY_ENVIRONMENT from env when user environment is undefined', () => {
+    const userOptions = { dsn: 'test-dsn' };
+    const env = { SENTRY_ENVIRONMENT: 'staging' };
+
+    const result = getFinalOptions(userOptions, env);
+
+    expect(result.environment).toBe('staging');
+  });
+
+  it('uses SENTRY_TRACES_SAMPLE_RATE from env when user tracesSampleRate is undefined', () => {
+    const userOptions = { dsn: 'test-dsn' };
+    const env = { SENTRY_TRACES_SAMPLE_RATE: '0.5' };
+
+    const result = getFinalOptions(userOptions, env);
+
+    expect(result.tracesSampleRate).toBe(0.5);
+  });
+
+  it('does not use SENTRY_TRACES_SAMPLE_RATE from env when it is gibberish', () => {
+    const env = { SENTRY_TRACES_SAMPLE_RATE: 'ʕっ•ᴥ•ʔっ' };
+
+    const result = getFinalOptions(undefined, env);
+
+    expect(result.tracesSampleRate).toBeUndefined();
+  });
+
+  it('prefers user dsn over env SENTRY_DSN', () => {
+    const userOptions = { dsn: 'user-dsn', release: 'user-release' };
+    const env = { SENTRY_DSN: 'https://env@ingest.sentry.io/1' };
+
+    const result = getFinalOptions(userOptions, env);
+
+    expect(result.dsn).toBe('user-dsn');
+  });
+
+  it('ignores SENTRY_DSN when not a string', () => {
+    const userOptions = { dsn: undefined };
+    const env = { SENTRY_DSN: 123 };
+
+    const result = getFinalOptions(userOptions, env);
+
+    expect(result.dsn).toBeUndefined();
+  });
+
   describe('CF_VERSION_METADATA', () => {
     it('uses CF_VERSION_METADATA.id as release when no other release is set', () => {
       const userOptions = { dsn: 'test-dsn' };

packages/core/src/index.ts

@@ -62,6 +62,7 @@ export {
   _INTERNAL_shouldSkipAiProviderWrapping,
   _INTERNAL_clearAiProviderSkips,
 } from './utils/ai/providerSkip';
+export { envToBool } from './utils/envToBool';
 export { applyScopeDataToEvent, mergeScopeData, getCombinedScopeData } from './utils/scopeData';
 export { prepareEvent } from './utils/prepareEvent';
 export type { ExclusiveEventHintOrCaptureContext } from './utils/prepareEvent';

packages/node-core/test/utils/envToBool.test.ts → packages/core/test/lib/utils/envToBool.test.ts

@@ -1,5 +1,5 @@
 import { describe, expect, it } from 'vitest';
-import { envToBool } from '../../src/utils/envToBool';
+import { envToBool } from '../../../src/utils/envToBool';
 
 describe('envToBool', () => {
   it.each([

packages/node-core/src/common-exports.ts

@@ -31,7 +31,6 @@ export { getRequestUrl } from './utils/getRequestUrl';
 export { initializeEsmLoader } from './sdk/esmLoader';
 export { isCjs } from './utils/detection';
 export { createMissingInstrumentationContext } from './utils/createMissingInstrumentationContext';
-export { envToBool } from './utils/envToBool';
 export { makeNodeTransport, type NodeTransportOptions } from './transports';
 export type { HTTPModuleRequestIncomingMessage } from './transports/http-module';
 export { cron } from './cron';
@@ -117,6 +116,7 @@ export {
   wrapMcpServerWithSentry,
   featureFlagsIntegration,
   metrics,
+  envToBool,
 } from '@sentry/core';
 
 export type {

packages/node-core/src/light/sdk.ts

@@ -4,6 +4,7 @@ import {
   consoleIntegration,
   consoleSandbox,
   debug,
+  envToBool,
   eventFiltersIntegration,
   functionToStringIntegration,
   getCurrentScope,
@@ -29,7 +30,6 @@ import { defaultStackParser, getSentryRelease } from '../sdk/api';
 import { makeNodeTransport } from '../transports';
 import type { NodeClientOptions, NodeOptions } from '../types';
 import { isCjs } from '../utils/detection';
-import { envToBool } from '../utils/envToBool';
 import { getSpotlightConfig } from '../utils/spotlight';
 import { setAsyncLocalStorageAsyncContextStrategy } from './asyncLocalStorageStrategy';
 import { LightNodeClient } from './client';

packages/node-core/src/sdk/index.ts

@@ -5,6 +5,7 @@ import {
   consoleSandbox,
   conversationIdIntegration,
   debug,
+  envToBool,
   functionToStringIntegration,
   getCurrentScope,
   getIntegrationsToSetup,
@@ -38,7 +39,6 @@ import { systemErrorIntegration } from '../integrations/systemError';
 import { makeNodeTransport } from '../transports';
 import type { NodeClientOptions, NodeOptions } from '../types';
 import { isCjs } from '../utils/detection';
-import { envToBool } from '../utils/envToBool';
 import { getSpotlightConfig } from '../utils/spotlight';
 import { defaultStackParser, getSentryRelease } from './api';
 import { NodeClient } from './client';

packages/node-core/src/utils/spotlight.ts

@@ -1,4 +1,4 @@
-import { envToBool } from './envToBool';
+import { envToBool } from '@sentry/core';
 
 /**
  * Parse the spotlight option with proper precedence: