nx 21.0.0 → 21.0.1

package/src/core/graph/styles.js

@@ -1 +1 @@
-"use strict";(self.webpackChunk=self.webpackChunk||[]).push([[869],{5663:()=>{}},s=>{var e;e=5663,s(s.s=e)}]);
+"use strict";(self.webpackChunk=self.webpackChunk||[]).push([[869],{2259:()=>{}},s=>{var e;e=2259,s(s.s=e)}]);

package/src/executors/run-commands/running-tasks.js

@@ -231,7 +231,7 @@ class RunningNodeProcess {
     }
     kill(signal) {
         return new Promise((res, rej) => {
-            if (process.platform === 'win32' || process.platform === 'darwin') {
+            if (process.platform === 'win32') {
                 if (this.childProcess.kill(signal)) {
                     res();
                 }

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

@@ -37,5 +37,6 @@ export declare class ForkedProcessTaskRunner {
     private forkProcessWithPrefixAndNotTTY;
     private forkProcessDirectOutputCapture;
     private writeTerminalOutput;
+    cleanup(signal?: NodeJS.Signals): void;
     private setupProcessEventListeners;
 }

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

@@ -233,6 +233,12 @@ class ForkedProcessTaskRunner {
     writeTerminalOutput(outputPath, content) {
         (0, fs_1.writeFileSync)(outputPath, content);
     }
+    cleanup(signal) {
+        this.processes.forEach((p) => {
+            p.kill(signal);
+        });
+        this.cleanUpBatchProcesses();
+    }
     setupProcessEventListeners() {
         const messageHandler = (message) => {
             this.pseudoTerminals.forEach((p) => {
@@ -246,29 +252,26 @@ class ForkedProcessTaskRunner {
         };
         // 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) => {
-                p.kill(signal);
-            });
-            process.off('message', messageHandler);
-            this.cleanUpBatchProcesses();
-        };
         // Terminate any task processes on exit
         process.once('exit', () => {
-            cleanUp();
+            this.cleanup();
+            process.off('message', messageHandler);
         });
         process.once('SIGINT', () => {
-            cleanUp('SIGTERM');
+            this.cleanup('SIGTERM');
+            process.off('message', messageHandler);
             // we exit here because we don't need to write anything to cache.
             process.exit((0, exit_codes_1.signalToCode)('SIGINT'));
         });
         process.once('SIGTERM', () => {
-            cleanUp('SIGTERM');
+            this.cleanup('SIGTERM');
+            process.off('message', messageHandler);
             // no exit here because we expect child processes to terminate which
             // will store results to the cache and will terminate this process
         });
         process.once('SIGHUP', () => {
-            cleanUp('SIGTERM');
+            this.cleanup('SIGTERM');
+            process.off('message', messageHandler);
             // 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

@@ -645,6 +645,7 @@ class TaskOrchestrator {
     }
     // endregion utils
     async cleanup() {
+        this.forkedProcessTaskRunner.cleanup();
         await Promise.all([
             ...Array.from(this.runningContinuousTasks).map(async ([taskId, t]) => {
                 try {