@nx/maven 23.0.0-beta.2 → 23.0.0-beta.20

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/dist/plugins/maven-data-cache.js

@@ -17,7 +17,7 @@ function readMavenCache(cachePath) {
  * Write the Maven targets cache to disk
  */
 function writeMavenCache(cachePath, cache) {
-    cache.writeToDisk(cachePath);
+    cache.writeToDisk();
 }
 /**
  * Get the cache path for a given options hash

package/dist/plugins/nodes.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"nodes.d.ts","sourceRoot":"","sources":["../../src/plugins/nodes.ts"],"names":[],"mappings":"AAAA,OAAO,EAAuB,aAAa,EAAa,MAAM,YAAY,CAAC;AAE3E,OAAO,EAAmB,kBAAkB,EAAE,MAAM,SAAS,CAAC;AAW9D;;GAEG;AACH,eAAO,MAAM,WAAW,EAAE,aAAa,CAAC,kBAAkB,CA+EzD,CAAC"}
+{"version":3,"file":"nodes.d.ts","sourceRoot":"","sources":["../../src/plugins/nodes.ts"],"names":[],"mappings":"AAAA,OAAO,EAAuB,aAAa,EAAa,MAAM,YAAY,CAAC;AAG3E,OAAO,EAAmB,kBAAkB,EAAE,MAAM,SAAS,CAAC;AAU9D;;GAEG;AACH,eAAO,MAAM,WAAW,EAAE,aAAa,CAAC,kBAAkB,CA+EzD,CAAC"}

package/dist/plugins/nodes.js

@@ -2,11 +2,11 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.createNodes = void 0;
 const devkit_1 = require("@nx/devkit");
+const internal_1 = require("@nx/devkit/internal");
 const path_1 = require("path");
 const types_1 = require("./types");
 const maven_analyzer_1 = require("./maven-analyzer");
 const maven_data_cache_1 = require("./maven-data-cache");
-const calculate_hash_for_create_nodes_1 = require("@nx/devkit/src/utils/calculate-hash-for-create-nodes");
 const devkit_internals_1 = require("nx/src/devkit-internals");
 /**
  * Maven plugin that analyzes Maven projects and returns configurations
@@ -34,7 +34,7 @@ exports.createNodes = [
         const mavenCache = (0, maven_data_cache_1.readMavenCache)(cachePath);
         // Calculate hashes for all pom.xml directories
         const projectRoots = configFiles.map((file) => (0, path_1.dirname)(file));
-        const hashes = await (0, calculate_hash_for_create_nodes_1.calculateHashesForCreateNodes)(projectRoots, opts, context);
+        const hashes = await (0, internal_1.calculateHashesForCreateNodes)(projectRoots, opts, context);
         // Combine all hashes into a single hash for the cache key
         const hash = (0, devkit_1.hashArray)(hashes);
         try {

package/package.json

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