Skip to content

Comments

fix(debug): should emit chunk data after computing cross chunk links#5156

Merged
hyf0 merged 1 commit intomainfrom
07-03-fix_debug_should_emit_chunk_data_after_computing_cross_chunk_links
Jul 3, 2025
Merged

fix(debug): should emit chunk data after computing cross chunk links#5156
hyf0 merged 1 commit intomainfrom
07-03-fix_debug_should_emit_chunk_data_after_computing_cross_chunk_links

Conversation

@hyf0
Copy link
Member

@hyf0 hyf0 commented Jul 3, 2025

No description provided.

@hyf0 hyf0 marked this pull request as ready for review July 3, 2025 11:33
@hyf0 Graphite App
Copy link
Member Author

hyf0 commented Jul 3, 2025 •
edited
Loading

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

@netlify
Copy link

netlify bot commented Jul 3, 2025

✅ Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit bbe73c6
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/68666a8570a55c0008f194c5

@hyf0 hyf0 added this pull request to the merge queue Jul 3, 2025
@pkg-pr-new
Copy link

pkg-pr-new bot commented Jul 3, 2025

Open in StackBlitz

@rolldown/browser

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

@rolldown/debug

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

@rolldown/pluginutils

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

rolldown

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

@rolldown/binding-darwin-arm64

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

@rolldown/binding-darwin-x64

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

@rolldown/binding-freebsd-x64

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

@rolldown/binding-linux-arm-gnueabihf

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

@rolldown/binding-linux-arm64-gnu

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

@rolldown/binding-linux-arm64-musl

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

@rolldown/binding-linux-x64-gnu

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

@rolldown/binding-linux-x64-musl

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

@rolldown/binding-wasm32-wasi

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

@rolldown/binding-win32-arm64-msvc

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

@rolldown/binding-win32-ia32-msvc

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

@rolldown/binding-win32-x64-msvc

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

commit: bbe73c6

Merged via the queue into main with commit 043500c Jul 3, 2025
46 checks passed
@hyf0 hyf0 deleted the 07-03-fix_debug_should_emit_chunk_data_after_computing_cross_chunk_links branch July 3, 2025 11:47
@github-actions
Copy link
Contributor

github-actions bot commented Jul 3, 2025

Benchmarks Rust

  • target: main(b37e4b4)
  • pr: 07-03-fix_debug_should_emit_chunk_data_after_computing_cross_chunk_links(bbe73c6)
group                                                        pr                                     target
-----                                                        --                                     ------
bundle/bundle@multi-duplicated-top-level-symbol              1.05     83.6±4.40ms        ? ?/sec    1.00     79.9±2.49ms        ? ?/sec
bundle/bundle@multi-duplicated-top-level-symbol-sourcemap    1.03     92.6±2.92ms        ? ?/sec    1.00     90.3±1.83ms        ? ?/sec
bundle/bundle@rome_ts                                        1.02    118.3±4.40ms        ? ?/sec    1.00    116.3±1.72ms        ? ?/sec
bundle/bundle@rome_ts-sourcemap                              1.00    137.9±3.16ms        ? ?/sec    1.00    137.2±1.92ms        ? ?/sec
bundle/bundle@threejs                                        1.00     42.3±1.11ms        ? ?/sec    1.02     42.9±2.09ms        ? ?/sec
bundle/bundle@threejs-sourcemap                              1.00     51.2±1.12ms        ? ?/sec    1.01     51.8±0.96ms        ? ?/sec
bundle/bundle@threejs10x                                     1.00    446.3±7.05ms        ? ?/sec    1.00    445.6±4.43ms        ? ?/sec
bundle/bundle@threejs10x-sourcemap                           1.00    508.6±5.71ms        ? ?/sec    1.00    509.3±5.30ms        ? ?/sec
scan/scan@rome_ts                                            1.00     90.5±1.73ms        ? ?/sec    1.00     90.2±1.55ms        ? ?/sec
scan/scan@threejs                                            1.00     30.8±0.81ms        ? ?/sec    1.01     31.3±2.22ms        ? ?/sec
scan/scan@threejs10x                                         1.00    326.7±5.02ms        ? ?/sec    1.00    325.7±8.26ms        ? ?/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

trigger: preview

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@hyf0 @IWANABETHATGUY