Skip to content

Feat/ssr static update - #5063

Merged
2heal1 merged 3 commits into
feat/mf-ssr-clear-cachefrom
feat/ssr-static-update
Sep 11, 2026
Merged

2heal1 merged 3 commits into
feat/mf-ssr-clear-cachefrom
feat/ssr-static-update

Conversation

@2heal1

@2heal1 2heal1 commented Sep 11, 2026

Copy link
Copy Markdown
Member

Description

Related Issue

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist

  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have updated the documentation.

@changeset-bot

changeset-bot Bot commented Sep 11, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 0bcfc10

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 48 packages
Name Type
@module-federation/modern-js-v3 Minor
@module-federation/sdk Minor
@module-federation/runtime-core Minor
@module-federation/webpack-bundler-runtime Minor
shared-tree-shaking-with-server-host Patch
shared-tree-shaking-with-server-provider Patch
@module-federation/devtools Minor
@module-federation/cli Minor
@module-federation/dts-plugin Minor
@module-federation/enhanced Minor
@module-federation/esbuild Patch
@module-federation/managers Minor
@module-federation/manifest Minor
@module-federation/metro Minor
@module-federation/modern-js Minor
@module-federation/nextjs-mf Patch
@module-federation/node Patch
@module-federation/observability-plugin Patch
@module-federation/retry-plugin Minor
@module-federation/rsbuild-plugin Minor
@module-federation/rspack Minor
@module-federation/rspress-plugin Minor
@module-federation/rstest Minor
@module-federation/runtime Minor
@module-federation/storybook-addon Patch
@module-federation/utilities Patch
@module-federation/bridge-react-webpack-plugin Minor
@module-federation/bridge-react Minor
@module-federation/bridge-vue3 Minor
@module-federation/runtime-tools Minor
shared-tree-shaking-no-server-host Patch
shared-tree-shaking-no-server-provider Patch
@module-federation/metro-plugin-rnc-cli Minor
@module-federation/metro-plugin-rnef Minor
@module-federation/metro-plugin-rock Minor
node-dynamic-remote-new-version Patch
node-dynamic-remote Patch
@module-federation/playground Patch
remote5 Patch
remote6 Patch
website-new Patch
@module-federation/inject-external-runtime-core-plugin Minor
create-module-federation Minor
@module-federation/error-codes Minor
@module-federation/third-party-dts-extractor Minor
@module-federation/treeshake-frontend Minor
@module-federation/treeshake-server Minor
@module-federation/bridge-shared Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 11, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-11T09:40:37.675692Z 0bcfc10 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@2heal1
2heal1 merged commit d33266f into feat/mf-ssr-clear-cache Sep 11, 2026
5 checks passed
@2heal1
2heal1 deleted the feat/ssr-static-update branch September 11, 2026 09:36

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0bcfc10974

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

chain.plugin('modern-ssr-dependencies').use(SSRDependencyPlugin, [
{
name: pluginOptions.ssrConfig.name!,
remotes: Object.keys(pluginOptions.ssrConfig.remotes || {}),

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Normalize array-form remotes before collecting ownership

When remotes uses the supported array form from ModuleFederationPluginOptions (for example ['shop@https://…'] or [{ shop: '…' }]), Object.keys returns numeric indexes rather than request scopes. SSRDependencyPlugin therefore cannot match incoming shop/... requests, records unmapped-remote, and makes plan() fall back to an application rebuild even when cacheUpdates and staticOnly are enabled. Normalize array entries to their inferred or declared scopes before passing them to the plugin.

AGENTS.md reference: AGENTS.md:L289-L289

Useful? React with 👍 / 👎.

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.

1 participant