You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
deno: Tracing integrations are added with tracing off #23893
deno/src/sdk.ts:46 spreads ...getTracingIntegrations() unconditionally. Node and Bun both gate on hasSpansEnabled(options).
$ deno run -A nospans.mjs # no tracesSampleRate
tracing OFF -> integration count: 39
... Graphql,Mongo,Mongoose,Mysql,Mysql2,Redis,Postgres,Prisma,Tedious,
GenericPool,Kafka,Amqplib,LruMemoizer,Aws,LangChain,LangGraph,VercelAI,
OpenAI,Anthropic_AI,Google_GenAI,PostgresJs,Firebase,SpanStreaming
On Deno these also subscribe eagerly rather than waiting for the module-injected event (instrumentation.ts:101), so this is real work, not just a longer list.
Work item. Gate getTracingIntegrations() on hasSpansEnabled(options) to match Node and Bun. Confirm first that no Deno integration in that set is relied on for error capture only; if one is, move it to getErrorIntegrations.
**Prior art **(tracked). #22306 (closed) "Automatically pull in all Orchestrion integrations to Deno" is the deliberate decision that produced the unconditional spread. It asked for all server-utils integrations to be pulled in automatically; it did not discuss the hasSpansEnabled gate that Node and Bun apply, which looks like an oversight rather than a choice. No issue covers the gate.
Activity
linear-code commentedon Sep 2, 2026
JS-3519
fix(deno): Gate tracing integrations on hasSpansEnabled (#24070)
github-actions commentedon Sep 23, 2026
A PR closing this issue has just been released 🚀
This issue was referenced by PR #24070, which was included in the 11.0.0 release.