dd-trace 6.0.0-pre-b4d004d → 6.0.0-pre-0bb1f17

package/LICENSE-3rdparty.csv

@@ -56,6 +56,7 @@ dev,chai,MIT,Copyright 2017 Chai.js Assertion Library
 dev,eslint,MIT,Copyright JS Foundation and other contributors https://js.foundation
 dev,eslint-plugin-cypress,MIT,Copyright (c) 2019 Cypress.io
 dev,eslint-plugin-import,MIT,Copyright 2015 Ben Mosher
+dev,eslint-plugin-jsdoc,BSD-3-Clause,Copyright Gajus Kuizinas
 dev,eslint-plugin-mocha,MIT,Copyright 2014 Mathias Schreck
 dev,eslint-plugin-n,MIT,Copyright 2015 Toru Nagashima
 dev,eslint-plugin-promise,ISC,jden and other contributors

package/index.d.ts

@@ -541,6 +541,12 @@ declare namespace tracer {
       eventLoop?: boolean
     }
 
+    /**
+     * Whether to add an auto-generated `runtime-id` tag to metrics.
+     * @default false
+     */
+    runtimeMetricsRuntimeId?: boolean
+
     /**
      * Custom function for DNS lookups when sending requests to the agent.
      * @default dns.lookup()
@@ -577,13 +583,6 @@ declare namespace tracer {
      */
     experimental?: {
       b3?: boolean
-      traceparent?: boolean
-
-      /**
-       * Whether to add an auto-generated `runtime-id` tag to metrics.
-       * @default false
-       */
-      runtimeId?: boolean
 
       /**
        * Whether to write traces to log output or agentless, rather than send to an agent
@@ -654,6 +653,13 @@ declare namespace tracer {
          * @default false
          */
         enabled?: boolean
+        /**
+         * Timeout in milliseconds for OpenFeature provider initialization.
+         * If configuration is not received within this time, initialization fails.
+         *
+         * @default 30000
+         */
+        initializationTimeoutMs?: number
       }
     };
 
@@ -680,13 +686,6 @@ declare namespace tracer {
      */
     tags?: { [key: string]: any };
 
-    /**
-     * Specifies which scope implementation to use. The default is to use the best
-     * implementation for the runtime. Only change this if you know what you are
-     * doing.
-     */
-    scope?: 'async_hooks' | 'async_local_storage' | 'async_resource' | 'sync' | 'noop'
-
     /**
      * Whether to report the hostname of the service host. This is used when the agent is deployed on a different host and cannot determine the hostname automatically.
      * @default false
@@ -699,13 +698,6 @@ declare namespace tracer {
      */
     logLevel?: 'error' | 'debug'
 
-    /**
-     * If false, require a parent in order to trace.
-     * @default true
-     * @deprecated since version 4.0
-     */
-    orphanable?: boolean
-
     /**
      * Enables DBM to APM link using tag injection.
      * @default 'disabled'
@@ -912,7 +904,7 @@ declare namespace tracer {
     /**
      * The selection and priority order of context propagation injection and extraction mechanisms.
      */
-    propagationStyle?: string[] | PropagationStyle
+    tracePropagationStyle?: string[] | PropagationStyle
 
     /**
      * Cloud payload report as tags
@@ -3011,15 +3003,15 @@ declare namespace tracer {
       label: string,
 
       /**
-       * The type of evaluation metric, one of 'categorical' or 'score'
+       * The type of evaluation metric, one of 'categorical', 'score', or 'boolean'
        */
-      metricType: 'categorical' | 'score',
+      metricType: 'categorical' | 'score' | 'boolean',
 
       /**
        * The value of the evaluation metric.
-       * Must be string for 'categorical' metrics and number for 'score' metrics.
+       * Must be string for 'categorical' metrics, number for 'score' metrics, and boolean for 'boolean' metrics.
        */
-      value: string | number,
+      value: string | number | boolean,
 
       /**
        * An object of string key-value pairs to tag the evaluation metric with.

package/initialize.mjs

@@ -1,14 +1,14 @@
 /**
-  * This file serves one of two purposes, depending on how it's used.
-  *
-  * If used with --import, it will import init.js and register the loader hook.
-  * If used with --loader, it will act as the loader hook, except that it will
-  * also import init.js inside the source code of the entrypoint file.
-  *
-  * The result is that no matter how this file is used, so long as it's with
-  * one of the two flags, the tracer will always be initialized, and the loader
-  * hook will always be active for ESM support.
-  */
+ * This file serves one of two purposes, depending on how it's used.
+ *
+ * If used with --import, it will import init.js and register the loader hook.
+ * If used with --loader, it will act as the loader hook, except that it will
+ * also import init.js inside the source code of the entrypoint file.
+ *
+ * The result is that no matter how this file is used, so long as it's with
+ * one of the two flags, the tracer will always be initialized, and the loader
+ * hook will always be active for ESM support.
+ */
 
 /* eslint n/no-unsupported-features/node-builtins: ['error', { ignores: ['module.register'] }] */
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dd-trace",
-  "version": "6.0.0-pre-b4d004d",
+  "version": "6.0.0-pre-0bb1f17",
   "description": "Datadog APM tracing client for JavaScript",
   "main": "index.js",
   "typings": "index.d.ts",
@@ -46,6 +46,8 @@
     "test:llmobs:sdk:ci": "nyc --no-clean --include \"packages/dd-trace/src/llmobs/**/*.js\" -- npm run test:llmobs:sdk",
     "test:llmobs:plugins": "mocha -r \"packages/dd-trace/test/setup/mocha.js\" \"packages/dd-trace/test/llmobs/plugins/@($(echo $PLUGINS))/*.spec.js\"",
     "test:llmobs:plugins:ci": "yarn services && nyc --no-clean --include \"packages/dd-trace/src/llmobs/**/*.js\" -- npm run test:llmobs:plugins",
+    "test:openfeature": "mocha -r \"packages/dd-trace/test/setup/mocha.js\" \"packages/dd-trace/test/openfeature/*.spec.js\"",
+    "test:openfeature:ci": "nyc --no-clean --include \"packages/dd-trace/src/openfeature/**/*.js\" -- npm run test:openfeature",
     "test:plugins": "mocha -r \"packages/dd-trace/test/setup/mocha.js\" \"packages/datadog-plugin-@($(echo $PLUGINS))/test/**/@($(echo ${SPEC:-'*'})).spec.js\"",
     "test:plugins:ci": "yarn services && nyc --no-clean --include \"packages/datadog-plugin-@($(echo $PLUGINS))/src/**/*.js\" -- npm run test:plugins",
     "test:plugins:ci:flaky": "yarn services && nyc --no-clean --include \"packages/datadog-plugin-@($(echo $PLUGINS))/src/**/*.js\" -- npm run test:plugins -- --bail --retries 2",
@@ -65,7 +67,7 @@
     "test:integration:playwright": "mocha --timeout 60000 -r \"packages/dd-trace/test/setup/core.js\" \"integration-tests/playwright/*.spec.js\"",
     "test:integration:selenium": "mocha --timeout 60000 -r \"packages/dd-trace/test/setup/core.js\" \"integration-tests/selenium/*.spec.js\"",
     "test:integration:vitest": "mocha --timeout 60000 -r \"packages/dd-trace/test/setup/core.js\" \"integration-tests/vitest/*.spec.js\"",
-    "test:integration:testopt": "mocha --timeout 90000 -r \"packages/dd-trace/test/setup/core.js\" \"integration-tests/ci-visibility/*.spec.js\"",
+    "test:integration:testopt": "mocha --timeout 120000 -r \"packages/dd-trace/test/setup/core.js\" \"integration-tests/ci-visibility/*.spec.js\"",
     "test:integration:profiler": "mocha --timeout 180000 -r \"packages/dd-trace/test/setup/core.js\" \"integration-tests/profiler/*.spec.js\"",
     "test:integration:plugins": "mocha -r \"packages/dd-trace/test/setup/mocha.js\" \"packages/datadog-plugin-@($(echo $PLUGINS))/test/integration-test/**/*.spec.js\"",
     "test:unit:plugins": "mocha -r \"packages/dd-trace/test/setup/mocha.js\" \"packages/datadog-instrumentations/test/@($(echo $PLUGINS)).spec.js\" \"packages/datadog-plugin-@($(echo $PLUGINS))/test/**/*.spec.js\" --exclude \"packages/datadog-plugin-@($(echo $PLUGINS))/test/integration-test/**/*.spec.js\"",
@@ -123,9 +125,9 @@
   "dependencies": {
     "@datadog/libdatadog": "0.7.0",
     "@datadog/native-appsec": "10.3.0",
-    "@datadog/native-iast-taint-tracking": "4.0.0",
+    "@datadog/native-iast-taint-tracking": "4.1.0",
     "@datadog/native-metrics": "3.1.1",
-    "@datadog/openfeature-node-server": "0.1.0-preview.15",
+    "@datadog/openfeature-node-server": "^0.2.0",
     "@datadog/pprof": "5.12.0",
     "@datadog/sketches-js": "2.1.1",
     "@datadog/wasm-js-rewriter": "5.0.1",
@@ -165,21 +167,22 @@
     "@eslint/js": "^9.39.0",
     "@msgpack/msgpack": "^3.1.2",
     "@openfeature/core": "^1.9.0",
-    "@openfeature/server-sdk": "^1.20.0",
+    "@openfeature/server-sdk": "~1.20.0",
     "@stylistic/eslint-plugin": "^5.5.0",
     "@types/chai": "^4.3.16",
     "@types/mocha": "^10.0.10",
     "@types/node": "^18.19.106",
-    "@types/sinon": "^17.0.4",
+    "@types/sinon": "^21.0.0",
     "@types/tap": "^15.0.12",
     "axios": "^1.12.2",
     "benchmark": "^2.1.4",
     "body-parser": "^2.2.0",
-    "bun": "1.3.2",
+    "bun": "1.3.3",
     "chai": "^4.5.0",
     "eslint": "^9.39.0",
     "eslint-plugin-cypress": "^5.2.0",
     "eslint-plugin-import": "^2.32.0",
+    "eslint-plugin-jsdoc": "^61.1.12",
     "eslint-plugin-mocha": "^11.2.0",
     "eslint-plugin-n": "^17.23.1",
     "eslint-plugin-promise": "^7.2.1",

package/packages/datadog-core/src/storage.js

@@ -15,7 +15,7 @@ const { AsyncLocalStorage } = require('async_hooks')
 class DatadogStorage extends AsyncLocalStorage {
   /**
    *
-   * @param store {Store}
+   * @param {Store} [store]
    * @override
    */
   enterWith (store) {
@@ -47,7 +47,7 @@ class DatadogStorage extends AsyncLocalStorage {
    * retrieved through `getHandle()` can also be passed in to be used as the
    * key. This is useful if you've stashed a handle somewhere and want to
    * retrieve the store with it.
-   * @param {{}} [handle]
+   * @param {object} [handle]
    * @returns {Store | undefined}
    * @override
    */
@@ -87,7 +87,7 @@ class DatadogStorage extends AsyncLocalStorage {
 
 /**
  * This is the map from handles to real stores, used in the class above.
- * @type {WeakMap<WeakKey, Store>}
+ * @type {WeakMap<WeakKey, Store|undefined>}
  */
 const stores = new WeakMap()
 
@@ -101,7 +101,7 @@ const storages = Object.create(null)
 
 /**
  *
- * @param namespace {string} the namespace to use
+ * @param {string} namespace The namespace to use
  * @returns {DatadogStorage}
  */
 function storage (namespace) {

package/packages/datadog-esbuild/src/utils.js

@@ -65,7 +65,11 @@ function getSource (url, { format }) {
 /**
  * Generates the pieces of code for the proxy module before the path
  *
- * @param {Object} moduleData { path, internal, context, excludeDefault }
+ * @param {object} moduleData
+ * @param {string} moduleData.path
+ * @param {boolean} moduleData.internal
+ * @param {object} moduleData.context
+ * @param {boolean} moduleData.excludeDefault
  * @returns {Promise<Map>}
  */
 async function processModule ({ path, internal, context, excludeDefault }) {

package/packages/datadog-instrumentations/src/aws-sdk.js

@@ -34,10 +34,11 @@ function wrapRequest (send) {
   }
 }
 
-function wrapDeserialize (deserialize, channelSuffix) {
+function wrapDeserialize (deserialize, channelSuffix, responseIndex = 0) {
   const headersCh = channel(`apm:aws:response:deserialize:${channelSuffix}`)
 
-  return function (response) {
+  return function () {
+    const response = arguments[responseIndex]
     if (headersCh.hasSubscribers) {
       headersCh.publish({ headers: response.headers })
     }
@@ -66,6 +67,12 @@ function wrapSmithySend (send) {
 
     if (typeof command.deserialize === 'function') {
       shimmer.wrap(command, 'deserialize', deserialize => wrapDeserialize(deserialize, channelSuffix))
+    } else if (this.config?.protocol?.deserializeResponse) {
+      shimmer.wrap(
+        this.config.protocol,
+        'deserializeResponse',
+        deserializeResponse => wrapDeserialize(deserializeResponse, channelSuffix, 2)
+      )
     }
 
     const ctx = {

package/packages/datadog-instrumentations/src/azure-service-bus.js

@@ -8,57 +8,64 @@ const shimmer = require('../../datadog-shimmer')
 const dc = require('dc-polyfill')
 
 const producerCh = dc.tracingChannel('apm:azure-service-bus:send')
+const isItDefault = new WeakSet()
 
-addHook({ name: '@azure/service-bus', versions: ['>=7.9.2'], patchDefault: false }, (obj) => {
+addHook({ name: '@azure/service-bus', versions: ['>=7.9.2'] }, (obj) => {
   const ServiceBusClient = obj.ServiceBusClient
-  let didItShim = false
   shimmer.wrap(ServiceBusClient.prototype, 'createSender',
     createSender => function (queueOrTopicName) {
       const sender = createSender.apply(this, arguments)
-      if (didItShim) return sender
       const senderPrototype = sender.constructor.prototype
       const senderSenderPrototype = sender._sender.constructor.prototype
-      shimmer.wrap(senderPrototype, 'scheduleMessages', scheduleMessages =>
-        function (msg, scheduledEnqueueTimeUtc) {
-          const functionName = scheduleMessages.name
+
+      if (!isItDefault.has(senderPrototype)) {
+        isItDefault.add(senderPrototype)
+
+        shimmer.wrap(senderPrototype, 'scheduleMessages', scheduleMessages =>
+          function (msg, scheduledEnqueueTimeUtc) {
+            const functionName = scheduleMessages.name
+            const config = this._context.config
+            const entityPath = this._entityPath
+            return producerCh.tracePromise(
+              scheduleMessages,
+              { config, entityPath, functionName, msg, scheduledEnqueueTimeUtc },
+              this, ...arguments
+            )
+          })
+
+        shimmer.wrap(senderPrototype, 'createMessageBatch', createMessageBatch => async function () {
+          const batch = await createMessageBatch.apply(this, arguments)
+          shimmer.wrap(batch, 'tryAddMessage', tryAddMessage => function (msg) {
+            const functionName = tryAddMessage.name
+            const config = this._context.config
+            return producerCh.tracePromise(
+              tryAddMessage, { config, functionName, batch, msg }, this, ...arguments)
+          })
+          return batch
+        })
+      }
+
+      if (!isItDefault.has(senderSenderPrototype)) {
+        isItDefault.add(senderSenderPrototype)
+
+        shimmer.wrap(senderSenderPrototype, 'send', send => function (msg) {
+          const functionName = send.name
           const config = this._context.config
-          const entityPath = this._entityPath
+          const entityPath = this.entityPath
           return producerCh.tracePromise(
-            scheduleMessages,
-            { config, entityPath, functionName, msg, scheduledEnqueueTimeUtc },
-            this, ...arguments
+            send, { config, entityPath, functionName, msg }, this, ...arguments
           )
         })
 
-      shimmer.wrap(senderPrototype, 'createMessageBatch', createMessageBatch => async function () {
-        const batch = await createMessageBatch.apply(this, arguments)
-        shimmer.wrap(batch.constructor.prototype, 'tryAddMessage', tryAddMessage => function (msg) {
-          const functionName = tryAddMessage.name
+        shimmer.wrap(senderSenderPrototype, 'sendBatch', sendBatch => function (msg) {
+          const functionName = sendBatch.name
           const config = this._context.config
+          const entityPath = this.entityPath
           return producerCh.tracePromise(
-            tryAddMessage, { config, functionName, batch, msg }, this, ...arguments)
+            sendBatch, { config, entityPath, functionName, msg }, this, ...arguments
+          )
         })
-        return batch
-      })
-
-      shimmer.wrap(senderSenderPrototype, 'send', send => function (msg) {
-        const functionName = send.name
-        const config = this._context.config
-        const entityPath = this.entityPath
-        return producerCh.tracePromise(
-          send, { config, entityPath, functionName, msg }, this, ...arguments
-        )
-      })
-
-      shimmer.wrap(senderSenderPrototype, 'sendBatch', sendBatch => function (msg) {
-        const functionName = sendBatch.name
-        const config = this._context.config
-        const entityPath = this.entityPath
-        return producerCh.tracePromise(
-          sendBatch, { config, entityPath, functionName, msg }, this, ...arguments
-        )
-      })
-      didItShim = true
+      }
       return sender
     })
   return obj

package/packages/datadog-instrumentations/src/helpers/hook.js

@@ -8,6 +8,7 @@ const ritm = require('../../../dd-trace/src/ritm')
  * In practice, `modules` is always an array with a single entry.
  *
  * @param {string[]} modules list of modules to hook into
+ * @param {object} hookOptions hook options
  * @param {Function} onrequire callback to be executed upon encountering module
  */
 function Hook (modules, hookOptions, onrequire) {

package/packages/datadog-instrumentations/src/helpers/instrument.js

@@ -28,7 +28,8 @@ exports.tracingChannel = function (name) {
  * @param {string[]} args.versions array of semver range strings
  * @param {string} [args.file='index.js'] path to file within package to instrument
  * @param {string} [args.filePattern] pattern to match files within package to instrument
- * @param Function hook
+ * @param {boolean} [args.patchDefault] whether to patch the default export
+ * @param {(moduleExports: unknown, version: string) => unknown} hook
  */
 exports.addHook = function addHook ({ name, versions, file, filePattern, patchDefault }, hook) {
   if (typeof name === 'string') {

package/packages/datadog-instrumentations/src/jest.js

@@ -1110,7 +1110,7 @@ function jestAdapterWrapper (jestAdapter, jestVersion) {
        * Child processes do not each request ITR configuration, so the jest's parent process
        * needs to pass them the configuration. This is done via _ddTestCodeCoverageEnabled, which
        * controls whether coverage is reported.
-      */
+       */
       if (environment.testEnvironmentOptions?._ddTestCodeCoverageEnabled) {
         const root = environment.repositoryRoot || environment.rootDir
 
@@ -1345,9 +1345,6 @@ addHook({
         }
       }
       return returnedValue
-    } catch (error) {
-      testSuiteErrorCh.publish({ error, testSuiteAbsolutePath: this._testPath })
-      throw error
     } finally {
       // Restore original prepareStackTrace
       Error.prepareStackTrace = originalPrepareStackTrace

package/packages/datadog-instrumentations/src/playwright.js

@@ -481,6 +481,15 @@ function dispatcherRunWrapper (run) {
 
 function dispatcherRunWrapperNew (run) {
   return function (testGroups) {
+    // Filter out disabled tests from testGroups before they get scheduled
+    if (isTestManagementTestsEnabled) {
+      testGroups.forEach(group => {
+        group.tests = group.tests.filter(test => !test._ddIsDisabled)
+      })
+      // Remove empty groups
+      testGroups = testGroups.filter(group => group.tests.length > 0)
+    }
+
     if (!this._allTests) {
       // Removed in https://github.com/microsoft/playwright/commit/1e52c37b254a441cccf332520f60225a5acc14c7
       // Not available from >=1.44.0
@@ -893,6 +902,9 @@ addHook({
         if (testProperties.disabled) {
           test._ddIsDisabled = true
           test.expectedStatus = 'skipped'
+          // setting test.expectedStatus to 'skipped' does not work for every case,
+          // so we need to filter out disabled tests in dispatcherRunWrapperNew,
+          // so they don't get to the workers
           continue
         }
         if (testProperties.quarantined) {
@@ -1270,6 +1282,10 @@ function generateSummaryWrapper (generateSummary) {
         const {
           _requireFile: testSuiteAbsolutePath,
           location: { line: testSourceLine },
+          _ddIsNew: isNew,
+          _ddIsDisabled: isDisabled,
+          _ddIsModified: isModified,
+          _ddIsQuarantined: isQuarantined
         } = test
         const browserName = getBrowserNameFromProjects(sessionProjects, test)
 
@@ -1278,6 +1294,10 @@ function generateSummaryWrapper (generateSummary) {
           testSuiteAbsolutePath,
           testSourceLine,
           browserName,
+          isNew,
+          isDisabled,
+          isModified,
+          isQuarantined
         })
       }
     }

package/packages/datadog-plugin-aws-sdk/src/services/bedrockruntime/utils.js

@@ -27,8 +27,9 @@ const PROVIDER = {
  * Coerce the chunks into a single response body.
  *
  * @param {Array<{ chunk: { bytes: Buffer } }>} chunks
- * @param {string} provider
- * @returns {Object}
+ * @param {string} modelProvider
+ * @param {string} modelName
+ * @returns {Generation | Record<never, never>}
  */
 function extractTextAndResponseReasonFromStream (chunks, modelProvider, modelName) {
   const modelProviderUpper = modelProvider.toUpperCase()

package/packages/datadog-plugin-azure-service-bus/src/producer.js

@@ -2,6 +2,7 @@
 
 const { getEnvironmentVariable } = require('../../dd-trace/src/config-helper')
 const ProducerPlugin = require('../../dd-trace/src/plugins/producer')
+const spanContexts = new WeakMap()
 
 class AzureServiceBusProducerPlugin extends ProducerPlugin {
   static get id () { return 'azure-service-bus' }
@@ -36,7 +37,12 @@ class AzureServiceBusProducerPlugin extends ProducerPlugin {
       }
 
       if (batchLinksAreEnabled()) {
-        ctx.batch._spanContexts.push(span.context())
+        const spanContext = spanContexts.get(ctx.batch)
+        if (spanContext) {
+          spanContext.push(span.context())
+        } else {
+          spanContexts.set(ctx.batch, [span.context()])
+        }
         injectTraceContext(this.tracer, span, ctx.msg)
       }
     }
@@ -47,9 +53,12 @@ class AzureServiceBusProducerPlugin extends ProducerPlugin {
       if (isBatch) {
         span.setTag('messaging.batch.message_count', messages.count)
         if (batchLinksAreEnabled()) {
-          messages._spanContexts.forEach(spanContext => {
-            span.addLink(spanContext)
-          })
+          const contexts = spanContexts.get(messages)
+          if (contexts) {
+            for (const spanContext of contexts) {
+              span.addLink(spanContext)
+            }
+          }
         }
       } else if (Array.isArray(messages)) {
         span.setTag('messaging.batch.message_count', messages.length)
@@ -64,7 +73,7 @@ class AzureServiceBusProducerPlugin extends ProducerPlugin {
   }
 
   asyncEnd (ctx) {
-    super.finish()
+    super.finish(ctx)
   }
 }
 

package/packages/datadog-plugin-cypress/src/cypress-plugin.js

@@ -133,6 +133,14 @@ function getCypressCommand (details) {
   return `${TEST_FRAMEWORK_NAME} ${details.specPattern || ''}`
 }
 
+function getIsTestIsolationEnabled (cypressConfig) {
+  if (!cypressConfig) {
+    // If we can't read testIsolation config parameter, we default to allowing retries
+    return true
+  }
+  return cypressConfig.testIsolation === undefined ? true : cypressConfig.testIsolation
+}
+
 function getLibraryConfiguration (tracer, testConfiguration) {
   return new Promise(resolve => {
     if (!tracer._tracer._exporter?.getLibraryConfiguration) {
@@ -296,6 +304,12 @@ class CypressPlugin {
     this.tracer = tracer
     this.cypressConfig = cypressConfig
 
+    this.isTestIsolationEnabled = getIsTestIsolationEnabled(cypressConfig)
+
+    if (!this.isTestIsolationEnabled) {
+      log.warn('Test isolation is disabled, retries will not be enabled')
+    }
+
     // we have to do it here because the tracer is not initialized in the constructor
     this.testEnvironmentMetadata[DD_TEST_IS_USER_PROVIDED_SERVICE] =
       tracer._tracer._config.isServiceUserProvided ? 'true' : 'false'
@@ -324,7 +338,7 @@ class CypressPlugin {
           this.isEarlyFlakeDetectionEnabled = isEarlyFlakeDetectionEnabled
           this.earlyFlakeDetectionNumRetries = earlyFlakeDetectionNumRetries
           this.isKnownTestsEnabled = isKnownTestsEnabled
-          if (isFlakyTestRetriesEnabled) {
+          if (isFlakyTestRetriesEnabled && this.isTestIsolationEnabled) {
             this.isFlakyTestRetriesEnabled = true
             this.cypressConfig.retries.runMode = flakyTestRetriesCount
           }
@@ -467,6 +481,7 @@ class CypressPlugin {
     // We need to make sure that the plugin is initialized before running the tests
     // This is for the case where the user has not returned the promise from the init function
     await this.libraryConfigurationPromise
+
     this.command = getCypressCommand(details)
     this.frameworkVersion = getCypressVersion(details)
     this.rootDir = getRootDir(details)
@@ -801,7 +816,8 @@ class CypressPlugin {
           testManagementTests: this.getTestSuiteProperties(testSuite),
           isImpactedTestsEnabled: this.isImpactedTestsEnabled,
           isModifiedTest: this.getIsTestModified(testSuiteAbsolutePath),
-          repositoryRoot: this.repositoryRoot
+          repositoryRoot: this.repositoryRoot,
+          isTestIsolationEnabled: this.isTestIsolationEnabled
         }
 
         if (this.testSuiteSpan) {
@@ -962,6 +978,11 @@ class CypressPlugin {
           this.activeTestSpan.addTags(tags)
         }
         return null
+      },
+      'dd:log': (message) => {
+        // eslint-disable-next-line no-console
+        console.log(`[datadog] ${message}`)
+        return null
       }
     }
   }