@nx/maven 22.7.1 → 22.7.2

package/dist/executors/maven/maven-batch.impl.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"maven-batch.impl.d.ts","sourceRoot":"","sources":["../../../src/executors/maven/maven-batch.impl.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,eAAe,EAAE,SAAS,EAAiB,MAAM,YAAY,CAAC;AAKvE,OAAO,EAAE,UAAU,EAAE,MAAM,+BAA+B,CAAC;AAC3D,OAAO,EAAE,kBAAkB,EAAE,MAAM,iDAAiD,CAAC;AACrF,OAAO,EAAE,mBAAmB,EAAE,MAAM,UAAU,CAAC;AAyE/C;;;GAGG;AACH,wBAA+B,kBAAkB,CAC/C,SAAS,EAAE,SAAS,EACpB,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,mBAAmB,CAAC,EAC3C,SAAS,EAAE,kBAAkB,EAC7B,OAAO,EAAE,eAAe,GACvB,cAAc,CAAC;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,UAAU,CAAA;CAAE,CAAC,CAuHtD"}
+{"version":3,"file":"maven-batch.impl.d.ts","sourceRoot":"","sources":["../../../src/executors/maven/maven-batch.impl.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,eAAe,EAAE,SAAS,EAAiB,MAAM,YAAY,CAAC;AAKvE,OAAO,EAAE,UAAU,EAAE,MAAM,+BAA+B,CAAC;AAC3D,OAAO,EAAE,kBAAkB,EAAE,MAAM,iDAAiD,CAAC;AACrF,OAAO,EAAE,mBAAmB,EAAE,MAAM,UAAU,CAAC;AAyE/C;;;GAGG;AACH,wBAA+B,kBAAkB,CAC/C,SAAS,EAAE,SAAS,EACpB,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,mBAAmB,CAAC,EAC3C,SAAS,EAAE,kBAAkB,EAC7B,OAAO,EAAE,eAAe,GACvB,cAAc,CAAC;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,UAAU,CAAA;CAAE,CAAC,CAmItD"}

package/dist/executors/maven/maven-batch.impl.js

@@ -108,64 +108,72 @@ async function* mavenBatchExecutor(taskGraph, inputs, overrides, context) {
     const stderrLines = [];
     // Collect terminal output from failed tasks
     const failedTaskOutputs = [];
-    // Yield results as they stream in
-    for await (const line of rl) {
-        if (line.startsWith('NX_RESULT:')) {
-            try {
-                const jsonStr = line.slice('NX_RESULT:'.length);
-                const data = JSON.parse(jsonStr);
-                const result = {
-                    success: data.result.success ?? false,
-                    terminalOutput: data.result.terminalOutput ?? '',
-                    startTime: data.result.startTime,
-                    endTime: data.result.endTime,
-                };
-                // Collect terminal output from failed tasks
-                if (!result.success && result.terminalOutput) {
-                    failedTaskOutputs.push(result.terminalOutput);
+    const taskIds = Object.keys(taskGraph.tasks);
+    const yielded = new Set();
+    try {
+        for await (const line of rl) {
+            if (line.startsWith('NX_RESULT:')) {
+                try {
+                    const data = JSON.parse(line.slice('NX_RESULT:'.length));
+                    const result = {
+                        success: data.result.success ?? false,
+                        status: data.result.status,
+                        terminalOutput: data.result.terminalOutput ?? '',
+                        startTime: data.result.startTime,
+                        endTime: data.result.endTime,
+                    };
+                    if (!result.success && result.terminalOutput) {
+                        failedTaskOutputs.push(result.terminalOutput);
+                    }
+                    yielded.add(data.task);
+                    yield { task: data.task, result };
+                }
+                catch (e) {
+                    console.error('[Maven Batch] Failed to parse result line:', line, e);
                 }
-                yield {
-                    task: data.task,
-                    result,
-                };
             }
-            catch (e) {
-                console.error('[Maven Batch] Failed to parse result line:', line, e);
+            else if (line.trim()) {
+                stderrLines.push(line);
             }
         }
-        else if (line.trim()) {
-            // Collect non-empty stderr lines for error reporting
-            stderrLines.push(line);
-        }
-    }
-    // Wait for process to exit
-    await new Promise((resolve, reject) => {
-        child.on('close', (code) => {
-            if (process.env.NX_VERBOSE_LOGGING === 'true') {
-                console.log(`[Maven Batch] Process exited with code: ${code}`);
-            }
-            // If process exited unexpectedly, print captured stderr
-            if (code !== 0 && stderrLines.length > 0) {
-                console.error(`[Maven Batch] Process exited with code ${code}. Stderr output:`);
+        const exitCode = await new Promise((resolvePromise, rejectPromise) => {
+            child.on('close', (code) => {
+                if (process.env.NX_VERBOSE_LOGGING === 'true') {
+                    console.log(`[Maven Batch] Process exited with code: ${code}`);
+                }
+                resolvePromise(code ?? 0);
+            });
+            child.on('error', rejectPromise);
+        });
+        if (exitCode !== 0) {
+            if (stderrLines.length > 0) {
+                console.error(`[Maven Batch] Process exited with code ${exitCode}. Stderr output:`);
                 for (const line of stderrLines) {
                     console.error(line);
                 }
             }
-            // Print failed task outputs to stderr so they appear in error.message
             if (failedTaskOutputs.length > 0) {
                 console.error('\n[Maven Batch] Failed task outputs:');
                 for (const output of failedTaskOutputs) {
                     console.error(output);
                 }
             }
-            // Reject promise if batch runner exited with non-zero code
-            if (code !== 0) {
-                reject(new Error(`Maven batch runner exited with code ${code}`));
-            }
-            else {
-                resolve();
+            throw new Error(`Maven batch runner exited with code ${exitCode}`);
+        }
+    }
+    catch (e) {
+        // Runner crashed before reporting on every task — backfill so Nx doesn't hang.
+        for (const taskId of taskIds) {
+            if (!yielded.has(taskId)) {
+                yielded.add(taskId);
+                yield {
+                    task: taskId,
+                    result: {
+                        success: false,
+                        terminalOutput: e.toString(),
+                    },
+                };
             }
-        });
-        child.on('error', reject);
-    });
+        }
+    }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/maven",
-  "version": "22.7.1",
+  "version": "22.7.2",
   "private": false,
   "description": "Nx plugin for Maven integration",
   "repository": {
@@ -47,7 +47,7 @@
   "author": "Victor Savkin",
   "license": "MIT",
   "dependencies": {
-    "@nx/devkit": "22.7.1",
+    "@nx/devkit": "22.7.2",
     "@xmldom/xmldom": "^0.8.10",
     "tree-kill": "^1.2.2",
     "tslib": "^2.3.0"
@@ -56,9 +56,9 @@
     "@types/jest": "30.0.0",
     "@types/node": "^20.19.10",
     "@types/xmldom": "^0.1.34",
-    "jest": "^30.0.2",
+    "jest": "30.3.0",
     "memfs": "^4.9.2",
-    "nx": "22.7.1",
+    "nx": "22.7.2",
     "ts-jest": "^29.4.0",
     "typescript": "~5.9.2"
   },