Skip to content

feat: add SOURCEMAP_BROKEN warning for transform hook#9498

Closed
sapphi-red wants to merge 1 commit into
mainfrom
05-21-feat_add_sourcemap_broken_warning_for_transform_hook
Closed

feat: add SOURCEMAP_BROKEN warning for transform hook#9498
sapphi-red wants to merge 1 commit into
mainfrom
05-21-feat_add_sourcemap_broken_warning_for_transform_hook

Conversation

@sapphi-red

@sapphi-red sapphi-red commented May 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Add SOURCEMAP_BROKEN warning and emit it for transform hook calls.

refs #6780

@sapphi-red Graphite App

sapphi-red commented May 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Member Author

How to use the Graphite Merge Queue

Add the label graphite: merge-when-ready to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

This was referenced May 21, 2026
Closed
Closed
Merged
@sapphi-red sapphi-red force-pushed the 05-21-feat_add_sourcemap_broken_warning_for_transform_hook branch from 723bd0c to 50986d9 Compare May 22, 2026 02:04
@sapphi-red sapphi-red force-pushed the 05-20-refactor_distinguish_map_null_vs_map_undefined_in_transform_hook_output branch from aa08857 to 5e26476 Compare May 22, 2026 02:04
@sapphi-red sapphi-red marked this pull request as ready for review May 22, 2026 02:09
@sapphi-red sapphi-red marked this pull request as draft May 22, 2026 02:10
@codspeed-hq

codspeed-hq Bot commented May 22, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 4 untouched benchmarks
⏩ 10 skipped benchmarks1

Comparing 05-21-feat_add_sourcemap_broken_warning_for_transform_hook (50986d9) with 05-20-refactor_distinguish_map_null_vs_map_undefined_in_transform_hook_output (5e26476)

Open in CodSpeed

Footnotes

  1. 10 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

@graphite-app graphite-app Bot changed the base branch from 05-20-refactor_distinguish_map_null_vs_map_undefined_in_transform_hook_output to graphite-base/9498 May 22, 2026 02:22
@graphite-app graphite-app Bot force-pushed the graphite-base/9498 branch from 5e26476 to e412df1 Compare May 22, 2026 02:25
@graphite-app graphite-app Bot force-pushed the 05-21-feat_add_sourcemap_broken_warning_for_transform_hook branch from 50986d9 to 83bfcb6 Compare May 22, 2026 02:25
@graphite-app graphite-app Bot changed the base branch from graphite-base/9498 to main May 22, 2026 02:26
@graphite-app graphite-app Bot force-pushed the 05-21-feat_add_sourcemap_broken_warning_for_transform_hook branch from 83bfcb6 to 7c3b79d Compare May 22, 2026 02:26
@netlify

netlify Bot commented May 22, 2026 •
edited
Loading

Copy link
Copy Markdown

✅ Deploy Preview for rolldown-rs ready!

Name Link
🔨 Latest commit 9a18438
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/6a191035a36e72000892b5ba
😎 Deploy Preview https://deploy-preview-9498--rolldown-rs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@sapphi-red sapphi-red force-pushed the 05-21-feat_add_sourcemap_broken_warning_for_transform_hook branch from 7c3b79d to cb2207e Compare May 25, 2026 13:01
@sapphi-red sapphi-red force-pushed the 05-21-feat_add_sourcemap_broken_warning_for_transform_hook branch from cb2207e to 9a18438 Compare May 29, 2026 04:04
@sapphi-red

sapphi-red commented May 29, 2026

Copy link
Copy Markdown
Member Author

=> #9600

@sapphi-red sapphi-red closed this May 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@sapphi-red sapphi-red

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@sapphi-red