dd-trace 6.16.0 → 6.18.0

package/packages/datadog-instrumentations/src/helpers/rewriter/instrumentations/supabase.js

@@ -0,0 +1,128 @@
+'use strict'
+
+const versionRange = '>=2.112.2'
+
+module.exports = [
+  {
+    module: {
+      name: '@supabase/storage-js',
+      versionRange,
+      filePath: 'dist/index.cjs',
+    },
+    functionQuery: { functionName: '_handleRequest', kind: 'Async' },
+    channelName: 'handleRequest',
+  },
+  {
+    module: {
+      name: '@supabase/storage-js',
+      versionRange,
+      filePath: 'dist/index.mjs',
+    },
+    functionQuery: { functionName: '_handleRequest', kind: 'Async' },
+    channelName: 'handleRequest',
+  },
+  {
+    module: {
+      name: '@supabase/auth-js',
+      versionRange,
+      filePath: 'dist/main/GoTrueClient.js',
+    },
+    functionQuery: {
+      methodName: 'getUser',
+      className: 'GoTrueClient',
+      kind: 'Async',
+    },
+    channelName: 'GoTrueClient_getUser',
+  },
+  {
+    module: {
+      name: '@supabase/auth-js',
+      versionRange,
+      filePath: 'dist/module/GoTrueClient.js',
+    },
+    functionQuery: {
+      methodName: 'getUser',
+      className: 'GoTrueClient',
+      kind: 'Async',
+    },
+    channelName: 'GoTrueClient_getUser',
+  },
+  {
+    module: {
+      name: '@supabase/realtime-js',
+      versionRange,
+      filePath: 'dist/main/RealtimeChannel.js',
+    },
+    functionQuery: {
+      methodName: 'send',
+      className: 'RealtimeChannel',
+      kind: 'Async',
+    },
+    channelName: 'RealtimeChannel_send',
+  },
+  {
+    module: {
+      name: '@supabase/realtime-js',
+      versionRange,
+      filePath: 'dist/module/RealtimeChannel.js',
+    },
+    functionQuery: {
+      methodName: 'send',
+      className: 'RealtimeChannel',
+      kind: 'Async',
+    },
+    channelName: 'RealtimeChannel_send',
+  },
+  {
+    module: {
+      name: '@supabase/functions-js',
+      versionRange,
+      filePath: 'dist/main/FunctionsClient.js',
+    },
+    functionQuery: {
+      methodName: 'invoke',
+      className: 'FunctionsClient',
+      kind: 'Async',
+    },
+    channelName: 'FunctionsClient_invoke',
+  },
+  {
+    module: {
+      name: '@supabase/functions-js',
+      versionRange,
+      filePath: 'dist/module/FunctionsClient.js',
+    },
+    functionQuery: {
+      methodName: 'invoke',
+      className: 'FunctionsClient',
+      kind: 'Async',
+    },
+    channelName: 'FunctionsClient_invoke',
+  },
+  {
+    module: {
+      name: '@supabase/postgrest-js',
+      versionRange,
+      filePath: 'dist/index.cjs',
+    },
+    functionQuery: {
+      methodName: 'then',
+      className: 'PostgrestBuilder',
+      kind: 'Async',
+    },
+    channelName: 'PostgrestBuilder_then',
+  },
+  {
+    module: {
+      name: '@supabase/postgrest-js',
+      versionRange,
+      filePath: 'dist/index.mjs',
+    },
+    functionQuery: {
+      methodName: 'then',
+      className: 'PostgrestBuilder',
+      kind: 'Async',
+    },
+    channelName: 'PostgrestBuilder_then',
+  },
+]

package/packages/datadog-instrumentations/src/helpers/rewriter/instrumentations/webdriverio.js

@@ -1,6 +1,19 @@
 'use strict'
 
 module.exports = [
+  {
+    module: {
+      name: '@wdio/config',
+      versionRange: '>=9.0.0',
+      filePath: 'build/node/index.js',
+    },
+    functionQuery: {
+      className: 'ConfigParser',
+      methodName: 'shard',
+      kind: 'Sync',
+    },
+    channelName: 'ConfigParser_shard',
+  },
   {
     module: {
       name: '@wdio/cli',

package/packages/datadog-instrumentations/src/helpers/rewriter/targets.js

@@ -1,7 +1,8 @@
 'use strict'
 
-// Generated by scripts/generate-rewriter-targets.js from ./instrumentations. Do not edit by hand.
+// targets.json is generated by scripts/generate-rewriter-targets.js from ./instrumentation-registry.
 const targets = require('./targets.json')
+const rewriteTargetNames = new Set(Object.values(targets))
 
 const NODE_MODULES = '/node_modules/'
 
@@ -9,6 +10,7 @@ const NODE_MODULES = '/node_modules/'
  * @typedef {object} RewriteTarget
  * @property {string} moduleName
  * @property {string} filePath
+ * @property {boolean} [activate]
  */
 
 /**
@@ -20,13 +22,36 @@ function getRewriteTarget (filename) {
   if (nodeModulesIndex === -1) return
 
   const modulePath = filename.slice(nodeModulesIndex + NODE_MODULES.length)
+  if (!Object.hasOwn(targets, modulePath)) return
   const moduleName = targets[modulePath]
   if (typeof moduleName !== 'string') return
+  const activate = isRewriteActivationEnabled(moduleName)
 
-  return {
+  const rewriteTarget = {
     moduleName,
     filePath: modulePath.slice(moduleName.length + 1),
   }
+  if (activate) rewriteTarget.activate = true
+  return rewriteTarget
 }
 
-module.exports = { getRewriteTarget }
+function getRewriteTargetNames () {
+  return rewriteTargetNames.values()
+}
+
+/**
+ * @param {string} name
+ */
+function isRewriteTargetName (name) {
+  return rewriteTargetNames.has(name)
+}
+
+/**
+ * @param {string} moduleName
+ */
+function isRewriteActivationEnabled (moduleName) {
+  // Keep target-name-only consumers from loading every instrumentation descriptor.
+  return require('./instrumentation-registry').isRewriteActivationEnabled(moduleName)
+}
+
+module.exports = { getRewriteTarget, getRewriteTargetNames, isRewriteActivationEnabled, isRewriteTargetName }

package/packages/datadog-instrumentations/src/helpers/rewriter/targets.json

@@ -30,7 +30,19 @@
   "@modelcontextprotocol/sdk/dist/esm/server/mcp.js": "@modelcontextprotocol/sdk",
   "@openai/agents-openai/dist/openaiChatCompletionsModel.js": "@openai/agents-openai",
   "@openai/agents-openai/dist/openaiChatCompletionsModel.mjs": "@openai/agents-openai",
+  "@playwright/test/lib/reporters/multiplexer.js": "@playwright/test",
+  "@supabase/auth-js/dist/main/GoTrueClient.js": "@supabase/auth-js",
+  "@supabase/auth-js/dist/module/GoTrueClient.js": "@supabase/auth-js",
+  "@supabase/functions-js/dist/main/FunctionsClient.js": "@supabase/functions-js",
+  "@supabase/functions-js/dist/module/FunctionsClient.js": "@supabase/functions-js",
+  "@supabase/postgrest-js/dist/index.cjs": "@supabase/postgrest-js",
+  "@supabase/postgrest-js/dist/index.mjs": "@supabase/postgrest-js",
+  "@supabase/realtime-js/dist/main/RealtimeChannel.js": "@supabase/realtime-js",
+  "@supabase/realtime-js/dist/module/RealtimeChannel.js": "@supabase/realtime-js",
+  "@supabase/storage-js/dist/index.cjs": "@supabase/storage-js",
+  "@supabase/storage-js/dist/index.mjs": "@supabase/storage-js",
   "@wdio/cli/build/index.js": "@wdio/cli",
+  "@wdio/config/build/node/index.js": "@wdio/config",
   "@wdio/jasmine-framework/build/index.js": "@wdio/jasmine-framework",
   "@wdio/local-runner/build/index.js": "@wdio/local-runner",
   "@wdio/runner/build/index.js": "@wdio/runner",
@@ -67,6 +79,14 @@
   "playwright-core/lib/coreBundle.js": "playwright-core",
   "playwright/lib/index.js": "playwright",
   "playwright/lib/runner/index.js": "playwright",
+  "playwright/lib/runner/lastRun.js": "playwright",
+  "playwright/lib/runner/runner.js": "playwright",
+  "postgres/cjs/src/connection.js": "postgres",
+  "postgres/cjs/src/index.js": "postgres",
+  "postgres/cjs/src/query.js": "postgres",
+  "postgres/src/connection.js": "postgres",
+  "postgres/src/index.js": "postgres",
+  "postgres/src/query.js": "postgres",
   "webdriver/build/index.js": "webdriver",
   "webdriver/build/node.js": "webdriver",
   "webdriverio/build/index.js": "webdriverio",

package/packages/datadog-instrumentations/src/helpers/rewriter/transforms/postgres.js

@@ -0,0 +1,388 @@
+'use strict'
+
+const assert = require('node:assert')
+
+const { parse, query } = require('../compiler')
+
+const POSTGRES_OPTIONS = '__ddTracePostgresOptions'
+const POSTGRES_QUERY_REGISTRY = '__ddTracePostgresQueries'
+const POSTGRES_QUERY_REGISTRY_SYMBOL = '__ddTracePostgresQueriesSymbol'
+const POSTGRES_READY = '__ddTracePostgresQueryReady'
+const POSTGRES_SIMPLE_DBM_QUERIES = '__ddTracePostgresSimpleDbmQueries'
+const POSTGRES_STATEMENT = 'this.string ?? (this.tagged && this.strings?.length !== 1 ? undefined : this.strings?.[0])'
+
+module.exports = {
+  postgresQueryHandlers,
+  postgresQueryLifecycle,
+  postgresQueryPreparation,
+}
+
+/**
+ * Starts a Postgres.js query at the pool handler that owns its connection options.
+ *
+ * @param {object} state
+ * @param {import('estree').Program} program
+ */
+function postgresQueryHandlers (state, program) {
+  const channelVariable = injectPostgresTracingChannel(state, program)
+  injectPostgresReadyCheck(program, findPostgresQueryIdentifier(program))
+  capturePostgresOptions(program)
+
+  const handlers = query(program, 'FunctionDeclaration[id.name="handler"]')
+  assert(handlers.length >= 2 && handlers.length <= 3, 'postgresQueryHandlers: unexpected handler count')
+
+  for (const handler of handlers) {
+    const queryParameter = handler.params[0]
+    assert(queryParameter?.type === 'Identifier', 'postgresQueryHandlers: handler query parameter changed')
+    wrapPostgresHandler(handler, queryParameter.name, channelVariable)
+  }
+}
+
+/**
+ * Publishes the final Postgres.js query settlement without changing its Promise implementation.
+ *
+ * @param {object} state
+ * @param {import('estree').Program} program
+ */
+function postgresQueryLifecycle (state, program) {
+  const channelVariable = injectPostgresTracingChannel(state, program)
+  injectPostgresQueryRegistration(program)
+
+  const resolveAssignments = query(
+    program,
+    'AssignmentExpression[left.object.type="ThisExpression"][left.property.name="resolve"]'
+  )
+  const rejectAssignments = query(
+    program,
+    'AssignmentExpression[left.object.type="ThisExpression"][left.property.name="reject"]'
+  )
+
+  assert(resolveAssignments.length === 2, 'postgresQueryLifecycle: unexpected resolve assignment count')
+  assert(rejectAssignments.length === 2, 'postgresQueryLifecycle: unexpected reject assignment count')
+
+  for (const assignment of resolveAssignments) {
+    wrapPostgresResolution(assignment, channelVariable)
+  }
+  for (const assignment of rejectAssignments) {
+    wrapPostgresRejection(assignment, channelVariable)
+  }
+}
+
+/**
+ * Injects DBM data after Postgres.js builds the final statement and before it creates wire messages.
+ *
+ * @param {object} state
+ * @param {import('estree').Program} program
+ */
+function postgresQueryPreparation (state, program) {
+  const channelVariable = injectPostgresTracingChannel(state, program)
+  const buildFunctions = query(program, 'FunctionDeclaration[id.name="build"]')
+  const toBufferFunctions = query(program, 'FunctionDeclaration[id.name="toBuffer"]')
+
+  assert(buildFunctions.length === 1, 'postgresQueryPreparation: build function changed')
+  assert(toBufferFunctions.length === 1, 'postgresQueryPreparation: toBuffer function changed')
+
+  const build = buildFunctions[0]
+  const queryParameter = build.params[0]
+  assert(queryParameter?.type === 'Identifier', 'postgresQueryPreparation: build query parameter changed')
+  assert(build.body.type === 'BlockStatement', 'postgresQueryPreparation: build body changed')
+
+  const queryName = queryParameter.name
+  const body = build.body.body
+  const stringDeclaration = body.find(statement =>
+    statement.type === 'VariableDeclaration' &&
+      statement.declarations.some(({ id }) => id.type === 'Identifier' && id.name === 'string')
+  )
+  assert(stringDeclaration?.kind === 'const', 'postgresQueryPreparation: string declaration changed')
+
+  const prepareIndex = body.findIndex(statement =>
+    statement.type === 'ExpressionStatement' &&
+      statement.expression.type === 'AssignmentExpression' &&
+      statement.expression.left.type === 'MemberExpression' &&
+      statement.expression.left.object.type === 'Identifier' &&
+      statement.expression.left.object.name === queryName &&
+      statement.expression.left.property.type === 'Identifier' &&
+      statement.expression.left.property.name === 'prepare'
+  )
+  assert(prepareIndex !== -1, 'postgresQueryPreparation: prepare assignment changed')
+
+  const legacySimpleString = findLegacyPostgresSimpleString(toBufferFunctions[0], queryName)
+  const contextStatements = parse(`
+    if (${channelVariable}.start.hasSubscribers) {
+      const __ddTraceContext = {
+        query: ${queryName},
+        statement: string,
+        prepared: Boolean(${queryName}.prepare && !${queryName}.options.simple)
+      };
+      ${channelVariable}.start.publish(__ddTraceContext);
+      ${legacySimpleString === undefined
+        ? ''
+        : `if (${queryName}.options.simple && __ddTraceContext.statement !== string) {
+          ${POSTGRES_SIMPLE_DBM_QUERIES}.add(${queryName});
+        }`}
+      string = __ddTraceContext.statement;
+    }
+  `).body
+
+  stringDeclaration.kind = 'let'
+  body.splice(prepareIndex + 1, 0, ...contextStatements)
+
+  if (legacySimpleString !== undefined) {
+    injectPostgresModuleStatements(
+      program,
+      parse(`const ${POSTGRES_SIMPLE_DBM_QUERIES} = new WeakSet();`).body
+    )
+    const replacement = parse(`
+      ${channelVariable}.start.hasSubscribers && ${POSTGRES_SIMPLE_DBM_QUERIES}.delete(${queryName})
+        ? ${queryName}.string
+        : ${queryName}.strings[0]
+    `).body[0].expression
+
+    for (const key of Object.keys(legacySimpleString)) {
+      delete legacySimpleString[key]
+    }
+    Object.assign(legacySimpleString, replacement)
+  }
+}
+
+/**
+ * @param {import('estree').FunctionDeclaration|import('estree').ArrowFunctionExpression} handler
+ * @param {string} queryParameter
+ * @param {string} channelVariable
+ */
+function wrapPostgresHandler (handler, queryParameter, channelVariable) {
+  assert(handler.body.type === 'BlockStatement', 'postgresQueryHandlers: handler body changed')
+  assert(!handler.async && !handler.generator, 'postgresQueryHandlers: unsupported handler kind')
+
+  const originalBody = handler.body.body
+  const wrapperBody = parse(`
+    function wrapper () {
+      if (${channelVariable}.start.hasSubscribers && ${POSTGRES_READY} && !${queryParameter}.cancelled) {
+        const __ddTraceContext = {
+          query: ${queryParameter},
+          database: ${POSTGRES_OPTIONS}.database,
+          user: ${POSTGRES_OPTIONS}.user
+        };
+        if (${POSTGRES_OPTIONS}.host.length === 1 && !${POSTGRES_OPTIONS}.path) {
+          __ddTraceContext.host = ${POSTGRES_OPTIONS}.host[0];
+          __ddTraceContext.port = ${POSTGRES_OPTIONS}.port[0];
+        }
+        return ${channelVariable}.start.runStores(__ddTraceContext, () => {});
+      }
+    }
+  `).body[0].body.body
+
+  wrapperBody[0].consequent.body.at(-1).argument.arguments[1].body.body = originalBody
+  handler.body.body = [...wrapperBody, ...originalBody]
+}
+
+/**
+ * @param {import('estree').Program} program
+ */
+function capturePostgresOptions (program) {
+  const postgresFunctions = query(program, 'FunctionDeclaration[id.name="Postgres"]')
+  assert(postgresFunctions.length === 1, 'postgresQueryHandlers: Postgres function changed')
+
+  const body = postgresFunctions[0].body.body
+  const optionsIndex = body.findIndex(statement =>
+    statement.type === 'VariableDeclaration' && statement.declarations.some(({ id }) => id.name === 'options')
+  )
+  assert(optionsIndex !== -1, 'postgresQueryHandlers: options declaration changed')
+
+  const capture = parse(`const ${POSTGRES_OPTIONS} = options;`).body[0]
+  body.splice(optionsIndex + 1, 0, capture)
+}
+
+/**
+ * @param {import('estree').Program} program
+ * @param {string} queryIdentifier
+ */
+function injectPostgresReadyCheck (program, queryIdentifier) {
+  const statements = parse(`
+    const ${POSTGRES_QUERY_REGISTRY} = globalThis[Symbol.for('dd-trace:postgres:query')];
+    const ${POSTGRES_READY} = ${POSTGRES_QUERY_REGISTRY} instanceof WeakSet &&
+      ${POSTGRES_QUERY_REGISTRY}.has(${queryIdentifier});
+  `).body
+  injectPostgresModuleStatements(program, statements)
+}
+
+/**
+ * @param {import('estree').FunctionDeclaration} toBuffer
+ * @param {string} queryName
+ * @returns {import('estree').MemberExpression | undefined}
+ */
+function findLegacyPostgresSimpleString (toBuffer, queryName) {
+  const simpleConditions = query(toBuffer, 'ConditionalExpression').filter(({ test }) =>
+    test.type === 'MemberExpression' &&
+      test.object.type === 'MemberExpression' &&
+      test.object.object.type === 'Identifier' &&
+      test.object.object.name === queryName &&
+      test.object.property.type === 'Identifier' &&
+      test.object.property.name === 'options' &&
+      test.property.type === 'Identifier' &&
+      test.property.name === 'simple'
+  )
+  assert(simpleConditions.length === 1, 'postgresQueryPreparation: simple query branch changed')
+
+  const legacyStrings = query(simpleConditions[0].consequent, 'MemberExpression').filter(node =>
+    node.computed &&
+      node.object.type === 'MemberExpression' &&
+      node.object.object.type === 'Identifier' &&
+      node.object.object.name === queryName &&
+      node.object.property.type === 'Identifier' &&
+      node.object.property.name === 'strings' &&
+      node.property.type === 'Literal' &&
+      node.property.value === 0
+  )
+  assert(legacyStrings.length <= 1, 'postgresQueryPreparation: simple query string changed')
+  return legacyStrings[0]
+}
+
+/**
+ * @param {import('estree').Program} program
+ * @param {import('estree').Statement[]} statements
+ */
+function injectPostgresModuleStatements (program, statements) {
+  let importIndex = program.body.length - 1
+  while (importIndex >= 0) {
+    const statement = program.body[importIndex]
+    if (statement.type === 'ImportDeclaration' || statement.type === 'VariableDeclaration') break
+    importIndex--
+  }
+
+  program.body.splice(importIndex + 1, 0, ...statements)
+}
+
+/**
+ * @param {import('estree').Program} program
+ */
+function findPostgresQueryIdentifier (program) {
+  const identifiers = []
+
+  const imports = query(program, 'ImportDeclaration')
+  for (const declaration of imports) {
+    if (declaration.source.value !== './query.js') continue
+
+    for (const specifier of declaration.specifiers) {
+      if (specifier.type === 'ImportSpecifier' && specifier.imported.name === 'Query') {
+        identifiers.push(specifier.local.name)
+      }
+    }
+  }
+
+  const declarations = query(program, 'VariableDeclarator[id.type="ObjectPattern"]')
+  for (const declaration of declarations) {
+    if (declaration.init?.type !== 'CallExpression' || declaration.init.callee.name !== 'require') continue
+    if (declaration.init.arguments[0]?.value !== './query.js') continue
+
+    for (const property of declaration.id.properties) {
+      if (property.type === 'Property' && property.key.name === 'Query' && property.value.type === 'Identifier') {
+        identifiers.push(property.value.name)
+      }
+    }
+  }
+
+  assert(identifiers.length === 1, 'postgresQueryHandlers: Query import changed')
+  return identifiers[0]
+}
+
+/**
+ * @param {import('estree').Program} program
+ */
+function injectPostgresQueryRegistration (program) {
+  const queryClassIndex = program.body.findIndex(statement =>
+    statement.type === 'VariableDeclaration' &&
+      statement.declarations.some(({ id }) => id.type === 'Identifier' && id.name === 'Query') ||
+      statement.type === 'ExportNamedDeclaration' && statement.declaration?.type === 'ClassDeclaration' &&
+        statement.declaration.id?.name === 'Query'
+  )
+  assert(queryClassIndex !== -1, 'postgresQueryLifecycle: Query class changed')
+
+  const statements = parse(`
+    const ${POSTGRES_QUERY_REGISTRY_SYMBOL} = Symbol.for('dd-trace:postgres:query');
+    const ${POSTGRES_QUERY_REGISTRY} = globalThis[${POSTGRES_QUERY_REGISTRY_SYMBOL}] instanceof WeakSet
+      ? globalThis[${POSTGRES_QUERY_REGISTRY_SYMBOL}]
+      : (globalThis[${POSTGRES_QUERY_REGISTRY_SYMBOL}] = new WeakSet());
+    ${POSTGRES_QUERY_REGISTRY}.add(Query);
+  `).body
+  program.body.splice(queryClassIndex + 1, 0, ...statements)
+}
+
+/**
+ * @param {import('estree').AssignmentExpression} assignment
+ * @param {string} channelVariable
+ */
+function wrapPostgresResolution (assignment, channelVariable) {
+  const resolution = assignment.right
+  assert(
+    resolution.type === 'ArrowFunctionExpression' && resolution.body.type !== 'BlockStatement',
+    'postgresQueryLifecycle: resolve function changed'
+  )
+
+  const originalBody = resolution.body
+  const body = parse(`
+    function wrapper () {
+      if (${channelVariable}.asyncEnd.hasSubscribers && (!this.streaming || !this.active)) {
+        const statement = ${POSTGRES_STATEMENT};
+        ${channelVariable}.asyncEnd.publish({ query: this, statement, pid: this.state?.pid });
+      }
+      return undefined;
+    }
+  `).body[0].body
+  resolution.body = body
+  resolution.expression = false
+  body.body.at(-1).argument = originalBody
+}
+
+/**
+ * @param {import('estree').AssignmentExpression} assignment
+ * @param {string} channelVariable
+ */
+function wrapPostgresRejection (assignment, channelVariable) {
+  const rejection = assignment.right
+  assert(
+    rejection.type === 'ArrowFunctionExpression' &&
+      rejection.body.type !== 'BlockStatement',
+    'postgresQueryLifecycle: reject function changed'
+  )
+  const errorParameter = rejection.params[0]
+  assert(errorParameter?.type === 'Identifier', 'postgresQueryLifecycle: reject parameter changed')
+
+  const originalBody = rejection.body
+  const body = parse(`
+    function wrapper () {
+      if (${channelVariable}.error.hasSubscribers || ${channelVariable}.asyncEnd.hasSubscribers) {
+        const statement = ${POSTGRES_STATEMENT};
+        const __ddTraceContext = { query: this, error: ${errorParameter.name}, statement, pid: this.state?.pid };
+        if (${channelVariable}.error.hasSubscribers) {
+          ${channelVariable}.error.publish(__ddTraceContext);
+        }
+        if (${channelVariable}.asyncEnd.hasSubscribers) {
+          ${channelVariable}.asyncEnd.publish(__ddTraceContext);
+        }
+      }
+      return undefined;
+    }
+  `).body[0].body
+  rejection.body = body
+  rejection.expression = false
+  body.body.at(-1).argument = originalBody
+}
+
+/**
+ * @param {object} state
+ * @param {import('estree').Program} program
+ */
+function injectPostgresTracingChannel (state, program) {
+  state.transforms.tracingChannelDeclaration(state, program)
+
+  const channelName = `orchestrion:${state.module.name}:${state.channelName}`
+  const declarations = query(
+    program,
+    'VariableDeclarator[init.type="CallExpression"][init.callee.name="tr_ch_apm_tracingChannel"]'
+  ).filter(({ init }) => init.arguments[0]?.value === channelName)
+
+  assert(declarations.length === 1, 'postgres: tracing channel declaration changed')
+  assert(declarations[0].id.type === 'Identifier', 'postgres: tracing channel identifier changed')
+  return declarations[0].id.name
+}

package/packages/datadog-instrumentations/src/helpers/rewriter/transforms.js

@@ -41,7 +41,6 @@ module.exports = {
  * @param {import('estree').FunctionExpression} node
  * @param {import('estree').Node} _parent
  * @param {import('estree').Node[]} ancestry
- * @returns {void}
  */
 function publishDurableOrchestrationFailure (state, node, _parent, ancestry) {
   // Class queries also visit the owning ClassDeclaration so Orchestrion can
@@ -108,7 +107,6 @@ function publishDurableOrchestrationFailure (state, node, _parent, ancestry) {
  * } node
  * @param {import('estree').Node} _parent
  * @param {import('estree').Node[]} ancestry
- * @returns {void}
  */
 function awaitContextCallback (state, node, _parent, ancestry) {
   let insertionTarget
@@ -358,7 +356,6 @@ function replaceIdentifier (root, name, replacement) {
  * @param {import('estree').Node} root
  * @param {import('estree').Node} target
  * @param {import('estree').Node[]} statements
- * @returns {boolean}
  */
 function insertBeforeStatement (root, target, statements) {
   for (const key of Object.keys(root)) {
@@ -668,7 +665,6 @@ function configureMercuriusRequest (_state, node) {
  *
  * @param {object} _state
  * @param {import('estree').CallExpression} node
- * @returns {void}
  */
 function waitForAsyncEnd (_state, node) {
   const onFulfilled = node.arguments[0]
@@ -688,7 +684,6 @@ function waitForAsyncEnd (_state, node) {
  * @param {import('estree').BlockStatement} body
  * @param {'ReturnStatement'|'ThrowStatement'} exitType
  * @param {'resolveCallback'|'rejectCallback'} callbackProperty
- * @returns {void}
  */
 function injectAsyncEndCallbackWait (body, exitType, callbackProperty) {
   const callbackVariable = `__apm$${callbackProperty}`

package/packages/datadog-instrumentations/src/helpers/router-helper.js

@@ -141,7 +141,6 @@ function getRouterMountPaths (router) {
 /**
  * @param {object} layer
  * @param {LayerMeta} meta
- * @returns {void}
  */
 function setLayerMeta (layer, meta) {
   layerMeta.set(layer, meta)