agentvibes 5.10.1 → 5.11.0

package/src/commands/install-mcp.js

@@ -571,7 +571,7 @@ async function downloadPiperVoices(agentVibesDir) {
         short: 'LibriTTS 900'
       },
       {
-        name: chalk.cyan('Full pack') + chalk.gray(' — 11 voices including LibriTTS (~250 MB, all BMAD agent voices)'),
+        name: chalk.cyan('Full pack') + chalk.gray(' — 17 voices including LibriTTS (~360 MB, all voices-tab curated + BMAD agent voices)'),
         value: 'full',
         short: 'Full pack'
       },
@@ -618,8 +618,8 @@ async function downloadPiperVoices(agentVibesDir) {
       console.log(chalk.yellow('\n⚠️  Download failed — try later with: /agent-vibes:add\n'));
     }
   } else if (voiceChoice === 'full') {
-    console.log(chalk.cyan('\n📥 Downloading full voice pack (~250 MB)...\n'));
-    console.log(chalk.gray('   This includes all BMAD agent voices plus LibriTTS 900 speakers.\n'));
+    console.log(chalk.cyan('\n📥 Downloading full voice pack (~360 MB)...\n'));
+    console.log(chalk.gray('   This includes all voices-tab curated voices, BMAD agent voices, and LibriTTS 900 speakers.\n'));
     try {
       if (fs.existsSync(fullPackScript)) {
         execFileSync('bash', [fullPackScript, '--yes'], { // NOSONAR

package/src/console/app.js

@@ -9,7 +9,8 @@
 
 import blessed from 'blessed';
 import path from 'node:path';
-import { readFileSync } from 'node:fs';
+import { readFileSync, existsSync } from 'node:fs';
+import os from 'node:os';
 import { fileURLToPath } from 'node:url';
 import { spawnSync, execFileSync } from 'node:child_process';
 import { NavigationService, TAB_ORDER } from '../services/navigation-service.js';
@@ -630,21 +631,36 @@ export class AgentVibesConsole {
       try { execFileSync('which', [bin], { stdio: 'ignore', timeout: 2000 }); return true; } // NOSONAR
       catch { return false; }
     };
+    const _pyHas = (mod) => {
+      try { execFileSync('python3', ['-c', `import ${mod}`], { stdio: 'ignore', timeout: 3000 }); return true; } // NOSONAR
+      catch { return false; }
+    };
+    const _elKeySet = () => {
+      if (process.env.ELEVENLABS_API_KEY) return true;
+      try {
+        const kf = path.join(os.homedir(), '.agentvibes', 'elevenlabs-key.txt');
+        return existsSync(kf) && readFileSync(kf, 'utf8').trim().length > 0;
+      } catch { return false; }
+    };
     const detected = {
-      piper:   _has('piper'),
-      soprano: _has('soprano'),
-      sapi:    process.platform === 'win32',
-      macos:   process.platform === 'darwin' && _has('say'),
+      piper:       _has('piper'),
+      kokoro:      _pyHas('kokoro'),
+      elevenlabs:  _elKeySet(),
+      soprano:     _has('soprano'),
+      sapi:        process.platform === 'win32',
+      macos:       process.platform === 'darwin' && _has('say'),
     };
 
     // Build provider status badges:  ● Name  (green if detected, grey if not)
     const on  = (label) => `{green-fg}●{/green-fg} ${label}`;
     const off = (label) => `{#546e7a-fg}● ${label}{/#546e7a-fg}`;
     const badges = [
-      detected.piper   ? on('Piper')        : off('Piper'),
-      detected.soprano ? on('Soprano')      : off('Soprano'),
-      detected.sapi    ? on('Windows SAPI') : off('Windows SAPI'),
-      detected.macos   ? on('Mac Say')      : off('Mac Say'),
+      detected.piper       ? on('Piper')       : off('Piper'),
+      detected.kokoro      ? on('Kokoro')       : off('Kokoro'),
+      detected.elevenlabs  ? on('ElevenLabs')   : off('ElevenLabs'),
+      detected.soprano     ? on('Soprano')      : off('Soprano'),
+      detected.sapi        ? on('Windows SAPI') : off('Windows SAPI'),
+      detected.macos       ? on('Mac Say')      : off('Mac Say'),
     ].join('  ');
 
     const footer = blessed.box({

package/src/console/audio-env.js

@@ -8,7 +8,7 @@
 import fs from 'node:fs';
 import path from 'node:path';
 import os from 'node:os';
-import { spawnSync } from 'node:child_process';
+import { spawn, spawnSync } from 'node:child_process';
 
 /**
  * Build a spawn environment with correct PULSE_SERVER handling.
@@ -151,6 +151,90 @@ export function detectMp3Player(env) {
   return _detect(MP3_PLAYERS, env);
 }
 
+/**
+ * Returns true when running headless with PulseAudio over TCP.
+ * SDL2-based players (ffplay) fail in this configuration with exit 123.
+ */
+function _isHeadlessPulse(env) {
+  return !env.DISPLAY && typeof env.PULSE_SERVER === 'string' && env.PULSE_SERVER.startsWith('tcp:');
+}
+
+/**
+ * Spawn an MP3 player for the given track path.
+ *
+ * On headless servers with PulseAudio TCP, SDL2-based players (ffplay) fail
+ * because DISPLAY is not set. In that case uses ffmpeg→pacat pipe.
+ *
+ * Returns a process-like object with:
+ *   .pid    — primary process PID
+ *   .kill() — stops playback, killing all spawned processes
+ *   .on(event, cb) — supports 'exit' and 'error'
+ *
+ * Returns null if no suitable player or required binaries are unavailable.
+ *
+ * @param {string} trackPath - Absolute path to the MP3 file
+ * @param {Object} [env]     - Environment (defaults to buildAudioEnv())
+ * @returns {{ pid: number, kill: Function, on: Function }|null}
+ */
+export function spawnMp3Player(trackPath, env) {
+  env = env ?? buildAudioEnv();
+
+  if (_isHeadlessPulse(env)) {
+    // Check required binaries are present.
+    // S4036: standard system media tools resolved via the user's local PATH on
+    // their own machine — no untrusted PATH, absolute paths aren't portable
+    // across distros/Homebrew/Nix. Risk accepted.
+    if (spawnSync('which', ['ffmpeg'], { stdio: 'ignore', env }).status !== 0) return null; // NOSONAR
+    if (spawnSync('which', ['pacat'],  { stdio: 'ignore', env }).status !== 0) return null; // NOSONAR
+
+    const ff = spawn('ffmpeg', // NOSONAR — see S4036 note above
+      ['-i', trackPath, '-f', 's16le', '-ac', '2', '-ar', '44100', 'pipe:1', '-loglevel', 'quiet'],
+      { stdio: ['ignore', 'pipe', 'ignore'], env },
+    );
+    const pa = spawn('pacat', // NOSONAR — see S4036 note above
+      ['--playback', '--format=s16le', '--channels=2', '--rate=44100'],
+      { stdio: ['pipe', 'ignore', 'ignore'], env },
+    );
+    ff.stdout.on('error', () => {});
+    pa.stdin.on('error', () => {});
+    ff.stdout.pipe(pa.stdin);
+
+    const _listeners = { exit: [], error: [] };
+    pa.on('exit', (code, sig) => _listeners.exit.forEach(cb => cb(code, sig)));
+    ff.on('error', err => _listeners.error.forEach(cb => cb(err)));
+    pa.on('error', err => _listeners.error.forEach(cb => cb(err)));
+
+    return {
+      pid: ff.pid,
+      kill() { try { ff.kill(); } catch {} try { pa.kill(); } catch {} },
+      on(event, cb) { if (_listeners[event]) _listeners[event].push(cb); },
+    };
+  }
+
+  const player = detectMp3Player(env);
+  if (!player) return null;
+
+  const isWin = process.platform === 'win32' && !process.env.WSL_DISTRO_NAME;
+  const proc = spawn(player.bin, player.args(trackPath), {
+    stdio: 'ignore', detached: !isWin, windowsHide: true, env,
+  });
+
+  return {
+    pid: proc.pid,
+    kill() {
+      try {
+        if (isWin) {
+          spawn('taskkill', ['/F', '/T', '/PID', String(proc.pid)], // NOSONAR
+            { stdio: 'ignore', windowsHide: true });
+        } else {
+          process.kill(-proc.pid, 'SIGTERM');
+        }
+      } catch (e) { if (e.code !== 'ESRCH') { /* ignore */ } }
+    },
+    on(event, cb) { proc.on(event, cb); },
+  };
+}
+
 /**
  * Detect the best available WAV player.
  * On Windows, uses PowerShell SoundPlayer as built-in fallback.

package/src/console/brand-colors.js

@@ -1,13 +1,13 @@
-/**
- * AgentVibes TUI — Brand Color Constants
- *
- * Single source of truth for the two primary brand colors.
- * Change BRAND_PINK or BRAND_BLUE here to update every modal title,
- * button, and the "Vibes" logotype across the entire TUI.
- */
-
-/** Magenta-pink used for modal titles and the "Vibes" logotype. */
-export const BRAND_PINK = '#f06292';  // Light magenta — Pink 300
-
-/** Indigo blue used for default button backgrounds and primary accents. */
-export const BRAND_BLUE = '#3949ab';
+/**
+ * AgentVibes TUI — Brand Color Constants
+ *
+ * Single source of truth for the two primary brand colors.
+ * Change BRAND_PINK or BRAND_BLUE here to update every modal title,
+ * button, and the "Vibes" logotype across the entire TUI.
+ */
+
+/** Magenta-pink used for modal titles and the "Vibes" logotype. */
+export const BRAND_PINK = '#f06292';  // Light magenta — Pink 300
+
+/** Indigo blue used for default button backgrounds and primary accents. */
+export const BRAND_BLUE = '#3949ab';

package/src/console/constants/personalities.js

@@ -1,44 +1,44 @@
-/**
- * AgentVibes — Canonical personality constants.
- *
- * Single source of truth for personality names and their associated emoji
- * glyphs.  All TUI modules (settings-tab, agents-tab, personality-picker …)
- * import from here; src/installer.js maintains its own copy because it
- * predates the TUI and uses a different module-load path.
- *
- * Exported:
- *   PERSONALITY_EMOJIS  — Map of personality name → emoji string
- *   PERSONALITIES       — Ordered array of personality names (canonical order
- *                         used for picker lists)
- */
-
-export const PERSONALITY_EMOJIS = Object.freeze({
-  angry:        '😠',
-  annoying:     '😤',
-  crass:        '🤬',
-  dramatic:     '🎭',
-  'dry-humor':  '😐',
-  flirty:       '😘',
-  funny:        '😂',
-  grandpa:      '👴',
-  millennial:   '🙄',
-  moody:        '😒',
-  none:         '😊',
-  normal:       '😊',
-  pirate:       '⚓',
-  poetic:       '📜',
-  professional: '👔',
-  rapper:       '🎤',
-  robot:        '🤖',
-  sarcastic:    '😏',
-  sassy:        '💁',
-  'surfer-dude':'🏄',
-  zen:          '🧘',
-});
-
-export const PERSONALITIES = Object.freeze([
-  'none', 'angry', 'annoying', 'crass', 'dramatic', 'dry-humor',
-  'flirty', 'funny', 'grandpa', 'millennial', 'moody', 'normal',
-  'pirate', 'poetic', 'professional', 'rapper', 'robot', 'sarcastic',
-  'sassy', 'surfer-dude', 'zen',
-]);
+/**
+ * AgentVibes — Canonical personality constants.
+ *
+ * Single source of truth for personality names and their associated emoji
+ * glyphs.  All TUI modules (settings-tab, agents-tab, personality-picker …)
+ * import from here; src/installer.js maintains its own copy because it
+ * predates the TUI and uses a different module-load path.
+ *
+ * Exported:
+ *   PERSONALITY_EMOJIS  — Map of personality name → emoji string
+ *   PERSONALITIES       — Ordered array of personality names (canonical order
+ *                         used for picker lists)
+ */
+
+export const PERSONALITY_EMOJIS = Object.freeze({
+  angry:        '😠',
+  annoying:     '😤',
+  crass:        '🤬',
+  dramatic:     '🎭',
+  'dry-humor':  '😐',
+  flirty:       '😘',
+  funny:        '😂',
+  grandpa:      '👴',
+  millennial:   '🙄',
+  moody:        '😒',
+  none:         '😊',
+  normal:       '😊',
+  pirate:       '⚓',
+  poetic:       '📜',
+  professional: '👔',
+  rapper:       '🎤',
+  robot:        '🤖',
+  sarcastic:    '😏',
+  sassy:        '💁',
+  'surfer-dude':'🏄',
+  zen:          '🧘',
+});
+
+export const PERSONALITIES = Object.freeze([
+  'none', 'angry', 'annoying', 'crass', 'dramatic', 'dry-humor',
+  'flirty', 'funny', 'grandpa', 'millennial', 'moody', 'normal',
+  'pirate', 'poetic', 'professional', 'rapper', 'robot', 'sarcastic',
+  'sassy', 'surfer-dude', 'zen',
+]);

package/src/console/navigation.js

@@ -40,8 +40,12 @@ const KEY_TO_TAB = {
 export function setupNavigation(screen, navigationService, focusMainTabBar) {
   // Tab switching shortcuts — one handler per key (both cases)
   // When a modal is open, force-close all modals then switch tabs.
+  // Exception: if already on the target tab, pressing its shortcut is a no-op —
+  // the user is trying to do something within the tab (e.g. 'I' in setup means
+  // "install", not "re-navigate to setup and close all my pickers").
   for (const [key, tabId] of Object.entries(KEY_TO_TAB)) {
     screen.key([key], () => {
+      if (navigationService.getActiveTab() === tabId) return;
       if (navigationService.isModalOpen()) {
         navigationService.forceCloseAll();
         setTimeout(() => navigationService.switchTab(tabId), 0);

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

@@ -20,6 +20,7 @@ import {
   getFavorites, getThumbsDown, toggleThumbsUp, toggleThumbsDown,
 } from './voices-tab.js';
 import { buildAudioEnv, detectWavPlayer, detectRemoteLlm } from '../audio-env.js';
+import { voicesForProvider } from '../../services/provider-voice-catalog.js';
 import { destroyList } from '../widgets/destroy-list.js';
 import { BRAND_PINK } from '../brand-colors.js';
 import { t } from '../../i18n/strings.js';
@@ -642,7 +643,7 @@ ${_tl('bmadDesc')}
     const FIELDS = [
       { key: 'voice',        label: 'Voice',       getValue: () => draft.voice || '(global default)' },
       { key: 'pretext',      label: 'Pretext',     getValue: () => draft.pretext || '(default)' },
-      { key: 'reverbPreset', label: 'Reverb',      getValue: () => formatReverbState(draft.reverbPreset) },
+      { key: 'reverbPreset', label: 'Audio Effects', getValue: () => formatReverbState(draft.reverbPreset) },
       { key: 'personality',  label: 'Personality', getValue: () => {
         const p = draft.personality;
         const emoji = PERSONALITY_EMOJIS[p] || '';
@@ -1494,13 +1495,18 @@ ${_tl('bmadDesc')}
   }
 
   function _autoAssignVoices() {
-    const installed = scanInstalledVoices();
+    // Pull the pool from whichever provider is currently active, not always Piper.
+    // Agent profiles store only a voice id (the active provider is global), so
+    // switching the provider and re-running auto-assign rebinds every agent to a
+    // voice from that provider (Kokoro, ElevenLabs, soprano, …).
+    const provider = providerService?.getActiveProvider?.() ?? 'piper';
+    const installed = voicesForProvider(provider, { scanInstalledVoices, getVoiceMeta });
     if (installed.length === 0) return false;
 
-    // Separate voices by gender
-    const femaleVoices = _shuffleArray(installed.filter(v => getVoiceMeta(v).gender === 'Female'));
-    const maleVoices   = _shuffleArray(installed.filter(v => getVoiceMeta(v).gender === 'Male'));
-    const otherVoices  = _shuffleArray(installed.filter(v => !['Male', 'Female'].includes(getVoiceMeta(v).gender)));
+    // installed is [{ id, gender }] — separate the ids by gender.
+    const femaleVoices = _shuffleArray(installed.filter(v => v.gender === 'Female').map(v => v.id));
+    const maleVoices   = _shuffleArray(installed.filter(v => v.gender === 'Male').map(v => v.id));
+    const otherVoices  = _shuffleArray(installed.filter(v => !['Male', 'Female'].includes(v.gender)).map(v => v.id));
 
     // Separate agents by gender
     const femaleAgents = _agents.filter(a => _inferAgentGender(a) === 'Female');