nx 21.0.0-beta.5 → 21.0.0-beta.6

package/package.json

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

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

@@ -140,6 +140,9 @@ class ForkedProcessTaskRunner {
             }
             this.pseudoTerminals.delete(pseudoTerminal);
             this.processes.delete(p);
+            if (!streamOutput) {
+                this.options.lifeCycle.printTaskTerminalOutput(task, code === 0 ? 'success' : 'failure', terminalOutput);
+            }
             this.writeTerminalOutput(temporaryOutputPath, terminalOutput);
         });
         return p;

package/src/tasks-runner/life-cycles/tui-summary-life-cycle.js

@@ -31,20 +31,19 @@ function getTuiTerminalSummaryLifeCycle({ projectNames, tasks, args, overrides,
         }
     };
     lifeCycle.printTaskTerminalOutput = (task, taskStatus, terminalOutput) => {
+        taskIdsInOrderOfCompletion.push(task.id);
         tasksToTerminalOutputs[task.id] = { terminalOutput, taskStatus };
     };
     lifeCycle.setTaskStatus = (taskId, taskStatus) => {
         if (taskStatus === 9 /* NativeTaskStatus.Stopped */) {
             totalStoppedTasks++;
-            taskIdsInOrderOfCompletion.push(taskId);
         }
     };
     lifeCycle.endTasks = (taskResults) => {
-        for (let t of taskResults) {
+        for (const { task, status } of taskResults) {
             totalCompletedTasks++;
-            inProgressTasks.delete(t.task.id);
-            taskIdsInOrderOfCompletion.push(t.task.id);
-            switch (t.status) {
+            inProgressTasks.delete(task.id);
+            switch (status) {
                 case 'remote-cache':
                 case 'local-cache':
                 case 'local-cache-kept-existing':
@@ -56,7 +55,7 @@ function getTuiTerminalSummaryLifeCycle({ projectNames, tasks, args, overrides,
                     break;
                 case 'failure':
                     totalFailedTasks++;
-                    failedTasks.add(t.task.id);
+                    failedTasks.add(task.id);
                     break;
             }
         }

package/src/tasks-runner/running-tasks/node-child-process.js

@@ -153,18 +153,11 @@ class NodeChildProcessWithDirectOutput {
         this.exitCallbacks.push(cb);
     }
     async getResults() {
-        const terminalOutput = this.getTerminalOutput();
-        if (this.exited) {
-            return Promise.resolve({
-                code: this.exitCode,
-                terminalOutput,
-            });
+        if (!this.exited) {
+            await this.waitForExit();
         }
-        await this.waitForExit();
-        return Promise.resolve({
-            code: this.exitCode,
-            terminalOutput,
-        });
+        const terminalOutput = this.getTerminalOutput();
+        return { code: this.exitCode, terminalOutput };
     }
     waitForExit() {
         return new Promise((res) => {

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

@@ -408,8 +408,16 @@ class TaskOrchestrator {
             // and release the threads
             await this.scheduleNextTasksAndReleaseThreads();
             if (this.initializingTaskIds.has(task.id)) {
-                // Hold the thread forever
-                await new Promise(() => { });
+                await new Promise((res) => {
+                    runningTask.onExit((code) => {
+                        if (!this.tuiEnabled) {
+                            if (code > 128) {
+                                process.exit(code);
+                            }
+                        }
+                        res();
+                    });
+                });
             }
             return runningTask;
         }
@@ -435,8 +443,16 @@ class TaskOrchestrator {
         });
         await this.scheduleNextTasksAndReleaseThreads();
         if (this.initializingTaskIds.has(task.id)) {
-            // Hold the thread forever
-            await new Promise(() => { });
+            await new Promise((res) => {
+                childProcess.onExit((code) => {
+                    if (!this.tuiEnabled) {
+                        if (code > 128) {
+                            process.exit(code);
+                        }
+                    }
+                    res();
+                });
+            });
         }
         return childProcess;
     }