⚗️[MFE] Source code context event enrichment

[bcaudan]

Motivation

In the context of micro frontend support, we add the ability to enrich events with source code context (service and version) from window.DD_SOURCE_CODE_CONTEXT.
DD_SOURCE_CODE_CONTEXT is injected at build time by the build plugin.

Changes

• Add sourceCodeContext.ts.
• Add rawRumEvent and domainContext to the assemble hook parameters to allow extracting the stack for sourceCodeContext.

Test instructions

Checklist

• Tested locally
• Tested on staging
• Added unit tests for this change.
• Added e2e/integration tests for this change.

[cit-pr-commenter]

Bundles Sizes Evolution

📦 Bundle Name	Base Size	Local Size	𝚫	𝚫%	Status
Rum	164.35 KiB	165.28 KiB	+956 B	+0.57%	✅
Rum Profiler	4.33 KiB	4.33 KiB	0 B	0.00%	✅
Rum Recorder	20.04 KiB	20.04 KiB	0 B	0.00%	✅
Logs	56.14 KiB	56.21 KiB	+72 B	+0.13%	✅
Flagging	944 B	944 B	0 B	0.00%	✅
Rum Slim	121.57 KiB	122.46 KiB	+918 B	+0.74%	✅
Worker	23.63 KiB	23.63 KiB	0 B	0.00%	✅

🚀 CPU Performance

Action Name	Base CPU Time (ms)	Local CPU Time (ms)	𝚫%
RUM - add global context	0.0048	0.0056	+16.67%
RUM - add action	0.0146	0.0206	+41.10%
RUM - add error	0.0134	0.0301	+124.63%
RUM - add timing	0.0031	0.0047	+51.61%
RUM - start view	0.0036	0.0037	+2.78%
RUM - start/stop session replay recording	0.0008	0.0012	+50.00%
Logs - log message	0.0151	0.0331	+119.21%

🧠 Memory Performance

Action Name	Base Memory Consumption	Local Memory Consumption	𝚫
RUM - add global context	26.64 KiB	25.69 KiB	-971 B
RUM - add action	50.13 KiB	51.49 KiB	+1.36 KiB
RUM - add timing	26.76 KiB	24.76 KiB	-2.00 KiB
RUM - add error	56.21 KiB	54.02 KiB	-2.19 KiB
RUM - start/stop session replay recording	25.57 KiB	24.64 KiB	-957 B
RUM - start view	424.67 KiB	425.07 KiB	+415 B
Logs - log message	47.45 KiB	107.43 KiB	+59.97 KiB

🔗 RealWorld

[datadog-datadog-prod-us1]

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

🎯 Code Coverage
• Patch Coverage: 75.00%
• Overall Coverage: 77.27% (-0.03%)

View detailed report

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 31e5ac1 | Docs | Datadog PR Page | Was this helpful? Give us feedback!}

[bcaudan]

❓ question: ‏What is the intent with the DeepReadonly?

[amortemousque]

I don’t want registered assemble hook callbacks to mutate the input, especially rawRumEvent. I avoided DeepReadonly<AssembleHookParams> because it forces casts for primitives (eventType, startTime, duration) throughout the code.

Example:

[bcaudan]

oh ok, what about adding a comment with the code to make the intent more explicit?

[bcaudan]

💬 suggestion: ‏It could be interesting to add e2e tests for this feature

[amortemousque]

Sure, added :) https://github.com/DataDog/browser-sdk/pull/3926/files#diff-88fb28376084e0b79b43bf21317d313440ea697f26ceeb4b8c093452be55d4e5

[bcaudan]

LGTM

[bcaudan]

💬 suggestion: ‏unclear to me what information we want to convey here and how it relates to create body 😕

[amortemousque]

Yeah, it’s a copy-paste from here. It means we use createBody instead of page.evaluate() because page.evaluate() messes up the stack trace and drops the URL. I’ll improve the comments.

[BeltranBulbarellaDD]

LGTM!