Skip to content

feat: Add support for sourcemap debugId property#151

Merged
SyMind merged 2 commits into
rstackjs:mainfrom
timfish:feat/debug-id-support
Jan 8, 2025
Merged

feat: Add support for sourcemap debugId property#151
SyMind merged 2 commits into
rstackjs:mainfrom
timfish:feat/debug-id-support

Conversation

@timfish
Copy link
Copy Markdown
Contributor

@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
Copy Markdown

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
@codspeed-hq
Copy link
Copy Markdown

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

@SyMind SyMind merged commit a847aae into rstackjs:main Jan 8, 2025
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.

3 participants