Skip to content

♻️ [RUM-16925] switch debug_ids to recommended URL/Debug-ID array format#4862

Merged
amortemousque merged 1 commit into
mainfrom
aymeric/change-debug-id-format
Jul 10, 2026
Merged

♻️ [RUM-16925] switch debug_ids to recommended URL/Debug-ID array format#4862
amortemousque merged 1 commit into
mainfrom
aymeric/change-debug-id-format

Conversation

@amortemousque

Copy link
Copy Markdown
Collaborator

Summary

  • _dd.debug_ids on RUM error and long task/LoAF events now serializes as an array of { url, id } entries instead of a { [url]: id } map, per the recommended format — the map shape was discouraged because keys containing dots get split into nested objects by the event platform.
  • Bumped @datadog/rum-events-format pin to 31cc845 (already merged upstream) and regenerated rumEvent.types.ts via yarn json-schemas:sync.
  • buildDebugIdByUrl() in sourceCodeContext.ts now returns DebugIdEntry[] | undefined instead of a map (same name/signature otherwise).
  • Added 'array' as a valid ModifiableFieldPaths type so beforeSend redaction still works on _dd.debug_ids.

Test plan

  • yarn typecheck passes
  • yarn lint passes
  • Updated unit tests in sourceCodeContext.spec.ts, error.spec.ts, errorCollection.spec.ts, longTaskCollection.spec.ts, assembly.spec.ts, createErrorFieldFromRawError.spec.ts — all green
  • Updated E2E assertions in microfrontend.scenario.ts to match the array shape

The map-of-URL-to-id shape was discouraged because keys containing dots
get split into nested objects by the event platform. Switch to an array
of `{ url, id }` entries per the RFC's recommended format.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@datadog-datadog-us1-prod

datadog-datadog-us1-prod Bot commented Jul 9, 2026

Copy link
Copy Markdown

Pipelines  Tests

Fix all issues with BitsAI

⚠️ Warnings

🚦 1 Pipeline job failed

Lint PR title | lint   View in Datadog   GitHub Actions

ℹ️ Info

No other issues found (see more)

🧪 All tests passed
❄️ No new flaky tests detected

🎯 Code Coverage (details)
Patch Coverage: 66.67%
Overall Coverage: 77.26% (+0.00%)

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: d5ccc16 | Docs | Datadog PR Page | Give us feedback!

@amortemousque amortemousque changed the title [RUM-16925] switch debug_ids to recommended URL/Debug-ID array format ♻️ [RUM-16925] switch debug_ids to recommended URL/Debug-ID array format Jul 9, 2026
@amortemousque amortemousque marked this pull request as ready for review July 9, 2026 15:39
@amortemousque amortemousque requested a review from a team as a code owner July 9, 2026 15:39
@cit-pr-commenter-54b7da

Copy link
Copy Markdown

Bundles Sizes Evolution

📦 Bundle Name Base Size Local Size 𝚫 𝚫% Status
Rum 178.25 KiB 178.37 KiB +119 B +0.07%
Rum Profiler 8.22 KiB 8.22 KiB 0 B 0.00%
Rum Recorder 21.14 KiB 21.14 KiB 0 B 0.00%
Logs 55.47 KiB 55.53 KiB +55 B +0.10%
Rum Slim 136.17 KiB 136.27 KiB +102 B +0.07%
Worker 22.96 KiB 22.96 KiB 0 B 0.00%

@sbarrio sbarrio requested a review from thomas-lebeau July 10, 2026 07:06
@amortemousque amortemousque merged commit c65e3bd into main Jul 10, 2026
31 of 32 checks passed
@amortemousque amortemousque deleted the aymeric/change-debug-id-format branch July 10, 2026 08:45
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 10, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants