Skip to content

feat(debug): only emit debug information for trace level #5108

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jun 30, 2025
Merged

feat(debug): only emit debug information for trace level #5108

merged 1 commit into from
Jun 30, 2025

Conversation

hyf0
Copy link
Member

@hyf0 hyf0 commented Jun 30, 2025

No description provided.

@hyf0 hyf0 marked this pull request as ready for review June 30, 2025 07:55
@hyf0 hyf0 mentioned this pull request Jun 30, 2025
Merged
@hyf0 Graphite App
Copy link
Member Author

hyf0 commented Jun 30, 2025 •
edited
Loading

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

Base automatically changed from 06-30-feat_debug_emit_chunk_related_informations to main June 30, 2025 07:57
@hyf0 hyf0 force-pushed the 06-30-feat_debug_only_emit_debug_information_for_trace_level branch from feef00f to f610124 Compare June 30, 2025 09:12
@netlify Netlify
Copy link

netlify bot commented Jun 30, 2025

✅ Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit f610124
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/686254fa02d414000877a3bd

@hyf0 hyf0 mentioned this pull request Jun 30, 2025
Merged
@github-actions GitHub Actions
Copy link
Contributor

Benchmarks Rust

  • target: main(67b295f)
  • pr: 06-30-feat_debug_only_emit_debug_information_for_trace_level(f610124)
group                                                        pr                                     target
-----                                                        --                                     ------
bundle/bundle@multi-duplicated-top-level-symbol              1.00     76.6±3.88ms        ? ?/sec    1.02     78.5±2.37ms        ? ?/sec
bundle/bundle@multi-duplicated-top-level-symbol-sourcemap    1.03     89.8±2.96ms        ? ?/sec    1.00     86.8±1.27ms        ? ?/sec
bundle/bundle@rome_ts                                        1.00    116.9±2.48ms        ? ?/sec    1.00    116.6±4.07ms        ? ?/sec
bundle/bundle@rome_ts-sourcemap                              1.01    135.7±2.91ms        ? ?/sec    1.00    134.1±1.83ms        ? ?/sec
bundle/bundle@threejs                                        1.00     42.0±2.39ms        ? ?/sec    1.01     42.4±2.62ms        ? ?/sec
bundle/bundle@threejs-sourcemap                              1.03     51.9±2.70ms        ? ?/sec    1.00     50.4±0.96ms        ? ?/sec
bundle/bundle@threejs10x                                     1.03    447.7±7.77ms        ? ?/sec    1.00    435.7±6.37ms        ? ?/sec
bundle/bundle@threejs10x-sourcemap                           1.01    515.7±6.57ms        ? ?/sec    1.00    511.9±6.78ms        ? ?/sec
scan/scan@rome_ts                                            1.03     90.8±1.96ms        ? ?/sec    1.00     88.0±1.86ms        ? ?/sec
scan/scan@threejs                                            1.01     31.9±0.74ms        ? ?/sec    1.00     31.6±1.83ms        ? ?/sec
scan/scan@threejs10x                                         1.02    327.7±4.41ms        ? ?/sec    1.00    321.4±4.54ms        ? ?/sec

@hyf0 hyf0 requested a review from shulaoda June 30, 2025 09:30
@hyf0 hyf0 added this pull request to the merge queue Jun 30, 2025
Merged via the queue into main with commit 9ccb35b Jun 30, 2025
22 checks passed
@hyf0 hyf0 deleted the 06-30-feat_debug_only_emit_debug_information_for_trace_level branch June 30, 2025 09:38
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

@shulaoda shulaoda shulaoda 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 @shulaoda