@nx/react 19.7.0-canary.20240816-ce2dff3 → 19.7.0-canary.20240817-b91d788

package/migrations.json

@@ -53,6 +53,12 @@
       "version": "19.6.0-beta.4",
       "description": "Update the server file for Module Federation SSR port value to be the same as the 'serve' target port value.",
       "factory": "./src/migrations/update-19-6-0/update-ssr-server-port"
+    },
+    "update-19-6-1-ensure-module-federation-target-defaults": {
+      "cli": "nx",
+      "version": "19.6.1-beta.0",
+      "description": "Ensure Target Defaults are set correctly for Module Federation.",
+      "factory": "./src/migrations/update-19-6-1/ensure-depends-on-for-mf"
     }
   },
   "packageJsonUpdates": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/react",
-  "version": "19.7.0-canary.20240816-ce2dff3",
+  "version": "19.7.0-canary.20240817-b91d788",
   "private": false,
   "description": "The React plugin for Nx contains executors and generators for managing React applications and libraries within an Nx workspace. It provides:\n\n\n- Integration with libraries such as Jest, Vitest, Playwright, Cypress, and Storybook.\n\n- Generators for applications, libraries, components, hooks, and more.\n\n- Library build support for publishing packages to npm or other registries.\n\n- Utilities for automatic workspace refactoring.",
   "repository": {
@@ -39,11 +39,11 @@
     "minimatch": "9.0.3",
     "tslib": "^2.3.0",
     "@module-federation/enhanced": "~0.2.3",
-    "@nx/devkit": "19.7.0-canary.20240816-ce2dff3",
-    "@nx/js": "19.7.0-canary.20240816-ce2dff3",
-    "@nx/eslint": "19.7.0-canary.20240816-ce2dff3",
-    "@nx/web": "19.7.0-canary.20240816-ce2dff3",
-    "@nrwl/react": "19.7.0-canary.20240816-ce2dff3"
+    "@nx/devkit": "19.7.0-canary.20240817-b91d788",
+    "@nx/js": "19.7.0-canary.20240817-b91d788",
+    "@nx/eslint": "19.7.0-canary.20240817-b91d788",
+    "@nx/web": "19.7.0-canary.20240817-b91d788",
+    "@nrwl/react": "19.7.0-canary.20240817-b91d788"
   },
   "publishConfig": {
     "access": "public"

package/src/generators/library/library.js

@@ -137,7 +137,11 @@ async function libraryGeneratorInternal(host, schema) {
                 "'react-dom'",
                 "'react/jsx-runtime'",
             ],
-            imports: [`import react from '@vitejs/plugin-react'`],
+            imports: [
+                options.compiler === 'swc'
+                    ? `import react from '@vitejs/plugin-react-swc'`
+                    : `import react from '@vitejs/plugin-react'`,
+            ],
             plugins: ['react()'],
         }, true);
     }

package/src/migrations/update-19-6-1/ensure-depends-on-for-mf.d.ts

@@ -0,0 +1,2 @@
+import { type Tree } from '@nx/devkit';
+export default function (tree: Tree): Promise<void>;

package/src/migrations/update-19-6-1/ensure-depends-on-for-mf.js

@@ -0,0 +1,46 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.default = default_1;
+const devkit_1 = require("@nx/devkit");
+const executor_options_utils_1 = require("@nx/devkit/src/generators/executor-options-utils");
+async function default_1(tree) {
+    let usesModuleFederation = false;
+    (0, executor_options_utils_1.forEachExecutorOptions)(tree, '@nx/webpack:webpack', (options, projectName, targetName) => {
+        const webpackConfig = options.webpackConfig;
+        if (!webpackConfig) {
+            return;
+        }
+        const webpackContents = tree.read(webpackConfig, 'utf-8');
+        if (['withModuleFederation', 'withModuleFederationForSSR'].some((p) => webpackContents.includes(p))) {
+            usesModuleFederation = true;
+        }
+    });
+    if (!usesModuleFederation) {
+        return;
+    }
+    const nxJson = (0, devkit_1.readNxJson)(tree);
+    const nxMFDevRemotesEnvVar = 'NX_MF_DEV_REMOTES';
+    if (!nxJson.targetDefaults ||
+        !nxJson.targetDefaults?.['@nx/webpack:webpack']) {
+        nxJson.targetDefaults ??= {};
+        nxJson.targetDefaults['@nx/webpack:webpack'] = {
+            cache: true,
+            inputs: ['production', '^production', { env: nxMFDevRemotesEnvVar }],
+            dependsOn: ['^build'],
+        };
+    }
+    else {
+        nxJson.targetDefaults['@nx/webpack:webpack'].dependsOn ??= [];
+        if (!nxJson.targetDefaults['@nx/webpack:webpack'].dependsOn.includes('^build')) {
+            nxJson.targetDefaults['@nx/webpack:webpack'].dependsOn.push('^build');
+        }
+        nxJson.targetDefaults['@nx/webpack:webpack'].inputs ??= [];
+        if (!nxJson.targetDefaults['@nx/webpack:webpack'].inputs.find((i) => typeof i === 'string' ? false : i['env'] === nxMFDevRemotesEnvVar)) {
+            nxJson.targetDefaults['@nx/webpack:webpack'].inputs.push({
+                env: nxMFDevRemotesEnvVar,
+            });
+        }
+    }
+    (0, devkit_1.updateNxJson)(tree, nxJson);
+    await (0, devkit_1.formatFiles)(tree);
+}

package/src/utils/add-mf-env-to-inputs.js

@@ -12,6 +12,7 @@ function addMfEnvToTargetDefaultInputs(tree) {
         'production',
         '^production',
     ];
+    nxJson.targetDefaults[webpackExecutor].dependsOn ??= ['^build'];
     let mfEnvVarExists = false;
     for (const input of nxJson.targetDefaults[webpackExecutor].inputs) {
         if (typeof input === 'object' && input['env'] === mfEnvVar) {