fix(deno): Clear pre-existing OTel global before registering TracerProvider (#19723)

## Summary

- Calls `trace.disable()` before `trace.setGlobalTracerProvider()` in
`@sentry/deno`'s OTel tracer setup
- This fixes silent registration failure when Supabase Edge Runtime (or
Deno's native OTel) pre-registers a `TracerProvider` on the
`@opentelemetry/api` global (`Symbol.for('opentelemetry.js.api.1')`)
- Without this fix, **OTel-instrumented spans** (e.g. `gen_ai.*` from AI
SDK, or any library using `@opentelemetry/api`) never reach Sentry
because Sentry's `TracerProvider` fails to register as the global.
Sentry's own `startSpan()` API is unaffected since it bypasses the OTel
global.

## Context

Supabase Edge Runtime (Deno 2.1.4+) registers its own `TracerProvider`
before user code runs. The OTel API's `trace.setGlobalTracerProvider()`
is a no-op if a provider is already registered (it only logs a diag
warning), so Sentry's tracer silently gets ignored.

**What works without the fix:** `Sentry.startSpan()` — goes through
Sentry's internal pipeline, not the OTel global.

**What breaks without the fix:** Any spans created via
`@opentelemetry/api` (AI SDK's `gen_ai.*` spans, HTTP instrumentations,
etc.) — these hit the pre-existing Supabase provider instead of
Sentry's.

Calling `trace.disable()` clears the global, allowing
`trace.setGlobalTracerProvider()` to succeed. This matches the pattern
already used in `cleanupOtel()` in the test file and is safe because:
1. It only runs once during `Sentry.init()`
2. Any pre-existing provider is immediately replaced by Sentry's
3. It's gated behind `skipOpenTelemetrySetup` so users with custom OTel
setups can opt out
4. The Cloudflare package was investigated and doesn't have the same
issue

## Test plan

- [x] Updated `should override pre-existing OTel provider with Sentry
provider` unit test — simulates a pre-existing provider and verifies
Sentry overrides it
- [x] Updated `should override native Deno OpenTelemetry when enabled`
unit test — verifies Sentry captures spans even when `OTEL_DENO=true`
- [x] **E2E test app**
(`dev-packages/e2e-tests/test-applications/deno/`) — Deno server with
pre-existing OTel provider, 5 tests:
  - Error capture (`Sentry.captureException`)
  - `Sentry.startSpan` transaction
- OTel `tracer.startSpan` despite pre-existing provider (core regression
test)
  - OTel `tracer.startActiveSpan` (AI SDK pattern)
  - Sentry + OTel interop (OTel child inside Sentry parent)
- [x] Verified manually with Supabase Edge Function + AI SDK:
`Sentry.startSpan()` spans appeared in Sentry both before and after the
fix, but `gen_ai.*` OTel spans only appeared after the fix

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Closes #19724

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: sergical

.github/workflows/build.yml

@@ -969,6 +969,11 @@ jobs:
         with:
           use-installer: true
           token: ${{ secrets.GITHUB_TOKEN }}
+      - name: Set up Deno
+        if: matrix.test-application == 'deno'
+        uses: denoland/setup-deno@v2.0.3
+        with:
+          deno-version: v2.1.5
       - name: Restore caches
         uses: ./.github/actions/restore-cache
         with:

dev-packages/e2e-tests/test-applications/deno/.npmrc

@@ -0,0 +1,2 @@
+@sentry:registry=http://127.0.0.1:4873
+@sentry-internal:registry=http://127.0.0.1:4873

dev-packages/e2e-tests/test-applications/deno/deno.json

@@ -0,0 +1,8 @@
+{
+  "imports": {
+    "@sentry/deno": "npm:@sentry/deno",
+    "@sentry/core": "npm:@sentry/core",
+    "@opentelemetry/api": "npm:@opentelemetry/api@^1.9.0"
+  },
+  "nodeModulesDir": "manual"
+}

dev-packages/e2e-tests/test-applications/deno/package.json

@@ -0,0 +1,23 @@
+{
+  "name": "deno-app",
+  "version": "1.0.0",
+  "private": true,
+  "scripts": {
+    "start": "deno run --allow-net --allow-env --allow-read src/app.ts",
+    "test": "playwright test",
+    "clean": "npx rimraf node_modules pnpm-lock.yaml",
+    "test:build": "pnpm install",
+    "test:assert": "pnpm test"
+  },
+  "dependencies": {
+    "@sentry/deno": "latest || *",
+    "@opentelemetry/api": "^1.9.0"
+  },
+  "devDependencies": {
+    "@playwright/test": "~1.56.0",
+    "@sentry-internal/test-utils": "link:../../../test-utils"
+  },
+  "volta": {
+    "extends": "../../package.json"
+  }
+}

dev-packages/e2e-tests/test-applications/deno/playwright.config.mjs

@@ -0,0 +1,8 @@
+import { getPlaywrightConfig } from '@sentry-internal/test-utils';
+
+const config = getPlaywrightConfig({
+  startCommand: `pnpm start`,
+  port: 3030,
+});
+
+export default config;

dev-packages/e2e-tests/test-applications/deno/src/app.ts

@@ -0,0 +1,90 @@
+import { trace } from '@opentelemetry/api';
+
+// Simulate a pre-existing OTel provider (like Supabase Edge Runtime registers
+// before user code runs). Without trace.disable() in Sentry's setup, this would
+// cause setGlobalTracerProvider to be a no-op, silently dropping all OTel spans.
+const fakeProvider = {
+  getTracer: () => ({
+    startSpan: () => ({ end: () => {}, setAttributes: () => {} }),
+    startActiveSpan: (_name: string, fn: Function) => fn({ end: () => {}, setAttributes: () => {} }),
+  }),
+};
+trace.setGlobalTracerProvider(fakeProvider as any);
+
+// Sentry.init() must call trace.disable() to clear the fake provider above
+import * as Sentry from '@sentry/deno';
+
+Sentry.init({
+  environment: 'qa',
+  dsn: Deno.env.get('E2E_TEST_DSN'),
+  debug: !!Deno.env.get('DEBUG'),
+  tunnel: 'http://localhost:3031/',
+  tracesSampleRate: 1,
+});
+
+const port = 3030;
+
+Deno.serve({ port }, (req: Request) => {
+  const url = new URL(req.url);
+
+  if (url.pathname === '/test-success') {
+    return new Response(JSON.stringify({ version: 'v1' }), {
+      headers: { 'Content-Type': 'application/json' },
+    });
+  }
+
+  if (url.pathname === '/test-error') {
+    const exceptionId = Sentry.captureException(new Error('This is an error'));
+    return new Response(JSON.stringify({ exceptionId }), {
+      headers: { 'Content-Type': 'application/json' },
+    });
+  }
+
+  // Test Sentry.startSpan — uses Sentry's internal pipeline
+  if (url.pathname === '/test-sentry-span') {
+    Sentry.startSpan({ name: 'test-sentry-span' }, () => {
+      // noop
+    });
+    return new Response(JSON.stringify({ status: 'ok' }), {
+      headers: { 'Content-Type': 'application/json' },
+    });
+  }
+
+  // Test OTel tracer.startSpan — goes through the global TracerProvider
+  if (url.pathname === '/test-otel-span') {
+    const tracer = trace.getTracer('test-tracer');
+    const span = tracer.startSpan('test-otel-span');
+    span.end();
+    return new Response(JSON.stringify({ status: 'ok' }), {
+      headers: { 'Content-Type': 'application/json' },
+    });
+  }
+
+  // Test OTel tracer.startActiveSpan — what AI SDK and most instrumentations use
+  if (url.pathname === '/test-otel-active-span') {
+    const tracer = trace.getTracer('test-tracer');
+    tracer.startActiveSpan('test-otel-active-span', span => {
+      span.setAttributes({ 'test.active': true });
+      span.end();
+    });
+    return new Response(JSON.stringify({ status: 'ok' }), {
+      headers: { 'Content-Type': 'application/json' },
+    });
+  }
+
+  // Test interop: OTel span inside a Sentry span
+  if (url.pathname === '/test-interop') {
+    Sentry.startSpan({ name: 'sentry-parent' }, () => {
+      const tracer = trace.getTracer('test-tracer');
+      const span = tracer.startSpan('otel-child');
+      span.end();
+    });
+    return new Response(JSON.stringify({ status: 'ok' }), {
+      headers: { 'Content-Type': 'application/json' },
+    });
+  }
+
+  return new Response('Not found', { status: 404 });
+});
+
+console.log(`Deno test app listening on port ${port}`);

dev-packages/e2e-tests/test-applications/deno/start-event-proxy.mjs

@@ -0,0 +1,6 @@
+import { startEventProxyServer } from '@sentry-internal/test-utils';
+
+startEventProxyServer({
+  port: 3031,
+  proxyServerName: 'deno',
+});

dev-packages/e2e-tests/test-applications/deno/tests/errors.test.ts

@@ -0,0 +1,15 @@
+import { expect, test } from '@playwright/test';
+import { waitForError } from '@sentry-internal/test-utils';
+
+test('Sends error event', async ({ baseURL }) => {
+  const errorEventPromise = waitForError('deno', event => {
+    return !event.type && event.exception?.values?.[0]?.value === 'This is an error';
+  });
+
+  await fetch(`${baseURL}/test-error`);
+
+  const errorEvent = await errorEventPromise;
+
+  expect(errorEvent.exception?.values).toHaveLength(1);
+  expect(errorEvent.exception?.values?.[0]?.value).toBe('This is an error');
+});

dev-packages/e2e-tests/test-applications/deno/tests/transactions.test.ts

@@ -0,0 +1,90 @@
+import { expect, test } from '@playwright/test';
+import { waitForTransaction } from '@sentry-internal/test-utils';
+
+test('Sends transaction with Sentry.startSpan', async ({ baseURL }) => {
+  const transactionPromise = waitForTransaction('deno', event => {
+    return event?.spans?.some(span => span.description === 'test-sentry-span') ?? false;
+  });
+
+  await fetch(`${baseURL}/test-sentry-span`);
+
+  const transaction = await transactionPromise;
+
+  expect(transaction.spans).toEqual(
+    expect.arrayContaining([
+      expect.objectContaining({
+        description: 'test-sentry-span',
+        origin: 'manual',
+      }),
+    ]),
+  );
+});
+
+test('Sends transaction with OTel tracer.startSpan despite pre-existing provider', async ({ baseURL }) => {
+  const transactionPromise = waitForTransaction('deno', event => {
+    return event?.spans?.some(span => span.description === 'test-otel-span') ?? false;
+  });
+
+  await fetch(`${baseURL}/test-otel-span`);
+
+  const transaction = await transactionPromise;
+
+  expect(transaction.spans).toEqual(
+    expect.arrayContaining([
+      expect.objectContaining({
+        description: 'test-otel-span',
+        op: 'otel.span',
+        origin: 'manual',
+      }),
+    ]),
+  );
+});
+
+test('Sends transaction with OTel tracer.startActiveSpan', async ({ baseURL }) => {
+  const transactionPromise = waitForTransaction('deno', event => {
+    return event?.spans?.some(span => span.description === 'test-otel-active-span') ?? false;
+  });
+
+  await fetch(`${baseURL}/test-otel-active-span`);
+
+  const transaction = await transactionPromise;
+
+  expect(transaction.spans).toEqual(
+    expect.arrayContaining([
+      expect.objectContaining({
+        description: 'test-otel-active-span',
+        op: 'otel.span',
+        origin: 'manual',
+      }),
+    ]),
+  );
+});
+
+test('OTel span appears as child of Sentry span (interop)', async ({ baseURL }) => {
+  const transactionPromise = waitForTransaction('deno', event => {
+    return event?.spans?.some(span => span.description === 'sentry-parent') ?? false;
+  });
+
+  await fetch(`${baseURL}/test-interop`);
+
+  const transaction = await transactionPromise;
+
+  expect(transaction.spans).toEqual(
+    expect.arrayContaining([
+      expect.objectContaining({
+        description: 'sentry-parent',
+        origin: 'manual',
+      }),
+      expect.objectContaining({
+        description: 'otel-child',
+        op: 'otel.span',
+        origin: 'manual',
+      }),
+    ]),
+  );
+
+  // Verify the OTel span is a child of the Sentry span
+  const sentrySpan = transaction.spans!.find((s: any) => s.description === 'sentry-parent');
+  const otelSpan = transaction.spans!.find((s: any) => s.description === 'otel-child');
+  expect(otelSpan!.parent_span_id).toBe(sentrySpan!.span_id);
+});

packages/deno/src/opentelemetry/tracer.ts

@@ -12,6 +12,9 @@ import {
  * This is not perfect but handles easy/common use cases.
  */
 export function setupOpenTelemetryTracer(): void {
+  // Clear any pre-existing OTel global registration (e.g. from Supabase Edge Runtime
+  // or Deno's built-in OTel) so Sentry's TracerProvider gets registered successfully.
+  trace.disable();
   trace.setGlobalTracerProvider(new SentryDenoTraceProvider());
 }