Skip to content

Comments

fix: moduleInfo is not updated when entry module is emitted by this.emitFile#5032

Merged
IWANABETHATGUY merged 1 commit intomainfrom
06-20-fix_5030
Jun 20, 2025
Merged

fix: moduleInfo is not updated when entry module is emitted by this.emitFile#5032
IWANABETHATGUY merged 1 commit intomainfrom
06-20-fix_5030

Conversation

@IWANABETHATGUY
Copy link
Member

@IWANABETHATGUY IWANABETHATGUY commented Jun 20, 2025 •
edited
Loading

closed #5030

@IWANABETHATGUY Graphite App
Copy link
Member Author

IWANABETHATGUY commented Jun 20, 2025

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

@IWANABETHATGUY IWANABETHATGUY marked this pull request as ready for review June 20, 2025 11:04
@IWANABETHATGUY IWANABETHATGUY changed the title fix: 5030 fix: moduleInfo is not updated when entry module is emitted by this.emitFile Jun 20, 2025
@netlify
Copy link

netlify bot commented Jun 20, 2025 •
edited
Loading

✅ Deploy Preview for rolldown-rs ready!

Name Link
🔨 Latest commit a2bc6f7
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/685541689029740008805eed
😎 Deploy Preview https://deploy-preview-5032--rolldown-rs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 20, 2025

Benchmarks Rust

group                                                        pr                                     target
-----                                                        --                                     ------
bundle/bundle@multi-duplicated-top-level-symbol              1.07     77.2±3.11ms        ? ?/sec    1.00     72.0±1.33ms        ? ?/sec
bundle/bundle@multi-duplicated-top-level-symbol-sourcemap    1.04     85.6±2.53ms        ? ?/sec    1.00     82.0±1.00ms        ? ?/sec
bundle/bundle@rome_ts                                        1.02    112.8±2.77ms        ? ?/sec    1.00    111.0±1.39ms        ? ?/sec
bundle/bundle@rome_ts-sourcemap                              1.01    127.3±2.26ms        ? ?/sec    1.00    125.9±1.50ms        ? ?/sec
bundle/bundle@threejs                                        1.03     41.0±1.74ms        ? ?/sec    1.00     39.9±0.67ms        ? ?/sec
bundle/bundle@threejs-sourcemap                              1.00     47.3±0.80ms        ? ?/sec    1.02     48.1±1.41ms        ? ?/sec
bundle/bundle@threejs10x                                     1.03    441.6±8.11ms        ? ?/sec    1.00    426.8±4.33ms        ? ?/sec
bundle/bundle@threejs10x-sourcemap                           1.01    513.2±7.64ms        ? ?/sec    1.00    509.0±9.40ms        ? ?/sec
scan/scan@rome_ts                                            1.00     87.7±1.62ms        ? ?/sec    1.05     91.7±3.20ms        ? ?/sec
scan/scan@threejs                                            1.00     30.6±0.80ms        ? ?/sec    1.06     32.3±2.16ms        ? ?/sec
scan/scan@threejs10x                                         1.00    320.6±3.78ms        ? ?/sec    1.03    329.7±8.37ms        ? ?/sec

@IWANABETHATGUY IWANABETHATGUY added this pull request to the merge queue Jun 20, 2025
Merged via the queue into main with commit 1cea807 Jun 20, 2025
23 checks passed
@IWANABETHATGUY IWANABETHATGUY deleted the 06-20-fix_5030 branch June 20, 2025 17:55
@github-actions github-actions bot mentioned this pull request Jun 22, 2025
Merged
hyf0 pushed a commit that referenced this pull request Jun 22, 2025
@github-actions @shulaoda
release: v1.0.0-beta.19 (#5038)
8950628 
## [1.0.0-beta.19] - 2025-06-22

### 🚀 Features

- support `OutputOptions#manualChunks` (#5037) by @hyf0
- advanced-chunks: support `advancedChunks#gruop#name` to be function
(#5035) by @hyf0
- rolldown_plugin_import_glob: align with `vitejs/vite#20163` (#5034) by
@shulaoda
- rust/advanced-chunks: support `MatchGroup#name` to be dynamic (#5033)
by @hyf0
- rolldown_plugin_build_import_analysis: align with `vitejs/vite#20117`
(#5027) by @shulaoda
- rolldown_plugin_build_import_analysis: align with `vitejs/vite#20115`
(#5020) by @shulaoda
- add validation warning for advanced chunks options without groups
(#5009) by @sapphi-red

### 🐛 Bug Fixes

- moduleInfo is not updated when entry module is emitted by
this.emitFile (#5032) by @IWANABETHATGUY
- preserveEntrySignatures: false generates circular imports that hangs
with TLA (#5029) by @IWANABETHATGUY
- rolldown_plugin_build_import_analysis: align pure dynamic import
handling with rolldown-vite (#5016) by @shulaoda
- plugin/vite-resolve: normalize leading slash (#5013) by @sapphi-red
- debug: `build_id` doesn't increase (#5015) by @hyf0
- side effects in this.emitFile({ type: 'chunk' }) is removed when
preserveEntrySignatures: false is set (#5012) by @IWANABETHATGUY

### 🚜 Refactor

- rolldown_utils: simplify `block_on_spawn_all` (#5019) by @shulaoda
- use `rolldown_utils::futures::block_on` for `WatcherImpl#start`
(#5018) by @shulaoda

### 📚 Documentation

- jsdoc: document `experimental.attachDebugInfo` (#5028) by @hyf0
- clarify that `advancedChunks` options are in bytes (#5022) by
@sapphi-red
- add a note that sequential conversion may break the code (#5024) by
@sapphi-red

### ⚙️ Miscellaneous Tasks

- infra: clean up `dist` before building `rolldown` (#5036) by @hyf0
- Align status notice in readme with documentation (#5021) by
@rijkvanzanten

### ❤️ New Contributors

* @rijkvanzanten made their first contribution in
[#5021](#5021)

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

@graphite-app graphite-app[bot] graphite-app[bot] left review comments

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

[Bug]: moduleInfo is not updated when entry module is emitted by this.emitFile

2 participants

@IWANABETHATGUY @hyf0