@nx/js 17.3.0-rc.1 → 17.3.1

package/babel.js

@@ -12,10 +12,10 @@ module.exports = function (api, options = {}) {
     const emitDecoratorMetadata = api.caller((caller) => caller?.emitDecoratorMetadata ?? true);
     // Determine settings  for `@babel//babel-plugin-transform-class-properties`,
     // so that we can sync the `loose` option with `@babel/preset-env`.
-    // TODO(v18): Remove classProperties since it's no longer needed, now that the class props transform is in preset-env.
+    // TODO(v19): Remove classProperties since it's no longer needed, now that the class props transform is in preset-env.
     const loose = options.classProperties?.loose ?? options.loose ?? true;
     if (options.classProperties) {
-        devkit_1.logger.warn(`Use =\`loose\` option instead of \`classProperties.loose\`. The \`classProperties\` option will be removed in Nx 18`);
+        devkit_1.logger.warn(`Use =\`loose\` option instead of \`classProperties.loose\`. The \`classProperties\` option will be removed in Nx 19`);
     }
     return {
         presets: [

package/executors.json

@@ -1,5 +1,5 @@
 {
-  "$schema": "http://json-schema.org/schema",
+  "$schema": "https://json-schema.org/schema",
   "executors": {
     "tsc": {
       "implementation": "./src/executors/tsc/tsc.impl",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/js",
-  "version": "17.3.0-rc.1",
+  "version": "17.3.1",
   "private": false,
   "description": "The JS plugin for Nx contains executors and generators that provide the best experience for developing JavaScript and TypeScript projects. ",
   "repository": {
@@ -30,11 +30,11 @@
   "generators": "./generators.json",
   "executors": "./executors.json",
   "dependencies": {
-    "@babel/core": "^7.22.9",
+    "@babel/core": "^7.23.2",
     "@babel/plugin-proposal-decorators": "^7.22.7",
-    "@babel/plugin-transform-runtime": "^7.22.9",
+    "@babel/plugin-transform-runtime": "^7.23.2",
     "@babel/plugin-transform-class-properties": "^7.22.5",
-    "@babel/preset-env": "^7.22.9",
+    "@babel/preset-env": "^7.23.2",
     "@babel/preset-typescript": "^7.22.5",
     "@babel/runtime": "^7.22.6",
     "@phenomnomnominal/tsquery": "~5.0.1",
@@ -57,9 +57,9 @@
     "semver": "7.5.3",
     "source-map-support": "0.5.19",
     "tslib": "^2.3.0",
-    "@nx/devkit": "17.3.0-rc.1",
-    "@nx/workspace": "17.3.0-rc.1",
-    "@nrwl/js": "17.3.0-rc.1"
+    "@nx/devkit": "17.3.1",
+    "@nx/workspace": "17.3.1",
+    "@nrwl/js": "17.3.1"
   },
   "peerDependencies": {
     "verdaccio": "^5.0.4"

package/src/executors/node/schema.json

@@ -1,7 +1,7 @@
 {
   "version": 2,
   "outputCapture": "direct-nodejs",
-  "$schema": "http://json-schema.org/schema",
+  "$schema": "https://json-schema.org/schema",
   "cli": "nx",
   "title": "Node executor",
   "description": "Execute Nodejs applications.",

package/src/executors/release-publish/schema.json

@@ -1,5 +1,5 @@
 {
-  "$schema": "http://json-schema.org/schema",
+  "$schema": "https://json-schema.org/schema",
   "version": 2,
   "title": "Implementation details of `nx release publish`",
   "description": "DO NOT INVOKE DIRECTLY WITH `nx run`. Use `nx release publish` instead.",

package/src/executors/swc/schema.json

@@ -1,7 +1,7 @@
 {
   "version": 2,
   "outputCapture": "direct-nodejs",
-  "$schema": "http://json-schema.org/schema",
+  "$schema": "https://json-schema.org/schema",
   "cli": "nx",
   "title": "Typescript Build Target",
   "description": "Builds using SWC.",

package/src/executors/verdaccio/schema.json

@@ -1,5 +1,5 @@
 {
-  "$schema": "http://json-schema.org/schema",
+  "$schema": "https://json-schema.org/schema",
   "version": 2,
   "title": "Verdaccio Local Registry",
   "description": "Start a local registry with Verdaccio.",

package/src/generators/convert-to-swc/schema.json

@@ -1,5 +1,5 @@
 {
-  "$schema": "http://json-schema.org/schema",
+  "$schema": "https://json-schema.org/schema",
   "$id": "NxTypescriptLibrary",
   "cli": "nx",
   "title": "Convert a TSC library to SWC",

package/src/generators/init/schema.json

@@ -1,5 +1,5 @@
 {
-  "$schema": "http://json-schema.org/schema",
+  "$schema": "https://json-schema.org/schema",
   "$id": "NxTypescriptInit",
   "cli": "nx",
   "title": "Init nx/js",

package/src/generators/library/schema.json

@@ -1,5 +1,5 @@
 {
-  "$schema": "http://json-schema.org/schema",
+  "$schema": "https://json-schema.org/schema",
   "$id": "NxTypescriptLibrary",
   "cli": "nx",
   "title": "Create a TypeScript Library",

package/src/generators/release-version/release-version.js

@@ -9,7 +9,6 @@ const git_1 = require("nx/src/command-line/release/utils/git");
 const resolve_semver_specifier_1 = require("nx/src/command-line/release/utils/resolve-semver-specifier");
 const semver_1 = require("nx/src/command-line/release/utils/semver");
 const version_1 = require("nx/src/command-line/release/version");
-const client_1 = require("nx/src/daemon/client/client");
 const utils_1 = require("nx/src/tasks-runner/utils");
 const ora = require("ora");
 const path_1 = require("path");
@@ -314,26 +313,7 @@ To fix this you will either need to add a package.json file at that location, or
             data: versionData,
             callback: async (tree, opts) => {
                 const cwd = tree.root;
-                const isDaemonEnabled = client_1.daemonClient.enabled();
-                if (isDaemonEnabled) {
-                    // temporarily stop the daemon, as it will error if the lock file is updated
-                    await client_1.daemonClient.stop();
-                }
-                const updatedFiles = (0, update_lock_file_1.updateLockFile)(cwd, opts);
-                if (isDaemonEnabled) {
-                    try {
-                        await client_1.daemonClient.startInBackground();
-                    }
-                    catch (e) {
-                        // If the daemon fails to start, we don't want to prevent the user from continuing, so we just log the error and move on
-                        if (opts.verbose) {
-                            devkit_1.output.warn({
-                                title: 'Unable to restart the Nx Daemon. It will be disabled until you run "nx reset"',
-                                bodyLines: [e.message],
-                            });
-                        }
-                    }
-                }
+                const updatedFiles = await (0, update_lock_file_1.updateLockFile)(cwd, opts);
                 return updatedFiles;
             },
         };

package/src/generators/release-version/schema.json

@@ -1,5 +1,5 @@
 {
-  "$schema": "http://json-schema.org/schema",
+  "$schema": "https://json-schema.org/schema",
   "$id": "NxJSReleaseVersionGenerator",
   "cli": "nx",
   "title": "Implementation details of `nx release version`",

package/src/generators/release-version/utils/update-lock-file.d.ts

@@ -2,4 +2,4 @@ export declare function updateLockFile(cwd: string, { dryRun, verbose, generator
     dryRun?: boolean;
     verbose?: boolean;
     generatorOptions?: Record<string, unknown>;
-}): string[];
+}): Promise<string[]>;

package/src/generators/release-version/utils/update-lock-file.js

@@ -3,16 +3,22 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.updateLockFile = void 0;
 const devkit_1 = require("@nx/devkit");
 const child_process_1 = require("child_process");
+const client_1 = require("nx/src/daemon/client/client");
 // eslint-disable-next-line @typescript-eslint/no-restricted-imports
 const lock_file_1 = require("nx/src/plugins/js/lock-file/lock-file");
 const semver_1 = require("semver");
-function updateLockFile(cwd, { dryRun, verbose, generatorOptions, }) {
+async function updateLockFile(cwd, { dryRun, verbose, generatorOptions, }) {
     if (generatorOptions?.skipLockFileUpdate) {
         if (verbose) {
             console.log('\nSkipped lock file update because skipLockFileUpdate was set.');
         }
         return [];
     }
+    const isDaemonEnabled = client_1.daemonClient.enabled();
+    if (isDaemonEnabled) {
+        // temporarily stop the daemon, as it will error if the lock file is updated
+        await client_1.daemonClient.stop();
+    }
     const packageManager = (0, devkit_1.detectPackageManager)(cwd);
     const packageManagerCommands = (0, devkit_1.getPackageManagerCommand)(packageManager);
     let installArgs = generatorOptions?.installArgs || '';
@@ -43,6 +49,20 @@ function updateLockFile(cwd, { dryRun, verbose, generatorOptions, }) {
         return [];
     }
     execLockFileUpdate(command, cwd, env);
+    if (isDaemonEnabled) {
+        try {
+            await client_1.daemonClient.startInBackground();
+        }
+        catch (e) {
+            // If the daemon fails to start, we don't want to prevent the user from continuing, so we just log the error and move on
+            if (verbose) {
+                devkit_1.output.warn({
+                    title: 'Unable to restart the Nx Daemon. It will be disabled until you run "nx reset"',
+                    bodyLines: [e.message],
+                });
+            }
+        }
+    }
     return [lockFile];
 }
 exports.updateLockFile = updateLockFile;

package/src/generators/setup-build/schema.json

@@ -1,5 +1,5 @@
 {
-  "$schema": "http://json-schema.org/schema",
+  "$schema": "https://json-schema.org/schema",
   "$id": "SetupBuild",
   "title": "Setup Build",
   "description": "Sets up build target for a project.",

package/src/generators/setup-verdaccio/schema.json

@@ -1,5 +1,5 @@
 {
-  "$schema": "http://json-schema.org/schema",
+  "$schema": "https://json-schema.org/schema",
   "$id": "SetupVerdaccio",
   "title": "Setup Verdaccio",
   "description": "Setup Verdaccio local-registry.",

package/src/utils/buildable-libs-utils.js

@@ -84,7 +84,8 @@ function calculateProjectDependencies(projGraph, root, projectName, targetName,
 exports.calculateProjectDependencies = calculateProjectDependencies;
 function collectDependencies(project, projGraph, acc, shallow, areTopLevelDeps = true) {
     (projGraph.dependencies[project] || []).forEach((dependency) => {
-        if (!acc.some((dep) => dep.name === dependency.target)) {
+        const existingEntry = acc.find((dep) => dep.name === dependency.target);
+        if (!existingEntry) {
             // Temporary skip this. Currently the set of external nodes is built from package.json, not lock file.
             // As a result, some nodes might be missing. This should not cause any issues, we can just skip them.
             if (dependency.target.startsWith('npm:') &&
@@ -96,6 +97,9 @@ function collectDependencies(project, projGraph, acc, shallow, areTopLevelDeps =
                 collectDependencies(dependency.target, projGraph, acc, shallow, false);
             }
         }
+        else if (areTopLevelDeps && !existingEntry.isTopLevel) {
+            existingEntry.isTopLevel = true;
+        }
     });
     return acc;
 }

package/src/utils/inline.js

@@ -161,9 +161,11 @@ function movePackage(from, to) {
     (0, fs_extra_1.copySync)(from, to, { overwrite: true });
 }
 function updateImports(destOutputPath, inlinedDepsDestOutputRecord) {
-    const importRegex = new RegExp(Object.keys(inlinedDepsDestOutputRecord)
-        .map((pathAlias) => `["'](${pathAlias})["']`)
-        .join('|'), 'g');
+    const pathAliases = Object.keys(inlinedDepsDestOutputRecord);
+    if (pathAliases.length == 0) {
+        return;
+    }
+    const importRegex = new RegExp(pathAliases.map((pathAlias) => `["'](${pathAlias})["']`).join('|'), 'g');
     recursiveUpdateImport(destOutputPath, importRegex, inlinedDepsDestOutputRecord);
 }
 function recursiveUpdateImport(dirPath, importRegex, inlinedDepsDestOutputRecord, rootParentDir) {

package/src/utils/typescript/ts-config.js

@@ -46,6 +46,7 @@ function getRootTsConfigFileName(tree) {
 exports.getRootTsConfigFileName = getRootTsConfigFileName;
 function addTsConfigPath(tree, importPath, lookupPaths) {
     (0, devkit_1.updateJson)(tree, getRootTsConfigPathInTree(tree), (json) => {
+        json.compilerOptions ??= {};
         const c = json.compilerOptions;
         c.paths ??= {};
         if (c.paths[importPath]) {