agentvibes 2.17.2 → 2.17.4

package/.mcp.json

@@ -0,0 +1,30 @@
+{
+  "mcpServers": {
+    "sonarqube": {
+      "command": "docker",
+      "args": [
+        "run",
+        "-i",
+        "--name",
+        "sonarqube-mcp-server-agentvibes",
+        "--rm",
+        "--dns",
+        "8.8.8.8",
+        "--dns",
+        "8.8.4.4",
+        "-e",
+        "SONARQUBE_TOKEN",
+        "-e",
+        "SONARQUBE_ORG",
+        "-e",
+        "STORAGE_PATH",
+        "mcp/sonarqube"
+      ],
+      "env": {
+        "SONARQUBE_TOKEN": "${SONARQUBE_TOKEN}",
+        "SONARQUBE_ORG": "${SONARQUBE_ORG}",
+        "STORAGE_PATH": "${STORAGE_PATH}"
+      }
+    }
+  }
+}

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.17.2
+**Author**: Paul Preibisch ([@997Fire](https://x.com/997Fire)) | **Version**: v2.17.4
 
 ---
 
@@ -19,12 +19,13 @@
 
 | I want to... | Go here |
 |--------------|---------|
-| **Install AgentVibes** | [Quick Start Guide](docs/quick-start.md) |
+| **Install AgentVibes** (just `npx`, no git!) | [Quick Start Guide](docs/quick-start.md) |
+| **Understand what I need** (spoiler: just Node.js!) | [Prerequisites](#-prerequisites) |
 | **Set up on Windows (Claude Desktop)** | [Windows Setup Guide](mcp-server/WINDOWS_SETUP.md) |
 | **Use natural language** | [MCP Setup](docs/mcp-setup.md) |
 | **Switch voices** | [Voice Library](docs/voice-library.md) |
 | **Learn Spanish while coding** | [Language Learning Mode](docs/language-learning-mode.md) |
-| **Fix issues** | [Troubleshooting](docs/troubleshooting.md) |
+| **Fix issues** (git-lfs? MCP tokens? Read this!) | [Troubleshooting](docs/troubleshooting.md) & [FAQ](#-frequently-asked-questions-faq) |
 
 ---
 
@@ -74,6 +75,7 @@ All 50+ Piper voices AgentVibes provides are sourced from Hugging Face's open-so
 
 ### Getting Started
 - [🚀 Quick Start](#-quick-start) - Install in 3 steps
+- [📋 Prerequisites](#-prerequisites) - **NEW!** What you need (spoiler: just Node.js!)
 - [✨ What is AgentVibes?](#-what-is-agentvibes) - Overview & key features
 - [📰 Latest Release](#-latest-release) - What's new
 - [🪟 Windows Setup Guide for Claude Desktop](mcp-server/WINDOWS_SETUP.md) - Complete Windows installation with WSL & Python
@@ -103,6 +105,7 @@ All 50+ Piper voices AgentVibes provides are sourced from Hugging Face's open-so
 ### Additional Resources
 - [🔗 Useful Links](#-useful-links) - Voice typing & AI tools
 - [🔄 Updating](#-updating) - Keep AgentVibes current
+- [❓ FAQ](#-frequently-asked-questions-faq) - **NEW!** Common questions answered (git-lfs, MCP tokens, installation)
 - [🍎 macOS Testing](docs/macos-testing.md) - Automated testing on macOS with GitHub Actions
 - [🙏 Credits](#-credits) - Acknowledgments
 - [🤝 Contributing](#-contributing) - Show support
@@ -111,17 +114,18 @@ All 50+ Piper voices AgentVibes provides are sourced from Hugging Face's open-so
 
 ## 📰 Latest Release
 
-**[v2.17.0 - Installer UX Revolution](https://github.com/paulpreibisch/AgentVibes/releases/tag/v2.17.2)** 🎉
+**[v2.17.4 - Code Quality & Maintainability](https://github.com/paulpreibisch/AgentVibes/releases/tag/v2.17.4)** ⚙️
 
-AgentVibes v2.17.0 delivers a complete installer user experience transformation with intelligent system dependency checking, paginated configuration flow, and comprehensive inline help.
+AgentVibes v2.17.4 focuses on code quality and maintainability with extensive SonarCloud-driven refactoring across the entire codebase, reducing cognitive complexity in critical files by up to 90%.
 
 **Key Highlights:**
-- 🔧 **System Dependency Checker** - Validates Node, Python, bash, sox, ffmpeg, curl, etc.
-- 📄 **Paginated Configuration** - Beautiful headers with Agent Vibes branding on every page
-- 💡 **Inline Help & Commands** - TTS explanations and examples throughout installer
-- 🎵 **New Music Track** - Salsa v2 background music now available
-- 🎨 **Professional UI** - Consistent sunshine yellow styling and dynamic page numbering
-- ✅ **Quality Assurance** - 140 tests passing, SonarCloud integration
+- ⚙️ **Major Code Refactoring** - Cognitive complexity reduced by up to 90% across critical files
+- 🧪 **Comprehensive Testing** - 377 new party mode tests added, 225 total tests passing
+- 📚 **Improved Documentation** - New Prerequisites and FAQ sections clarify installation
+- 🎭 **Conversational Agent Intros** - BMAD party mode now uses "Hi! I'm John, your Product Manager" format
+- 🔊 **Better Audio Balance** - Background music reduced 15% for clearer TTS voice
+- 🏗️ **installer.js** - Complexity: 84 → ~8 (90% reduction)
+- 📦 **dependency-checker.js** - Complexity: 43 → ~5, 23 → ~8 (88% & 65% reductions)
 
 💡 **Tip:** If `npx agentvibes` shows an older version or missing commands, clear your npm cache: `npm cache clean --force && npx agentvibes@latest --help`
 
@@ -183,6 +187,51 @@ npx agentvibes install
 
 ---
 
+## 📋 Prerequisites
+
+### For Users (Installing & Running AgentVibes)
+
+**✅ Required:**
+- **Node.js** ≥16.0 (`node --version`)
+- That's it! No git required, no git-lfs required, no repo cloning needed.
+
+**❌ NOT Required:**
+- ❌ Git or git-lfs (you're installing via npm, not cloning)
+- ❌ Repository cloning (npm handles everything)
+- ❌ Build tools (pre-built package ready to use)
+
+**📦 Installation Methods:**
+
+| Method | Command | Use Case |
+|--------|---------|----------|
+| **✅ RECOMMENDED: NPM** | `npx agentvibes install` | Users wanting to use AgentVibes |
+| **⚠️ Developer Clone** | `git clone ...` | **Only for contributing code** |
+
+**💡 Why npm installation?**
+- Zero git operations - npm downloads pre-built package
+- No large file downloads - audio generated on-demand
+- Instant setup - works in seconds
+- No git-lfs, no cloning, no build steps
+
+### For Developers (Contributing to AgentVibes)
+
+**Only if contributing code to the project:**
+- Node.js 16+
+- Git (no git-lfs needed - repo has no large files)
+- Familiarity with `npm link` for local development
+
+**Developer installation:**
+```bash
+git clone https://github.com/paulpreibisch/AgentVibes.git
+cd AgentVibes
+npm install
+npm link
+```
+
+[↑ Back to top](#-table-of-contents)
+
+---
+
 ## 📋 System Requirements
 
 AgentVibes requires certain system dependencies for optimal audio processing and playback. Requirements vary by operating system and TTS provider.
@@ -591,9 +640,33 @@ AgentVibes supports **custom personalities** and **custom voices**.
 
 **Common Issues:**
 
+**❌ Error: "git-lfs is not installed"**
+
+**AgentVibes does NOT require git-lfs.** This error suggests:
+
+1. **Wrong installation method** - Use npm, not git clone:
+   ```bash
+   # ✅ CORRECT - Use this:
+   npx agentvibes install
+
+   # ❌ WRONG - Don't clone unless contributing:
+   git clone https://github.com/paulpreibisch/AgentVibes.git
+   ```
+
+2. **Different project** - You may be in a BMAD-METHOD or other repo that uses git-lfs
+
+3. **Global git config** - Your git may have lfs enabled globally:
+   ```bash
+   git config --global --list | grep lfs
+   ```
+
+**Solution:** Use `npx agentvibes install` - no git operations needed!
+
+---
+
 **No Audio Playing?**
-2. Verify hook is installed: `ls -la .claude/hooks/session-start-tts.sh`
-3. Test: `/agent-vibes:sample Aria`
+1. Verify hook is installed: `ls -la .claude/hooks/session-start-tts.sh`
+2. Test: `/agent-vibes:sample Aria`
 
 **Commands Not Found?**
 ```bash
@@ -630,6 +703,86 @@ npm update -g agentvibes && agentvibes update --yes
 
 ---
 
+## ❓ Frequently Asked Questions (FAQ)
+
+### Installation & Setup
+
+**Q: Does AgentVibes require git-lfs?**
+**A:** **NO.** AgentVibes has zero git-lfs requirement. Use `npx agentvibes install` - no git operations needed.
+
+**Q: Do I need to clone the GitHub repository?**
+**A:** **NO** (unless you're contributing code). Normal users should use `npx agentvibes install`. Repository cloning is only for developers who want to contribute to the project.
+
+**Q: Why is the GitHub repo so large?**
+**A:** The repo includes demo files and development dependencies (node_modules). The actual npm package you download is **< 50MB** and optimized for users.
+
+**Q: What's the difference between npm install and git clone?**
+**A:**
+- `npx agentvibes install` → **For users** - Downloads pre-built package, zero git operations, instant setup
+- `git clone ...` → **For developers only** - Full source code, development setup, contributing code
+
+**Q: I saw an error about git-lfs, is something wrong?**
+**A:** You're likely:
+1. Using wrong installation method (use `npx` not `git clone`)
+2. In a different project directory that uses git-lfs
+3. Have global git config with lfs enabled
+
+AgentVibes itself does NOT use or require git-lfs.
+
+### Features & Usage
+
+**Q: Does MCP consume tokens from my context window?**
+**A:** **YES.** Every MCP tool schema adds to the context window. AgentVibes MCP is designed to be minimal (~1500-2000 tokens), but if you're concerned about token usage, you can use slash commands instead of MCP.
+
+**Q: What's the difference between using MCP vs slash commands?**
+**A:**
+- **MCP**: Natural language ("Switch to Aria voice"), uses ~1500-2000 context tokens
+- **Slash commands**: Explicit commands (`/agent-vibes:switch Aria`), zero token overhead
+
+Both do the exact same thing - MCP is more convenient, slash commands are more token-efficient.
+
+**Q: Is AgentVibes just a bash script?**
+**A:** No. AgentVibes includes:
+- Multi-provider TTS abstraction (Piper, macOS Say, ElevenLabs)
+- Voice management system with 50+ voices
+- Personality & sentiment system
+- Language learning mode with bilingual playback
+- Audio effects processing (reverb, EQ, compression)
+- MCP server for natural language control
+- BMAD integration for multi-agent voice switching
+- Remote audio optimization for SSH/RDP sessions
+
+**Q: Can I use AgentVibes without BMAD?**
+**A:** **YES.** AgentVibes works standalone. BMAD integration is optional - only activates if you install BMAD separately.
+
+**Q: What are the audio dependencies?**
+**A:**
+- **Required**: Node.js 16+, Python 3.10+ (for Piper TTS)
+- **Optional**: sox (audio effects), ffmpeg (background music, padding)
+- All TTS generation works without optional dependencies - they just enhance the experience
+
+### Troubleshooting
+
+**Q: Why isn't Claude speaking?**
+**A:** Common causes:
+1. Hook not installed - Run `npx agentvibes install --yes`
+2. Audio player missing - Install `sox` and `ffmpeg`
+3. TTS protocol not enabled in settings
+4. Test with `/agent-vibes:sample Aria`
+
+**Q: Can I use this on Windows?**
+**A:** Yes, but requires WSL (Windows Subsystem for Linux). See [Windows Setup Guide](mcp-server/WINDOWS_SETUP.md).
+
+**Q: How do I reduce token usage?**
+**A:**
+1. Use slash commands instead of MCP (zero context token overhead)
+2. Set verbosity to LOW (`/agent-vibes:verbosity low`)
+3. Disable BMAD integration if not using it
+
+[↑ Back to top](#-table-of-contents)
+
+---
+
 ## ⚠️ Important Disclaimers
 
 **API Costs & Usage:**

package/mcp-server/server.py

@@ -54,6 +54,18 @@ import mcp.server.stdio
 class AgentVibesServer:
     """MCP Server for AgentVibes TTS functionality"""
 
+    # Script name constants (addresses SonarCloud S1192)
+    VOICE_MANAGER_SCRIPT = "voice-manager.sh"
+    PERSONALITY_MANAGER_SCRIPT = "personality-manager.sh"
+    LANGUAGE_MANAGER_SCRIPT = "language-manager.sh"
+    BACKGROUND_MUSIC_MANAGER_SCRIPT = "background-music-manager.sh"
+    EFFECTS_MANAGER_SCRIPT = "effects-manager.sh"
+
+    # Path constants (addresses SonarCloud S1192)
+    CLAUDE_DIR_NAME = ".claude"
+    MUTE_FILE_NAME = ".agentvibes-muted"
+    SEPARATOR = "━" * 39
+
     def __init__(self):
         """Initialize the AgentVibes MCP server"""
         # Find the .claude directory (project-local or global)
@@ -67,7 +79,7 @@ class AgentVibesServer:
         # Get the AgentVibes root directory (parent of mcp-server)
         script_dir = Path(__file__).resolve().parent  # mcp-server/
         agentvibes_root = script_dir.parent  # AgentVibes/
-        claude_dir = agentvibes_root / ".claude"
+        claude_dir = agentvibes_root / self.CLAUDE_DIR_NAME
 
         # ALWAYS use package .claude for hooks (even in NPX cache)
         # The package ALWAYS has .claude/ with all the hooks
@@ -75,7 +87,7 @@ class AgentVibesServer:
             return claude_dir
 
         # Fallback to global ~/.claude (should never happen in properly installed package)
-        return Path.home() / ".claude"
+        return Path.home() / self.CLAUDE_DIR_NAME
 
     async def text_to_speech(
         self,
@@ -105,13 +117,13 @@ class AgentVibesServer:
             if personality:
                 original_personality = await self._get_personality()
                 await self._run_script(
-                    "personality-manager.sh", ["set", personality]
+                    self.PERSONALITY_MANAGER_SCRIPT, ["set", personality]
                 )
 
             # Temporarily set language if specified
             if language:
                 original_language = await self._get_language()
-                await self._run_script("language-manager.sh", ["set", language])
+                await self._run_script(self.LANGUAGE_MANAGER_SCRIPT, ["set", language])
 
             # Call the TTS script via bash explicitly
             play_tts = self.hooks_dir / "play-tts.sh"
@@ -127,7 +139,7 @@ class AgentVibesServer:
             # 2. Otherwise → Use global ~/.claude/ (Claude Desktop, Warp, etc.)
             # Note: Hooks are ALWAYS from package .claude/ (self.claude_dir)
             cwd = Path.cwd()
-            if (cwd / ".claude").is_dir() and cwd != self.agentvibes_root:
+            if (cwd / self.CLAUDE_DIR_NAME).is_dir() and cwd != self.agentvibes_root:
                 # Real Claude Code project with .claude directory
                 env["CLAUDE_PROJECT_DIR"] = str(cwd)
             # else: Don't set CLAUDE_PROJECT_DIR, let scripts fall back to ~/.claude
@@ -177,11 +189,11 @@ class AgentVibesServer:
             # Restore original settings
             if original_personality:
                 await self._run_script(
-                    "personality-manager.sh", ["set", original_personality]
+                    self.PERSONALITY_MANAGER_SCRIPT, ["set", original_personality]
                 )
             if original_language:
                 await self._run_script(
-                    "language-manager.sh", ["set", original_language]
+                    self.LANGUAGE_MANAGER_SCRIPT, ["set", original_language]
                 )
 
     async def list_voices(self) -> str:
@@ -196,17 +208,17 @@ class AgentVibesServer:
         current_voice = await self._get_current_voice()
 
         # voice-manager.sh list-simple is now provider-aware
-        result = await self._run_script("voice-manager.sh", ["list-simple"])
+        result = await self._run_script(self.VOICE_MANAGER_SCRIPT, ["list-simple"])
         if result:
             voices = result.strip().split("\n")
             voices = [v for v in voices if v]  # Filter empty strings
 
             if not voices:
                 return (
-                    "📦 No voices available\n"
-                    "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n"
-                    "For Piper: Download voices using /agent-vibes:provider download <voice-name>\n"
-                    "Example: en_US-lessac-medium, en_GB-alba-medium"
+                    f"📦 No voices available\n"
+                    f"{self.SEPARATOR}\n"
+                    f"For Piper: Download voices using /agent-vibes:provider download <voice-name>\n"
+                    f"Example: en_US-lessac-medium, en_GB-alba-medium"
                 )
 
             # Determine provider label and alternative provider
@@ -221,11 +233,11 @@ class AgentVibesServer:
                 alternative_provider = None
 
             output = f"🎤 Available {provider_label} Voices:\n"
-            output += "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n"
+            output += f"{self.SEPARATOR}\n"
             for voice in voices:
                 marker = " ✓ (current)" if voice == current_voice else ""
                 output += f"  • {voice}{marker}\n"
-            output += "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n"
+            output += f"{self.SEPARATOR}\n"
 
             # Add provider switch hint
             if alternative_provider:
@@ -245,7 +257,7 @@ class AgentVibesServer:
             Success or error message
         """
         result = await self._run_script(
-            "voice-manager.sh", ["switch", voice_name, "--silent"]
+            self.VOICE_MANAGER_SCRIPT, ["switch", voice_name, "--silent"]
         )
         if result and "✅" in result:
             return f"✅ Voice switched to: {voice_name}"
@@ -258,7 +270,7 @@ class AgentVibesServer:
         Returns:
             Formatted list of personalities with descriptions
         """
-        result = await self._run_script("personality-manager.sh", ["list"])
+        result = await self._run_script(self.PERSONALITY_MANAGER_SCRIPT, ["list"])
         return result if result else "❌ Failed to list personalities"
 
     async def set_personality(self, personality: str) -> str:
@@ -272,7 +284,7 @@ class AgentVibesServer:
             Success or error message
         """
         result = await self._run_script(
-            "personality-manager.sh", ["set", personality]
+            self.PERSONALITY_MANAGER_SCRIPT, ["set", personality]
         )
         if result and "🎭" in result:
             return result
@@ -291,12 +303,12 @@ class AgentVibesServer:
         provider = await self._get_provider()
 
         output = "🎤 Current AgentVibes Configuration\n"
-        output += "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n"
+        output += f"{self.SEPARATOR}\n"
         output += f"Provider: {provider}\n"
         output += f"Voice: {voice}\n"
         output += f"Personality: {personality}\n"
         output += f"Language: {language}\n"
-        output += "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n"
+        output += f"{self.SEPARATOR}\n"
         return output
 
     async def set_language(self, language: str) -> str:
@@ -309,7 +321,7 @@ class AgentVibesServer:
         Returns:
             Success or error message
         """
-        result = await self._run_script("language-manager.sh", ["set", language])
+        result = await self._run_script(self.LANGUAGE_MANAGER_SCRIPT, ["set", language])
         if result and "✓" in result:
             return result
         return f"❌ Failed to set language: {result}"
@@ -324,7 +336,7 @@ class AgentVibesServer:
         Returns:
             Success or error message
         """
-        result = await self._run_script("voice-manager.sh", ["replay", str(n)])
+        result = await self._run_script(self.VOICE_MANAGER_SCRIPT, ["replay", str(n)])
         if result and "🔊" in result:
             return result
         return f"❌ Failed to replay audio: {result}"
@@ -489,7 +501,7 @@ class AgentVibesServer:
         Returns:
             Success message confirming mute is active
         """
-        mute_file = Path.home() / ".agentvibes-muted"
+        mute_file = Path.home() / self.MUTE_FILE_NAME
         try:
             mute_file.touch()
             return "🔇 AgentVibes TTS muted. All voice output is now silenced.\n\n💡 To unmute, use: unmute()"
@@ -503,8 +515,8 @@ class AgentVibesServer:
         Returns:
             Success message confirming TTS is restored
         """
-        global_mute = Path.home() / ".agentvibes-muted"
-        project_mute = Path.cwd() / ".claude" / "agentvibes-muted"
+        global_mute = Path.home() / self.MUTE_FILE_NAME
+        project_mute = Path.cwd() / self.CLAUDE_DIR_NAME / "agentvibes-muted"
 
         removed = []
         try:
@@ -529,8 +541,8 @@ class AgentVibesServer:
         Returns:
             Current mute status
         """
-        global_mute = Path.home() / ".agentvibes-muted"
-        project_mute = Path.cwd() / ".claude" / "agentvibes-muted"
+        global_mute = Path.home() / self.MUTE_FILE_NAME
+        project_mute = Path.cwd() / self.CLAUDE_DIR_NAME / "agentvibes-muted"
 
         if global_mute.exists() or project_mute.exists():
             return "🔇 TTS is currently MUTED\n\n💡 To unmute, use: unmute()"
@@ -544,7 +556,7 @@ class AgentVibesServer:
         Returns:
             Formatted list of all pre-packaged background music files
         """
-        result = await self._run_script("background-music-manager.sh", ["list"])
+        result = await self._run_script(self.BACKGROUND_MUSIC_MANAGER_SCRIPT, ["list"])
         return result if result else "❌ Failed to list background music"
 
     async def set_background_music(self, track_name: str, agent_name: Optional[str] = None) -> str:
@@ -561,7 +573,7 @@ class AgentVibesServer:
         import re
 
         # Get list of available tracks for fuzzy matching
-        list_result = await self._run_script("background-music-manager.sh", ["list"])
+        list_result = await self._run_script(self.BACKGROUND_MUSIC_MANAGER_SCRIPT, ["list"])
         if not list_result or "❌" in list_result:
             return "❌ Failed to list background music tracks"
 
@@ -597,13 +609,13 @@ class AgentVibesServer:
         # Determine which command to use based on agent_name
         if agent_name and agent_name.lower() == "all":
             # Set for all agents
-            result = await self._run_script("background-music-manager.sh", ["set-all", matched_track])
+            result = await self._run_script(self.BACKGROUND_MUSIC_MANAGER_SCRIPT, ["set-all", matched_track])
         elif agent_name:
             # Set for specific agent
-            result = await self._run_script("background-music-manager.sh", ["set-agent", agent_name, matched_track])
+            result = await self._run_script(self.BACKGROUND_MUSIC_MANAGER_SCRIPT, ["set-agent", agent_name, matched_track])
         else:
             # Set as default
-            result = await self._run_script("background-music-manager.sh", ["set-default", matched_track])
+            result = await self._run_script(self.BACKGROUND_MUSIC_MANAGER_SCRIPT, ["set-default", matched_track])
 
         if result and "✅" in result:
             if matched_track.lower() != track_name.lower():
@@ -622,7 +634,7 @@ class AgentVibesServer:
             Success or error message
         """
         command = "on" if enabled else "off"
-        result = await self._run_script("background-music-manager.sh", [command])
+        result = await self._run_script(self.BACKGROUND_MUSIC_MANAGER_SCRIPT, [command])
         return result if result else f"❌ Failed to {'enable' if enabled else 'disable'} background music"
 
     async def set_background_music_volume(self, volume: float) -> str:
@@ -635,7 +647,7 @@ class AgentVibesServer:
         Returns:
             Success or error message
         """
-        result = await self._run_script("background-music-manager.sh", ["volume", str(volume)])
+        result = await self._run_script(self.BACKGROUND_MUSIC_MANAGER_SCRIPT, ["volume", str(volume)])
         return result if result else "❌ Failed to set background music volume"
 
     async def get_background_music_status(self) -> str:
@@ -645,7 +657,7 @@ class AgentVibesServer:
         Returns:
             Status information
         """
-        result = await self._run_script("background-music-manager.sh", ["status"])
+        result = await self._run_script(self.BACKGROUND_MUSIC_MANAGER_SCRIPT, ["status"])
         return result if result else "❌ Failed to get background music status"
 
     async def set_reverb(self, level: str, agent: str = "default", apply_all: bool = False) -> str:
@@ -663,7 +675,7 @@ class AgentVibesServer:
         args = ["set-reverb", level, agent]
         if apply_all:
             args.append("--all")
-        result = await self._run_script("effects-manager.sh", args)
+        result = await self._run_script(self.EFFECTS_MANAGER_SCRIPT, args)
         return result if result else f"✅ Set reverb to {level}"
 
     async def get_reverb(self, agent: str = "default") -> str:
@@ -676,7 +688,7 @@ class AgentVibesServer:
         Returns:
             Current reverb level
         """
-        result = await self._run_script("effects-manager.sh", ["get-reverb", agent])
+        result = await self._run_script(self.EFFECTS_MANAGER_SCRIPT, ["get-reverb", agent])
         if result:
             return f"Current reverb level for {agent}: {result.strip()}"
         return f"❌ Failed to get reverb for {agent}"
@@ -688,7 +700,7 @@ class AgentVibesServer:
         Returns:
             Effects configuration
         """
-        result = await self._run_script("effects-manager.sh", ["list"])
+        result = await self._run_script(self.EFFECTS_MANAGER_SCRIPT, ["list"])
         return result if result else "❌ Failed to list audio effects"
 
     # Helper methods
@@ -749,7 +761,7 @@ class AgentVibesServer:
 
     async def _get_current_voice(self) -> str:
         """Get the currently active voice"""
-        result = await self._run_script("voice-manager.sh", ["get"])
+        result = await self._run_script(self.VOICE_MANAGER_SCRIPT, ["get"])
         return result.strip() if result else "Unknown"
 
     async def _get_personality(self) -> str:
@@ -757,7 +769,7 @@ class AgentVibesServer:
         personality_file = self.claude_dir / "tts-personality.txt"
         if not personality_file.exists():
             # Try global
-            personality_file = Path.home() / ".claude" / "tts-personality.txt"
+            personality_file = Path.home() / self.CLAUDE_DIR_NAME / "tts-personality.txt"
 
         try:
             if personality_file.exists():
@@ -770,14 +782,14 @@ class AgentVibesServer:
 
     async def _get_language(self) -> str:
         """Get the current language setting"""
-        result = await self._run_script("language-manager.sh", ["code"])
+        result = await self._run_script(self.LANGUAGE_MANAGER_SCRIPT, ["code"])
         return result.strip() if result else "english"
 
     async def _get_provider(self) -> str:
         """Get the active TTS provider"""
         provider_file = self.claude_dir / "tts-provider.txt"
         if not provider_file.exists():
-            provider_file = Path.home() / ".claude" / "tts-provider.txt"
+            provider_file = Path.home() / self.CLAUDE_DIR_NAME / "tts-provider.txt"
 
         try:
             if provider_file.exists():

package/package.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://json.schemastore.org/package.json",
   "name": "agentvibes",
-  "version": "2.17.2",
+  "version": "2.17.4",
   "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": [
@@ -64,9 +64,10 @@
     "install-local": "node src/installer.js install",
     "postinstall": "node mcp-server/install-deps.js",
     "install-mcp-deps": "node mcp-server/install-deps.js",
-    "test": "AGENTVIBES_TEST_MODE=true bats test/unit/*.bats && node --test test/unit/*.test.js",
+    "test": "AGENTVIBES_TEST_MODE=true bats test/unit/*.bats && npm run test:coverage",
     "test:bats": "AGENTVIBES_TEST_MODE=true bats test/unit/*.bats",
     "test:node": "node --test test/unit/*.test.js",
+    "test:coverage": "c8 --reporter=lcov --reporter=text node --test test/unit/*.test.js",
     "test:verbose": "AGENTVIBES_TEST_MODE=true bats -t test/unit/*.bats"
   },
   "dependencies": {
@@ -83,5 +84,8 @@
   },
   "publishConfig": {
     "access": "public"
+  },
+  "devDependencies": {
+    "c8": "^10.1.3"
   }
 }