nx 20.8.0-canary.20250412-07ad2d7 → 20.8.0

package/package.json

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

package/src/tasks-runner/forked-process-task-runner.d.ts

@@ -8,10 +8,12 @@ export declare class ForkedProcessTaskRunner {
     cliPath: string;
     private readonly verbose;
     private processes;
+    private finishedProcesses;
     private pseudoTerminal;
     constructor(options: DefaultTasksRunnerOptions);
     init(): Promise<void>;
     forkProcessForBatch({ executorName, taskGraph: batchTaskGraph }: Batch, projectGraph: ProjectGraph, fullTaskGraph: TaskGraph, env: NodeJS.ProcessEnv): Promise<BatchResults>;
+    cleanUpBatchProcesses(): void;
     forkProcessLegacy(task: Task, { temporaryOutputPath, streamOutput, pipeOutput, taskGraph, env, }: {
         temporaryOutputPath: string;
         streamOutput: boolean;

package/src/tasks-runner/forked-process-task-runner.js

@@ -20,6 +20,7 @@ class ForkedProcessTaskRunner {
         this.cliPath = (0, utils_1.getCliPath)();
         this.verbose = process.env.NX_VERBOSE_LOGGING === 'true';
         this.processes = new Set();
+        this.finishedProcesses = new Set();
         this.pseudoTerminal = pseudo_terminal_1.PseudoTerminal.isSupported()
             ? (0, pseudo_terminal_1.getPseudoTerminal)()
             : null;
@@ -33,6 +34,7 @@ class ForkedProcessTaskRunner {
     // TODO: vsavkin delegate terminal output printing
     forkProcessForBatch({ executorName, taskGraph: batchTaskGraph }, projectGraph, fullTaskGraph, env) {
         return new Promise((res, rej) => {
+            let p;
             try {
                 const count = Object.keys(batchTaskGraph.tasks).length;
                 if (count > 1) {
@@ -42,7 +44,7 @@ class ForkedProcessTaskRunner {
                     const args = (0, utils_1.getPrintableCommandArgsForTask)(Object.values(batchTaskGraph.tasks)[0]);
                     output_1.output.logCommand(args.join(' '));
                 }
-                const p = (0, child_process_1.fork)(workerPath, {
+                p = (0, child_process_1.fork)(workerPath, {
                     stdio: ['inherit', 'inherit', 'inherit', 'ipc'],
                     env,
                 });
@@ -52,20 +54,18 @@ class ForkedProcessTaskRunner {
                     if (code === null)
                         code = (0, exit_codes_1.signalToCode)(signal);
                     if (code !== 0) {
-                        const results = {};
-                        for (const rootTaskId of batchTaskGraph.roots) {
-                            results[rootTaskId] = {
-                                success: false,
-                                terminalOutput: '',
-                            };
-                        }
                         rej(new Error(`"${executorName}" exited unexpectedly with code: ${code}`));
                     }
                 });
+                p.on('error', (err) => {
+                    this.processes.delete(p);
+                    rej(err || new Error(`"${executorName}" exited unexpectedly`));
+                });
                 p.on('message', (message) => {
                     switch (message.type) {
                         case batch_messages_1.BatchMessageType.CompleteBatchExecution: {
                             res(message.results);
+                            this.finishedProcesses.add(p);
                             break;
                         }
                         case batch_messages_1.BatchMessageType.RunTasks: {
@@ -90,9 +90,23 @@ class ForkedProcessTaskRunner {
             }
             catch (e) {
                 rej(e);
+                if (p) {
+                    this.processes.delete(p);
+                    p.kill();
+                }
             }
         });
     }
+    cleanUpBatchProcesses() {
+        if (this.finishedProcesses.size > 0) {
+            this.finishedProcesses.forEach((p) => {
+                if ('connected' in p ? p.connected : p.isAlive) {
+                    p.kill();
+                }
+            });
+            this.finishedProcesses.clear();
+        }
+    }
     async forkProcessLegacy(task, { temporaryOutputPath, streamOutput, pipeOutput, taskGraph, env, }) {
         return pipeOutput
             ? await this.forkProcessPipeOutputCapture(task, {
@@ -157,6 +171,7 @@ class ForkedProcessTaskRunner {
         });
         return new Promise((res) => {
             p.onExit((code) => {
+                this.processes.delete(p);
                 // If the exit code is greater than 128, it's a special exit code for a signal
                 if (code >= 128) {
                     process.exit(code);
@@ -227,7 +242,7 @@ class ForkedProcessTaskRunner {
                 p.stderr.on('data', (chunk) => {
                     outWithErr.push(chunk.toString());
                 });
-                p.on('exit', (code, signal) => {
+                p.once('exit', (code, signal) => {
                     this.processes.delete(p);
                     if (code === null)
                         code = (0, exit_codes_1.signalToCode)(signal);
@@ -272,7 +287,8 @@ class ForkedProcessTaskRunner {
                     taskGraph,
                     isVerbose: this.verbose,
                 });
-                p.on('exit', (code, signal) => {
+                p.once('exit', (code, signal) => {
+                    this.processes.delete(p);
                     if (code === null)
                         code = (0, exit_codes_1.signalToCode)(signal);
                     // we didn't print any output as we were running the command
@@ -317,8 +333,7 @@ class ForkedProcessTaskRunner {
                 process.send(message);
             });
         }
-        // When the nx process gets a message, it will be sent into the task's process
-        process.on('message', (message) => {
+        const messageHandler = (message) => {
             // this.publisher.publish(message.toString());
             if (this.pseudoTerminal) {
                 this.pseudoTerminal.sendMessageToChildren(message);
@@ -328,39 +343,34 @@ class ForkedProcessTaskRunner {
                     p.send(message);
                 }
             });
-        });
-        // Terminate any task processes on exit
-        process.on('exit', () => {
+        };
+        // When the nx process gets a message, it will be sent into the task's process
+        process.on('message', messageHandler);
+        const cleanUp = (signal) => {
             this.processes.forEach((p) => {
                 if ('connected' in p ? p.connected : p.isAlive) {
-                    p.kill();
+                    p.kill(signal);
                 }
             });
+            process.off('message', messageHandler);
+            this.cleanUpBatchProcesses();
+        };
+        // Terminate any task processes on exit
+        process.once('exit', () => {
+            cleanUp();
         });
-        process.on('SIGINT', () => {
-            this.processes.forEach((p) => {
-                if ('connected' in p ? p.connected : p.isAlive) {
-                    p.kill('SIGTERM');
-                }
-            });
+        process.once('SIGINT', () => {
+            cleanUp('SIGTERM');
             // we exit here because we don't need to write anything to cache.
             process.exit((0, exit_codes_1.signalToCode)('SIGINT'));
         });
-        process.on('SIGTERM', () => {
-            this.processes.forEach((p) => {
-                if ('connected' in p ? p.connected : p.isAlive) {
-                    p.kill('SIGTERM');
-                }
-            });
+        process.once('SIGTERM', () => {
+            cleanUp('SIGTERM');
             // no exit here because we expect child processes to terminate which
             // will store results to the cache and will terminate this process
         });
-        process.on('SIGHUP', () => {
-            this.processes.forEach((p) => {
-                if ('connected' in p ? p.connected : p.isAlive) {
-                    p.kill('SIGTERM');
-                }
-            });
+        process.once('SIGHUP', () => {
+            cleanUp('SIGTERM');
             // no exit here because we expect child processes to terminate which
             // will store results to the cache and will terminate this process
         });

package/src/tasks-runner/task-orchestrator.js

@@ -171,6 +171,7 @@ class TaskOrchestrator {
             results.push(...batchResults);
         }
         await this.postRunSteps(tasks, results, doNotSkipCache, { groupId });
+        this.forkedProcessTaskRunner.cleanUpBatchProcesses();
         const tasksCompleted = taskEntries.filter(([taskId]) => this.completedTasks[taskId]);
         // Batch is still not done, run it again
         if (tasksCompleted.length !== taskEntries.length) {