nx 20.0.0-beta.5 → 20.0.0-beta.6

package/.eslintrc.json

@@ -9,8 +9,18 @@
         "no-restricted-imports": [
           "error",
           {
-            "name": "fs-extra",
-            "message": "Please use equivalent utilities from `node:fs` instead."
+            "paths": [
+              {
+                "name": "fs-extra",
+                "message": "Please use equivalent utilities from `node:fs` instead."
+              }
+            ],
+            "patterns": [
+              {
+                "group": ["**/devkit-exports"],
+                "message": "Do not import from devkit-exports from the nx package"
+              }
+            ]
           }
         ]
       }

package/migrations.json

@@ -1,42 +1,5 @@
 {
   "generators": {
-    "16.0.0-remove-nrwl-cli": {
-      "cli": "nx",
-      "version": "16.0.0-beta.0",
-      "description": "Remove @nrwl/cli.",
-      "implementation": "./src/migrations/update-16-0-0/remove-nrwl-cli"
-    },
-    "16.0.0-tokens-for-depends-on": {
-      "cli": "nx",
-      "version": "16.0.0-beta.9",
-      "description": "Replace `dependsOn.projects` and `inputs` definitions with new configuration format.",
-      "implementation": "./src/migrations/update-16-0-0/update-depends-on-to-tokens"
-    },
-    "16.0.0-update-nx-cloud-runner": {
-      "cli": "nx",
-      "version": "16.0.0-beta.0",
-      "description": "Replace @nrwl/nx-cloud with nx-cloud",
-      "implementation": "./src/migrations/update-16-0-0/update-nx-cloud-runner"
-    },
-    "16.2.0-remove-output-path-from-run-commands": {
-      "cli": "nx",
-      "version": "16.2.0-beta.0",
-      "description": "Remove outputPath from run commands",
-      "implementation": "./src/migrations/update-16-2-0/remove-run-commands-output-path"
-    },
-    "16.6.0-prefix-outputs": {
-      "cli": "nx",
-      "version": "16.6.0-beta.6",
-      "description": "Prefix outputs with {workspaceRoot}/{projectRoot} if needed",
-      "implementation": "./src/migrations/update-15-0-0/prefix-outputs"
-    },
-    "16.8.0-escape-dollar-sign-env": {
-      "cli": "nx",
-      "version": "16.8.0-beta.3",
-      "description": "Escape $ in env variables",
-      "implementation": "./src/migrations/update-16-8-0/escape-dollar-sign-env-variables",
-      "x-repair-skip": true
-    },
     "17.0.0-move-cache-directory": {
       "cli": "nx",
       "version": "17.0.0-beta.1",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nx",
-  "version": "20.0.0-beta.5",
+  "version": "20.0.0-beta.6",
   "private": false,
   "description": "The core Nx plugin contains the core functionality of Nx like the project graph, nx commands and task orchestration.",
   "repository": {
@@ -80,16 +80,16 @@
     }
   },
   "optionalDependencies": {
-    "@nx/nx-darwin-x64": "20.0.0-beta.5",
-    "@nx/nx-darwin-arm64": "20.0.0-beta.5",
-    "@nx/nx-linux-x64-gnu": "20.0.0-beta.5",
-    "@nx/nx-linux-x64-musl": "20.0.0-beta.5",
-    "@nx/nx-win32-x64-msvc": "20.0.0-beta.5",
-    "@nx/nx-linux-arm64-gnu": "20.0.0-beta.5",
-    "@nx/nx-linux-arm64-musl": "20.0.0-beta.5",
-    "@nx/nx-linux-arm-gnueabihf": "20.0.0-beta.5",
-    "@nx/nx-win32-arm64-msvc": "20.0.0-beta.5",
-    "@nx/nx-freebsd-x64": "20.0.0-beta.5"
+    "@nx/nx-darwin-x64": "20.0.0-beta.6",
+    "@nx/nx-darwin-arm64": "20.0.0-beta.6",
+    "@nx/nx-linux-x64-gnu": "20.0.0-beta.6",
+    "@nx/nx-linux-x64-musl": "20.0.0-beta.6",
+    "@nx/nx-win32-x64-msvc": "20.0.0-beta.6",
+    "@nx/nx-linux-arm64-gnu": "20.0.0-beta.6",
+    "@nx/nx-linux-arm64-musl": "20.0.0-beta.6",
+    "@nx/nx-linux-arm-gnueabihf": "20.0.0-beta.6",
+    "@nx/nx-win32-arm64-msvc": "20.0.0-beta.6",
+    "@nx/nx-freebsd-x64": "20.0.0-beta.6"
   },
   "nx-migrations": {
     "migrations": "./migrations.json",

package/src/command-line/release/index.d.ts

@@ -5,7 +5,7 @@ import type { NxReleaseConfiguration } from '../../config/nx-json';
 export declare class ReleaseClient {
     private overrideReleaseConfig;
     releaseChangelog: (args: import("./command-object").ChangelogOptions) => Promise<import("./changelog").NxReleaseChangelogResult>;
-    releasePublish: (args: import("./command-object").PublishOptions, isCLI?: boolean) => Promise<number>;
+    releasePublish: (args: import("./command-object").PublishOptions) => Promise<import("./publish").PublishProjectsResult>;
     releaseVersion: (args: import("./command-object").VersionOptions) => Promise<import("./version").NxReleaseVersionResult>;
     release: (args: import("./command-object").ReleaseOptions) => Promise<import("./version").NxReleaseVersionResult | number>;
     constructor(overrideReleaseConfig: NxReleaseConfiguration);
@@ -15,6 +15,10 @@ declare const defaultClient: ReleaseClient;
  * @public
  */
 export declare const releaseChangelog: typeof defaultClient.releaseChangelog;
+/**
+ * @public
+ */
+export { PublishProjectsResult } from './publish';
 /**
  * @public
  */
@@ -27,4 +31,3 @@ export declare const releaseVersion: typeof defaultClient.releaseVersion;
  * @public
  */
 export declare const release: typeof defaultClient.release;
-export {};

package/src/command-line/release/publish.d.ts

@@ -1,4 +1,9 @@
 import { NxReleaseConfiguration } from '../../config/nx-json';
 import { PublishOptions } from './command-object';
+export interface PublishProjectsResult {
+    [projectName: string]: {
+        code: number;
+    };
+}
 export declare const releasePublishCLIHandler: (args: PublishOptions) => Promise<number>;
-export declare function createAPI(overrideReleaseConfig: NxReleaseConfiguration): (args: PublishOptions, isCLI?: boolean) => Promise<number>;
+export declare function createAPI(overrideReleaseConfig: NxReleaseConfiguration): (args: PublishOptions) => Promise<PublishProjectsResult>;

package/src/command-line/release/publish.js

@@ -7,15 +7,21 @@ const file_map_utils_1 = require("../../project-graph/file-map-utils");
 const project_graph_1 = require("../../project-graph/project-graph");
 const run_command_1 = require("../../tasks-runner/run-command");
 const command_line_utils_1 = require("../../utils/command-line-utils");
-const output_1 = require("../../utils/output");
 const handle_errors_1 = require("../../utils/handle-errors");
+const output_1 = require("../../utils/output");
 const project_graph_utils_1 = require("../../utils/project-graph-utils");
 const graph_1 = require("../graph/graph");
 const config_1 = require("./config/config");
 const deep_merge_json_1 = require("./config/deep-merge-json");
 const filter_release_groups_1 = require("./config/filter-release-groups");
 const print_config_1 = require("./utils/print-config");
-const releasePublishCLIHandler = (args) => (0, handle_errors_1.handleErrors)(args.verbose, () => createAPI({})(args, true));
+const releasePublishCLIHandler = (args) => (0, handle_errors_1.handleErrors)(args.verbose, async () => {
+    const publishProjectsResult = await createAPI({})(args);
+    // If all projects are published successfully, return 0, otherwise return 1
+    return Object.values(publishProjectsResult).every((result) => result.code === 0)
+        ? 0
+        : 1;
+});
 exports.releasePublishCLIHandler = releasePublishCLIHandler;
 function createAPI(overrideReleaseConfig) {
     /**
@@ -23,7 +29,7 @@ function createAPI(overrideReleaseConfig) {
      * of Nx. We intentionally do not wrap the implementation with handleErrors because users need
      * to have control over their own error handling when using the API.
      */
-    return async function releasePublish(args, isCLI = false) {
+    return async function releasePublish(args) {
         /**
          * When used via the CLI, the args object will contain a __overrides_unparsed__ property that is
          * important for invoking the relevant executor behind the scenes.
@@ -60,38 +66,40 @@ function createAPI(overrideReleaseConfig) {
         const shouldExcludeTaskDependencies = _args.projects?.length > 0 ||
             _args.groups?.length > 0 ||
             args.excludeTaskDependencies;
-        let overallExitStatus = 0;
+        let overallPublishProjectsResult = {};
         if (args.projects?.length) {
             /**
              * Run publishing for all remaining release groups and filtered projects within them
              */
             for (const releaseGroup of releaseGroups) {
-                const status = await runPublishOnProjects(_args, projectGraph, nxJson, Array.from(releaseGroupToFilteredProjects.get(releaseGroup)), isCLI, {
+                const publishProjectsResult = await runPublishOnProjects(_args, projectGraph, nxJson, Array.from(releaseGroupToFilteredProjects.get(releaseGroup)), {
                     excludeTaskDependencies: shouldExcludeTaskDependencies,
                     loadDotEnvFiles: process.env.NX_LOAD_DOT_ENV_FILES !== 'false',
                 });
-                if (status !== 0) {
-                    overallExitStatus = status || 1;
-                }
+                overallPublishProjectsResult = {
+                    ...overallPublishProjectsResult,
+                    ...publishProjectsResult,
+                };
             }
-            return overallExitStatus;
+            return overallPublishProjectsResult;
         }
         /**
          * Run publishing for all remaining release groups
          */
         for (const releaseGroup of releaseGroups) {
-            const status = await runPublishOnProjects(_args, projectGraph, nxJson, releaseGroup.projects, isCLI, {
+            const publishProjectsResult = await runPublishOnProjects(_args, projectGraph, nxJson, releaseGroup.projects, {
                 excludeTaskDependencies: shouldExcludeTaskDependencies,
                 loadDotEnvFiles: process.env.NX_LOAD_DOT_ENV_FILES !== 'false',
             });
-            if (status !== 0) {
-                overallExitStatus = status || 1;
-            }
+            overallPublishProjectsResult = {
+                ...overallPublishProjectsResult,
+                ...publishProjectsResult,
+            };
         }
-        return overallExitStatus;
+        return overallPublishProjectsResult;
     };
 }
-async function runPublishOnProjects(args, projectGraph, nxJson, projectNames, isCLI, extraOptions) {
+async function runPublishOnProjects(args, projectGraph, nxJson, projectNames, extraOptions) {
     const projectsToRun = projectNames.map((projectName) => projectGraph.nodes[projectName]);
     const overrides = (0, command_line_utils_1.createOverrides)(args.__overrides_unparsed__);
     if (args.registry) {
@@ -132,7 +140,7 @@ async function runPublishOnProjects(args, projectGraph, nxJson, projectNames, is
             projects: projectNamesWithTarget,
             file,
         }, projectNamesWithTarget);
-        return 0;
+        return {};
     }
     const projectsWithTarget = projectsToRun.filter((project) => (0, project_graph_utils_1.projectHasTarget)(project, requiredTargetName));
     if (projectsWithTarget.length === 0) {
@@ -145,20 +153,18 @@ async function runPublishOnProjects(args, projectGraph, nxJson, projectNames, is
     /**
      * Run the relevant nx-release-publish executor on each of the selected projects.
      */
-    const status = await (0, run_command_1.runCommand)(projectsWithTarget, projectGraph, { nxJson }, {
+    const commandResults = await (0, run_command_1.runCommandForTasks)(projectsWithTarget, projectGraph, { nxJson }, {
         targets: [requiredTargetName],
         outputStyle: 'static',
         ...args,
         // It is possible for workspaces to have circular dependencies between packages and still release them to a registry
         nxIgnoreCycles: true,
     }, overrides, null, {}, extraOptions);
-    if (status !== 0) {
-        // In order to not add noise to the overall CLI output, do not throw an additional error
-        if (isCLI) {
-            return status;
-        }
-        // Throw an additional error for programmatic API usage
-        throw new Error('One or more of the selected projects could not be published');
+    const publishProjectsResult = {};
+    for (const taskData of Object.values(commandResults)) {
+        publishProjectsResult[taskData.task.target.project] = {
+            code: taskData.code,
+        };
     }
-    return 0;
+    return publishProjectsResult;
 }

package/src/command-line/release/utils/print-changes.js

@@ -5,8 +5,10 @@ exports.printAndFlushChanges = printAndFlushChanges;
 const chalk = require("chalk");
 const jest_diff_1 = require("jest-diff");
 const node_fs_1 = require("node:fs");
-const devkit_exports_1 = require("../../../devkit-exports");
 const tree_1 = require("../../../generators/tree");
+const workspace_root_1 = require("../../../utils/workspace-root");
+const path_1 = require("../../../utils/path");
+const logger_1 = require("../../../utils/logger");
 // jest-diff does not export this constant
 const NO_DIFF_MESSAGE = 'Compared values have no visual difference.';
 function printDiff(before, after, contextLines = 1, noDiffMessage = NO_DIFF_MESSAGE) {
@@ -43,7 +45,7 @@ function printAndFlushChanges(tree, isDryRun, diffContextLines = 1, shouldPrintD
         }
         else if (f.type === 'UPDATE') {
             console.error(`${chalk.white('UPDATE')} ${f.path}${isDryRun ? chalk.keyword('orange')(' [dry-run]') : ''}`);
-            const currentContentsOnDisk = (0, node_fs_1.readFileSync)((0, devkit_exports_1.joinPathFragments)(tree.root, f.path)).toString();
+            const currentContentsOnDisk = (0, node_fs_1.readFileSync)((0, path_1.joinPathFragments)(tree.root, f.path)).toString();
             printDiff(currentContentsOnDisk, f.content?.toString() || '', diffContextLines, noDiffMessage);
         }
         else if (f.type === 'DELETE') {
@@ -51,9 +53,9 @@ function printAndFlushChanges(tree, isDryRun, diffContextLines = 1, shouldPrintD
         }
     });
     if (!isDryRun) {
-        (0, tree_1.flushChanges)(devkit_exports_1.workspaceRoot, changes);
+        (0, tree_1.flushChanges)(workspace_root_1.workspaceRoot, changes);
     }
     if (isDryRun && shouldPrintDryRunMessage) {
-        devkit_exports_1.logger.warn(`\nNOTE: The "dryRun" flag means no changes were made.`);
+        logger_1.logger.warn(`\nNOTE: The "dryRun" flag means no changes were made.`);
     }
 }

package/src/command-line/release/utils/resolve-nx-json-error-message.js

@@ -3,10 +3,11 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.resolveNxJsonConfigErrorMessage = resolveNxJsonConfigErrorMessage;
 const node_fs_1 = require("node:fs");
 const node_path_1 = require("node:path");
-const devkit_exports_1 = require("../../../devkit-exports");
+const path_1 = require("../../../utils/path");
+const workspace_root_1 = require("../../../utils/workspace-root");
 async function resolveNxJsonConfigErrorMessage(propPath) {
-    const errorLines = await getJsonConfigLinesForErrorMessage((0, node_fs_1.readFileSync)((0, devkit_exports_1.joinPathFragments)(devkit_exports_1.workspaceRoot, 'nx.json'), 'utf-8'), propPath);
-    let nxJsonMessage = `The relevant config is defined here: ${(0, node_path_1.relative)(process.cwd(), (0, devkit_exports_1.joinPathFragments)(devkit_exports_1.workspaceRoot, 'nx.json'))}`;
+    const errorLines = await getJsonConfigLinesForErrorMessage((0, node_fs_1.readFileSync)((0, path_1.joinPathFragments)(workspace_root_1.workspaceRoot, 'nx.json'), 'utf-8'), propPath);
+    let nxJsonMessage = `The relevant config is defined here: ${(0, node_path_1.relative)(process.cwd(), (0, path_1.joinPathFragments)(workspace_root_1.workspaceRoot, 'nx.json'))}`;
     if (errorLines) {
         nxJsonMessage +=
             errorLines.startLine === errorLines.endLine

package/src/command-line/release/version.d.ts

@@ -26,10 +26,10 @@ export interface ReleaseVersionGeneratorSchema {
     conventionalCommitsConfig?: NxReleaseConfig['conventionalCommits'];
     deleteVersionPlans?: boolean;
     /**
-     * 'auto' allows users to opt into dependents being updated (a patch version bump) when a dependency is versioned.
-     * This is only applicable to independently released projects.
+     * 'auto' is the default and will cause dependents to be updated (a patch version bump) when a dependency is versioned.
+     * This is only applicable to independently released projects. 'never' will cause dependents to not be updated.
      */
-    updateDependents?: 'never' | 'auto';
+    updateDependents?: 'auto' | 'never';
     /**
      * Whether or not to completely omit project logs when that project has no applicable changes. This can be useful for
      * large monorepos which have a large number of projects, especially when only a subset are released together.

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

@@ -108,6 +108,20 @@ export interface ProjectMetadata {
     description?: string;
     technologies?: string[];
     targetGroups?: Record<string, string[]>;
+    owners?: {
+        [ownerId: string]: {
+            ownedFiles: {
+                files: ['*'] | string[];
+                fromConfig?: {
+                    filePath: string;
+                    location: {
+                        startLine: number;
+                        endLine: number;
+                    };
+                };
+            }[];
+        };
+    };
 }
 export interface TargetMetadata {
     [k: string]: any;