agentvibes 5.1.1 → 5.1.2

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**: v5.1.1
+**Author**: Paul Preibisch ([@997Fire](https://x.com/997Fire)) | **Version**: v5.1.2
 
 ---
 
@@ -43,6 +43,16 @@ Whether you're using Claude Code, GitHub Copilot, OpenAI Codex, Claude Desktop,
 
 ---
 
+## 🔀 v5.1.2 — MCP Per-LLM Routing Hotfix
+
+- **MCP server now reads `AGENTVIBES_LLM` env var** instead of hardcoding `copilot` — Codex / Copilot / Claude Code each get routed to their own per-LLM voice / pretext / music / effects config from `audio-effects.cfg`.
+- **MCP launcher templates set the env var** automatically — `.codex/config.toml`, `.vscode/mcp.json`, and `.mcp.json` all include `AGENTVIBES_LLM` for the right provider.
+- **24 new regression tests** prevent this class of bug from shipping again, including a `npm pack` content guard that fails the test suite if the working tree has uncommitted changes (the v5.1.0 disaster guard).
+
+If you already have AgentVibes installed, re-run the per-provider configure step or add `"env": { "AGENTVIBES_LLM": "<your-llm>" }` manually to your MCP config.
+
+---
+
 ## 🩹 v5.1.1 — Windows TTS Hook Hotfix
 
 - **`play-tts.ps1 -llm` parameter restored** — npm-published v5.1.0 shipped a regressed copy without `-llm` support, breaking the Setup tab Preview button and the agentvibes MCP `text_to_speech` tool on Windows. Fixed in v5.1.1. If you hit the error, clear your npx cache: `npm cache clean --force` then reinstall.

package/RELEASE_NOTES.md

@@ -1,5 +1,42 @@
 # AgentVibes Release Notes
 
+## 🔀 v5.1.2 — MCP Per-LLM Routing Hotfix
+
+**Release Date:** April 2026
+
+### Bug Fixes
+
+- **MCP server no longer hardcodes `-llm copilot`** — `mcp-server/server.py` was passing `-llm copilot` for every caller, regardless of whether the actual client was Codex, Copilot, or Claude Code. This meant per-LLM voice/pretext/music routing in `audio-effects.cfg` was effectively broken: all three providers fell through to the same `llm:copilot` lookup (or worse, the global default config). The server now reads `AGENTVIBES_LLM` from the environment and forwards that as the `-llm` value, falling back to no `-llm` flag if unset.
+
+- **MCP launcher templates set `AGENTVIBES_LLM`** — Each provider's MCP config now sets the env var so the server knows which LLM is calling:
+  - Codex: `.codex/config.toml` → `env = { AGENTVIBES_LLM = "codex" }`
+  - Copilot: `.vscode/mcp.json` → `"env": { "AGENTVIBES_LLM": "copilot" }`
+  - Claude Code: `.mcp.json` → `"env": { "AGENTVIBES_LLM": "claude-code" }`
+
+  Both freshly-installed configs and the manual-instruction snippets shown by the installer include the env var.
+
+### Testing & Hardening
+
+- **24 new regression tests** in `test/unit/llm-provider-mcp-routing.test.js` and `test/unit/npm-pack-contents.test.js` enforce the contract end-to-end:
+  - Each MCP launcher template MUST set `AGENTVIBES_LLM` to the correct value
+  - `mcp-server/server.py` MUST read `AGENTVIBES_LLM` from env and MUST NOT hardcode `-llm copilot`
+  - `play-tts.ps1` MUST declare its `$llm` parameter, contain per-LLM lookup logic, export `AGENTVIBES_LLM_KEY`, and be at least 400 lines (catches the v5.1.0 mass-deletion regression)
+  - `play-tts.sh` MUST parse `--llm` and do per-LLM lookup against `audio-effects.cfg`
+  - **Working tree must be clean** before publishing (the v5.1.0 disaster guard — `npm publish` packs the working tree, not the git tag, so any uncommitted local edits get shipped)
+
+- The contract tests are **mutation-tested**: temporarily replacing `play-tts.ps1` with a broken stub fires 4 assertions with clear messages, proving the exact v5.1.0 regression would now block publish.
+
+### How to Update
+
+```
+npm cache clean --force
+npx --yes agentvibes@5.1.2
+```
+
+If you already have AgentVibes installed for one or more providers, re-run the installer's per-provider configure step (or manually add `"env": { "AGENTVIBES_LLM": "<your-llm>" }` to your `.mcp.json` / `.vscode/mcp.json` / `.codex/config.toml`).
+
+---
+
 ## 🩹 v5.1.1 — Windows TTS Hook Hotfix
 
 **Release Date:** April 2026

package/mcp-server/server.py

@@ -194,18 +194,33 @@ class AgentVibesServer:
                 original_language = await self._get_language()
                 await self._run_script(self.LANGUAGE_MANAGER_SCRIPT, ["set", language])
 
-            # Call the TTS script via appropriate shell
+            # Call the TTS script via appropriate shell.
+            #
+            # The LLM key is read from the AGENTVIBES_LLM env var so each
+            # caller (Claude Code, GitHub Copilot, OpenAI Codex) gets routed
+            # to its own per-LLM voice / pretext / music / effects config.
+            # The MCP launcher in each provider's config file should set
+            # this env var (e.g. .codex/config.toml [mcp_servers.agentvibes]
+            # env = { AGENTVIBES_LLM = "codex" }).
+            #
+            # If unset, no -llm flag is passed and play-tts falls back to
+            # the project / global default config — which is the correct
+            # behavior for ad-hoc / unconfigured callers.
+            llm_key = os.environ.get("AGENTVIBES_LLM", "").strip()
             tts_script = "play-tts.ps1" if self.is_windows else "play-tts.sh"
             play_tts = self.hooks_dir / tts_script
             if self.is_windows:
-                args = ["powershell", "-NoProfile", "-ExecutionPolicy", "Bypass", "-File", str(play_tts), text, "-llm", "copilot"]
+                args = ["powershell", "-NoProfile", "-ExecutionPolicy", "Bypass", "-File", str(play_tts), text]
                 if voice:
                     args.extend(["-VoiceOverride", voice])
+                if llm_key:
+                    args.extend(["-llm", llm_key])
             else:
                 args = ["bash", str(play_tts), text]
                 if voice:
                     args.append(voice)
-                args.extend(["--llm", "copilot"])
+                if llm_key:
+                    args.extend(["--llm", llm_key])
 
             env = self._build_script_env()
             # Set agent name for audio effects lookup (audio-effects.cfg, background music config)

package/package.json

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

@@ -4297,12 +4297,15 @@ function isPathSafe(targetPath, basePath) {
 async function handleMcpConfiguration(targetDir, options) {
   const mcpConfigPath = path.join(targetDir, '.mcp.json');
 
-  // MCP server configuration for AgentVibes
+  // MCP server configuration for AgentVibes.
+  // The AGENTVIBES_LLM env var tells the MCP server which LLM is calling
+  // so per-LLM voice / pretext / music / effects routing works correctly.
   const mcpConfig = {
     mcpServers: {
       agentvibes: {
         command: 'npx',
-        args: ['-y', '--package=agentvibes', 'agentvibes-mcp-server']
+        args: ['-y', '--package=agentvibes', 'agentvibes-mcp-server'],
+        env: { AGENTVIBES_LLM: 'claude-code' }
       }
     }
   };
@@ -4337,7 +4340,8 @@ async function handleMcpConfiguration(targetDir, options) {
     console.log(
       '\n"agentvibes": {\n' +
       '  "command": "npx",\n' +
-      '  "args": ["-y", "--package=agentvibes", "agentvibes-mcp-server"]\n' +
+      '  "args": ["-y", "--package=agentvibes", "agentvibes-mcp-server"],\n' +
+      '  "env": { "AGENTVIBES_LLM": "claude-code" }\n' +
       '}\n'
     );
 
@@ -4442,7 +4446,8 @@ async function handleMcpConfiguration(targetDir, options) {
       '  "mcpServers": {\n' +
       '    "agentvibes": {\n' +
       '      "command": "npx",\n' +
-      '      "args": ["-y", "--package=agentvibes", "agentvibes-mcp-server"]\n' +
+      '      "args": ["-y", "--package=agentvibes", "agentvibes-mcp-server"],\n' +
+      '      "env": { "AGENTVIBES_LLM": "claude-code" }\n' +
       '    }\n' +
       '  }\n' +
       '}\n'

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

@@ -228,6 +228,9 @@ export async function installCopilotMcp(targetDir) {
     type: 'stdio',
     command: 'npx',
     args: ['-y', '--package=agentvibes', 'agentvibes-mcp-server'],
+    // Tells the MCP server which LLM is calling so per-LLM voice / pretext
+    // / music / effects routing in audio-effects.cfg works correctly.
+    env: { AGENTVIBES_LLM: 'copilot' },
   };
 
   try {
@@ -337,6 +340,9 @@ export function buildCodexToml(existingContent = '') {
     '[mcp_servers.agentvibes]',
     'command = "npx"',
     'args = ["-y", "--package=agentvibes", "agentvibes-mcp-server"]',
+    // Tells the MCP server which LLM is calling so per-LLM voice / pretext
+    // / music / effects routing in audio-effects.cfg works correctly.
+    'env = { AGENTVIBES_LLM = "codex" }',
   ].join('\n');
 
   if (!existingContent.trim()) return serverBlock + '\n';