agentvibes 2.14.2 → 2.14.3

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

@@ -116,6 +116,9 @@ speak_text() {
     piper)
       "$SCRIPT_DIR/play-tts-piper.sh" "$text" "$voice"
       ;;
+    macos)
+      "$SCRIPT_DIR/play-tts-macos.sh" "$text" "$voice"
+      ;;
     *)
       echo "❌ Unknown provider: $provider" >&2
       return 1
@@ -232,6 +235,9 @@ case "$ACTIVE_PROVIDER" in
   piper)
     exec "$SCRIPT_DIR/play-tts-piper.sh" "$TEXT" "$VOICE_OVERRIDE"
     ;;
+  macos)
+    exec "$SCRIPT_DIR/play-tts-macos.sh" "$TEXT" "$VOICE_OVERRIDE"
+    ;;
   *)
     echo "❌ Unknown provider: $ACTIVE_PROVIDER"
     echo "   Run: /agent-vibes:provider list"

package/README.md

@@ -11,7 +11,7 @@
 [![Publish](https://github.com/paulpreibisch/AgentVibes/actions/workflows/publish.yml/badge.svg)](https://github.com/paulpreibisch/AgentVibes/actions/workflows/publish.yml)
 [![License](https://img.shields.io/badge/License-Apache_2.0-blue.svg)](https://opensource.org/licenses/Apache-2.0)
 
-**Author**: Paul Preibisch ([@997Fire](https://x.com/997Fire)) | **Version**: v2.14.2
+**Author**: Paul Preibisch ([@997Fire](https://x.com/997Fire)) | **Version**: v2.14.3
 
 ---
 
@@ -94,17 +94,15 @@ Whether you're coding in Claude Code, chatting in Claude Desktop, or using Warp
 
 ## 📰 Latest Release
 
-**[v2.14.2 - Native macOS TTS Provider](https://github.com/paulpreibisch/AgentVibes/releases/tag/v2.14.2)** 🎉
+**[v2.14.3 - macOS Provider Routing Fix](https://github.com/paulpreibisch/AgentVibes/releases/tag/v2.14.3)** 🎉
 
-AgentVibes v2.14.1 introduces native macOS TTS support via the built-in `say` command. Mac users can now use AgentVibes with **zero setup required** - no API keys, no downloads, no configuration. Simply switch to the macOS provider and start talking! The new provider automatically detects 100+ built-in Apple voices across 40+ languages, with Siri-quality enhanced voices available on macOS Mojave (10.14) and later.
+AgentVibes v2.14.3 fixes a critical bug where the macOS TTS provider would not speak when selected. The TTS router (`play-tts.sh`) was missing the case handler for the macOS provider, causing it to fail silently. This release adds the missing routing and includes comprehensive test coverage to prevent regression.
 
 **Key Highlights:**
-- 🍎 **macOS Say Provider** - Native TTS using macOS `say` command (zero dependencies!)
-- 🎤 **100+ Built-in Voices** - Access all Apple voices including enhanced Siri voices
-- 🌍 **40+ Languages** - Full language support from Apple's voice library
-- 🔄 **Three-Way Provider Switching** - Seamless migration between ElevenLabs, Piper, and macOS
-- 🎉 **Smarter BMAD Detection** - Improved installer message when BMAD-METHOD™ is detected
-- ✅ **Zero Breaking Changes** - Fully backward compatible with v2.14.0
+- 🐛 **Fixed macOS Provider Routing** - macOS `say` provider now works when selected (Issue #52)
+- ✅ **New Provider Tests** - 4 new tests ensure macOS provider is properly detected and routed
+- 🔧 **Fixed Translator Tests** - Fixed 3 pre-existing failing tests related to PWD and locale warnings
+- 🎯 **133 Tests Passing** - Full test suite now passes with comprehensive macOS coverage
 
 💡 **Tip:** If `npx agentvibes` shows an older version or missing commands, clear your npm cache: `npm cache clean --force && npx agentvibes@latest --help`
 

package/RELEASE_NOTES.md

@@ -1,3 +1,120 @@
+# Release v2.14.3 - macOS Provider Routing Fix
+
+**Release Date:** 2025-11-30
+**Type:** Patch Release (Bug Fix)
+
+## AI Summary
+
+AgentVibes v2.14.3 fixes a critical bug where the macOS TTS provider would not speak when selected. The TTS router (`play-tts.sh`) was missing the case handler for the macOS provider, causing it to fail silently. This release adds the missing routing and includes comprehensive test coverage to prevent regression.
+
+**Key Highlights:**
+- 🐛 **Fixed macOS Provider Routing** - macOS `say` provider now works when selected (Issue #52)
+- ✅ **New Provider Tests** - 4 new tests ensure macOS provider is properly detected and routed
+- 🔧 **Fixed Translator Tests** - Fixed 3 pre-existing failing tests related to PWD and locale warnings
+- 🎯 **133 Tests Passing** - Full test suite now passes with comprehensive macOS coverage
+
+---
+
+## Bug Fixes
+
+### macOS Provider Routing Fix (Issue #52)
+**File:** `.claude/hooks/play-tts.sh`
+
+The TTS router was missing case handlers for the `macos` provider in two locations:
+1. The `speak_text()` function (used by translation/learning modes)
+2. The main routing case statement
+
+**Before:** Switching to macOS provider would silently fail - no TTS output
+**After:** macOS provider correctly routes to `play-tts-macos.sh`
+
+```bash
+# This now works correctly:
+/agent-vibes:provider switch macos
+# TTS output will now use macOS say command
+```
+
+---
+
+## Test Improvements
+
+### New macOS Provider Tests
+**Files:** `test/unit/provider-manager.bats`, `test/unit/play-tts.bats`
+
+Added 4 new tests to ensure macOS provider is properly supported:
+
+| Test | Description |
+|------|-------------|
+| `provider-manager list shows available providers` | Verifies `macos` appears in provider list |
+| `provider-manager switch to macos` | Tests switching to macOS provider |
+| `provider-manager validate macos provider exists` | Validates macOS provider script exists |
+| `play-tts routes to macos provider when configured` | Tests actual TTS routing to macOS |
+| `play-tts speak_text function routes to macos` | Verifies internal routing function |
+
+### Fixed Translator Tests
+**File:** `test/unit/translator.bats`
+
+Fixed 3 pre-existing failing tests:
+- Tests now properly `cd` to project directory before running commands
+- Fixed locale warning filtering using `env LC_ALL=C bash -c`
+
+---
+
+## Files Modified
+
+| File | Changes |
+|------|---------|
+| `.claude/hooks/play-tts.sh` | Added macOS case handlers (+6 lines) |
+| `test/unit/provider-manager.bats` | Added macOS provider tests (+24 lines) |
+| `test/unit/play-tts.bats` | Added macOS routing tests (+45 lines) |
+| `test/unit/translator.bats` | Fixed PWD and locale issues (+13 lines) |
+
+---
+
+## Changes Summary
+
+**Commits:** 2
+- `fix: Add macOS provider routing to play-tts.sh`
+- `test: Add macOS provider tests and fix translator tests`
+
+**Files Changed:** 4
+**Lines Added:** 88
+**Lines Removed:** 11
+
+---
+
+## Migration Notes
+
+**No migration required** - This is a bug fix release.
+
+**Who is affected:** Mac users who updated to v2.14.1/v2.14.2 and tried to use the macOS provider.
+
+**Fix verification:**
+```bash
+# Update AgentVibes
+npx agentvibes update
+
+# Switch to macOS provider - should now speak
+/agent-vibes:provider switch macos
+```
+
+---
+
+## Testing
+
+All 133 tests pass:
+
+```bash
+npm test
+
+# Key tests:
+# ok 21 play-tts routes to macos provider when configured
+# ok 22 play-tts speak_text function routes to macos
+# ok 29 provider-manager switch to macos
+# ok 30 provider-manager validate macos provider exists
+```
+
+---
+
 # Release v2.14.2 - Native macOS TTS Provider
 
 **Release Date:** 2025-11-29

package/package.json

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

package/src/installer.js

@@ -128,18 +128,17 @@ function showReleaseInfo() {
   console.log(
     boxen(
       chalk.white.bold('═══════════════════════════════════════════════════════════════\n') +
-      chalk.cyan.bold('  📦 AgentVibes v2.14.2 - Native macOS TTS Provider\n') +
+      chalk.cyan.bold('  📦 AgentVibes v2.14.3 - macOS Provider Routing Fix\n') +
       chalk.white.bold('═══════════════════════════════════════════════════════════════\n\n') +
       chalk.green.bold('🎙️ WHAT\'S NEW:\n\n') +
-      chalk.cyan('AgentVibes v2.14.1 introduces native macOS TTS support via the\n') +
-      chalk.cyan('built-in `say` command. Mac users can now use AgentVibes with\n') +
-      chalk.cyan('zero setup - no API keys, no downloads, no configuration!\n\n') +
+      chalk.cyan('AgentVibes v2.14.3 fixes a critical bug where the macOS TTS\n') +
+      chalk.cyan('provider would not speak when selected. The TTS router was\n') +
+      chalk.cyan('missing case handlers for the macOS provider.\n\n') +
       chalk.green.bold('✨ KEY HIGHLIGHTS:\n\n') +
-      chalk.gray('   🍎 macOS Say Provider - Native TTS using macOS `say` (zero deps!)\n') +
-      chalk.gray('   🎤 100+ Built-in Voices - Access all Apple voices incl. Siri\n') +
-      chalk.gray('   🌍 40+ Languages - Full Apple voice library support\n') +
-      chalk.gray('   🔄 Three-Way Provider Switching - ElevenLabs, Piper, and macOS\n') +
-      chalk.gray('   🎉 Smarter BMAD Detection - Party mode recommendation on install\n\n') +
+      chalk.gray('   🐛 Fixed macOS Provider Routing - macOS `say` now works (Issue #52)\n') +
+      chalk.gray('   ✅ New Provider Tests - 4 tests ensure macOS is properly routed\n') +
+      chalk.gray('   🔧 Fixed Translator Tests - 3 pre-existing test failures resolved\n') +
+      chalk.gray('   🎯 133 Tests Passing - Full test suite with macOS coverage\n\n') +
       chalk.white.bold('═══════════════════════════════════════════════════════════════\n\n') +
       chalk.gray('📖 Full Release Notes: RELEASE_NOTES.md\n') +
       chalk.gray('🌐 Website: https://agentvibes.org\n') +

package/test/unit/play-tts.bats

@@ -90,3 +90,48 @@ teardown() {
   [ "$status" -eq 1 ]
   assert_output_contains "Error: No text provided"
 }
+
+# ============================================================================
+# Provider Routing Tests (Issue #52)
+# ============================================================================
+
+@test "play-tts routes to macos provider when configured" {
+  # Issue #52: macOS provider routing was missing
+  # This test ensures the macos case exists in play-tts.sh
+  echo "macos" > "$CLAUDE_PROJECT_DIR/.claude/tts-provider.txt"
+
+  # Create a mock play-tts-macos.sh that we can detect was called
+  local macos_script="$TEST_CLAUDE_DIR/hooks/play-tts-macos.sh"
+  cat > "$macos_script" << 'EOF'
+#!/usr/bin/env bash
+echo "MACOS_PROVIDER_CALLED"
+echo "Text: $1"
+echo "Voice: $2"
+exit 0
+EOF
+  chmod +x "$macos_script"
+
+  run "$PLAY_TTS" "Test macOS routing"
+
+  [ "$status" -eq 0 ]
+  # Should have called the macos provider
+  assert_output_contains "MACOS_PROVIDER_CALLED"
+}
+
+@test "play-tts speak_text function routes to macos" {
+  # Test the speak_text function also routes to macos (for translation/learning modes)
+  echo "macos" > "$CLAUDE_PROJECT_DIR/.claude/tts-provider.txt"
+
+  # Source play-tts.sh and test speak_text function
+  # We verify by checking the macos case exists in the script
+  run grep -A2 'case "\$provider" in' "$PLAY_TTS"
+
+  [ "$status" -eq 0 ]
+  # The macos case should exist in the speak_text function
+  run grep 'macos)' "$PLAY_TTS"
+  [ "$status" -eq 0 ]
+
+  # Count occurrences - should be 2 (one in speak_text, one in main routing)
+  local count=$(grep -c 'macos)' "$PLAY_TTS")
+  [ "$count" -ge 2 ]
+}

package/test/unit/provider-manager.bats

@@ -27,9 +27,10 @@ teardown() {
 
   [ "$status" -eq 0 ]
 
-  # Should show at least elevenlabs and piper
+  # Should show all three providers: elevenlabs, piper, and macos
   assert_output_contains "elevenlabs"
   assert_output_contains "piper"
+  assert_output_contains "macos"
 }
 
 @test "provider-manager list finds providers from play-tts files" {
@@ -38,9 +39,10 @@ teardown() {
   [ "$status" -eq 0 ]
 
   # Verify it's detecting actual provider scripts
-  # Both elevenlabs and piper should be present
+  # All three providers should be present (Issue #52)
   [[ "$output" =~ "elevenlabs" ]]
   [[ "$output" =~ "piper" ]]
+  [[ "$output" =~ "macos" ]]
 }
 
 @test "provider-manager list works with no providers (edge case)" {
@@ -94,6 +96,24 @@ teardown() {
   assert_file_contains "$PROVIDER_FILE" "elevenlabs"
 }
 
+@test "provider-manager switch to macos" {
+  # Issue #52: macOS provider must be switchable
+  run "$PROVIDER_MANAGER" switch "macos"
+
+  [ "$status" -eq 0 ]
+  assert_output_contains "Active provider set to: macos"
+
+  assert_file_exists "$PROVIDER_FILE"
+  assert_file_contains "$PROVIDER_FILE" "macos"
+}
+
+@test "provider-manager validate macos provider exists" {
+  # Issue #52: macOS provider must be recognized as valid
+  run "$PROVIDER_MANAGER" validate "macos"
+
+  [ "$status" -eq 0 ]
+}
+
 @test "provider-manager switch with invalid provider fails" {
   run "$PROVIDER_MANAGER" switch "nonexistent"
 

package/test/unit/translator.bats

@@ -132,7 +132,9 @@ teardown() {
 @test "translate-manager.sh get-translate-to returns language" {
   "$TRANSLATE_MANAGER" set french > /dev/null 2>&1
 
-  run bash -c "LC_ALL=C $TRANSLATE_MANAGER get-translate-to 2>/dev/null"
+  # Must run from same directory where 'set' wrote the config
+  # Set LC_ALL=C before bash to suppress locale warnings
+  run env LC_ALL=C bash -c "cd '$CLAUDE_PROJECT_DIR' && $TRANSLATE_MANAGER get-translate-to" 2>/dev/null
 
   [ "$status" -eq 0 ]
   [[ "$output" == "french" ]]
@@ -171,29 +173,33 @@ teardown() {
 # ============================================
 
 @test "translate-manager.sh detects BMAD communication_language" {
-  # Create mock BMAD config
-  mkdir -p ".bmad/core"
-  cat > ".bmad/core/config.yaml" << 'EOF'
+  # Create mock BMAD config in the project directory
+  mkdir -p "$CLAUDE_PROJECT_DIR/.bmad/core"
+  cat > "$CLAUDE_PROJECT_DIR/.bmad/core/config.yaml" << 'EOF'
 communication_language: Spanish
 document_output_language: Spanish
 EOF
 
-  run bash -c "LC_ALL=C $TRANSLATE_MANAGER get-bmad-language 2>/dev/null"
+  # Must run from project directory where BMAD config exists
+  # Set LC_ALL=C before bash to suppress locale warnings
+  run env LC_ALL=C bash -c "cd '$CLAUDE_PROJECT_DIR' && $TRANSLATE_MANAGER get-bmad-language" 2>/dev/null
 
   [ "$status" -eq 0 ]
   [[ "$output" == "spanish" ]]
 }
 
 @test "translate-manager.sh auto uses BMAD language when set" {
-  # Create mock BMAD config
-  mkdir -p ".bmad/core"
-  cat > ".bmad/core/config.yaml" << 'EOF'
+  # Create mock BMAD config in the project directory
+  mkdir -p "$CLAUDE_PROJECT_DIR/.bmad/core"
+  cat > "$CLAUDE_PROJECT_DIR/.bmad/core/config.yaml" << 'EOF'
 communication_language: French
 EOF
 
   "$TRANSLATE_MANAGER" auto > /dev/null 2>&1
 
-  run bash -c "LC_ALL=C $TRANSLATE_MANAGER get-translate-to 2>/dev/null"
+  # Must run from project directory where BMAD config and tts-translate-to.txt exist
+  # Set LC_ALL=C before bash to suppress locale warnings
+  run env LC_ALL=C bash -c "cd '$CLAUDE_PROJECT_DIR' && $TRANSLATE_MANAGER get-translate-to" 2>/dev/null
 
   [ "$status" -eq 0 ]
   [[ "$output" == "french" ]]