Skip to content

Comments

feat(debug): emit asset-related data#5124

Merged
hyf0 merged 1 commit intomainfrom
07-01-feat_debug_emit_asset-related_data
Jul 1, 2025
Merged

feat(debug): emit asset-related data#5124
hyf0 merged 1 commit intomainfrom
07-01-feat_debug_emit_asset-related_data

Conversation

@hyf0
Copy link
Member

@hyf0 hyf0 commented Jul 1, 2025

No description provided.

@hyf0 hyf0 marked this pull request as ready for review July 1, 2025 05:32
@hyf0 Graphite App
Copy link
Member Author

hyf0 commented Jul 1, 2025 •
edited
Loading

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

@netlify
Copy link

netlify bot commented Jul 1, 2025

✅ Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit 4b4afcb
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/686372d550e124000846187a

@github-actions
Copy link
Contributor

github-actions bot commented Jul 1, 2025

Benchmarks Rust

  • target: main(7c1ee27)
  • pr: 07-01-feat_debug_emit_asset-related_data(4b4afcb)
group                                                        pr                                     target
-----                                                        --                                     ------
bundle/bundle@multi-duplicated-top-level-symbol              1.00     81.3±3.43ms        ? ?/sec    1.04     84.5±3.30ms        ? ?/sec
bundle/bundle@multi-duplicated-top-level-symbol-sourcemap    1.00     93.2±3.40ms        ? ?/sec    1.03     95.7±3.24ms        ? ?/sec
bundle/bundle@rome_ts                                        1.00    115.9±2.62ms        ? ?/sec    1.05    121.3±3.38ms        ? ?/sec
bundle/bundle@rome_ts-sourcemap                              1.00    137.4±2.64ms        ? ?/sec    1.01    139.2±2.49ms        ? ?/sec
bundle/bundle@threejs                                        1.00     43.2±1.76ms        ? ?/sec    1.01     43.8±2.74ms        ? ?/sec
bundle/bundle@threejs-sourcemap                              1.00     51.9±1.17ms        ? ?/sec    1.03     53.4±0.94ms        ? ?/sec
bundle/bundle@threejs10x                                     1.00    453.8±8.57ms        ? ?/sec    1.01    457.5±7.17ms        ? ?/sec
bundle/bundle@threejs10x-sourcemap                           1.00    523.6±8.45ms        ? ?/sec    1.01    528.5±8.60ms        ? ?/sec
scan/scan@rome_ts                                            1.00     91.7±1.82ms        ? ?/sec    1.01     92.7±2.06ms        ? ?/sec
scan/scan@threejs                                            1.00     32.1±0.54ms        ? ?/sec    1.01     32.4±1.86ms        ? ?/sec
scan/scan@threejs10x                                         1.00    330.0±3.58ms        ? ?/sec    1.01    331.9±5.83ms        ? ?/sec

@antfu antfu mentioned this pull request Jul 1, 2025
Merged
@hyf0 hyf0 mentioned this pull request Jul 1, 2025
Merged
@hyf0 hyf0 added this pull request to the merge queue Jul 1, 2025
Merged via the queue into main with commit 7f7651e Jul 1, 2025
24 checks passed
@hyf0 hyf0 deleted the 07-01-feat_debug_emit_asset-related_data branch July 1, 2025 06:46
github-merge-queue bot pushed a commit that referenced this pull request Jul 1, 2025
@antfu @hyf0
refactor(debug)!: rename debug logs fields to be consitent (#5125)
a62e0b0 
This is an incompleted PR to suggest on debug log fields renaming, based
on #5124

I only changed the interface but not the implementation, as I don't have
Rust installed yet. @hyf0 please review the changes and help me update
the implementation if they looks good to you.

Co-authored-by: Yunfei He <[email protected]>
This was referenced Jul 1, 2025
Closed
Merged
hyf0 pushed a commit that referenced this pull request Jul 1, 2025
@github-actions @shulaoda
release: v1.0.0-beta.23 (#5133)
fbab789 
## [1.0.0-beta.23] - 2025-07-01

### 💥 BREAKING CHANGES

- debug: rename debug logs fields to be consitent (#5125) by @antfu

### 🚀 Features

- debug: append `content` for asset data and unify `source` field to
`content`. (#5130) by @hyf0
- debug: emit asset-related data (#5124) by @hyf0
- commonjs tree shaking. (#5095) by @IWANABETHATGUY
- rolldown_plugin_wasm_helper: align behaviors for expected
functionality (#5120) by @shulaoda
- remove `experimental.enableComposingJsPlugins` (#5122) by @shulaoda
- use same option object reference (#5114) by @sapphi-red
- add util function to inspect why the facade symbol is created (#5115)
by @IWANABETHATGUY
- types: expose `ChunkingContext` type (#5112) by @sapphi-red
- scan commonjs exports (#5111) by @IWANABETHATGUY
- debug: remove unneeded source info of render chunk events (#5109) by
@hyf0
- debug: only emit debug information for trace level (#5108) by @hyf0
- debug: emit chunk related informations (#5106) by @hyf0

### 🐛 Bug Fixes

- debug: add hook render chunk start and end events to Meta enum (#5117)
by @hyf0

### 🚜 Refactor

- rust: unify outdated namings (#5127) by @hyf0

### 📚 Documentation

- advanced-chunks: clarify module capturing behavior (#5129) by @hyf0

### 🧪 Testing

- support array output options (#5113) by @sapphi-red

### ⚙️ Miscellaneous Tasks

- remove unused code (#5128) by @IWANABETHATGUY
- remove unnecessary `skipComposingJsPlugin` (#5123) by @shulaoda
- deps: update crate-ci/typos action to v1.34.0 (#5119) by
@renovate[bot]
- deps: update NAPI-RS to 3.0.0-beta.11 (#5110) by @shulaoda

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

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@hyf0 @IWANABETHATGUY