chore: remove rolldown_plugin_vite_wasm_helper#8207
Merged
graphite-app[bot] merged 1 commit intomainfrom Feb 5, 2026
Merged
Conversation
Member
Author
How to use the Graphite Merge QueueAdd the label graphite: merge-when-ready 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. |
✅ Deploy Preview for rolldown-rs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes the rolldown_plugin_vite_wasm_helper plugin from the codebase in preparation for refactoring in Vite's upstream repository. The plugin is being temporarily removed and will be re-ported later once the Vite refactoring (related to wasmHelperPlugin and fileToUrl behavior alignment) is complete.
Changes:
- Removed the entire
rolldown_plugin_vite_wasm_helpercrate and its implementation - Removed test fixtures for the wasm-helper plugin
- Cleaned up all TypeScript/JavaScript exports and type definitions
Reviewed changes
Copilot reviewed 16 out of 18 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
packages/rolldown/tests/fixtures/builtin-plugin/wasm-helper/ |
Removed test fixtures directory including test config, source files, and .gitignore |
packages/rolldown/src/experimental-index.ts |
Removed viteWasmHelperPlugin export from experimental API |
packages/rolldown/src/builtin-plugin/constructors.ts |
Removed viteWasmHelperPlugin factory function and related type imports |
packages/rolldown/src/binding.d.cts |
Removed TypeScript type definitions for BindingViteWasmHelperPluginConfig and associated types, plus removed plugin name from union type |
crates/rolldown_plugin_vite_wasm_helper/ |
Removed entire crate directory including lib.rs, wasm-runtime.js, README.md, and Cargo.toml |
crates/rolldown_binding/src/options/plugin/types/binding_builtin_plugin_name.rs |
Removed ViteWasmHelper enum variant |
crates/rolldown_binding/src/options/plugin/config/mod.rs |
Removed module declaration and re-export for wasm helper config |
crates/rolldown_binding/src/options/plugin/config/binding_vite_wasm_helper_plugin_config.rs |
Removed entire configuration file |
crates/rolldown_binding/src/options/plugin/binding_builtin_plugin.rs |
Removed import, config type usage, and match arm for ViteWasmHelper plugin |
crates/rolldown_binding/Cargo.toml |
Removed dependency on rolldown_plugin_vite_wasm_helper |
Cargo.toml |
Removed workspace member definition for rolldown_plugin_vite_wasm_helper |
Cargo.lock |
Removed package entry for rolldown_plugin_vite_wasm_helper and its dependency references |
Contributor
Benchmarks Rust |
sapphi-red
approved these changes
Feb 5, 2026
Contributor
Merge activity
|
Related to vitejs/vite#21566 This is because vitejs/vite#21102 will refactor the logic around `wasmHelperPlugin`, and `nativeWasmHelperPlugin` is not aligned with the `fileToUrl` behavior. So I plan to remove it for now and re-port it when it’s needed again later.
graphite-app
bot
force-pushed
the
02-05-chore_remove_rolldown_plugin_vite_wasm_helper
branch
from
af43384 to
049c53a
Compare
graphite-app
bot
deleted the
02-05-chore_remove_rolldown_plugin_vite_wasm_helper
branch
This was referenced Feb 11, 2026
Closed
Merged
shulaoda
added a commit
that referenced
this pull request
Feb 11, 2026
## [1.0.0-rc.4] - 2026-02-11 💡 Granular `comments` Option - New `output.comments` option provides fine-grained control over comment preservation - The `output.legalComments` option is now deprecated and use `comments.legal` instead ### 🚀 Features - rename error name to `RolldownError` from `RollupError` (#8262) by @sapphi-red - add hidden `resolve_tsconfig` function for Vite (#8257) by @sapphi-red - rust: introduce `rolldown_watcher` (#8161) by @hyf0 - unify `comments` and `legalComments` into a single granular `comments` option (#8229) by @IWANABETHATGUY - add builtin plugin for visualizing chunk graph (#8162) by @IWANABETHATGUY - show import declaration location in AssignToImport errors (#8222) by @Copilot - show import declaration span in CannotCallNamespace error (#8223) by @Copilot - emit error when plugin accidentally removes runtime module symbols (#8203) by @IWANABETHATGUY - support tsconfig loading & inputMap for `transform` (#8180) by @sapphi-red - rolldown_plugin_vite_reporter: update warning message to link to Rolldown docs (#8205) by @sapphi-red ### 🐛 Bug Fixes - avoid panic on untranspiled JSX syntax by reporting a diagnostic error (#8226) by @IWANABETHATGUY - rolldown_plugin_vite_import_glob: relax absolute path check and improve invalid glob warning (#8219) by @shulaoda - merge chunks after detect circular reference (#8154) by @IWANABETHATGUY - rust: detect runtime module side effects based on its content (#8209) by @hyf0 ### 🚜 Refactor - rename `other` to `jsdoc` in comments options (#8256) by @IWANABETHATGUY - rename chunk-visualize plugin with bundle-analyzer plugin (#8255) by @IWANABETHATGUY - remove EXPORT_UNDEFINED_VARIABLE error (#8228) by @Copilot - consolidate missing runtime symbol errors into a single diagnostic (#8220) by @IWANABETHATGUY - stabilize `parse` and `parseSync` (#8215) by @sapphi-red - return errors instead of panicking on builtin plugin conversion failure (#8217) by @shulaoda - expose `parse` / `minify` / `transform` from `rolldown/utils` (#8214) by @sapphi-red - prepare defer chunk merging (#8153) by @IWANABETHATGUY ### 📚 Documentation - remove `<script>` escape behavior difference note from `platform` option (#8253) by @sapphi-red - TypeScript & JSX support by plugins (#8183) by @sapphi-red ### 🧪 Testing - ensure runtime module is preserved even if it's not used but has side effects (#8213) by @hyf0 ### ⚙️ Miscellaneous Tasks - deps: update oxc to v0.113.0 (#8267) by @renovate[bot] - deps: update dependency oxlint-tsgolint to v0.12.0 (#8272) by @renovate[bot] - deps: update oxc apps (#8269) by @renovate[bot] - deps: update test262 submodule for tests (#8261) by @sapphi-red - deps: update crate-ci/typos action to v1.43.4 (#8260) by @renovate[bot] - deps: update dependency esbuild to v0.27.3 (#8250) by @renovate[bot] - deps: update rust crates (#8244) by @renovate[bot] - deps: update dependency semver to v7.7.4 (#8247) by @renovate[bot] - deps: update github-actions (#8243) by @renovate[bot] - deps: update npm packages (#8245) by @renovate[bot] - deps: update oxc resolver to v11.17.1 (#8240) by @renovate[bot] - deps: update rust crate oxc_sourcemap to v6.0.2 (#8241) by @renovate[bot] - rust: handle ignored `RUSTSEC-2025-0141` cargo check error (#8235) by @hyf0 - deps: update dependency oxlint-tsgolint to v0.11.5 (#8233) by @renovate[bot] - deps: update dependency rolldown-plugin-dts to ^0.22.0 (#8232) by @renovate[bot] - deps: update crate-ci/typos action to v1.43.3 (#8225) by @renovate[bot] - deps: update dependency rolldown-plugin-dts to v0.21.9 (#8224) by @renovate[bot] - deps: update crate-ci/typos action to v1.43.2 (#8212) by @renovate[bot] - remove rolldown_plugin_vite_wasm_helper (#8207) by @shulaoda - build docs for production (#8206) by @sapphi-red Co-authored-by: shulaoda <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related to vitejs/vite#21566
This is because vitejs/vite#21102 will refactor the logic around
wasmHelperPlugin, andnativeWasmHelperPluginis not aligned with thefileToUrlbehavior. So I plan to remove it for now and re-port it when it’s needed again later.