Skip to content

Comments

fix(strictExecutionOrder): ensure initialization chain of entry exports#4933

Merged
hyf0 merged 1 commit intomainfrom
06-12-fix_strictexecutionorder_ensure_initialization_chain_of_entry_exports
Jun 12, 2025
Merged

fix(strictExecutionOrder): ensure initialization chain of entry exports#4933
hyf0 merged 1 commit intomainfrom
06-12-fix_strictexecutionorder_ensure_initialization_chain_of_entry_exports

Conversation

@hyf0
Copy link
Member

@hyf0 hyf0 commented Jun 12, 2025 •
edited
Loading

Fixes #4920.

@netlify
Copy link

netlify bot commented Jun 12, 2025

✅ Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit 8b09c32
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/684ad77f76ed770008294f57

@hyf0 Graphite App
Copy link
Member Author

hyf0 commented Jun 12, 2025

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

@hyf0 hyf0 marked this pull request as ready for review June 12, 2025 13:38
@github-actions
Copy link
Contributor

github-actions bot commented Jun 12, 2025

Benchmarks Rust

  • target: main(3ac8c50)
  • pr: 06-12-fix_strictexecutionorder_ensure_initialization_chain_of_entry_exports(8b09c32)
group                                                        pr                                     target
-----                                                        --                                     ------
bundle/bundle@multi-duplicated-top-level-symbol              1.01     77.4±3.17ms        ? ?/sec    1.00     76.8±1.77ms        ? ?/sec
bundle/bundle@multi-duplicated-top-level-symbol-sourcemap    1.00     87.3±2.08ms        ? ?/sec    1.01     88.5±2.28ms        ? ?/sec
bundle/bundle@rome_ts                                        1.01    115.5±2.75ms        ? ?/sec    1.00    114.2±2.69ms        ? ?/sec
bundle/bundle@rome_ts-sourcemap                              1.00    128.4±1.82ms        ? ?/sec    1.01    129.2±1.90ms        ? ?/sec
bundle/bundle@threejs                                        1.01     42.4±2.43ms        ? ?/sec    1.00     41.8±2.36ms        ? ?/sec
bundle/bundle@threejs-sourcemap                              1.00     47.5±1.42ms        ? ?/sec    1.00     47.5±0.76ms        ? ?/sec
bundle/bundle@threejs10x                                     1.00    440.3±6.23ms        ? ?/sec    1.00    440.0±6.06ms        ? ?/sec
bundle/bundle@threejs10x-sourcemap                           1.01    513.1±6.26ms        ? ?/sec    1.00    510.0±5.16ms        ? ?/sec
scan/scan@rome_ts                                            1.03     91.3±2.60ms        ? ?/sec    1.00     88.5±1.23ms        ? ?/sec
scan/scan@threejs                                            1.03     32.3±1.99ms        ? ?/sec    1.00     31.4±1.85ms        ? ?/sec
scan/scan@threejs10x                                         1.00    325.6±4.39ms        ? ?/sec    1.00    325.5±5.23ms        ? ?/sec

@pkg-pr-new
Copy link

pkg-pr-new bot commented Jun 12, 2025

Open in StackBlitz

@rolldown/browser

npm i https://pkg.pr.new/@rolldown/browser@4933

@rolldown/debug

npm i https://pkg.pr.new/@rolldown/debug@4933

@rolldown/pluginutils

npm i https://pkg.pr.new/@rolldown/pluginutils@4933

rolldown

npm i https://pkg.pr.new/rolldown@4933

@rolldown/binding-darwin-arm64

npm i https://pkg.pr.new/@rolldown/binding-darwin-arm64@4933

@rolldown/binding-darwin-x64

npm i https://pkg.pr.new/@rolldown/binding-darwin-x64@4933

@rolldown/binding-freebsd-x64

npm i https://pkg.pr.new/@rolldown/binding-freebsd-x64@4933

@rolldown/binding-linux-arm-gnueabihf

npm i https://pkg.pr.new/@rolldown/binding-linux-arm-gnueabihf@4933

@rolldown/binding-linux-arm64-gnu

npm i https://pkg.pr.new/@rolldown/binding-linux-arm64-gnu@4933

@rolldown/binding-linux-arm64-musl

npm i https://pkg.pr.new/@rolldown/binding-linux-arm64-musl@4933

@rolldown/binding-linux-x64-gnu

npm i https://pkg.pr.new/@rolldown/binding-linux-x64-gnu@4933

@rolldown/binding-linux-x64-musl

npm i https://pkg.pr.new/@rolldown/binding-linux-x64-musl@4933

@rolldown/binding-wasm32-wasi

npm i https://pkg.pr.new/@rolldown/binding-wasm32-wasi@4933

@rolldown/binding-win32-arm64-msvc

npm i https://pkg.pr.new/@rolldown/binding-win32-arm64-msvc@4933

@rolldown/binding-win32-ia32-msvc

npm i https://pkg.pr.new/@rolldown/binding-win32-ia32-msvc@4933

@rolldown/binding-win32-x64-msvc

npm i https://pkg.pr.new/@rolldown/binding-win32-x64-msvc@4933

commit: 8b09c32

@hyf0 hyf0 mentioned this pull request Jun 12, 2025
Closed
6 tasks
@hyf0 hyf0 added this pull request to the merge queue Jun 12, 2025
Merged via the queue into main with commit 0e8686c Jun 12, 2025
47 checks passed
@hyf0 hyf0 deleted the 06-12-fix_strictexecutionorder_ensure_initialization_chain_of_entry_exports branch June 12, 2025 14:27
github-actions bot pushed a commit that referenced this pull request Jun 15, 2025
@shulaoda @github-actions
release: v1.0.0-beta.16
5a8075e 
## [1.0.0-beta.16] - 2025-06-15

### 🚀 Features

- rolldown: feat(rolldown): oxc v0.73.0 by @Boshen in [#4948](#4948)
- types: feat(types): generate HMR runtime types by @sapphi-red in [#4928](#4928)
- rolldown_plugin_build_import_analysis: feat(rolldown_plugin_build_import_analysis): align with rolldown-vite by @shulaoda in [#4931](#4931)
- feat: allow to configure`experimental#attachDebugInfo: 'none'` to disable generating `#region` comments by @TheAlexLichter in [#4918](#4918)

### 🐛 Bug Fixes

- fix: deduplicate entry point by @IWANABETHATGUY in [#4956](#4956)
- fix: diagnostic message on minified code is too long to be readable by @IWANABETHATGUY in [#4950](#4950)
- fix: deduplicated entry points when have same `id`, `name`, and `filename` by @IWANABETHATGUY in [#4947](#4947)
- fix: unbundle does not work with outExtensions by @IWANABETHATGUY in [#4945](#4945)
- types: fix(types): remove `undefined` from return type of `generateHmrPatch` and `hmrInvalidate` by @sapphi-red in [#4943](#4943)
- fix: handle error in `hmrInvalidate` by @sapphi-red in [#4942](#4942)
- rolldown_plugin_build_import_analysis: fix(rolldown_plugin_build_import_analysis): preload helper is imported even if it’s not needed by @shulaoda in [#4936](#4936)
- hmr: fix(hmr): avoid panic after syntax error by @sapphi-red in [#4898](#4898)
- strictExecutionOrder: fix(strictExecutionOrder): ensure initialization chain of entry exports by @hyf0 in [#4933](#4933)
- fix: should not panic when meet a syntax error in watch mode. by @IWANABETHATGUY in [#4926](#4926)
- fix: regression, id filter normalization for windows is not working by @IWANABETHATGUY in [#4922](#4922)
- rolldown_plugin_json: fix(rolldown_plugin_json): incorrect default JSON export by @shulaoda in [#4924](#4924)

### 🚜 Refactor

- refactor: avoid unnecessary collect by @IWANABETHATGUY in [#4949](#4949)
- refactor: get normalized options via binding by @IWANABETHATGUY in [#4938](#4938)
- refactor: include DevRuntime base class even if custom HMR implementation is passed by @sapphi-red in [#4929](#4929)

### 📚 Documentation

- docs: clarify minifier state by @TheAlexLichter in [#4953](#4953)
- docs: drafting code splitting by @hyf0 in [#4951](#4951)
- docs: add link to repl by @sxzz in [#4917](#4917)
- pluginutils: docs(pluginutils): add example to simple filter functions by @sapphi-red in [#4915](#4915)

### 🧪 Testing

- test: add testcase for normalized output options by @IWANABETHATGUY in [#4939](#4939)
- hmr: test(hmr): enable `expectExecuted` for `generatePatchError` test by @sapphi-red in [#4937](#4937)
- test: evaluate hmr tests in node by @sapphi-red in [#4930](#4930)
- test: add "non used export" hmr test by @sapphi-red in [#4900](#4900)
- test: enable expectExecuted in HMR tests by @sapphi-red in [#4927](#4927)

### ⚙️ Miscellaneous Tasks

- deps: chore(deps): lock file maintenance npm packages by @renovate[bot] in [#4866](#4866)
- infra: chore(infra): remove comments related to `cspell` by @shulaoda in [#4940](#4940)
- ci: remove cargo bench test by @IWANABETHATGUY in [#4925](#4925)
@Boshen Boshen mentioned this pull request Jun 15, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shulaoda shulaoda shulaoda approved these changes

Assignees

No one assigned

Labels

trigger: preview

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug]: strictExecutionOrder: true makes re-exported values undefined

2 participants

@hyf0 @shulaoda