agentvibes 4.4.0 → 4.4.1

package/package.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/package.json",
   "name": "agentvibes",
-  "version": "4.4.0",
+  "version": "4.4.1",
   "description": "Now your AI Agents can finally talk back! Professional TTS voice for Claude Code, Claude Desktop (via MCP), and Clawdbot with multi-provider support.",
   "homepage": "https://agentvibes.org",
   "keywords": [

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

@@ -37,8 +37,18 @@ function _resolvePiperBin() {
     const lad = process.env.LOCALAPPDATA ||
       (process.env.USERPROFILE ? path.join(process.env.USERPROFILE, 'AppData', 'Local') : null);
     if (lad) {
+      // Standalone binary install
       const exe = path.join(lad, 'Programs', 'Piper', 'piper.exe');
       if (fs.existsSync(exe)) return exe;
+      // pip-installed piper (Python Scripts directory)
+      const pyScripts = path.join(lad, 'Programs', 'Python');
+      try {
+        const pyDirs = fs.readdirSync(pyScripts).filter(d => d.startsWith('Python'));
+        for (const d of pyDirs) {
+          const pipExe = path.join(pyScripts, d, 'Scripts', 'piper.exe');
+          if (fs.existsSync(pipExe)) return pipExe;
+        }
+      } catch { /* no Python installs */ }
     }
   }
   return 'piper';
@@ -906,13 +916,18 @@ export function createSettingsTab(screen, services) {
       if (!_samplePlaying) { try { fs.unlinkSync(tempWav); } catch {} return; }
       if (code !== 0) {
         _killSample(); playBtn.setContent('▶ Play'); _focusButton(playBtn);
+        _showNotice(screen, 'Voice synthesis failed — check voice model');
         try { fs.unlinkSync(tempWav); } catch {}
         return;
       }
       playBtn.setContent('■ Stop');
       screen.render();
       const _wavPlayer2 = detectWavPlayer(_sampleEnv);
-      if (!_wavPlayer2) { _stopSpinner(); _killSample(); playBtn.setContent('▶ Play'); screen.render(); return; }
+      if (!_wavPlayer2) {
+        _stopSpinner(); _killSample(); playBtn.setContent('▶ Play');
+        _showNotice(screen, 'No audio player found — install ffplay, sox, or mpv');
+        screen.render(); return;
+      }
       const playProc = spawn(_wavPlayer2.bin, _wavPlayer2.args(tempWav), _spawnOpts(_sampleEnv));
       _sampleProcess = playProc;
       const _done = () => { _killSample(); playBtn.setContent('▶ Play'); _focusButton(playBtn); try { fs.unlinkSync(tempWav); } catch {} };
@@ -1027,23 +1042,59 @@ export function createSettingsTab(screen, services) {
       // Piper (default): pipe text via stdin
       _startSpinner(playBtn, 'Synthesizing…');
       const voiceId   = providerService.getActiveVoiceId();
-      if (!voiceId) { _stopSpinner(); _killSample(); playBtn.setContent('▶ Play'); screen.render(); return; }
+      if (!voiceId) {
+        _stopSpinner(); _killSample(); playBtn.setContent('▶ Play');
+        _showNotice(screen, 'No voice selected — choose a voice first');
+        screen.render(); return;
+      }
       const _ms2 = parseMultiSpeaker(voiceId);
       const voicePath = path.resolve(PIPER_VOICES_DIR, _ms2.model + '.onnx');
       const safeBase  = path.resolve(PIPER_VOICES_DIR);
       if (!voicePath.startsWith(safeBase + path.sep) && voicePath !== safeBase) {
-        _stopSpinner(); _killSample(); playBtn.setContent('▶ Play'); screen.render(); return;
+        _stopSpinner(); _killSample(); playBtn.setContent('▶ Play');
+        _showNotice(screen, 'Invalid voice path');
+        screen.render(); return;
+      }
+      const piperBin2 = _resolvePiperBin();
+      if (piperBin2 === 'piper') {
+        // Bare command — verify it exists in PATH before spawning
+        const whichCmd = _IS_WINDOWS ? 'where' : 'which';
+        const whichResult = spawnSync(whichCmd, [_IS_WINDOWS ? 'piper.exe' : 'piper'], { stdio: 'pipe', env: _sampleEnv });
+        if (whichResult.status !== 0) {
+          _stopSpinner(); _killSample(); playBtn.setContent('▶ Play');
+          _showNotice(screen, 'Piper not installed — run the installer or: pip install piper-tts');
+          _focusButton(playBtn); screen.render(); return;
+        }
       }
       const _piperArgs2 = ['--model', voicePath, '--output_file', tempWav];
       if (_ms2.speakerId != null) _piperArgs2.push('--speaker', String(_ms2.speakerId));
-      const piper = spawn(_resolvePiperBin(), _piperArgs2, {
-        stdio: ['pipe', 'ignore', 'ignore'], detached: !_IS_WINDOWS, windowsHide: true, env: _sampleEnv,
+      const piper = spawn(piperBin2, _piperArgs2, {
+        stdio: ['pipe', 'ignore', 'pipe'], detached: !_IS_WINDOWS, windowsHide: true, env: _sampleEnv,
       });
+      let _piperStderr = '';
+      piper.stderr.on('data', (d) => { _piperStderr += d.toString(); });
       piper.stdin.write(phrase + '\n');
       piper.stdin.end();
       _sampleProcess = piper;
-      piper.on('exit', _onSynthDone);
-      piper.on('error', () => { _stopSpinner(); _killSample(); playBtn.setContent('▶ Play'); _focusButton(playBtn); });
+      piper.on('exit', (code) => {
+        if (code !== 0 && _piperStderr) {
+          // Python tracebacks: actual error is the LAST non-empty line
+          const lines = _piperStderr.split('\n').map(l => l.trim()).filter(Boolean);
+          const errLine = lines[lines.length - 1] || lines[0] || 'unknown error';
+          _stopSpinner();
+          if (!_samplePlaying) { try { fs.unlinkSync(tempWav); } catch {} return; }
+          _killSample(); playBtn.setContent('▶ Play'); _focusButton(playBtn);
+          _showNotice(screen, errLine.length > 100 ? errLine.substring(0, 97) + '…' : errLine);
+          try { fs.unlinkSync(tempWav); } catch {}
+          return;
+        }
+        _onSynthDone(code);
+      });
+      piper.on('error', (e) => {
+        _stopSpinner(); _killSample(); playBtn.setContent('▶ Play');
+        _showNotice(screen, `Piper failed: ${e.message}`);
+        _focusButton(playBtn);
+      });
     }
   });
   playBtn.top = 7;

package/src/installer.js

@@ -5139,6 +5139,19 @@ Troubleshooting:
     }
     await fs.writeFile(voiceConfigPath, defaultVoice);
 
+    // Sync voice + provider to global .agentvibes/config.json so TUI finds them
+    // regardless of which directory it's launched from
+    const globalAvDir = path.join(process.env.HOME || process.env.USERPROFILE, '.agentvibes');
+    try {
+      await fs.mkdir(globalAvDir, { recursive: true });
+      const globalCfgPath = path.join(globalAvDir, 'config.json');
+      let globalCfg = {};
+      try { globalCfg = JSON.parse(await fs.readFile(globalCfgPath, 'utf8')); } catch { /* new file */ }
+      globalCfg.voice = defaultVoice;
+      globalCfg.provider = selectedProvider;
+      await fs.writeFile(globalCfgPath, JSON.stringify(globalCfg, null, 2), { mode: 0o600 });
+    } catch { /* best-effort global sync */ }
+
     // Detect and migrate old configuration
     await detectAndMigrateOldConfig(targetDir, silentSpinner);
 

package/src/services/provider-service.js

@@ -39,6 +39,7 @@ export class ProviderService {
    */
   setActiveProvider(provider) {
     this._config.set('provider', provider);
+    this._config.setGlobal('provider', provider);
     this._syncProviderFile(provider);
   }
 
@@ -95,19 +96,29 @@ export class ProviderService {
 
   /**
    * Returns the currently active voice ID from config.
-   * Defaults to 'en_US-amy-medium' if not configured.
-   * @returns {string}
+   * Falls back to first installed voice if not configured.
+   * @returns {string|null}
    */
   getActiveVoiceId() {
-    return this._config.getConfig().voice ?? 'en_US-amy-medium';
+    const voice = this._config.getConfig().voice;
+    if (voice) return voice;
+    // Detect first installed voice instead of hardcoding a default that may not exist
+    const voicesDir = path.join(os.homedir(), '.claude', 'piper-voices');
+    try {
+      const models = fs.readdirSync(voicesDir).filter(f => f.endsWith('.onnx'));
+      if (models.length > 0) return models[0].replace(/\.onnx$/, '');
+    } catch { /* dir may not exist */ }
+    return null;
   }
 
   /**
    * Sets the active voice ID in config.
+   * Writes to both project (if exists) and global config for portability.
    * @param {string} voiceId
    */
   setActiveVoice(voiceId) {
     this._config.set('voice', voiceId);
+    this._config.setGlobal('voice', voiceId);
   }
 
   // ---------------------------------------------------------------------------