From 66b446bbf67c27e5f353fb47883a658a9435d0d4 Mon Sep 17 00:00:00 2001 From: Mikayla Toffler Date: Thu, 30 Oct 2025 15:19:59 -0400 Subject: [PATCH 1/2] Migrate some CI Vis calls to new Config getters --- ci/init.js | 8 ++++---- packages/dd-trace/src/config.js | 11 +++++++++++ 2 files changed, 15 insertions(+), 4 deletions(-) diff --git a/ci/init.js b/ci/init.js index a1ef9054f8b..83918486ccf 100644 --- a/ci/init.js +++ b/ci/init.js @@ -2,9 +2,9 @@ /* eslint-disable no-console */ const tracer = require('../packages/dd-trace') -const { isTrue, isFalse } = require('../packages/dd-trace/src/util') const log = require('../packages/dd-trace/src/log') const { getEnvironmentVariable } = require('../packages/dd-trace/src/config-helper') +const { isCIVisibilityEnabled, isCiVisibilityAgentlessEnabled, getAPIKey } = require('../packages/dd-trace/src/config') const PACKAGE_MANAGERS = ['npm', 'yarn', 'pnpm'] const DEFAULT_FLUSH_INTERVAL = 5000 @@ -42,8 +42,8 @@ const baseOptions = { flushInterval: isJestWorker ? JEST_FLUSH_INTERVAL : DEFAULT_FLUSH_INTERVAL } -let shouldInit = !isFalse(getEnvironmentVariable('DD_CIVISIBILITY_ENABLED')) -const isAgentlessEnabled = isTrue(getEnvironmentVariable('DD_CIVISIBILITY_AGENTLESS_ENABLED')) +let shouldInit = isCIVisibilityEnabled() +const isAgentlessEnabled = isCiVisibilityAgentlessEnabled() if (!isTestWorker && isPackageManager()) { log.debug('dd-trace is not initialized in a package manager.') @@ -57,7 +57,7 @@ if (isTestWorker) { } } else { if (isAgentlessEnabled) { - if (getEnvironmentVariable('DD_API_KEY')) { + if (getAPIKey()) { baseOptions.experimental = { exporter: 'datadog' } diff --git a/packages/dd-trace/src/config.js b/packages/dd-trace/src/config.js index fd02d3766ad..65affc70f1a 100644 --- a/packages/dd-trace/src/config.js +++ b/packages/dd-trace/src/config.js @@ -1464,6 +1464,17 @@ class Config { } } } + + isCIVisibilityEnabled () { + return this.isCiVisibility + } + + isCiVisibilityAgentlessEnabled () { + return this.isCiVisibilityAgentless + } + + getAPIKey () { + return this.apiKey } function handleOtel (tagString) { From fe44133481a2acea89b64cf91d84b468d8002fe9 Mon Sep 17 00:00:00 2001 From: Mikayla Toffler Date: Thu, 30 Oct 2025 15:24:28 -0400 Subject: [PATCH 2/2] fix syntax issue --- packages/dd-trace/src/config.js | 1 + 1 file changed, 1 insertion(+) diff --git a/packages/dd-trace/src/config.js b/packages/dd-trace/src/config.js index 65affc70f1a..5111dd19754 100644 --- a/packages/dd-trace/src/config.js +++ b/packages/dd-trace/src/config.js @@ -1475,6 +1475,7 @@ class Config { getAPIKey () { return this.apiKey + } } function handleOtel (tagString) {