Skip to content

feat(sourcemaps): Set has_debug_ids when assembling an artifact bundle - #130422

Open
sentry-junior[bot] wants to merge 2 commits into
feat/artifact-bundle-has-debug-ids-schemafrom
feat/artifact-bundle-write-has-debug-ids
Open

sentry-junior[bot] wants to merge 2 commits into
feat/artifact-bundle-has-debug-ids-schemafrom
feat/artifact-bundle-write-has-debug-ids

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

ArtifactBundle.has_debug_ids tells whether a bundle has debug-ID rows, so the source map debugger does not need to read those rows.

The upload sets the flag when it creates a new bundle. This is the same step that writes the debug-ID rows. A new upload of an existing bundle writes no debug-ID rows, so it does not change the flag either.

via @jan-auer.

--

View Session [Sentry]

A new bundle gets the flag from whether its archive has files with debug IDs, which is also when its DebugIdArtifactBundle rows are written. A re-upload of an existing bundle writes no debug-ID rows, so it leaves the flag as it is.

Co-Authored-By: Jan Michael Auer <1433023+jan-auer@users.noreply.github.com>
@sentry-junior
sentry-junior Bot requested a review from a team as a code owner November 9, 2026 14:00
@sentry-junior
sentry-junior Bot added this pull request to stack #130423 November 9, 2026 14:01
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 9, 2026
…tifact-bundle-write-has-debug-ids

Co-Authored-By: Jan Michael Auer <1433023+jan-auer@users.noreply.github.com>
@mchen-sentry mchen-sentry added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Nov 9, 2026

This branch was successfully deployed

1 active deployment
Preview — f35549cc Deployed Nov 9, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants