feat(vite-dev-server): migrate to async fs operations for better perf… #32910
Conversation
…ormance - Replace fs.readFileSync with async fs.readFile - Add proper error handling with debug logs - Improve performance by avoiding event loop blocking - Remove TODO comment about async fs methods This follows Node.js best practices and improves DX with faster build times.
|
|
|
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
| } | ||
|
|
||
| // Initialize loader promise | ||
| loaderPromise = loadInitFile() |
There was a problem hiding this comment.
Bug: Unhandled Promise Rejection in Plugin Loader
Potential unhandled promise rejection. The loaderPromise is initialized immediately when the plugin is created, but if the file read fails and transformIndexHtml is never called (e.g., in certain Vite configurations or error scenarios), the rejected promise will never be awaited, causing an unhandled promise rejection warning in Node.js. The old synchronous code would fail immediately at plugin creation, providing clearer error feedback. Consider adding a .catch() handler to the promise initialization or deferring the file read until it's actually needed in transformIndexHtml.
3 participants
…ormance
This follows Node.js best practices and improves DX with faster build times.
Additional details
Steps to test
How has the user experience changed?
PR Tasks
cypress-documentation?type definitions?Note
Switches
cypress.tsto non-blocking file reads, preloads the init script, and adds debug-backed error handling.npm/vite-dev-server/src/plugins/cypress.ts):fs.readFileSyncand directfs.promises.readFilewith a shared promisifiedreadFilehelper.INIT_FILEPATHvialoaderPromiseand inject its contents intransformIndexHtml.loadInitFile()with debug logs and explicit error propagation.index.htmlvia asyncreadFileand inject discovered<script>tags and the preloaded loader script.Written by Cursor Bugbot for commit 8103f1c. This will update automatically on new commits. Configure here.