Skip to content

Conversation

sapphi-red
Copy link
Member

Description

Adds some code to use native refresh wrapper plugin in rolldown-vite.

@ArnaudBarre
Copy link
Member

The perf gain comes from the fact that you skip round-trip for all JSX? files?

@sapphi-red sapphi-red marked this pull request as ready for review September 24, 2025 12:16
@sapphi-red
Copy link
Member Author

I believe so (I didn't dig down much though). The other potential reason I can think of is since all files that is processed by JS would be processed serially, even if each process doesn't take that much time, it can be slower than other transforms done on the native side. It also mean that there might be a long queue which would be bad for mutex.

@sapphi-red sapphi-red merged commit bd1487d into main Sep 25, 2025
19 of 20 checks passed
@sapphi-red sapphi-red deleted the perf/react-use-native-refresh-wrapper-plugin branch September 25, 2025 02:06
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Sep 28, 2025
| datasource | package              | from  | to    |
| ---------- | -------------------- | ----- | ----- |
| npm        | @vitejs/plugin-react | 5.0.3 | 5.0.4 |


## [v5.0.4](https://github.com/vitejs/vite-plugin-react/blob/HEAD/packages/plugin-react/CHANGELOG.md#504-2025-09-27)

##### Perf: use native refresh wrapper plugin in rolldown-vite ([#881](vitejs/vite-plugin-react#881))
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Sep 28, 2025
| datasource | package              | from  | to    |
| ---------- | -------------------- | ----- | ----- |
| npm        | @vitejs/plugin-react | 5.0.3 | 5.0.4 |


## [v5.0.4](https://github.com/vitejs/vite-plugin-react/blob/HEAD/packages/plugin-react/CHANGELOG.md#504-2025-09-27)

##### Perf: use native refresh wrapper plugin in rolldown-vite ([#881](vitejs/vite-plugin-react#881))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants