nx 20.1.0-canary.20241106-51f47f0 → 20.1.0-canary.20241107-fd2e8d0

package/bin/nx.js

@@ -16,6 +16,7 @@ const assert_supported_platform_1 = require("../src/native/assert-supported-plat
 const perf_hooks_1 = require("perf_hooks");
 const workspace_context_1 = require("../src/utils/workspace-context");
 const client_1 = require("../src/daemon/client/client");
+const db_connection_1 = require("../src/utils/db-connection");
 function main() {
     if (process.argv[2] !== 'report' &&
         process.argv[2] !== '--version' &&
@@ -213,4 +214,11 @@ const getLatestVersionOfNx = ((fn) => {
     let cache = null;
     return () => cache || (cache = fn());
 })(_getLatestVersionOfNx);
+function nxCleanup() {
+    (0, db_connection_1.removeDbConnections)();
+}
+process.on('exit', nxCleanup);
+process.on('SIGINT', nxCleanup);
+process.on('SIGTERM', nxCleanup);
+process.on('SIGHUP', nxCleanup);
 main();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nx",
-  "version": "20.1.0-canary.20241106-51f47f0",
+  "version": "20.1.0-canary.20241107-fd2e8d0",
   "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.1.0-canary.20241106-51f47f0",
-    "@nx/nx-darwin-arm64": "20.1.0-canary.20241106-51f47f0",
-    "@nx/nx-linux-x64-gnu": "20.1.0-canary.20241106-51f47f0",
-    "@nx/nx-linux-x64-musl": "20.1.0-canary.20241106-51f47f0",
-    "@nx/nx-win32-x64-msvc": "20.1.0-canary.20241106-51f47f0",
-    "@nx/nx-linux-arm64-gnu": "20.1.0-canary.20241106-51f47f0",
-    "@nx/nx-linux-arm64-musl": "20.1.0-canary.20241106-51f47f0",
-    "@nx/nx-linux-arm-gnueabihf": "20.1.0-canary.20241106-51f47f0",
-    "@nx/nx-win32-arm64-msvc": "20.1.0-canary.20241106-51f47f0",
-    "@nx/nx-freebsd-x64": "20.1.0-canary.20241106-51f47f0"
+    "@nx/nx-darwin-x64": "20.1.0-canary.20241107-fd2e8d0",
+    "@nx/nx-darwin-arm64": "20.1.0-canary.20241107-fd2e8d0",
+    "@nx/nx-linux-x64-gnu": "20.1.0-canary.20241107-fd2e8d0",
+    "@nx/nx-linux-x64-musl": "20.1.0-canary.20241107-fd2e8d0",
+    "@nx/nx-win32-x64-msvc": "20.1.0-canary.20241107-fd2e8d0",
+    "@nx/nx-linux-arm64-gnu": "20.1.0-canary.20241107-fd2e8d0",
+    "@nx/nx-linux-arm64-musl": "20.1.0-canary.20241107-fd2e8d0",
+    "@nx/nx-linux-arm-gnueabihf": "20.1.0-canary.20241107-fd2e8d0",
+    "@nx/nx-win32-arm64-msvc": "20.1.0-canary.20241107-fd2e8d0",
+    "@nx/nx-freebsd-x64": "20.1.0-canary.20241107-fd2e8d0"
   },
   "nx-migrations": {
     "migrations": "./migrations.json",

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

@@ -4,6 +4,7 @@ exports.loginHandler = loginHandler;
 const update_manager_1 = require("../../nx-cloud/update-manager");
 const get_cloud_options_1 = require("../../nx-cloud/utilities/get-cloud-options");
 const handle_errors_1 = require("../../utils/handle-errors");
+const resolution_helpers_1 = require("../../nx-cloud/resolution-helpers");
 function loginHandler(args) {
     if (args.nxCloudUrl) {
         process.env.NX_CLOUD_API = args.nxCloudUrl;
@@ -11,6 +12,8 @@ function loginHandler(args) {
     return (0, handle_errors_1.handleErrors)(args.verbose, async () => {
         const nxCloudClient = (await (0, update_manager_1.verifyOrUpdateNxCloudClient)((0, get_cloud_options_1.getCloudOptions)()))
             .nxCloudClient;
+        const paths = (0, resolution_helpers_1.findAncestorNodeModules)(__dirname, []);
+        nxCloudClient.configureLightClientRequire()(paths);
         await nxCloudClient.commands.login();
     });
 }

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

@@ -4,10 +4,13 @@ exports.logoutHandler = logoutHandler;
 const update_manager_1 = require("../../nx-cloud/update-manager");
 const get_cloud_options_1 = require("../../nx-cloud/utilities/get-cloud-options");
 const handle_errors_1 = require("../../utils/handle-errors");
+const resolution_helpers_1 = require("../../nx-cloud/resolution-helpers");
 function logoutHandler(args) {
     return (0, handle_errors_1.handleErrors)(args.verbose, async () => {
         const nxCloudClient = (await (0, update_manager_1.verifyOrUpdateNxCloudClient)((0, get_cloud_options_1.getCloudOptions)()))
             .nxCloudClient;
+        const paths = (0, resolution_helpers_1.findAncestorNodeModules)(__dirname, []);
+        nxCloudClient.configureLightClientRequire()(paths);
         await nxCloudClient.commands.logout();
     });
 }

package/src/daemon/server/shutdown-utils.js

@@ -15,6 +15,7 @@ const socket_utils_1 = require("../socket-utils");
 const cache_1 = require("../cache");
 const plugins_1 = require("./plugins");
 const error_types_1 = require("../../project-graph/error-types");
+const db_connection_1 = require("../../utils/db-connection");
 exports.SERVER_INACTIVITY_TIMEOUT_MS = 10800000; // 10800000 ms = 3 hours
 let watcherInstance;
 function storeWatcherInstance(instance) {
@@ -50,6 +51,7 @@ async function handleServerProcessTermination({ server, reason, sockets, }) {
         }
         (0, cache_1.deleteDaemonJsonProcessCache)();
         (0, plugins_1.cleanupPlugins)();
+        (0, db_connection_1.removeDbConnections)();
         logger_1.serverLogger.log(`Server stopped because: "${reason}"`);
     }
     finally {

package/src/native/index.d.ts

@@ -97,6 +97,8 @@ export interface CachedResult {
   outputsPath: string
 }
 
+export declare export function closeDbConnection(connection: ExternalObject<NxDbConnection>): void
+
 export declare export function connectToNxDb(cacheDir: string, nxVersion: string, dbName?: string | undefined | null): ExternalObject<NxDbConnection>
 
 export declare export function copy(src: string, dest: string): void

package/src/native/native-bindings.js

@@ -371,6 +371,7 @@ module.exports.TaskDetails = nativeBinding.TaskDetails
 module.exports.TaskHasher = nativeBinding.TaskHasher
 module.exports.Watcher = nativeBinding.Watcher
 module.exports.WorkspaceContext = nativeBinding.WorkspaceContext
+module.exports.closeDbConnection = nativeBinding.closeDbConnection
 module.exports.connectToNxDb = nativeBinding.connectToNxDb
 module.exports.copy = nativeBinding.copy
 module.exports.EventType = nativeBinding.EventType

package/src/project-graph/utils/project-configuration-utils.js

@@ -253,8 +253,10 @@ plugins) {
                 else {
                     errorBodyLines.push(`  - ${e.message}`);
                 }
-                const innerStackTrace = '    ' + e.stack.split('\n').join('\n    ');
-                errorBodyLines.push(innerStackTrace);
+                if (e.stack) {
+                    const innerStackTrace = '    ' + e.stack.split('\n')?.join('\n    ');
+                    errorBodyLines.push(innerStackTrace);
+                }
             }
             error.stack = errorBodyLines.join('\n');
             // This represents a single plugin erroring out with a hard error.

package/src/tasks-runner/create-task-graph.d.ts

@@ -22,22 +22,19 @@ export declare class ProcessTasks {
     createTask(id: string, project: ProjectGraphProjectNode, target: string, resolvedConfiguration: string | undefined, overrides: Object): Task;
     resolveConfiguration(project: ProjectGraphProjectNode, target: string, configuration: string | undefined): string;
     getId(project: string, target: string, configuration: string | undefined): string;
-    /**
-     * this function is used to get the non dummy dependencies of a task recursively
-     * For example, when we have the following dependencies:
-     * {
-     *   'app1:compile': [ 'app2:__nx_dummy_task__' ],
-     *   'app2:__nx_dummy_task__': [ 'app3:__nx_dummy_task__' ],
-     *   'app3:__nx_dummy_task__': [ 'app4:precompile' ],
-     *   'app4:precompile': []
-     * }
-     * getNonDummyDeps('app1:compile') will return ['app1:compile']
-     * getNonDummyDeps('app2:__nx_dummy_task__') will return ['app4:precompile']
-     * getNonDummyDeps('app3:__nx_dummy_task__') will return ['app4:precompile']
-     * getNonDummyDeps('app4:precompile') will return ['app4:precompile']
-     */
-    private getNonDummyDeps;
-    private filterDummyTasks;
 }
 export declare function createTaskGraph(projectGraph: ProjectGraph, extraTargetDependencies: TargetDependencies, projectNames: string[], targets: string[], configuration: string | undefined, overrides: Object, excludeTaskDependencies?: boolean): TaskGraph;
 export declare function mapTargetDefaultsToDependencies(defaults: TargetDefaults | undefined): TargetDependencies;
+/**
+ * This function is used to filter out the dummy tasks from the dependencies
+ * It will manipulate the dependencies object in place
+ */
+export declare function filterDummyTasks(dependencies: {
+    [k: string]: string[];
+}): void;
+/**
+ * this function is used to get the non dummy dependencies of a task recursively
+ */
+export declare function getNonDummyDeps(currentTask: string, dependencies: {
+    [k: string]: string[];
+}, cycles?: Set<string>, seen?: Set<string>): string[];

package/src/tasks-runner/create-task-graph.js

@@ -3,6 +3,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.ProcessTasks = void 0;
 exports.createTaskGraph = createTaskGraph;
 exports.mapTargetDefaultsToDependencies = mapTargetDefaultsToDependencies;
+exports.filterDummyTasks = filterDummyTasks;
+exports.getNonDummyDeps = getNonDummyDeps;
 const utils_1 = require("./utils");
 const project_graph_utils_1 = require("../utils/project-graph-utils");
 const output_1 = require("../utils/output");
@@ -54,7 +56,7 @@ class ProcessTasks {
                 this.dependencies[d] = this.dependencies[d].filter((dd) => !!initialTasks[dd]);
             }
         }
-        this.filterDummyTasks();
+        filterDummyTasks(this.dependencies);
         for (const taskId of Object.keys(this.dependencies)) {
             if (this.dependencies[taskId].length > 0) {
                 this.dependencies[taskId] = [
@@ -189,52 +191,6 @@ class ProcessTasks {
         }
         return id;
     }
-    /**
-     * this function is used to get the non dummy dependencies of a task recursively
-     * For example, when we have the following dependencies:
-     * {
-     *   'app1:compile': [ 'app2:__nx_dummy_task__' ],
-     *   'app2:__nx_dummy_task__': [ 'app3:__nx_dummy_task__' ],
-     *   'app3:__nx_dummy_task__': [ 'app4:precompile' ],
-     *   'app4:precompile': []
-     * }
-     * getNonDummyDeps('app1:compile') will return ['app1:compile']
-     * getNonDummyDeps('app2:__nx_dummy_task__') will return ['app4:precompile']
-     * getNonDummyDeps('app3:__nx_dummy_task__') will return ['app4:precompile']
-     * getNonDummyDeps('app4:precompile') will return ['app4:precompile']
-     */
-    getNonDummyDeps(currentTask, originalTask, cycle) {
-        if (currentTask === originalTask) {
-            return [];
-        }
-        else if (currentTask.endsWith(DUMMY_TASK_TARGET)) {
-            if (cycle?.length && cycle?.includes(currentTask)) {
-                return [];
-            }
-            // if not a cycle, recursively get the non dummy dependencies
-            return (this.dependencies[currentTask]?.flatMap((dep) => this.getNonDummyDeps(dep, originalTask, cycle)) ?? []);
-        }
-        else {
-            return [currentTask];
-        }
-    }
-    filterDummyTasks() {
-        const cycle = (0, task_graph_utils_1.findCycle)({ dependencies: this.dependencies });
-        for (const [key, deps] of Object.entries(this.dependencies)) {
-            if (!key.endsWith(DUMMY_TASK_TARGET)) {
-                const normalizedDeps = [];
-                for (const dep of deps) {
-                    normalizedDeps.push(...this.getNonDummyDeps(dep, key, cycle));
-                }
-                this.dependencies[key] = normalizedDeps;
-            }
-        }
-        for (const key of Object.keys(this.dependencies)) {
-            if (key.endsWith(DUMMY_TASK_TARGET)) {
-                delete this.dependencies[key];
-            }
-        }
-    }
 }
 exports.ProcessTasks = ProcessTasks;
 function createTaskGraph(projectGraph, extraTargetDependencies, projectNames, targets, configuration, overrides, excludeTaskDependencies = false) {
@@ -268,3 +224,43 @@ function interpolateOverrides(args, projectName, project) {
     });
     return interpolatedArgs;
 }
+/**
+ * This function is used to filter out the dummy tasks from the dependencies
+ * It will manipulate the dependencies object in place
+ */
+function filterDummyTasks(dependencies) {
+    const cycles = (0, task_graph_utils_1.findCycles)({ dependencies });
+    for (const [key, deps] of Object.entries(dependencies)) {
+        if (!key.endsWith(DUMMY_TASK_TARGET)) {
+            const normalizedDeps = [];
+            for (const dep of deps) {
+                normalizedDeps.push(...getNonDummyDeps(dep, dependencies, cycles, new Set([key])));
+            }
+            dependencies[key] = normalizedDeps;
+        }
+    }
+    for (const key of Object.keys(dependencies)) {
+        if (key.endsWith(DUMMY_TASK_TARGET)) {
+            delete dependencies[key];
+        }
+    }
+}
+/**
+ * this function is used to get the non dummy dependencies of a task recursively
+ */
+function getNonDummyDeps(currentTask, dependencies, cycles, seen = new Set()) {
+    if (seen.has(currentTask)) {
+        return [];
+    }
+    seen.add(currentTask);
+    if (currentTask.endsWith(DUMMY_TASK_TARGET)) {
+        if (cycles?.has(currentTask)) {
+            return [];
+        }
+        // if not a cycle, recursively get the non dummy dependencies
+        return (dependencies[currentTask]?.flatMap((dep) => getNonDummyDeps(dep, dependencies, cycles, seen)) ?? []);
+    }
+    else {
+        return [currentTask];
+    }
+}

package/src/tasks-runner/task-graph-utils.d.ts

@@ -1,8 +1,19 @@
 import { ProjectGraph } from '../config/project-graph';
 import { TaskGraph } from '../config/task-graph';
+/**
+ * This function finds a cycle in the graph.
+ * @returns the first cycle found, or null if no cycle is found.
+ */
 export declare function findCycle(graph: {
     dependencies: Record<string, string[]>;
 }): string[] | null;
+/**
+ * This function finds all cycles in the graph.
+ * @returns a list of unique task ids in all cycles found, or null if no cycle is found.
+ */
+export declare function findCycles(graph: {
+    dependencies: Record<string, string[]>;
+}): Set<string> | null;
 export declare function makeAcyclic(graph: {
     roots: string[];
     dependencies: Record<string, string[]>;

package/src/tasks-runner/task-graph-utils.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.findCycle = findCycle;
+exports.findCycles = findCycles;
 exports.makeAcyclic = makeAcyclic;
 exports.validateNoAtomizedTasks = validateNoAtomizedTasks;
 const output_1 = require("../utils/output");
@@ -17,6 +18,10 @@ function _findCycle(graph, id, visited, path) {
     }
     return null;
 }
+/**
+ * This function finds a cycle in the graph.
+ * @returns the first cycle found, or null if no cycle is found.
+ */
 function findCycle(graph) {
     const visited = {};
     for (const t of Object.keys(graph.dependencies)) {
@@ -29,6 +34,24 @@ function findCycle(graph) {
     }
     return null;
 }
+/**
+ * This function finds all cycles in the graph.
+ * @returns a list of unique task ids in all cycles found, or null if no cycle is found.
+ */
+function findCycles(graph) {
+    const visited = {};
+    const cycles = new Set();
+    for (const t of Object.keys(graph.dependencies)) {
+        visited[t] = false;
+    }
+    for (const t of Object.keys(graph.dependencies)) {
+        const cycle = _findCycle(graph, t, visited, [t]);
+        if (cycle) {
+            cycle.forEach((t) => cycles.add(t));
+        }
+    }
+    return cycles.size ? cycles : null;
+}
 function _makeAcyclic(graph, id, visited, path) {
     if (visited[id])
         return;

package/src/utils/db-connection.d.ts

@@ -3,3 +3,4 @@ export declare function getDbConnection(opts?: {
     directory?: string;
     dbName?: string;
 }): ExternalObject<NxDbConnection>;
+export declare function removeDbConnections(): void;

package/src/utils/db-connection.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getDbConnection = getDbConnection;
+exports.removeDbConnections = removeDbConnections;
 const native_1 = require("../native");
 const cache_directory_1 = require("./cache-directory");
 const package_json_1 = require("../../package.json");
@@ -11,6 +12,12 @@ function getDbConnection(opts = {}) {
     const connection = getEntryOrSet(dbConnectionMap, key, () => (0, native_1.connectToNxDb)(opts.directory, package_json_1.version, opts.dbName));
     return connection;
 }
+function removeDbConnections() {
+    for (const connection of dbConnectionMap.values()) {
+        (0, native_1.closeDbConnection)(connection);
+    }
+    dbConnectionMap.clear();
+}
 function getEntryOrSet(map, key, defaultValue) {
     const existing = map.get(key);
     if (existing) {