nx 19.1.0-beta.0 → 19.1.0-beta.2

package/.eslintrc.json

@@ -100,7 +100,8 @@
               "events", // This is coming from @storybook/builder-manager since it uses the browser polyfill
               "process", // This is coming from @storybook/builder-manager since it uses the browser polyfill
               "prettier", // This is coming from @storybook/builder-manager since it uses the browser polyfill
-              "util" // This is coming from @storybook/builder-manager since it uses the browser polyfill
+              "util", // This is coming from @storybook/builder-manager since it uses the browser polyfill
+              "js-yaml" // The rule is failing with alias dependencies
             ]
           }
         ]

package/bin/init-local.js

@@ -132,7 +132,7 @@ function handleAngularCLIFallbacks(workspace) {
     else if (process.argv[2] === 'cache') {
         console.log(`"ng cache" is not natively supported by Nx.
 To clear the cache, you can delete the ".angular/cache" directory (or the directory configured by "cli.cache.path" in the "nx.json" file).
-To update the cache configuration, you can directly update the relevant options in your "nx.json" file (https://angular.io/guide/workspace-config#cache-options).`);
+To update the cache configuration, you can directly update the relevant options in your "nx.json" file (https://angular.dev/reference/configs/workspace-config#cache-options).`);
     }
     else {
         try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nx",
-  "version": "19.1.0-beta.0",
+  "version": "19.1.0-beta.2",
   "private": false,
   "description": "The core Nx plugin contains the core functionality of Nx like the project graph, nx commands and task orchestration.",
   "repository": {
@@ -20,7 +20,11 @@
     "Nest",
     "Jest",
     "Cypress",
-    "CLI"
+    "CLI",
+    "Testing",
+    "Front-end",
+    "Backend",
+    "Mobile"
   ],
   "bin": {
     "nx": "./bin/nx.js",
@@ -35,7 +39,6 @@
   "dependencies": {
     "@yarnpkg/lockfile": "^1.1.0",
     "@yarnpkg/parsers": "3.0.0-rc.46",
-    "@zkochan/js-yaml": "0.0.6",
     "axios": "^1.6.0",
     "chalk": "^4.1.0",
     "cli-cursor": "3.1.0",
@@ -49,7 +52,7 @@
     "fs-extra": "^11.1.0",
     "ignore": "^5.0.4",
     "jest-diff": "^29.4.1",
-    "js-yaml": "4.1.0",
+    "js-yaml": "npm:@zkochan/js-yaml@0.0.7",
     "jsonc-parser": "3.2.0",
     "lines-and-columns": "~2.0.3",
     "minimatch": "9.0.3",
@@ -66,7 +69,7 @@
     "yargs-parser": "21.1.1",
     "node-machine-id": "1.1.12",
     "ora": "5.3.0",
-    "@nrwl/tao": "19.1.0-beta.0"
+    "@nrwl/tao": "19.1.0-beta.2"
   },
   "peerDependencies": {
     "@swc-node/register": "^1.8.0",
@@ -81,16 +84,16 @@
     }
   },
   "optionalDependencies": {
-    "@nx/nx-darwin-x64": "19.1.0-beta.0",
-    "@nx/nx-darwin-arm64": "19.1.0-beta.0",
-    "@nx/nx-linux-x64-gnu": "19.1.0-beta.0",
-    "@nx/nx-linux-x64-musl": "19.1.0-beta.0",
-    "@nx/nx-win32-x64-msvc": "19.1.0-beta.0",
-    "@nx/nx-linux-arm64-gnu": "19.1.0-beta.0",
-    "@nx/nx-linux-arm64-musl": "19.1.0-beta.0",
-    "@nx/nx-linux-arm-gnueabihf": "19.1.0-beta.0",
-    "@nx/nx-win32-arm64-msvc": "19.1.0-beta.0",
-    "@nx/nx-freebsd-x64": "19.1.0-beta.0"
+    "@nx/nx-darwin-x64": "19.1.0-beta.2",
+    "@nx/nx-darwin-arm64": "19.1.0-beta.2",
+    "@nx/nx-linux-x64-gnu": "19.1.0-beta.2",
+    "@nx/nx-linux-x64-musl": "19.1.0-beta.2",
+    "@nx/nx-win32-x64-msvc": "19.1.0-beta.2",
+    "@nx/nx-linux-arm64-gnu": "19.1.0-beta.2",
+    "@nx/nx-linux-arm64-musl": "19.1.0-beta.2",
+    "@nx/nx-linux-arm-gnueabihf": "19.1.0-beta.2",
+    "@nx/nx-win32-arm64-msvc": "19.1.0-beta.2",
+    "@nx/nx-freebsd-x64": "19.1.0-beta.2"
   },
   "nx-migrations": {
     "migrations": "./migrations.json",

package/schemas/nx-schema.json

@@ -366,7 +366,7 @@
         "packageManager": {
           "type": "string",
           "description": "The default package manager to use.",
-          "enum": ["yarn", "pnpm", "npm"]
+          "enum": ["yarn", "pnpm", "npm", "bun"]
         }
       }
     },

package/src/adapter/ngcli-adapter.js

@@ -791,12 +791,10 @@ async function getWrappedWorkspaceNodeModulesArchitectHost(workspace, root, proj
             const [packageName, builderName] = builderStr.split(':');
             const { executorsFilePath, executorConfig } = this.readExecutorsJson(packageName, builderName);
             const builderInfo = this.readExecutor(packageName, builderName);
-            const { builders, executors } = (0, fileutils_1.readJsonFile)(executorsFilePath);
             return {
                 name: builderStr,
                 builderName,
-                description: builders?.[builderName]?.description ??
-                    executors?.[builderName]?.description,
+                description: executorConfig.description,
                 optionSchema: builderInfo.schema,
                 import: (0, schema_utils_1.resolveImplementation)(executorConfig.implementation, (0, path_1.dirname)(executorsFilePath)),
             };
@@ -813,6 +811,11 @@ async function getWrappedWorkspaceNodeModulesArchitectHost(workspace, root, proj
             if (!executorConfig) {
                 throw new Error(`Cannot find builder '${builder}' in ${executorsFilePath}.`);
             }
+            if (typeof executorConfig === 'string') {
+                // Angular CLI can have a builder pointing to another package:builder
+                const [packageName, executorName] = executorConfig.split(':');
+                return this.readExecutorsJson(packageName, executorName);
+            }
             return { executorsFilePath, executorConfig, isNgCompat: true };
         }
         readExecutor(nodeModule, executor) {

package/src/command-line/format/format.js

@@ -1,23 +1,23 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.format = void 0;
-const child_process_1 = require("child_process");
-const path = require("path");
+const node_child_process_1 = require("node:child_process");
+const path = require("node:path");
+const file_utils_1 = require("../../project-graph/file-utils");
 const command_line_utils_1 = require("../../utils/command-line-utils");
-const ignore_1 = require("../../utils/ignore");
 const fileutils_1 = require("../../utils/fileutils");
-const file_utils_1 = require("../../project-graph/file-utils");
+const ignore_1 = require("../../utils/ignore");
 const prettier = require("prettier");
-const object_sort_1 = require("../../utils/object-sort");
-const package_json_1 = require("../../utils/package-json");
+const configuration_1 = require("../../config/configuration");
 const typescript_1 = require("../../plugins/js/utils/typescript");
-const project_graph_1 = require("../../project-graph/project-graph");
 const affected_project_graph_1 = require("../../project-graph/affected/affected-project-graph");
-const configuration_1 = require("../../config/configuration");
-const chunkify_1 = require("../../utils/chunkify");
+const project_graph_1 = require("../../project-graph/project-graph");
 const all_file_data_1 = require("../../utils/all-file-data");
-const workspace_root_1 = require("../../utils/workspace-root");
+const chunkify_1 = require("../../utils/chunkify");
+const object_sort_1 = require("../../utils/object-sort");
 const output_1 = require("../../utils/output");
+const package_json_1 = require("../../utils/package-json");
+const workspace_root_1 = require("../../utils/workspace-root");
 const PRETTIER_PATH = getPrettierPath();
 async function format(command, args) {
     const { nxArgs } = (0, command_line_utils_1.splitArgsIntoNxArgsAndOverrides)(args, 'affected', { printWarnings: false }, (0, configuration_1.readNxJson)());
@@ -33,12 +33,31 @@ async function format(command, args) {
             addRootConfigFiles(chunkList, nxArgs);
             chunkList.forEach((chunk) => write(chunk));
             break;
-        case 'check':
-            const pass = chunkList.reduce((pass, chunk) => check(chunk) && pass, true);
-            if (!pass) {
+        case 'check': {
+            const filesWithDifferentFormatting = [];
+            for (const chunk of chunkList) {
+                const files = await check(chunk);
+                filesWithDifferentFormatting.push(...files);
+            }
+            if (filesWithDifferentFormatting.length > 0) {
+                if (nxArgs.verbose) {
+                    output_1.output.error({
+                        title: 'The following files are not formatted correctly based on your Prettier configuration',
+                        bodyLines: [
+                            '- Run "nx format:write" and commit the resulting diff to fix these files.',
+                            '- Please note, Prettier does not support a native way to diff the output of its check logic (https://github.com/prettier/prettier/issues/6885).',
+                            '',
+                            ...filesWithDifferentFormatting,
+                        ],
+                    });
+                }
+                else {
+                    console.log(filesWithDifferentFormatting.join('\n'));
+                }
                 process.exit(1);
             }
             break;
+        }
     }
 }
 exports.format = format;
@@ -113,29 +132,32 @@ function write(patterns) {
             result[pattern.includes('.swcrc') ? 0 : 1].push(pattern);
             return result;
         }, [[], []]);
-        (0, child_process_1.execSync)(`node "${PRETTIER_PATH}" --write --list-different ${regularPatterns.join(' ')}`, {
+        (0, node_child_process_1.execSync)(`node "${PRETTIER_PATH}" --write --list-different ${regularPatterns.join(' ')}`, {
             stdio: [0, 1, 2],
         });
         if (swcrcPatterns.length > 0) {
-            (0, child_process_1.execSync)(`node "${PRETTIER_PATH}" --write --list-different ${swcrcPatterns.join(' ')} --parser json`, {
+            (0, node_child_process_1.execSync)(`node "${PRETTIER_PATH}" --write --list-different ${swcrcPatterns.join(' ')} --parser json`, {
                 stdio: [0, 1, 2],
             });
         }
     }
 }
-function check(patterns) {
+async function check(patterns) {
     if (patterns.length === 0) {
-        return true;
+        return [];
     }
-    try {
-        (0, child_process_1.execSync)(`node "${PRETTIER_PATH}" --list-different ${patterns.join(' ')}`, {
-            stdio: [0, 1, 2],
+    return new Promise((resolve) => {
+        (0, node_child_process_1.exec)(`node "${PRETTIER_PATH}" --list-different ${patterns.join(' ')}`, { encoding: 'utf-8' }, (error, stdout) => {
+            if (error) {
+                // The command failed so there are files with different formatting. Prettier writes them to stdout, newline separated.
+                resolve(stdout.trim().split('\n'));
+            }
+            else {
+                // The command succeeded so there are no files with different formatting
+                resolve([]);
+            }
         });
-        return true;
-    }
-    catch {
-        return false;
-    }
+    });
 }
 function sortTsConfig() {
     try {

package/src/command-line/init/implementation/angular/legacy-angular-versions.js

@@ -15,9 +15,10 @@ const connect_to_nx_cloud_1 = require("../../../connect/connect-to-nx-cloud");
 // key is major Angular version and value is Nx version to use
 const nxAngularLegacyVersionMap = {
     14: '~17.0.0',
+    15: '~19.0.0',
 };
 // min major angular version supported in latest Nx
-const minMajorAngularVersionSupported = 15;
+const minMajorAngularVersionSupported = 16;
 // version when the Nx CLI changed from @nrwl/tao & @nrwl/cli to nx
 const versionWithConsolidatedPackages = '13.9.0';
 // version when packages were rescoped from @nrwl/* to @nx/*

package/src/command-line/init/implementation/angular/standalone-workspace.js

@@ -98,8 +98,6 @@ function createNxJson(repoRoot, angularJson, cacheableOperations, { eslintProjec
             inputs: ['default', '^production'],
         };
     }
-    // Angular 15 workspaces still support defaultProject. Support was removed in Angular 16.
-    nxJson.defaultProject = angularJson.defaultProject;
     (0, fileutils_1.writeJsonFile)((0, path_1.join)(repoRoot, 'nx.json'), nxJson);
 }
 function updateProjectOutputs(repoRoot, project) {

package/src/command-line/init/implementation/react/index.js

@@ -155,6 +155,7 @@ function moveFilesToTempWorkspace(options) {
         options.packageManager === 'yarn' ? 'yarn.lock' : null,
         options.packageManager === 'pnpm' ? 'pnpm-lock.yaml' : null,
         options.packageManager === 'npm' ? 'package-lock.json' : null,
+        options.packageManager === 'bun' ? 'bun.lockb' : null,
     ];
     const optionalCraFiles = ['README.md'];
     const filesToMove = [...requiredCraFiles, ...optionalCraFiles].filter(Boolean);

package/src/command-line/migrate/migrate.js

@@ -377,9 +377,9 @@ const LEGACY_NRWL_PACKAGE_GROUP = [
 ];
 async function normalizeVersionWithTagCheck(pkg, version) {
     // This doesn't seem like a valid version, lets check if its a tag on the registry.
-    if (version && !(0, semver_1.coerce)(version)) {
+    if (version && !(0, semver_1.parse)(version)) {
         try {
-            return (0, package_manager_1.packageRegistryView)(pkg, version, 'version');
+            return (0, package_manager_1.resolvePackageVersionUsingRegistry)(pkg, version);
         }
         catch {
             // fall through to old logic
@@ -634,6 +634,9 @@ async function getPackageMigrationsUsingInstall(packageName, packageVersion) {
         }
         result = { ...migrations, packageGroup, version: packageJson.version };
     }
+    catch (e) {
+        logger_1.logger.warn(`Unable to fetch migrations for ${packageName}@${packageVersion}: ${e.message}`);
+    }
     finally {
         await cleanup();
     }
@@ -871,7 +874,14 @@ function runInstall() {
     (0, child_process_1.execSync)(pmCommands.install, { stdio: [0, 1, 2] });
 }
 async function executeMigrations(root, migrations, isVerbose, shouldCreateCommits, commitPrefix) {
-    const depsBeforeMigrations = getStringifiedPackageJsonDeps(root);
+    let initialDeps = getStringifiedPackageJsonDeps(root);
+    const installDepsIfChanged = () => {
+        const currentDeps = getStringifiedPackageJsonDeps(root);
+        if (initialDeps !== currentDeps) {
+            runInstall();
+        }
+        initialDeps = currentDeps;
+    };
     const migrationsWithNoChanges = [];
     const sortedMigrations = migrations.sort((a, b) => {
         // special case for the split configuration migration to run first
@@ -912,6 +922,7 @@ async function executeMigrations(root, migrations, isVerbose, shouldCreateCommit
                 loggingQueue.forEach((log) => logger_1.logger.info('  ' + log));
             }
             if (shouldCreateCommits) {
+                installDepsIfChanged();
                 const commitMessage = `${commitPrefix}${m.name}`;
                 try {
                     const committedSha = commitChanges(commitMessage);
@@ -935,9 +946,8 @@ async function executeMigrations(root, migrations, isVerbose, shouldCreateCommit
             throw e;
         }
     }
-    const depsAfterMigrations = getStringifiedPackageJsonDeps(root);
-    if (depsBeforeMigrations !== depsAfterMigrations) {
-        runInstall();
+    if (!shouldCreateCommits) {
+        installDepsIfChanged();
     }
     return migrationsWithNoChanges;
 }

package/src/command-line/release/utils/github.js

@@ -223,7 +223,7 @@ async function resolveGithubToken() {
     const ghCLIPath = (0, path_1.joinPathFragments)(process.env.XDG_CONFIG_HOME || (0, path_1.joinPathFragments)((0, node_os_1.homedir)(), '.config'), 'gh', 'hosts.yml');
     if ((0, node_fs_1.existsSync)(ghCLIPath)) {
         const yamlContents = await node_fs_1.promises.readFile(ghCLIPath, 'utf8');
-        const { load } = require('@zkochan/js-yaml');
+        const { load } = require('js-yaml');
         const ghCLIConfig = load(yamlContents);
         if (ghCLIConfig['github.com']) {
             // Web based session (the token is already embedded in the config)

package/src/command-line/release/utils/shared.js

@@ -162,9 +162,11 @@ function createGitTagValues(releaseGroups, releaseGroupToFilteredProjects, versi
         }
         // For fixed groups we want one tag for the overall group
         const projectVersionData = versionData[releaseGroupProjectNames[0]]; // all at the same version, so we can just pick the first one
-        tags.push((0, utils_1.interpolate)(releaseGroup.releaseTagPattern, {
-            version: projectVersionData.newVersion,
-        }));
+        if (projectVersionData.newVersion !== null) {
+            tags.push((0, utils_1.interpolate)(releaseGroup.releaseTagPattern, {
+                version: projectVersionData.newVersion,
+            }));
+        }
     }
     return tags;
 }

package/src/command-line/run/executor-utils.js

@@ -68,6 +68,11 @@ function readExecutorJson(nodeModule, executor, root, projects) {
     if (!executorConfig) {
         throw new Error(`Cannot find executor '${executor}' in ${executorsFilePath}.`);
     }
+    if (typeof executorConfig === 'string') {
+        // Angular CLI can have a builder pointing to another package:builder
+        const [packageName, executorName] = executorConfig.split(':');
+        return readExecutorJson(packageName, executorName, root, projects);
+    }
     const isNgCompat = !executorsJson.executors?.[executor];
     return { executorsFilePath, executorConfig, isNgCompat };
 }

package/src/config/misc-interfaces.d.ts

@@ -25,13 +25,14 @@ export interface GeneratorsJsonEntry {
     'x-use-standalone-layout'?: boolean;
 }
 export type OutputCaptureMethod = 'direct-nodejs' | 'pipe';
-export interface ExecutorsJsonEntry {
+export interface ExecutorJsonEntryConfig {
     schema: string;
     implementation: string;
     batchImplementation?: string;
     description?: string;
     hasher?: string;
 }
+export type ExecutorsJsonEntry = string | ExecutorJsonEntryConfig;
 export type Dependencies = 'dependencies' | 'devDependencies';
 export interface PackageJsonUpdateForPackage {
     version: string;

package/src/config/workspace-json-project-json.d.ts

@@ -109,6 +109,7 @@ export interface ProjectMetadata {
     targetGroups?: Record<string, string[]>;
 }
 export interface TargetMetadata {
+    [k: string]: any;
     description?: string;
     technologies?: string[];
 }