agentgui 1.0.236 → 1.0.238

package/bin/gmgui.cjs

@@ -16,19 +16,29 @@ async function gmgui(args = []) {
     const installer = 'npm';
 
     // Ensure dependencies are installed only if node_modules is missing
-    // Skip this for bunx which manages dependencies independently
+    // Skip this for bunx/npx which manage dependencies independently
     const nodeModulesPath = path.join(projectRoot, 'node_modules');
-    const isBunx = process.env.npm_execpath && process.env.npm_execpath.includes('bunx');
+    const execPath = process.env.npm_execpath || '';
+    const isBunx = execPath.includes('bun') || process.env.BUN_INSTALL;
+    const isNpx = execPath.includes('npx') || process.env._.includes('npx');
+    
+    // Also skip if running from temp/cache directory (bunx/npm cache)
+    const isFromCache = projectRoot.includes('node_modules') && 
+                        (projectRoot.includes('.bun') || projectRoot.includes('_npx') || projectRoot.includes('npm-cache'));
 
-    if (!isBunx && !fs.existsSync(nodeModulesPath)) {
+    if (!isBunx && !isNpx && !isFromCache && !fs.existsSync(nodeModulesPath)) {
       console.log(`Installing dependencies with ${installer}...`);
       const installResult = spawnSync(installer, ['install'], {
         cwd: projectRoot,
-        stdio: 'inherit'
+        stdio: 'inherit',
+        shell: true
       });
-      if (installResult.status !== 0) {
+      if (installResult.status !== 0 && installResult.status !== null) {
         throw new Error(`${installer} install failed with code ${installResult.status}`);
       }
+      if (installResult.error) {
+        throw new Error(`${installer} install failed: ${installResult.error.message}`);
+      }
     }
 
     const port = process.env.PORT || 3000;

package/lib/claude-runner.js

@@ -1,5 +1,15 @@
 import { spawn } from 'child_process';
 
+const isWindows = process.platform === 'win32';
+
+function getSpawnOptions(cwd, additionalOptions = {}) {
+  const options = { cwd, ...additionalOptions };
+  if (isWindows) {
+    options.shell = true;
+  }
+  return options;
+}
+
 /**
  * Agent Framework
  * Extensible registry for AI agent CLI integrations
@@ -51,7 +61,7 @@ class AgentRunner {
       } = config;
 
       const args = this.buildArgs(prompt, config);
-      const proc = spawn(this.command, args, { cwd });
+      const proc = spawn(this.command, args, getSpawnOptions(cwd));
 
       if (config.onPid) {
         try { config.onPid(proc.pid); } catch (e) {}
@@ -211,7 +221,7 @@ class AgentRunner {
       const baseArgs = this.requiresAdapter && this.adapterCommand ? this.adapterArgs : this.buildArgs(prompt, config);
       const args = [...baseArgs];
 
-      const proc = spawn(cmd, args, { cwd });
+      const proc = spawn(cmd, args, getSpawnOptions(cwd));
 
       if (config.onPid) {
         try { config.onPid(proc.pid); } catch (e) {}
@@ -474,13 +484,15 @@ class AgentRegistry {
 
   listACPAvailable() {
     const { spawnSync } = require('child_process');
+    const isWindows = process.platform === 'win32';
     return this.list().filter(agent => {
       try {
-        const which = spawnSync('which', [agent.command], { encoding: 'utf-8', timeout: 3000 });
+        const whichCmd = isWindows ? 'where' : 'which';
+        const which = spawnSync(whichCmd, [agent.command], { encoding: 'utf-8', timeout: 3000 });
         if (which.status !== 0) return false;
-        const binPath = (which.stdout || '').trim();
+        const binPath = (which.stdout || '').trim().split('\n')[0].trim();
         if (!binPath) return false;
-        const check = spawnSync(binPath, ['--version'], { encoding: 'utf-8', timeout: 10000 });
+        const check = spawnSync(binPath, ['--version'], { encoding: 'utf-8', timeout: 10000, shell: isWindows });
         return check.status === 0 && (check.stdout || '').trim().length > 0;
       } catch {
         return false;

package/lib/speech.js

@@ -29,7 +29,14 @@ const POCKET_TTS_VOICES = [
 const PREDEFINED_IDS = new Set(POCKET_TTS_VOICES.filter(v => v.id !== 'default').map(v => v.id));
 const POCKET_PORT = 8787;
 
-const needsPatch = !serverTTS.getVoices(EXTRA_VOICE_DIRS).some(v => v.id === 'alba' && !v.isCustom);
+function safeGetVoices(extraDirs) {
+  if (typeof serverTTS.getVoices === 'function') {
+    return serverTTS.getVoices(extraDirs || []);
+  }
+  return [];
+}
+
+const needsPatch = !safeGetVoices(EXTRA_VOICE_DIRS).some(v => v.id === 'alba' && !v.isCustom);
 
 function synthesizeDirect(text, voiceId) {
   const voicePath = serverTTS.findVoiceFile(voiceId, EXTRA_VOICE_DIRS);
@@ -96,7 +103,7 @@ function synthesizeStream(text, voiceId) {
 }
 
 function getVoices() {
-  const upstream = serverTTS.getVoices(EXTRA_VOICE_DIRS);
+  const upstream = safeGetVoices(EXTRA_VOICE_DIRS);
   const custom = upstream.filter(v => v.isCustom);
   return [...POCKET_TTS_VOICES, ...custom];
 }
@@ -116,6 +123,10 @@ function getStatus() {
 }
 
 function preloadTTS() {
+  if (typeof serverTTS.findVoiceFile !== 'function' || typeof serverTTS.start !== 'function') {
+    console.log('[TTS] pocket-tts functions not available');
+    return;
+  }
   const defaultVoice = serverTTS.findVoiceFile('custom_cleetus', EXTRA_VOICE_DIRS) || '/config/voices/cleetus.wav';
   const voicePath = fs.existsSync(defaultVoice) ? defaultVoice : null;
   serverTTS.start(voicePath, {}).then(ok => {
@@ -127,15 +138,15 @@ function preloadTTS() {
 }
 
 function ttsCacheKey(text, voiceId) {
-  return serverTTS.ttsCacheKey(text, voiceId);
+  return typeof serverTTS.ttsCacheKey === 'function' ? serverTTS.ttsCacheKey(text, voiceId) : null;
 }
 
 function ttsCacheGet(key) {
-  return serverTTS.ttsCacheGet(key);
+  return typeof serverTTS.ttsCacheGet === 'function' ? serverTTS.ttsCacheGet(key) : null;
 }
 
 function splitSentences(text) {
-  return serverTTS.splitSentences(text);
+  return typeof serverTTS.splitSentences === 'function' ? serverTTS.splitSentences(text) : [text];
 }
 
 export { transcribe, synthesize, synthesizeStream, getSTT, getStatus, getVoices, preloadTTS, ttsCacheKey, ttsCacheGet, splitSentences };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentgui",
-  "version": "1.0.236",
+  "version": "1.0.238",
   "description": "Multi-agent ACP client with real-time communication",
   "type": "module",
   "main": "server.js",

package/server.js

@@ -38,9 +38,17 @@ const modelDownloadState = {
 async function ensureModelsDownloaded() {
   const { createRequire: cr } = await import('module');
   const r = cr(import.meta.url);
-  const { checkAllFilesExist, downloadModels } = r('sttttsmodels');
+  const models = r('sttttsmodels');
+  const checkAllFilesExist = models.checkAllFilesExist;
+  const downloadModels = models.downloadModels;
   
-  if (checkAllFilesExist()) {
+  if (checkAllFilesExist && checkAllFilesExist()) {
+    modelDownloadState.complete = true;
+    return true;
+  }
+  
+  if (!downloadModels) {
+    console.log('[MODELS] Download function not available, skipping');
     modelDownloadState.complete = true;
     return true;
   }
@@ -1260,7 +1268,8 @@ const server = http.createServer(async (req, res) => {
       delete childEnv.PORT;
       delete childEnv.BASE_URL;
       delete childEnv.HOT_RELOAD;
-      const child = spawn('npm', ['run', script], { cwd: wd, stdio: ['ignore', 'pipe', 'pipe'], detached: true, env: childEnv });
+      const isWindows = os.platform() === 'win32';
+      const child = spawn('npm', ['run', script], { cwd: wd, stdio: ['ignore', 'pipe', 'pipe'], detached: true, env: childEnv, shell: isWindows });
       activeScripts.set(conversationId, { process: child, script, startTime: Date.now() });
       broadcastSync({ type: 'script_started', conversationId, script, timestamp: Date.now() });
 
@@ -1508,7 +1517,8 @@ const server = http.createServer(async (req, res) => {
 
       const child = spawn(authCmd.cmd, authCmd.args, {
         stdio: ['pipe', 'pipe', 'pipe'],
-        env: { ...process.env, FORCE_COLOR: '1' }
+        env: { ...process.env, FORCE_COLOR: '1' },
+        shell: os.platform() === 'win32'
       });
       activeScripts.set(conversationId, { process: child, script: 'auth-' + agentId, startTime: Date.now() });
       broadcastSync({ type: 'script_started', conversationId, script: 'auth-' + agentId, agentId, timestamp: Date.now() });
@@ -1721,12 +1731,14 @@ const server = http.createServer(async (req, res) => {
         return;
       }
       try {
-        execSync(`git clone https://github.com/${repo}.git`, {
+        const isWindows = os.platform() === 'win32';
+        execSync('git clone https://github.com/' + repo + '.git', {
           cwd: cloneDir,
           encoding: 'utf-8',
           timeout: 120000,
           stdio: ['pipe', 'pipe', 'pipe'],
-          env: { ...process.env, GIT_TERMINAL_PROMPT: '0' }
+          env: { ...process.env, GIT_TERMINAL_PROMPT: '0' },
+          shell: isWindows
         });
         sendJSON(req, res, 200, { ok: true, repo, path: targetPath, name: repoName });
       } catch (err) {
@@ -1756,7 +1768,8 @@ const server = http.createServer(async (req, res) => {
 
     if (pathOnly === '/api/git/check-remote-ownership' && req.method === 'GET') {
       try {
-        const result = execSync('git remote get-url origin 2>/dev/null', { encoding: 'utf-8', cwd: STARTUP_CWD });
+        const isWindows = os.platform() === 'win32';
+        const result = execSync('git remote get-url origin' + (isWindows ? '' : ' 2>/dev/null'), { encoding: 'utf-8', cwd: STARTUP_CWD, shell: isWindows });
         const remoteUrl = result.trim();
         const statusResult = execSync('git status --porcelain', { encoding: 'utf-8', cwd: STARTUP_CWD });
         const hasChanges = statusResult.trim().length > 0;
@@ -1770,7 +1783,8 @@ const server = http.createServer(async (req, res) => {
 
     if (pathOnly === '/api/git/push' && req.method === 'POST') {
       try {
-        execSync('git add -A && git commit -m "Auto-commit" && git push', { encoding: 'utf-8', cwd: STARTUP_CWD });
+        const isWindows = os.platform() === 'win32';
+        execSync('git add -A && git commit -m "Auto-commit" && git push', { encoding: 'utf-8', cwd: STARTUP_CWD, shell: isWindows });
         sendJSON(req, res, 200, { success: true });
       } catch (err) {
         sendJSON(req, res, 500, { error: err.message });
@@ -1784,7 +1798,9 @@ const server = http.createServer(async (req, res) => {
       const expandedPath = decodedPath.startsWith('~') ?
         decodedPath.replace('~', os.homedir()) : decodedPath;
       const normalizedPath = path.normalize(expandedPath);
-      if (!normalizedPath.startsWith('/') || normalizedPath.includes('..')) {
+      const isWindows = os.platform() === 'win32';
+      const isAbsolute = isWindows ? /^[A-Za-z]:[\\\/]/.test(normalizedPath) : normalizedPath.startsWith('/');
+      if (!isAbsolute || normalizedPath.includes('..')) {
         res.writeHead(403); res.end('Forbidden'); return;
       }
       try {

package/static/js/conversations.js

@@ -4,6 +4,15 @@
  * Includes folder browser for selecting working directory on new conversation
  */
 
+function pathSplit(p) {
+  return p.split(/[\/\\]/).filter(Boolean);
+}
+
+function pathBasename(p) {
+  const parts = pathSplit(p);
+  return parts.length ? parts.pop() : '';
+}
+
 class ConversationManager {
   constructor() {
     this.conversations = [];
@@ -176,7 +185,8 @@ class ConversationManager {
         li.innerHTML = `<span class="folder-list-item-icon">&#128193;</span><span class="folder-list-item-name">${this.escapeHtml(folder.name)}</span>`;
         li.addEventListener('click', () => {
           const expandedBase = dirPath === '~' ? this.folderBrowser.homePath : dirPath;
-          const newPath = expandedBase + '/' + folder.name;
+          const separator = expandedBase.includes('\\') ? '\\' : '/';
+          const newPath = expandedBase + separator + folder.name;
           this.loadFolders(newPath);
         });
         this.folderBrowser.listEl.appendChild(li);
@@ -189,26 +199,31 @@ class ConversationManager {
 
   getParentPath(dirPath) {
     const expanded = dirPath === '~' ? this.folderBrowser.homePath : dirPath;
-    const parts = expanded.split('/').filter(Boolean);
-    if (parts.length <= 1) return '/';
+    const parts = pathSplit(expanded);
+    if (parts.length <= 1) {
+      const separator = expanded.includes('\\') ? '\\' : '/';
+      return separator;
+    }
     parts.pop();
-    return '/' + parts.join('/');
+    const separator = expanded.includes('\\') ? '\\' : '/';
+    return separator + parts.join(separator);
   }
 
   renderBreadcrumb(dirPath) {
     if (!this.folderBrowser.breadcrumbEl) return;
 
     const expanded = dirPath === '~' ? this.folderBrowser.homePath : dirPath;
-    const parts = expanded.split('/').filter(Boolean);
+    const parts = pathSplit(expanded);
+    const separator = expanded.includes('\\') ? '\\' : '/';
 
     let html = '';
-    html += '<span class="folder-breadcrumb-segment" data-path="/">/ </span>';
+    html += `<span class="folder-breadcrumb-segment" data-path="${separator}">${separator} </span>`;
 
     let accumulated = '';
     for (let i = 0; i < parts.length; i++) {
-      accumulated += '/' + parts[i];
+      accumulated += separator + parts[i];
       const isLast = i === parts.length - 1;
-      html += '<span class="folder-breadcrumb-separator">/</span>';
+      html += `<span class="folder-breadcrumb-separator">${separator}</span>`;
       html += `<span class="folder-breadcrumb-segment${isLast ? '' : ''}" data-path="${this.escapeHtml(accumulated)}">${this.escapeHtml(parts[i])}</span>`;
     }
 
@@ -227,7 +242,7 @@ class ConversationManager {
     const expanded = currentPath === '~' ? this.folderBrowser.homePath : currentPath;
     this.closeFolderBrowser();
 
-    const dirName = expanded.split('/').filter(Boolean).pop() || 'root';
+    const dirName = pathBasename(expanded) || 'root';
     window.dispatchEvent(new CustomEvent('create-new-conversation', {
       detail: { workingDirectory: expanded, title: dirName }
     }));
@@ -398,7 +413,7 @@ class ConversationManager {
     const timestamp = conv.created_at ? new Date(conv.created_at).toLocaleDateString() : 'Unknown';
     const agent = this.getAgentDisplayName(conv.agentType);
     const modelLabel = conv.model ? ` (${conv.model})` : '';
-    const wd = conv.workingDirectory ? conv.workingDirectory.split('/').pop() : '';
+    const wd = conv.workingDirectory ? pathBasename(conv.workingDirectory) : '';
     const metaParts = [agent + modelLabel, timestamp];
     if (wd) metaParts.push(wd);
 

package/static/js/streaming-renderer.js

@@ -4,6 +4,15 @@
  * for Claude Code streaming execution display
  */
 
+function pathSplit(p) {
+  return p.split(/[\/\\]/).filter(Boolean);
+}
+
+function pathBasename(p) {
+  const parts = pathSplit(p);
+  return parts.length ? parts.pop() : '';
+}
+
 class StreamingRenderer {
   constructor(config = {}) {
     // Configuration
@@ -523,7 +532,7 @@ class StreamingRenderer {
    */
   renderFilePath(filePath) {
     if (!filePath) return '';
-    const parts = filePath.split('/');
+    const parts = pathSplit(filePath);
     const fileName = parts.pop();
     const dir = parts.join('/');
     return `<div class="tool-param-file"><span class="file-icon">&#128196;</span>${dir ? `<span class="file-dir">${this.escapeHtml(dir)}/</span>` : ''}<span class="file-name">${this.escapeHtml(fileName)}</span></div>`;
@@ -660,18 +669,16 @@ class StreamingRenderer {
   getToolUseTitle(toolName, input) {
     const normalizedName = toolName.replace(/^mcp__[^_]+__/, '');
     if (normalizedName === 'Edit' && input.file_path) {
-      const parts = input.file_path.split('/');
+      const parts = pathSplit(input.file_path);
       const fileName = parts.pop();
       const dir = parts.slice(-2).join('/');
       return dir ? `${dir}/${fileName}` : fileName;
     }
     if (normalizedName === 'Read' && input.file_path) {
-      const parts = input.file_path.split('/');
-      return parts.pop();
+      return pathBasename(input.file_path);
     }
     if (normalizedName === 'Write' && input.file_path) {
-      const parts = input.file_path.split('/');
-      return parts.pop();
+      return pathBasename(input.file_path);
     }
     if (normalizedName === 'Bash' || normalizedName === 'bash') {
       const cmd = input.command || input.commands || '';
@@ -684,7 +691,7 @@ class StreamingRenderer {
       try { return new URL(input.url).hostname; } catch (e) { return input.url.substring(0, 40); }
     }
     if (normalizedName === 'WebSearch' && input.query) return input.query.substring(0, 50);
-    if (input.file_path) return input.file_path.split('/').pop();
+    if (input.file_path) return pathBasename(input.file_path);
     if (input.command) {
       const c = typeof input.command === 'string' ? input.command : JSON.stringify(input.command);
       return c.length > 50 ? c.substring(0, 47) + '...' : c;
@@ -748,11 +755,14 @@ class StreamingRenderer {
     }
 
     const lines = trimmed.split('\n');
-    const allFilePaths = lines.length > 1 && lines.every(l => l.trim() === '' || l.trim().startsWith('/'));
+    const allFilePaths = lines.length > 1 && lines.every(l => {
+      const t = l.trim();
+      return t === '' || t.startsWith('/') || /^[A-Za-z]:[\\\/]/.test(t);
+    });
     if (allFilePaths && lines.filter(l => l.trim()).length > 0) {
       const fileHtml = lines.filter(l => l.trim()).map(l => {
         const p = l.trim();
-        const parts = p.split('/');
+        const parts = pathSplit(p);
         const name = parts.pop();
         const dir = parts.join('/');
         return `<div style="display:flex;align-items:center;gap:0.375rem;padding:0.1875rem 0;font-family:'Monaco','Menlo','Ubuntu Mono',monospace;font-size:0.75rem"><span style="opacity:0.5">&#128196;</span><span style="color:var(--color-text-secondary)">${this.escapeHtml(dir)}/</span><span style="font-weight:600">${this.escapeHtml(name)}</span></div>`;
@@ -785,7 +795,8 @@ class StreamingRenderer {
         const lines = data.split('\n').length;
         return `<div style="font-family:'Monaco','Menlo','Ubuntu Mono',monospace;font-size:0.75rem;white-space:pre-wrap;word-break:break-all;max-height:200px;overflow-y:auto;background:var(--color-bg-code);color:#d1d5db;padding:0.5rem;border-radius:0.375rem;line-height:1.5">${this.escapeHtml(data.substring(0, 1000))}${data.length > 1000 ? '\n... (' + (data.length - 1000) + ' more chars, ' + lines + ' lines)' : ''}</div>`;
       }
-      if (data.startsWith('/') && !data.includes(' ') && data.includes('.')) return this.renderFilePath(data);
+      const looksLikePath = data.startsWith('/') || /^[A-Za-z]:[\\\/]/.test(data);
+      if (looksLikePath && !data.includes(' ') && data.includes('.')) return this.renderFilePath(data);
       return `<span style="color:var(--color-text-primary)">${this.escapeHtml(data)}</span>`;
     }
 
@@ -986,11 +997,14 @@ class StreamingRenderer {
       return html;
     }
 
-    const allFilePaths = lines.length > 1 && lines.every(l => l.trim() === '' || l.trim().startsWith('/'));
+    const allFilePaths = lines.length > 1 && lines.every(l => {
+      const t = l.trim();
+      return t === '' || t.startsWith('/') || /^[A-Za-z]:[\\\/]/.test(t);
+    });
     if (allFilePaths && lines.filter(l => l.trim()).length > 0) {
       const fileHtml = lines.filter(l => l.trim()).map(l => {
         const p = l.trim();
-        const parts = p.split('/');
+        const parts = pathSplit(p);
         const name = parts.pop();
         const dir = parts.join('/');
         return `<div style="display:flex;align-items:center;gap:0.375rem;padding:0.1875rem 0;font-family:'Monaco','Menlo','Ubuntu Mono',monospace;font-size:0.75rem"><span style="opacity:0.5">&#128196;</span><span style="color:var(--color-text-secondary)">${esc(dir)}/</span><span style="font-weight:600">${esc(name)}</span></div>`;
@@ -1103,15 +1117,15 @@ class StreamingRenderer {
 
   static getToolTitle(toolName, input) {
     const n = toolName.replace(/^mcp__[^_]+__/, '');
-    if (n === 'Edit' && input.file_path) { const p = input.file_path.split('/'); const f = p.pop(); const d = p.slice(-2).join('/'); return d ? d+'/'+f : f; }
-    if (n === 'Read' && input.file_path) return input.file_path.split('/').pop();
-    if (n === 'Write' && input.file_path) return input.file_path.split('/').pop();
+    if (n === 'Edit' && input.file_path) { const p = pathSplit(input.file_path); const f = p.pop(); const d = p.slice(-2).join('/'); return d ? d+'/'+f : f; }
+    if (n === 'Read' && input.file_path) return pathBasename(input.file_path);
+    if (n === 'Write' && input.file_path) return pathBasename(input.file_path);
     if ((n === 'Bash' || n === 'bash') && (input.command || input.commands)) { const c = typeof (input.command||input.commands) === 'string' ? (input.command||input.commands) : JSON.stringify(input.command||input.commands); return c.length > 60 ? c.substring(0,57)+'...' : c; }
     if (n === 'Glob' && input.pattern) return input.pattern;
     if (n === 'Grep' && input.pattern) return input.pattern;
     if (n === 'WebFetch' && input.url) { try { return new URL(input.url).hostname; } catch(e) { return input.url.substring(0,40); } }
     if (n === 'WebSearch' && input.query) return input.query.substring(0,50);
-    if (input.file_path) return input.file_path.split('/').pop();
+    if (input.file_path) return pathBasename(input.file_path);
     if (input.command) { const c = typeof input.command === 'string' ? input.command : JSON.stringify(input.command); return c.length > 50 ? c.substring(0,47)+'...' : c; }
     if (input.query) return input.query.substring(0,50);
     return '';
@@ -1134,8 +1148,9 @@ class StreamingRenderer {
       if (data.length > 500) {
         return `<div style="font-family:'Monaco','Menlo','Ubuntu Mono',monospace;font-size:0.75rem;white-space:pre-wrap;word-break:break-all;max-height:200px;overflow-y:auto;background:var(--color-bg-code);color:#d1d5db;padding:0.5rem;border-radius:0.375rem;line-height:1.5">${esc(data.substring(0, 1000))}${data.length > 1000 ? '\n... (' + (data.length - 1000) + ' more chars)' : ''}</div>`;
       }
-      if (data.startsWith('/') && !data.includes(' ') && data.includes('.')) {
-        const parts = data.split('/');
+      const looksLikePath = /^[A-Za-z]:[\\\/]/.test(data) || data.startsWith('/');
+      if (looksLikePath && !data.includes(' ') && data.includes('.')) {
+        const parts = pathSplit(data);
         const name = parts.pop();
         const dir = parts.join('/');
         return `<div style="display:flex;align-items:center;gap:0.375rem;font-family:'Monaco','Menlo','Ubuntu Mono',monospace;font-size:0.8rem"><span style="opacity:0.5">&#128196;</span><span style="color:var(--color-text-secondary)">${esc(dir)}/</span><span style="font-weight:600">${esc(name)}</span></div>`;

package/telemetry-id

@@ -0,0 +1 @@
+814cd4b3-ffd4-44c7-abbd-b2d9e757f0fc