Skip to content

feat(bundler-plugins): Enable hidden source maps and remove them after upload in the webpack plugin - #25647

Open
timfish wants to merge 1 commit into
feat/bundler-plugins-delete-sourcemaps-vitefrom
feat/bundler-plugins-delete-sourcemaps-webpack
Open

timfish wants to merge 1 commit into
feat/bundler-plugins-delete-sourcemaps-vitefrom
feat/bundler-plugins-delete-sourcemaps-webpack

Conversation

@timfish

@timfish timfish commented Oct 22, 2026

Copy link
Copy Markdown
Collaborator

Part of the stack for #25609. When devtool is unset and an upload can run, the plugin sets hidden-source-map; a visible devtool with deleteSourcemapsAfterUpload: true gets the hidden- prefix. The maps are staged at the end of processAssets, after the real content hashes are in place, and deleted from the compilation together with any compressed variants, so webpack never emits them.

Development builds and a devtool: false with a manual SourceMapDevToolPlugin stay untouched. Rspack uses the same hooks but is not covered by tests, because @rspack/core is not installed in the repo.

@timfish
timfish added this pull request to stack #25649 October 22, 2026 10:47
@github-actions

github-actions Bot commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

size-limit report 📦

Compared: caef574 (PR test merge using develop at 16f4f87)

Against: e17a480 (develop) · base run #63967

⚠️ Baseline mismatch: 6 develop commit(s) in the PR build are missing from the baseline. Their size changes also affect this comparison. Compare commits. Re-run once develop's baseline build for 16f4f87 finishes.

Path Size % Change Change
@sentry/browser 30.34 kB - -
@sentry/browser - with treeshaking flags 28.46 kB - -
@sentry/browser - with treeshaking flags tracing without tracing 28.35 kB - -
@sentry/browser (incl. Tracing) 52.61 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 52.62 kB - -
@sentry/browser (incl. Tracing, Profiling) 55.54 kB - -
@sentry/browser (incl. Tracing, Replay) 92.81 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 81.43 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 97.54 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 110.52 kB - -
@sentry/browser (incl. Feedback) 47.9 kB - -
@sentry/browser (incl. sendFeedback) 35.41 kB - -
@sentry/browser (incl. FeedbackAsync) 40.52 kB - -
@sentry/browser (incl. Metrics) 31.35 kB - -
@sentry/browser (incl. Logs) 31.63 kB - -
@sentry/browser (incl. Metrics & Logs) 32.31 kB - -
@sentry/react 32.16 kB - -
@sentry/react (incl. Tracing) 54.89 kB - -
@sentry/vue 38.61 kB - -
@sentry/vue (incl. Tracing) 55.5 kB - -
@sentry/svelte 30.37 kB - -
@sentry/remix (Remix 3 client bundle) 57.65 kB - -
CDN Bundle 32.07 kB - -
CDN Bundle (incl. Tracing) 53.1 kB - -
CDN Bundle (incl. Logs, Metrics) 34.27 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 55.05 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 75.69 kB - -
CDN Bundle (incl. Tracing, Replay) 91.23 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 93.17 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 97.36 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 99.34 kB - -
CDN Bundle - uncompressed 94.31 kB - -
CDN Bundle (incl. Tracing) - uncompressed 157.3 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 100.86 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 163.27 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 232.36 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 278.93 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 284.88 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 292.63 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 298.58 kB - -
@sentry/nextjs (client) 57.75 kB - -
@sentry/sveltekit (client) 53.06 kB - -
@sentry/core/server 44.29 kB - -
@sentry/core/browser 13.95 kB - -
@sentry/node 155 kB +0.01% +10 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection) 81.62 kB - -
@sentry/node - without tracing 95.72 kB +0.01% +8 B 🔺
@sentry/node - without channel injection 135.61 kB +0.01% +8 B 🔺
@sentry/aws-serverless 103.89 kB +0.01% +9 B 🔺
@sentry/cloudflare (withSentry) - minified 197.89 kB - -
@sentry/cloudflare (withSentry) 491.82 kB - -
@sentry/nextjs/cloudflare (withSentry) - minified 218.47 kB - -

View base workflow run

@timfish
timfish force-pushed the feat/bundler-plugins-delete-sourcemaps-webpack branch from c0e6cee to 4709561 Compare October 22, 2026 10:57
@github-actions

github-actions Bot commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Optional E2E tests

Some optional checks were not successful. They do not block the merge. 3 not successful, 64 successful. See the run.

Check Status
❌ CI: Optional E2E Tests / E2E node-anthropic-send-to-sentry (latest) Test (optional) Failing after 3m Details
❌ CI: Optional E2E Tests / E2E supabase-nextjs Test (optional) Failing after 27s Details
❌ CI: Optional E2E Tests / E2E node-express-send-to-sentry-static Test (optional) Failing after 3m Details
Successful checks
Check Status
✅ CI: Optional E2E Tests / E2E cloudflare-flue-send-to-sentry Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E nextjs-15 (turbo) Test (optional) Successful in 2m Details
✅ CI: Optional E2E Tests / E2E angular (canary) Test (optional) Successful in 53s Details
✅ CI: Optional E2E Tests / E2E nextjs-15 (latest) Test (optional) Successful in 3m Details
✅ CI: Optional E2E Tests / E2E nextjs-16 (vinext) Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E debug-id-sourcemaps Test (optional) Successful in 29s Details
✅ CI: Optional E2E Tests / E2E nextjs-16 (vinext, bun) Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E nextjs-14 (canary) Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E node-fastify-6 (tracer provider) Test (optional) Successful in 22s Details
✅ CI: Optional E2E Tests / E2E node-express-mcp-v1 Test (optional) Successful in 18s Details
✅ CI: Optional E2E Tests / E2E cloudflare-agent Test (optional) Successful in 32s Details
✅ CI: Optional E2E Tests / E2E cloudflare-think Test (optional) Successful in 39s Details
✅ CI: Optional E2E Tests / E2E gen-ai-libraries (cloudflare) Test (optional) Successful in 25s Details
✅ CI: Optional E2E Tests / E2E nextjs-16-tunnel (cloudflare) Test (optional) Successful in 42s Details
✅ CI: Optional E2E Tests / E2E angular (canary) Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E tanstackstart-react-cloudflare Test (optional) Successful in 24s Details
✅ CI: Optional E2E Tests / E2E nextjs-16 (canary, turbopack) Test (optional) Successful in 4m Details
✅ CI: Optional E2E Tests / E2E gen-ai-libraries Test (optional) Successful in 52s Details
✅ CI: Optional E2E Tests / E2E deno (latest) Test (optional) Successful in 48s Details
✅ CI: Optional E2E Tests / E2E node-eve (latest) Test (optional) Successful in 37s Details
✅ CI: Optional E2E Tests / E2E nextjs-14 (latest) Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E cloudflare-mastra Test (optional) Successful in 28s Details
✅ CI: Optional E2E Tests / E2E cloudflare-think (ai v6) Test (optional) Successful in 37s Details
✅ CI: Optional E2E Tests / E2E cloudflare-pi-durable (latest) Test (optional) Successful in 27s Details
✅ CI: Optional E2E Tests / E2E astro-7 Test (optional) Successful in 59s Details
✅ CI: Optional E2E Tests / E2E nuxt-4 (canary) Test (optional) Successful in 2m Details
✅ CI: Optional E2E Tests / E2E cloudflare-pi-durable Test (optional) Successful in 23s Details
✅ CI: Optional E2E Tests / E2E cloudflare-workers-send-to-sentry Test (optional) Successful in 2m Details
✅ CI: Optional E2E Tests / E2E nextjs-16 (canary, webpack) Test (optional) Successful in 4m Details
✅ CI: Optional E2E Tests / E2E node-anthropic-send-to-sentry Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E nextjs-orpc Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E node-express-mcp-v2 Test (optional) Successful in 15s Details
✅ CI: Optional E2E Tests / E2E nuxt-4-static (canary) Test (optional) Successful in 2m Details
✅ CI: Optional E2E Tests / E2E node-express-send-to-sentry Test (optional) Successful in 41s Details
✅ CI: Optional E2E Tests / E2E node-flue-send-to-sentry Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E node-flue Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E node-pi-durable Test (optional) Successful in 24s Details
✅ CI: Optional E2E Tests / E2E node-eve (orchestrion) Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E nextjs-16-standalone (canary, turbopack) Test (optional) Successful in 24s Details
✅ CI: Optional E2E Tests / E2E nextjs-16 (cloudflare, latest) Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E node-fastify-6 Test (optional) Successful in 47s Details
✅ CI: Optional E2E Tests / E2E nuxt-5-sourcemaps (static / nuxt generate) Test (optional) Successful in 19s Details
✅ CI: Optional E2E Tests / E2E nextjs-16 (deno) Test (optional) Successful in 2m Details
✅ CI: Optional E2E Tests / E2E nuxt-5-sourcemaps (client source maps kept) Test (optional) Successful in 22s Details
✅ CI: Optional E2E Tests / E2E node-openai-decisions-send-to-sentry Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E nuxt-5-sourcemaps Test (optional) Successful in 22s Details
✅ CI: Optional E2E Tests / E2E node-flue (latest) Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E react-send-to-sentry Test (optional) Successful in 2m Details
✅ CI: Optional E2E Tests / E2E react-router-7-framework-instrumentation Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E node-openai-decisions-send-to-sentry (latest) Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E node-mastra (latest) Test (optional) Successful in 50s Details
✅ CI: Optional E2E Tests / E2E nuxt-3 (canary) Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E node-openai-send-to-sentry (latest) Test (optional) Successful in 2m Details
✅ CI: Optional E2E Tests / E2E node-openai-send-to-sentry Test (optional) Successful in 2m Details
✅ CI: Optional E2E Tests / E2E nuxt-5 Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E node-mastra Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E node-flue-send-to-sentry (latest) Test (optional) Successful in 1m Details
✅ CI: Optional E2E Tests / E2E node-eve Test (optional) Successful in 53s Details
✅ CI: Optional E2E Tests / E2E solidstart-2 Test (optional) Successful in 36s Details
✅ CI: Optional E2E Tests / E2E node-pi-durable (latest) Test (optional) Successful in 23s Details
✅ CI: Optional E2E Tests / E2E astro-7-static Test (optional) Successful in 48s Details
✅ CI: Optional E2E Tests / E2E vue-3 (no Options API) Test (optional) Successful in 52s Details
✅ CI: Optional E2E Tests / E2E tanstackstart-react-sourcemaps Test (optional) Successful in 24s Details
✅ CI: Optional E2E Tests / E2E vue-3 (canary) Test (optional) Successful in 53s Details

@timfish
timfish force-pushed the feat/bundler-plugins-delete-sourcemaps-webpack branch from 4709561 to 63db386 Compare October 22, 2026 11:49
@timfish
timfish force-pushed the feat/bundler-plugins-delete-sourcemaps-webpack branch 2 times, most recently from 247acf4 to e7fe1ce Compare October 27, 2026 16:32
@timfish
timfish marked this pull request as ready for review October 27, 2026 18:02

@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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit e7fe1ce. Configure here.

Comment thread packages/bundler-plugins/src/webpack/index.ts
@timfish
timfish requested review from a team as code owners October 28, 2026 12:15
@timfish
timfish requested review from isaacs, logaretm, msonnb, mydea and nicohrubec and removed request for a team October 28, 2026 12:15
@timfish
timfish force-pushed the feat/bundler-plugins-delete-sourcemaps-webpack branch from e7fe1ce to 24328a1 Compare October 28, 2026 12:16
@timfish
timfish force-pushed the feat/bundler-plugins-delete-sourcemaps-webpack branch from 24328a1 to 4e1417f Compare October 28, 2026 12:31
@timfish
timfish force-pushed the feat/bundler-plugins-delete-sourcemaps-webpack branch from 4e1417f to 4163a84 Compare October 28, 2026 12:54
@timfish
timfish force-pushed the feat/bundler-plugins-delete-sourcemaps-webpack branch from 4163a84 to 8613941 Compare October 28, 2026 13:04
…r upload in the webpack plugin

When `devtool` is unset and an upload can run, the plugin sets
`hidden-source-map`, copies each JS asset and its map into the upload
staging folder at the end of `processAssets` and deletes the map
assets, so webpack never emits them. A `devtool` with a manual
`SourceMapDevToolPlugin` and development builds stay untouched.
@timfish
timfish force-pushed the feat/bundler-plugins-delete-sourcemaps-webpack branch from 8613941 to 5ad49fc Compare October 28, 2026 14:49

This branch has not been deployed

No deployments
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