v5.82.0 proposal #7136
Draft
v5.82.0 proposal #7136
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
…on by default (#7119) * Update to pprof 5.13.1 * Profiler can now use AsyncContextFrame starting with Node.js 22.9.0 * Use AsyncContextFrame-based profiling sample context tracking by default, where available. * Add tests * Do not use options.useAsyncContextFrame
* Check DD_AZURE_RESOURCE_GROUP env var when detecting resource group metadata for Azure Flex Consumption Functions * Add tests, add DD_AZURE_RESOURCE_GROUP to supported configurations * Make flex consumption function detection a function
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## v5.x #7136 +/- ##
==========================================
+ Coverage 83.19% 84.64% +1.45%
==========================================
Files 476 522 +46
Lines 20153 22233 +2080
==========================================
+ Hits 16766 18819 +2053
- Misses 3387 3414 +27 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Overall package sizeSelf size: 4.36 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 |
This comment has been minimized.
This comment has been minimized.
The dd-license-attribution tool requires a GitHub token to avoid API rate limits. This change explicitly passes the GITHUB_TOKEN as an environment variable to the license regeneration step.
* Added support for iitm v2.0.0 * Update iitm with master package.json and yarn.lock
Automated rewrites including a lint:fix call. Afterwards manual fixes and small improvements.
Adds unit tests with mocked time for the debugger snapshot collector deadline mechanism and removes strict timing assertions from the flaky 1ms budget integration test. The integration test was failing in CI due to unpredictable execution times in GitHub Actions environments. The test set a 1ms capture timeout and expected thread pause time ≤16ms, which was too tight for resource-constrained CI environments.
Add a new "Sign your commits" section to the contributing guidelines that requires all commits in pull requests to be signed. The section includes: - Explanation of the commit signing requirement - Step-by-step instructions for setting up GPG key signing - Instructions for automatic commit signing configuration - Guide for retroactively signing existing commits This ensures the authenticity and integrity of contributions to the project.
dd-octo-sts
bot
force-pushed
the
v5.82.0-proposal
branch
from
b21ef1d to
ac50e2d
Compare
Version 2.0.1 was just released which causes issue with some of our CI tests. It's too early to say if this is only a CI issue, but until we can investigate, it's better to pin it.
Add comprehensive developer guide (AGENTS.md) and enhance CONTRIBUTING.md with detailed documentation covering: - Project setup, structure, and overview - Testing workflows (unit tests, integration tests, plugin tests) - Code style and linting standards (import ordering, ECMAScript/Node.js APIs) - Performance and memory considerations (async/await restrictions, array iteration) - Development workflow best practices (backportability, refactoring principles) - How to add new configuration options and instrumentations - Debugging, logging, and error handling patterns The AGENTS.md file follows the agents.md spec and provides a quick reference for common development tasks and established best practices to help both AI coding assistants and human contributors navigate the codebase effectively. Includes CLAUDE.md symlink for compatibility with Claude-based tools. This reduces onboarding friction, enforces consistency, prevents common mistakes, and improves code quality by making best practices discoverable.
dd-octo-sts
bot
force-pushed
the
v5.82.0-proposal
branch
from
ac50e2d to
4a3bcf0
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.
4b40f00f78] - (SEMVER-PATCH) docs: add AGENTS.md and update CONTRIBUTING.md (Thomas Watson) #71097f5a9d30c0] - (SEMVER-PATCH) refactor(config): use getEnv instead of getEnvironmentVariables (Thomas Watson) #714795d4d2625e] - (SEMVER-PATCH) chore(datastreams): separate DSM specific test behavior and add to CODEOWNERS (robcarlan-datadog) #7076261e8ece5c] - (SEMVER-MINOR) feat(ws): add websocket context propagation (Thomas Hunter II) #7077244b082d00] - (SEMVER-PATCH) fix: pin import-in-the-middle to v2.0.0 (Thomas Watson) #7148edfc7c6928] - (SEMVER-PATCH) docs: add commit signing requirement to CONTRIBUTING.md (Thomas Watson) #7108a37816bd2e] - (SEMVER-PATCH) test(debugger): fix flaky time budget integration test (Thomas Watson) #7103a1f53d8fb1] - (SEMVER-PATCH) test: rewrite chai to assert (Ruben Bridgewater) #7134b26100bbeb] - (SEMVER-PATCH) add maximum node version in guardrails (Roch Devost) #67888822838ef3] - (SEMVER-PATCH) test: use partialDeepStrictEqual instead of individual assertions (Ruben Bridgewater) #7133d9a3a8ee81] - (SEMVER-PATCH) fix: failing appsec tests after iitm bump to v2.0.0 (Pablo Erhard) #69357c979d23df] - (SEMVER-PATCH) ci: pass GITHUB_TOKEN to license attribution step (Thomas Watson) #71408f86699871] - (SEMVER-PATCH) [SVLS-8161] Update Azure Function metadata detection logic (Kathie Huang) #7100b08f6e9614] - (SEMVER-MINOR) feat(profiler): AsyncContextFrame used by profiler with Node.js 22+, on by default (Attila Szegedi) #7119