voicesmith-mcp 1.0.2 → 1.0.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "voicesmith-mcp",
-  "version": "1.0.2",
+  "version": "1.0.4",
   "description": "Local AI voice for coding assistants — TTS & STT via MCP. Kokoro ONNX + faster-whisper, fully offline.",
   "bin": {
     "voicesmith-mcp": "bin/cli.js"

package/server.py

@@ -206,7 +206,8 @@ class _VoiceHTTPHandler(BaseHTTPRequestHandler):
             self._json_response(400, {"error": "invalid_json"})
             return
 
-        name = params.get("name", _session_info.get("name", "Eric") if _session_info else "Eric")
+        default_name = _config.main_agent if _config else "Eric"
+        name = params.get("name", _session_info.get("name", default_name) if _session_info else default_name)
         text = params.get("text", "")
         speed = params.get("speed", 1.0)
 
@@ -806,12 +807,28 @@ def main():
 
 
 def _start_preheat_intro():
-    """Speak a brief intro after server starts. Preheats TTS engine."""
+    """Speak a brief intro after server starts. Preheats TTS engine.
+
+    Only speaks if this session got its preferred name. If the preferred name
+    was taken (e.g., during a session resume where the old server is still
+    running), skip the intro to avoid confusing double introductions.
+    """
     if _tts_engine is None or _audio_player is None:
         return
 
-    name = _session_info.get("name", "Eric") if _session_info else "Eric"
-    voice = _session_info.get("voice", "am_eric") if _session_info else "am_eric"
+    default_name = _config.main_agent if _config else "Eric"
+    default_voice = _config.tts.default_voice if _config else "am_eric"
+
+    name = _session_info.get("name", default_name) if _session_info else default_name
+    voice = _session_info.get("voice", default_voice) if _session_info else default_voice
+
+    # Determine what name we wanted
+    preferred = (_config.last_voice_name or default_name) if _config else default_name
+
+    # Skip intro if we didn't get our preferred name — another session has it
+    if name != preferred:
+        logger.info(f"Skipping preheat intro: wanted '{preferred}' but got '{name}'")
+        return
 
     def _intro():
         # Wait for server to settle

package/session_registry.py

@@ -65,7 +65,7 @@ def _write_sessions(path: Path, sessions: list[dict]) -> None:
 _STALE_ACTIVITY_THRESHOLD = 300  # 5 minutes
 
 
-def _session_healthy(session: dict) -> bool:
+def _session_healthy(session: dict, activity_threshold: int = _STALE_ACTIVITY_THRESHOLD) -> bool:
     """Check if a session is alive and actively used.
 
     Three checks:
@@ -97,7 +97,7 @@ def _session_healthy(session: dict) -> bool:
             # Check activity age — if server hasn't had MCP tool calls
             # in a while, it's orphaned
             age = data.get("last_tool_call_age_s")
-            if age is not None and age > _STALE_ACTIVITY_THRESHOLD:
+            if age is not None and age > activity_threshold:
                 logger.info(
                     f"Session '{session.get('name')}' (pid {pid}) inactive "
                     f"for {age}s — treating as stale"
@@ -117,11 +117,18 @@ def _session_healthy(session: dict) -> bool:
         return False
 
 
-def _clean_stale(sessions: list[dict]) -> list[dict]:
-    """Remove sessions that are dead or unresponsive."""
+def _clean_stale(sessions: list[dict], aggressive: bool = False) -> list[dict]:
+    """Remove sessions that are dead or unresponsive.
+
+    Args:
+        aggressive: If True, use a shorter activity threshold (10s instead of 5min).
+                    Used during startup registration to quickly reclaim names from
+                    orphaned servers that haven't fully shut down yet.
+    """
+    threshold = 10 if aggressive else _STALE_ACTIVITY_THRESHOLD
     alive = []
     for s in sessions:
-        if _session_healthy(s):
+        if _session_healthy(s, activity_threshold=threshold):
             alive.append(s)
         else:
             logger.info(f"Removed stale session: {s.get('name')} (pid {s.get('pid')})")
@@ -214,10 +221,22 @@ def register_session(
         fcntl.flock(f, fcntl.LOCK_EX)
 
         sessions = _read_sessions(path)
-        sessions = _clean_stale(sessions)
+        # Aggressive cleanup on startup — use short activity threshold
+        # to quickly reclaim names from orphaned servers
+        sessions = _clean_stale(sessions, aggressive=True)
 
         taken_names = {s["name"] for s in sessions}
 
+        if preferred_name in taken_names:
+            # Wait briefly and retry — the old server may be shutting down
+            fcntl.flock(f, fcntl.LOCK_UN)
+            time.sleep(2)
+            fcntl.flock(f, fcntl.LOCK_EX)
+            sessions = _read_sessions(path)
+            sessions = _clean_stale(sessions, aggressive=True)
+            _write_sessions(path, sessions)
+            taken_names = {s["name"] for s in sessions}
+
         if preferred_name in taken_names:
             name, voice = _find_available_name(taken_names, preferred_name)
             logger.warning(