chore: make manifest as false by default

Author: 2heal1

packages/rspack/src/container/ModuleFederationPlugin.ts

@@ -54,7 +54,7 @@ export class ModuleFederationPlugin {
 			enhanced: true
 		}).apply(compiler);
 
-		if (this._options.manifest !== false) {
+		if (this._options.manifest) {
 			const manifestOptions: ModuleFederationManifestPluginOptions =
 				this._options.manifest === true ? {} : { ...this._options.manifest };
 			const containerName = manifestOptions.name ?? this._options.name;

tests/rspack-test/configCases/container-1-5/manifest/rspack.config.js

@@ -14,6 +14,7 @@ module.exports = {
 			name: "container",
 			filename: "container.[chunkhash:8].js",
 			library: { type: "commonjs-module" },
+			manifest: true,
 			exposes: {
 				'./expose-a': {
 					import: './module.js',

tests/rspack-test/serialCases/container/0-container-full/rspack.config.js

@@ -27,7 +27,6 @@ module.exports = [
 			new ModuleFederationPlugin({
 				library: { type: "commonjs-module" },
 				filename: "container.js",
-				manifest: false,
 				remotes: {
 					containerA: {
 						external: "./container.js"
@@ -49,7 +48,6 @@ module.exports = [
 			new ModuleFederationPlugin({
 				library: { type: "module" },
 				filename: "module/container.mjs",
-				manifest: false,
 				remotes: {
 					containerA: {
 						external: "./container.mjs"

tests/rspack-test/serialCases/container/1-container-full/rspack.config.js

@@ -37,7 +37,6 @@ module.exports = [
 					containerA: "../0-container-full/container.js",
 					containerB: "./container.js"
 				},
-				manifest: false,
 				...commonMF
 			})
 		]
@@ -60,7 +59,6 @@ module.exports = [
 					containerA: "../../0-container-full/module/container.mjs",
 					containerB: "./container.mjs"
 				},
-				manifest: false,
 				...commonMF
 			})
 		],

tests/rspack-test/serialCases/container/2-container-full/rspack.config.js

@@ -20,7 +20,6 @@ module.exports = {
 				]
 			},
 			exposes: ["./Self"],
-			manifest: false,
 			shared: {
 				react: "react",
 				"old-react": {

tests/rspack-test/serialCases/container/3-container-full/rspack.config.js

@@ -6,7 +6,6 @@ module.exports = {
 	plugins: [
 		new ModuleFederationPlugin({
 			remoteType: "commonjs-module",
-			manifest: false,
 			remotes: {
 				containerB: "../1-container-full/container.js"
 			},

tests/rspack-test/serialCases/container/eager-shared/rspack.config.js

@@ -13,7 +13,6 @@ module.exports = {
 			remotes: {
 				service: "../0-eager-shared/container.js"
 			},
-			manifest: false,
 			shared: {
 				"tiny-emitter": {
 					eager: true,

tests/rspack-test/serialCases/container/module-federation-with-shareScope/rspack.config.js

@@ -34,7 +34,6 @@ module.exports = [
 				name: "container",
 				library: { type: "commonjs-module" },
 				filename: "container.js",
-				manifest: false,
 				remotes: {
 					containerA: "../0-container-full/container.js",
 					containerB: "./container.js"
@@ -61,7 +60,6 @@ module.exports = [
 					containerA: "../../0-container-full/module/container.mjs",
 					containerB: "./container.mjs"
 				},
-				manifest: false,
 				...commonMF
 			})
 		],

tests/rspack-test/serialCases/container/module-federation/rspack.config.js

@@ -11,7 +11,6 @@ function createConfig() {
 				filename: "container.js",
 				library: { type: "system" },
 				exposes: ["./other", "./self", "./dep"],
-				manifest: false,
 				remotes: {
 					abc: "ABC",
 					def: "DEF",
@@ -23,7 +22,6 @@ function createConfig() {
 				name: "container2",
 				filename: "container2.js",
 				library: { type: "system" },
-				manifest: false,
 				exposes: ["./other", "./self", "./dep"],
 				remotes: {
 					abc: "ABC",