@nx/remix 23.2.0-beta.1 → 23.2.0-beta.11

package/README.md

@@ -7,9 +7,9 @@
 
 <div style="text-align: center;">
 
-[![License](https://img.shields.io/npm/l/@nx/workspace.svg?style=flat-square)]()
+[![License](https://img.shields.io/npm/l/@nx/workspace.svg?style=flat-square)](<>)
 [![NPM Version](https://badge.fury.io/js/nx.svg)](https://www.npmjs.com/package/nx)
-[![Semantic Release](https://img.shields.io/badge/%20%20%F0%9F%93%A6%F0%9F%9A%80-semantic--release-e10079.svg?style=flat-square)]()
+[![Semantic Release](https://img.shields.io/badge/%20%20%F0%9F%93%A6%F0%9F%9A%80-semantic--release-e10079.svg?style=flat-square)](<>)
 [![Commitizen friendly](https://img.shields.io/badge/commitizen-friendly-brightgreen.svg)](http://commitizen.github.io/cz-cli/)
 [![Join us on the Official Nx Discord Server](https://img.shields.io/discord/1143497901675401286?label=discord)](https://go.nx.dev/community)
 [![Nx Sandboxing](https://staging.nx.app/workspaces/62d013ea0852fe0a2df74438/sandbox-badge.svg)](https://nx.dev/docs/features/ci-features/sandboxing)

package/dist/src/executors/build/build.impl.js

@@ -51,6 +51,7 @@ async function buildExecutor(options, context) {
     if (!outputIsDirectory) {
         (0, fs_extra_1.mkdir)(options.outputPath);
     }
+    const packageManager = (0, devkit_1.detectPackageManager)(context.root);
     let packageJson;
     if (options.generatePackageJson) {
         packageJson = (0, js_1.createPackageJson)(context.projectName, context.projectGraph, {
@@ -66,22 +67,19 @@ async function buildExecutor(options, context) {
             packageJson.scripts['start'] = 'remix-serve ./build/index.js';
         }
         updatePackageJson(packageJson, context);
-        (0, devkit_1.writeJsonFile)(`${options.outputPath}/package.json`, packageJson);
     }
     else {
         packageJson = (0, devkit_1.readJsonFile)((0, path_1.join)(projectRoot, 'package.json'));
     }
     if (options.generateLockfile) {
-        const packageManager = (0, devkit_1.detectPackageManager)(context.root);
-        if (packageManager === 'bun') {
-            devkit_1.logger.warn('Bun lockfile generation is not supported. The generated package.json will not include a lockfile. Run "bun install" in the output directory after deployment if needed.');
-        }
-        else {
-            const lockFile = (0, js_1.createLockFile)(packageJson, context.projectGraph, packageManager);
-            (0, fs_extra_1.writeFileSync)(`${options.outputPath}/${(0, js_1.getLockFileName)(packageManager)}`, lockFile, {
-                encoding: 'utf-8',
-            });
-        }
+        (0, js_1.generatePrunedDeployOutput)(packageJson, context.projectGraph, projectRoot, {
+            outputDirectory: options.outputPath,
+            packageManager,
+            workspaceRoot: context.root,
+        });
+    }
+    if (options.generatePackageJson) {
+        (0, devkit_1.writeJsonFile)(`${options.outputPath}/package.json`, packageJson);
     }
     // If output path is different from source path, then copy over the config and public files.
     // This is the default behavior when running `nx build <app>`.

package/dist/src/generators/application/application.impl.js

@@ -6,13 +6,12 @@ const tslib_1 = require("tslib");
 const internal_1 = require("@nx/devkit/internal");
 const devkit_1 = require("@nx/devkit");
 const js_1 = require("@nx/js");
-const onboarding_1 = require("nx/src/nx-cloud/utilities/onboarding");
+const internal_2 = require("@nx/js/internal");
 const testing_config_utils_1 = require("../../utils/testing-config-utils");
 const versions_1 = require("../../utils/versions");
 const init_1 = tslib_1.__importDefault(require("../init/init"));
 const update_dependencies_1 = require("../utils/update-dependencies");
 const lib_1 = require("./lib");
-const internal_2 = require("@nx/js/internal");
 function remixApplicationGenerator(tree, options) {
     return remixApplicationGeneratorInternal(tree, {
         addPlugin: true,
@@ -56,9 +55,9 @@ async function remixApplicationGeneratorInternal(tree, _options) {
     }
     const installTask = (0, update_dependencies_1.updateDependencies)(tree);
     tasks.push(installTask);
-    const onBoardingStatus = await (0, onboarding_1.createNxCloudOnboardingURLForWelcomeApp)(tree, options.nxCloudToken);
+    const onBoardingStatus = await (0, internal_1.createNxCloudOnboardingURLForWelcomeApp)(tree, options.nxCloudToken);
     const connectCloudUrl = onBoardingStatus === 'unclaimed' &&
-        (await (0, onboarding_1.getNxCloudAppOnBoardingUrl)(options.nxCloudToken));
+        (await (0, internal_1.getNxCloudAppOnBoardingUrl)(options.nxCloudToken));
     const vars = {
         ...options,
         tmpl: '',
@@ -148,7 +147,16 @@ async function remixApplicationGeneratorInternal(tree, _options) {
     else {
         tree.delete((0, devkit_1.joinPathFragments)(options.projectRoot, `tests/routes/_index.spec.tsx`));
     }
-    if (options.linter !== 'none') {
+    if (options.linter !== 'eslint' && options.linter !== 'none') {
+        tasks.push(await (0, internal_2.addLintingToProject)(tree, {
+            oxlintPlugins: ['react', 'react-perf', 'jsx-a11y'],
+            unitTestRunner: options.unitTestRunner,
+            linter: options.linter,
+            project: options.projectName,
+            addPlugin: options.addPlugin,
+        }));
+    }
+    if (options.linter === 'eslint') {
         const { lintProjectGenerator } = (0, devkit_1.ensurePackage)('@nx/eslint', versions_1.nxVersion);
         const { addIgnoresToLintConfig, } = require('@nx/eslint/internal');
         const eslintTask = await lintProjectGenerator(tree, {

package/dist/src/generators/application/files/common/app/entry.server.tsx__tmpl__

@@ -21,8 +21,7 @@ export default function handleRequest(
   remixContext: EntryContext,
   // This is ignored so we can keep it in the template for visibility.  Feel
   // free to delete this parameter in your app if you're not using it!
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  loadContext: AppLoadContext
+  _loadContext: AppLoadContext
 ) {
   return isbot(request.headers.get("user-agent") || "")
     ? handleBotRequest(

package/dist/src/generators/application/files/common/tests/routes/_index.spec.tsx__tmpl__

@@ -13,5 +13,5 @@ test('renders loader data', async () => {
 
   render(<RemixStub />);
 
-  await screen.findByText('Hello there,');
+  expect(await screen.findByText('Hello there,')).toBeTruthy();
 });

package/dist/src/generators/application/files/nx-welcome/claimed/app/nx-welcome.tsx__tmpl__

@@ -5,6 +5,8 @@
  Delete this file and get started with your project!
  * * * * * * * * * * * * * * * * * * * * * * * * * * * *
  */
+/* oxlint-disable jsx-a11y/prefer-tag-over-role -- the icons below
+   are inline SVGs, which have no tag to swap the role for. */
 export function NxWelcome({ title }: { title: string }) {
   return (
     <>

package/dist/src/generators/application/files/nx-welcome/not-configured/app/nx-welcome.tsx__tmpl__

@@ -5,6 +5,8 @@
  Delete this file and get started with your project!
  * * * * * * * * * * * * * * * * * * * * * * * * * * * *
  */
+/* oxlint-disable jsx-a11y/prefer-tag-over-role -- the icons below
+   are inline SVGs, which have no tag to swap the role for. */
 export function NxWelcome({ title }: { title: string }) {
   return (
     <>

package/dist/src/generators/application/files/nx-welcome/unclaimed/app/nx-welcome.tsx__tmpl__

@@ -5,6 +5,8 @@
  Delete this file and get started with your project!
  * * * * * * * * * * * * * * * * * * * * * * * * * * * *
  */
+/* oxlint-disable jsx-a11y/prefer-tag-over-role -- the icons below
+   are inline SVGs, which have no tag to swap the role for. */
  export function NxWelcome({ title }: { title: string }) {
   return (
     <>

package/dist/src/generators/application/lib/add-e2e.js

@@ -82,7 +82,7 @@ async function addE2E(tree, options) {
             directory: 'src',
             js: false,
             linter: options.linter,
-            setParserOptionsProject: false,
+            enableTypedLinting: false,
             webServerCommand: e2eWebsServerInfo.e2eCiWebServerCommand,
             webServerAddress: e2eWebsServerInfo.e2eCiBaseUrl,
             rootProject: options.rootProject,

package/dist/src/generators/application/lib/ignore-vite-temp-files.d.ts

@@ -1,3 +1,2 @@
 import { type Tree } from '@nx/devkit';
 export declare function ignoreViteTempFiles(tree: Tree, projectRoot?: string | undefined): Promise<void>;
-export declare function isEslintInstalled(tree: Tree): boolean;

package/dist/src/generators/application/lib/ignore-vite-temp-files.js

@@ -1,8 +1,8 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ignoreViteTempFiles = ignoreViteTempFiles;
-exports.isEslintInstalled = isEslintInstalled;
 const devkit_1 = require("@nx/devkit");
+const internal_1 = require("@nx/js/internal");
 const versions_1 = require("../../../utils/versions");
 async function ignoreViteTempFiles(tree, projectRoot) {
     addViteTempFilesToGitIgnore(tree);
@@ -23,7 +23,9 @@ function addViteTempFilesToGitIgnore(tree) {
     tree.write('.gitignore', gitIgnoreContents);
 }
 async function ignoreViteTempFilesInEslintConfig(tree, projectRoot) {
-    if (!isEslintInstalled(tree)) {
+    // Checked before `ensurePackage` so an Oxlint workspace does not install
+    // `@nx/eslint` only for `isEslintConfigSupported` to send it straight back.
+    if (!(0, internal_1.detectLinters)(tree).includes('eslint')) {
         return;
     }
     (0, devkit_1.ensurePackage)('@nx/eslint', versions_1.nxVersion);
@@ -46,15 +48,3 @@ async function ignoreViteTempFilesInEslintConfig(tree, projectRoot) {
         '**/vitest.config.*.timestamp*',
     ]);
 }
-function isEslintInstalled(tree) {
-    try {
-        require('eslint');
-        return true;
-    }
-    catch { }
-    // it might not be installed yet, but it might be in the tree pending install
-    const { devDependencies, dependencies } = tree.exists('package.json')
-        ? (0, devkit_1.readJson)(tree, 'package.json')
-        : {};
-    return !!devDependencies?.['eslint'] || !!dependencies?.['eslint'];
-}

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

@@ -26,7 +26,7 @@ async function normalizeOptions(tree, options) {
         : [];
     return {
         ...options,
-        linter: options.linter ?? 'eslint',
+        linter: await (0, internal_2.normalizeLinterOption)(tree, options.linter),
         projectName: appProjectName,
         projectRoot,
         importPath,

package/dist/src/generators/application/schema.d.ts

@@ -1,10 +1,10 @@
-import type { Linter, LinterType } from '@nx/eslint';
+import type { LinterType } from '@nx/js';
 
 export interface NxRemixGeneratorSchema {
   directory: string;
   name?: string;
   tags?: string;
-  linter?: Linter | LinterType;
+  linter?: LinterType;
   unitTestRunner?: 'vitest' | 'jest' | 'none';
   e2eTestRunner?: 'cypress' | 'playwright' | 'none';
   skipFormat?: boolean;
@@ -13,6 +13,6 @@ export interface NxRemixGeneratorSchema {
   addPlugin?: boolean;
   nxCloudToken?: string;
   useTsSolution?: boolean;
-  formatter?: 'prettier' | 'none';
+  formatter?: 'prettier' | 'oxfmt' | 'none';
   useProjectJson?: boolean;
 }

package/dist/src/generators/application/schema.json

@@ -21,13 +21,16 @@
       "x-priority": "important"
     },
     "linter": {
-      "description": "The tool to use for running lint checks.",
+      "description": "The tool to use for running lint checks. Defaults to the linter the workspace already uses.",
       "type": "string",
-      "enum": ["eslint", "none"],
-      "default": "none",
-      "x-prompt": "Which linter would you like to use?",
+      "enum": ["eslint", "oxlint", "none"],
       "x-priority": "important"
     },
+    "formatter": {
+      "description": "The tool to use for code formatting.",
+      "type": "string",
+      "enum": ["none", "prettier", "oxfmt"]
+    },
     "unitTestRunner": {
       "type": "string",
       "enum": ["vitest", "jest", "none"],

package/dist/src/generators/library/lib/normalize-options.js

@@ -18,6 +18,9 @@ async function normalizeOptions(tree, options) {
     const isUsingTsSolutionConfig = (0, internal_2.isUsingTsSolutionSetup)(tree);
     return {
         ...options,
+        // Resolved after the spread: the framework's ESLint shaping is guarded on
+        // `=== 'eslint'`, and an unresolved `undefined` would skip all of it.
+        linter: await (0, internal_2.normalizeLinterOption)(tree, options.linter),
         unitTestRunner: options.unitTestRunner ?? 'vitest',
         projectName: isUsingTsSolutionConfig && !options.name ? importPath : projectName,
         projectRoot,

package/dist/src/generators/library/library.impl.js

@@ -22,6 +22,9 @@ async function remixLibraryGeneratorInternal(tree, schema) {
     const tasks = [];
     tasks.push((0, assert_and_pin_remix_typescript_1.assertAndPinRemixTypescript)(tree));
     const addTsPlugin = (0, internal_1.shouldConfigureTsSolutionSetup)(tree, schema.addPlugin);
+    // Detected before `updateDependencies`, which installs eslint unconditionally.
+    // Detecting afterwards would just observe what this generator itself added.
+    const linter = await (0, internal_1.normalizeLinterOption)(tree, schema.linter);
     const installTask = (0, update_dependencies_1.updateDependencies)(tree);
     tasks.push(installTask);
     const jsInitTask = await (0, js_1.initGenerator)(tree, {
@@ -30,7 +33,7 @@ async function remixLibraryGeneratorInternal(tree, schema) {
         skipFormat: true,
     });
     tasks.push(jsInitTask);
-    const options = await (0, lib_1.normalizeOptions)(tree, schema);
+    const options = await (0, lib_1.normalizeOptions)(tree, { ...schema, linter });
     if (options.isUsingTsSolutionConfig) {
         await (0, internal_1.addProjectToTsSolutionWorkspace)(tree, options.projectRoot);
     }

package/dist/src/generators/library/schema.d.ts

@@ -1,3 +1,4 @@
+import type { LinterType } from '@nx/js';
 import { SupportedStyles } from '@nx/react';
 
 export interface NxRemixGeneratorSchema {
@@ -9,7 +10,7 @@ export interface NxRemixGeneratorSchema {
   /** @deprecated Use bundler instead. */
   buildable?: boolean;
   bundler?: 'none' | 'vite' | 'rollup';
-  linter?: 'none' | 'eslint';
+  linter?: LinterType;
   unitTestRunner?: 'jest' | 'vitest' | 'none';
   js?: boolean;
   skipFormat?: boolean;

package/dist/src/generators/library/schema.json

@@ -46,11 +46,9 @@
       "x-priority": "important"
     },
     "linter": {
-      "description": "The tool to use for running lint checks.",
+      "description": "The tool to use for running lint checks. Defaults to the linter the workspace already uses.",
       "type": "string",
-      "enum": ["eslint", "none"],
-      "default": "none",
-      "x-prompt": "Which linter would you like to use?",
+      "enum": ["eslint", "oxlint", "none"],
       "x-priority": "important"
     },
     "unitTestRunner": {

package/dist/src/generators/preset/preset.impl.js

@@ -3,7 +3,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.default = default_1;
 const tslib_1 = require("tslib");
 const devkit_1 = require("@nx/devkit");
-const devkit_2 = require("@nx/devkit");
 const application_impl_1 = tslib_1.__importDefault(require("../application/application.impl"));
 const setup_impl_1 = tslib_1.__importDefault(require("../setup/setup.impl"));
 const normalize_options_1 = require("./lib/normalize-options");
@@ -31,5 +30,5 @@ async function default_1(tree, _options) {
     tree.delete('apps');
     tree.delete('libs');
     await (0, devkit_1.formatFiles)(tree);
-    return (0, devkit_2.runTasksInSerial)(...tasks);
+    return (0, devkit_1.runTasksInSerial)(...tasks);
 }

package/dist/src/generators/storybook-configuration/schema.d.ts

@@ -1,4 +1,4 @@
-import { Linter, LinterType } from '@nx/eslint';
+import { LinterType } from '@nx/js';
 
 export interface StorybookConfigurationSchema {
   project: string;
@@ -6,7 +6,7 @@ export interface StorybookConfigurationSchema {
   generateStories?: boolean;
   js?: boolean;
   tsConfiguration?: boolean;
-  linter?: Linter | LinterType;
+  linter?: LinterType;
   ignorePaths?: string[];
   configureTestRunner?: boolean;
   configureStaticServe?: boolean;

package/dist/src/generators/storybook-configuration/schema.json

@@ -53,8 +53,7 @@
     "linter": {
       "description": "The tool to use for running lint checks.",
       "type": "string",
-      "enum": ["eslint"],
-      "default": "eslint"
+      "enum": ["eslint", "oxlint"]
     },
     "ignorePaths": {
       "type": "array",

package/dist/src/plugins/plugin.js

@@ -2,8 +2,6 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.createNodesV2 = exports.createNodes = exports.createDependencies = void 0;
 const internal_1 = require("@nx/devkit/internal");
-const cache_directory_1 = require("nx/src/utils/cache-directory");
-const file_hasher_1 = require("nx/src/hasher/file-hasher");
 const devkit_1 = require("@nx/devkit");
 const js_1 = require("@nx/js");
 const path_1 = require("path");
@@ -21,8 +19,8 @@ const remixConfigGlob = '**/{remix,vite}.config.{js,cjs,mjs,ts,cts,mts}';
 exports.createNodes = [
     remixConfigGlob,
     async (configFilePaths, options, context) => {
-        const optionsHash = (0, file_hasher_1.hashObject)(options);
-        const cachePath = (0, path_1.join)(cache_directory_1.workspaceDataDirectory, `remix-${optionsHash}.hash`);
+        const optionsHash = (0, internal_1.hashObject)(options);
+        const cachePath = (0, path_1.join)(internal_1.workspaceDataDirectory, `remix-${optionsHash}.hash`);
         const targetsCache = new internal_1.PluginCache(cachePath);
         const packageManager = (0, devkit_1.detectPackageManager)(context.workspaceRoot);
         const pmc = (0, devkit_1.getPackageManagerCommand)(packageManager);

package/dist/src/utils/create-watch-paths.js

@@ -3,9 +3,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.createWatchPaths = createWatchPaths;
 exports.getRelativeDependencyPaths = getRelativeDependencyPaths;
 const devkit_1 = require("@nx/devkit");
-const find_project_for_path_1 = require("nx/src/project-graph/utils/find-project-for-path");
-const project_graph_utils_1 = require("nx/src/utils/project-graph-utils");
 const path_1 = require("path");
+const internal_1 = require("@nx/devkit/internal");
 /**
  * Generates an array of paths to watch based on the project dependencies.
  *
@@ -13,9 +12,9 @@ const path_1 = require("path");
  */
 async function createWatchPaths(dirname) {
     const graph = await (0, devkit_1.createProjectGraphAsync)();
-    const projectRootMappings = (0, find_project_for_path_1.createProjectRootMappings)(graph.nodes);
-    const projectName = (0, find_project_for_path_1.findProjectForPath)((0, path_1.relative)(devkit_1.workspaceRoot, dirname), projectRootMappings);
-    const deps = (0, project_graph_utils_1.findAllProjectNodeDependencies)(projectName, graph);
+    const projectRootMappings = (0, internal_1.createProjectRootMappings)(graph.nodes);
+    const projectName = (0, internal_1.findProjectForPath)((0, path_1.relative)(devkit_1.workspaceRoot, dirname), projectRootMappings);
+    const deps = (0, internal_1.findAllProjectNodeDependencies)(projectName, graph);
     return getRelativeDependencyPaths(graph.nodes[projectName], deps, graph);
 }
 // Exported for testing

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/remix",
-  "version": "23.2.0-beta.1",
+  "version": "23.2.0-beta.11",
   "description": "The Remix plugin for Nx contains executors and generators for managing Remix applications and libraries within an Nx workspace. It provides:\n\n\n- Integration with libraries such as Vitest, Jest, Playwright, Cypress, and Storybook.\n\n- Generators for applications, libraries, routes, loaders, and more.\n\n- Library build support for publishing packages to npm or other registries.\n\n- Utilities for automatic workspace refactoring.",
   "repository": {
     "type": "git",
@@ -56,20 +56,28 @@
   "dependencies": {
     "tslib": "^2.3.0",
     "@phenomnomnominal/tsquery": "~6.2.0",
-    "@nx/devkit": "23.2.0-beta.1",
-    "@nx/react": "23.2.0-beta.1",
-    "@nx/web": "23.2.0-beta.1",
-    "@nx/js": "23.2.0-beta.1"
+    "@nx/devkit": "23.2.0-beta.11",
+    "@nx/js": "23.2.0-beta.11",
+    "@nx/react": "23.2.0-beta.11",
+    "@nx/web": "23.2.0-beta.11"
   },
   "devDependencies": {
-    "@nx/eslint": "23.2.0-beta.1",
-    "nx": "23.2.0-beta.1",
-    "@nx/vitest": "23.2.0-beta.1"
+    "@nx/eslint": "23.2.0-beta.11",
+    "nx": "23.2.0-beta.11",
+    "@nx/vitest": "23.2.0-beta.11"
   },
   "peerDependencies": {
-    "@remix-run/dev": "^2.0.0"
+    "@remix-run/dev": "^2.0.0",
+    "@nx/cypress": "23.2.0-beta.11",
+    "@nx/playwright": "23.2.0-beta.11"
   },
   "peerDependenciesMeta": {
+    "@nx/cypress": {
+      "optional": true
+    },
+    "@nx/playwright": {
+      "optional": true
+    },
     "@remix-run/dev": {
       "optional": true
     }