dd-trace 6.13.0 → 6.14.0

package/packages/dd-trace/src/exporters/common/url.js

@@ -1,6 +1,7 @@
 'use strict'
 
 const net = require('node:net')
+const { format } = require('node:url')
 
 const { urlToHttpOptions } = require('./url-to-http-options-polyfill')
 
@@ -16,6 +17,44 @@ function isLoopbackHost (hostname) {
     (hostname.startsWith('127.') && net.isIPv4(hostname))
 }
 
+/**
+ * @param {string|undefined} protocol
+ * @param {string|undefined} hostname
+ * @returns {boolean}
+ */
+function canSendApiKey (protocol, hostname) {
+  return protocol === 'https:' || protocol === 'unix:' ||
+    typeof hostname === 'string' && isLoopbackHost(hostname)
+}
+
+/**
+ * @param {string} site
+ * @param {string} [intake]
+ * @returns {URL | undefined}
+ */
+function createSiteUrl (site, intake) {
+  const hostname = `${intake === undefined ? '' : `${intake}.`}${site}`.toLowerCase()
+
+  try {
+    const url = new URL(format({
+      protocol: 'https:',
+      hostname,
+    }))
+    if (
+      url.hostname !== hostname ||
+      url.username ||
+      url.password ||
+      url.port ||
+      url.pathname !== '/' ||
+      url.search ||
+      url.hash
+    ) {
+      return
+    }
+    return url
+  } catch {}
+}
+
 /**
  * Convert an agent/intake URL into Node http(s) request options.
  *
@@ -43,4 +82,4 @@ function parseUrl (urlObjOrString) {
   return url
 }
 
-module.exports = { isLoopbackHost, parseUrl }
+module.exports = { canSendApiKey, createSiteUrl, isLoopbackHost, parseUrl }

package/packages/dd-trace/src/exporters/common/writer.js

@@ -8,13 +8,16 @@ const request = require('./request')
 const { safeJSONStringify } = require('./util')
 
 const firstFlushChannel = channel('dd-trace:exporter:first-flush')
+const noop = () => {}
 
 class Writer {
   #deliveryTracker
+  #retainOnBackpressure
 
-  constructor ({ url, beforeFirstFlush, deliveryTracker }) {
+  constructor ({ url, beforeFirstFlush, deliveryTracker, retainOnBackpressure = false }) {
     this._url = url
     this._beforeFirstFlush = beforeFirstFlush
+    this.#retainOnBackpressure = retainOnBackpressure
     this.#deliveryTracker = deliveryTracker
   }
 
@@ -39,10 +42,10 @@ class Writer {
    * @param {{ deadline?: number }} [options]
    * @returns {void}
    */
-  flushDirect (done = () => {}, options) {
+  flushDirect (done = noop, options) {
     const count = this._encoder.count()
 
-    if (!request.writable && options?.deadline === undefined) {
+    if (!request.writable && options?.deadline === undefined && !this.#retainOnBackpressure) {
       this._encoder.reset()
       done()
     } else if (count > 0) {
@@ -77,7 +80,7 @@ class Writer {
   }
 
   append (payload, options) {
-    if (!request.writable && options?.deadline === undefined) {
+    if (!request.writable && options?.deadline === undefined && !this.#retainOnBackpressure) {
       // eslint-disable-next-line eslint-rules/eslint-log-printf-style
       log.debug(() => `Maximum number of active requests reached. Payload discarded: ${safeJSONStringify(payload)}`)
       return false
@@ -86,12 +89,11 @@ class Writer {
     // eslint-disable-next-line eslint-rules/eslint-log-printf-style
     log.debug(() => `Encoding payload: ${safeJSONStringify(payload)}`)
 
-    this._encode(payload)
-    return true
+    return this._encode(payload) !== false
   }
 
   _encode (payload) {
-    this._encoder.encode(payload)
+    return this._encoder.encode(payload)
   }
 
   setUrl (url) {

package/packages/dd-trace/src/flush.js

@@ -43,12 +43,16 @@ function registerTelemetryFlusher (flusher, options) {
  * @param {{ timeout?: number }} [options]
  * @param {TraceFlushers} [traceFlushers]
  */
-function flushServerlessTelemetry (done, options, traceFlushers = {}) {
-  const { trace: traceFlusher, spanStats: spanStatsFlusher } = traceFlushers
+function flushServerlessTelemetry (done, options, traceFlushers) {
+  const traceFlusher = traceFlushers?.trace
+  const spanStatsFlusher = traceFlushers?.spanStats
   // TODO: Include DSM after DataStreamsProcessor exposes a completion-aware flush API.
   let pending = telemetryFlushers.size + postTraceTelemetryFlushers.size +
     (typeof traceFlusher === 'function' ? 1 : 0) +
     (typeof spanStatsFlusher === 'function' ? 1 : 0)
+
+  if (pending === 0) return done?.()
+
   let completed = false
   let timeout
 
@@ -62,7 +66,6 @@ function flushServerlessTelemetry (done, options, traceFlushers = {}) {
     if (--pending === 0) finish()
   }
 
-  if (pending === 0) return finish()
   if (options?.timeout) {
     timeout = setTimeout(() => {
       log.warn('Timed out waiting for telemetry flush after %dms', options.timeout)

package/packages/dd-trace/src/id.js

@@ -2,6 +2,8 @@
 
 const { randomFillSync } = require('crypto')
 
+const { channel } = require('dc-polyfill')
+
 const UINT_MAX = 4_294_967_296
 
 const data = new Uint8Array(8 * 8192)
@@ -9,6 +11,8 @@ const zeroId = new Uint8Array(8)
 
 let batch = 0
 
+channel('datadog:identity:update').subscribe(reseed)
+
 // Internal representation of a trace or span ID.
 class Identifier {
   /** @type {number[] | Uint8Array} */
@@ -254,6 +258,16 @@ function writeUInt32BE (buffer, value, offset) {
   buffer[0 + offset] = value & 255
 }
 
+/**
+ * Resets the batch cursor, forcing the next ID batch to draw a fresh
+ * randomFillSync() call on MicroVM clone resume. Node's crypto RNG is
+ * re-seeded from the kernel CSPRNG on snapshot resume, so re-invoking it
+ * is sufficient — no need to read /dev/urandom directly.
+ */
+function reseed () {
+  batch = 0
+}
+
 /**
  * @param {string} [value]
  * @param {number} [radix]

package/packages/dd-trace/src/index.js

@@ -1,6 +1,7 @@
 'use strict'
 
-const { getValueFromEnvSources } = require('./config/helper')
+const { getEnvironmentVariable, getValueFromEnvSources } = require('./config/helper')
+const { isTrue } = require('./util')
 
 // Global `jest` is only present in Jest workers.
 const inJestWorker = typeof jest !== 'undefined'
@@ -8,8 +9,10 @@ const inJestWorker = typeof jest !== 'undefined'
 // skipDefault: distinguish an unset DD_TRACE_ENABLED (fall back to the OTel signal) from an
 // explicit value; the registered default would otherwise mask the OTEL_TRACES_EXPORTER check.
 const ddTraceEnabled = getValueFromEnvSources('DD_TRACE_ENABLED', true)
+const agentlessTracingEnabled = getValueFromEnvSources('DD_AGENTLESS_ENABLED') ||
+  isTrue(getEnvironmentVariable('_DD_APM_TRACING_AGENTLESS_ENABLED'))
 const ddTraceDisabled = ddTraceEnabled === undefined
-  ? getValueFromEnvSources('OTEL_TRACES_EXPORTER') === 'none'
+  ? !agentlessTracingEnabled && getValueFromEnvSources('OTEL_TRACES_EXPORTER') === 'none'
   : ddTraceEnabled === false
 
 module.exports = ddTraceDisabled || inJestWorker

package/packages/dd-trace/src/llmobs/experiments/dataset.js

@@ -1,7 +1,8 @@
 'use strict'
 
 const { randomUUID } = require('node:crypto')
-const snapshotPayload = require('../../../../../vendor/dist/rfdc')({ proto: false, circles: false })
+const createRfdc = require('../../../../../vendor/dist/rfdc')
+const snapshotPayload = createRfdc({ proto: false, circles: false })
 
 /** @typedef {{add?: string[], remove?: string[], replace?: string[]}} TagOperations */
 /**

package/packages/dd-trace/src/llmobs/index.js

@@ -2,7 +2,6 @@
 
 const { channel } = require('dc-polyfill')
 
-const { readDatadogTags, writeDatadogTags } = require('../carrier')
 const { registerTelemetryFlusher } = require('../flush')
 const log = require('../log')
 const { createServerlessDeliveryTracker } = require('../serverless')
@@ -25,7 +24,7 @@ const {
   PROPAGATED_TRACE_ID_KEY,
 } = require('./constants/tags')
 const { storage } = require('./storage')
-const { agentNameWireSafe, appendOptionalPropagatedTag, resolveAgentAttribution, stripTagsetEntry } = require('./util')
+const { agentNameWireSafe, resolveAgentAttribution } = require('./util')
 const telemetry = require('./telemetry')
 const LLMObsSpanProcessor = require('./span_processor')
 const LLMObsEvalMetricsWriter = require('./writers/evaluations')
@@ -64,6 +63,13 @@ let unregisterTelemetryFlusher
 /** @type {import('../config/config-base')} */
 let globalTracerConfig
 
+/**
+ * @typedef {object} TraceTagInjection
+ * @property {import('../opentracing/span_context')} spanContext
+ * @property {Array<string | undefined>} [traceTagReplacements]
+ * @property {number} [optionalTraceTagCount]
+ */
+
 /**
  * @param {@type import('../config/config-base')} config
  */
@@ -163,7 +169,8 @@ function retireWriters (retiredSpanWriter, retiredEvalWriter) {
 
 // since LLMObs traces can extend between services and be the same trace,
 // we need to propagate the parent id, mlApp, session id, and sampling rate/decision.
-function handleLLMObsInjection ({ carrier }) {
+/** @param {TraceTagInjection} injection */
+function handleLLMObsInjection (injection) {
   // Respect the standard propagator's gate: when trace tag propagation is
   // disabled, don't write `x-datadog-tags` for LLMObs either.
   if (globalTracerConfig.DD_TRACE_X_DATADOG_TAGS_MAX_LENGTH === 0) return
@@ -201,37 +208,30 @@ function handleLLMObsInjection ({ carrier }) {
     mlObsSpanTags, parent
   )
 
-  // `_injectTags` only writes `x-datadog-tags` when the trace has `_dd.p.*`
-  // tags, so it may be undefined here — coalesce before appending.
-  const existing = readDatadogTags(carrier)
-  let tags = existing || ''
-  if (parentId) tags += `${tags ? ',' : ''}${PROPAGATED_PARENT_ID_KEY}=${parentId}`
-  if (mlApp) tags += `${tags ? ',' : ''}${PROPAGATED_ML_APP_KEY}=${mlApp}`
-  if (sessionId) tags += `${tags ? ',' : ''}${PROPAGATED_SESSION_ID_KEY}=${sessionId}`
-  if (sampleRate != null) tags += `${tags ? ',' : ''}${PROPAGATED_SAMPLE_RATE_KEY}=${sampleRate}`
-  if (samplingDecision != null) tags += `${tags ? ',' : ''}${PROPAGATED_SAMPLING_DECISION_KEY}=${samplingDecision}`
-  if (propagatedTraceId != null) tags += `${tags ? ',' : ''}${PROPAGATED_TRACE_ID_KEY}=${propagatedTraceId}`
-  // When a local agent attribution is resolved, strip any stale upstream pagent entries that
-  // `_injectTags` may have already written into the carrier (it propagates all `_dd.p.*` from
-  // `_trace.tags`). This ensures the downstream sees a consistent id-only or id+name pair
-  // rather than a mix from different hops. The id is always digit-safe; an unsafe name is wiped.
-  if (parentAgentSpanId) {
-    tags = stripTagsetEntry(tags, PROPAGATED_PARENT_AGENT_ID_KEY)
-    tags = stripTagsetEntry(tags, PROPAGATED_PARENT_AGENT_NAME_KEY)
+  const traceTagReplacements = []
+  if (parentId) traceTagReplacements.push(PROPAGATED_PARENT_ID_KEY, parentId)
+  if (mlApp) traceTagReplacements.push(PROPAGATED_ML_APP_KEY, mlApp)
+  if (sessionId) traceTagReplacements.push(PROPAGATED_SESSION_ID_KEY, sessionId)
+  if (sampleRate != null) traceTagReplacements.push(PROPAGATED_SAMPLE_RATE_KEY, sampleRate.toString())
+  if (samplingDecision != null) {
+    traceTagReplacements.push(PROPAGATED_SAMPLING_DECISION_KEY, samplingDecision.toString())
   }
-  const maxLength = globalTracerConfig.DD_TRACE_X_DATADOG_TAGS_MAX_LENGTH
-  const tagsWithId = appendOptionalPropagatedTag(
-    tags, PROPAGATED_PARENT_AGENT_ID_KEY, parentAgentSpanId, null, maxLength
-  )
-  // Only append the name when the id fit: a name without an id is unresolvable by the backend.
-  if (tagsWithId === tags) {
-    tags = tagsWithId
-  } else {
-    tags = appendOptionalPropagatedTag(
-      tagsWithId, PROPAGATED_PARENT_AGENT_NAME_KEY, parentAgentName, agentNameWireSafe, maxLength
-    )
+  if (propagatedTraceId != null) traceTagReplacements.push(PROPAGATED_TRACE_ID_KEY, propagatedTraceId)
+
+  let optionalTraceTagCount = 0
+  if (parentAgentSpanId) {
+    traceTagReplacements.push(PROPAGATED_PARENT_AGENT_ID_KEY, parentAgentSpanId)
+    optionalTraceTagCount++
+    if (parentAgentName && agentNameWireSafe(parentAgentName)) {
+      traceTagReplacements.push(PROPAGATED_PARENT_AGENT_NAME_KEY, parentAgentName)
+    } else {
+      traceTagReplacements.push(PROPAGATED_PARENT_AGENT_NAME_KEY, undefined)
+    }
+    optionalTraceTagCount++
   }
-  if (tags !== existing) writeDatadogTags(carrier, tags)
+
+  injection.traceTagReplacements = traceTagReplacements
+  injection.optionalTraceTagCount = optionalTraceTagCount
 }
 
 /**

package/packages/dd-trace/src/llmobs/plugins/ai/ddTelemetry.js

@@ -4,7 +4,6 @@ const { channel } = require('dc-polyfill')
 const BaseLLMObsPlugin = require('../base')
 const { getModelProvider } = require('../../../../../datadog-plugin-ai/src/utils')
 
-const toolCreationCh = channel('tracing:orchestrion:ai:tool:start')
 const setAttributesCh = channel('dd-trace:vercel-ai:span:setAttributes')
 
 const { MODEL_NAME, MODEL_PROVIDER, NAME } = require('../../constants/tags')
@@ -21,16 +20,10 @@ const {
   getTelemetryMetadata,
 } = require('./util')
 
-/**
- * @typedef {Record<string, unknown> & { description?: string, id?: string }} AvailableToolArgs
- */
-
 /**
  * @typedef {string | number | boolean | null | undefined | string[] | number[] | boolean[]} TagValue
  * @typedef {Record<string, TagValue>} SpanTags
  *
- * @typedef {{ name?: string, description?: string }} ToolForModel
- *
  * @typedef {{ type: 'text' | 'reasoning' | 'redacted-reasoning', text?: string, data?: string }} TextPart
  * @typedef {{ type: 'tool-call', toolName: string, toolCallId: string, args?: unknown, input?: unknown }} ToolCallPart
  * @typedef {(
@@ -76,58 +69,14 @@ class DdTelemetryPlugin extends BaseLLMObsPlugin {
   static integration = 'ai'
   static prefix = 'tracing:dd-trace:vercel-ai'
 
-  /**
-   * The available tools within the runtime scope of this integration.
-   * This essentially acts as a global registry for all tools made through the Vercel AI SDK.
-   * @type {Set<AvailableToolArgs>}
-   */
-  #availableTools
-
-  /**
-   * A mapping of tool call IDs to tool names.
-   * This is used to map the tool call ID to the tool name for the output message.
-   * @type {Record<string, string>}
-   */
-  #toolCallIdsToName
-
   constructor (...args) {
     super(...args)
 
-    this.#toolCallIdsToName = {}
-    this.#availableTools = new Set()
-    toolCreationCh.subscribe(ctx => {
-      const toolArgs = ctx.arguments
-      const tool = toolArgs[0] ?? {}
-      this.#availableTools.add(tool)
-    })
-
     setAttributesCh.subscribe(({ ctx, attributes }) => {
       Object.assign(ctx.attributes, attributes)
     })
   }
 
-  /**
-   * Does a best-effort attempt to find the right tool name for the given tool description.
-   * This is because the Vercel AI SDK does not tag tools by name properly, but
-   * rather by the index they were passed in. Tool names appear nowhere in the span tags.
-   *
-   * We use the tool description as the next best identifier for a tool.
-   *
-   * @param {string} toolName
-   * @param {string | undefined} toolDescription
-   * @returns {string | undefined}
-   */
-  findToolName (toolName, toolDescription) {
-    if (Number.isNaN(Number.parseInt(toolName, 10))) return toolName
-
-    for (const availableTool of this.#availableTools) {
-      const description = availableTool.description
-      if (description === toolDescription && availableTool.id) {
-        return availableTool.id
-      }
-    }
-  }
-
   /**
    * @override
    */
@@ -232,7 +181,13 @@ class DdTelemetryPlugin extends BaseLLMObsPlugin {
     const lastUserPrompt =
       promptInfo.prompt ??
       promptInfo.messages.reverse().find(message => message.role === 'user')?.content
-    const prompt = Array.isArray(lastUserPrompt) ? lastUserPrompt.map(part => part.text ?? '').join('') : lastUserPrompt
+    let prompt = lastUserPrompt
+    if (Array.isArray(lastUserPrompt)) {
+      prompt = ''
+      for (const part of lastUserPrompt) {
+        if (typeof part.text === 'string') prompt += part.text
+      }
+    }
 
     const output = tags['ai.response.object']
 
@@ -248,7 +203,13 @@ class DdTelemetryPlugin extends BaseLLMObsPlugin {
     const lastUserPrompt =
       promptInfo.prompt ??
       promptInfo.messages.reverse().find(message => message.role === 'user')?.content
-    const prompt = Array.isArray(lastUserPrompt) ? lastUserPrompt.map(part => part.text ?? '').join('') : lastUserPrompt
+    let prompt = lastUserPrompt
+    if (Array.isArray(lastUserPrompt)) {
+      prompt = ''
+      for (const part of lastUserPrompt) {
+        if (typeof part.text === 'string') prompt += part.text
+      }
+    }
 
     const output = tags['ai.response.text']
 
@@ -263,16 +224,14 @@ class DdTelemetryPlugin extends BaseLLMObsPlugin {
    * @param {SpanTags} tags
    */
   setLLMOperationTags (span, tags) {
-    const toolsForModel = tags['ai.prompt.tools']?.map(getJsonStringValue)
-
     const inputMessages = getJsonStringValue(tags['ai.prompt.messages'], [])
     const parsedInputMessages = []
     for (const message of inputMessages) {
-      const formattedMessages = this.formatMessage(message, toolsForModel)
+      const formattedMessages = this.formatMessage(message)
       parsedInputMessages.push(...formattedMessages)
     }
 
-    const outputMessage = this.formatOutputMessage(tags, toolsForModel)
+    const outputMessage = this.formatOutputMessage(tags)
 
     this._tagger.tagLLMIO(span, parsedInputMessages, outputMessage)
 
@@ -284,8 +243,8 @@ class DdTelemetryPlugin extends BaseLLMObsPlugin {
   }
 
   setToolTags (span, tags) {
-    const toolCallId = tags['ai.toolCall.id']
-    const name = getToolNameFromTags(tags) ?? this.#toolCallIdsToName[toolCallId]
+    const name = getToolNameFromTags(tags)
+
     if (name) this._tagger._setTag(span, NAME, name)
 
     const input = tags['ai.toolCall.args']
@@ -294,7 +253,7 @@ class DdTelemetryPlugin extends BaseLLMObsPlugin {
     this._tagger.tagTextIO(span, input, output)
   }
 
-  formatOutputMessage (tags, toolsForModel) {
+  formatOutputMessage (tags) {
     const outputMessageText = tags['ai.response.text'] ?? tags['ai.response.object']
     const outputMessageToolCalls = getJsonStringValue(tags['ai.response.toolCalls'], [])
 
@@ -302,13 +261,9 @@ class DdTelemetryPlugin extends BaseLLMObsPlugin {
     for (const toolCall of outputMessageToolCalls) {
       const toolArgs = toolCall.args ?? toolCall.input
       const toolCallArgs = typeof toolArgs === 'string' ? getJsonStringValue(toolArgs, {}) : toolArgs
-      const toolDescription = toolsForModel?.find(tool => toolCall.toolName === tool.name)?.description
-      const name = this.findToolName(toolCall.toolName, toolDescription)
-      this.#toolCallIdsToName[toolCall.toolCallId] = name
-
       formattedToolCalls.push({
         arguments: toolCallArgs,
-        name,
+        name: toolCall.toolName,
         toolId: toolCall.toolCallId,
         type: toolCall.toolCallType ?? 'function',
       })
@@ -328,11 +283,10 @@ class DdTelemetryPlugin extends BaseLLMObsPlugin {
    * need to split into multiple messages.
    *
    * @param {AiSdkMessage} message
-   * @param {ToolForModel[] | null | undefined} toolsForModel
    * @returns {Array<{role: string, content: string, toolId?: string,
    *   toolCalls?: Array<{arguments: string, name: string, toolId: string, type: string}>}>}
    */
-  formatMessage (message, toolsForModel) {
+  formatMessage (message) {
     const { role, content } = message
 
     if (role === 'system') {
@@ -357,12 +311,9 @@ class DdTelemetryPlugin extends BaseLLMObsPlugin {
         if (['text', 'reasoning', 'redacted-reasoning'].includes(type)) {
           finalContent += part.text ?? part.data
         } else if (type === 'tool-call') {
-          const toolDescription = toolsForModel?.find(tool => part.toolName === tool.name)?.description
-          const name = this.findToolName(part.toolName, toolDescription)
-
           toolCalls.push({
             arguments: part.args ?? part.input,
-            name,
+            name: part.toolName,
             toolId: part.toolCallId,
             type: 'function',
           })

package/packages/dd-trace/src/llmobs/plugins/ai/vercelTelemetry.js

@@ -38,11 +38,15 @@ function formatLanguageModelInputMessages (instructions, messages) {
   const inputMessages = []
 
   if (instructions) {
-    const systemPrompt = typeof instructions === 'string'
-      ? instructions
-      : Array.isArray(instructions)
-        ? instructions.map(instruction => instruction.content).join('')
-        : instructions.content
+    let systemPrompt = instructions
+    if (typeof instructions !== 'string') {
+      if (Array.isArray(instructions)) {
+        systemPrompt = ''
+        for (const instruction of instructions) systemPrompt += instruction.content
+      } else {
+        systemPrompt = instructions.content
+      }
+    }
 
     inputMessages.push({ role: 'system', content: systemPrompt })
   }
@@ -53,13 +57,13 @@ function formatLanguageModelInputMessages (instructions, messages) {
     if (role === 'system') {
       inputMessages.push({ role, content })
     } else if (role === 'user') {
-      const userMessageContent =
-      typeof content === 'string'
-        ? content
-        : content
-          .filter(part => part.type === 'text')
-          .map(part => part.text)
-          .join('')
+      let userMessageContent = content
+      if (typeof content !== 'string') {
+        userMessageContent = ''
+        for (const part of content) {
+          if (part.type === 'text') userMessageContent += part.text
+        }
+      }
 
       inputMessages.push({ role, content: userMessageContent })
     } else if (role === 'assistant') {
@@ -292,7 +296,13 @@ class VercelAiTelemetryPlugin extends BaseLLMObsPlugin {
     const { event, result } = ctx
 
     const lastUserPrompt = event.messages.findLast(message => message.role === 'user')?.content
-    const input = Array.isArray(lastUserPrompt) ? lastUserPrompt.map(part => part.text ?? '').join('') : lastUserPrompt
+    let input = lastUserPrompt
+    if (Array.isArray(lastUserPrompt)) {
+      input = ''
+      for (const part of lastUserPrompt) {
+        if (part.type === 'text') input += part.text
+      }
+    }
 
     const output =
       ctx.isStream

package/packages/dd-trace/src/llmobs/plugins/langchain/index.js

@@ -46,6 +46,13 @@ class BaseLangChainLLMObsPlugin extends LLMObsPlugin {
     }
   }
 
+  error (ctx) {
+    // NodeInterrupt or GraphInterrupt "errors" for control flow, do not mark as real errors
+    if (ctx.error?.is_bubble_up) return
+
+    super.error(ctx)
+  }
+
   getLLMObsSpanRegisterOptions (ctx) {
     const span = ctx.currentStore?.span
     const spanContext = span?.context()

package/packages/dd-trace/src/llmobs/util.js

@@ -337,37 +337,6 @@ function resolveAgentAttribution (tags, span) {
   return { name: tags[PARENT_AGENT_NAME], spanId: tags[PARENT_AGENT_SPAN_ID] }
 }
 
-/**
- * Removes all `key=value` entries for the given key from a comma-separated tagset string.
- *
- * @param {string} tags - Existing tagset string (may be empty).
- * @param {string} key
- * @returns {string}
- */
-function stripTagsetEntry (tags, key) {
-  if (!tags.includes(key)) return tags
-  return tags.split(',').filter(entry => !entry.startsWith(`${key}=`)).join(',')
-}
-
-/**
- * Appends `key=value` to the tagset string with a comma separator, but only when `value` is
- * truthy, passes the optional `safeguard` predicate, and fits within `maxTagSetLength`. Returns
- * the original `tags` unchanged when the value is absent, unsafe, or would overflow the budget.
- *
- * @param {string} tags - Existing tagset string (may be empty).
- * @param {string} key
- * @param {string | undefined} value
- * @param {((v: string) => boolean) | null} [safeguard]
- * @param {number} [maxTagSetLength]
- * @returns {string}
- */
-function appendOptionalPropagatedTag (tags, key, value, safeguard, maxTagSetLength) {
-  if (!value || (safeguard && !safeguard(value))) return tags
-  const entry = `${tags ? ',' : ''}${key}=${value}`
-  if (maxTagSetLength != null && tags.length + entry.length > maxTagSetLength) return tags
-  return `${tags}${entry}`
-}
-
 function spanHasError (span) {
   const spanContext = span.context()
   return !!(spanContext.getTag('error') || spanContext.getTag('error.type'))
@@ -518,7 +487,6 @@ function formatAudioPart (data, mimeType) {
 
 module.exports = {
   agentNameWireSafe,
-  appendOptionalPropagatedTag,
   audioMimeTypeFromFormat,
   encodeUnicode,
   findGenAIAncestorSpanId,
@@ -527,7 +495,6 @@ module.exports = {
   normalizeLlmObsTraceId,
   formatAudioPart,
   resolveAgentAttribution,
-  stripTagsetEntry,
   validateCostTags,
   validateKind,
   getFunctionArguments,