Skip to content

Comments

fix: diagnostic message on minified code is too long to be readable#4950

Merged
shulaoda merged 2 commits intomainfrom
06-13-fix_4941
Jun 13, 2025
Merged

fix: diagnostic message on minified code is too long to be readable#4950
shulaoda merged 2 commits intomainfrom
06-13-fix_4941

Conversation

@IWANABETHATGUY
Copy link
Member

@IWANABETHATGUY IWANABETHATGUY commented Jun 13, 2025 •
edited
Loading

Closed #4941

@IWANABETHATGUY Graphite App
Copy link
Member Author

IWANABETHATGUY commented Jun 13, 2025

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

@netlify
Copy link

netlify bot commented Jun 13, 2025 •
edited
Loading

✅ Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit 0919af9
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/684c48d60568690008763ccf

@IWANABETHATGUY IWANABETHATGUY changed the title fix: 4941 fix: Diagnostic message on minified code is too long to be readable Jun 13, 2025
@IWANABETHATGUY IWANABETHATGUY marked this pull request as ready for review June 13, 2025 12:29
@socket-security
Copy link

socket-security bot commented Jun 13, 2025 •
edited
Loading

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security		 Vulnerability Quality Maintenance License
Addedpeggy@​5.0.31001009692100

View full report

graphite-app[bot]
graphite-app bot reviewed Jun 13, 2025
View reviewed changes
@github-actions
Copy link
Contributor

github-actions bot commented Jun 13, 2025 •
edited
Loading

Benchmarks Rust

group                                                        pr                                     target
-----                                                        --                                     ------
bundle/bundle@multi-duplicated-top-level-symbol              1.00     74.9±1.63ms        ? ?/sec    1.09     81.3±3.19ms        ? ?/sec
bundle/bundle@multi-duplicated-top-level-symbol-sourcemap    1.00     87.9±1.39ms        ? ?/sec    1.06     93.3±2.93ms        ? ?/sec
bundle/bundle@rome_ts                                        1.00    114.9±2.21ms        ? ?/sec    1.01    115.8±1.93ms        ? ?/sec
bundle/bundle@rome_ts-sourcemap                              1.01    131.2±2.16ms        ? ?/sec    1.00    130.5±2.26ms        ? ?/sec
bundle/bundle@threejs                                        1.01     41.9±2.19ms        ? ?/sec    1.00     41.5±2.69ms        ? ?/sec
bundle/bundle@threejs-sourcemap                              1.00     48.1±1.21ms        ? ?/sec    1.01     48.8±1.11ms        ? ?/sec
bundle/bundle@threejs10x                                     1.00    440.3±7.19ms        ? ?/sec    1.01    442.7±7.03ms        ? ?/sec
bundle/bundle@threejs10x-sourcemap                           1.00    512.2±6.68ms        ? ?/sec    1.01    517.1±8.26ms        ? ?/sec
scan/scan@rome_ts                                            1.00     88.0±1.39ms        ? ?/sec    1.02     90.1±1.69ms        ? ?/sec
scan/scan@threejs                                            1.00     30.8±0.58ms        ? ?/sec    1.01     31.2±1.73ms        ? ?/sec
scan/scan@threejs10x                                         1.00    323.8±3.80ms        ? ?/sec    1.02    329.5±6.57ms        ? ?/sec

graphite-app[bot]
graphite-app bot reviewed Jun 13, 2025
View reviewed changes
@shulaoda shulaoda changed the title fix: Diagnostic message on minified code is too long to be readable fix: diagnostic message on minified code is too long to be readable Jun 13, 2025
@graphite-app
Copy link
Contributor

graphite-app bot commented Jun 13, 2025 •
edited
Loading

Merge activity

  • Jun 13, 3:43 PM UTC: This pull request can not be added to the Graphite merge queue. Please try rebasing and resubmitting to merge when ready.
  • Jun 13, 3:43 PM UTC: Graphite disabled "merge when ready" on this PR due to: a merge conflict with the target branch; resolve the conflict and try again..

graphite-app[bot]
graphite-app bot reviewed Jun 13, 2025
View reviewed changes
@shulaoda shulaoda added this pull request to the merge queue Jun 13, 2025
Merged via the queue into main with commit d2c42c3 Jun 13, 2025
21 checks passed
@shulaoda shulaoda deleted the 06-13-fix_4941 branch June 13, 2025 16:17
github-actions bot pushed a commit that referenced this pull request Jun 15, 2025
@shulaoda @github-actions
release: v1.0.0-beta.16
5a8075e 
## [1.0.0-beta.16] - 2025-06-15

### 🚀 Features

- rolldown: feat(rolldown): oxc v0.73.0 by @Boshen in [#4948](#4948)
- types: feat(types): generate HMR runtime types by @sapphi-red in [#4928](#4928)
- rolldown_plugin_build_import_analysis: feat(rolldown_plugin_build_import_analysis): align with rolldown-vite by @shulaoda in [#4931](#4931)
- feat: allow to configure`experimental#attachDebugInfo: 'none'` to disable generating `#region` comments by @TheAlexLichter in [#4918](#4918)

### 🐛 Bug Fixes

- fix: deduplicate entry point by @IWANABETHATGUY in [#4956](#4956)
- fix: diagnostic message on minified code is too long to be readable by @IWANABETHATGUY in [#4950](#4950)
- fix: deduplicated entry points when have same `id`, `name`, and `filename` by @IWANABETHATGUY in [#4947](#4947)
- fix: unbundle does not work with outExtensions by @IWANABETHATGUY in [#4945](#4945)
- types: fix(types): remove `undefined` from return type of `generateHmrPatch` and `hmrInvalidate` by @sapphi-red in [#4943](#4943)
- fix: handle error in `hmrInvalidate` by @sapphi-red in [#4942](#4942)
- rolldown_plugin_build_import_analysis: fix(rolldown_plugin_build_import_analysis): preload helper is imported even if it’s not needed by @shulaoda in [#4936](#4936)
- hmr: fix(hmr): avoid panic after syntax error by @sapphi-red in [#4898](#4898)
- strictExecutionOrder: fix(strictExecutionOrder): ensure initialization chain of entry exports by @hyf0 in [#4933](#4933)
- fix: should not panic when meet a syntax error in watch mode. by @IWANABETHATGUY in [#4926](#4926)
- fix: regression, id filter normalization for windows is not working by @IWANABETHATGUY in [#4922](#4922)
- rolldown_plugin_json: fix(rolldown_plugin_json): incorrect default JSON export by @shulaoda in [#4924](#4924)

### 🚜 Refactor

- refactor: avoid unnecessary collect by @IWANABETHATGUY in [#4949](#4949)
- refactor: get normalized options via binding by @IWANABETHATGUY in [#4938](#4938)
- refactor: include DevRuntime base class even if custom HMR implementation is passed by @sapphi-red in [#4929](#4929)

### 📚 Documentation

- docs: clarify minifier state by @TheAlexLichter in [#4953](#4953)
- docs: drafting code splitting by @hyf0 in [#4951](#4951)
- docs: add link to repl by @sxzz in [#4917](#4917)
- pluginutils: docs(pluginutils): add example to simple filter functions by @sapphi-red in [#4915](#4915)

### 🧪 Testing

- test: add testcase for normalized output options by @IWANABETHATGUY in [#4939](#4939)
- hmr: test(hmr): enable `expectExecuted` for `generatePatchError` test by @sapphi-red in [#4937](#4937)
- test: evaluate hmr tests in node by @sapphi-red in [#4930](#4930)
- test: add "non used export" hmr test by @sapphi-red in [#4900](#4900)
- test: enable expectExecuted in HMR tests by @sapphi-red in [#4927](#4927)

### ⚙️ Miscellaneous Tasks

- deps: chore(deps): lock file maintenance npm packages by @renovate[bot] in [#4866](#4866)
- infra: chore(infra): remove comments related to `cspell` by @shulaoda in [#4940](#4940)
- ci: remove cargo bench test by @IWANABETHATGUY in [#4925](#4925)
@Boshen Boshen mentioned this pull request Jun 15, 2025
Merged
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]: Diagnostic message on minified code is too long to be readable

3 participants

@IWANABETHATGUY @hyf0 @shulaoda