v5.85.0 proposal - #7399
Merged
Merged
Conversation
Otherwise the CI jobs would not trigger anymore. This happens if commits use the former GITHUB_TOKEN.
This adds a new CI job to sign deduping commits and that commit is added to any opened PR. In addition, husky is added with a manual install to install a pre-commit hook to run dedupe while committing locally.
…ss 1 directory with 3 updates (#7392) Bumps the dev-minor-and-patch-dependencies group with 3 updates in the / directory: [@stylistic/eslint-plugin](https://github.com/eslint-stylistic/eslint-stylistic/tree/HEAD/packages/eslint-plugin), [eslint-plugin-jsdoc](https://github.com/gajus/eslint-plugin-jsdoc) and [globals](https://github.com/sindresorhus/globals). Updates `@stylistic/eslint-plugin` from 5.7.0 to 5.7.1 - [Release notes](https://github.com/eslint-stylistic/eslint-stylistic/releases) - [Changelog](https://github.com/eslint-stylistic/eslint-stylistic/blob/main/CHANGELOG.md) - [Commits](https://github.com/eslint-stylistic/eslint-stylistic/commits/v5.7.1/packages/eslint-plugin) Updates `eslint-plugin-jsdoc` from 62.0.0 to 62.4.1 - [Release notes](https://github.com/gajus/eslint-plugin-jsdoc/releases) - [Commits](gajus/eslint-plugin-jsdoc@v62.0.0...v62.4.1) Updates `globals` from 17.0.0 to 17.1.0 - [Release notes](https://github.com/sindresorhus/globals/releases) - [Commits](sindresorhus/globals@v17.0.0...v17.1.0) --- updated-dependencies: - dependency-name: "@stylistic/eslint-plugin" dependency-version: 5.7.1 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: dev-minor-and-patch-dependencies - dependency-name: eslint-plugin-jsdoc dependency-version: 62.4.1 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: dev-minor-and-patch-dependencies - dependency-name: globals dependency-version: 17.1.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: dev-minor-and-patch-dependencies ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
It is unclear why this is failing.
This could cause the all green check to believe everything passed while some checks did not pass due to former name conflicts. To verify this is not causing issues in the future, a CI check is added that tests against duplicate names.
Bun does not emit the same events as Node.js does. This fixes it by adding additional listeners that might indicate the end as well as only counting down once, no matter how many listeners indicate an end. --------- Co-authored-by: Ruben Bridgewater <ruben@bridgewater.de>
Prevent event listeners in setupAssertionListeners from firing multiple times and overwriting trace IDs. The 'message' listener now removes itself after capturing the first valid span, and the 'debugger-input' listener uses .once() instead of .on(). This fixes flaky test failures where subsequent HTTP requests would update the traceId/spanId variables, causing assertion mismatches.
Parallel integration tests were bottlenecked by multiple processes simultaneously packing dd-trace.tgz during sandbox creation. This caused severe I/O contention. Cache the tarball at the sandboxRoot level so it's only created once per test run, then reused by all parallel workers. Debugger integration tests performance improvement example: - Sequential: 5:06 - Parallel (--jobs 2) before: 6:47 (slower due to contention) - Parallel (--jobs 2) after: 3:02 (40% faster than sequential) - System CPU overhead reduced from 424s to 164s.
When a probe is hit, stack traces now correctly show original source file locations (e.g., TypeScript files) instead of transpiled code, matching the behavior of breakpoint locations. Added async source map transformation to getStackFromCallFrames using the existing SourceMapConsumer.originalPositionFor method. Stack frames gracefully fall back to generated positions if transformation fails.
The added commit does not match our permission claim, so we just skip that part in case it is not pushed by dependabot anymore.
Contributor
Overall package sizeSelf size: 4.58 MB Dependency sizes| name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 2.0.6 | 81.92 kB | 813.08 kB | | dc-polyfill | 0.1.10 | 26.73 kB | 26.73 kB |🤖 This report was automatically generated by heaviest-objects-in-the-universe |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## v5.x #7399 +/- ##
==========================================
- Coverage 83.19% 80.26% -2.94%
==========================================
Files 476 731 +255
Lines 20153 31093 +10940
==========================================
+ Hits 16766 24956 +8190
- Misses 3387 6137 +2750 Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This is just running auto-fix on the code without further changes.
* refactor(proxy): extract serverless check to module Remove Config._isInServerlessEnvironment() method and use isInServerlessEnvironment() directly from serverless module throughout the codebase. Simplifies lazyProxy signature by removing config parameter that was only needed for this check. * perf: cache serverless environment check at startup Cache the result of isInServerlessEnvironment() at module load time to avoid repeated checks during runtime. The serverless environment status is static and doesn't change during execution, making this a safe optimization for hot path code.
…6807) Don't throw if the probe config doesn't contain a `capture` object when `captureSnapshot: true`. Currently, all probe config objects does contain a `capture` object, but in case this ever changes in the future we'll be prepared.
Our test optimization product is using fixture files that are automatically picked up by CodeCov. Instead of having to declare our coverage files in all CI scripts, this just renames these fixture files, since they are just imported to read their content. To prevent future issues, a small lint job is added that prevents names that might be detected as coverage. The scripts are now way simpler due to the fix and a dynamic config is used for nyc. While this officially 'decreases' our coverage, the opposite is actually the case. We now track 9000 more lines and 200 files of which we cover 6000 lines. That is why our overall coverage drops while we actually properly handle the coverage now. All our CodeCov uploaded files are now correct and none create errors. This PR makes our coverage way more solid and is a first step towards also tracking the coverage of integration tests. Coverage is now tracked in individual folders per Node.js version and plugin name. That way CodeCov is merging the coverage instead of nyc and we never replace existing coverage.
…pdates (#7405) Bumps the test-versions group with 2 updates in the /integration-tests/esbuild directory: [axios](https://github.com/axios/axios) and [openai](https://github.com/openai/openai-node). Updates `axios` from 1.13.3 to 1.13.4 - [Release notes](https://github.com/axios/axios/releases) - [Changelog](https://github.com/axios/axios/blob/v1.x/CHANGELOG.md) - [Commits](axios/axios@v1.13.3...v1.13.4) Updates `openai` from 6.16.0 to 6.17.0 - [Release notes](https://github.com/openai/openai-node/releases) - [Changelog](https://github.com/openai/openai-node/blob/master/CHANGELOG.md) - [Commits](openai/openai-node@v6.16.0...v6.17.0) --- updated-dependencies: - dependency-name: axios dependency-version: 1.13.4 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: test-versions - dependency-name: openai dependency-version: 6.17.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: test-versions ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
dd-octo-sts
Bot
force-pushed
the
v5.85.0-proposal
branch
from
February 2, 2026 05:34
88cd958 to
0fd2c9b
Compare
…ess methods (#7408) * Move stateless methods outside of Config class * Group private/public methods in Config class * Use proper private methods instead of underscore-prefix * Remove whitespace * Move functions below Config class + export above * Capitalize constants + use Maps for map-like objects * Use node: prefix for core modules * Group imports correctly * Move configInstance variable to the top
Add support for the v2 debugger input endpoint (/debugger/v2/input) with intelligent fallback to the diagnostics endpoint (/api/v2/debugger) when the agent does not support v2 or returns 404. The implementation: - Detects v2 endpoint support from agent /info response - Automatically falls back to diagnostics endpoint if v2 is not advertised - Handles runtime 404 errors from v2 endpoint by falling back to diagnostics - Ensures remote config handler is registered synchronously to avoid race conditions This change maintains backward compatibility with older agents while enabling new v2 functionality when available.
…st (#7356) * feat(test-optimization): create final_status tag on test event for jest * fix: errors after resolving rebase conflicts * test: add assertion to test name * refactor: extract functions to get final state for better readability * refactor: merge finalStatus function into restuls function for better readability * fix: efd enabled but not active caused final_status tag to be set on every retry * style: fix lint violations
* update active function * comments --------- Co-authored-by: Thomas Watson <thomas.watson@datadoghq.com>
Co-authored-by: campaigner-prod[bot] <87874424+campaigner-prod[bot]@users.noreply.github.com>
Replace string concatenation and callback-style logging with printf-style formatting (%d, %s placeholders) for better performance and consistency. Merge multi-line concatenated strings into single strings with eslint max-len disables where needed to avoid unnecessary string operations.
While transforming assertions, some variables were accidentally detected as name. Thus, the assertion lost their meaning. This fixes these mistakes and makes sure the assertions work again.
* add container id to dogstatsd metrics * add docker helper tests * prevent real container id from being used in test
…gnored in cypress (#7442)
) The instrumentation was noticable by a user so far and hooked into the public event listener addition / removal. The event handlers being added could not be detected as not being the ones by the user. That is improved here by using an internal only instrumentation that should also reduce overhead and needs less coordination between the additions and removals. The plugin is refactored to have less memory retantion. So far, the whole span was attached and in case long living ws connections existed with a high load, the spans could add up without ever being freed. Now that tracking overhead is much lower and the span is created when needed.
Makes the ESM tests for bullmq comply with the current norms and verifies we properly instrument all esm exports.
* added filter into ts definition file * moved queryStringObfuscation and resourceRenamingEnabled * Added e2e tests for filter config in http http2 and redis * added tests for resourceRenamingEnabled in http * added tests for queryStringObfuscation * added test to verify multiple filters don't conflict
* initial changes * Update tests * removed unsused tests * remove require * remove unhelpful comments * Update packages/datadog-plugin-prisma/test/index.spec.js Co-authored-by: Ruben Bridgewater <ruben@bridgewater.de> --------- Co-authored-by: Ruben Bridgewater <ruben@bridgewater.de>
dd-octo-sts
Bot
force-pushed
the
v5.85.0-proposal
branch
from
February 5, 2026 20:32
b720fd5 to
188cf16
Compare
BridgeAR
approved these changes
Feb 5, 2026
BridgeAR
marked this pull request as ready for review
February 5, 2026 20:41
BridgeAR
enabled auto-merge (rebase)
February 5, 2026 20:43
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
11 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
1630815773] - (SEMVER-PATCH) refactor(prisma): Prisma instrumentation rewrite (Bowen Brooks) #7447de724d0a73] - (SEMVER-PATCH) chore: update pr template to not show warning in pr (Roch Devost) #742880571e0927] - (SEMVER-PATCH) fix(ts): added filter into ts definition file (Pablo Erhard) #736893f7c4b262] - (SEMVER-PATCH) Upgrade pprof-nodejs to 5.13.3 (Attila Szegedi) #7449d86c16b52f] - (SEMVER-PATCH) chore(tests): added varySandbox for bullmq ESM tests (Pablo Erhard) #7448c195ee802f] - (SEMVER-PATCH) fix: reduce memory allocation and use transparent instrumentation (Ruben Bridgewater) #7422d5e899b9d1] - (SEMVER-PATCH) [test optimization] Fix quarantined tests being skipped rather than ignored in cypress (Juan Antonio Fernández de Alba) #744261ead8491c] - (SEMVER-PATCH) fix: add container id to dogstatsd metrics (Roch Devost) #7435c5e98c0a86] - (SEMVER-PATCH) test: fix assertion transform regression (Juan Antonio Fernández de Alba) #7443fe7175a117] - (SEMVER-PATCH) refactor(esbuild): extract logging into module (Thomas Watson) #74393c1cd8aefd] - (SEMVER-PATCH) chore(types): fix TypeScript errors in the esbuild package (Thomas Watson) #743833297e0eb0] - (SEMVER-PATCH) chore(lint): run linting on the root of package folders (Thomas Watson) #74374afd74ae7a] - (SEMVER-PATCH) ci: sign commits created by bots (Ruben Bridgewater) #7397313fe22142] - (SEMVER-PATCH) fix: fix minor azure-function and mariadb bugs (simon-id) #7007da07fea5f8] - (SEMVER-PATCH) chore(tests): added ESM test for light-my-request (Pablo Erhard) #743309945cd21f] - (SEMVER-PATCH) chore(debugger): add name to worker thread (Thomas Watson) #7432ff416e6ff1] - (SEMVER-PATCH) chore(deps-dev): bump p-limit from 3.1.0 to 7.2.0 (dependabot[bot]) #74255e2f01325c] - (SEMVER-PATCH) chore(deps): bump the gh-actions-packages group across 4 directories with 3 updates (dependabot[bot]) #74263acebd003e] - (SEMVER-PATCH) chore(deps-dev): bump the dev-minor-and-patch-dependencies group across 1 directory with 4 updates (dependabot[bot]) #7424a679e827c0] - (SEMVER-PATCH) chore: update iitm and fix initialize (Ruben Bridgewater) #738776e93d7934] - (SEMVER-PATCH) fix(ws): handle removing untracked event listeners (Crystal Luc-Magloire) #7419d308b93ad0] - (SEMVER-PATCH) chore: activate unicorn/no-array-for-each eslint rule (Ruben Bridgewater) #74039ea0f34154] - (SEMVER-PATCH) fix: update vendoring to keep class names and function names (Roch Devost) #741641c8ca6162] - (SEMVER-PATCH) refactor(debugger): add JSDoc types for CDP getProperties (Thomas Watson) #7418e19ce95452] - (SEMVER-PATCH) test: added sqlite3 into knex externals and add process.exit patch (Pablo Erhard) #741368206c0dbb] - (SEMVER-PATCH) chore: fix flakiness report, lint scripts, and fix appsec test (Ruben Bridgewater) #73962e337bd1be] - (SEMVER-PATCH) refactor(logging): use printf-style formatting (Thomas Watson) #7409627ba54479] - (SEMVER-PATCH) move bun cache to under node_modules (Roch Devost) #74073eb1d7a879] - (SEMVER-PATCH) chore(ci) update one-pipeline (campaigner-prod[bot]) #741160565f1c6e] - (SEMVER-PATCH) Fix: Update Opentelemety Active Functionality (Bowen Brooks) #7319d399d0f641] - (SEMVER-MINOR) feat(test-optimization): create final_status tag on test event for jest (Calvin Bayer) #73560b299bca72] - (SEMVER-MINOR) feat(debugger): add support for v2 input endpoint detection (Thomas Watson) #73080ac57880dc] - (SEMVER-PATCH) refactor(config): modernize Config class structure and extract stateless methods (Thomas Watson) #7408c80bbc29c0] - (SEMVER-PATCH) chore(deps): bump the test-versions group across 1 directory with 2 updates (dependabot[bot]) #7405657b4f922d] - (SEMVER-PATCH) ci: fix coverage reports and prevent detecting fixture files (Ruben Bridgewater) #7391d19c433535] - (SEMVER-PATCH) fix(debugger): don't expect probe config to containcaptureobject (Thomas Watson) #680775b67c268a] - (SEMVER-PATCH) perf: cache serverless environment detection at startup (Thomas Watson) #74044a8a494c6e] - (SEMVER-PATCH) chore: activate comma-dangle eslint rule (Ruben Bridgewater) #7402a894629556] - (SEMVER-PATCH) ci: fix added commits breaking CI validation (Ruben Bridgewater) #73931f1a5024a8] - (SEMVER-MINOR) Emit the number of loaded source maps in the profile metadata (Attila Szegedi) #7382f8c43343e7] - (SEMVER-PATCH) fix(debugger): apply source maps to probe stack traces (Thomas Watson) #733662d6580053] - (SEMVER-PATCH) perf(tests): cache dd-trace tarball for sandbox creation (Thomas Watson) #735138b60f7e9f] - (SEMVER-PATCH) test(debugger): fix race condition in test helpers (Thomas Watson) #739499aba5a361] - (SEMVER-MINOR) [test optimization][SDTEST-3004] Upload code coverage reports automatically (Juan Antonio Fernández de Alba) #7335b42c5f3da3] - (SEMVER-PATCH) fix: maximum attempts on running uppon bun-js (Everton Segur) #72843523cd6828] - (SEMVER-PATCH) ci: fix workflow name conflicts (Ruben Bridgewater) #7388c273d49ba1] - (SEMVER-PATCH) ci: debug flakiness report (Ruben Bridgewater) #7380905f387f4b] - (SEMVER-PATCH) ci: fix github rate limit (Ruben Bridgewater) #73897d6c9f0e66] - (SEMVER-PATCH) chore(deps-dev): bump the dev-minor-and-patch-dependencies group across 1 directory with 3 updates (dependabot[bot]) #7392237283029b] - (SEMVER-PATCH) ci: fix yarn deduping working for any PR (Ruben Bridgewater) #7364d0c5b45957] - (SEMVER-PATCH) ci: commit vendored files with octo-sts (Ruben Bridgewater) #7381bdb76d3219] - (SEMVER-PATCH) chore(test): Fix iast overhead controller flaky test (Ugaitz Urien) #7384