@nx/react 20.6.1 → 20.6.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/react",
-  "version": "20.6.1",
+  "version": "20.6.3",
   "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": {
@@ -38,11 +38,11 @@
     "minimatch": "9.0.3",
     "picocolors": "^1.1.0",
     "tslib": "^2.3.0",
-    "@nx/devkit": "20.6.1",
-    "@nx/js": "20.6.1",
-    "@nx/eslint": "20.6.1",
-    "@nx/web": "20.6.1",
-    "@nx/module-federation": "20.6.1",
+    "@nx/devkit": "20.6.3",
+    "@nx/js": "20.6.3",
+    "@nx/eslint": "20.6.3",
+    "@nx/web": "20.6.3",
+    "@nx/module-federation": "20.6.3",
     "express": "^4.21.2",
     "http-proxy-middleware": "^3.0.3",
     "semver": "^7.6.3"

package/src/generators/application/application.js

@@ -46,25 +46,26 @@ async function applicationGeneratorInternal(tree, schema) {
     });
     tasks.push(jsInitTask);
     const options = await (0, normalize_options_1.normalizeOptions)(tree, schema);
-    options.useReactRouter = options.routing
-        ? options.useReactRouter ??
-            (await (0, prompt_1.promptWhenInteractive)({
-                name: 'response',
-                message: 'Would you like to use react-router for server-side rendering?',
-                type: 'autocomplete',
-                choices: [
-                    {
-                        name: 'Yes',
-                        message: 'I want to use react-router   [ https://reactrouter.com/start/framework/routing   ]',
-                    },
-                    {
-                        name: 'No',
-                        message: 'I do not want to use react-router for server-side rendering',
-                    },
-                ],
-                initial: 0,
-            }, { response: 'No' }).then((r) => r.response === 'Yes'))
-        : false;
+    options.useReactRouter =
+        options.routing && options.bundler === 'vite'
+            ? options.useReactRouter ??
+                (await (0, prompt_1.promptWhenInteractive)({
+                    name: 'response',
+                    message: 'Would you like to use react-router for server-side rendering?',
+                    type: 'autocomplete',
+                    choices: [
+                        {
+                            name: 'Yes',
+                            message: 'I want to use react-router   [ https://reactrouter.com/start/framework/routing   ]',
+                        },
+                        {
+                            name: 'No',
+                            message: 'I do not want to use react-router for server-side rendering',
+                        },
+                    ],
+                    initial: 0,
+                }, { response: 'No' }).then((r) => r.response === 'Yes'))
+            : false;
     (0, show_possible_warnings_1.showPossibleWarnings)(tree, options);
     const initTask = await (0, init_1.default)(tree, {
         ...options,

package/src/generators/application/lib/normalize-options.js

@@ -31,7 +31,10 @@ async function normalizeOptions(host, options) {
         ? null
         : options.style;
     (0, assertion_1.assertValidStyle)(options.style);
-    options.bundler = options.useReactRouter ? 'vite' : options.bundler;
+    (0, assertion_1.assertValidReactRouter)(options.useReactRouter, options.bundler);
+    if (options.useReactRouter && !options.bundler) {
+        options.bundler = 'vite';
+    }
     options.useReactRouter = options.routing ? options.useReactRouter : false;
     const normalized = {
         ...options,

package/src/generators/library/library.js

@@ -190,7 +190,7 @@ async function libraryGeneratorInternal(host, schema) {
         });
         tasks.push(componentTask);
     }
-    if (options.publishable || options.buildable) {
+    if (options.publishable) {
         const projectConfiguration = (0, devkit_1.readProjectConfiguration)(host, options.name);
         if (options.isUsingTsSolutionConfig) {
             await (0, add_release_config_1.addReleaseConfigForTsSolution)(host, options.name, projectConfiguration);

package/src/utils/assertion.d.ts

@@ -1 +1,3 @@
+import { type Schema } from '../generators/application/schema';
 export declare function assertValidStyle(style: string): void;
+export declare function assertValidReactRouter(reactRouter: boolean, bundler: Schema['bundler']): void;

package/src/utils/assertion.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.assertValidStyle = assertValidStyle;
+exports.assertValidReactRouter = assertValidReactRouter;
 const VALID_STYLES = [
     'css',
     'scss',
@@ -16,3 +17,8 @@ function assertValidStyle(style) {
         throw new Error(`Unsupported style option found: ${style}. Valid values are: "${VALID_STYLES.join('", "')}"`);
     }
 }
+function assertValidReactRouter(reactRouter, bundler) {
+    if (reactRouter && bundler !== 'vite') {
+        throw new Error(`Unsupported bundler found: ${bundler}. React Router is only supported with 'vite'.`);
+    }
+}

package/src/utils/ct-utils.d.ts

@@ -10,5 +10,5 @@ export declare function getBundlerFromTarget(found: FoundTarget, tree: Tree): Pr
 export declare function getActualBundler(tree: Tree, options: {
     buildTarget?: string;
     bundler?: 'vite' | 'webpack';
-}, found: FoundTarget): Promise<"vite" | "webpack">;
+}, found: FoundTarget): Promise<"webpack" | "vite">;
 export declare function isComponent(tree: Tree, filePath: string): boolean;

package/src/utils/format-file.d.ts

@@ -1 +0,0 @@
-export declare function formatFile(content: any, ...values: any[]): string;

package/src/utils/format-file.js

@@ -1,11 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.formatFile = formatFile;
-const prettier_1 = require("prettier");
-const devkit_1 = require("@nx/devkit");
-function formatFile(content, ...values) {
-    return (0, prettier_1.format)((0, devkit_1.stripIndents)(content, values), {
-        singleQuote: true,
-        parser: 'typescript',
-    });
-}