Skip to content

Conversation

@watson
Copy link
Collaborator

@watson watson commented Nov 15, 2025

What does this PR do?

As the title says...

Motivation

Make the code easier to understand.

@github-actions
Copy link

github-actions bot commented Nov 15, 2025

Overall package size

Self size: 13.27 MB
Deduped: 116.17 MB
No deduping: 131.19 MB

Dependency sizes | name | version | self size | total size | |------|---------|-----------|------------| | @datadog/libdatadog | 0.7.0 | 35.02 MB | 35.02 MB | | @datadog/native-appsec | 10.3.0 | 20.73 MB | 20.74 MB | | @datadog/native-iast-taint-tracking | 4.0.0 | 11.72 MB | 11.73 MB | | @datadog/pprof | 5.12.0 | 11.19 MB | 11.57 MB | | @opentelemetry/resources | 1.30.1 | 557.67 kB | 7.71 MB | | @opentelemetry/core | 1.30.1 | 908.66 kB | 7.16 MB | | protobufjs | 7.5.4 | 2.95 MB | 5.83 MB | | @datadog/wasm-js-rewriter | 5.0.1 | 2.82 MB | 3.53 MB | | @datadog/native-metrics | 3.1.1 | 1.02 MB | 1.43 MB | | @opentelemetry/api-logs | 0.208.0 | 199.48 kB | 1.42 MB | | @opentelemetry/api | 1.9.0 | 1.22 MB | 1.22 MB | | jsonpath-plus | 10.3.0 | 617.18 kB | 1.08 MB | | import-in-the-middle | 1.15.0 | 127.66 kB | 856.24 kB | | lru-cache | 10.4.3 | 804.3 kB | 804.3 kB | | @datadog/openfeature-node-server | 0.2.0 | 118.51 kB | 437.19 kB | | opentracing | 0.14.7 | 194.81 kB | 194.81 kB | | source-map | 0.7.6 | 185.63 kB | 185.63 kB | | pprof-format | 2.2.1 | 163.06 kB | 163.06 kB | | @datadog/sketches-js | 2.1.1 | 109.9 kB | 109.9 kB | | @isaacs/ttlcache | 2.1.1 | 90.58 kB | 90.58 kB | | lodash.sortby | 4.7.0 | 75.76 kB | 75.76 kB | | ignore | 7.0.5 | 63.38 kB | 63.38 kB | | istanbul-lib-coverage | 3.2.2 | 34.37 kB | 34.37 kB | | rfdc | 1.4.1 | 27.15 kB | 27.15 kB | | dc-polyfill | 0.1.10 | 26.73 kB | 26.73 kB | | tlhunter-sorted-set | 0.1.0 | 24.94 kB | 24.94 kB | | shell-quote | 1.8.3 | 23.74 kB | 23.74 kB | | limiter | 1.1.5 | 23.17 kB | 23.17 kB | | retry | 0.13.1 | 18.85 kB | 18.85 kB | | semifies | 1.0.0 | 15.84 kB | 15.84 kB | | jest-docblock | 29.7.0 | 8.99 kB | 12.76 kB | | crypto-randomuuid | 1.0.0 | 11.18 kB | 11.18 kB | | ttl-set | 1.0.0 | 4.61 kB | 9.69 kB | | mutexify | 1.4.0 | 5.71 kB | 8.74 kB | | path-to-regexp | 0.1.12 | 6.6 kB | 6.6 kB | | module-details-from-path | 1.0.4 | 3.96 kB | 3.96 kB | | escape-string-regexp | 5.0.0 | 3.66 kB | 3.66 kB |

🤖 This report was automatically generated by heaviest-objects-in-the-universe

Copy link
Collaborator Author

watson commented Nov 15, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@codecov
Copy link

codecov bot commented Nov 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.61%. Comparing base (2dd3573) to head (90970a9).
⚠️ Report is 8 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #6921   +/-   ##
=======================================
  Coverage   84.61%   84.61%           
=======================================
  Files         505      505           
  Lines       21171    21171           
=======================================
  Hits        17913    17913           
  Misses       3258     3258           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pr-commenter
Copy link

pr-commenter bot commented Nov 15, 2025

Benchmarks

Benchmark execution time: 2025-11-21 11:15:31

Comparing candidate commit 90970a9 in PR branch watson/DEBUG-4727/refactor-snapshot-collector with baseline commit 2dd3573 in branch master.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 292 metrics, 28 unstable metrics.

@watson watson force-pushed the watson/DEBUG-4727/debugger-snapshot-time-budget branch from 495678c to cd88600 Compare November 17, 2025 10:43
@watson watson force-pushed the watson/DEBUG-4727/refactor-snapshot-collector branch from b1e29d7 to c134d8c Compare November 17, 2025 10:43
@watson watson force-pushed the watson/DEBUG-4727/debugger-snapshot-time-budget branch from cd88600 to 8b643eb Compare November 17, 2025 10:51
@watson watson force-pushed the watson/DEBUG-4727/refactor-snapshot-collector branch from c134d8c to 4e91f48 Compare November 17, 2025 10:51
@watson watson changed the base branch from watson/DEBUG-4727/debugger-snapshot-time-budget to graphite-base/6921 November 17, 2025 14:36
@watson watson force-pushed the watson/DEBUG-4727/refactor-snapshot-collector branch from 4e91f48 to d9b3bae Compare November 20, 2025 09:04
@watson watson force-pushed the graphite-base/6921 branch from 1f3788c to bed66ba Compare November 20, 2025 09:58
@watson watson force-pushed the watson/DEBUG-4727/refactor-snapshot-collector branch from d9b3bae to 163850e Compare November 20, 2025 09:58
@watson watson changed the base branch from graphite-base/6921 to watson/DEBUG-4727/debugger-snapshot-time-budget November 20, 2025 09:59
@datadog-official

This comment has been minimized.

@watson watson force-pushed the watson/DEBUG-4727/debugger-snapshot-time-budget branch from bed66ba to 5976953 Compare November 20, 2025 12:50
@watson watson force-pushed the watson/DEBUG-4727/refactor-snapshot-collector branch from 163850e to 265ce0f Compare November 20, 2025 12:50
Base automatically changed from watson/DEBUG-4727/debugger-snapshot-time-budget to master November 21, 2025 11:00
Rename a few function names that were ambiguous.
@watson watson force-pushed the watson/DEBUG-4727/refactor-snapshot-collector branch from 265ce0f to 90970a9 Compare November 21, 2025 11:07
@watson watson marked this pull request as ready for review November 21, 2025 11:08
@watson watson requested review from a team as code owners November 21, 2025 11:08
@watson watson added the debugger Dynamic Instrumentation & Live Debugger label Nov 21, 2025
@BridgeAR BridgeAR merged commit fcddd26 into master Nov 23, 2025
787 checks passed
@BridgeAR BridgeAR deleted the watson/DEBUG-4727/refactor-snapshot-collector branch November 23, 2025 03:06
dd-octo-sts bot pushed a commit that referenced this pull request Nov 23, 2025
Rename a few function names that were ambiguous.
@dd-octo-sts dd-octo-sts bot mentioned this pull request Nov 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

debugger Dynamic Instrumentation & Live Debugger semver-patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants