💥 Rename chunk files to include "datadog" prefix for cross-bundler naming

[thomas-lebeau]

Motivation

Chunk files produced by the SDK (recorder, profiler) use generic names like recorder-<hash> and profiler-<hash>. These can collide with chunks from other libraries or the application itself. Prefixing them with datadog- makes them easily identifiable and avoids naming conflicts across bundlers.

Changes

• Rename startRecording.ts to datadog-recorder.ts and profiler.ts to datadog-profiler.ts
• Update webpack chunk names from recorder/profiler to datadogRecorder/datadogProfiler
• Update deploy scripts, bundle size computation, and associated tests to match the new naming convention

Test instructions

• yarn build && yarn build:apps and verify:
  • [webpack] the chunk files under packages/rum/bundle/chunks/ are named datadogRecorder_<hash>-datadog-rum.js and datadogProfiler_<hash>-datadog-rum.js
  • [vite/rollup] the chunk files under developer-extension/dist/chrome-mv3/chunks are named datadogRecorder_<hash>.js and datadogProfiler_<hash>.js
• Run yarn test:unit --spec scripts/deploy/deploy.spec.ts to verify deploy tests pass
• bundle size evolution comment still work as expected

Checklist

• Tested locally
• Tested on staging
• Added unit tests for this change.
• Added e2e/integration tests for this change.
• Updated documentation and/or relevant AGENTS.md file

[cit-pr-commenter-54b7da]

Bundles Sizes Evolution

📦 Bundle Name	Base Size	Local Size	𝚫	𝚫%	Status
Rum	N/A	172.16 KiB	N/A	N/A	N/A
Rum Profiler	N/A	6.19 KiB	N/A	N/A	N/A
Rum Recorder	N/A	26.97 KiB	N/A	N/A	N/A
Logs	N/A	56.50 KiB	N/A	N/A	N/A
Rum Slim	N/A	129.50 KiB	N/A	N/A	N/A
Worker	N/A	23.63 KiB	N/A	N/A	N/A

🚀 CPU Performance

Action Name	Base CPU Time (ms)	Local CPU Time (ms)	𝚫%
addglobalcontext	N/A	0.0049	N/A
addaction	N/A	0.0161	N/A
adderror	N/A	0.0166	N/A
addtiming	N/A	0.0034	N/A
startview	N/A	0.0145	N/A
startstopsessionreplayrecording	N/A	0.0007	N/A
logmessage	N/A	0.0175	N/A

🧠 Memory Performance

Action Name	Base Memory Consumption	Local Memory Consumption	𝚫
addglobalcontext	N/A	33.45 KiB	N/A
addaction	N/A	58.80 KiB	N/A
addtiming	N/A	34.38 KiB	N/A
adderror	N/A	63.46 KiB	N/A
startstopsessionreplayrecording	N/A	33.70 KiB	N/A
startview	N/A	464.25 KiB	N/A
logmessage	N/A	50.29 KiB	N/A

🔗 RealWorld

[datadog-official]

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

🎯 Code Coverage (details)
• Patch Coverage: 0.00%
• Overall Coverage: 77.29% (+0.00%)

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: ae34631 | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback!}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b20a529b8c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[BenoitZugmeyer]

suggestion: could we stick with the naming convention and use datadogRecorder.ts?

[mormubis]

startRecording.spec.ts was renamed to datadog-recorder.spec.ts, but profiler.spec.ts kept its old name. Is this intentional?