fix(enhanced): skip ensure file when virtualEntry (#3037)

Author: ScriptedAlchemy

.changeset/ai-calm-bear.md

@@ -0,0 +1,7 @@
+---
+"@module-federation/enhanced": patch
+---
+
+Added a check to skip processing when virtualRuntimeEntry is present.
+
+- Added an early return in `FederationRuntimePlugin` to skip processing if `options.virtualRuntimeEntry` is defined.

packages/enhanced/src/lib/container/runtime/FederationRuntimePlugin.ts

@@ -198,6 +198,10 @@ class FederationRuntimePlugin {
     if (!this.options) {
       return;
     }
+    // skip virtual entry
+    if (this.options?.virtualRuntimeEntry) {
+      return;
+    }
     const filePath = this.getFilePath(compiler);
     try {
       fs.readFileSync(filePath);

packages/enhanced/test/configCases/container/virtual-entry/App.js

@@ -0,0 +1,6 @@
+import React from 'react';
+import ComponentA from 'containerA/ComponentA';
+
+export default () => {
+  return `App rendered with [${React()}] and [${ComponentA()}]`;
+};

packages/enhanced/test/configCases/container/virtual-entry/ComponentA.js

@@ -0,0 +1,5 @@
+import React from 'react';
+
+export default () => {
+  return `ComponentA rendered with [${React()}]`;
+};

packages/enhanced/test/configCases/container/virtual-entry/index.js

@@ -0,0 +1,15 @@
+it('should load the component from container', () => {
+  return import('./App').then(({ default: App }) => {
+    const rendered = App();
+    expect(rendered).toBe(
+      'App rendered with [This is react 0.1.2] and [ComponentA rendered with [This is react 0.1.2]]',
+    );
+    return import('./upgrade-react').then(({ default: upgrade }) => {
+      upgrade();
+      const rendered = App();
+      expect(rendered).toBe(
+        'App rendered with [This is react 1.2.3] and [ComponentA rendered with [This is react 1.2.3]]',
+      );
+    });
+  });
+});

packages/enhanced/test/configCases/container/virtual-entry/node_modules/react.js

@@ -0,0 +1,5 @@
+module.exports = {
+  findBundle: function (i, options) {
+    return i === 0 ? './main.js' : './module/main.mjs';
+  },
+};

packages/enhanced/test/configCases/container/virtual-entry/test.config.js

@@ -0,0 +1,5 @@
+import { setVersion } from 'react';
+
+export default function upgrade() {
+  setVersion('1.2.3');
+}

packages/enhanced/test/configCases/container/virtual-entry/upgrade-react.js

@@ -0,0 +1,61 @@
+const { ModuleFederationPlugin } = require('../../../../dist/src');
+
+const common = {
+  name: 've',
+  exposes: {
+    './ComponentA': {
+      import: './ComponentA',
+    },
+  },
+  shared: {
+    react: {
+      version: false,
+      requiredVersion: false,
+    },
+  },
+};
+
+module.exports = [
+  {
+    output: {
+      filename: '[name].js',
+      uniqueName: '0-ve-full',
+    },
+    plugins: [
+      new ModuleFederationPlugin({
+        library: { type: 'commonjs-module' },
+        virtualRuntimeEntry: true,
+        filename: 'container.js',
+        remotes: {
+          containerA: {
+            external: './container.js',
+          },
+        },
+        ...common,
+      }),
+    ],
+  },
+  {
+    experiments: {
+      outputModule: true,
+    },
+    output: {
+      filename: 'module/[name].mjs',
+      uniqueName: '0-ve-full-mjs',
+    },
+    plugins: [
+      new ModuleFederationPlugin({
+        library: { type: 'module' },
+        virtualRuntimeEntry: true,
+        filename: 'module/container.mjs',
+        remotes: {
+          containerA: {
+            external: './container.mjs',
+          },
+        },
+        ...common,
+      }),
+    ],
+    target: 'node14',
+  },
+];