fix(indexHtml): ensure correct URL when querying module graph #19601
+22
−17
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.
Description
closes: #19135
Just adjust the order of code execution.
The main reason is that the URL during
preTransformRequest
may be inconsistent. For example, converting./src/main.js
to/src/main.js
.vite/packages/vite/src/node/server/middlewares/indexHtml.ts
Lines 157 to 166 in 0254645
Later, storing
/src/main.js
inurlToModuleMap
.vite/packages/vite/src/node/server/middlewares/indexHtml.ts
Line 174 in 0254645
When
HMR
is triggered, the value will be retrieved based on./src/main.js
instead of/src/main.js
, which will not access the previously stored data.vite/packages/vite/src/node/server/middlewares/indexHtml.ts
Lines 131 to 136 in 0254645