Fix dest already exists
error during Vite build
#9305
Merged
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.
This fixes the issue raised in #9239.
This issue occurs when a server asset is present in the client build but isn't present in any of the client manifest's
assets
arrays. Unfortunately I wasn't able to work out why this can happen, but we can still handle this case regardless.The list of client assets was only used to determine whether to move a server asset or remove it. This means it's safe to expand the list of client assets to be the list of all files in the client build, regardless of whether they're considered assets or not. The server asset detection logic remains the same so we won't be moving additional files.