@nx/maven 23.2.0-beta.7 → 23.2.0-beta.8

package/dist/plugins/maven-analyzer.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"maven-analyzer.d.ts","sourceRoot":"","sources":["../../src/plugins/maven-analyzer.ts"],"names":[],"mappings":"AAIA,OAAO,EAAE,iBAAiB,EAAE,kBAAkB,EAAE,MAAM,SAAS,CAAC;AAShE;;;GAGG;AACH,wBAAgB,0BAA0B,IAAI,IAAI,CAKjD;AAaD;;GAEG;AACH,wBAAsB,gBAAgB,CACpC,aAAa,EAAE,MAAM,EACrB,OAAO,EAAE,kBAAkB,GAC1B,OAAO,CAAC,iBAAiB,CAAC,CA6J5B"}
+{"version":3,"file":"maven-analyzer.d.ts","sourceRoot":"","sources":["../../src/plugins/maven-analyzer.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,iBAAiB,EAAE,kBAAkB,EAAE,MAAM,SAAS,CAAC;AAahE;;;GAGG;AACH,wBAAgB,0BAA0B,IAAI,IAAI,CAKjD;AAaD;;GAEG;AACH,wBAAsB,gBAAgB,CACpC,aAAa,EAAE,MAAM,EACrB,OAAO,EAAE,kBAAkB,GAC1B,OAAO,CAAC,iBAAiB,CAAC,CA8J5B"}

package/dist/plugins/maven-analyzer.js

@@ -2,13 +2,10 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.cancelPendingMavenAnalysis = cancelPendingMavenAnalysis;
 exports.runMavenAnalysis = runMavenAnalysis;
-const tslib_1 = require("tslib");
 const path_1 = require("path");
 const fs_1 = require("fs");
-const child_process_1 = require("child_process");
 const devkit_1 = require("@nx/devkit");
 const detect_maven_executable_1 = require("../utils/detect-maven-executable");
-const tree_kill_1 = tslib_1.__importDefault(require("tree-kill"));
 const internal_1 = require("@nx/devkit/internal");
 const DEFAULT_ANALYSIS_TIMEOUT_SECONDS = (0, internal_1.isCI)() ? 600 : 120;
 let currentAbortController;
@@ -77,17 +74,18 @@ async function runMavenAnalysis(workspaceRoot, options) {
     devkit_1.logger.verbose(`[Maven Analyzer] Spawning Maven process...`);
     try {
         await new Promise((resolve, reject) => {
-            const child = (0, child_process_1.spawn)(mavenExecutable, mavenArgs, {
+            const child = (0, internal_1.safeSpawn)(mavenExecutable, mavenArgs, {
                 cwd: workspaceRoot,
-                windowsHide: true,
-                shell: true,
                 stdio: 'pipe', // Always use pipe so we can control output
             });
-            // Use tree-kill on abort to kill the entire process tree
+            // On abort, kill the entire process tree and settle immediately — a
+            // wedged process that outlives the kill signal would otherwise keep
+            // this promise pending and the abort error would never surface.
             const onAbort = () => {
                 if (child.pid) {
-                    (0, tree_kill_1.default)(child.pid);
+                    (0, internal_1.killProcessTreeGraceful)(child.pid).catch(() => { });
                 }
+                reject(new Error('Maven analysis aborted'));
             };
             signal.addEventListener('abort', onAbort, { once: true });
             devkit_1.logger.verbose(`[Maven Analyzer] Process spawned with PID: ${child.pid}`);

package/dist/utils/detect-maven-executable.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"detect-maven-executable.d.ts","sourceRoot":"","sources":["../../src/utils/detect-maven-executable.ts"],"names":[],"mappings":"AAQA;;GAEG;AACH,wBAAgB,kBAAkB,CAAC,aAAa,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAwBvE;AAED;;GAEG;AACH,wBAAgB,QAAQ,CAAC,aAAa,EAAE,MAAM,GAAG,OAAO,CAGvD;AAED;;GAEG;AACH,wBAAgB,qBAAqB,CAAC,aAAa,EAAE,MAAM,GAAG,MAAM,CA4BnE"}
+{"version":3,"file":"detect-maven-executable.d.ts","sourceRoot":"","sources":["../../src/utils/detect-maven-executable.ts"],"names":[],"mappings":"AAOA;;GAEG;AACH,wBAAgB,kBAAkB,CAAC,aAAa,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAqBvE;AAED;;GAEG;AACH,wBAAgB,QAAQ,CAAC,aAAa,EAAE,MAAM,GAAG,OAAO,CAGvD;AAED;;GAEG;AACH,wBAAgB,qBAAqB,CAAC,aAAa,EAAE,MAAM,GAAG,MAAM,CA4BnE"}

package/dist/utils/detect-maven-executable.js

@@ -4,10 +4,9 @@ exports.detectMavenVersion = detectMavenVersion;
 exports.isMaven4 = isMaven4;
 exports.detectMavenExecutable = detectMavenExecutable;
 const fs_1 = require("fs");
-const child_process_1 = require("child_process");
 const path_1 = require("path");
 const devkit_1 = require("@nx/devkit");
-// Cache Maven version to avoid repeated execSync calls
+const internal_1 = require("@nx/devkit/internal");
 let cachedMavenVersion = null;
 /**
  * Detect Maven version (e.g., "4.0.0-rc-4" or "3.9.9")
@@ -18,11 +17,8 @@ function detectMavenVersion(workspaceRoot) {
     }
     const executable = detectMavenExecutable(workspaceRoot);
     try {
-        const output = (0, child_process_1.execSync)(`${executable} --version`, {
+        const output = (0, internal_1.safeExecFileSync)(executable, ['--version'], {
             cwd: workspaceRoot,
-            stdio: 'pipe',
-            encoding: 'utf-8',
-            windowsHide: true,
         });
         // Parse version from output like "Apache Maven 4.0.0-rc-4" or "Apache Maven 3.9.9"
         const match = output.match(/Apache Maven (\d+\.\d+\.\d+[^\s]*)/);
@@ -50,7 +46,7 @@ function isMaven4(workspaceRoot) {
 function detectMavenExecutable(workspaceRoot) {
     // First priority: Check for Maven Daemon
     try {
-        (0, child_process_1.execSync)('mvnd --version', { stdio: 'pipe', windowsHide: true });
+        (0, internal_1.safeExecFileSync)('mvnd', ['--version']);
         devkit_1.logger.verbose(`[Maven] Found Maven Daemon, using: mvnd`);
         return 'mvnd';
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/maven",
-  "version": "23.2.0-beta.7",
+  "version": "23.2.0-beta.8",
   "private": false,
   "description": "Nx plugin for Maven integration",
   "repository": {
@@ -50,9 +50,8 @@
   "license": "MIT",
   "dependencies": {
     "@xmldom/xmldom": "^0.8.10",
-    "tree-kill": "^1.2.2",
     "tslib": "^2.3.0",
-    "@nx/devkit": "23.2.0-beta.7"
+    "@nx/devkit": "23.2.0-beta.8"
   },
   "devDependencies": {
     "@types/jest": "30.0.0",
@@ -62,7 +61,7 @@
     "memfs": "^4.9.2",
     "ts-jest": "^29.4.7",
     "typescript": "~6.0.3",
-    "nx": "23.2.0-beta.7"
+    "nx": "23.2.0-beta.8"
   },
   "publishConfig": {
     "access": "public"