nx 19.6.2 → 19.6.4

package/.eslintrc.json

@@ -100,7 +100,18 @@
               "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
+              // The native modules are optional and only one of them will ever be installed on a given machine
+              "@nx/nx-darwin-x64",
+              "@nx/nx-darwin-arm64",
+              "@nx/nx-linux-x64-gnu",
+              "@nx/nx-linux-x64-musl",
+              "@nx/nx-win32-x64-msvc",
+              "@nx/nx-linux-arm64-gnu",
+              "@nx/nx-linux-arm64-musl",
+              "@nx/nx-linux-arm-gnueabihf",
+              "@nx/nx-win32-arm64-msvc",
+              "@nx/nx-freebsd-x64"
             ]
           }
         ]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nx",
-  "version": "19.6.2",
+  "version": "19.6.4",
   "private": false,
   "description": "The core Nx plugin contains the core functionality of Nx like the project graph, nx commands and task orchestration.",
   "repository": {
@@ -71,7 +71,7 @@
     "yargs-parser": "21.1.1",
     "node-machine-id": "1.1.12",
     "ora": "5.3.0",
-    "@nrwl/tao": "19.6.2"
+    "@nrwl/tao": "19.6.4"
   },
   "peerDependencies": {
     "@swc-node/register": "^1.8.0",
@@ -86,16 +86,16 @@
     }
   },
   "optionalDependencies": {
-    "@nx/nx-darwin-x64": "19.6.2",
-    "@nx/nx-darwin-arm64": "19.6.2",
-    "@nx/nx-linux-x64-gnu": "19.6.2",
-    "@nx/nx-linux-x64-musl": "19.6.2",
-    "@nx/nx-win32-x64-msvc": "19.6.2",
-    "@nx/nx-linux-arm64-gnu": "19.6.2",
-    "@nx/nx-linux-arm64-musl": "19.6.2",
-    "@nx/nx-linux-arm-gnueabihf": "19.6.2",
-    "@nx/nx-win32-arm64-msvc": "19.6.2",
-    "@nx/nx-freebsd-x64": "19.6.2"
+    "@nx/nx-darwin-x64": "19.6.4",
+    "@nx/nx-darwin-arm64": "19.6.4",
+    "@nx/nx-linux-x64-gnu": "19.6.4",
+    "@nx/nx-linux-x64-musl": "19.6.4",
+    "@nx/nx-win32-x64-msvc": "19.6.4",
+    "@nx/nx-linux-arm64-gnu": "19.6.4",
+    "@nx/nx-linux-arm64-musl": "19.6.4",
+    "@nx/nx-linux-arm-gnueabihf": "19.6.4",
+    "@nx/nx-win32-arm64-msvc": "19.6.4",
+    "@nx/nx-freebsd-x64": "19.6.4"
   },
   "nx-migrations": {
     "migrations": "./migrations.json",

package/src/command-line/graph/graph.d.ts

@@ -1,4 +1,4 @@
-import { ProjectFileMap, ProjectGraphDependency, ProjectGraphProjectNode } from '../../config/project-graph';
+import { ProjectFileMap, ProjectGraph, ProjectGraphDependency, ProjectGraphProjectNode } from '../../config/project-graph';
 import { TaskGraph } from '../../config/task-graph';
 export interface GraphError {
     message: string;
@@ -48,3 +48,20 @@ export declare function generateGraph(args: {
     exclude?: string[];
     affected?: boolean;
 }, affectedProjects: string[]): Promise<void>;
+/**
+ * The data type that `nx graph --file graph.json` or `nx build --graph graph.json` contains
+ */
+export interface GraphJson {
+    /**
+     * A graph of tasks populated with `nx build --graph`
+     */
+    tasks?: TaskGraph;
+    /**
+     * The plans for hashing a task in the task graph
+     */
+    taskPlans?: Record<string, string[]>;
+    /**
+     * The project graph
+     */
+    graph: ProjectGraph;
+}

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

@@ -675,8 +675,10 @@ function expandInputs(inputs, project, allWorkspaceFiles, depGraphClientResponse
     const externalInputs = [];
     const otherInputs = [];
     inputs.forEach((input) => {
-        if (input.startsWith('{workspaceRoot}')) {
-            workspaceRootInputs.push(input);
+        // grouped workspace inputs look like workspace:[pattern,otherPattern]
+        if (input.startsWith('workspace:[')) {
+            const inputs = input.substring(11, input.length - 1).split(',');
+            workspaceRootInputs.push(...inputs);
             return;
         }
         const maybeProjectName = input.split(':')[0];
@@ -696,22 +698,7 @@ function expandInputs(inputs, project, allWorkspaceFiles, depGraphClientResponse
             return;
         }
     });
-    const workspaceRootsExpanded = workspaceRootInputs.flatMap((input) => {
-        const matches = [];
-        const withoutWorkspaceRoot = input.substring(16);
-        const matchingFile = allWorkspaceFiles.find((t) => t.file === withoutWorkspaceRoot);
-        if (matchingFile) {
-            matches.push(matchingFile.file);
-        }
-        else {
-            allWorkspaceFiles
-                .filter((f) => (0, minimatch_1.minimatch)(f.file, withoutWorkspaceRoot))
-                .forEach((f) => {
-                matches.push(f.file);
-            });
-        }
-        return matches;
-    });
+    const workspaceRootsExpanded = getExpandedWorkspaceRoots(workspaceRootInputs, allWorkspaceFiles);
     const otherInputsExpanded = otherInputs.map((input) => {
         if (input === 'TsConfig') {
             return (0, path_1.relative)(workspace_root_1.workspaceRoot, (0, typescript_1.getRootTsConfigPath)());
@@ -744,6 +731,36 @@ function expandInputs(inputs, project, allWorkspaceFiles, depGraphClientResponse
         external: externalInputs,
     };
 }
+function getExpandedWorkspaceRoots(workspaceRootInputs, allWorkspaceFiles) {
+    const workspaceRootsExpanded = [];
+    const negativeWRPatterns = [];
+    const positiveWRPatterns = [];
+    for (const fileset of workspaceRootInputs) {
+        if (fileset.startsWith('!')) {
+            negativeWRPatterns.push(fileset.substring(17));
+        }
+        else {
+            positiveWRPatterns.push(fileset.substring(16));
+        }
+    }
+    for (const pattern of positiveWRPatterns) {
+        const matchingFile = allWorkspaceFiles.find((t) => t.file === pattern);
+        if (matchingFile &&
+            !negativeWRPatterns.some((p) => (0, minimatch_1.minimatch)(matchingFile.file, p))) {
+            workspaceRootsExpanded.push(matchingFile.file);
+        }
+        else {
+            allWorkspaceFiles
+                .filter((f) => (0, minimatch_1.minimatch)(f.file, pattern) &&
+                !negativeWRPatterns.some((p) => (0, minimatch_1.minimatch)(f.file, p)))
+                .forEach((f) => {
+                workspaceRootsExpanded.push(f.file);
+            });
+        }
+    }
+    workspaceRootsExpanded.sort();
+    return workspaceRootsExpanded;
+}
 async function createJsonOutput(prunedGraph, rawGraph, projects, targets) {
     const response = {
         graph: prunedGraph,

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

@@ -1079,55 +1079,24 @@ function getImplementationPath(collection, collectionPath, name) {
     }
     return { path: implPath, fnSymbol };
 }
-// TODO (v17): This should just become something like:
-// ```
-// return !collection.generators[name] && collection.schematics[name]
-// ```
+// TODO (v21): Remove CLI determination of Angular Migration
 function isAngularMigration(collection, collectionPath, name) {
     const entry = collection.generators?.[name] || collection.schematics?.[name];
-    // In the future we will determine this based on the location of the entry in the collection.
-    // If the entry is under `schematics`, it will be assumed to be an angular cli migration.
-    // If the entry is under `generators`, it will be assumed to be an nx migration.
-    // For now, we will continue to obey the cli property, if it exists.
-    // If it doesn't exist, we will check if the implementation references @angular/devkit.
     const shouldBeNx = !!collection.generators?.[name];
     const shouldBeNg = !!collection.schematics?.[name];
-    let useAngularDevkitToRunMigration = false;
-    const { path: implementationPath } = getImplementationPath(collection, collectionPath, name);
-    const implStringContents = (0, fs_1.readFileSync)(implementationPath, 'utf-8');
-    // TODO (v17): Remove this check and the cli property access - it is only here for backwards compatibility.
-    if (['@angular/material', '@angular/cdk'].includes(collection.name) ||
-        [
-            "import('@angular-devkit",
-            'import("@angular-devkit',
-            "require('@angular-devkit",
-            'require("@angular-devkit',
-            "from '@angular-devkit",
-            'from "@angular-devkit',
-        ].some((s) => implStringContents.includes(s))) {
-        useAngularDevkitToRunMigration = true;
-    }
-    if (useAngularDevkitToRunMigration && shouldBeNx) {
+    if (entry.cli && entry.cli !== 'nx' && collection.generators?.[name]) {
         output_1.output.warn({
             title: `The migration '${collection.name}:${name}' appears to be an Angular CLI migration, but is located in the 'generators' section of migrations.json.`,
             bodyLines: [
-                'In Nx 17, migrations inside `generators` will be treated as Angular Devkit migrations.',
-                "Please open an issue on the plugin's repository if you believe this is an error.",
-            ],
-        });
-    }
-    if (!useAngularDevkitToRunMigration && entry.cli === 'nx' && shouldBeNg) {
-        output_1.output.warn({
-            title: `The migration '${collection.name}:${name}' appears to be an Nx migration, but is located in the 'schematics' section of migrations.json.`,
-            bodyLines: [
-                'In Nx 17, migrations inside `generators` will be treated as nx devkit migrations.',
+                'In Nx 21, migrations inside `generators` will be treated as Nx Devkit migrations and therefore may not run correctly if they are using Angular Devkit.',
+                'If the migration should be run with Angular Devkit, please place the migration inside `schematics` instead.',
                 "Please open an issue on the plugin's repository if you believe this is an error.",
             ],
         });
     }
     // Currently, if the cli property exists we listen to it. If its nx, its not an ng cli migration.
     // If the property is not set, we will fall back to our intuition.
-    return entry.cli ? entry.cli !== 'nx' : useAngularDevkitToRunMigration;
+    return entry.cli ? entry.cli !== 'nx' : !shouldBeNx && shouldBeNg;
 }
 const getNgCompatLayer = (() => {
     let _ngCliAdapter;

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

@@ -10,19 +10,21 @@ export declare class ReleaseClient {
     release: (args: import("./command-object").ReleaseOptions) => Promise<import("./version").NxReleaseVersionResult | number>;
     constructor(overrideReleaseConfig: NxReleaseConfiguration);
 }
+declare const defaultClient: ReleaseClient;
 /**
  * @public
  */
-export declare const releaseChangelog: any;
+export declare const releaseChangelog: typeof defaultClient.releaseChangelog;
 /**
  * @public
  */
-export declare const releasePublish: any;
+export declare const releasePublish: typeof defaultClient.releasePublish;
 /**
  * @public
  */
-export declare const releaseVersion: any;
+export declare const releaseVersion: typeof defaultClient.releaseVersion;
 /**
  * @public
  */
-export declare const release: any;
+export declare const release: typeof defaultClient.release;
+export {};

package/src/command-line/yargs-utils/shared-options.d.ts

@@ -1,7 +1,8 @@
-import { Argv } from 'yargs';
+import { Argv, ParserConfigurationOptions } from 'yargs';
 interface ExcludeOptions {
     exclude: string[];
 }
+export declare const defaultYargsParserConfiguration: Partial<ParserConfigurationOptions>;
 export declare function withExcludeOption(yargs: Argv): Argv<ExcludeOptions>;
 export interface RunOptions {
     exclude: string;

package/src/command-line/yargs-utils/shared-options.js

@@ -1,5 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.defaultYargsParserConfiguration = void 0;
 exports.withExcludeOption = withExcludeOption;
 exports.withRunOptions = withRunOptions;
 exports.withTargetAndConfigurationOption = withTargetAndConfigurationOption;
@@ -12,6 +13,13 @@ exports.withOverrides = withOverrides;
 exports.withOutputStyleOption = withOutputStyleOption;
 exports.withRunOneOptions = withRunOneOptions;
 exports.parseCSV = parseCSV;
+exports.defaultYargsParserConfiguration = {
+    'strip-dashed': true,
+    'unknown-options-as-args': true,
+    'populate--': true,
+    'parse-numbers': false,
+    'parse-positional-numbers': false,
+};
 function withExcludeOption(yargs) {
     return yargs.option('exclude', {
         describe: 'Exclude certain projects from being processed',
@@ -128,11 +136,7 @@ function withBatch(yargs) {
 }
 function withAffectedOptions(yargs) {
     return withExcludeOption(yargs)
-        .parserConfiguration({
-        'strip-dashed': true,
-        'unknown-options-as-args': true,
-        'populate--': true,
-    })
+        .parserConfiguration(exports.defaultYargsParserConfiguration)
         .option('files', {
         describe: 'Change the way Nx is calculating the affected command by providing directly changed files, list of files delimited by commas or spaces',
         type: 'string',
@@ -169,11 +173,7 @@ function withAffectedOptions(yargs) {
 }
 function withRunManyOptions(yargs) {
     return withRunOptions(yargs)
-        .parserConfiguration({
-        'strip-dashed': true,
-        'unknown-options-as-args': true,
-        'populate--': true,
-    })
+        .parserConfiguration(exports.defaultYargsParserConfiguration)
         .option('projects', {
         type: 'string',
         alias: 'p',
@@ -225,11 +225,7 @@ function withOutputStyleOption(yargs, choices = [
 function withRunOneOptions(yargs) {
     const executorShouldShowHelp = !(process.argv[2] === 'run' && process.argv[3] === '--help');
     const res = withRunOptions(withOutputStyleOption(withConfiguration(yargs), allOutputStyles))
-        .parserConfiguration({
-        'strip-dashed': true,
-        'unknown-options-as-args': true,
-        'populate--': true,
-    })
+        .parserConfiguration(exports.defaultYargsParserConfiguration)
         .option('project', {
         describe: 'Target project',
         type: 'string',