fix(enhanced): Add fallback parsing for container module exposes (#4034)

packages/enhanced/test/configCases/container/expose-edge-cases/index.js

@@ -0,0 +1,6 @@
+it('should be able to handle spaces in path to exposes', async () => {
+  const { default: test1 } = await import('./test 1');
+  const { default: test2 } = await import('./path with spaces/test-2');
+  expect(test1()).toBe('test 1');
+  expect(test2()).toBe('test 2');
+});

packages/enhanced/test/configCases/container/expose-edge-cases/path with spaces/test-2.js

@@ -0,0 +1,3 @@
+export default function test() {
+  return 'test 2';
+}

packages/enhanced/test/configCases/container/expose-edge-cases/test 1.js

@@ -0,0 +1,3 @@
+export default function test() {
+  return 'test 1';
+}

packages/enhanced/test/configCases/container/expose-edge-cases/webpack.config.js

@@ -0,0 +1,20 @@
+const { ModuleFederationPlugin } = require('../../../../dist/src');
+
+module.exports = {
+  mode: 'development',
+  devtool: false,
+  output: {
+    publicPath: 'http://localhost:3000/',
+  },
+  plugins: [
+    new ModuleFederationPlugin({
+      name: 'remote',
+      filename: 'remoteEntry.js',
+      manifest: true,
+      exposes: {
+        './test-1': './test 1.js',
+        './test-2': './path with spaces/test-2.js',
+      },
+    }),
+  ],
+};

packages/manifest/src/ModuleHandler.ts

@@ -292,7 +292,25 @@ class ModuleHandler {
     // identifier: container entry (default) [[".",{"import":["./src/routes/page.tsx"],"name":"__federation_expose_default_export"}]]'
     const data = identifier.split(' ');
 
-    JSON.parse(data[3]).forEach(([prefixedName, file]) => {
+    let entries: Array<
+      [exposeKey: string, { import: string[]; name?: string }]
+    > = [];
+
+    try {
+      // Prefer parsing exposes from stats first
+      entries = JSON.parse(data[3]);
+    } catch {
+      // If that fails, fallback to the original options
+      const exposes = this._options.exposes;
+
+      if (!exposes || typeof exposes !== 'object') {
+        return;
+      }
+
+      entries = Object.entries(exposes);
+    }
+
+    entries.forEach(([prefixedName, file]) => {
       // TODO: support multiple import
       exposesMap[getFileNameWithOutExt(file.import[0])] = getExposeItem({
         exposeKey: prefixedName,