fix(manifest): record all remotes which are declared in build config (#3082)

Author: 2heal1

.changeset/eleven-flowers-agree.md

@@ -0,0 +1,5 @@
+---
+'@module-federation/manifest': patch
+---
+
+fix(manifest): record all remotes which are declared in build config

packages/managers/__tests__/__snapshots__/RemoteManager.spec.ts.snap

@@ -15,22 +15,22 @@ exports[`RemoteManager statsRemoteWithEmptyUsedIn 1`] = `
     "consumingFederationContainerName": "@module-federation/remote-managers-test",
     "entry": "http://localhost:3000/remoteEntry.js",
     "federationContainerName": "remote1",
-    "moduleName": "remote1",
+    "moduleName": "UNKNOWN",
     "usedIn": [],
   },
   {
     "alias": "remote-version",
     "consumingFederationContainerName": "@module-federation/remote-managers-test",
     "federationContainerName": "remote2",
-    "moduleName": "remote2",
+    "moduleName": "UNKNOWN",
     "usedIn": [],
     "version": "1.2.3",
   },
   {
     "alias": "remote-default",
     "consumingFederationContainerName": "@module-federation/remote-managers-test",
     "federationContainerName": "remote3",
-    "moduleName": "remote3",
+    "moduleName": "UNKNOWN",
     "usedIn": [],
     "version": "*",
   },

packages/managers/src/RemoteManager.ts

@@ -7,6 +7,7 @@ import {
   moduleFederationPlugin,
 } from '@module-federation/sdk';
 import { BasicPluginOptionsManager } from './BasicPluginOptionsManager';
+import { UNKNOWN_MODULE_NAME } from './constant';
 
 interface NormalizedRemote {
   [remoteName: string]: RemoteEntryInfo & {
@@ -50,15 +51,15 @@ class RemoteManager extends BasicPluginOptionsManager<moduleFederationPlugin.Mod
         curObj = {
           entry: normalizedOption.entry,
           alias: normalizedOption.alias,
-          moduleName: normalizedOption.name,
+          moduleName: UNKNOWN_MODULE_NAME,
           federationContainerName: normalizedOption.name,
           consumingFederationContainerName: name!,
           usedIn: [],
         };
       } else {
         curObj = {
           alias: normalizedOption.alias,
-          moduleName: normalizedOption.name,
+          moduleName: UNKNOWN_MODULE_NAME,
           version: normalizedOption.version,
           federationContainerName: normalizedOption.name,
           consumingFederationContainerName: name!,

packages/managers/src/constant.ts

@@ -1 +1,2 @@
 export const LOCAL_BUILD_VERSION = 'local';
+export const UNKNOWN_MODULE_NAME = 'UNKNOWN';

packages/manifest/src/StatsManager.ts

@@ -364,10 +364,21 @@ class StatsManager {
 
       await Promise.all([
         new Promise<void>((resolve) => {
-          stats.remotes = remotes.map((remote) => ({
-            ...remote,
-            usedIn: Array.from(remote.usedIn.values()),
-          }));
+          const remoteMemo: Set<string> = new Set();
+          stats.remotes = remotes.map((remote) => {
+            remoteMemo.add(remote.federationContainerName);
+            return {
+              ...remote,
+              usedIn: Array.from(remote.usedIn.values()),
+            };
+          });
+          const statsRemoteWithEmptyUsedIn =
+            this._remoteManager.statsRemoteWithEmptyUsedIn;
+          statsRemoteWithEmptyUsedIn.forEach((remoteInfo) => {
+            if (!remoteMemo.has(remoteInfo.federationContainerName)) {
+              stats.remotes.push(remoteInfo);
+            }
+          });
           resolve();
         }),
         new Promise<void>((resolve) => {