agentvibes 5.6.7 → 5.6.9

package/package.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/package.json",
   "name": "agentvibes",
-  "version": "5.6.7",
+  "version": "5.6.9",
   "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/app.js

@@ -30,6 +30,7 @@ import { createReceiverTab } from './tabs/receiver-tab.js';
 import { createAgentsTab } from './tabs/agents-tab.js';
 import { ConfigService } from '../services/config-service.js';
 import { ProviderService } from '../services/provider-service.js';
+import { seedAllLlmDefaultsSync } from '../services/llm-provider-service.js';
 
 const _dir = path.dirname(fileURLToPath(import.meta.url));
 const _pkg = JSON.parse(readFileSync(path.join(_dir, '../../package.json'), 'utf8'));
@@ -893,6 +894,11 @@ export class AgentVibesConsole {
  * @returns {Promise<AgentVibesConsole>}
  */
 export async function launchConsole(opts = {}) {
+  // Seed per-LLM piper defaults into the project config on every TUI launch.
+  // This guarantees play-tts.ps1 always finds a piper engine row for each LLM
+  // and never silently falls back to the global windows-sapi provider.
+  seedAllLlmDefaultsSync(process.cwd());
+
   const app = new AgentVibesConsole(opts);
   await app.init();
   return app;

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

@@ -225,6 +225,19 @@ function _setMusic(configService, update) {
   configService.set('backgroundMusic', { ...current, ...update });
 }
 
+/**
+ * Sync the background-music-enabled.txt flag file so bash hooks see the change.
+ * The TUI stores enabled state in config.json, but audio-processor.sh and
+ * play-tts-piper.sh read from this .txt file — they must stay in sync.
+ * @param {boolean} enabled
+ */
+export function applyEnabledToFile(enabled) {
+  const enabledFile = path.join(process.cwd(), '.claude', 'config', 'background-music-enabled.txt');
+  try {
+    fs.writeFileSync(enabledFile, enabled ? 'true' : 'false', 'utf-8');
+  } catch { /* non-fatal */ }
+}
+
 /**
  * Patch the 'default' entry in audio-effects.cfg to use the given track.
  * play-tts-piper.sh reads the track from audio-effects.cfg (not from config.json),
@@ -431,7 +444,9 @@ export function createMusicTab(screen, services) {
 
   const toggleBtn = _createBtn(_tl('musicToggleBtn'), () => {
     const { enabled } = _getMusic(configService);
-    _setMusic(configService, { enabled: !enabled });
+    const next = !enabled;
+    _setMusic(configService, { enabled: next });
+    applyEnabledToFile(next);  // sync bash hooks (audio-processor.sh reads this file)
     refreshDisplay();
   });
   toggleBtn.bottom = 4;
@@ -772,8 +787,9 @@ export function createMusicTab(screen, services) {
     }
 
     function _saveLocally() {
-      _setMusic(configService, { track: trackId });
+      _setMusic(configService, { track: trackId, enabled: true });
       applyTrackToAudioEffects(trackId);
+      applyEnabledToFile(true);  // saving a track implies enabling music
       refreshDisplay();
       _showTrackChangedNotice(displayName);
     }

package/src/console/widgets/format-utils.js

@@ -5,6 +5,8 @@
  * circular imports between widgets and tabs.
  */
 
+import { uniquifyVoiceName } from '../../utils/voice-names.js';
+
 const TRACK_NAMES = Object.freeze({
   'agentvibes_soft_flamenco_loop.mp3':                 '🎻 Soft Flamenco',
   'agent_vibes_bachata_v1_loop.mp3':                   '🎺 Bachata',
@@ -60,8 +62,15 @@ export function formatVoiceName(voice) {
 
   let name;
   if (voice.includes('::')) {
-    // 16Speakers::Rose_Ibex → extract after '::'
-    name = voice.split('::')[1];
+    const speakerPart = voice.split('::')[1];
+    if (speakerPart.includes('_')) {
+      // 16Speakers format (Rose_Ibex) — already a complete name, just normalise display
+      name = speakerPart.replace(/_/g, ' ');
+    } else {
+      // LibriTTS / single-word names: append deterministic surname
+      // "Mary" → "Mary Bell", "Mary-7" → "Mary Hayes"
+      name = uniquifyVoiceName(speakerPart);
+    }
   } else {
     const parts = voice.split('-');
     const QUALITIES = new Set(['high', 'medium', 'low']);

package/src/installer.js

@@ -4054,17 +4054,18 @@ async function configureSessionStartHook(targetDir, spinner) {
       existingSettings.hooks = {};
     }
 
-    if (!existingSettings.hooks.SessionStart) {
-      if (isNativeWindows()) {
-        existingSettings.hooks.SessionStart = [{
-          hooks: [{
-            type: 'command',
-            command: 'powershell -NoProfile -ExecutionPolicy Bypass -File "$CLAUDE_PROJECT_DIR\\.claude\\hooks-windows\\session-start-tts.ps1"'
-          }]
-        }];
-      } else {
-        existingSettings.hooks.SessionStart = templateSettings.hooks.SessionStart;
-      }
+    // Windows uses SessionStart; Linux/macOS/WSL uses UserPromptSubmit.
+    // Both point to the global $HOME path so updates to the hook script
+    // take effect immediately without needing per-project reinstalls.
+    const hookKey = isNativeWindows() ? 'SessionStart' : 'UserPromptSubmit';
+    const hookCommand = isNativeWindows()
+      ? 'powershell -NoProfile -ExecutionPolicy Bypass -File "$HOME\\.claude\\hooks-windows\\session-start-tts.ps1"'
+      : 'bash "$HOME/.claude/hooks/session-start-tts.sh"';
+
+    if (!existingSettings.hooks[hookKey]) {
+      existingSettings.hooks[hookKey] = [{
+        hooks: [{ type: 'command', command: hookCommand }]
+      }];
 
       if (!existingSettings.$schema) {
         existingSettings.$schema = templateSettings.$schema;
@@ -5045,8 +5046,8 @@ async function updateCommandFiles(targetDir, spinner) {
  * These hooks contain bug fixes (e.g. markdown stripping) that must propagate
  * on every `npx agentvibes update` regardless of target directory.
  */
-const CRITICAL_HOOKS = ['stop-tts.sh', 'stop.sh', 'play-tts.sh', 'session-start-tts.sh', 'bmad-party-speak.sh'];
-const CRITICAL_HOOKS_WINDOWS = ['play-tts.ps1', 'session-start-tts.ps1', 'bmad-speak.ps1', 'bmad-party-speak.ps1'];
+const CRITICAL_HOOKS = ['stop-tts.sh', 'stop.sh', 'play-tts.sh', 'play-tts-piper.sh', 'audio-processor.sh', 'session-start-tts.sh', 'bmad-party-speak.sh'];
+const CRITICAL_HOOKS_WINDOWS = ['play-tts.ps1', 'play-tts-piper.ps1', 'audio-processor.ps1', 'session-start-tts.ps1', 'bmad-speak.ps1', 'bmad-party-speak.ps1'];
 
 /**
  * Update critical hooks in the global ~/.claude/hooks/ directory if it exists.
@@ -5056,45 +5057,38 @@ const CRITICAL_HOOKS_WINDOWS = ['play-tts.ps1', 'session-start-tts.ps1', 'bmad-s
  * @returns {Promise<number>} Number of hooks updated
  */
 async function updateGlobalHooks(srcHooksDir, homeDirOverride) {
-  const globalHooksDir = path.join(homeDirOverride || os.homedir(), '.claude', 'hooks');
+  const homeDir = homeDirOverride || os.homedir();
+  const globalHooksDir = path.join(homeDir, '.claude', 'hooks');
   let updated = 0;
-  try {
-    await fs.access(globalHooksDir);
-  } catch {
-    return 0; // global hooks dir not present — nothing to do
-  }
+
+  // Always ensure the global hooks dir exists so registered $HOME hooks resolve.
+  await fs.mkdir(globalHooksDir, { recursive: true });
 
   for (const hook of CRITICAL_HOOKS) {
     const destPath = path.join(globalHooksDir, hook);
     const srcPath = path.join(srcHooksDir, hook);
     try {
-      await fs.access(destPath); // only update if already installed
       await fs.copyFile(srcPath, destPath);
       await fs.chmod(destPath, 0o750);
       updated++;
     } catch {
-      // file not in global dir or src missing — skip silently
+      // src missing — skip silently
     }
   }
 
-  // Also update Windows global hooks-windows dir if present
-  const globalHooksWindowsDir = path.join(homeDirOverride || os.homedir(), '.claude', 'hooks-windows');
+  // Also ensure Windows global hooks-windows dir and scripts are up to date.
+  const globalHooksWindowsDir = path.join(homeDir, '.claude', 'hooks-windows');
   const srcHooksWindowsDir = path.join(path.dirname(srcHooksDir), 'hooks-windows');
-  try {
-    await fs.access(globalHooksWindowsDir);
-    for (const hook of CRITICAL_HOOKS_WINDOWS) {
-      const destPath = path.join(globalHooksWindowsDir, hook);
-      const srcPath = path.join(srcHooksWindowsDir, hook);
-      try {
-        await fs.access(destPath); // only update if already installed
-        await fs.copyFile(srcPath, destPath);
-        updated++;
-      } catch {
-        // file not in global dir or src missing — skip silently
-      }
+  await fs.mkdir(globalHooksWindowsDir, { recursive: true });
+  for (const hook of CRITICAL_HOOKS_WINDOWS) {
+    const destPath = path.join(globalHooksWindowsDir, hook);
+    const srcPath = path.join(srcHooksWindowsDir, hook);
+    try {
+      await fs.copyFile(srcPath, destPath);
+      updated++;
+    } catch {
+      // src missing — skip silently
     }
-  } catch {
-    // hooks-windows dir not present — nothing to do
   }
 
   return updated;
@@ -5337,6 +5331,13 @@ async function install(options = {}) {
     await copyBackgroundMusicFiles(targetDir, silentSpinner);
     await copyConfigFiles(targetDir, silentSpinner);
     await copyCodexFiles(targetDir, silentSpinner);
+
+    // Populate global ~/.claude/hooks[/-windows]/ so $HOME hook paths resolve
+    // on first install (not just on update).
+    const hooksSubdirInstall = isNativeWindows() ? 'hooks-windows' : 'hooks';
+    const srcHooksDirInstall = path.join(__dirname, '..', '.claude', hooksSubdirInstall);
+    await updateGlobalHooks(srcHooksDirInstall);
+
     await configureSessionStartHook(targetDir, silentSpinner);
     await configurePartyModeHook(targetDir, silentSpinner);
     await installPluginManifest(targetDir, silentSpinner);

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

@@ -92,8 +92,17 @@ const DEFAULT_LLM_CONFIGS = {
 };
 
 function ensureDefaultLlmConfigSync(llmKey, targetDir) {
-  const existing = loadLlmConfigSync(llmKey, targetDir);
-  if (existing.sourcePath) return;
+  // Check only the project dir — a global config row must not prevent seeding
+  // per-project defaults (otherwise new installs silently inherit a different
+  // project's voice/pretext and the per-LLM piper engine is never written,
+  // causing play-tts.ps1 to fall through to the global SAPI provider).
+  const resolvedDir = targetDir || process.env.INIT_CWD || process.cwd();
+  const projectCfgPath = path.join(resolvedDir, '.claude', 'config', 'audio-effects.cfg');
+  const cfgKey = `llm:${llmKey}`;
+  try {
+    const content = fsSync.readFileSync(projectCfgPath, 'utf8');
+    if (content.split('\n').some(line => line.startsWith(cfgKey + '|'))) return;
+  } catch { /* file not found — continue to seed */ }
 
   const defaults = DEFAULT_LLM_CONFIGS[llmKey];
   if (!defaults) return;
@@ -101,6 +110,17 @@ function ensureDefaultLlmConfigSync(llmKey, targetDir) {
   saveLlmConfigSync(llmKey, defaults, targetDir);
 }
 
+/**
+ * Seed piper defaults for every LLM in DEFAULT_LLM_CONFIGS into the project dir.
+ * Called at install time so play-tts.ps1 always finds a per-LLM piper row in the
+ * project config and never silently falls back to the global windows-sapi provider.
+ */
+export function seedAllLlmDefaultsSync(targetDir) {
+  for (const llmKey of Object.keys(DEFAULT_LLM_CONFIGS)) {
+    ensureDefaultLlmConfigSync(llmKey, targetDir);
+  }
+}
+
 // ── Provider install-checks ─────────────────────────────────────────────────
 
 export async function checkClaudeInstalled(targetDir) {
@@ -902,13 +922,12 @@ export function saveLlmConfigSync(llmKey, config, targetDir) {
   const sanitize = (v) => (v || '').replace(/[\|\n\r\x00]/g, '');
   const cfgLine = `${cfgKey}|${sanitize(config.effects)}|${sanitize(config.bgTrack)}|${sanitize(config.bgVolume)}|${sanitize(config.voice)}|${sanitize(config.pretext)}|${sanitize(config.ttsEngine)}`;
   const resolvedTargetDir = targetDir || process.env.INIT_CWD || process.cwd();
-  // When targetDir is explicitly passed, write there directly (do not fall back to global).
-  // resolveCfgPath falls back to ~/.claude when the local file doesn't yet exist,
-  // which would silently redirect writes away from the intended directory.
-  const cfgPath = config.sourcePath ||
-    (targetDir
-      ? path.join(resolvedTargetDir, '.claude', 'config', 'audio-effects.cfg')
-      : resolveCfgPath(resolvedTargetDir));
+  // When targetDir is explicitly passed, always write to the project dir — never follow
+  // config.sourcePath back to the global ~/.claude/config (it may have been loaded from there
+  // as a "default seed" for a new project, and writing back would pollute other projects).
+  const cfgPath = targetDir
+    ? path.join(resolvedTargetDir, '.claude', 'config', 'audio-effects.cfg')
+    : (config.sourcePath || resolveCfgPath(resolvedTargetDir));
 
   try {
     let content = '';

package/src/utils/voice-names.js

@@ -35,6 +35,8 @@ export function uniquifyVoiceName(rawName) {
       const idx = (n - 1) % SURNAME_POOL.length;
       return `${base} ${SURNAME_POOL[idx]}`;
     }
+    // n=1 (or 0): strip the suffix — "Mary-1" → "Mary Bell" is cleaner than "Mary-1 Bell"
+    return `${base} ${SURNAME_POOL[0]}`;
   }
   if (/\s/.test(rawName)) return rawName;
   return `${rawName} ${SURNAME_POOL[0]}`;

package/.claude/hooks/play-tts-agentvibes-receiver.sh

@@ -1 +0,0 @@
-play-tts-agentvibes-receiver-for-voiceless-connections.sh