Skip to content

Comments

feat: support OutputOptions#manualChunks#5037

Merged
hyf0 merged 1 commit intomainfrom
06-21-feat_support_outputoptions_manualchunks_
Jun 21, 2025
Merged

feat: support OutputOptions#manualChunks#5037
hyf0 merged 1 commit intomainfrom
06-21-feat_support_outputoptions_manualchunks_

Conversation

@hyf0
Copy link
Member

@hyf0 hyf0 commented Jun 21, 2025

Closes #4311.

Copy link
Member Author

hyf0 commented Jun 21, 2025

@hyf0 hyf0 marked this pull request as ready for review June 21, 2025 11:58
@graphite-app graphite-app bot changed the base branch from 06-21-feat_advanced-chunks_support_advancedchunks_gruop_name_to_be_function to graphite-base/5037 June 21, 2025 12:09
@hyf0 hyf0 force-pushed the 06-21-feat_support_outputoptions_manualchunks_ branch from 41681ce to 70a0e58 Compare June 21, 2025 12:21
@hyf0 hyf0 force-pushed the graphite-base/5037 branch from 9925f4f to e94c643 Compare June 21, 2025 12:21
@hyf0 hyf0 changed the base branch from graphite-base/5037 to 06-21-feat_advanced-chunks_support_advancedchunks_gruop_name_to_be_function June 21, 2025 12:21
@hyf0
Copy link
Member Author

hyf0 commented Jun 21, 2025

That's so weird the benchmark ci keeps failing.

Base automatically changed from 06-21-feat_advanced-chunks_support_advancedchunks_gruop_name_to_be_function to main June 21, 2025 13:51
@hyf0 hyf0 force-pushed the 06-21-feat_support_outputoptions_manualchunks_ branch from 70a0e58 to ad16701 Compare June 21, 2025 13:52
@graphite-app
Copy link
Contributor

graphite-app bot commented Jun 21, 2025

Merge activity

  • Jun 21, 1:53 PM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.

@netlify
Copy link

netlify bot commented Jun 21, 2025

Deploy Preview for rolldown-rs ready!

Name Link
🔨 Latest commit ad16701
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/6856b93c006de300080b9ace
😎 Deploy Preview https://deploy-preview-5037--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

Benchmarks Rust

  • target: main(9eed85e)
  • pr: 06-21-feat_support_outputoptions_manualchunks_(ad16701)
group                                                        pr                                     target
-----                                                        --                                     ------
bundle/bundle@multi-duplicated-top-level-symbol              1.00     74.6±1.92ms        ? ?/sec    1.00     74.4±1.92ms        ? ?/sec
bundle/bundle@multi-duplicated-top-level-symbol-sourcemap    1.01     85.5±1.93ms        ? ?/sec    1.00     85.0±1.90ms        ? ?/sec
bundle/bundle@rome_ts                                        1.00    112.0±1.47ms        ? ?/sec    1.00    111.8±1.72ms        ? ?/sec
bundle/bundle@rome_ts-sourcemap                              1.01    127.0±2.52ms        ? ?/sec    1.00    126.3±1.36ms        ? ?/sec
bundle/bundle@threejs                                        1.00     39.9±1.60ms        ? ?/sec    1.02     40.6±2.24ms        ? ?/sec
bundle/bundle@threejs-sourcemap                              1.00     46.5±0.46ms        ? ?/sec    1.00     46.6±0.67ms        ? ?/sec
bundle/bundle@threejs10x                                     1.01    433.6±4.70ms        ? ?/sec    1.00    428.9±3.73ms        ? ?/sec
bundle/bundle@threejs10x-sourcemap                           1.01    505.5±5.98ms        ? ?/sec    1.00    502.7±3.57ms        ? ?/sec
scan/scan@rome_ts                                            1.01     89.0±2.16ms        ? ?/sec    1.00     88.0±1.49ms        ? ?/sec
scan/scan@threejs                                            1.03     31.4±1.23ms        ? ?/sec    1.00     30.6±0.89ms        ? ?/sec
scan/scan@threejs10x                                         1.02    324.0±4.71ms        ? ?/sec    1.00    318.0±4.43ms        ? ?/sec

@hyf0 hyf0 added this pull request to the merge queue Jun 21, 2025
Merged via the queue into main with commit 13571e8 Jun 21, 2025
22 checks passed
@hyf0 hyf0 deleted the 06-21-feat_support_outputoptions_manualchunks_ branch June 21, 2025 14:15
@VillainsRule
Copy link

thanks 😁

hyf0 pushed a commit that referenced this pull request Jun 22, 2025
## [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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request]: add rollup manualChunks option

3 participants