Skip to content

fix: fix up race condition with missing await#605

Merged
gioboa merged 1 commit intomainfrom
fix/603
Apr 9, 2026
Merged

fix: fix up race condition with missing await#605
gioboa merged 1 commit intomainfrom
fix/603

Conversation

@gioboa
Copy link
Copy Markdown
Collaborator

@gioboa gioboa commented Apr 9, 2026

Close #603

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Apr 9, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@module-federation/vite@605

commit: 8bdc034

Copy link
Copy Markdown
Collaborator Author

@gioboa gioboa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests With Different Frameworks

Image

Copy link
Copy Markdown
Collaborator Author

@gioboa gioboa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@robinengeldmde is this pre-release working for you?
I tested with the reproduction repo and it's working fine.

@gioboa gioboa merged commit 4c60947 into main Apr 9, 2026
22 checks passed
@gioboa gioboa deleted the fix/603 branch April 9, 2026 19:23
@klassm
Copy link
Copy Markdown

klassm commented Apr 10, 2026

I had the same issue. The pre-release works great for me.

@robinengeldmde
Copy link
Copy Markdown

Can also confirm that the pre-release fixes the Issue in our real world scenario. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Vite 8 + React prod mode crash - race condition because of missing await

3 participants