dd-trace 6.16.0 → 6.17.0

package/packages/dd-trace/src/ci-visibility/exporters/ci-visibility-exporter.js

@@ -6,6 +6,10 @@ const URL = require('url').URL
 
 const { version: tracerVersion } = require('../../../../../package.json')
 const { EMPTY_EFD_RETRY_POLICY, createEfdRetryPolicy } = require('../efd-retry-policy')
+const {
+  getDynamicAtrBuckets,
+  isDynamicAtrEnabled,
+} = require('../dynamic-atr-retries')
 const { getLibraryConfiguration: getLibraryConfigurationRequest } = require('../requests/get-library-configuration')
 const { getCachePath, withCache, writeToCache } = require('../requests/fs-cache')
 const { getSkippableSuites: getSkippableSuitesRequest } = require('../intelligent-test-runner/get-skippable-suites')
@@ -31,7 +35,7 @@ const {
   FINAL_FLUSH_FALLBACK_DELAY,
   FINAL_FLUSH_TIMEOUT,
 } = require('../final-flush')
-const { incrementCountMetric, TELEMETRY_ENDPOINT_PAYLOAD_DROPPED } = require('../telemetry')
+const { incrementCountMetric, recordDynamicAtrRetries, TELEMETRY_ENDPOINT_PAYLOAD_DROPPED } = require('../telemetry')
 const { sendGitMetadata: sendGitMetadataRequest } = require('./git/git_metadata')
 const buildSettingsCacheKey = require('./settings-cache-key')
 
@@ -108,7 +112,6 @@ function isValidRetryCount (value) {
  * Checks whether a cached EFD retry policy has the complete parsed shape.
  *
  * @param {unknown} retryPolicy - Candidate retry policy.
- * @returns {boolean}
  */
 function isValidCachedEfdRetryPolicy (retryPolicy) {
   if (retryPolicy === null || typeof retryPolicy !== 'object' || Array.isArray(retryPolicy)) return false
@@ -208,6 +211,11 @@ class CiVisibilityExporter extends BufferingExporter {
     this._coverageTimer = undefined
     this._logsTimer = undefined
     this._coverageBuffer = []
+    const dynamicAtrEnabled = config?.testOptimization?.DD_CIVISIBILITY_DYNAMIC_ATR_ENABLED
+    const dynamicAtrBuckets = config?.testOptimization?.DD_CIVISIBILITY_DYNAMIC_ATR_BUCKETS
+    this._dynamicAtrEnabled = isDynamicAtrEnabled(dynamicAtrEnabled)
+    this._dynamicAtrBuckets = this._dynamicAtrEnabled ? getDynamicAtrBuckets(dynamicAtrBuckets) ?? undefined : undefined
+    this._hasRecordedDynamicAtrTelemetry = false
     this._testOptimizationHttpCache = options.testOptimizationHttpCache || new TestOptimizationHttpCache()
     this._isTestOptimizationCacheOnly = options.cacheOnly === true
     const coverageReportFlags = parsers.ARRAY(config?.testOptimization?.DD_CODE_COVERAGE_FLAGS)
@@ -357,7 +365,6 @@ class CiVisibilityExporter extends BufferingExporter {
    *
    * @param {TestConfiguration} testConfiguration
    * @param {(error: Error | null, libraryConfig?: Readonly<Record<string, unknown>>) => void} callback
-   * @returns {void}
    */
   getLibraryConfiguration (testConfiguration, callback) {
     const { repositoryUrl } = testConfiguration
@@ -410,6 +417,7 @@ class CiVisibilityExporter extends BufferingExporter {
           // `_gitUploadPromise` itself. Do not call `_resolveGit()`.
           writeSettingsToCache(libraryConfig)
           this._libraryConfig = this.filterConfiguration(libraryConfig)
+          this._recordDynamicAtrTelemetry()
           return callback(null, this._libraryConfig)
         }
         // Filesystem cache hit: no git upload was started in this process.
@@ -444,11 +452,11 @@ class CiVisibilityExporter extends BufferingExporter {
    * @param {string} repositoryUrl - Repository URL for git metadata upload.
    * @param {boolean} isFilesystemCache - Whether settings came from the cross-process cache.
    * @param {Function} callback - Completion callback.
-   * @returns {void}
    */
   _applyCachedSettings (settings, configuration, repositoryUrl, isFilesystemCache, callback) {
     writeSettingsToCache(settings)
     this._libraryConfig = this.filterConfiguration(settings)
+    this._recordDynamicAtrTelemetry()
     const canUseCachedSkippableSuites = !this.shouldRequestSkippableSuites() ||
       this._testOptimizationHttpCache.hasValidSkippableSuites({
         testLevel: configuration.testLevel,
@@ -471,7 +479,6 @@ class CiVisibilityExporter extends BufferingExporter {
    * @param {string} repositoryUrl - Repository URL for git metadata upload.
    * @param {string|null} cacheKey - Filesystem cache key when this process owns the lock, null otherwise.
    * @param {Function} done - Completion callback.
-   * @returns {void}
    */
   _fetchLibraryConfigurationFromBackend (configuration, repositoryUrl, cacheKey, done) {
     this.sendGitMetadata(repositoryUrl)
@@ -532,6 +539,7 @@ class CiVisibilityExporter extends BufferingExporter {
       DD_TEST_MANAGEMENT_ATTEMPT_TO_FIX_RETRIES: configuredAttemptToFixRetries = 0,
       DD_TEST_MANAGEMENT_ENABLED: isTestManagementAllowed,
     } = testOptimization
+
     const earlyFlakeDetectionRetryPolicy = earlyFlakeDetectionRetryCount === undefined
       ? remoteConfiguration.earlyFlakeDetectionRetryPolicy ?? EMPTY_EFD_RETRY_POLICY
       : createEfdRetryPolicy({
@@ -543,6 +551,20 @@ class CiVisibilityExporter extends BufferingExporter {
     const testManagementAttemptToFixRetries =
       remoteConfiguration.testManagementAttemptToFixRetries ?? configuredAttemptToFixRetries
 
+    const isFlakyTestRetriesEnabled =
+      remoteConfiguration.isFlakyTestRetriesEnabled === true && isFlakyTestRetriesAllowed === true
+    // Dynamic ATR only replaces the flat ATR policy when backend ATR is enabled.
+    const isDynamicAtrEnabled = isFlakyTestRetriesEnabled && this._dynamicAtrEnabled
+    let dynamicAtrBuckets
+    if (isDynamicAtrEnabled) {
+      // Resolve the backend fallback before local EFD overrides can reach any runner.
+      const backendPolicy = remoteConfiguration.earlyFlakeDetectionRetryPolicy ?? EMPTY_EFD_RETRY_POLICY
+      dynamicAtrBuckets = this._dynamicAtrBuckets ?? Object.freeze([
+        ...backendPolicy.durationRetryCounts.map(({ retryCount }) => Math.max(1, retryCount)),
+        1,
+      ])
+    }
+
     return Object.freeze({
       isCodeCoverageEnabled: remoteConfiguration.isCodeCoverageEnabled === true,
       isSuitesSkippingEnabled: remoteConfiguration.isSuitesSkippingEnabled === true,
@@ -552,9 +574,10 @@ class CiVisibilityExporter extends BufferingExporter {
         remoteConfiguration.isEarlyFlakeDetectionEnabled === true && isEarlyFlakeDetectionAllowed === true,
       earlyFlakeDetectionRetryPolicy,
       earlyFlakeDetectionFaultyThreshold: remoteConfiguration.earlyFlakeDetectionFaultyThreshold ?? 30,
-      isFlakyTestRetriesEnabled:
-        remoteConfiguration.isFlakyTestRetriesEnabled === true && isFlakyTestRetriesAllowed === true,
+      isFlakyTestRetriesEnabled,
       flakyTestRetriesCount,
+      isDynamicAtrEnabled,
+      dynamicAtrBuckets,
       isDiEnabled: remoteConfiguration.isDiEnabled === true && isFailedTestReplayAllowed === true,
       isKnownTestsEnabled: remoteConfiguration.isKnownTestsEnabled === true,
       isTestManagementEnabled:
@@ -566,6 +589,13 @@ class CiVisibilityExporter extends BufferingExporter {
     })
   }
 
+  _recordDynamicAtrTelemetry () {
+    if (!this._hasRecordedDynamicAtrTelemetry && this._libraryConfig?.isDynamicAtrEnabled) {
+      this._hasRecordedDynamicAtrTelemetry = true
+      recordDynamicAtrRetries(this._dynamicAtrBuckets !== undefined)
+    }
+  }
+
   sendGitMetadata (repositoryUrl) {
     if (!this._config.testOptimization.DD_CIVISIBILITY_GIT_UPLOAD_ENABLED) {
       this._resolveGit()
@@ -607,7 +637,6 @@ class CiVisibilityExporter extends BufferingExporter {
    * Exports spans that are not retained for late updates to session, module, or suite events.
    *
    * @param {Array<object>} trace
-   * @returns {void}
    */
   #exportTrace (trace) {
     // Until it's initialized, we just store the traces as is
@@ -633,7 +662,6 @@ class CiVisibilityExporter extends BufferingExporter {
    * Retries session, module, and suite traces rejected by writer backpressure within the final deadline.
    *
    * @param {{ deadline?: number }} options final-flush options
-   * @returns {void}
    */
   #exportDeferredTestSessionTraces (options) {
     if (!this._writer || !this.canReportSessionTraces()) return
@@ -859,7 +887,6 @@ class CiVisibilityExporter extends BufferingExporter {
   /**
    * Allows later test activity to establish a new finalization boundary.
    *
-   * @returns {void}
    */
   #resetFinalFlush () {
     this.#finalFlush = undefined
@@ -933,7 +960,6 @@ class CiVisibilityExporter extends BufferingExporter {
   /**
    * Returns whether the exporter can upload test failure screenshots.
    *
-   * @returns {boolean}
    */
   canUploadTestScreenshots () {
     return Boolean(this._testScreenshotUploadUrl) && this._isTestFailureScreenshotsEnabled
@@ -942,7 +968,6 @@ class CiVisibilityExporter extends BufferingExporter {
   /**
    * Returns whether the exporter can upload test failure videos.
    *
-   * @returns {boolean}
    */
   canUploadTestVideos () {
     return Boolean(this._testScreenshotUploadUrl) && this._isTestFailureVideosEnabled
@@ -985,7 +1010,6 @@ class CiVisibilityExporter extends BufferingExporter {
    * @param {number} options.capturedAtMs - Capture time in epoch milliseconds
    * @param {AbortSignal} [options.signal] - Additional signal used to cancel the upload
    * @param {Function} callback - Callback function (err)
-   * @returns {void}
    */
   uploadTestVideo (options, callback) {
     if (!this._testScreenshotUploadUrl) {
@@ -1006,7 +1030,6 @@ class CiVisibilityExporter extends BufferingExporter {
    * @param {number} options.capturedAtMs - Capture time in epoch milliseconds
    * @param {AbortSignal} [options.signal] - Additional signal used to cancel the upload
    * @param {Function} callback - Callback function (err)
-   * @returns {void}
    */
   uploadTestSuiteVideo (options, callback) {
     if (!this._testScreenshotUploadUrl) {
@@ -1024,7 +1047,6 @@ class CiVisibilityExporter extends BufferingExporter {
    * @param {AbortSignal} [uploadOptions.signal] - Additional signal used to cancel the upload
    * @param {Function} callback - Callback function (err)
    * @param {number} [timeoutMs] - Maximum background upload duration
-   * @returns {void}
    */
   #uploadTestMedia (uploadRequest, uploadOptions, callback, timeoutMs = FINAL_FLUSH_TIMEOUT) {
     const { signal } = uploadOptions

package/packages/dd-trace/src/ci-visibility/exporters/git/git_metadata.js

@@ -70,7 +70,6 @@ function getCommonRequestOptions (url) {
  *
  * @param {GitUploadTarget & { latestCommits: string[] }} options
  * @param {(error: Error | null, commitsToUpload?: string[]) => void} callback
- * @returns {void}
  */
 function getCommitsToUpload ({ url, repositoryUrl, latestCommits, isEvpProxy, evpProxyPrefix }, callback) {
   const commonOptions = getCommonRequestOptions(url)
@@ -139,7 +138,6 @@ function getCommitsToUpload ({ url, repositoryUrl, latestCommits, isEvpProxy, ev
  *
  * @param {GitUploadTarget & { packFileToUpload: string, headCommit: string }} options
  * @param {(error: Error | null, uploadSize?: number) => void} callback
- * @returns {void}
  */
 function uploadPackFile ({ url, isEvpProxy, evpProxyPrefix, packFileToUpload, repositoryUrl, headCommit }, callback) {
   const form = new FormData()
@@ -263,7 +261,6 @@ function generateAndUploadPackFiles ({
  * @param {EvpProxyConfiguration} evpProxyConfiguration
  * @param {string | undefined} configRepositoryUrl repository URL from the configuration, if set
  * @param {(error?: Error | null) => void} callback
- * @returns {void}
  */
 function sendGitMetadata (url, { isEvpProxy, evpProxyPrefix }, configRepositoryUrl, callback) {
   if (!isGitAvailable()) {

package/packages/dd-trace/src/ci-visibility/exporters/request.js

@@ -35,7 +35,6 @@ const backpressureWaiters = new Set()
 /**
  * Keeps the shared polling timer alive only while an owned request is waiting.
  *
- * @returns {void}
  */
 function updateBackpressureTimerRef () {
   if (ownedBackpressureWaiters > 0) backpressureTimer.ref?.()
@@ -46,7 +45,6 @@ function updateBackpressureTimerRef () {
  * Schedules the next shared backpressure poll.
  *
  * @param {number} due
- * @returns {void}
  */
 function setBackpressureTimer (due) {
   clearTimeout(backpressureTimer)
@@ -61,7 +59,6 @@ function setBackpressureTimer (due) {
  * @param {BackpressureWaiter} waiter
  * @param {number} delay
  * @param {number} attemptIndex
- * @returns {void}
  */
 function scheduleBackpressureRetry (waiter, delay, attemptIndex) {
   waiter.attemptIndex = attemptIndex
@@ -81,7 +78,6 @@ function scheduleBackpressureRetry (waiter, delay, attemptIndex) {
  * Removes a settled request from the shared backpressure poll.
  *
  * @param {BackpressureWaiter} waiter
- * @returns {void}
  */
 function removeBackpressureWaiter (waiter) {
   if (!backpressureWaiters.delete(waiter)) return
@@ -99,7 +95,6 @@ function removeBackpressureWaiter (waiter) {
 /**
  * Retries every request whose shared backpressure delay has elapsed.
  *
- * @returns {void}
  */
 function runBackpressureWaiters () {
   const now = Date.now()
@@ -125,7 +120,6 @@ function runBackpressureWaiters () {
 
 /**
  * @param {Buffer|string|Array<Buffer|string>|(() => Readable)} data
- * @returns {number}
  */
 function getPayloadSize (data) {
   // Factory-backed payloads are streamed by their custom transport and enforce their own concurrency limit.
@@ -166,7 +160,6 @@ function createRequestTimeoutError () {
 
 /**
  * @param {number} statusCode
- * @returns {boolean}
  */
 function isRetriableHttpStatusCode (statusCode) {
   return statusCode === 408 || statusCode === 429 || (statusCode >= 500 && statusCode <= 599)
@@ -186,7 +179,6 @@ function getAbortError (signal) {
  * @param {Readable} data
  * @param {AbortSignal} [signal]
  * @param {(error: Error|null, data?: Buffer, payloadSize?: number) => void} callback
- * @returns {void}
  */
 function bufferReadable (data, signal, callback) {
   const chunks = []
@@ -256,7 +248,6 @@ function bufferReadable (data, signal, callback) {
  * @param {object} options
  * @param {(error: Error|null, result?: string|null, statusCode?: number,
  *   headers?: import('node:http').IncomingHttpHeaders) => void} callback
- * @returns {void}
  */
 function request (data, options, callback) {
   const { signal } = options
@@ -281,7 +272,6 @@ function request (data, options, callback) {
  * @param {(error: Error|null, result?: string|null, statusCode?: number,
  *   headers?: import('node:http').IncomingHttpHeaders) => void} callback
  * @param {number} [reservedPayloadSize]
- * @returns {void}
  */
 function requestBuffered (data, options, callback, reservedPayloadSize) {
   const { signal } = options
@@ -344,7 +334,6 @@ function requestBuffered (data, options, callback, reservedPayloadSize) {
 
   /**
    * @param {number} attemptIndex
-   * @returns {void}
    */
   function attempt (attemptIndex) {
     if (settled) return

package/packages/dd-trace/src/ci-visibility/exporters/settings-cache-key.js

@@ -10,7 +10,6 @@ const getConfig = require('../../config')
  * applied while parsing the response.
  *
  * @param {object} configuration - Request configuration for the settings endpoint.
- * @returns {string}
  */
 function buildSettingsCacheKey (configuration) {
   const config = getConfig()

package/packages/dd-trace/src/ci-visibility/intelligent-test-runner/get-skippable-suites.js

@@ -86,7 +86,6 @@ function parseSkippableSuitesResponse (
  * }} result - Parsed skippable response.
  * @param {'suite'|'test'} testLevel - Test optimization skipping level.
  * @param {boolean} shouldConsiderLineCoverage - Whether line coverage can affect suite skipping.
- * @returns {void}
  */
 function logSkippableSuitesResponse (result, testLevel, shouldConsiderLineCoverage) {
   if (!shouldConsiderLineCoverage || result.numReceivedSkippableItems === undefined) {

package/packages/dd-trace/src/ci-visibility/requests/fs-cache.js

@@ -16,7 +16,6 @@ const CACHE_LOCK_HEARTBEAT_MS = 30_000 // 30 seconds
 /**
  * Returns whether the filesystem cache is enabled via config.
  *
- * @returns {boolean}
  */
 function isCacheEnabled () {
   return getConfig().DD_EXPERIMENTAL_TEST_REQUESTS_FS_CACHE
@@ -27,7 +26,6 @@ function isCacheEnabled () {
  *
  * @param {string} prefix - Cache file prefix (e.g. 'known-tests', 'skippable', 'test-mgmt')
  * @param {Array<unknown>} parts - Values that uniquely identify the cached response
- * @returns {string}
  */
 function buildCacheKey (prefix, parts) {
   const hash = createHash('sha256').update(JSON.stringify(parts)).digest('hex').slice(0, 16)
@@ -36,7 +34,6 @@ function buildCacheKey (prefix, parts) {
 
 /**
  * @param {string} cacheKey
- * @returns {string}
  */
 function getCachePath (cacheKey) {
   return path.join(tmpdir(), `dd-${cacheKey}.json`)
@@ -44,7 +41,6 @@ function getCachePath (cacheKey) {
 
 /**
  * @param {string} cacheKey
- * @returns {string}
  */
 function getLockPath (cacheKey) {
   return path.join(tmpdir(), `dd-${cacheKey}.lock`)
@@ -138,7 +134,6 @@ function tryAcquireLock (cacheKey) {
  * Removes the lock file.
  *
  * @param {string} cacheKey
- * @returns {boolean}
  */
 function releaseLock (cacheKey) {
   try {
@@ -185,7 +180,6 @@ function startLockHeartbeat (cacheKey) {
  * Checks whether the lock file is stale (older than the lock timeout).
  *
  * @param {string} cacheKey
- * @returns {boolean}
  */
 function isLockStale (cacheKey) {
   try {

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

@@ -88,7 +88,7 @@ function parseSlowTestRetries (value) {
  * @returns {EarlyFlakeDetectionSettings}
  */
 function parseEarlyFlakeDetectionSettings (value, isKnownTestsEnabled) {
-  if (!isRecord(value) || value.enabled !== true) {
+  if (!isRecord(value)) {
     return {
       enabled: false,
       retryPolicy: DEFAULT_EARLY_FLAKE_DETECTION_RETRY_POLICY,
@@ -117,7 +117,7 @@ function parseEarlyFlakeDetectionSettings (value, isKnownTestsEnabled) {
   }
 
   return {
-    enabled: isKnownTestsEnabled && isValid,
+    enabled: value.enabled === true && isKnownTestsEnabled && isValid,
     retryPolicy,
     faultyThreshold,
   }

package/packages/dd-trace/src/ci-visibility/requests/rate-limit.js

@@ -6,7 +6,6 @@ const EPOCH_SECONDS_THRESHOLD = 1_000_000_000
  * Returns the delay requested by a Test Optimization intake rate-limit response.
  *
  * @param {import('node:http').IncomingHttpHeaders} [headers]
- * @returns {number}
  */
 function getRateLimitResetDelay (headers) {
   let retryAfter = headers?.['retry-after']

package/packages/dd-trace/src/ci-visibility/requests/upload-test-screenshot.js

@@ -46,7 +46,6 @@ function isValidId (value) {
  * Renders an idempotency key into a value safe to carry in the upload's query string.
  *
  * @param {string} idempotencyKey - Raw per-artifact idempotency key
- * @returns {string} Query-safe, deterministic representation of the key
  */
 function toIdempotencyQueryValue (idempotencyKey) {
   const separatorIndex = idempotencyKey.indexOf(':')
@@ -76,7 +75,6 @@ function toIdempotencyQueryValue (idempotencyKey) {
  * @param {number} [options.deadline] - Absolute finalization deadline in epoch milliseconds
  * @param {AbortSignal} [options.signal] - Signal used to cancel the upload
  * @param {Function} callback - Callback function (err)
- * @returns {void}
  */
 function uploadTestMedia (options, callback) {
   const {
@@ -216,7 +214,6 @@ function uploadTestMedia (options, callback) {
  *
  * @param {object} options - Upload options
  * @param {Function} callback - Callback function (err)
- * @returns {void}
  */
 function uploadTestScreenshot (options, callback) {
   uploadTestMedia({ ...options, kind: 'screenshot' }, callback)
@@ -227,7 +224,6 @@ function uploadTestScreenshot (options, callback) {
  *
  * @param {object} options - Upload options
  * @param {Function} callback - Callback function (err)
- * @returns {void}
  */
 function uploadTestVideo (options, callback) {
   uploadTestMedia({ ...options, kind: 'video' }, callback)
@@ -238,7 +234,6 @@ function uploadTestVideo (options, callback) {
  *
  * @param {object} options - Upload options
  * @param {Function} callback - Callback function (err)
- * @returns {void}
  */
 function uploadTestSuiteVideo (options, callback) {
   uploadTestMedia({ ...options, kind: 'video' }, callback)

package/packages/dd-trace/src/ci-visibility/requests/video-request.js

@@ -21,7 +21,6 @@ let activeRequests = 0
  * @param {object} options - HTTP request options
  * @param {(error: Error|null, result?: string|null, statusCode?: number,
  *   headers?: import('node:http').IncomingHttpHeaders) => void} callback
- * @returns {void}
  */
 function requestVideo (body, options, callback) {
   if (activeRequests >= MAX_ACTIVE_REQUESTS) {

package/packages/dd-trace/src/ci-visibility/telemetry.js

@@ -75,6 +75,16 @@ function distributionMetric (name, tags, measure) {
   ciVisibilityMetrics.distribution(name, formatMetricTags(tags)).track(measure)
 }
 
+/**
+ * Records that dynamic, duration-based ATR retries are enabled for this session.
+ *
+ * @param {boolean} hasCustomBuckets
+ */
+function recordDynamicAtrRetries (hasCustomBuckets) {
+  const tags = hasCustomBuckets ? { has_custom_buckets: 'true' } : {}
+  incrementCountMetric(TELEMETRY_DYNAMIC_ATR_RETRIES_ENABLED, tags)
+}
+
 // CI Visibility telemetry events
 const TELEMETRY_TEST_SESSION = 'test_session'
 const TELEMETRY_EVENT_CREATED = 'event_created'
@@ -134,6 +144,7 @@ const TELEMETRY_TEST_MANAGEMENT_TESTS_MS = 'test_management_tests.request_ms'
 const TELEMETRY_TEST_MANAGEMENT_TESTS_ERRORS = 'test_management_tests.request_errors'
 const TELEMETRY_TEST_MANAGEMENT_TESTS_RESPONSE_TESTS = 'test_management_tests.response_tests'
 const TELEMETRY_TEST_MANAGEMENT_TESTS_RESPONSE_BYTES = 'test_management_tests.response_bytes'
+const TELEMETRY_DYNAMIC_ATR_RETRIES_ENABLED = 'dynamic_atr_retries.enabled'
 
 function getErrorTypeFromStatusCode (statusCode) {
   if (typeof statusCode !== 'number') return 'network'
@@ -149,6 +160,7 @@ function getErrorTypeFromStatusCode (statusCode) {
 module.exports = {
   incrementCountMetric,
   distributionMetric,
+  recordDynamicAtrRetries,
   TELEMETRY_TEST_SESSION,
   TELEMETRY_EVENT_CREATED,
   TELEMETRY_EVENT_FINISHED,
@@ -204,4 +216,5 @@ module.exports = {
   TELEMETRY_TEST_MANAGEMENT_TESTS_ERRORS,
   TELEMETRY_TEST_MANAGEMENT_TESTS_RESPONSE_TESTS,
   TELEMETRY_TEST_MANAGEMENT_TESTS_RESPONSE_BYTES,
+  TELEMETRY_DYNAMIC_ATR_RETRIES_ENABLED,
 }

package/packages/dd-trace/src/ci-visibility/test-optimization-cache.js

@@ -22,7 +22,6 @@ function getSettingsCachePath () {
 /**
  * Sets up the test optimization settings cache file path.
  * Returns the existing path if already set, otherwise creates a new one.
- * @returns {string} The cache file path.
  */
 function setupSettingsCachePath () {
   const existing = getSettingsCachePath()

package/packages/dd-trace/src/ci-visibility/test-screenshot.js

@@ -17,7 +17,6 @@ const SCREENSHOT_UPLOAD_RESULT_ERROR = 'error'
  *
  * @param {object|string} screenshot - Framework screenshot metadata or its file path
  * @param {string} filePath - Resolved screenshot file path
- * @returns {number} Capture time in epoch milliseconds
  */
 function getScreenshotCapturedAtMs (screenshot, filePath) {
   const takenAt = screenshot !== null && typeof screenshot === 'object' ? screenshot.takenAt : undefined
@@ -73,7 +72,6 @@ function getScreenshotUploadTag (uploadResult) {
  *
  * @param {object} testSpan - Test span to tag
  * @param {string|undefined} uploadResult - Aggregate screenshot upload result
- * @returns {void}
  */
 function setScreenshotUploadTags (testSpan, uploadResult) {
   const uploadTag = getScreenshotUploadTag(uploadResult)

package/packages/dd-trace/src/ci-visibility/test-video.js

@@ -42,7 +42,6 @@ function getVideoUploadTag (uploadResult) {
  * @param {object} span - Test or test suite span to tag
  * @param {string|undefined} uploadResult - Aggregate video upload result
  * @param {string} [scope] - Media lookup scope
- * @returns {void}
  */
 function setVideoUploadTags (span, uploadResult, scope) {
   const uploadTag = getVideoUploadTag(uploadResult)

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

@@ -47,10 +47,11 @@ function warnInvalidValue (value, optionName, source, baseMessage, error, outcom
       error.stack = error.toString()
       message += `\n\n${util.inspect(error)}`
     }
+    const telemetryError = { code: null, message }
     if (telemetryEntry) {
-      telemetryEntry.error = { message }
+      telemetryEntry.error = telemetryError
     } else {
-      parseErrors.set(telemetryKey, { message })
+      parseErrors.set(telemetryKey, telemetryError)
     }
     log ??= require('../log')
     const logLevel = error ? 'error' : 'warn'