Skip to content

feat(rust/dev): support disable_watcher#6329

Merged
graphite-app[bot] merged 1 commit intomainfrom
09-25-feat_rust_dev_support_disable_watcher_
Sep 25, 2025
Merged

feat(rust/dev): support disable_watcher#6329
graphite-app[bot] merged 1 commit intomainfrom
09-25-feat_rust_dev_support_disable_watcher_

Conversation

@hyf0
Copy link
Member

@hyf0 hyf0 commented Sep 25, 2025

For integerating with rust hmr test infra.

Copy link
Member Author

hyf0 commented Sep 25, 2025


How to use the Graphite Merge Queue

Add the label graphite: merge to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

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

@hyf0 hyf0 marked this pull request as ready for review September 25, 2025 09:40
@hyf0 hyf0 requested review from sapphi-red and shulaoda September 25, 2025 09:41
@netlify
Copy link

netlify bot commented Sep 25, 2025

Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit c9b69cc
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/68d516e8f309e10008f7e482

@github-actions
Copy link
Contributor

github-actions bot commented Sep 25, 2025

Benchmarks Rust

  • target: main(6c7fe56)
  • pr: 09-25-feat_rust_dev_support_disable_watcher_(c9b69cc)
group                                                        pr                                     target
-----                                                        --                                     ------
bundle/bundle@multi-duplicated-top-level-symbol              1.03     70.7±1.97ms        ? ?/sec    1.00     68.9±1.87ms        ? ?/sec
bundle/bundle@multi-duplicated-top-level-symbol-sourcemap    1.00     76.5±1.63ms        ? ?/sec    1.03     78.6±2.95ms        ? ?/sec
bundle/bundle@rome_ts                                        1.02    113.8±2.39ms        ? ?/sec    1.00    112.1±2.30ms        ? ?/sec
bundle/bundle@rome_ts-sourcemap                              1.01    129.4±2.58ms        ? ?/sec    1.00    128.7±2.73ms        ? ?/sec
bundle/bundle@threejs                                        1.00     44.5±1.01ms        ? ?/sec    1.00     44.5±2.48ms        ? ?/sec
bundle/bundle@threejs-sourcemap                              1.00     51.6±0.89ms        ? ?/sec    1.00     51.6±1.16ms        ? ?/sec
bundle/bundle@threejs10x                                     1.00    433.7±5.58ms        ? ?/sec    1.00    433.6±7.87ms        ? ?/sec
bundle/bundle@threejs10x-sourcemap                           1.00    495.4±6.23ms        ? ?/sec    1.02    502.8±5.55ms        ? ?/sec
scan/scan@rome_ts                                            1.00     87.4±1.69ms        ? ?/sec    1.02     89.5±1.76ms        ? ?/sec
scan/scan@threejs                                            1.00     32.3±0.51ms        ? ?/sec    1.04     33.6±2.18ms        ? ?/sec
scan/scan@threejs10x                                         1.00    334.0±4.60ms        ? ?/sec    1.01    338.9±4.57ms        ? ?/sec

@graphite-app
Copy link
Contributor

graphite-app bot commented Sep 25, 2025

Merge activity

For integerating with rust hmr test infra.
@graphite-app graphite-app bot force-pushed the 09-25-feat_rust_dev_support_disable_watcher_ branch from d633feb to c9b69cc Compare September 25, 2025 10:18
@graphite-app graphite-app bot merged commit c9b69cc into main Sep 25, 2025
28 checks passed
@graphite-app graphite-app bot deleted the 09-25-feat_rust_dev_support_disable_watcher_ branch September 25, 2025 10:31
shulaoda added a commit that referenced this pull request Sep 30, 2025
## [1.0.0-beta.41] - 2025-09-29

### 🚀 Features

- support `output.generatedCode.symbols` (#6335) by @IWANABETHATGUY
- rolldown: oxc v0.93.0 (#6364) by @Boshen
- rolldown_plugin_vite_html: finish script tag logic (#6355) by @shulaoda
- rolldown_plugin_vite_html: support inline html proxy (#6353) by @shulaoda
- rolldown_plugin_vite_html: transform script tag into js import (#6351) by @shulaoda
- rolldown_plugin_vite_html: overwrite script src url (#6349) by @shulaoda
- rolldown_plugin_vite_html: remove `vite-ignore` attribute (#6348) by @shulaoda
- rolldown_plugin_vite_html: use `html5ever` and `markup5ever_rcdom` (#6327) by @shulaoda
- rust/dev: support `on_output` callback (#6330) by @hyf0
- rust/dev: support `disable_watcher` (#6329) by @hyf0
- dev: introduce client/session concept into dev engine (#6297) by @hyf0

### 🐛 Bug Fixes

- 'asset' module type and CJS format produces warnings and wrong output (#6369) by @IWANABETHATGUY
- inlining constants in CJS by optimization.inlineConst incorrectly inlines non-constant values if the value is assigned more than once (#6328) by @IWANABETHATGUY
- dev: add `hasLatestBuildOutput` instead of `scheduleBuildIfStale` and add edit-reload test (#6321) by @sapphi-red

### 🚜 Refactor

- rust/dev: rename `HmrManager` to `HmrState` and tweak namings (#6358) by @hyf0
- rust/dev: use `Bundler` to expose hmr related methods (#6356) by @hyf0
- rolldown_plugin_vite_html: use `html5gum` instead (#6343) by @shulaoda
- dev: remove deprecated `new` field from HMR options and related configurations (#6337) by @hyf0
- dev: remove unused hmr API (#6336) by @hyf0
- make `cjs_ast_analyzer` immutable for better reuse (#6326) by @IWANABETHATGUY
- simplify logic for determining entry is_lived (#6325) by @IWANABETHATGUY

### ⚡ Performance

- dev: compute depended data incrementally (#6357) by @hyf0

### 🧪 Testing

- rust/hmr: use dev engine to test hmr (#6334) by @hyf0

### ⚙️ Miscellaneous Tasks

- deps: update notify (#6368) by @sapphi-red
- deps: lock file maintenance npm packages (#6366) by @renovate[bot]
- deps: update github-actions (#6365) by @renovate[bot]
- deps: update dependency rolldown-plugin-dts to v0.16.9 (#6354) by @renovate[bot]
- deps: update dependency tsdown to v0.15.5 (#6350) by @renovate[bot]
- node: use `catalog:` to unify version of all dependencies (#6339) by @hyf0
- ci: unify node installation via oxc-project/setup-node (#6338) by @Boshen
- deps: update crate-ci/typos action to v1.36.3 (#6341) by @renovate[bot]
- remove unused snapshots (#6331) by @IWANABETHATGUY

Co-authored-by: shulaoda <165626830+shulaoda@users.noreply.github.com>
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.

2 participants