Skip to content

fix(bundledDev): serve sourcemap for lazy chunks in bundledDev mode - #23480

Closed
btea wants to merge 1 commit into
vitejs:mainfrom
btea:fix/bundled-dev-lazy-chunk-sourcemap
Closed

btea wants to merge 1 commit into
vitejs:mainfrom
btea:fix/bundled-dev-lazy-chunk-sourcemap

Conversation

@btea

@btea btea commented Sep 13, 2026 •

Copy link
Copy Markdown
Contributor

In bundledDev mode, modules loaded via import.meta.glob or dynamic import will cause sourcemaps to become invalid.

https://github.com/btea/rolldown-panic/tree/fix/import-bundledDev
This is the repository for reproducing the issue. Simply install the dependencies, then run pnpm dev. You should then see the information shown in the image below when you open your browser's developer tools.

image

@btea
btea requested review from h-a-n-a and shulaoda September 15, 2026 10:35
@h-a-n-a

h-a-n-a commented Sep 15, 2026

Copy link
Copy Markdown
Member

I think it's duplicated with #23026

@btea

btea commented Sep 15, 2026 •

Copy link
Copy Markdown
Contributor Author

Oh, it does seem so. I hadn't noticed that before. 👍

It seems there's a conflict in the code over there. Should we handle it directly on the page or leave a message asking him to fix it.

@h-a-n-a

h-a-n-a commented Sep 17, 2026

Copy link
Copy Markdown
Member

Oh, it does seem so. I hadn't noticed that before. 👍

It seems there's a conflict in the code over there. Should we handle it directly on the page or leave a message asking him to fix it.

If they don't mind, we might just rebase that PR ourselves and get it merged?

@btea

btea commented Sep 18, 2026

Copy link
Copy Markdown
Contributor Author

Duplicate of #23026

@btea btea marked this as a duplicate of #23026 Sep 18, 2026
@btea btea closed this Sep 18, 2026
@btea
btea deleted the fix/bundled-dev-lazy-chunk-sourcemap branch September 18, 2026 13:10
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