Skip to content

fix(nuxt): Delete Sentry-enabled source maps from Nitro output directories - #25450

Merged
s1gr1d merged 1 commit into
developfrom
sig/nuxt-source-maps-nitro
Oct 20, 2026
Merged

s1gr1d merged 1 commit into
developfrom
sig/nuxt-source-maps-nitro

Conversation

@s1gr1d

@s1gr1d s1gr1d commented Oct 16, 2026

Copy link
Copy Markdown
Member

When Sentry enables 'hidden' source maps, it deleted them with folder-name globs like .*/**/public/**/*.map. As discussed in #25362, a static glob cannot know where a preset writes its output.

  • Reads nitro.options.output.publicDir and serverDir instead (available since nitropack 2.6.1, Nuxt 3.7.0).
  • Fixes client maps leaking in dist on Netlify and Cloudflare Pages, and keeps the server maps in dist/_worker.js.
  • Keeps server maps enabled with nitro.rollupConfig.output.sourcemap.

Tested with unit tests on real files, and with new netlify and cloudflare-pages E2E variants.

@s1gr1d
s1gr1d requested a review from a team as a code owner October 16, 2026 11:48
@s1gr1d
s1gr1d requested review from chargome and nicohrubec and removed request for a team October 16, 2026 11:48
@s1gr1d

s1gr1d commented Oct 16, 2026

Copy link
Copy Markdown
Member Author

bugbot run

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit e945b1f. Configure here.

@s1gr1d
s1gr1d merged commit 42f1ac1 into develop Oct 20, 2026
64 checks passed
@s1gr1d
s1gr1d deleted the sig/nuxt-source-maps-nitro branch October 20, 2026 15:40
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.

2 participants