agentvibes 5.2.1 → 5.4.0

package/src/console/tabs/setup-tab.js

@@ -1278,29 +1278,32 @@ export function createSetupTab(screen, services) {
       const phrase = SAMPLE_PHRASES[Math.floor(Math.random() * SAMPLE_PHRASES.length)];
 
       // Route through remote provider if active
+      // Search order: CLAUDE_PROJECT_DIR → cwd → package root → home
       const _remoteProviders = ['ssh-remote', 'agentvibes-receiver'];
       let _activeProvider = '';
       try {
-        const _projectRoot = path.resolve(__dirname, '..', '..');
+        const _pkgRoot = path.resolve(__dirname, '..', '..');
         const _provPaths = [
-          path.join(_projectRoot, '.claude', 'tts-provider.txt'),
+          process.env.CLAUDE_PROJECT_DIR && path.join(process.env.CLAUDE_PROJECT_DIR, '.claude', 'tts-provider.txt'),
+          path.join(process.cwd(), '.claude', 'tts-provider.txt'),
+          path.join(_pkgRoot, '.claude', 'tts-provider.txt'),
           path.join(os.homedir(), '.claude', 'tts-provider.txt'),
-        ];
+        ].filter(Boolean);
         for (const p of _provPaths) {
           if (fs.existsSync(p)) { _activeProvider = fs.readFileSync(p, 'utf8').trim(); break; }
         }
       } catch {}
 
       if (_remoteProviders.includes(_activeProvider)) {
-        const _projectRoot = path.resolve(__dirname, '..', '..');
+        const _hooksBase = process.env.CLAUDE_PROJECT_DIR || process.cwd();
         let rProc;
         if (_isWin) {
-          const _playTts = path.join(_projectRoot, '.claude', 'hooks-windows', 'play-tts.ps1');
+          const _playTts = path.join(_hooksBase, '.claude', 'hooks-windows', 'play-tts.ps1');
           rProc = spawn('powershell', ['-NoProfile', '-ExecutionPolicy', 'Bypass', '-File', _playTts, phrase, voiceId], {
             stdio: 'ignore', detached: false, windowsHide: true, env: _spawnEnv,
           });
         } else {
-          const _playTts = path.join(_projectRoot, '.claude', 'hooks', 'play-tts.sh');
+          const _playTts = path.join(_hooksBase, '.claude', 'hooks', 'play-tts.sh');
           rProc = spawn('bash', [_playTts, phrase, voiceId], {
             stdio: 'ignore', detached: true, env: _spawnEnv,
           });

package/src/console/tabs/voices-tab.js

@@ -868,14 +868,17 @@ export function createVoicesTab(screen, services) {
     _playingVoiceId = null;
 
     // Check if we should route through remote provider (ssh-remote / agentvibes-receiver)
+    // Search order: CLAUDE_PROJECT_DIR (actual project) → cwd → package root → home
     const projectRoot = path.resolve(__dirname, '..', '..');
     const remoteProviders = ['ssh-remote', 'agentvibes-receiver'];
     let activeProvider = '';
     try {
       const providerPaths = [
+        process.env.CLAUDE_PROJECT_DIR && path.join(process.env.CLAUDE_PROJECT_DIR, '.claude', 'tts-provider.txt'),
+        path.join(process.cwd(), '.claude', 'tts-provider.txt'),
         path.join(projectRoot, '.claude', 'tts-provider.txt'),
         path.join(os.homedir(), '.claude', 'tts-provider.txt'),
-      ];
+      ].filter(Boolean);
       for (const p of providerPaths) {
         if (fs.existsSync(p)) { activeProvider = fs.readFileSync(p, 'utf8').trim(); break; }
       }
@@ -884,14 +887,17 @@ export function createVoicesTab(screen, services) {
     if (remoteProviders.includes(activeProvider)) {
       const isWindows = process.platform === 'win32' && !process.env.WSL_DISTRO_NAME;
       const phrase = SAMPLE_PHRASES[Math.floor(Math.random() * SAMPLE_PHRASES.length)];
+      // Resolve play-tts from the actual project (CLAUDE_PROJECT_DIR / cwd),
+      // not the npm package root — hooks live in the user's project dir.
+      const hooksBase = process.env.CLAUDE_PROJECT_DIR || process.cwd();
       let proc;
       if (isWindows) {
-        const playTts = path.join(projectRoot, '.claude', 'hooks-windows', 'play-tts.ps1');
+        const playTts = path.join(hooksBase, '.claude', 'hooks-windows', 'play-tts.ps1');
         proc = spawn('powershell', ['-NoProfile', '-ExecutionPolicy', 'Bypass', '-File', playTts, phrase, voiceId], {
           stdio: 'ignore', detached: false, windowsHide: true, env: _spawnEnv,
         });
       } else {
-        const playTts = path.join(projectRoot, '.claude', 'hooks', 'play-tts.sh');
+        const playTts = path.join(hooksBase, '.claude', 'hooks', 'play-tts.sh');
         proc = spawn('bash', [playTts, phrase, voiceId], {
           stdio: 'ignore', detached: true, env: _spawnEnv,
         });

package/src/installer.js

@@ -132,6 +132,36 @@ function isNativeWindows() {
   return process.platform === 'win32' && !process.env.WSL_DISTRO_NAME;
 }
 
+/**
+ * Wrap an ora spinner with safe fallbacks for all methods.
+ * Ensures compatibility across platforms (Windows, WSL, macOS, Linux) and ora versions.
+ * @param {Object} spinner - An ora spinner instance (or any spinner-like object)
+ * @returns {Object} A spinner proxy with guaranteed methods
+ */
+function createRobustSpinner(spinner) {
+  const safe = (method, fallback) => (...args) => {
+    if (typeof spinner[method] === 'function') {
+      try { spinner[method](...args); } catch { fallback?.(...args); }
+    } else {
+      fallback?.(...args);
+    }
+    return proxy;
+  };
+  const proxy = {
+    start:          safe('start'),
+    stop:           safe('stop'),
+    succeed:        safe('succeed', (t) => { if (t) process.stdout.write(`✓ ${t}\n`); }),
+    fail:           safe('fail',    (t) => { if (t) process.stderr.write(`✗ ${t}\n`); }),
+    warn:           safe('warn',    (t) => { if (t) process.stdout.write(`⚠ ${t}\n`); }),
+    info:           safe('info',    (t) => { if (t) process.stdout.write(`ℹ ${t}\n`); }),
+    stopAndPersist: safe('stopAndPersist'),
+    get text()      { return spinner.text ?? ''; },
+    set text(t)     { try { spinner.text = t; } catch { /* non-TTY */ } },
+    get isSpinning(){ return spinner.isSpinning ?? false; },
+  };
+  return proxy;
+}
+
 /**
  * Get the Piper provider name (always 'piper' on all platforms)
  * @returns {string} The piper provider identifier
@@ -4215,7 +4245,7 @@ async function checkAndInstallPiperWindows(targetDir, options) {
   }
   const piperDir = path.join(localAppData, 'Programs', 'Piper');
   const piperExe = path.join(piperDir, 'piper.exe');
-  const spinner = ora();
+  const spinner = createRobustSpinner(ora());
 
   if (fsSync.existsSync(piperExe)) {
     console.log(chalk.green('✓ Piper TTS is already installed at ' + piperDir + '\n'));
@@ -4844,7 +4874,14 @@ async function updatePersonalityFiles(targetDir, srcPersonalitiesDir) {
  * @returns {Object} Mock spinner object
  */
 function createSilentSpinner() {
-  const s = { start: () => s, succeed: () => s, info: () => s, fail: () => s, warn: () => s, stop: () => s };
+  const s = {
+    start: () => s, succeed: () => s, info: () => s,
+    fail: () => s, warn: () => s, stop: () => s,
+    stopAndPersist: () => s,
+    get text() { return ''; },
+    set text(_) {},
+    get isSpinning() { return false; },
+  };
   return s;
 }
 
@@ -5025,7 +5062,7 @@ function displayUpdateSummary(results) {
  * @param {Object} options - Update options
  */
 async function updateAgentVibes(targetDir, options) {
-  const spinner = ora('Updating AgentVibes...').start();
+  const spinner = createRobustSpinner(ora('Updating AgentVibes...').start());
 
   try {
     // Perform all update operations
@@ -5145,7 +5182,7 @@ async function install(options = {}) {
   const silentSpinner = createSilentSpinner();
 
   console.log('');
-  const spinner = ora('Installing AgentVibes...').start();
+  const spinner = createRobustSpinner(ora('Installing AgentVibes...').start());
 
   try {
     // Create .claude directory structure
@@ -5599,7 +5636,7 @@ program
       console.log(chalk.gray('✓ Auto-confirmed (--yes flag)\n'));
     }
 
-    const spinner = ora('Uninstalling AgentVibes...').start();
+    const spinner = createRobustSpinner(ora('Uninstalling AgentVibes...').start());
 
     try {
       let removedCount = 0;

package/src/services/llm-provider-service.js

@@ -209,6 +209,19 @@ export async function installClaudeMcp(targetDir) {
     await installer.copyBackgroundMusicFiles(targetDir, silentSpinner);
     ensureDefaultLlmConfigSync('claude-code', targetDir);
 
+    // Explicitly write tts-provider.txt so `get_active_provider()` in
+    // provider-manager.sh doesn't silently fall back to "piper".  Without
+    // this, headless servers with no audio device hit a confusing failure
+    // mode where TTS tries to synth locally and fails silently.  Users
+    // can still change the provider via the Setup TUI or slash command.
+    const ttsProviderPath = path.join(targetDir, '.claude', 'tts-provider.txt');
+    try {
+      await fs.access(ttsProviderPath);
+      // Already exists — user has explicitly set a provider, don't clobber
+    } catch {
+      await fs.writeFile(ttsProviderPath, 'piper\n');
+    }
+
     return { success: true, mcpCreated, mcpError };
   } catch (err) {
     return { success: false, error: err.message, mcpError };