agentvibes 5.13.0 → 5.14.0

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

@@ -20,7 +20,7 @@ import { promisify } from 'node:util';
 import fs from 'node:fs';
 import { promises as _fsP } from 'node:fs';
 import { SUPPORTED_LANGUAGES, t } from '../../i18n/strings.js';
-import { ELEVENLABS_VOICES, KOKORO_VOICE_IDS } from '../../services/provider-voice-catalog.js';
+import { ELEVENLABS_VOICES, KOKORO_VOICE_IDS, voicesForProvider } from '../../services/provider-voice-catalog.js';
 import {
   PROVIDERS,
   checkClaudeInstalled, checkCopilotInstalled, checkCodexInstalled,
@@ -36,14 +36,14 @@ import {
   loadLlmConfigSync, saveLlmConfigSync, resolveCfgPath,
 } from '../../services/llm-provider-service.js';
 import {
-  getAvailableEngines, getEngineStatuses, checkEngineInstalled,
+  getAvailableEngines, getAllEngines, checkEngineInstalled, receiverProviderId,
 } from '../../services/tts-engine-service.js';
 import { openReverbPicker, formatEffectLabel } from '../widgets/reverb-picker.js';
 import { openTrackPicker, openVolumeInput } from '../widgets/track-picker.js';
 import { renderHelpBar, selectorTitle } from '../widgets/help-bar.js';
 import { formatTrackName } from '../widgets/format-utils.js';
 import { destroyList } from '../widgets/destroy-list.js';
-import { scanInstalledVoices, getVoiceMeta, genderIconTag, formatVoiceRow, voiceRowHeader, PIPER_VOICES_DIR, SAMPLE_PHRASES, parseMultiSpeaker, getFavorites, getThumbsDown, toggleFavorite, toggleThumbsUp, toggleThumbsDown } from './voices-tab.js';
+import { scanInstalledVoices, getVoiceMeta, previewPhrase, genderIconTag, formatVoiceRow, voiceRowHeader, PIPER_VOICES_DIR, SAMPLE_PHRASES, parseMultiSpeaker, getFavorites, getThumbsDown, toggleFavorite, toggleThumbsUp, toggleThumbsDown } from './voices-tab.js';
 import { attachBtnBlink } from './agents-tab.js';
 import { buildAudioEnv, detectWavPlayer } from '../audio-env.js';
 import { buildBlingCommand, playBlingCue } from '../bling.js';
@@ -175,6 +175,20 @@ const NATIVE_ENGINE_VOICES = {
   elevenlabs:  { id: 'EXAVITQu4vr4xnSDxMaL', label: 'ElevenLabs'           },
 };
 
+// SAPI and macOS Say are NATIVE but MULTI-voice (many built-in system voices),
+// unlike soprano (a genuine single-voice engine). They get the full voice picker
+// and a real default voice, not the engine-id placeholder.
+const MULTI_VOICE_NATIVE = new Set(['sapi', 'macos-say']);
+
+/** Default voice to assign when an engine is first selected in a per-LLM draft. */
+function defaultVoiceForEngine(engine) {
+  if (MULTI_VOICE_NATIVE.has(engine)) {
+    const list = voicesForProvider(engine, { scanInstalledVoices, getVoiceMeta });
+    return list[0]?.id || NATIVE_ENGINE_VOICES[engine]?.id || '';
+  }
+  return NATIVE_ENGINE_VOICES[engine]?.id || '';
+}
+
 // Static built-in ElevenLabs premade voices — universal to every ElevenLabs
 // account. The stored config value is the raw ElevenLabs voice_id; the TTS hook
 // accepts raw IDs directly (its ^[a-zA-Z0-9]{10,40}$ path), so there is no
@@ -810,7 +824,6 @@ export function createSetupTab(screen, services) {
   const providerStatusTexts = [];
 
   // Transport provider row widgets (Configure-only, no Install/Remove)
-  const transportRows = [];
 
   // Info box for provider details
   const infoBox = blessed.box({
@@ -1229,8 +1242,14 @@ export function createSetupTab(screen, services) {
       style: { bg: COLORS.contentBg },
     });
 
+    // _bgRestoreFn is modal-scoped so _killPreview can restore bg music synchronously,
+    // eliminating the race condition when Preview is clicked twice rapidly.
     let _previewModalProc = null;
+    let _bgRestoreFn = null;
     function _killPreview() {
+      // Restore bg music immediately (synchronously) before killing the process —
+      // otherwise a second Preview reads bgWas=true before the first's exit fires.
+      if (_bgRestoreFn) { _bgRestoreFn(); _bgRestoreFn = null; }
       if (_previewModalProc) { try { _previewModalProc.kill(); } catch {} _previewModalProc = null; }
     }
 
@@ -1266,6 +1285,24 @@ export function createSetupTab(screen, services) {
       const hooksSubdir = process.platform === 'win32' ? 'hooks-windows' : 'hooks';
       const isWin = process.platform === 'win32' && !process.env.WSL_DISTRO_NAME;
       const sampleText = 'This is how your Hermes audio settings sound right now.';
+
+      // Temporarily enable background music for preview if a track is configured.
+      // Preview must play everything combined — voice, effects AND music. Write to
+      // targetDir (project): audio-processor.sh checks CLAUDE_PROJECT_DIR first
+      // (set to targetDir in the subprocess env below).
+      if (!!draft.bgTrack) {
+        const bgEnabledFile = path.join(targetDir, '.claude', 'config', 'background-music-enabled.txt');
+        let bgWas = false;
+        try { bgWas = fs.readFileSync(bgEnabledFile, 'utf8').trim() === 'true'; } catch {}
+        if (!bgWas) {
+          try {
+            fs.mkdirSync(path.dirname(bgEnabledFile), { recursive: true });
+            fs.writeFileSync(bgEnabledFile, 'true', 'utf8');
+          } catch {}
+          _bgRestoreFn = () => { try { fs.writeFileSync(bgEnabledFile, 'false', 'utf8'); } catch {} };
+        }
+      }
+
       let cmd, args;
       if (draft.ttsEngine === 'elevenlabs') {
         // ElevenLabs has no Windows provider; route through the bash orchestrator
@@ -1288,12 +1325,19 @@ export function createSetupTab(screen, services) {
         stdio: 'ignore', windowsHide: true,
         env: {
           ...process.env, CLAUDE_PROJECT_DIR: targetDir,
+          // audition = keep the draft's EXACT voice (F1). Force the receiver engine
+          // ONLY when this row has an explicit engine — an empty engine means
+          // "use the global/voice-derived engine", and forcing piper here would
+          // shadow a kokoro/SAPI row voice on the receiver (Fable review).
+          AGENTVIBES_VOICE_SOURCE: 'audition',
+          ...(draft.ttsEngine ? { AGENTVIBES_RECEIVER_PROVIDER_OVERRIDE: receiverProviderId(draft.ttsEngine) } : {}),
           ...(draft.ttsEngine === 'elevenlabs' ? { AGENTVIBES_FORCE_PROVIDER: 'elevenlabs' } : {}),
         },
       });
       _previewModalProc = proc;
       proc.on('exit', (code) => {
         _previewModalProc = null;
+        if (_bgRestoreFn) { _bgRestoreFn(); _bgRestoreFn = null; }
         if (!_closed) {
           if (code !== 0 && code !== null) {
             const engineLabel = NATIVE_ENGINE_VOICES[draft.ttsEngine]?.label || draft.ttsEngine || 'engine';
@@ -1305,7 +1349,11 @@ export function createSetupTab(screen, services) {
           }
         }
       });
-      proc.on('error', () => { _previewModalProc = null; if (!_closed) { previewLine.setContent('{red-fg}Preview failed{/red-fg}'); screen.render(); } });
+      proc.on('error', () => {
+        _previewModalProc = null;
+        if (_bgRestoreFn) { _bgRestoreFn(); _bgRestoreFn = null; }
+        if (!_closed) { previewLine.setContent('{red-fg}Preview failed{/red-fg}'); screen.render(); }
+      });
     }
 
     const previewBtn = _modalBtn('Preview', 4, _playPreview);
@@ -1889,14 +1937,9 @@ export function createSetupTab(screen, services) {
     }
 
     function _autoSave(silent) {
-      saveTransportConfig(provider.id, draft).then(() => {
-        // Refresh status text below provider name
-        const row = transportRows.find(r => r.id === provider.id);
-        if (row && draft.host) {
-          row.statusText.setContent(`{#9e9e9e-fg}→ ${draft.host}:${draft.port}{/#9e9e9e-fg}`);
-          screen.render();
-        }
-      }).catch(() => {});
+      // (Removed a dead status-text refresh keyed off `transportRows`, which was
+      // always empty — Sonar S2583. The config still saves below.)
+      saveTransportConfig(provider.id, draft).catch(() => {});
       if (!silent) _showSavedToast(`${provider.name} Config`, `~/.agentvibes/transport-config.json`);
     }
 
@@ -2225,6 +2268,12 @@ export function createSetupTab(screen, services) {
           windowsHide: true,
           env: {
             ...process.env, CLAUDE_PROJECT_DIR: targetDir, AGENTVIBES_LLM_KEY: `llm:${llmKey}`,
+            // audition = keep the draft's EXACT voice (F1). Force the receiver
+            // engine ONLY when this row has an explicit engine (empty = use the
+            // global/voice-derived engine; forcing piper would shadow a kokoro/
+            // SAPI row voice on the receiver — Fable review).
+            AGENTVIBES_VOICE_SOURCE: 'audition',
+            ...(draft.ttsEngine ? { AGENTVIBES_RECEIVER_PROVIDER_OVERRIDE: receiverProviderId(draft.ttsEngine) } : {}),
             ...(draft.ttsEngine === 'elevenlabs' ? { AGENTVIBES_FORCE_PROVIDER: 'elevenlabs' } : {}),
           },
         });
@@ -2478,8 +2527,24 @@ export function createSetupTab(screen, services) {
     }
     navigationService?.openModal(null, _closePicker);
 
-    const engines = getEngineStatuses();
+    // Show ALL engines (like the Settings → Default TTS Engine picker), not just
+    // the platform-native ones — otherwise Windows SAPI / macOS Say vanish on a
+    // Linux/other host even though a remote receiver could synthesize them. When
+    // this LLM (or the global default) is in remote mode the receiver renders the
+    // audio, so off-platform engines are valid, selectable choices tagged
+    // "(Receiver)"; locally they stay greyed "(Not supported)".
+    const _isRemote = draft.mode === 'remote'
+      || (configService?.getConfig?.()?.audio_destination ?? 'local') === 'remote';
+    const engines = getAllEngines().map(e => ({
+      ...e,
+      installed: e.supported ? checkEngineInstalled(e.id) : false,
+    }));
     const items = engines.map(e => {
+      if (!e.supported) {
+        return _isRemote
+          ? `{cyan-fg}  ${e.name.padEnd(20)} (Receiver){/cyan-fg}  ${e.desc}`
+          : `{#607d8b-fg}  ${e.name.padEnd(20)} (Not supported){/#607d8b-fg}`;
+      }
       const status = e.installed ? '{green-fg}[OK]{/green-fg}' : '{yellow-fg}[Not Found]{/yellow-fg}';
       return `  ${e.name.padEnd(20)} ${status}  ${e.desc}`;
     });
@@ -2533,18 +2598,22 @@ export function createSetupTab(screen, services) {
       const engine = idx > 0 ? engines[idx - 1] : null;
       if (idx > 0 && !engine) { _closePicker(); return; }
       const selectedEngine = engine ? engine.id : '';
-      // Guard: block selection of non-installed optional engines
+      // Guard selection. In remote mode the receiver synthesizes, so any engine
+      // (incl. off-platform SAPI/macOS) is allowed. Locally, block engines that
+      // can't run here: off-platform ones, and optional engines not installed.
       if (selectedEngine) {
         const engineStatus = engines.find(e => e.id === selectedEngine);
-        if (engineStatus && !engineStatus.installed && !engineStatus.native) {
-          box.setLabel(` {red-fg} ${engineStatus.name} is not installed — go to Setup > TTS Engines to install {/red-fg} `);
+        const _flash = (msg) => {
+          box.setLabel(` {red-fg} ${msg} {/red-fg} `);
           screen.render();
-          setTimeout(() => {
-            if (!box.destroyed) {
-              box.setLabel(ENGINE_TITLE);
-              screen.render();
-            }
-          }, 3000);
+          setTimeout(() => { if (!box.destroyed) { box.setLabel(ENGINE_TITLE); screen.render(); } }, 3000);
+        };
+        if (engineStatus && !_isRemote && !engineStatus.supported) {
+          _flash(`${engineStatus.name} runs on the receiver — switch this LLM to a remote receiver to use it`);
+          return;
+        }
+        if (engineStatus && !_isRemote && engineStatus.supported && !engineStatus.installed && !engineStatus.native) {
+          _flash(`${engineStatus.name} is not installed — go to Setup > TTS Engines to install`);
           return;
         }
       }
@@ -2558,8 +2627,12 @@ export function createSetupTab(screen, services) {
         const looksLikeElId = /^[A-Za-z0-9]{10,40}$/.test(draft.voice || '');
         draft.voice = (elevenLabsVoiceName(draft.voice) || looksLikeElId)
           ? draft.voice : ELEVENLABS_DEFAULT_VOICE_ID;
+      } else if (MULTI_VOICE_NATIVE.has(selectedEngine)
+        && voicesForProvider(selectedEngine, { scanInstalledVoices, getVoiceMeta }).some(v => v.id === draft.voice)) {
+        // Re-selecting the SAME multi-voice native engine must NOT discard an
+        // already-valid chosen voice (e.g. Zira -> David) — keep it (Fable review).
       } else {
-        draft.voice = NATIVE_ENGINE_VOICES[selectedEngine]?.id || '';
+        draft.voice = defaultVoiceForEngine(selectedEngine);
       }
       _closePicker();
     });
@@ -3196,7 +3269,7 @@ export function createSetupTab(screen, services) {
           ? '안녕하세요, 코코로 한국어 음성 미리보기입니다.'
           : _pfx2 === 'jf' || _pfx2 === 'jm'
             ? 'こんにちは、これはKokoroの日本語音声プレビューです。'
-            : `Hi, I am the ${voiceId.slice(3)} Kokoro voice.`;
+            : previewPhrase(voiceId);
 
       // Bling now — a real preview is committed (past the install-prompt and
       // toggle-off early returns). Fires for both local and remote paths.
@@ -3726,7 +3799,7 @@ export function createSetupTab(screen, services) {
       const elScript = path.join(packageDir, '.claude', 'hooks', 'play-tts-elevenlabs.sh');
       let proc;
       try {
-        proc = _spawnAudio(resolveBash(), [elScript, `Hi, I am ${v.name}.`, v.id], { // NOSONAR — local hook on user's PATH
+        proc = _spawnAudio(resolveBash(), [elScript, `Hi, I'm ${v.name} from ElevenLabs.`, v.id], { // NOSONAR — local hook on user's PATH
           stdio: 'ignore',
           env: { ...process.env, CLAUDE_PROJECT_DIR: targetDir },
         });
@@ -3827,9 +3900,13 @@ export function createSetupTab(screen, services) {
       destroyList(vpModal, screen, onDone);
     }
 
-    // AVI-S5.1/5.2: Single-item overlay for non-Piper engines.
-    // scanInstalledVoices() is NOT called; Space previews via the correct engine binary.
-    const nativeVoice = NATIVE_ENGINE_VOICES[draft.ttsEngine];
+    // AVI-S5.1/5.2: Single-item overlay for TRUE single-voice engines (soprano).
+    // SAPI / macOS Say are native but MULTI-voice — they fall through to the full
+    // voice picker below (populated from their built-in catalog) so a specific
+    // voice (e.g. Zira) can be chosen and previewed, not just the engine name.
+    const nativeVoice = MULTI_VOICE_NATIVE.has(draft.ttsEngine)
+      ? null
+      : NATIVE_ENGINE_VOICES[draft.ttsEngine];
     if (nativeVoice) {
       draft.voice = nativeVoice.id;
       let _nvClosed = false;
@@ -3890,7 +3967,10 @@ export function createSetupTab(screen, services) {
           screen.render();
           return;
         }
-        const phrase = `Hi, I am the ${nativeVoice.label} voice.`;
+        const _engineLabel = nativeVoice.id === 'macos' ? 'macOS'
+          : (nativeVoice.id && nativeVoice.id.includes('sapi')) ? 'Windows SAPI'
+            : (nativeVoice.id || 'your system');
+        const phrase = `Hi, I'm ${nativeVoice.label} from ${_engineLabel}.`;
         const engine = nativeVoice.id;
 
         function _spawnAndTrack(cmd, args, opts) {
@@ -4091,7 +4171,10 @@ export function createSetupTab(screen, services) {
       if (_vpClosed) return;
       const savedIdx = vpList.selected ?? 0;
       const savedScroll = vpList.childBase ?? 0;
-      _allVoices = scanInstalledVoices();
+      // SAPI / macOS Say list their built-in catalog voices; Piper scans disk.
+      _allVoices = MULTI_VOICE_NATIVE.has(draft.ttsEngine)
+        ? voicesForProvider(draft.ttsEngine, { scanInstalledVoices, getVoiceMeta }).map(v => v.id)
+        : scanInstalledVoices();
       // Sort by display name so the first-letter quick jump is intuitive
       _allVoices.sort((a, b) => getVoiceMeta(a).displayName.localeCompare(
         getVoiceMeta(b).displayName, undefined, { sensitivity: 'base' }));
@@ -4106,7 +4189,7 @@ export function createSetupTab(screen, services) {
       if (_previewVoiceId === voiceId) { _killVP(); vpPreviewLine.setContent(''); _refreshVP(); return; }
       _killVP();
 
-      const phrase = `Hi, my name is ${getVoiceMeta(voiceId).displayName}.`;
+      const phrase = previewPhrase(voiceId);
 
       // Route through remote provider if active
       // Search order: targetDir → cwd → package root → home
@@ -4133,6 +4216,10 @@ export function createSetupTab(screen, services) {
           : targetDir;
         const _rEnv = {
           ..._spawnEnv, CLAUDE_PROJECT_DIR: targetDir,
+          // audition = keep the EXACT previewed voice (F1); force the receiver
+          // engine so SAPI/macOS voices render in their own engine, not the default.
+          AGENTVIBES_VOICE_SOURCE: 'audition',
+          AGENTVIBES_RECEIVER_PROVIDER_OVERRIDE: receiverProviderId(draft.ttsEngine || 'piper'),
           ...(llmKey ? { AGENTVIBES_LLM_KEY: `llm:${llmKey}` } : {}),
         };
         let rProc;
@@ -4163,6 +4250,41 @@ export function createSetupTab(screen, services) {
         return;
       }
 
+      // Local preview of the native MULTI-voice engines (Windows SAPI, macOS Say):
+      // the piper synth below cannot render these. Restores the local preview the
+      // single-item overlay used to provide before SAPI/macOS became multi-voice
+      // (Fable review — otherwise a local SAPI preview mis-ran piper and showed a
+      // bogus "is Piper installed?" error).
+      if (MULTI_VOICE_NATIVE.has(draft.ttsEngine)) {
+        let nProc = null;
+        if (_isWin && draft.ttsEngine === 'sapi') {
+          let _sapiScript = '';
+          for (const n of ['play-tts-sapi.ps1', 'play-tts-windows-sapi.ps1']) {
+            for (const base of [packageDir, targetDir]) {
+              const _p = path.join(base, '.claude', 'hooks-windows', n);
+              if (fs.existsSync(_p)) { _sapiScript = _p; break; }
+            }
+            if (_sapiScript) break;
+          }
+          if (_sapiScript) {
+            nProc = _spawnAudio('powershell', ['-NoProfile', '-ExecutionPolicy', 'Bypass', '-File', _sapiScript, '-Text', phrase, '-VoiceOverride', voiceId], { stdio: 'ignore', windowsHide: true }); // NOSONAR
+          }
+        } else if (process.platform === 'darwin' && draft.ttsEngine === 'macos-say') {
+          nProc = _spawnAudio('say', ['-v', voiceId, phrase], { stdio: 'ignore' }); // NOSONAR
+        }
+        if (!nProc) {
+          // Engine can't run on THIS OS and this LLM isn't remote — nothing to play.
+          if (!_vpClosed) { vpPreviewLine.setContent('{yellow-fg}♪ Runs on the receiver — set this LLM to remote to preview{/yellow-fg}'); _refreshVP(); }
+          return;
+        }
+        _previewProc = nProc;
+        _previewVoiceId = voiceId;
+        if (!_vpClosed) { vpPreviewLine.setContent(`{cyan-fg}♪ Playing: ${voiceId}...{/cyan-fg}`); _refreshVP(); }
+        nProc.on('exit', () => { if (_previewVoiceId === voiceId) { _previewVoiceId = null; _previewProc = null; if (!_vpClosed) { vpPreviewLine.setContent(''); _refreshVP(); } } });
+        nProc.on('error', () => { _previewProc = null; _previewVoiceId = null; });
+        return;
+      }
+
       const _ms = parseMultiSpeaker(voiceId);
       const voicePath = path.resolve(PIPER_VOICES_DIR, _ms.model + '.onnx');
       const safeBase = path.resolve(PIPER_VOICES_DIR);

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

@@ -17,6 +17,7 @@ import { buildAudioEnv, detectWavPlayer, getAllWavPlayers } from '../audio-env.j
 import { SURNAME_POOL, uniquifyVoiceName } from '../../utils/voice-names.js';
 import { voicesForProvider, ELEVENLABS_VOICES, KOKORO_VOICE_IDS, WINDOWS_SAPI_VOICES, MACOS_VOICES, kokoroGender } from '../../services/provider-voice-catalog.js';
 import { getProvider as _catalogGetProvider } from '../../services/provider-catalog.js';
+import { receiverProviderId } from '../../services/tts-engine-service.js';
 
 /**
  * Resolve display name / gender / provider label for a NON-Piper catalog voice
@@ -531,7 +532,7 @@ export function scanInstalledVoices() {
       } catch { /* fall through to add as single voice */ }
       result.push(voiceId);
     }
-    return result.sort();
+    return result.sort((a, b) => a.localeCompare(b));
   } catch {
     return [];
   }
@@ -724,6 +725,27 @@ export function getVoiceMeta(voiceId) {
   return result;
 }
 
+/**
+ * Build the spoken preview phrase for a voice — announces BOTH the voice's name
+ * and the engine it renders on, consistently across every voice picker in the
+ * app (Piper / Kokoro / Windows SAPI / macOS / ElevenLabs). Centralised so a
+ * preview reads the same everywhere: "Hi, I'm Bella from Piper."
+ * @param {string} voiceId
+ * @returns {string}
+ */
+export function previewPhrase(voiceId) {
+  const meta = getVoiceMeta(voiceId);
+  // Strip a parenthetical model note ("Piper (libritts)" -> "Piper") for a clean
+  // spoken line; fall back to a neutral label if the provider can't be resolved.
+  const engine = String(meta.provider || '').replace(/\s*\(.*\)\s*$/, '').trim() || 'AgentVibes';
+  // Kokoro ids (af_bella, am_michael) carry a 2-letter lang/gender prefix that the
+  // display name keeps ("Af Bella") — awkward aloud, so speak just the name part.
+  let name = meta.displayName;
+  const kok = voiceId.match(/^[a-z]{2}_([a-z0-9_]+)$/);
+  if (kok) name = kok[1].replace(/_/g, ' ').replace(/\b\w/g, c => c.toUpperCase());
+  return `Hi, I'm ${name} from ${engine}.`;
+}
+
 // ---------------------------------------------------------------------------
 
 /**
@@ -1037,7 +1059,27 @@ export function createVoicesTab(screen, services) {
     if (!_isPiperLocal) {
       const _remote = _remoteProviders.includes(activeProvider);
       const isWindows = process.platform === 'win32' && !process.env.WSL_DISTRO_NAME;
-      const phrase = `Hi, my name is ${getVoiceMeta(voiceId).displayName}.`;
+      const phrase = previewPhrase(voiceId);
+      // A PREVIEW must render in the previewed voice's OWN engine — never the
+      // receiver's persisted default (receiver-provider.txt). Compute the engine
+      // these voices belong to (the Default TTS Engine, same rule the list uses)
+      // and forward it as a ONE-OFF override the remote sender honours for this
+      // invocation only. This changes no settings; it just makes preview preview.
+      let _previewEngine = configService?.getConfig?.()?.ttsEngine
+        || providerService?.getActiveProvider?.() || 'piper';
+      // Guard: if that resolves to a transport (not a real engine), fall back to
+      // piper so we never tell the receiver to "synthesize with ssh-remote".
+      if (_remoteProviders.includes(_previewEngine)) _previewEngine = 'piper';
+      // AGENTVIBES_VOICE_SOURCE=audition: the resolver must keep the EXACT voice
+      // we're previewing (F1), not demote it to the per-LLM/provider voice —
+      // required for engines whose ids aren't self-identifying (SAPI/macOS).
+      // receiverProviderId maps the engine id to the receiver's provider id the
+      // sender validates (sapi -> windows-sapi, macos-say -> macos).
+      const _previewEnv = {
+        ..._spawnEnv,
+        AGENTVIBES_VOICE_SOURCE: 'audition',
+        AGENTVIBES_RECEIVER_PROVIDER_OVERRIDE: receiverProviderId(_previewEngine),
+      };
       // Hooks live in the AgentVibes package (projectRoot), not the user's project dir.
       // Fall back to CLAUDE_PROJECT_DIR / cwd only if the hook isn't at projectRoot
       // (e.g. when running from a published npm package with a different layout).
@@ -1052,13 +1094,13 @@ export function createVoicesTab(screen, services) {
         const playTts = path.join(hooksBase, '.claude', 'hooks-windows', 'play-tts.ps1');
         const _pdArgs = activeProjectDir ? ['-ProjectDir', activeProjectDir] : [];
         proc = spawn('powershell', ['-NoProfile', '-ExecutionPolicy', 'Bypass', '-File', playTts, phrase, voiceId, ..._pdArgs], { // NOSONAR
-          stdio: 'ignore', detached: false, windowsHide: true, env: _spawnEnv,
+          stdio: 'ignore', detached: false, windowsHide: true, env: _previewEnv,
         });
       } else {
         const playTts = path.join(hooksBase, '.claude', 'hooks', 'play-tts.sh');
         const _pdArgs = activeProjectDir ? ['--project-dir', activeProjectDir] : [];
         proc = spawn('bash', [playTts, phrase, voiceId, ..._pdArgs], { // NOSONAR
-          stdio: ['ignore', 'ignore', 'pipe'], detached: true, env: _spawnEnv,
+          stdio: ['ignore', 'ignore', 'pipe'], detached: true, env: _previewEnv,
           cwd: process.cwd(),
         });
       }
@@ -1103,7 +1145,7 @@ export function createVoicesTab(screen, services) {
     }
 
     const tempWav = path.join(os.tmpdir(), `agentvibes-preview-${Date.now()}.wav`);
-    const phrase = `Hi, my name is ${getVoiceMeta(voiceId).displayName}.`;
+    const phrase = previewPhrase(voiceId);
 
     // Synthesize: spawn piper; on Windows use the exe path directly
     const isWindows = process.platform === 'win32' && !process.env.WSL_DISTRO_NAME;
@@ -1351,7 +1393,7 @@ export function createVoicesTab(screen, services) {
       try { fs.unlinkSync(path.join(globalClaudeDir, 'tts-piper-speaker-id.txt')); } catch { /* ok */ }
     }
     // Also write global tts-voice.txt for shell scripts
-    try { fs.writeFileSync(path.join(globalClaudeDir, 'tts-voice.txt'), ms.isMultiSpeaker ? voiceId : voiceId, 'utf8'); } catch { /* ok */ }
+    try { fs.writeFileSync(path.join(globalClaudeDir, 'tts-voice.txt'), voiceId, 'utf8'); } catch { /* ok */ }
   }
 
   function _openSelectVoiceModal(voiceId) {

package/src/console/widgets/destroy-list.js

@@ -1,25 +1,25 @@
-/**
- * AgentVibes TUI — Shared Widget: Modal Destroy Helper
- *
- * Force-invalidates blessed's olines buffer after destroying a modal widget.
- * Without this, blessed skips repainting cells where lines==olines and the
- * terminal retains stale modal content as ghost artifacts.
- */
-
-/**
- * Destroy a blessed list/box widget and force full screen repaint.
- *
- * @param {object} widget  - blessed widget to destroy
- * @param {object} screen  - blessed screen instance
- * @param {Function} [onClose] - optional callback after destruction
- */
-export function destroyList(widget, screen, onClose) {
-  widget.destroy();
-  try {
-    for (let r = 0; r < screen.height; r++)
-      for (let c = 0; c < screen.width; c++)
-        if (screen.olines[r]?.[c]) screen.olines[r][c][0] = -1;
-  } catch {}
-  onClose?.();
-  screen.render();
-}
+/**
+ * AgentVibes TUI — Shared Widget: Modal Destroy Helper
+ *
+ * Force-invalidates blessed's olines buffer after destroying a modal widget.
+ * Without this, blessed skips repainting cells where lines==olines and the
+ * terminal retains stale modal content as ghost artifacts.
+ */
+
+/**
+ * Destroy a blessed list/box widget and force full screen repaint.
+ *
+ * @param {object} widget  - blessed widget to destroy
+ * @param {object} screen  - blessed screen instance
+ * @param {Function} [onClose] - optional callback after destruction
+ */
+export function destroyList(widget, screen, onClose) {
+  widget.destroy();
+  try {
+    for (let r = 0; r < screen.height; r++)
+      for (let c = 0; c < screen.width; c++)
+        if (screen.olines[r]?.[c]) screen.olines[r][c][0] = -1;
+  } catch {}
+  onClose?.();
+  screen.render();
+}

package/src/console/widgets/notice.js

@@ -1,55 +1,55 @@
-/**
- * AgentVibes TUI — Shared Widget: Notice Toast
- *
- * Displays a short auto-dismissing notice modal centred on screen.
- * Usable from any tab; no settings-specific state required.
- */
-
-import { destroyList } from './destroy-list.js';
-
-const IS_TEST = process.env.AGENTVIBES_TEST_MODE === 'true';
-let blessed;
-if (!IS_TEST) {
-  const { default: b } = await import('blessed');
-  blessed = b;
-}
-
-/**
- * Show a temporary notice that auto-dismisses after 2.5 seconds.
- *
- * @param {object} screen  - blessed screen instance
- * @param {string} message - text to display
- * @param {object} [opts]
- * @param {string} [opts.bg='#0a0e1a']       - background colour
- * @param {string} [opts.fg='#e3f2fd']       - foreground colour
- * @param {string} [opts.borderFg='bright-cyan'] - border colour
- * @param {number} [opts.durationMs=2500]    - auto-dismiss delay in ms
- */
-export function showNotice(screen, message, opts = {}) {
-  const bg         = opts.bg         ?? '#0a0e1a';
-  const fg         = opts.fg         ?? '#e3f2fd';
-  const borderFg   = opts.borderFg   ?? 'bright-cyan';
-  const durationMs = opts.durationMs ?? 2500;
-
-  const width = Math.max(28, message.length + 6);
-  const modal = blessed.box({
-    parent: screen,
-    top: 'center',
-    left: 'center',
-    width,
-    height: 3,
-    border: { type: 'line' },
-    tags: true,
-    content: `{center}${message}{/center}`,
-    style: {
-      fg,
-      bg,
-      border: { fg: borderFg },
-    },
-  });
-  screen.render();
-
-  setTimeout(() => {
-    destroyList(modal, screen);
-  }, durationMs);
-}
+/**
+ * AgentVibes TUI — Shared Widget: Notice Toast
+ *
+ * Displays a short auto-dismissing notice modal centred on screen.
+ * Usable from any tab; no settings-specific state required.
+ */
+
+import { destroyList } from './destroy-list.js';
+
+const IS_TEST = process.env.AGENTVIBES_TEST_MODE === 'true';
+let blessed;
+if (!IS_TEST) {
+  const { default: b } = await import('blessed');
+  blessed = b;
+}
+
+/**
+ * Show a temporary notice that auto-dismisses after 2.5 seconds.
+ *
+ * @param {object} screen  - blessed screen instance
+ * @param {string} message - text to display
+ * @param {object} [opts]
+ * @param {string} [opts.bg='#0a0e1a']       - background colour
+ * @param {string} [opts.fg='#e3f2fd']       - foreground colour
+ * @param {string} [opts.borderFg='bright-cyan'] - border colour
+ * @param {number} [opts.durationMs=2500]    - auto-dismiss delay in ms
+ */
+export function showNotice(screen, message, opts = {}) {
+  const bg         = opts.bg         ?? '#0a0e1a';
+  const fg         = opts.fg         ?? '#e3f2fd';
+  const borderFg   = opts.borderFg   ?? 'bright-cyan';
+  const durationMs = opts.durationMs ?? 2500;
+
+  const width = Math.max(28, message.length + 6);
+  const modal = blessed.box({
+    parent: screen,
+    top: 'center',
+    left: 'center',
+    width,
+    height: 3,
+    border: { type: 'line' },
+    tags: true,
+    content: `{center}${message}{/center}`,
+    style: {
+      fg,
+      bg,
+      border: { fg: borderFg },
+    },
+  });
+  screen.render();
+
+  setTimeout(() => {
+    destroyList(modal, screen);
+  }, durationMs);
+}