dd-trace 5.79.0 → 5.80.0

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

@@ -9,7 +9,7 @@ const BaseWriter = require('../common/writer')
 
 const METRIC_PREFIX = 'datadog.tracer.node.exporter.agent'
 
-class Writer extends BaseWriter {
+class AgentWriter extends BaseWriter {
   constructor ({ prioritySampler, lookup, protocolVersion, headers, config = {} }) {
     super(...arguments)
     const AgentEncoder = getEncoder(protocolVersion)
@@ -62,12 +62,6 @@ class Writer extends BaseWriter {
   }
 }
 
-function setHeader (headers, key, value) {
-  if (value) {
-    headers[key] = value
-  }
-}
-
 function getEncoder (protocolVersion) {
   return protocolVersion === '0.5'
     ? require('../../encode/0.5').AgentEncoder
@@ -82,16 +76,15 @@ function makeRequest (version, data, count, url, headers, lookup, needsStartupLo
       ...headers,
       'Content-Type': 'application/msgpack',
       'Datadog-Meta-Tracer-Version': tracerVersion,
-      'X-Datadog-Trace-Count': String(count)
+      'X-Datadog-Trace-Count': String(count),
+      'Datadog-Meta-Lang': 'nodejs',
+      'Datadog-Meta-Lang-Version': process.version,
+      'Datadog-Meta-Lang-Interpreter': process.jsEngine || 'v8'
     },
     lookup,
     url
   }
 
-  setHeader(options.headers, 'Datadog-Meta-Lang', 'nodejs')
-  setHeader(options.headers, 'Datadog-Meta-Lang-Version', process.version)
-  setHeader(options.headers, 'Datadog-Meta-Lang-Interpreter', process.jsEngine || 'v8')
-
   log.debug('Request to the agent: %j', options)
 
   request(data, options, (err, res, status) => {
@@ -105,4 +98,4 @@ function makeRequest (version, data, count, url, headers, lookup, needsStartupLo
   })
 }
 
-module.exports = Writer
+module.exports = AgentWriter

package/packages/dd-trace/src/lambda/runtime/ritm.js

@@ -53,8 +53,7 @@ function _extractModuleNameAndHandlerPath (handler) {
   const FUNCTION_EXPR = /^([^.]*)\.(.*)$/
   const match = handler.match(FUNCTION_EXPR)
   if (!match || match.length !== 3) {
-    // Malformed Handler Name
-    return // TODO: throw error
+    throw new Error(`Malformed handler name: ${handler}`)
   }
   return [match[1], match[2]] // [module, handler-path]
 }

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

@@ -12,7 +12,7 @@ const telemetry = require('./telemetry')
 const LLMObsSpanProcessor = require('./span_processor')
 
 const { channel } = require('dc-polyfill')
-const spanProcessCh = channel('dd-trace:span:process')
+const spanFinishCh = channel('dd-trace:span:finish')
 const evalMetricAppendCh = channel('llmobs:eval-metric:append')
 const flushCh = channel('llmobs:writers:flush')
 const injectCh = channel('dd-trace:span:inject')
@@ -62,7 +62,7 @@ function enable (config) {
   // span processing
   spanProcessor = new LLMObsSpanProcessor(config)
   spanProcessor.setWriter(spanWriter)
-  spanProcessCh.subscribe(handleSpanProcess)
+  spanFinishCh.subscribe(handleSpanProcess)
 
   // distributed tracing for llmobs
   injectCh.subscribe(handleLLMObsParentIdInjection)
@@ -86,7 +86,7 @@ function enable (config) {
 function disable () {
   if (evalMetricAppendCh.hasSubscribers) evalMetricAppendCh.unsubscribe(handleEvalMetricAppend)
   if (flushCh.hasSubscribers) flushCh.unsubscribe(handleFlush)
-  if (spanProcessCh.hasSubscribers) spanProcessCh.unsubscribe(handleSpanProcess)
+  if (spanFinishCh.hasSubscribers) spanFinishCh.unsubscribe(handleSpanProcess)
   if (injectCh.hasSubscribers) injectCh.unsubscribe(handleLLMObsParentIdInjection)
   if (registerUserSpanProcessorCh.hasSubscribers) registerUserSpanProcessorCh.unsubscribe(handleRegisterProcessor)
 
@@ -133,8 +133,8 @@ function handleRegisterProcessor (userSpanProcessor) {
   spanProcessor.setUserSpanProcessor(userSpanProcessor)
 }
 
-function handleSpanProcess (data) {
-  spanProcessor.process(data)
+function handleSpanProcess (span) {
+  spanProcessor.process(span)
 }
 
 function handleEvalMetricAppend (payload) {

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

@@ -75,6 +75,12 @@ class NoopLLMObs {
   submitEvaluation (llmobsSpanContext, options) {}
 
   flush () {}
+
+  registerProcessor (processor) {}
+
+  deregisterProcessor () {}
+
+  annotationContext (options, fn) { return fn() }
 }
 
 module.exports = NoopLLMObs

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

@@ -79,6 +79,7 @@ class VercelAILLMObsPlugin extends BaseLLMObsPlugin {
    *
    * We use the tool description as the next best identifier for a tool.
    *
+   * @param {string} toolName
    * @param {string} toolDescription
    * @returns {string | undefined}
    */

package/packages/dd-trace/src/llmobs/plugins/openai.js

@@ -67,6 +67,12 @@ class OpenAiLLMObsPlugin extends LLMObsPlugin {
     if (!error) {
       const metrics = this._extractMetrics(response)
       this._tagger.tagMetrics(span, metrics)
+
+      const responseModel = response.model
+      if (responseModel) {
+        // override the model name with the response model (more accurate)
+        this._tagger.tagModelName(span, responseModel)
+      }
     }
   }
 
@@ -85,11 +91,11 @@ class OpenAiLLMObsPlugin extends LLMObsPlugin {
 
     if (tokenUsage) {
       // Responses API uses input_tokens, Chat/Completions use prompt_tokens
-      const inputTokens = tokenUsage.input_tokens ?? tokenUsage.prompt_tokens
+      const inputTokens = tokenUsage.input_tokens ?? tokenUsage.prompt_tokens ?? 0
       if (inputTokens !== undefined) metrics.inputTokens = inputTokens
 
       // Responses API uses output_tokens, Chat/Completions use completion_tokens
-      const outputTokens = tokenUsage.output_tokens ?? tokenUsage.completion_tokens
+      const outputTokens = tokenUsage.output_tokens ?? tokenUsage.completion_tokens ?? 0
       if (outputTokens !== undefined) metrics.outputTokens = outputTokens
 
       const totalTokens = tokenUsage.total_tokens || (inputTokens + outputTokens)
@@ -105,7 +111,7 @@ class OpenAiLLMObsPlugin extends LLMObsPlugin {
       } else if (tokenUsage.prompt_tokens_details) {
         // Chat/Completions API - only include if > 0
         const cacheReadTokens = tokenUsage.prompt_tokens_details.cached_tokens
-        if (cacheReadTokens) {
+        if (cacheReadTokens != null) {
           metrics.cacheReadTokens = cacheReadTokens
         }
       }
@@ -159,6 +165,16 @@ class OpenAiLLMObsPlugin extends LLMObsPlugin {
   _tagChatCompletion (span, inputs, response, error) {
     const { messages, model, ...parameters } = inputs
 
+    const metadata = Object.entries(parameters).reduce((obj, [key, value]) => {
+      if (!['tools', 'functions'].includes(key)) {
+        obj[key] = value
+      }
+
+      return obj
+    }, {})
+
+    this._tagger.tagMetadata(span, metadata)
+
     if (error) {
       this._tagger.tagLLMIO(span, messages, [{ content: '' }])
       return
@@ -200,16 +216,6 @@ class OpenAiLLMObsPlugin extends LLMObsPlugin {
     }
 
     this._tagger.tagLLMIO(span, messages, outputMessages)
-
-    const metadata = Object.entries(parameters).reduce((obj, [key, value]) => {
-      if (!['tools', 'functions'].includes(key)) {
-        obj[key] = value
-      }
-
-      return obj
-    }, {})
-
-    this._tagger.tagMetadata(span, metadata)
   }
 
   #tagResponse (span, inputs, response, error) {
@@ -269,6 +275,15 @@ class OpenAiLLMObsPlugin extends LLMObsPlugin {
       inputMessages.push({ role: 'user', content: input })
     }
 
+    const inputMetadata = Object.entries(parameters).reduce((obj, [key, value]) => {
+      if (allowedParamKeys.has(key)) {
+        obj[key] = value
+      }
+      return obj
+    }, {})
+
+    this._tagger.tagMetadata(span, inputMetadata)
+
     if (error) {
       this._tagger.tagLLMIO(span, inputMessages, [{ content: '' }])
       return
@@ -287,17 +302,13 @@ class OpenAiLLMObsPlugin extends LLMObsPlugin {
       for (const item of response.output) {
         // Handle reasoning type (reasoning responses)
         if (item.type === 'reasoning') {
-          // Extract reasoning text from summary
-          let reasoningText = ''
-          if (Array.isArray(item.summary) && item.summary.length > 0) {
-            const summaryItem = item.summary[0]
-            if (summaryItem.type === 'summary_text' && summaryItem.text) {
-              reasoningText = summaryItem.text
-            }
-          }
           outputMessages.push({
             role: 'reasoning',
-            content: reasoningText
+            content: JSON.stringify({
+              summary: item.summary ?? [],
+              encrypted_content: item.encrypted_content ?? null,
+              id: item.id ?? ''
+            })
           })
         } else if (item.type === 'function_call') {
           // Handle function_call type (responses API tool calls)
@@ -369,24 +380,19 @@ class OpenAiLLMObsPlugin extends LLMObsPlugin {
 
     this._tagger.tagLLMIO(span, inputMessages, outputMessages)
 
-    const metadata = Object.entries(parameters).reduce((obj, [key, value]) => {
-      if (allowedParamKeys.has(key)) {
-        obj[key] = value
-      }
-      return obj
-    }, {})
+    const outputMetadata = {}
 
     // Add fields from response object (convert numbers to floats)
-    if (response.temperature !== undefined) metadata.temperature = Number(response.temperature)
-    if (response.top_p !== undefined) metadata.top_p = Number(response.top_p)
-    if (response.tool_choice !== undefined) metadata.tool_choice = response.tool_choice
-    if (response.truncation !== undefined) metadata.truncation = response.truncation
-    if (response.text !== undefined) metadata.text = response.text
+    if (response.temperature !== undefined) outputMetadata.temperature = Number(response.temperature)
+    if (response.top_p !== undefined) outputMetadata.top_p = Number(response.top_p)
+    if (response.tool_choice !== undefined) outputMetadata.tool_choice = response.tool_choice
+    if (response.truncation !== undefined) outputMetadata.truncation = response.truncation
+    if (response.text !== undefined) outputMetadata.text = response.text
     if (response.usage?.output_tokens_details?.reasoning_tokens !== undefined) {
-      metadata.reasoning_tokens = response.usage.output_tokens_details.reasoning_tokens
+      outputMetadata.reasoning_tokens = response.usage.output_tokens_details.reasoning_tokens
     }
 
-    this._tagger.tagMetadata(span, metadata)
+    this._tagger.tagMetadata(span, outputMetadata) // update the metadata with the output metadata
   }
 }
 

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

@@ -369,7 +369,7 @@ class LLMObs extends NoopLLMObs {
         err = 'invalid_metric_label'
         throw new Error('label must be the specified name of the evaluation metric')
       }
-      if (!metricType || !['categorical', 'score'].includes(metricType)) {
+      if (!metricType || !['categorical', 'score', 'boolean'].includes(metricType)) {
         err = 'invalid_metric_type'
         throw new Error('metricType must be one of "categorical" or "score"')
       }
@@ -381,6 +381,10 @@ class LLMObs extends NoopLLMObs {
         err = 'invalid_metric_value'
         throw new Error('value must be a number for a score metric.')
       }
+      if (metricType === 'boolean' && typeof value !== 'boolean') {
+        err = 'invalid_metric_value'
+        throw new Error('value must be a boolean for a boolean metric')
+      }
 
       const evaluationTags = {
         'ddtrace.version': tracerVersion,

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

@@ -72,7 +72,7 @@ class LLMObsSpanProcessor {
   }
 
   // TODO: instead of relying on the tagger's weakmap registry, can we use some namespaced storage correlation?
-  process ({ span }) {
+  process (span) {
     if (!this.#config.llmobs.enabled) return
     // if the span is not in our private tagger map, it is not an llmobs span
     if (!LLMObsTagger.tagMap.has(span)) return
@@ -155,7 +155,7 @@ class LLMObsSpanProcessor {
     llmObsSpan._tags = tags
 
     const processedSpan = this.#runProcessor(llmObsSpan)
-    if (processedSpan === null) return null
+    if (processedSpan === undefined) return null
 
     if (processedSpan.input) {
       if (inputType === 'messages') {
@@ -269,7 +269,7 @@ class LLMObsSpanProcessor {
   /**
    * Runs the user span processor, emitting telemetry and adding some guardrails against invalid return types
    * @param {LLMObservabilitySpan} span
-   * @returns {LLMObservabilitySpan | null}
+   * @returns {LLMObservabilitySpan | undefined}
    */
   #runProcessor (span) {
     const processor = this.#userSpanProcessor
@@ -279,12 +279,12 @@ class LLMObsSpanProcessor {
 
     try {
       const processedLLMObsSpan = processor(span)
-      if (processedLLMObsSpan === null) return null
+      if (processedLLMObsSpan === null) return
 
       if (!(processedLLMObsSpan instanceof LLMObservabilitySpan)) {
         error = true
         logger.warn('User span processor must return an instance of an LLMObservabilitySpan or null, dropping span.')
-        return null
+        return
       }
 
       return processedLLMObsSpan

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

@@ -85,7 +85,7 @@ class LLMObsTagger {
     if (name) this._setTag(span, NAME, name)
 
     this._setTag(span, SPAN_KIND, kind)
-    if (modelName) this._setTag(span, MODEL_NAME, modelName)
+    if (modelName) this.tagModelName(span, modelName)
     if (modelProvider) this._setTag(span, MODEL_PROVIDER, modelProvider)
 
     sessionId = sessionId || registry.get(parent)?.[SESSION_ID]
@@ -194,6 +194,10 @@ class LLMObsTagger {
     this._setTag(span, SPAN_KIND, newKind)
   }
 
+  tagModelName (span, modelName) {
+    this._setTag(span, MODEL_NAME, modelName)
+  }
+
   #tagText (span, data, key) {
     if (data) {
       if (typeof data === 'string') {
@@ -324,7 +328,7 @@ class LLMObsTagger {
 
     for (const message of data) {
       if (typeof message === 'string') {
-        messages.push({ content: message })
+        messages.push({ content: message, role: '' })
         continue
       }
       if (message == null || typeof message !== 'object') {
@@ -332,20 +336,30 @@ class LLMObsTagger {
         continue
       }
 
-      const { content = '', role } = message
-      const toolCalls = message.toolCalls
-      const toolResults = message.toolResults
-      const toolId = message.toolId
-      const messageObj = { content }
+      const {
+        role = '',
+        content,
+        toolCalls,
+        toolResults,
+        toolId
+      } = message
+      const messageObj = {}
 
       let condition = this.#tagConditionalString(role, 'Message role', messageObj, 'role')
 
-      const valid = typeof content === 'string'
-      if (!valid) {
-        this.#handleFailure('Message content must be a string.', 'invalid_io_messages')
+      if (
+        content == null &&
+        toolCalls == null &&
+        toolResults == null
+      ) {
+        messageObj.content = ''
+      }
+
+      if (content != null) {
+        condition = this.#tagConditionalString(content, 'Message content', messageObj, 'content') && condition
       }
 
-      if (toolCalls) {
+      if (toolCalls != null) {
         const filteredToolCalls = this.#filterToolCalls(toolCalls)
 
         if (filteredToolCalls.length) {
@@ -353,7 +367,7 @@ class LLMObsTagger {
         }
       }
 
-      if (toolResults) {
+      if (toolResults != null) {
         const filteredToolResults = this.#filterToolResults(toolResults)
 
         if (filteredToolResults.length) {
@@ -363,13 +377,13 @@ class LLMObsTagger {
 
       if (toolId) {
         if (role === 'tool') {
-          condition = this.#tagConditionalString(toolId, 'Tool ID', messageObj, 'tool_id')
+          condition = this.#tagConditionalString(toolId, 'Tool ID', messageObj, 'tool_id') && condition
         } else {
           log.warn(`Tool ID for tool message not associated with a "tool" role, instead got "${role}"`)
         }
       }
 
-      if (valid && condition) {
+      if (condition) {
         messages.push(messageObj)
       }
     }
@@ -380,7 +394,7 @@ class LLMObsTagger {
   }
 
   #tagConditionalString (data, type, carrier, key) {
-    if (!data) return true
+    if (data == null) return true
     if (typeof data !== 'string') {
       this.#handleFailure(`"${type}" must be a string.`)
       return false
@@ -390,7 +404,7 @@ class LLMObsTagger {
   }
 
   #tagConditionalNumber (data, type, carrier, key) {
-    if (!data) return true
+    if (data == null) return true
     if (typeof data !== 'number') {
       this.#handleFailure(`"${type}" must be a number.`)
       return false
@@ -400,7 +414,7 @@ class LLMObsTagger {
   }
 
   #tagConditionalObject (data, type, carrier, key) {
-    if (!data) return true
+    if (data == null) return true
     if (typeof data !== 'object') {
       this.#handleFailure(`"${type}" must be an object.`)
       return false

package/packages/dd-trace/src/msgpack/chunk.js

@@ -7,7 +7,7 @@ const DEFAULT_MIN_SIZE = 2 * 1024 * 1024 // 2MB
  * interfaces so that it can be used seamlessly by any encoder code that expects
  * either.
  */
-class Chunk {
+class MsgpackChunk {
   constructor (minSize = DEFAULT_MIN_SIZE) {
     this.buffer = Buffer.allocUnsafe(minSize)
     this.view = new DataView(this.buffer.buffer)
@@ -66,4 +66,4 @@ class Chunk {
   }
 }
 
-module.exports = Chunk
+module.exports = MsgpackChunk

package/packages/dd-trace/src/msgpack/encoder.js

@@ -1,11 +1,10 @@
 'use strict'
 
-const Chunk = require('./chunk')
+const MsgpackChunk = require('./chunk')
 
 class MsgpackEncoder {
   encode (value) {
-    const bytes = new Chunk()
-
+    const bytes = new MsgpackChunk()
     this.encodeValue(bytes, value)
 
     return bytes.buffer.subarray(0, bytes.length)

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

@@ -1,6 +1,6 @@
 'use strict'
 
-const Chunk = require('./chunk')
+const MsgpackChunk = require('./chunk')
 const { MsgpackEncoder } = require('./encoder')
 
-module.exports = { Chunk, MsgpackEncoder }
+module.exports = { MsgpackChunk, MsgpackEncoder }

package/packages/dd-trace/src/openfeature/flagging_provider.js

@@ -15,15 +15,17 @@ class FlaggingProvider extends DatadogNodeServerProvider {
    * @param {import('../config')} config - Tracer configuration object
    */
   constructor (tracer, config) {
-    // Call parent constructor with required options
+    // Call parent constructor with required options and timeout
     super({
-      exposureChannel: channel(EXPOSURE_CHANNEL)
+      exposureChannel: channel(EXPOSURE_CHANNEL),
+      initializationTimeoutMs: config.experimental.flaggingProvider.initializationTimeoutMs
     })
 
     this._tracer = tracer
     this._config = config
 
-    log.debug(this.constructor.name + ' created')
+    log.debug(this.constructor.name + ' created with timeout: ' +
+      config.experimental.flaggingProvider.initializationTimeoutMs + 'ms')
   }
 
   /**

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

@@ -7,7 +7,7 @@ const os = require('os')
  */
 
 /**
- * @fileoverview OpenTelemetry Logs Implementation for dd-trace-js
+ * OpenTelemetry Logs Implementation for `dd-trace-js`
  *
  * This package provides a custom OpenTelemetry Logs implementation that integrates
  * with the Datadog tracing library. It includes all necessary components for

package/packages/dd-trace/src/opentelemetry/logs/logger.js

@@ -5,10 +5,12 @@ const { context } = require('@opentelemetry/api')
 const packageVersion = require('../../../../../package.json').version
 /**
  * @typedef {import('@opentelemetry/api-logs').LogRecord} LogRecord
+ * @typedef {import('@opentelemetry/api-logs').LoggerProvider} LoggerProvider
  * @typedef {import('@opentelemetry/api').SpanContext} SpanContext
  * @typedef {import('@opentelemetry/api').Attributes} Attributes
  * @typedef {import('@opentelemetry/resources').Resource} Resource
  * @typedef {import('@opentelemetry/core').InstrumentationScope} InstrumentationScope
+ * @typedef {import('@opentelemetry/core').InstrumentationLibrary} InstrumentationLibrary
  */
 
 /**
@@ -26,12 +28,15 @@ class Logger {
    * Creates a new Logger instance.
    *
    * @param {LoggerProvider} loggerProvider - Parent logger provider
-   * @param {InstrumentationScope} [instrumentationScope] - Instrumentation scope information (newer API)
-   * @param {Object} [instrumentationLibrary] - Instrumentation library information (legacy API) [DEPRECATED in v1.3.0]
-   * @param {InstrumentationScope} [instrumentationScope.name] - Library name (defaults to 'dd-trace-js')
-   * @param {InstrumentationScope} [instrumentationScope.version] - Library version (defaults to tracer version)
-   * @param {string} [instrumentationLibrary.name] - Library name (legacy, defaults to 'dd-trace-js')
-   * @param {string} [instrumentationLibrary.version] - Library version (legacy, defaults to tracer version)
+   * @param {InstrumentationScope} [instrumentationScope] - Instrumentation scope information (newer API).
+   *  `name` defaults to 'dd-trace-js';
+   *  `version` defaults to tracer version;
+   *  `schemaUrl` defaults to '';
+   * @param {InstrumentationLibrary} [instrumentationLibrary]
+   *  - Instrumentation library information (legacy API) [DEPRECATED in v1.3.0].
+   *  `name` defaults to 'dd-trace-js';
+   *  `version` defaults to tracer version;
+   *  `schemaUrl` defaults to '';
    */
   constructor (loggerProvider, instrumentationScope, instrumentationLibrary) {
     this.loggerProvider = loggerProvider

package/packages/dd-trace/src/opentelemetry/logs/otlp_http_log_exporter.js

@@ -15,7 +15,7 @@ const OtlpTransformer = require('./otlp_transformer')
  * https://opentelemetry.io/docs/specs/otlp/#otlphttp
  *
  * @class OtlpHttpLogExporter
- * @extends OtlpHttpExporterBase
+ * @augments OtlpHttpExporterBase
  */
 class OtlpHttpLogExporter extends OtlpHttpExporterBase {
   /**

package/packages/dd-trace/src/opentelemetry/logs/otlp_transformer.js

@@ -44,7 +44,7 @@ const SEVERITY_MAP = {
  * https://opentelemetry.io/docs/specs/otlp/#log-data-model
  *
  * @class OtlpTransformer
- * @extends OtlpTransformerBase
+ * @augments OtlpTransformerBase
  */
 class OtlpTransformer extends OtlpTransformerBase {
   /**
@@ -73,7 +73,6 @@ class OtlpTransformer extends OtlpTransformerBase {
    * Transforms log records to protobuf format.
    * @param {LogRecord[]} logRecords - Array of enriched log records to transform
    * @returns {Buffer} Protobuf-encoded log records
-   * @private
    */
   #transformToProtobuf (logRecords) {
     const { protoLogsService } = getProtobufTypes()
@@ -92,7 +91,6 @@ class OtlpTransformer extends OtlpTransformerBase {
    * Transforms log records to JSON format.
    * @param {LogRecord[]} logRecords - Array of enriched log records to transform
    * @returns {Buffer} JSON-encoded log records
-   * @private
    */
   #transformToJson (logRecords) {
     const logsData = {
@@ -108,7 +106,6 @@ class OtlpTransformer extends OtlpTransformerBase {
    * Creates scope logs grouped by instrumentation library.
    * @param {LogRecord[]} logRecords - Array of log records to transform
    * @returns {Object[]} Array of scope log objects
-   * @private
    */
   #transformScope (logRecords) {
     const groupedRecords = this._groupByInstrumentationScope(logRecords)
@@ -135,7 +132,6 @@ class OtlpTransformer extends OtlpTransformerBase {
    * Transforms a single log record to OTLP format.
    * @param {LogRecord} logRecord - Log record to transform
    * @returns {Object} OTLP log record object
-   * @private
    */
   #transformLogRecord (logRecord) {
     const spanContext = this.#extractSpanContext(logRecord.context)
@@ -182,7 +178,6 @@ class OtlpTransformer extends OtlpTransformerBase {
    * Extracts span context from the log record's context.
    * @param {Object} logContext - The log record's context
    * @returns {Object|null} Span context or null if not available
-   * @private
    */
   #extractSpanContext (logContext) {
     if (!logContext) return null
@@ -199,7 +194,6 @@ class OtlpTransformer extends OtlpTransformerBase {
    * Maps OpenTelemetry severity number to protobuf severity number.
    * @param {number} severityNumber - OpenTelemetry severity number
    * @returns {number} Protobuf severity number
-   * @private
    */
   #mapSeverityNumber (severityNumber) {
     const { protoSeverityNumber } = getProtobufTypes()
@@ -211,7 +205,6 @@ class OtlpTransformer extends OtlpTransformerBase {
    * Converts a hex string to a Buffer.
    * @param {string} hexString - Hex string to convert
    * @returns {Buffer} Buffer containing the hex data
-   * @private
    */
   #hexToBytes (hexString) {
     const cleanHex = hexString ? (hexString.startsWith('0x') ? hexString.slice(2) : hexString) : ''
@@ -223,7 +216,6 @@ class OtlpTransformer extends OtlpTransformerBase {
    * Transforms log body to OTLP AnyValue format.
    * @param {any} body - Log body to transform
    * @returns {Object} OTLP AnyValue object
-   * @private
    */
   #transformBody (body) {
     if (typeof body === 'string') {

package/packages/dd-trace/src/opentelemetry/otlp/protobuf_loader.js

@@ -1,7 +1,7 @@
 'use strict'
 
 /**
- * @fileoverview Protobuf Loader for OpenTelemetry Logs and Metrics
+ * Protobuf Loader for OpenTelemetry Logs and Metrics
  *
  * This module loads protobuf definitions for OpenTelemetry logs and metrics.
  *

package/packages/dd-trace/src/plugins/database.js

@@ -77,6 +77,7 @@ class DatabasePlugin extends StoragePlugin {
       return servicePropagation
     } else if (mode === 'full') {
       span.setTag('_dd.dbm_trace_injected', 'true')
+      span._processor.sample(span)
       const traceparent = span._spanContext.toTraceparent()
       return `${servicePropagation},traceparent='${traceparent}'`
     }