Skip to content

fix: use sentinel id for browser: false ignored modules#9192

Merged
shulaoda merged 3 commits intomainfrom
04-22-fix_preserve_ignored_flag_through_plugin_this.resolve_round-trip
Apr 26, 2026
Merged

fix: use sentinel id for browser: false ignored modules#9192
shulaoda merged 3 commits intomainfrom
04-22-fix_preserve_ignored_flag_through_plugin_this.resolve_round-trip

Conversation

@shulaoda
Copy link
Copy Markdown
Member

closes #9107

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 22, 2026

Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit b6286aa
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/69ed758e9b9eea0008d64adb

Comment thread crates/rolldown_plugin/src/utils/resolve_id_with_plugins.rs Outdated
@shulaoda shulaoda force-pushed the 04-22-fix_preserve_ignored_flag_through_plugin_this.resolve_round-trip branch from 7941a4f to d25f33f Compare April 24, 2026 03:23
@shulaoda shulaoda force-pushed the 04-22-fix_preserve_ignored_flag_through_plugin_this.resolve_round-trip branch from d25f33f to d382e00 Compare April 24, 2026 03:31
@shulaoda shulaoda marked this pull request as ready for review April 24, 2026 03:32
@shulaoda shulaoda requested review from hyf0 and sapphi-red April 24, 2026 03:32
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Apr 24, 2026

Merging this PR will not alter performance

✅ 4 untouched benchmarks
⏩ 10 skipped benchmarks1


Comparing 04-22-fix_preserve_ignored_flag_through_plugin_this.resolve_round-trip (b6286aa) with main (31d0403)

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.

Copy link
Copy Markdown
Member

@hyf0 hyf0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why there's no snapshot change?

I assume the comment should print the id directly

@shulaoda
Copy link
Copy Markdown
Member Author

shulaoda commented Apr 24, 2026

Why there's no snapshot change?

I assume the comment should print the id directly

I think it may because we construct something like Path::new("\0rolldown/empty?C:\\a\\b\\c.js") and then use file_stem() to get the file name c, which keeps the behavior somewhat consistent with the original. Do you think we should adjust this?

@shulaoda shulaoda merged commit 1d79b02 into main Apr 26, 2026
33 checks passed
@shulaoda shulaoda deleted the 04-22-fix_preserve_ignored_flag_through_plugin_this.resolve_round-trip branch April 26, 2026 15:13
This was referenced Apr 29, 2026
shulaoda added a commit that referenced this pull request Apr 29, 2026
## [1.0.0-rc.18] - 2026-04-29

### 💥 BREAKING CHANGES

- optimization: default unspecified inlineConst.mode to smart (#9248) by @IWANABETHATGUY

### 🐛 Bug Fixes

- rolldown_plugin_vite_import_glob: return error instead of panicking when virtual module uses a relative glob (#9241) by @shulaoda
- binding: treat empty inlineConst object as omitted (#9247) by @IWANABETHATGUY
- rolldown: keep enum declaration for optional-chain access (#9229) by @Dunqing
- link_stage: restore inline let-else in exports-kind filter (#9237) by @IWANABETHATGUY
- dev/lazy: avoid module reinitialization in lazy compilation patches (#9179) by @h-a-n-a
- dev: visit identifier references for runtime rewrites in HMR finalizer (#9191) by @h-a-n-a
- chunk-optimizer: pick dominator for runtime placement to avoid cycles (#9164) by @IWANABETHATGUY
- make `this.emitFile` chunk path synchronous to avoid deadlock (#9031) by @lazarv
- use sentinel id for `browser: false` ignored modules (#9192) by @shulaoda
- prevent chunk optimizer from creating import cycles (#9228) by @IWANABETHATGUY

### 🚜 Refactor

- replace tokio::sync::Mutex with std::sync::Mutex for non-IO data (#9176) by @shulaoda
- rolldown_plugin_vite_import_glob: do not rewrite import path for absolute base (#9195) by @shulaoda
- runtime_helper: wrap DependedRuntimeHelperMap in a struct (#9215) by @IWANABETHATGUY
- drop redundant clear() in determine_safely_merge_cjs_ns (#9206) by @IWANABETHATGUY
- clean up generate_lazy_export (#9208) by @IWANABETHATGUY
- bitset: return bool from set_bit to fuse guard-and-set (#9207) by @IWANABETHATGUY
- link_stage: simplify exports-kind filter and clarify safety comments (#9205) by @IWANABETHATGUY

### 📚 Documentation

- determine_module_exports_kind (#9252) by @IWANABETHATGUY
- fix dead link to esbuild ESM/CJS interop tests (#9230) by @Copilot
- remove CSS bundling references (#9234) by @shulaoda
- correct IncrementalFullBuild row in BundleMode table (#9214) by @IWANABETHATGUY
- design: add bundler data lifecycle design doc (#9212) by @hyf0
- remove minifier alpha status notices (#9202) by @sapphi-red

### ⚙️ Miscellaneous Tasks

- upgrade oxc to 0.128.0 (#9260) by @shulaoda
- deps: bump rolldown-ariadne to 0.6.0 (#9254) by @IWANABETHATGUY
- deps: update github actions (#9259) by @renovate[bot]
- deps: update github actions (#9258) by @renovate[bot]
- remove renovate overrides (#9257) by @Boshen
- use ubuntu-latest for security workflow (#9256) by @Boshen
- notify Discord around release publish (#9251) by @Boshen
- add release environment to npm publish workflow (#9250) by @Boshen
- justfile: drop the `--` separator before forwarded args in `vp run` (#9246) by @shulaoda
- deps: update test262 submodule for tests (#9243) by @sapphi-red
- add more tracing instrumentations (#9220) by @sapphi-red
- rolldown_plugin_vite_import_glob: remove outdated sourcemap doc comment (#9213) by @shulaoda
- update security workflow (#9201) by @Boshen

### ❤️ New Contributors

* @lazarv made their first contribution in [#9031](#9031)

Co-authored-by: shulaoda <165626830+shulaoda@users.noreply.github.com>
@rolldown-guard rolldown-guard Bot mentioned this pull request Apr 29, 2026
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.

[Bug]: Returning this.resolve() from resolveId misresolves browser: false imports

3 participants