Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add support for sourcemap debugId property #151

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

timfish
Copy link

@timfish timfish commented Jan 3, 2025

Debug IDs are detailed in the TC39 proposal.

Debug IDs are unique IDs generated from a deterministic hash of the source code which are then injected/included in both source and sourcemap. They can be used to easily identify the correct sourcemaps in production.

Although the proposal is only at stage 2, the debug ID in source and sourcemap is well defined and it's only the JavaScript API which is likely to change before stage 3. Sentry already process hundreds of millions of source files per month containing debug ids!

I've recently added support for injecting debug IDs to webpack, Rollup, and Rolldown and wanted to get them working in Rspack too with the same API as webpack.

@CLAassistant
Copy link

CLAassistant commented Jan 3, 2025

CLA assistant check
All committers have signed the CLA.

@timfish timfish changed the title feat: Add support for sourcemap debugid property feat: Add support for sourcemap debugId property Jan 3, 2025
Copy link

codspeed-hq bot commented Jan 6, 2025

CodSpeed Performance Report

Merging #151 will not alter performance

Comparing timfish:feat/debug-id-support (ee21a73) with main (10b6a4c)

Summary

✅ 7 untouched benchmarks

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