Skip to content

fix(injection): keep Rollup debug IDs in upload prefix - #488

Open
jhssilva wants to merge 1 commit into
masterfrom
hugo.silva/rum-18038-rollup-debug-id
Open

fix(injection): keep Rollup debug IDs in upload prefix#488
jhssilva wants to merge 1 commit into
masterfrom
hugo.silva/rum-18038-rollup-debug-id

Conversation

@jhssilva

@jhssilva jhssilva commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

What does this change?

Fixes Rollup sourcemap uploads that can miss the entry bundle's debug ID when a later transform pushes the complete ddDebugId value outside Error Tracking's initial read.

  • Runs Datadog's Rollup renderChunk injection with order: 'post', after normal transforms such as Terser.
  • Places ddDebugId first in the injected source-code context.

Progressive debug-ID discovery is intentionally handled separately by the follow-up PR linked below, because it changes shared Error Tracking behavior for every bundler.

Related fixes:

  • PR #487 fixes the esbuild artifact-readiness race.
  • PR #489 adds progressive debug-ID discovery for all bundlers.

Tracking issue: RUM-18038

Validation

  • Focused regression tests: 10/10 passed.
  • Lint, full typechecking, integrity, and dependency checks passed.
  • Static production builds and browser runtime registration passed for webpack, Vite, Rollup, esbuild, and Rspack.
  • Live Rollup staging test: 4/4 debug IDs extracted before upload and 2/2 errors deobfuscated.
  • Live webpack, Vite, and Rspack staging tests: all uploaded sourcemaps contained debug IDs and 2/2 errors deobfuscated for each bundler.
  • Exact esbuild PR fix(error-tracking): wait for debug ID injection before sourcemap upload #487 package: 7/7 debug IDs extracted and 2/2 errors deobfuscated.

@jhssilva
jhssilva marked this pull request as ready for review August 14, 2026 11:56
@jhssilva
jhssilva requested review from a team as code owners August 14, 2026 11:56
@jhssilva
jhssilva requested review from tyffical and removed request for a team August 14, 2026 11:56
@jhssilva

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create a Codex account and connect to github.

@jhssilva
jhssilva requested a review from buranmert August 14, 2026 12:06
@jhssilva jhssilva changed the title fix(injection): keep Rollup debug IDs in upload prefix fix(error-tracking): find debug IDs progressively Aug 14, 2026
@jhssilva jhssilva changed the title fix(error-tracking): find debug IDs progressively fix(injection): keep Rollup debug IDs in upload prefix Aug 14, 2026
@jhssilva
jhssilva force-pushed the hugo.silva/rum-18038-rollup-debug-id branch from 817cc2d to 83a82fb Compare August 14, 2026 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant