dd-trace 5.36.0 → 5.37.0

package/packages/dd-trace/src/ci-visibility/quarantined-tests/get-quarantined-tests.js

@@ -0,0 +1,62 @@
+const request = require('../../exporters/common/request')
+const id = require('../../id')
+
+function getQuarantinedTests ({
+  url,
+  isEvpProxy,
+  evpProxyPrefix,
+  isGzipCompatible,
+  repositoryUrl
+}, done) {
+  const options = {
+    path: '/api/v2/test/libraries/test-management/tests',
+    method: 'POST',
+    headers: {
+      'Content-Type': 'application/json'
+    },
+    timeout: 20000,
+    url
+  }
+
+  if (isGzipCompatible) {
+    options.headers['accept-encoding'] = 'gzip'
+  }
+
+  if (isEvpProxy) {
+    options.path = `${evpProxyPrefix}/api/v2/test/libraries/test-management/tests`
+    options.headers['X-Datadog-EVP-Subdomain'] = 'api'
+  } else {
+    const apiKey = process.env.DATADOG_API_KEY || process.env.DD_API_KEY
+    if (!apiKey) {
+      return done(new Error('Quarantined tests were not fetched because Datadog API key is not defined.'))
+    }
+
+    options.headers['dd-api-key'] = apiKey
+  }
+
+  const data = JSON.stringify({
+    data: {
+      id: id().toString(10),
+      type: 'ci_app_libraries_tests_request',
+      attributes: {
+        repository_url: repositoryUrl
+      }
+    }
+  })
+
+  request(data, options, (err, res) => {
+    if (err) {
+      done(err)
+    } else {
+      try {
+        const { data: { attributes: { modules: quarantinedTests } } } = JSON.parse(res)
+
+        done(null, quarantinedTests)
+      } catch (err) {
+        done(err)
+      }
+    }
+  })
+}
+
+module.exports = { getQuarantinedTests }

package/packages/dd-trace/src/ci-visibility/requests/get-library-configuration.js

@@ -94,7 +94,8 @@ function getLibraryConfiguration ({
               early_flake_detection: earlyFlakeDetectionConfig,
               flaky_test_retries_enabled: isFlakyTestRetriesEnabled,
               di_enabled: isDiEnabled,
-              known_tests_enabled: isKnownTestsEnabled
+              known_tests_enabled: isKnownTestsEnabled,
+              test_management: testManagementConfig
             }
           }
         } = JSON.parse(res)
@@ -111,7 +112,9 @@ function getLibraryConfiguration ({
             earlyFlakeDetectionConfig?.faulty_session_threshold ?? DEFAULT_EARLY_FLAKE_DETECTION_ERROR_THRESHOLD,
           isFlakyTestRetriesEnabled,
           isDiEnabled: isDiEnabled && isFlakyTestRetriesEnabled,
-          isKnownTestsEnabled
+          isKnownTestsEnabled,
+          // TODO: should it be test management?
+          isQuarantinedTestsEnabled: (testManagementConfig?.enabled ?? false)
         }
 
         log.debug(() => `Remote settings: ${JSON.stringify(settings)}`)

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

@@ -497,6 +497,7 @@ class Config {
     this._setValue(defaults, 'iast.redactionNamePattern', null)
     this._setValue(defaults, 'iast.redactionValuePattern', null)
     this._setValue(defaults, 'iast.requestSampling', 30)
+    this._setValue(defaults, 'iast.securityControlsConfiguration', null)
     this._setValue(defaults, 'iast.telemetryVerbosity', 'INFORMATION')
     this._setValue(defaults, 'iast.stackTrace.enabled', true)
     this._setValue(defaults, 'injectionEnabled', [])
@@ -519,6 +520,8 @@ class Config {
     this._setValue(defaults, 'legacyBaggageEnabled', true)
     this._setValue(defaults, 'isTestDynamicInstrumentationEnabled', false)
     this._setValue(defaults, 'isServiceUserProvided', false)
+    this._setValue(defaults, 'testManagementAttemptToFixRetries', 20)
+    this._setValue(defaults, 'isTestManagementEnabled', false)
     this._setValue(defaults, 'logInjection', false)
     this._setValue(defaults, 'lookup', undefined)
     this._setValue(defaults, 'inferredProxyServicesEnabled', false)
@@ -625,6 +628,7 @@ class Config {
       DD_IAST_REDACTION_NAME_PATTERN,
       DD_IAST_REDACTION_VALUE_PATTERN,
       DD_IAST_REQUEST_SAMPLING,
+      DD_IAST_SECURITY_CONTROLS_CONFIGURATION,
       DD_IAST_TELEMETRY_VERBOSITY,
       DD_IAST_STACK_TRACE_ENABLED,
       DD_INJECTION_ENABLED,
@@ -793,6 +797,7 @@ class Config {
       this._setValue(env, 'iast.requestSampling', iastRequestSampling)
     }
     this._envUnprocessed['iast.requestSampling'] = DD_IAST_REQUEST_SAMPLING
+    this._setString(env, 'iast.securityControlsConfiguration', DD_IAST_SECURITY_CONTROLS_CONFIGURATION)
     this._setString(env, 'iast.telemetryVerbosity', DD_IAST_TELEMETRY_VERBOSITY)
     this._setBoolean(env, 'iast.stackTrace.enabled', DD_IAST_STACK_TRACE_ENABLED)
     this._setArray(env, 'injectionEnabled', DD_INJECTION_ENABLED)
@@ -985,14 +990,15 @@ class Config {
       this._setValue(opts, 'iast.requestSampling', iastRequestSampling)
       this._optsUnprocessed['iast.requestSampling'] = options.iast?.requestSampling
     }
-    this._setString(opts, 'iast.telemetryVerbosity', options.iast && options.iast.telemetryVerbosity)
+    this._setValue(opts, 'iast.securityControlsConfiguration', options.iast?.securityControlsConfiguration)
     this._setBoolean(opts, 'iast.stackTrace.enabled', options.iast?.stackTrace?.enabled)
+    this._setString(opts, 'iast.telemetryVerbosity', options.iast && options.iast.telemetryVerbosity)
     this._setBoolean(opts, 'isCiVisibility', options.isCiVisibility)
     this._setBoolean(opts, 'legacyBaggageEnabled', options.legacyBaggageEnabled)
     this._setBoolean(opts, 'llmobs.agentlessEnabled', options.llmobs?.agentlessEnabled)
     this._setString(opts, 'llmobs.mlApp', options.llmobs?.mlApp)
     this._setBoolean(opts, 'logInjection', options.logInjection)
-    this._setString(opts, 'lookup', options.lookup)
+    this._setValue(opts, 'lookup', options.lookup)
     this._setBoolean(opts, 'middlewareTracingEnabled', options.middlewareTracingEnabled)
     this._setBoolean(opts, 'openAiLogsEnabled', options.openAiLogsEnabled)
     this._setValue(opts, 'peerServiceMapping', options.peerServiceMapping)
@@ -1138,7 +1144,9 @@ class Config {
       DD_CIVISIBILITY_FLAKY_RETRY_COUNT,
       DD_TEST_SESSION_NAME,
       DD_AGENTLESS_LOG_SUBMISSION_ENABLED,
-      DD_TEST_DYNAMIC_INSTRUMENTATION_ENABLED
+      DD_TEST_DYNAMIC_INSTRUMENTATION_ENABLED,
+      DD_TEST_MANAGEMENT_ENABLED,
+      DD_TEST_MANAGEMENT_ATTEMPT_TO_FIX_RETRIES
     } = process.env
 
     if (DD_CIVISIBILITY_AGENTLESS_URL) {
@@ -1158,6 +1166,11 @@ class Config {
       this._setBoolean(calc, 'ciVisAgentlessLogSubmissionEnabled', isTrue(DD_AGENTLESS_LOG_SUBMISSION_ENABLED))
       this._setBoolean(calc, 'isTestDynamicInstrumentationEnabled', isTrue(DD_TEST_DYNAMIC_INSTRUMENTATION_ENABLED))
       this._setBoolean(calc, 'isServiceUserProvided', !!this._env.service)
+      this._setBoolean(calc, 'isTestManagementEnabled', !isFalse(DD_TEST_MANAGEMENT_ENABLED))
+      this._setValue(calc,
+        'testManagementAttemptToFixRetries',
+        coalesce(maybeInt(DD_TEST_MANAGEMENT_ATTEMPT_TO_FIX_RETRIES), 20)
+      )
     }
     this._setString(calc, 'dogstatsd.hostname', this._getHostname())
     this._setBoolean(calc, 'isGitUploadEnabled',

package/packages/dd-trace/src/debugger/devtools_client/breakpoints.js

@@ -1,5 +1,6 @@
 'use strict'
 
+const { getGeneratedPosition } = require('./source-maps')
 const session = require('./session')
 const { MAX_SNAPSHOTS_PER_SECOND_PER_PROBE, MAX_NON_SNAPSHOTS_PER_SECOND_PER_PROBE } = require('./defaults')
 const { findScriptFromPartialPath, probes, breakpoints } = require('./state')
@@ -16,10 +17,11 @@ async function addBreakpoint (probe) {
   if (!sessionStarted) await start()
 
   const file = probe.where.sourceFile
-  const line = Number(probe.where.lines[0]) // Tracer doesn't support multiple-line breakpoints
+  let lineNumber = Number(probe.where.lines[0]) // Tracer doesn't support multiple-line breakpoints
+  let columnNumber = 0 // Probes do not contain/support column information
 
   // Optimize for sending data to /debugger/v1/input endpoint
-  probe.location = { file, lines: [String(line)] }
+  probe.location = { file, lines: [String(lineNumber)] }
   delete probe.where
 
   // Optimize for fast calculations when probe is hit
@@ -34,17 +36,22 @@ async function addBreakpoint (probe) {
   // not continue untill all scripts have been parsed?
   const script = findScriptFromPartialPath(file)
   if (!script) throw new Error(`No loaded script found for ${file} (probe: ${probe.id}, version: ${probe.version})`)
-  const [path, scriptId] = script
+  const { url, scriptId, sourceMapURL, source } = script
+
+  if (sourceMapURL) {
+    ({ line: lineNumber, column: columnNumber } = await getGeneratedPosition(url, source, lineNumber, sourceMapURL))
+  }
 
   log.debug(
-    '[debugger:devtools_client] Adding breakpoint at %s:%d (probe: %s, version: %d)',
-    path, line, probe.id, probe.version
+    '[debugger:devtools_client] Adding breakpoint at %s:%d:%d (probe: %s, version: %d)',
+    url, lineNumber, columnNumber, probe.id, probe.version
   )
 
   const { breakpointId } = await session.post('Debugger.setBreakpoint', {
     location: {
       scriptId,
-      lineNumber: line - 1 // Beware! lineNumber is zero-indexed
+      lineNumber: lineNumber - 1, // Beware! lineNumber is zero-indexed
+      columnNumber
     }
   })
 
@@ -66,7 +73,7 @@ async function removeBreakpoint ({ id }) {
   probes.delete(id)
   breakpoints.delete(breakpointId)
 
-  if (breakpoints.size === 0) await stop()
+  if (breakpoints.size === 0) return stop() // return instead of await to reduce number of promises created
 }
 
 async function start () {

package/packages/dd-trace/src/debugger/devtools_client/source-maps.js

@@ -0,0 +1,50 @@
+'use strict'
+
+const { join, dirname } = require('path')
+const { readFileSync } = require('fs')
+const { readFile } = require('fs/promises')
+const { SourceMapConsumer } = require('source-map')
+
+const cache = new Map()
+let cacheTimer = null
+
+const self = module.exports = {
+  async loadSourceMap (dir, url) {
+    if (url.startsWith('data:')) return loadInlineSourceMap(url)
+    const path = join(dir, url)
+    if (cache.has(path)) return cache.get(path)
+    return cacheIt(path, JSON.parse(await readFile(path, 'utf8')))
+  },
+
+  loadSourceMapSync (dir, url) {
+    if (url.startsWith('data:')) return loadInlineSourceMap(url)
+    const path = join(dir, url)
+    if (cache.has(path)) return cache.get(path)
+    return cacheIt(path, JSON.parse(readFileSync(path, 'utf8')))
+  },
+
+  async getGeneratedPosition (url, source, line, sourceMapURL) {
+    const dir = dirname(new URL(url).pathname)
+    return await SourceMapConsumer.with(
+      await self.loadSourceMap(dir, sourceMapURL),
+      null,
+      (consumer) => consumer.generatedPositionFor({ source, line, column: 0 })
+    )
+  }
+}
+
+function cacheIt (key, value) {
+  clearTimeout(cacheTimer)
+  cacheTimer = setTimeout(function () {
+    // Optimize for app boot, where a lot of reads might happen
+    // Clear cache a few seconds after it was last used
+    cache.clear()
+  }, 10_000).unref()
+  cache.set(key, value)
+  return value
+}
+
+function loadInlineSourceMap (data) {
+  data = data.slice(data.indexOf('base64,') + 7)
+  return JSON.parse(Buffer.from(data, 'base64').toString('utf8'))
+}

package/packages/dd-trace/src/debugger/devtools_client/state.js

@@ -1,10 +1,13 @@
 'use strict'
 
+const { join, dirname } = require('path')
+const { loadSourceMapSync } = require('./source-maps')
 const session = require('./session')
+const log = require('../../log')
 
 const WINDOWS_DRIVE_LETTER_REGEX = /[a-zA-Z]/
 
-const scriptIds = []
+const loadedScripts = []
 const scriptUrls = new Map()
 
 module.exports = {
@@ -15,18 +18,17 @@ module.exports = {
    * Find the script to inspect based on a partial or absolute path. Handles both Windows and POSIX paths.
    *
    * @param {string} path - Partial or absolute path to match against loaded scripts
-   * @returns {[string, string, string | undefined] | null} - Array containing [url, scriptId, sourceMapURL]
-   *   or null if no match
+   * @returns {Object | null} - Object containing `url`, `scriptId`, `sourceMapURL`, and `source` - or null if no match
    */
   findScriptFromPartialPath (path) {
     if (!path) return null // This shouldn't happen, but better safe than sorry
 
     path = path.toLowerCase()
 
-    const bestMatch = new Array(3)
+    const bestMatch = { url: null, scriptId: null, sourceMapURL: null, source: null }
     let maxMatchLength = -1
 
-    for (const [url, scriptId, sourceMapURL] of scriptIds) {
+    for (const { url, sourceUrl, scriptId, sourceMapURL, source } of loadedScripts) {
       let i = url.length - 1
       let j = path.length - 1
       let matchLength = 0
@@ -75,12 +77,13 @@ module.exports = {
       // If we found a valid match and it's better than our previous best
       if (atBoundary && (
         lastBoundaryPos > maxMatchLength ||
-        (lastBoundaryPos === maxMatchLength && url.length < bestMatch[0].length) // Prefer shorter paths
+        (lastBoundaryPos === maxMatchLength && url.length < bestMatch.url.length) // Prefer shorter paths
       )) {
         maxMatchLength = lastBoundaryPos
-        bestMatch[0] = url
-        bestMatch[1] = scriptId
-        bestMatch[2] = sourceMapURL
+        bestMatch.url = sourceUrl || url
+        bestMatch.scriptId = scriptId
+        bestMatch.sourceMapURL = sourceMapURL
+        bestMatch.source = source
       }
     }
 
@@ -112,6 +115,31 @@ module.exports = {
 session.on('Debugger.scriptParsed', ({ params }) => {
   scriptUrls.set(params.scriptId, params.url)
   if (params.url.startsWith('file:')) {
-    scriptIds.push([params.url, params.scriptId, params.sourceMapURL])
+    if (params.sourceMapURL) {
+      const dir = dirname(new URL(params.url).pathname)
+      let sources
+      try {
+        sources = loadSourceMapSync(dir, params.sourceMapURL).sources
+      } catch (err) {
+        if (typeof params.sourceMapURL === 'string' && params.sourceMapURL.startsWith('data:')) {
+          log.error('[debugger:devtools_client] could not load inline source map for "%s"', params.url, err)
+        } else {
+          log.error('[debugger:devtools_client] could not load source map "%s" from "%s" for "%s"',
+            params.sourceMapURL, dir, params.url, err)
+        }
+        return
+      }
+      for (const source of sources) {
+        // TODO: Take source map `sourceRoot` into account?
+        loadedScripts.push({
+          ...params,
+          sourceUrl: params.url,
+          url: new URL(join(dir, source), 'file:').href,
+          source
+        })
+      }
+    } else {
+      loadedScripts.push(params)
+    }
   }
 })

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

@@ -1,11 +1,11 @@
 'use strict'
 
-const semver = require('semver')
+const satisfies = require('semifies')
 const logger = require('./log')
 const { addHook } = require('import-in-the-middle')
 const dc = require('dc-polyfill')
 
-if (semver.satisfies(process.versions.node, '>=14.13.1')) {
+if (satisfies(process.versions.node, '>=14.13.1')) {
   const moduleLoadStartChannel = dc.channel('dd-trace:moduleLoadStart')
   addHook((name, namespace) => {
     if (moduleLoadStartChannel.hasSubscribers) {

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

@@ -100,7 +100,12 @@ class LLMObsTagger {
   }
 
   tagMetadata (span, metadata) {
-    this._setTag(span, METADATA, metadata)
+    const existingMetadata = registry.get(span)?.[METADATA]
+    if (existingMetadata) {
+      Object.assign(existingMetadata, metadata)
+    } else {
+      this._setTag(span, METADATA, metadata)
+    }
   }
 
   tagMetrics (span, metrics) {
@@ -128,7 +133,12 @@ class LLMObsTagger {
       }
     }
 
-    this._setTag(span, METRICS, filterdMetrics)
+    const existingMetrics = registry.get(span)?.[METRICS]
+    if (existingMetrics) {
+      Object.assign(existingMetrics, filterdMetrics)
+    } else {
+      this._setTag(span, METRICS, filterdMetrics)
+    }
   }
 
   tagSpanTags (span, tags) {

package/packages/dd-trace/src/opentracing/span.js

@@ -4,7 +4,7 @@
 const { performance } = require('perf_hooks')
 const now = performance.now.bind(performance)
 const dateNow = Date.now
-const semver = require('semver')
+const satisfies = require('semifies')
 const SpanContext = require('./span_context')
 const id = require('../id')
 const tagger = require('../tagger')
@@ -365,7 +365,7 @@ class DatadogSpan {
 }
 
 function createRegistry (type) {
-  if (!semver.satisfies(process.version, '>=14.6')) return
+  if (!satisfies(process.version, '>=14.6')) return
 
   return new global.FinalizationRegistry(name => {
     runtimeMetrics.decrement(`runtime.node.spans.${type}`)

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

@@ -50,7 +50,7 @@ module.exports = class CiPlugin extends Plugin {
 
     this.addSub(`ci:${this.constructor.id}:library-configuration`, ({ onDone }) => {
       if (!this.tracer._exporter || !this.tracer._exporter.getLibraryConfiguration) {
-        return onDone({ err: new Error('CI Visibility was not initialized correctly') })
+        return onDone({ err: new Error('Test optimization was not initialized correctly') })
       }
       this.tracer._exporter.getLibraryConfiguration(this.testConfiguration, (err, libraryConfig) => {
         if (err) {
@@ -64,7 +64,7 @@ module.exports = class CiPlugin extends Plugin {
 
     this.addSub(`ci:${this.constructor.id}:test-suite:skippable`, ({ onDone }) => {
       if (!this.tracer._exporter?.getSkippableSuites) {
-        return onDone({ err: new Error('CI Visibility was not initialized correctly') })
+        return onDone({ err: new Error('Test optimization was not initialized correctly') })
       }
       this.tracer._exporter.getSkippableSuites(this.testConfiguration, (err, skippableSuites, itrCorrelationId) => {
         if (err) {
@@ -153,7 +153,7 @@ module.exports = class CiPlugin extends Plugin {
 
     this.addSub(`ci:${this.constructor.id}:known-tests`, ({ onDone }) => {
       if (!this.tracer._exporter?.getKnownTests) {
-        return onDone({ err: new Error('CI Visibility was not initialized correctly') })
+        return onDone({ err: new Error('Test optimization was not initialized correctly') })
       }
       this.tracer._exporter.getKnownTests(this.testConfiguration, (err, knownTests) => {
         if (err) {
@@ -164,6 +164,19 @@ module.exports = class CiPlugin extends Plugin {
         onDone({ err, knownTests })
       })
     })
+
+    this.addSub(`ci:${this.constructor.id}:quarantined-tests`, ({ onDone }) => {
+      if (!this.tracer._exporter?.getQuarantinedTests) {
+        return onDone({ err: new Error('Test optimization was not initialized correctly') })
+      }
+      this.tracer._exporter.getQuarantinedTests(this.testConfiguration, (err, quarantinedTests) => {
+        if (err) {
+          log.error('Quarantined tests could not be fetched. %s', err.message)
+          this.libraryConfig.isQuarantinedTestsEnabled = false
+        }
+        onDone({ err, quarantinedTests })
+      })
+    })
   }
 
   get telemetry () {

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

@@ -63,25 +63,35 @@ class DatabasePlugin extends StoragePlugin {
     return tracerService
   }
 
-  injectDbmQuery (span, query, serviceName, isPreparedStatement = false) {
+  createDbmComment (span, serviceName, isPreparedStatement = false) {
     const mode = this.config.dbmPropagationMode
     const dbmService = this.getDbmServiceName(span, serviceName)
 
     if (mode === 'disabled') {
-      return query
+      return null
     }
 
     const servicePropagation = this.createDBMPropagationCommentService(dbmService, span)
 
     if (isPreparedStatement || mode === 'service') {
-      return `/*${servicePropagation}*/ ${query}`
+      return servicePropagation
     } else if (mode === 'full') {
       span.setTag('_dd.dbm_trace_injected', 'true')
       const traceparent = span._spanContext.toTraceparent()
-      return `/*${servicePropagation},traceparent='${traceparent}'*/ ${query}`
+      return `${servicePropagation},traceparent='${traceparent}'`
     }
   }
 
+  injectDbmQuery (span, query, serviceName, isPreparedStatement = false) {
+    const dbmTraceComment = this.createDbmComment(span, serviceName, isPreparedStatement)
+
+    if (!dbmTraceComment) {
+      return query
+    }
+
+    return `/*${dbmTraceComment}*/ ${query}`
+  }
+
   maybeTruncate (query) {
     const maxLength = typeof this.config.truncate === 'number'
       ? this.config.truncate

package/packages/dd-trace/src/plugins/util/inferred_proxy.js

@@ -2,7 +2,6 @@ const log = require('../../log')
 const tags = require('../../../../../ext/tags')
 
 const RESOURCE_NAME = tags.RESOURCE_NAME
-const SPAN_KIND = tags.SPAN_KIND
 const SPAN_TYPE = tags.SPAN_TYPE
 const HTTP_URL = tags.HTTP_URL
 const HTTP_METHOD = tags.HTTP_METHOD
@@ -49,7 +48,6 @@ function createInferredProxySpan (headers, childOf, tracer, context) {
       tags: {
         service: proxyContext.domainName || tracer._config.service,
         component: proxySpanInfo.component,
-        [SPAN_KIND]: 'internal',
         [SPAN_TYPE]: 'web',
         [HTTP_METHOD]: proxyContext.method,
         [HTTP_URL]: proxyContext.domainName + proxyContext.path,
@@ -71,7 +69,7 @@ function createInferredProxySpan (headers, childOf, tracer, context) {
 
 function setInferredProxySpanTags (span, proxyContext) {
   span.setTag(RESOURCE_NAME, `${proxyContext.method} ${proxyContext.path}`)
-  span.setTag('_dd.inferred_span', '1')
+  span.setTag('_dd.inferred_span', 1)
   return span
 }
 

package/packages/dd-trace/src/plugins/util/test.js

@@ -52,8 +52,6 @@ const TEST_MODULE_ID = 'test_module_id'
 const TEST_SUITE_ID = 'test_suite_id'
 const TEST_TOOLCHAIN = 'test.toolchain'
 const TEST_SKIPPED_BY_ITR = 'test.skipped_by_itr'
-// Browser used in browser test. Namespaced by test.configuration because it affects the fingerprint
-const TEST_CONFIGURATION_BROWSER_NAME = 'test.configuration.browser_name'
 // Early flake detection
 const TEST_IS_NEW = 'test.is_new'
 const TEST_IS_RETRY = 'test.is_retry'
@@ -117,6 +115,9 @@ const DI_DEBUG_ERROR_SNAPSHOT_ID_SUFFIX = 'snapshot_id'
 const DI_DEBUG_ERROR_FILE_SUFFIX = 'file'
 const DI_DEBUG_ERROR_LINE_SUFFIX = 'line'
 
+const TEST_MANAGEMENT_IS_QUARANTINED = 'test.test_management.is_quarantined'
+const TEST_MANAGEMENT_ENABLED = 'test.test_management.enabled'
+
 module.exports = {
   TEST_CODE_OWNERS,
   TEST_SESSION_NAME,
@@ -143,7 +144,6 @@ module.exports = {
   MOCHA_WORKER_TRACE_PAYLOAD_CODE,
   TEST_SOURCE_START,
   TEST_SKIPPED_BY_ITR,
-  TEST_CONFIGURATION_BROWSER_NAME,
   TEST_IS_NEW,
   TEST_IS_RETRY,
   TEST_EARLY_FLAKE_ENABLED,
@@ -202,7 +202,9 @@ module.exports = {
   DI_DEBUG_ERROR_FILE_SUFFIX,
   DI_DEBUG_ERROR_LINE_SUFFIX,
   getFormattedError,
-  DD_TEST_IS_USER_PROVIDED_SERVICE
+  DD_TEST_IS_USER_PROVIDED_SERVICE,
+  TEST_MANAGEMENT_IS_QUARANTINED,
+  TEST_MANAGEMENT_ENABLED
 }
 
 // Returns pkg manager and its version, separated by '-', e.g. npm-8.15.0 or yarn-1.22.19

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

@@ -201,7 +201,11 @@ class Tracer extends NoopProxy {
     try {
       return require('./profiler').start(config)
     } catch (e) {
-      log.error('Error starting profiler', e)
+      log.error(
+        'Error starting profiler. For troubleshooting tips, see ' +
+        '<https://dtdg.co/nodejs-profiler-troubleshooting>',
+        e
+      )
     }
   }
 

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

@@ -94,10 +94,11 @@ function Hook (modules, options, onrequire) {
     if (moduleLoadStartChannel.hasSubscribers) {
       moduleLoadStartChannel.publish(payload)
     }
-    const exports = origRequire.apply(this, arguments)
+    let exports = origRequire.apply(this, arguments)
     payload.module = exports
     if (moduleLoadEndChannel.hasSubscribers) {
       moduleLoadEndChannel.publish(payload)
+      exports = payload.module
     }
 
     // The module has already been loaded,

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

@@ -83,7 +83,6 @@ module.exports.addSpan = function (span) {
 
   const now = Date.now()
   const expiration = now + LIFETIME
-  // eslint-disable-next-line no-undef
   const wrapped = new WeakRef(span)
   spans.add(wrapped, expiration)
   // registry.register(span, span._name)

package/packages/memwatch/package.json

@@ -1,9 +0,0 @@
-{
-  "name": "memwatch",
-  "version": "1.0.0",
-  "license": "BSD-3-Clause",
-  "private": true,
-  "dependencies": {
-    "@airbnb/node-memwatch": "^1.0.2"
-  }
-}