Skip to content

fix: fix symlinked workspace singleton sharing#679

Merged
gioboa merged 5 commits intomodule-federation:mainfrom
francesc79:fix/676-remote-entry-init
May 4, 2026
Merged

fix: fix symlinked workspace singleton sharing#679
gioboa merged 5 commits intomodule-federation:mainfrom
francesc79:fix/676-remote-entry-init

Conversation

@francesc79
Copy link
Copy Markdown
Contributor

@francesc79 francesc79 commented May 1, 2026

Close #676

…kspace ESM paths via realpath for correct lazy loading
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 1, 2026

Open in StackBlitz

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

commit: 79bc35f

Copy link
Copy Markdown
Collaborator

@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.

Thanks @francesc79 i will check it in the next days 👍

Copy link
Copy Markdown
Collaborator

@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.

build now emits top-level await import(...) and this is breaking the logic. We are running await from TLAs.

gioboa added 2 commits May 4, 2026 12:31
…olve workspace ESM paths via realpath for correct lazy loading"

This reverts commit 16b5912.
@gioboa gioboa changed the title fix: avoid top-level await for non-workspace singleton shares in build fix: fix symlinked workspace singleton sharing May 4, 2026
@gioboa
Copy link
Copy Markdown
Collaborator

gioboa commented May 4, 2026

@francesc79 I tested this with your repo, it's working great 😊

Copy link
Copy Markdown
Collaborator

@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

@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.

Thanks @francesc79 for your help

@gioboa gioboa merged commit d3b2e6c into module-federation:main May 4, 2026
19 checks passed
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]: Production singleton shared library using Vaadin is duplicated/evaluated in remote

2 participants