fix: include version manager bins in expanded PATH

src/__tests__/shared/pathUtils.test.ts

@@ -14,6 +14,8 @@
 
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
 import * as os from 'os';
+import * as fs from 'fs';
+import * as path from 'path';
 import {
 	expandTilde,
 	parseVersion,
@@ -28,6 +30,7 @@ vi.mock('os', async () => {
 	return {
 		...actual,
 		homedir: vi.fn(() => '/Users/testuser'),
+		tmpdir: () => '/tmp',
 	};
 });
 
@@ -276,6 +279,35 @@ describe('buildExpandedPath', () => {
 			expect(homebrewIndex).toBeLessThan(customIndex);
 		});
 
+		it('should prepend detected Node version manager bin paths', () => {
+			process.env.PATH = '/usr/bin';
+			const originalNvmDir = process.env.NVM_DIR;
+			const tempNvmDir = fs.mkdtempSync(path.join(os.tmpdir(), 'maestro-nvm-'));
+			process.env.NVM_DIR = tempNvmDir;
+			fs.mkdirSync(path.join(tempNvmDir, 'current', 'bin'), { recursive: true });
+			fs.mkdirSync(path.join(tempNvmDir, 'versions', 'node', 'v22.10.0', 'bin'), {
+				recursive: true,
+			});
+
+			try {
+				const result = buildExpandedPath();
+				const pathParts = result.split(':');
+				const currentBin = path.join(tempNvmDir, 'current', 'bin');
+				const versionedBin = path.join(tempNvmDir, 'versions', 'node', 'v22.10.0', 'bin');
+
+				expect(pathParts[0]).toBe(currentBin);
+				expect(pathParts).toContain(versionedBin);
+				expect(pathParts.indexOf(currentBin)).toBeLessThan(pathParts.indexOf(versionedBin));
+			} finally {
+				if (originalNvmDir === undefined) {
+					delete process.env.NVM_DIR;
+				} else {
+					process.env.NVM_DIR = originalNvmDir;
+				}
+				fs.rmSync(tempNvmDir, { recursive: true, force: true });
+			}
+		});
+
 		it('should accept custom paths that are prepended first', () => {
 			process.env.PATH = '/usr/bin';
 			const result = buildExpandedPath(['/my/custom/bin', '/another/path']);

src/main/process-manager/utils/__tests__/envBuilder.test.ts

@@ -11,6 +11,7 @@
  */
 
 import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import * as fs from 'fs';
 import * as os from 'os';
 import * as path from 'path';
 import { buildChildProcessEnv, buildPtyTerminalEnv } from '../envBuilder';
@@ -304,6 +305,37 @@ describe('envBuilder - Global Environment Variables', () => {
 			// The actual value depends on the system, but it should exist
 			expect((env.PATH as string).length).toBeGreaterThan(0);
 		});
+
+		it('should include detected Node version manager bins in PATH', () => {
+			const originalPlatform = process.platform;
+			Object.defineProperty(process, 'platform', { value: 'darwin' });
+			const originalNvmDir = process.env.NVM_DIR;
+			const tempNvmDir = fs.mkdtempSync(path.join(os.tmpdir(), 'maestro-nvm-'));
+			process.env.NVM_DIR = tempNvmDir;
+			fs.mkdirSync(path.join(tempNvmDir, 'current', 'bin'), { recursive: true });
+			fs.mkdirSync(path.join(tempNvmDir, 'versions', 'node', 'v22.10.0', 'bin'), {
+				recursive: true,
+			});
+
+			try {
+				const env = buildChildProcessEnv();
+				const pathParts = env.PATH?.split(path.delimiter) || [];
+				const currentBin = path.join(tempNvmDir, 'current', 'bin');
+				const versionedBin = path.join(tempNvmDir, 'versions', 'node', 'v22.10.0', 'bin');
+
+				expect(pathParts[0]).toBe(currentBin);
+				expect(pathParts).toContain(versionedBin);
+				expect(pathParts.indexOf(currentBin)).toBeLessThan(pathParts.indexOf(versionedBin));
+			} finally {
+				Object.defineProperty(process, 'platform', { value: originalPlatform });
+				if (originalNvmDir === undefined) {
+					delete process.env.NVM_DIR;
+				} else {
+					process.env.NVM_DIR = originalNvmDir;
+				}
+				fs.rmSync(tempNvmDir, { recursive: true, force: true });
+			}
+		});
 	});
 
 	describe('Test 2.6: Complex Precedence Chain', () => {

src/shared/pathUtils.ts

@@ -189,7 +189,7 @@
 
 	// nvm: Check for ~/.nvm and find installed node versions
 	const nvmDir = process.env.NVM_DIR || path.join(home, '.nvm');
 	if (fs.existsSync(nvmDir)) {
 		// Check nvm/current symlink first (preferred)
 		const nvmCurrentBin = path.join(nvmDir, 'current', 'bin');
 		if (fs.existsSync(nvmCurrentBin)) {
@@ -302,6 +302,7 @@
 export function buildExpandedPath(customPaths?: string[]): string {
 	const delimiter = path.delimiter;
 	const home = os.homedir();
+	const versionManagerPaths = detectNodeVersionManagerBinPaths();
 
 	// Start with current PATH
 	const currentPath = process.env.PATH || '';
@@ -370,6 +371,7 @@
 	} else {
 		// Unix-like paths (macOS/Linux)
 		additionalPaths = [
+			...versionManagerPaths,
 			'/opt/homebrew/bin', // Homebrew on Apple Silicon
 			'/opt/homebrew/sbin',
 			'/usr/local/bin', // Homebrew on Intel, common install location
@@ -386,17 +388,20 @@
 		];
 	}
 
-	// Add custom paths first (if provided)
+	// Iterate in reverse because each entry is prepended with unshift().
+	// This preserves the caller's intended left-to-right path precedence.
 	if (customPaths && customPaths.length > 0) {
-		for (const p of customPaths) {
+		for (let i = customPaths.length - 1; i >= 0; i--) {
+			const p = customPaths[i];
 			if (!pathParts.includes(p)) {
 				pathParts.unshift(p);
 			}
 		}
 	}
 
 	// Add standard additional paths
-	for (const p of additionalPaths) {
+	for (let i = additionalPaths.length - 1; i >= 0; i--) {
+		const p = additionalPaths[i];
 		if (!pathParts.includes(p)) {
 			pathParts.unshift(p);
 		}