v5.83.0 proposal #7270
Draft
v5.83.0 proposal #7270
+3,687
−1,242
Conversation
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
Bump `dd-license-attribution` to the latest version which fixes two issues with `LICENSE-3rdparty.csv`: - Consistently use Windows style line endings - Removes incorrect `@datadog/source-map` line from `LICENSE-3rdparty.csv` Also update `scripts/check_licenses.js` to support resolving npm aliases.
This detects test files that are not actually checked in our CI. This happened multiple times so far and also currently test files exist that are not run with our scripts. Add verify-exercised-tests workflow and add missing test files. Skip crashtracking tests on Windows. Crashtracking is not supported on Windows.
Add validation to scripts/verify-exercised-tests.js to ensure all test:integration* scripts are invoked by GitHub Actions workflows. This prevents accidentally adding integration test scripts without corresponding CI coverage. The check excludes test:integration:plugins, which is a local convenience script whose tests are already fully covered by test:plugins:ci in CI. This validation complements the existing check for :ci suffixed scripts and helps maintain comprehensive test coverage in CI.
…ss 1 directory with 7 updates (#7238) Bumps the dev-minor-and-patch-dependencies group with 7 updates in the / directory: | Package | From | To | | --- | --- | --- | | [@eslint/js](https://github.com/eslint/eslint/tree/HEAD/packages/js) | `9.39.1` | `9.39.2` | | [@msgpack/msgpack](https://github.com/msgpack/msgpack-javascript) | `3.1.2` | `3.1.3` | | [@stylistic/eslint-plugin](https://github.com/eslint-stylistic/eslint-stylistic/tree/HEAD/packages/eslint-plugin) | `5.6.1` | `5.7.0` | | [body-parser](https://github.com/expressjs/body-parser) | `2.2.1` | `2.2.2` | | [bun](https://github.com/oven-sh/bun) | `1.3.3` | `1.3.5` | | [eslint](https://github.com/eslint/eslint) | `9.39.1` | `9.39.2` | | [eslint-plugin-cypress](https://github.com/cypress-io/eslint-plugin-cypress) | `5.2.0` | `5.2.1` | Updates `@eslint/js` from 9.39.1 to 9.39.2 - [Release notes](https://github.com/eslint/eslint/releases) - [Commits](https://github.com/eslint/eslint/commits/v9.39.2/packages/js) Updates `@msgpack/msgpack` from 3.1.2 to 3.1.3 - [Release notes](https://github.com/msgpack/msgpack-javascript/releases) - [Changelog](https://github.com/msgpack/msgpack-javascript/blob/main/CHANGELOG.md) - [Commits](msgpack/msgpack-javascript@v3.1.2...v3.1.3) Updates `@stylistic/eslint-plugin` from 5.6.1 to 5.7.0 - [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.0/packages/eslint-plugin) Updates `body-parser` from 2.2.1 to 2.2.2 - [Release notes](https://github.com/expressjs/body-parser/releases) - [Changelog](https://github.com/expressjs/body-parser/blob/master/HISTORY.md) - [Commits](expressjs/body-parser@v2.2.1...v2.2.2) Updates `bun` from 1.3.3 to 1.3.5 - [Release notes](https://github.com/oven-sh/bun/releases) - [Commits](oven-sh/bun@bun-v1.3.3...bun-v1.3.5) Updates `eslint` from 9.39.1 to 9.39.2 - [Release notes](https://github.com/eslint/eslint/releases) - [Commits](eslint/eslint@v9.39.1...v9.39.2) Updates `eslint-plugin-cypress` from 5.2.0 to 5.2.1 - [Release notes](https://github.com/cypress-io/eslint-plugin-cypress/releases) - [Commits](cypress-io/eslint-plugin-cypress@v5.2.0...v5.2.1) --- updated-dependencies: - dependency-name: "@eslint/js" dependency-version: 9.39.2 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: dev-minor-and-patch-dependencies - dependency-name: "@msgpack/msgpack" dependency-version: 3.1.3 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: dev-minor-and-patch-dependencies - dependency-name: "@stylistic/eslint-plugin" dependency-version: 5.7.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: dev-minor-and-patch-dependencies - dependency-name: body-parser dependency-version: 2.2.2 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: dev-minor-and-patch-dependencies - dependency-name: bun dependency-version: 1.3.5 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: dev-minor-and-patch-dependencies - dependency-name: eslint dependency-version: 9.39.2 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: dev-minor-and-patch-dependencies - dependency-name: eslint-plugin-cypress dependency-version: 5.2.1 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: dev-minor-and-patch-dependencies ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…with 2 updates (#7237) Bumps the gh-actions-packages group with 1 update in the / directory: [actions/checkout](https://github.com/actions/checkout). Bumps the gh-actions-packages group with 1 update in the /.github/actions/node directory: [oven-sh/setup-bun](https://github.com/oven-sh/setup-bun). Bumps the gh-actions-packages group with 1 update in the /.github/workflows directory: [actions/checkout](https://github.com/actions/checkout). Updates `actions/checkout` from 5.0.0 to 6.0.1 - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v5...8e8c483) Updates `oven-sh/setup-bun` from 2.0.2 to 2.1.0 - [Release notes](https://github.com/oven-sh/setup-bun/releases) - [Commits](oven-sh/setup-bun@735343b...b7a1c7c) Updates `actions/checkout` from 5.0.0 to 6.0.1 - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v5...8e8c483) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: 6.0.1 dependency-type: direct:production update-type: version-update:semver-major dependency-group: gh-actions-packages - dependency-name: oven-sh/setup-bun dependency-version: 2.1.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: gh-actions-packages - dependency-name: actions/checkout dependency-version: 6.0.1 dependency-type: direct:production update-type: version-update:semver-major dependency-group: gh-actions-packages ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…7236) * Fixed varySandbox and added new error out when no default import match * Fixed prisma not having default export and added functionality to varySandbox * test destructure import when possible and added variant into test description
* fix: remove event emitter leak warning This is done by two steps: a) only add a single connect listener instead of two and b) increase the limit by one to behave as if there would be no limit. The downside of the second part is that the user may check for this value and would now get a different value than the originally defined one. * refactor: use .once() instead of .on() where possible The event listeners added in many circumstances should only be used once, so using once should be best. * fix: use less event beforeExit listeners This limits the usage of the beforeExit listeners by using a global dd-trace symbol across the code and using a single set of handlers that are run before the process exits. In addition, reload dogstatsd in tests to limit adding too many listeners there as well. The newly added global may be used in multiple circumstances and is a better abstraction than having multiple different symbols that are added to the global. This symbol is also non enumerable, non writable, and not configurable. * fix: use a WeakSet instead of manipulating internals When instrumenting `ai`, we now use a WeakSet for tracking instances. This prevents the need for adding a symbol to the tracer instances to know if it has been instrumented or not. * fixup! * fixup! address comments
The NoopProxy baggage methods (setBaggageItem, getBaggageItem, removeBaggageItem) had incompatible signatures with their implementations in the Tracer class, causing TypeScript errors when Tracer extended NoopProxy. Updated the method signatures to accept the expected parameters while maintaining no-op behavior.
* support for koa router v15.2.0 * skip tests for koa/router v15 in node versions less than 20
Closed
Overall package sizeSelf size: 4.4 MB Dependency sizes| name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 2.0.0 | 68.46 kB | 797.03 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 #7270 +/- ##
==========================================
+ Coverage 83.19% 85.06% +1.86%
==========================================
Files 476 532 +56
Lines 20153 22782 +2629
==========================================
+ Hits 16766 19379 +2613
- Misses 3387 3403 +16 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…7271) This config option has been available for a long time, but wasn't typed correctly.
dd-octo-sts
bot
force-pushed
the
v5.83.0-proposal
branch
from
d7dea92 to
fdee2dc
Compare
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
e3ec10f18d] - (SEMVER-PATCH) feat(esbuild): warn when using --minify w/o --keep-names (Thomas Hunter II) #72482dca0291e8] - (SEMVER-PATCH) fix(types): add dynamicInstrumentation config object to index.d.ts (Thomas Watson) #72717e8ea24386] - (SEMVER-MINOR) feat(profiling): add process tags (Thomas Hunter II) #7268234b7c8f72] - (SEMVER-PATCH) fix(integration): support for koa router v15.2.0 (Pablo Erhard) #7266de37230b67] - (SEMVER-PATCH) chore(types): add parameters to NoopProxy baggage methods (Thomas Watson) #726439c85a4da8] - (SEMVER-PATCH) fix: remove event emitter leak warning (Ruben Bridgewater) #7201a07f1f87dd] - (SEMVER-MINOR) feat: support undici via diagnostics_channel events (Stephen Belanger) #720613c1f5c901] - (SEMVER-PATCH) chore(tests): fixed varySandbox and added it into missing ESM tests (Pablo Erhard) #72364e1c2ce236] - (SEMVER-PATCH) chore(deps): bump the gh-actions-packages group across 3 directories with 2 updates (dependabot[bot]) #72372591eb1ddd] - (SEMVER-PATCH) chore(deps-dev): bump the dev-minor-and-patch-dependencies group across 1 directory with 7 updates (dependabot[bot]) #72386a3f0ec36d] - (SEMVER-PATCH) chore: verify all test:integration* scripts are exercised in CI (Thomas Watson) #72603d6e6089af] - (SEMVER-PATCH) chore(tests): add type annotations to integration test helpers (Thomas Watson) #72400b7c8ccb77] - (SEMVER-PATCH) [test optimization] Fix mocha test management logic (Juan Antonio Fernández de Alba) #7242168f7ce368] - (SEMVER-PATCH) Disable telemetry for OCE integration test (Carles Capell) #7255033993d07e] - (SEMVER-PATCH) ci: add test verify script and activate missing tests (Ruben Bridgewater) #7225f68705799a] - (SEMVER-PATCH) chore(dep): bump dd-license-attribution tool to newest version (Thomas Watson) #72462321594d45] - (SEMVER-PATCH) chore: update AGENTS.md for improved types and assertions (Ruben Bridgewater) #7209