agentvibes 4.0.1 → 4.4.0

package/.claude/hooks/tts-queue-worker.sh

@@ -1,114 +1,145 @@
-#!/usr/bin/env bash
-#
-# File: .claude/hooks/tts-queue-worker.sh
-#
-# TTS Queue Worker - Background process that plays queued TTS sequentially
-# Automatically exits when queue is empty for 5 seconds
-
-set -euo pipefail
-
-# Security: Use secure temp directory with restrictive permissions
-# Must match the logic in tts-queue.sh exactly
-if [[ -n "${XDG_RUNTIME_DIR:-}" ]] && [[ -d "$XDG_RUNTIME_DIR" ]]; then
-  QUEUE_DIR="$XDG_RUNTIME_DIR/agentvibes-tts-queue"
-else
-  # Fallback to user-specific temp directory
-  QUEUE_DIR="/tmp/agentvibes-tts-queue-$USER"
-fi
-
-# Security: Validate queue directory exists and has correct ownership
-if [[ ! -d "$QUEUE_DIR" ]]; then
-  echo "Error: Queue directory does not exist: $QUEUE_DIR" >&2
-  exit 1
-fi
-
-# Security: Verify we own the queue directory (prevent symlink attacks)
-if [[ "$(stat -c '%u' "$QUEUE_DIR" 2>/dev/null || stat -f '%u' "$QUEUE_DIR" 2>/dev/null)" != "$(id -u)" ]]; then
-  echo "Error: Queue directory not owned by current user" >&2
-  exit 1
-fi
-
-WORKER_PID_FILE="$QUEUE_DIR/worker.pid"
-IDLE_TIMEOUT=5  # Exit after 5 seconds of no new requests
-
-# Get script directory
-SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-
-# Configurable delay between speakers (seconds)
-# Can be overridden by .claude/tts-speaker-delay.txt or ~/.claude/tts-speaker-delay.txt
-SPEAKER_DELAY=4  # Default: 4 seconds between speakers
-
-# Check for custom delay configuration
-if [[ -f ".claude/tts-speaker-delay.txt" ]]; then
-  CUSTOM_DELAY=$(cat .claude/tts-speaker-delay.txt 2>/dev/null | tr -d '[:space:]')
-  if [[ "$CUSTOM_DELAY" =~ ^[0-9]+$ ]]; then
-    SPEAKER_DELAY=$CUSTOM_DELAY
-  fi
-elif [[ -f "$HOME/.claude/tts-speaker-delay.txt" ]]; then
-  CUSTOM_DELAY=$(cat "$HOME/.claude/tts-speaker-delay.txt" 2>/dev/null | tr -d '[:space:]')
-  if [[ "$CUSTOM_DELAY" =~ ^[0-9]+$ ]]; then
-    SPEAKER_DELAY=$CUSTOM_DELAY
-  fi
-fi
-
-# Trap to clean up on exit
-trap 'rm -f "$WORKER_PID_FILE"' EXIT
-
-# Process queue items
-process_queue() {
-  local idle_count=0
-
-  while true; do
-    # Find oldest queue item
-    local queue_item=$(ls -1 "$QUEUE_DIR"/*.queue 2>/dev/null | sort | head -1)
-
-    if [[ -z "$queue_item" ]]; then
-      # Queue is empty, increment idle counter
-      idle_count=$((idle_count + 1))
-
-      if [[ $idle_count -ge $IDLE_TIMEOUT ]]; then
-        # No new items for timeout period, exit worker
-        exit 0
-      fi
-
-      # Wait 1 second and check again
-      sleep 1
-      continue
-    fi
-
-    # Reset idle counter - we have work
-    idle_count=0
-
-    # Load TTS request
-    source "$queue_item"
-
-    # Decode base64 values
-    TEXT=$(echo -n "$TEXT_B64" | base64 -d)
-    VOICE=$(echo -n "$VOICE_B64" | base64 -d)
-    AGENT=$(echo -n "${AGENT_B64:-}" | base64 -d 2>/dev/null || echo "default")
-
-    # Use enhanced TTS with agent-specific background music if agent is specified
-    # and background music is enabled
-    if [[ -f "$SCRIPT_DIR/play-tts-enhanced.sh" ]] && [[ "$AGENT" != "default" ]] && [[ -n "$AGENT" ]]; then
-      # Party mode: each agent gets their unique background music from audio-effects.cfg
-      bash "$SCRIPT_DIR/play-tts-enhanced.sh" "$TEXT" "$AGENT" "$VOICE" || true
-    else
-      # Standard TTS without background music
-      # Display output to show file location (GitHub Issue #39)
-      if [[ -n "${VOICE:-}" ]]; then
-        bash "$SCRIPT_DIR/play-tts.sh" "$TEXT" "$VOICE" || true
-      else
-        bash "$SCRIPT_DIR/play-tts.sh" "$TEXT" || true
-      fi
-    fi
-
-    # Add configurable pause between speakers for natural conversation flow
-    sleep $SPEAKER_DELAY
-
-    # Remove processed item
-    rm -f "$queue_item"
-  done
-}
-
-# Start processing
-process_queue
+#!/usr/bin/env bash
+#
+# File: .claude/hooks/tts-queue-worker.sh
+#
+# TTS Queue Worker - Background process that plays queued TTS sequentially
+# Automatically exits when queue is empty for 5 seconds
+
+set -euo pipefail
+
+# Security: Use secure temp directory with restrictive permissions
+# Must match the logic in tts-queue.sh exactly
+if [[ -n "${XDG_RUNTIME_DIR:-}" ]] && [[ -d "$XDG_RUNTIME_DIR" ]]; then
+  QUEUE_DIR="$XDG_RUNTIME_DIR/agentvibes-tts-queue"
+else
+  # Fallback to user-specific temp directory
+  QUEUE_DIR="/tmp/agentvibes-tts-queue-$(id -u)"
+fi
+
+# Security: Validate queue directory exists and has correct ownership
+if [[ ! -d "$QUEUE_DIR" ]]; then
+  echo "Error: Queue directory does not exist: $QUEUE_DIR" >&2
+  exit 1
+fi
+
+# Security: Verify we own the queue directory (prevent symlink attacks)
+if [[ "$(stat -c '%u' "$QUEUE_DIR" 2>/dev/null || stat -f '%u' "$QUEUE_DIR" 2>/dev/null)" != "$(id -u)" ]]; then
+  echo "Error: Queue directory not owned by current user" >&2
+  exit 1
+fi
+
+WORKER_PID_FILE="$QUEUE_DIR/worker.pid"
+IDLE_TIMEOUT=5  # Exit after 5 seconds of no new requests
+
+# Get script directory
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+
+# Configurable delay between speakers (seconds)
+# Can be overridden by .claude/tts-speaker-delay.txt or ~/.claude/tts-speaker-delay.txt
+SPEAKER_DELAY=4  # Default: 4 seconds between speakers
+
+# Check for custom delay configuration
+if [[ -f ".claude/tts-speaker-delay.txt" ]]; then
+  CUSTOM_DELAY=$(cat .claude/tts-speaker-delay.txt 2>/dev/null | tr -d '[:space:]')
+  if [[ "$CUSTOM_DELAY" =~ ^[0-9]+$ ]]; then
+    SPEAKER_DELAY=$CUSTOM_DELAY
+  fi
+elif [[ -f "$HOME/.claude/tts-speaker-delay.txt" ]]; then
+  CUSTOM_DELAY=$(cat "$HOME/.claude/tts-speaker-delay.txt" 2>/dev/null | tr -d '[:space:]')
+  if [[ "$CUSTOM_DELAY" =~ ^[0-9]+$ ]]; then
+    SPEAKER_DELAY=$CUSTOM_DELAY
+  fi
+fi
+
+# Trap to clean up on exit
+trap 'rm -f "$WORKER_PID_FILE"' EXIT
+
+# Process queue items
+process_queue() {
+  local idle_count=0
+
+  while true; do
+    # Find oldest queue item
+    local queue_item=$(ls -1 "$QUEUE_DIR"/*.queue 2>/dev/null | sort | head -1)
+
+    if [[ -z "$queue_item" ]]; then
+      # Queue is empty, increment idle counter
+      idle_count=$((idle_count + 1))
+
+      if [[ $idle_count -ge $IDLE_TIMEOUT ]]; then
+        # No new items for timeout period, exit worker
+        exit 0
+      fi
+
+      # Wait for a new queue item — use inotifywait if available to avoid polling
+      # Use a 1-second timeout (-t 1) so the idle counter still advances correctly
+      if command -v inotifywait &>/dev/null; then
+        inotifywait -q -e create -t 1 "$QUEUE_DIR" 2>/dev/null || true
+      else
+        sleep 1
+      fi
+      continue
+    fi
+
+    # Reset idle counter - we have work
+    idle_count=0
+
+    # Load queue item — explicit key=value parsing (SECURITY: never source untrusted files)
+    TEXT_FILE=""
+    VOICE=""
+    AGENT=""
+    PROFILE_PATH=""
+    PLAY_WAV=""
+    while IFS='=' read -r _key _val; do
+      case "$_key" in
+        TEXT_FILE)     TEXT_FILE="$_val" ;;
+        VOICE)         VOICE="$_val" ;;
+        AGENT)         AGENT="$_val" ;;
+        PROFILE_PATH)  PROFILE_PATH="$_val" ;;
+        PLAY_WAV)      PLAY_WAV="$_val" ;;
+      esac
+    done < "$queue_item"
+
+    # Check if this is a pre-generated WAV playback item
+    if [[ -n "${PLAY_WAV:-}" ]] && [[ -f "$PLAY_WAV" ]]; then
+      # Play the pre-generated WAV directly (synthesis already done by bmad-speak)
+      if command -v paplay &>/dev/null; then
+        paplay "$PLAY_WAV" 2>/dev/null || true
+      elif command -v aplay &>/dev/null; then
+        aplay -q "$PLAY_WAV" 2>/dev/null || true
+      elif command -v ffplay &>/dev/null; then
+        ffplay -nodisp -autoexit -loglevel quiet "$PLAY_WAV" 2>/dev/null || true
+      fi
+    else
+      # Full TTS request — read text from companion file, use voice/agent directly
+      TEXT=""
+      if [[ -n "${TEXT_FILE:-}" ]] && [[ -f "$TEXT_FILE" ]]; then
+        TEXT=$(cat "$TEXT_FILE")
+        rm -f "$TEXT_FILE"
+      fi
+      AGENT_PROFILE="${PROFILE_PATH:-}"
+
+      export AGENTVIBES_AGENT_PROFILE="$AGENT_PROFILE"
+
+      if [[ -n "${VOICE:-}" ]]; then
+        bash "$SCRIPT_DIR/play-tts.sh" "$TEXT" "${VOICE}" || true
+      else
+        bash "$SCRIPT_DIR/play-tts.sh" "$TEXT" || true
+      fi
+
+      if [[ -n "$AGENT_PROFILE" ]] && [[ -f "$AGENT_PROFILE" ]]; then
+        rm -f "$AGENT_PROFILE"
+      fi
+      unset AGENTVIBES_AGENT_PROFILE
+    fi
+
+    # Add configurable pause between speakers for natural conversation flow
+    sleep $SPEAKER_DELAY
+
+    # Remove processed item and any companion text file
+    rm -f "$queue_item" "${queue_item%.queue}.txt"
+  done
+}
+
+# Start processing
+process_queue

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

@@ -1,136 +1,165 @@
-#!/usr/bin/env bash
-#
-# File: .claude/hooks/tts-queue.sh
-#
-# TTS Queue Manager for Party Mode
-# Queues TTS requests and plays them sequentially in the background
-# This allows Claude to continue generating responses while audio plays in order
-
-set -euo pipefail
-
-# Security: Use secure temp directory with restrictive permissions
-# Check if XDG_RUNTIME_DIR is available (more secure than /tmp)
-if [[ -n "${XDG_RUNTIME_DIR:-}" ]] && [[ -d "$XDG_RUNTIME_DIR" ]]; then
-  QUEUE_DIR="$XDG_RUNTIME_DIR/agentvibes-tts-queue"
-else
-  # Fallback to user-specific temp directory
-  QUEUE_DIR="/tmp/agentvibes-tts-queue-$USER"
-fi
-
-QUEUE_LOCK="$QUEUE_DIR/queue.lock"
-WORKER_PID_FILE="$QUEUE_DIR/worker.pid"
-
-# Get script directory
-SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-
-# Initialize queue directory with restrictive permissions
-if [[ ! -d "$QUEUE_DIR" ]]; then
-  mkdir -p "$QUEUE_DIR"
-  chmod 700 "$QUEUE_DIR"  # Only owner can read/write/execute
-fi
-
-# @function add_to_queue
-# @intent Add a TTS request to the queue for sequential playback
-# @param $1 dialogue text
-# @param $2 voice name (optional)
-# @param $3 agent name (optional, for background music in party mode)
-add_to_queue() {
-  local text="$1"
-  local voice="${2:-}"
-  local agent="${3:-default}"
-
-  # Create unique queue item with timestamp
-  local timestamp=$(date +%s%N)
-  local queue_file="$QUEUE_DIR/$timestamp.queue"
-
-  # Write request to queue file (base64 encoded to handle all special chars)
-  cat > "$queue_file" <<EOF
-TEXT_B64=$(echo -n "$text" | base64 -w0)
-VOICE_B64=$(echo -n "$voice" | base64 -w0)
-AGENT_B64=$(echo -n "$agent" | base64 -w0)
-EOF
-
-  # Start queue worker if not already running
-  start_worker_if_needed
-}
-
-# @function start_worker_if_needed
-# @intent Start the queue worker process if it's not already running
-start_worker_if_needed() {
-  # Security: Use file locking to prevent race condition
-  # Open file descriptor 200 for locking
-  exec 200>"$QUEUE_LOCK"
-
-  # Acquire exclusive lock (flock -x) with timeout
-  if ! flock -x -w 5 200; then
-    echo "Warning: Could not acquire queue lock" >&2
-    return 1
-  fi
-
-  # Check if worker is already running (within lock)
-  if [[ -f "$WORKER_PID_FILE" ]]; then
-    local pid=$(cat "$WORKER_PID_FILE")
-    if kill -0 "$pid" 2>/dev/null; then
-      # Worker is running, release lock and return
-      flock -u 200
-      exec 200>&-
-      return 0
-    fi
-  fi
-
-  # Start worker in background
-  "$SCRIPT_DIR/tts-queue-worker.sh" &
-  local worker_pid=$!
-  echo $worker_pid > "$WORKER_PID_FILE"
-
-  # Release lock
-  flock -u 200
-  exec 200>&-
-}
-
-# @function clear_queue
-# @intent Clear all pending TTS requests (emergency stop)
-clear_queue() {
-  rm -f "$QUEUE_DIR"/*.queue
-  echo "✅ Queue cleared"
-}
-
-# @function show_queue
-# @intent Display current queue status
-show_queue() {
-  local count=$(ls -1 "$QUEUE_DIR"/*.queue 2>/dev/null | wc -l)
-  echo "📊 Queue status: $count items pending"
-
-  if [[ -f "$WORKER_PID_FILE" ]]; then
-    local pid=$(cat "$WORKER_PID_FILE")
-    if kill -0 "$pid" 2>/dev/null; then
-      echo "✅ Worker process running (PID: $pid)"
-    else
-      echo "❌ Worker process not running"
-    fi
-  else
-    echo "❌ Worker process not running"
-  fi
-}
-
-# Main command dispatcher
-case "${1:-help}" in
-  add)
-    add_to_queue "${2:-}" "${3:-}" "${4:-default}"
-    ;;
-  clear)
-    clear_queue
-    ;;
-  status)
-    show_queue
-    ;;
-  *)
-    echo "Usage: tts-queue.sh {add|clear|status}"
-    echo ""
-    echo "Commands:"
-    echo "  add <text> [voice] [agent]  Add TTS request to queue with optional agent for background music"
-    echo "  clear                       Clear all pending requests"
-    echo "  status                      Show queue status"
-    exit 1
-    ;;
-esac
+#!/usr/bin/env bash
+#
+# File: .claude/hooks/tts-queue.sh
+#
+# TTS Queue Manager for Party Mode
+# Queues TTS requests and plays them sequentially in the background
+# This allows Claude to continue generating responses while audio plays in order
+
+set -euo pipefail
+
+# Security: Use secure temp directory with restrictive permissions
+# Check if XDG_RUNTIME_DIR is available (more secure than /tmp)
+if [[ -n "${XDG_RUNTIME_DIR:-}" ]] && [[ -d "$XDG_RUNTIME_DIR" ]]; then
+  QUEUE_DIR="$XDG_RUNTIME_DIR/agentvibes-tts-queue"
+else
+  # Fallback to user-specific temp directory
+  QUEUE_DIR="/tmp/agentvibes-tts-queue-$(id -u)"
+fi
+
+QUEUE_LOCK="$QUEUE_DIR/queue.lock"
+WORKER_PID_FILE="$QUEUE_DIR/worker.pid"
+
+# Get script directory
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+
+# Initialize queue directory with restrictive permissions
+if [[ ! -d "$QUEUE_DIR" ]]; then
+  mkdir -p "$QUEUE_DIR"
+  chmod 700 "$QUEUE_DIR"  # Only owner can read/write/execute
+fi
+
+# @function add_to_queue
+# @intent Add a TTS request to the queue for sequential playback
+# @param $1 dialogue text
+# @param $2 voice name (optional)
+# @param $3 agent name (optional, for background music in party mode)
+# @param $4 agent profile path (optional, PID-scoped temp JSON with reverb/personality/music overrides)
+add_to_queue() {
+  local text="$1"
+  local voice="${2:-}"
+  local agent="${3:-default}"
+  local profile_path="${4:-}"
+
+  # Create unique queue item with timestamp
+  local timestamp=$(date +%s%N)
+  local queue_file="$QUEUE_DIR/$timestamp.queue"
+
+  # Write request to queue file using direct storage
+  # Text is stored in a separate .txt file (handles newlines and special chars safely)
+  # Voice and agent are simple identifiers with no special chars
+  printf '%s' "$text" > "${queue_file%.queue}.txt"
+  cat > "$queue_file" <<EOF
+TEXT_FILE=${queue_file%.queue}.txt
+VOICE=$voice
+AGENT=$agent
+PROFILE_PATH=$profile_path
+EOF
+
+  # Start queue worker if not already running
+  start_worker_if_needed
+}
+
+# @function start_worker_if_needed
+# @intent Start the queue worker process if it's not already running
+start_worker_if_needed() {
+  # Security: Use file locking to prevent race condition
+  # Open file descriptor 200 for locking
+  exec 200>"$QUEUE_LOCK"
+
+  # Acquire exclusive lock (flock -x) with timeout
+  if ! flock -x -w 5 200; then
+    echo "Warning: Could not acquire queue lock" >&2
+    return 1
+  fi
+
+  # Check if worker is already running (within lock)
+  if [[ -f "$WORKER_PID_FILE" ]]; then
+    local pid=$(cat "$WORKER_PID_FILE")
+    if kill -0 "$pid" 2>/dev/null; then
+      # Worker is running, release lock and return
+      flock -u 200
+      exec 200>&-
+      return 0
+    fi
+  fi
+
+  # Start worker in background
+  "$SCRIPT_DIR/tts-queue-worker.sh" &
+  local worker_pid=$!
+  echo $worker_pid > "$WORKER_PID_FILE"
+
+  # Release lock
+  flock -u 200
+  exec 200>&-
+}
+
+# @function clear_queue
+# @intent Clear all pending TTS requests (emergency stop)
+clear_queue() {
+  rm -f "$QUEUE_DIR"/*.queue
+  echo "✅ Queue cleared"
+}
+
+# @function show_queue
+# @intent Display current queue status
+show_queue() {
+  local count=$(ls -1 "$QUEUE_DIR"/*.queue 2>/dev/null | wc -l)
+  echo "📊 Queue status: $count items pending"
+
+  if [[ -f "$WORKER_PID_FILE" ]]; then
+    local pid=$(cat "$WORKER_PID_FILE")
+    if kill -0 "$pid" 2>/dev/null; then
+      echo "✅ Worker process running (PID: $pid)"
+    else
+      echo "❌ Worker process not running"
+    fi
+  else
+    echo "❌ Worker process not running"
+  fi
+}
+
+# @function play_wav
+# @intent Queue a pre-generated WAV file for sequential playback
+# @param $1 path to WAV file
+play_wav() {
+  local wav_file="$1"
+  [[ -z "$wav_file" ]] && return 1
+  [[ ! -f "$wav_file" ]] && return 1
+
+  local timestamp=$(date +%s%N)
+  local queue_file="$QUEUE_DIR/$timestamp.queue"
+
+  # Write a playback-only queue item (no synthesis needed)
+  cat > "$queue_file" <<EOF
+PLAY_WAV=$wav_file
+EOF
+
+  start_worker_if_needed
+}
+
+# Main command dispatcher
+case "${1:-help}" in
+  add)
+    add_to_queue "${2:-}" "${3:-}" "${4:-default}" "${5:-}"
+    ;;
+  play)
+    play_wav "${2:-}"
+    ;;
+  clear)
+    clear_queue
+    ;;
+  status)
+    show_queue
+    ;;
+  *)
+    echo "Usage: tts-queue.sh {add|play|clear|status}"
+    echo ""
+    echo "Commands:"
+    echo "  add <text> [voice] [agent]  Add TTS request to queue"
+    echo "  play <wav_file>             Queue a pre-generated WAV for playback"
+    echo "  clear                       Clear all pending requests"
+    echo "  status                      Show queue status"
+    exit 1
+    ;;
+esac