@nx/devkit 16.7.1 → 16.7.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/devkit",
-  "version": "16.7.1",
+  "version": "16.7.3",
   "private": false,
   "description": "The Nx Devkit is used to customize Nx for different technologies and use cases. It contains many utility functions for reading and writing files, updating configuration, working with Abstract Syntax Trees(ASTs), and more.",
   "repository": {
@@ -28,7 +28,7 @@
   },
   "homepage": "https://nx.dev",
   "dependencies": {
-    "@nrwl/devkit": "16.7.1",
+    "@nrwl/devkit": "16.7.3",
     "ejs": "^3.1.7",
     "enquirer": "~2.3.6",
     "ignore": "^5.0.4",
@@ -46,5 +46,5 @@
     "migrations": "./migrations.json"
   },
   "type": "commonjs",
-  "gitHead": "8b38708d6c18496711695499d07c95f3ecc31cc9"
+  "gitHead": "6ff5487c5f4ed9ed008df2af39e37d98fd289259"
 }

package/src/executors/parse-target-string.js

@@ -1,10 +1,10 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.targetToTargetString = exports.parseTargetString = void 0;
-// eslint-disable-next-line @typescript-eslint/no-restricted-imports
-const split_target_1 = require("nx/src/utils/split-target");
 const nx_1 = require("../../nx");
-const { readCachedProjectGraph } = (0, nx_1.requireNx)();
+let { readCachedProjectGraph, splitTarget } = (0, nx_1.requireNx)();
+// TODO: Remove this in Nx 18 when Nx 16.7.0 is no longer supported
+splitTarget = splitTarget !== null && splitTarget !== void 0 ? splitTarget : require('nx/src/utils/split-target').splitTarget;
 function parseTargetString(targetString, projectGraph) {
     if (!projectGraph) {
         try {
@@ -14,7 +14,7 @@ function parseTargetString(targetString, projectGraph) {
             projectGraph = { nodes: {} };
         }
     }
-    const [project, target, configuration] = (0, split_target_1.splitTarget)(targetString, projectGraph);
+    const [project, target, configuration] = splitTarget(targetString, projectGraph);
     if (!project || !target) {
         throw new Error(`Invalid Target String: ${targetString}`);
     }

package/src/executors/read-target-options.js

@@ -1,11 +1,12 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.readTargetOptions = void 0;
-// eslint-disable-next-line @typescript-eslint/no-restricted-imports
-const params_1 = require("nx/src/utils/params");
 const nx_1 = require("../../nx");
 const path_1 = require("path");
-const { Workspaces, getExecutorInformation, calculateDefaultProjectName } = (0, nx_1.requireNx)();
+let { Workspaces, getExecutorInformation, calculateDefaultProjectName, combineOptionsForExecutor, } = (0, nx_1.requireNx)();
+// TODO: Remove this in Nx 18 when Nx 16.7.0 is no longer supported
+combineOptionsForExecutor =
+    combineOptionsForExecutor !== null && combineOptionsForExecutor !== void 0 ? combineOptionsForExecutor : require('nx/src/utils/params').combineOptionsForExecutor;
 /**
  * Reads and combines options for a given target.
  *
@@ -26,6 +27,6 @@ function readTargetOptions({ project, target, configuration }, context) {
         ? calculateDefaultProjectName(context.cwd, context.root, { version: 2, projects: context.projectsConfigurations.projects }, context.nxJsonConfiguration)
         : // TODO(v18): remove calculateDefaultProjectName. This is to be backwards compatible with Nx 16.5 and below.
             ws.calculateDefaultProjectName(context.cwd, { version: 2, projects: context.projectsConfigurations.projects }, context.nxJsonConfiguration);
-    return (0, params_1.combineOptionsForExecutor)({}, (_a = configuration !== null && configuration !== void 0 ? configuration : targetConfiguration.defaultConfiguration) !== null && _a !== void 0 ? _a : '', targetConfiguration, schema, defaultProject, (0, path_1.relative)(context.cwd, context.root));
+    return combineOptionsForExecutor({}, (_a = configuration !== null && configuration !== void 0 ? configuration : targetConfiguration.defaultConfiguration) !== null && _a !== void 0 ? _a : '', targetConfiguration, schema, defaultProject, (0, path_1.relative)(context.cwd, context.root));
 }
 exports.readTargetOptions = readTargetOptions;

package/src/generators/format-files.js

@@ -3,10 +3,11 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.formatFiles = void 0;
 const tslib_1 = require("tslib");
 const path = require("path");
-// eslint-disable-next-line @typescript-eslint/no-restricted-imports
-const object_sort_1 = require("nx/src/utils/object-sort");
 const nx_1 = require("../../nx");
-const { updateJson, readJson } = (0, nx_1.requireNx)();
+let { updateJson, readJson, sortObjectByKeys } = (0, nx_1.requireNx)();
+// TODO: Remove this in Nx 18 when Nx 16.7.0 is no longer supported
+sortObjectByKeys =
+    sortObjectByKeys !== null && sortObjectByKeys !== void 0 ? sortObjectByKeys : require('nx/src/utils/object-sort').sortObjectByKeys;
 /**
  * Formats all the created or updated files using Prettier
  * @param tree - the file system tree
@@ -37,7 +38,9 @@ function formatFiles(tree) {
                 if (support.ignored || !support.inferredParser) {
                     return;
                 }
-                tree.write(file.path, prettier.format(file.content.toString('utf-8'), options));
+                tree.write(file.path, 
+                // In prettier v3 the format result is a promise
+                yield prettier.format(file.content.toString('utf-8'), options));
             }
             catch (e) {
                 console.warn(`Could not format ${file.path}. Error: "${e.message}"`);
@@ -52,7 +55,7 @@ function sortTsConfig(tree) {
         if (!tsConfigPath) {
             return;
         }
-        updateJson(tree, tsConfigPath, (tsconfig) => (Object.assign(Object.assign({}, tsconfig), { compilerOptions: Object.assign(Object.assign({}, tsconfig.compilerOptions), { paths: (0, object_sort_1.sortObjectByKeys)(tsconfig.compilerOptions.paths) }) })));
+        updateJson(tree, tsConfigPath, (tsconfig) => (Object.assign(Object.assign({}, tsconfig), { compilerOptions: Object.assign(Object.assign({}, tsconfig.compilerOptions), { paths: sortObjectByKeys(tsconfig.compilerOptions.paths) }) })));
     }
     catch (e) {
         // catch noop

package/src/tasks/install-packages-task.js

@@ -29,9 +29,6 @@ function installPackagesTask(tree, alwaysRun = false, cwd = '', packageManager =
             cwd: (0, path_1.join)(tree.root, cwd),
             stdio: process.env.NX_GENERATE_QUIET === 'true' ? 'ignore' : 'inherit',
         };
-        if (pmc.preInstall) {
-            (0, child_process_1.execSync)(pmc.preInstall, execSyncOptions);
-        }
         (0, child_process_1.execSync)(pmc.install, execSyncOptions);
     }
 }

package/src/utils/convert-nx-executor.js

@@ -21,7 +21,7 @@ function convertNxExecutor(executor) {
             const projectsConfigurations = retrieveProjectConfigurationsWithAngularProjects
                 ? {
                     version: 2,
-                    projects: yield retrieveProjectConfigurationsWithAngularProjects(builderContext.workspaceRoot, nxJsonConfiguration),
+                    projects: yield retrieveProjectConfigurationsWithAngularProjects(builderContext.workspaceRoot, nxJsonConfiguration).then((p) => p.projectNodes),
                 }
                 : // TODO(v18): remove retrieveProjectConfigurations. This is to be backwards compatible with Nx 16.5 and below.
                     workspaces.readProjectsConfigurations({

package/src/utils/invoke-nx-generator.js

@@ -2,11 +2,11 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.convertNxGenerator = void 0;
 const tslib_1 = require("tslib");
-// eslint-disable-next-line @typescript-eslint/no-restricted-imports
-const logger_1 = require("nx/src/utils/logger");
 const path_1 = require("path");
 const nx_1 = require("../../nx");
-const { logger } = (0, nx_1.requireNx)();
+let { logger, stripIndent } = (0, nx_1.requireNx)();
+// TODO: Remove this in Nx 18 when Nx 16.7.0 is no longer supported
+stripIndent = stripIndent !== null && stripIndent !== void 0 ? stripIndent : require('nx/src/utils/logger').stripIndent;
 class RunCallbackTask {
     constructor(callback) {
         this.callback = callback;
@@ -167,7 +167,7 @@ class DevkitTreeFromAngularDevkitTree {
         this.warnUnsupportedFilePermissionsChange(filePath, mode);
     }
     warnUnsupportedFilePermissionsChange(filePath, mode) {
-        logger.warn((0, logger_1.stripIndent)(`The Angular DevKit tree does not support changing a file permissions.
+        logger.warn(stripIndent(`The Angular DevKit tree does not support changing a file permissions.
                   Ignoring changing ${filePath} permissions to ${mode}.`));
     }
 }

package/src/utils/module-federation/secondary-entry-points.js

@@ -3,10 +3,11 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.collectPackageSecondaryEntryPoints = exports.recursivelyCollectSecondaryEntryPointsFromDirectory = exports.getNonNodeModulesSubDirs = exports.collectWorkspaceLibrarySecondaryEntryPoints = void 0;
 const path_1 = require("path");
 const fs_1 = require("fs");
-// eslint-disable-next-line @typescript-eslint/no-restricted-imports
-const package_json_1 = require("nx/src/utils/package-json");
 const nx_1 = require("../../../nx");
-const { readJsonFile, joinPathFragments, workspaceRoot } = (0, nx_1.requireNx)();
+let { readJsonFile, joinPathFragments, workspaceRoot, readModulePackageJson } = (0, nx_1.requireNx)();
+// TODO: Remove this in Nx 18 when Nx 16.7.0 is no longer supported
+readModulePackageJson =
+    readModulePackageJson !== null && readModulePackageJson !== void 0 ? readModulePackageJson : require('nx/src/utils/package-json').readModulePackageJson;
 function collectWorkspaceLibrarySecondaryEntryPoints(library, tsconfigPathAliases) {
     const libraryRoot = (0, path_1.join)(workspaceRoot, library.root);
     const needsSecondaryEntryPointsCollected = (0, fs_1.existsSync)((0, path_1.join)(libraryRoot, 'ng-package.json'));
@@ -83,7 +84,7 @@ function collectPackageSecondaryEntryPoints(pkgName, pkgVersion, collectedPackag
     let packageJsonPath;
     let packageJson;
     try {
-        ({ path: packageJsonPath, packageJson } = (0, package_json_1.readModulePackageJson)(pkgName));
+        ({ path: packageJsonPath, packageJson } = readModulePackageJson(pkgName));
         pathToPackage = (0, path_1.dirname)(packageJsonPath);
     }
     catch (_a) {