Skip to content

Comments

fix: should include modules contain exports * from '...' forcefully if they reexport any wrapped module#5143

Merged
hyf0 merged 1 commit intomainfrom
07-02-fix_should_include_modules_contain_exports_from_._forcefully_if_they_reexport_any_wrapped_module
Jul 2, 2025
Merged

fix: should include modules contain exports * from '...' forcefully if they reexport any wrapped module#5143
hyf0 merged 1 commit intomainfrom
07-02-fix_should_include_modules_contain_exports_from_._forcefully_if_they_reexport_any_wrapped_module

Conversation

@hyf0
Copy link
Member

@hyf0 hyf0 commented Jul 2, 2025 •
edited
Loading

@hyf0 hyf0 marked this pull request as ready for review July 2, 2025 08:43
@hyf0 Graphite App
Copy link
Member Author

hyf0 commented Jul 2, 2025

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

@netlify
Copy link

netlify bot commented Jul 2, 2025 •
edited
Loading

✅ Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit 85ba9ec
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/6864f1ec76518900083b885a

@hyf0 hyf0 force-pushed the 07-02-fix_should_include_modules_contain_exports_from_._forcefully_if_they_reexport_any_wrapped_module branch from faab330 to 85ba9ec Compare July 2, 2025 08:46
@hyf0 hyf0 added this pull request to the merge queue Jul 2, 2025
Merged via the queue into main with commit c673996 Jul 2, 2025
23 checks passed
@hyf0 hyf0 deleted the 07-02-fix_should_include_modules_contain_exports_from_._forcefully_if_they_reexport_any_wrapped_module branch July 2, 2025 08:59
@github-actions
Copy link
Contributor

github-actions bot commented Jul 2, 2025

Benchmarks Rust

  • target: main(8df7c03)
  • pr: 07-02-fix_should_include_modules_contain_exports_from_._forcefully_if_they_reexport_any_wrapped_module(85ba9ec)
group                                                        pr                                     target
-----                                                        --                                     ------
bundle/bundle@multi-duplicated-top-level-symbol              1.00     77.9±2.26ms        ? ?/sec    1.00     78.0±2.05ms        ? ?/sec
bundle/bundle@multi-duplicated-top-level-symbol-sourcemap    1.00     87.1±1.96ms        ? ?/sec    1.02     88.6±1.72ms        ? ?/sec
bundle/bundle@rome_ts                                        1.01    115.5±4.05ms        ? ?/sec    1.00    114.7±1.55ms        ? ?/sec
bundle/bundle@rome_ts-sourcemap                              1.01    135.3±1.86ms        ? ?/sec    1.00    134.5±1.73ms        ? ?/sec
bundle/bundle@threejs                                        1.00     41.0±1.18ms        ? ?/sec    1.00     40.9±1.08ms        ? ?/sec
bundle/bundle@threejs-sourcemap                              1.00     50.6±0.99ms        ? ?/sec    1.00     50.8±0.74ms        ? ?/sec
bundle/bundle@threejs10x                                     1.00    438.5±5.74ms        ? ?/sec    1.00    438.4±3.65ms        ? ?/sec
bundle/bundle@threejs10x-sourcemap                           1.00    509.5±2.92ms        ? ?/sec    1.00    509.2±6.34ms        ? ?/sec
scan/scan@rome_ts                                            1.00     88.3±1.77ms        ? ?/sec    1.02     90.2±4.65ms        ? ?/sec
scan/scan@threejs                                            1.00     30.9±1.80ms        ? ?/sec    1.00     30.7±1.96ms        ? ?/sec
scan/scan@threejs10x                                         1.00    321.3±4.50ms        ? ?/sec    1.00    321.9±3.82ms        ? ?/sec

@github-actions github-actions bot mentioned this pull request Jul 5, 2025
Merged
hyf0 pushed a commit that referenced this pull request Jul 5, 2025
@github-actions @shulaoda
release: v1.0.0-beta.24 (#5182)
2c9c25c 
## [1.0.0-beta.24] - 2025-07-05

### 🚀 Features

- support `inlineConst` for imported variable (#5165) by @IWANABETHATGUY
- rust: skip transform hook for modules starts with `rolldown:` (#5169)
by @hyf0
- print rolldown version (#5163) by @btea
- add optimization.inlineConst option (#5164) by @IWANABETHATGUY
- hmr: introduce `rolldown_plugin_hmr` to inject runtime code (#5159) by
@hyf0
- rolldown: oxc v0.75.1 (#5157) by @Boshen

### 🐛 Bug Fixes

- generate correct CSS filename in `preserveModules` mode (#5175) by
@shulaoda
- rolldown_plugin_build_import_analysis: avoid `default` keyword error
(#5166) by @shulaoda
- watch: add paths to watcher in batch (#5068) by @branchseer
- debug: should emit chunk data after computing cross chunk links
(#5156) by @hyf0
- simplify commonjs interop `default` prop optimization (#5142) by
@IWANABETHATGUY
- should include modules contain `exports * from '...'` forcefully if
they reexport any wrapped module (#5143) by @hyf0
- use canonical runtime time for `__name` (#5140) by @IWANABETHATGUY
- `options.jsx` is ignored when `options.transform` is set (#5137) by
@shulaoda

### 🚜 Refactor

- polyfill `flatten_iter` for wasm target (#5180) by @IWANABETHATGUY
- rolldown_plugin_dynamic_import_vars: remove unnecessary validity
checks (#5172) by @shulaoda
- hmr: remove hmr-related hacky hard-code (#5168) by @hyf0
- rolldown_plugin_dynamic_import_vars: align with rolldown-vite (#5171)
by @shulaoda
- hmr: use `rolldown:hmr` to load hmr runtime code (#5162) by @hyf0
- rust: remove special fields of `Asset` (#5155) by @hyf0
- rust: make `Asset#originate_from` optional (#5153) by @hyf0
- rust: isolate mutations on assets data (#5152) by @hyf0
- rust: remove unnecessary usage of index vec on assets (#5151) by @hyf0
- merge different member expr in `try_rewrite_member_expr` (#5146) by
@IWANABETHATGUY
- remove unused code (#5144) by @IWANABETHATGUY
- use `self.result.ast_usage` directly (#5135) by @IWANABETHATGUY

### 📚 Documentation

- update syntax lowering transform info (#5145) by @TheAlexLichter

### 🧪 Testing

- rust/hmr: hide hmr runtime code in snapshots (#5160) by @hyf0

### ⚙️ Miscellaneous Tasks

- rolldown_testing: avoid unnecessary snapshots (#5181) by @shulaoda
- fix `flatten_iter` compile error (#5179) by @IWANABETHATGUY
- expose `optimization.inlineConst` option (#5177) by @IWANABETHATGUY
- infra: update `pnpm-lock.yaml` (#5178) by @shulaoda
- rust: remove unnecessary crate `rolldown_loader_utils` (#5173) by
@shulaoda
- print build time use green color (#5158) by @btea
- deps: update dependency rolldown-plugin-dts to v0.13.13 (#5148) by
@renovate[bot]
- deps: update dependency rolldown-plugin-dts to v0.13.13 (#5138) by
@renovate[bot]
- allow only test without generate snapshot (#5134) by @IWANABETHATGUY

Co-authored-by: shulaoda <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@IWANABETHATGUY IWANABETHATGUY IWANABETHATGUY approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Vite + Vue Build Issue - Rolldown build breaks with Vuedraggable imported, works with Rollup

2 participants

@hyf0 @IWANABETHATGUY