Skip to content

Comments

fix: unbundle does not work with outExtensions#4945

Merged
IWANABETHATGUY merged 1 commit intomainfrom
06-13-fix_4944
Jun 13, 2025
Merged

fix: unbundle does not work with outExtensions#4945
IWANABETHATGUY merged 1 commit intomainfrom
06-13-fix_4944

Conversation

@IWANABETHATGUY
Copy link
Member

@IWANABETHATGUY IWANABETHATGUY commented Jun 13, 2025 •
edited
Loading

Closed #4944

@IWANABETHATGUY Graphite App
Copy link
Member Author

IWANABETHATGUY commented Jun 13, 2025

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

@netlify
Copy link

netlify bot commented Jun 13, 2025 •
edited
Loading

✅ Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit 004950e
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/684be134271fa00008167726

@pkg-pr-new
Copy link

pkg-pr-new bot commented Jun 13, 2025

Open in StackBlitz

@rolldown/browser

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

@rolldown/debug

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

@rolldown/pluginutils

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

rolldown

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

@rolldown/binding-darwin-arm64

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

@rolldown/binding-darwin-x64

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

@rolldown/binding-freebsd-x64

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

@rolldown/binding-linux-arm-gnueabihf

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

@rolldown/binding-linux-arm64-gnu

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

@rolldown/binding-linux-arm64-musl

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

@rolldown/binding-linux-x64-gnu

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

@rolldown/binding-linux-x64-musl

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

@rolldown/binding-wasm32-wasi

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

@rolldown/binding-win32-arm64-msvc

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

@rolldown/binding-win32-ia32-msvc

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

@rolldown/binding-win32-x64-msvc

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

commit: ea5c094

@github-actions
Copy link
Contributor

github-actions bot commented Jun 13, 2025 •
edited
Loading

Benchmarks Rust

group                                                        pr                                     target
-----                                                        --                                     ------
bundle/bundle@multi-duplicated-top-level-symbol              1.02     78.9±3.07ms        ? ?/sec    1.00     77.1±3.77ms        ? ?/sec
bundle/bundle@multi-duplicated-top-level-symbol-sourcemap    1.05     89.9±2.89ms        ? ?/sec    1.00     85.5±1.73ms        ? ?/sec
bundle/bundle@rome_ts                                        1.01    117.7±3.07ms        ? ?/sec    1.00    116.1±2.66ms        ? ?/sec
bundle/bundle@rome_ts-sourcemap                              1.02    133.2±5.50ms        ? ?/sec    1.00    130.1±2.48ms        ? ?/sec
bundle/bundle@threejs                                        1.02     41.9±2.71ms        ? ?/sec    1.00     41.1±2.71ms        ? ?/sec
bundle/bundle@threejs-sourcemap                              1.02     48.0±1.27ms        ? ?/sec    1.00     47.2±0.73ms        ? ?/sec
bundle/bundle@threejs10x                                     1.00    444.5±6.56ms        ? ?/sec    1.00    445.5±6.60ms        ? ?/sec
bundle/bundle@threejs10x-sourcemap                           1.00    518.1±6.58ms        ? ?/sec    1.00    517.5±7.60ms        ? ?/sec
scan/scan@rome_ts                                            1.01     91.3±2.16ms        ? ?/sec    1.00     90.5±2.25ms        ? ?/sec
scan/scan@threejs                                            1.00     31.0±0.81ms        ? ?/sec    1.03     31.8±2.17ms        ? ?/sec
scan/scan@threejs10x                                         1.00    330.1±6.52ms        ? ?/sec    1.00    328.5±7.41ms        ? ?/sec

@IWANABETHATGUY IWANABETHATGUY force-pushed the 06-13-fix_4944 branch 3 times, most recently from 62d00d4 to 224ea70 Compare June 13, 2025 08:13
@IWANABETHATGUY IWANABETHATGUY marked this pull request as ready for review June 13, 2025 10:18
@IWANABETHATGUY IWANABETHATGUY added this pull request to the merge queue Jun 13, 2025
@IWANABETHATGUY IWANABETHATGUY changed the title fix: 4944 fix: unbundle does not work with outExtensions Jun 13, 2025
Merged via the queue into main with commit 59e6bcd Jun 13, 2025
25 checks passed
@IWANABETHATGUY IWANABETHATGUY deleted the 06-13-fix_4944 branch June 13, 2025 10:29
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

@hyf0 hyf0 hyf0 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.

unbundle does not work with outExtensions

2 participants

@IWANABETHATGUY @hyf0