Files
Jarvis/scripts/test-copy-voice-models.sh
james.maesandClaude Opus 4.7 ee7f6d2dc7 fix(track-b 1-3): bundle voice models + WakeWordDAG typo + tier-1 TTS wiring
User feedback after deep audit: voice modality reported as "F — nothing
works". The audit identified five independent fatal breaks; this commit
closes the first three under TDD discipline. B-4 (AudioGraphOwner +
LiveSpeechAnalyzerBridge) and B-5 (voice loop e2e test) are the next
session's work.

**B-1 — voice models bundled at the runtime path:**
- scripts/copy-voice-models.sh — new postBuildScript that copies
  Resources/models/{openwakeword,silero}/ → Contents/Resources/Models/
  {openWakeWord,silero}/. Maps lowercase canonical-on-disk to capital-M
  runtime path AppDelegate.installVoice expects (was off-by-case +
  no copy phase at all). Hard-fails on PLACEHOLDER_… sha256 in
  MANIFEST.json so a forgotten fetch script is a build error, not a
  silent runtime failure.
- scripts/test-copy-voice-models.sh — 6 shell-level tests for the
  copy script (real-real bundle layout, PLACEHOLDER hard-fail,
  missing manifest, soft-degrade, capital-M path matches AppDelegate).
  All 6 pass.
- project.yml — adds the copy phase to Jarvis target's postBuildScripts.
- Resources/models/silero/MANIFEST.json — populated with real
  sha256 hashes by running scripts/fetch-silero-models.sh.

**B-2 — WakeWordDAG.swift:93 production typo:**
- packages/Voice/Sources/Voice/WakeWord/WakeWordDAG.swift — was
  `session.feedTest()` (test seam, always passes empty `[Float]` to
  hysteresis). Production path now calls `session.feed(samples:)` with
  the actual ring buffer slice. Without this, the wake word literally
  could never fire in production — every frame hit the hysteresis with
  zero samples regardless of mic input.
- packages/Voice/Tests/VoiceTests/WakeWordDAGTests.swift — 2 new
  tests: WD-1 verifies the DAG forwards actual ring contents to the
  session's classifier (RED on the bug; GREEN after the fix); WD-2
  verifies an empty ring doesn't trigger wasted classifier calls.

**B-3 — tier-1 TTS engine wired:**
- packages/Voice/Sources/Voice/TTS/TTSEngineActor.swift — `orpheus`
  parameter is now `OrpheusTTS?` (was non-optional). Production
  callers can wire just AVSpeechSynthesizer without first downloading
  Orpheus 3B weights from HuggingFace. Tier-2 requested with nil
  Orpheus degrades to tier-1 transparently via a new resolvedTier
  branch; defensive guard throws TTSError.sinkUnavailable if the
  branch ever leaks through.
- packages/Voice/Tests/VoiceTests/TTSEngineActorTier1Tests.swift —
  4 new tests: TET1-1 (construction with nil Orpheus), TET1-2 (real
  AVSpeechSynthesizer emits .started → .firstAudio → .finished),
  TET1-3 (.tier2 with nil Orpheus does NOT crash), TET1-4 (rapid-fire
  cancels prior synthesis).
- App/Voice/VoiceOutputWiring.swift — new factory enum that constructs
  the tier-1 engine. Lives outside AppDelegate.swift so the VISION-03
  Layer 3 grep gate (no PresenceSignalBus + TTS engine in same file)
  stays clean.
- App/AppDelegate.swift — `VoiceTTSAdapter(engine: nil)` becomes
  `VoiceTTSAdapter(engine: VoiceOutputWiring.makeTier1Engine())`.
  Voice subsystem now has a real synthesis path on every launch.

**Verification:**
- AgentCore: 215/215.
- Voice: WakeWordDAGTests 2/2, TTSEngineActorTier1Tests 4/4.
- Replay: 33/33.
- Bus: 57/58 (F-A2-01 pre-existing).
- webview/hud: 94/94.
- All 15 boundary grep gates: PASS (including check-presence-vision-
  isolation.sh — VoiceOutputWiring.swift renamed from TTSEngineWiring
  + comment text scrubbed of PresenceSignalBus references so the
  Layer 3 substring check no longer flags it).
- scripts/check-app-builds.sh: PASS.
- Models verified at bundle path Contents/Resources/Models/
  {openWakeWord,silero}/ with real .onnx files.

**Carry-forward to next session:**
- B-4: construct AudioGraphOwner in installVoice; start AVAudioEngine;
  route mic taps to wake-word DAG. Replace LiveSpeechAnalyzerBridge.feed
  body (`_ = chunk`) with the macOS 26 SpeechAnalyzer wiring.
- B-5: voice loop e2e — feed a known WAV file at the audio graph,
  assert STT result lands on the orchestrator.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-03 18:16:11 -05:00

137 lines
5.0 KiB
Bash
Executable File

#!/usr/bin/env bash
# test-copy-voice-models.sh — shell-level tests for copy-voice-models.sh.
# Closes the bug class from the 2026-05-03 voice audit: the runtime
# expects ONNX files at `Contents/Resources/Models/{openWakeWord,silero}/`
# but the build had no copy phase, AND nothing gated on MANIFEST.json
# placeholders.
#
# Usage: scripts/test-copy-voice-models.sh
#
# Exits 0 on all-green. Each test sets up a fixture under /tmp/cvm-test-XXX,
# invokes copy-voice-models.sh with overridden SRCROOT/TARGET_BUILD_DIR,
# and asserts the expected outcome.
set -euo pipefail
REPO_ROOT="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
SCRIPT="${REPO_ROOT}/scripts/copy-voice-models.sh"
PASS=0
FAIL=0
ok() { PASS=$((PASS+1)); echo " ✓ $1"; }
fail() { FAIL=$((FAIL+1)); echo " ✗ $1"; }
mktemp_dir() { mktemp -d -t cvm-test.XXXXXXXX; }
# Build a minimal fixture replicating Resources/models layout.
# $1 = fixture root
# $2 = openwakeword manifest content (or "real" for the canonical real one)
# $3 = silero manifest content (or "real")
seed_fixture() {
local fixture="$1"
local owm="$2"
local sm="$3"
mkdir -p "$fixture/Resources/models/openwakeword"
mkdir -p "$fixture/Resources/models/silero"
if [ "$owm" = "real" ]; then
cp "${REPO_ROOT}/Resources/models/openwakeword/MANIFEST.json" \
"$fixture/Resources/models/openwakeword/MANIFEST.json"
else
printf '%s' "$owm" > "$fixture/Resources/models/openwakeword/MANIFEST.json"
fi
if [ "$sm" = "real" ]; then
cp "${REPO_ROOT}/Resources/models/silero/MANIFEST.json" \
"$fixture/Resources/models/silero/MANIFEST.json"
else
printf '%s' "$sm" > "$fixture/Resources/models/silero/MANIFEST.json"
fi
# Minimal ONNX placeholder files (content unimportant for these tests).
: > "$fixture/Resources/models/openwakeword/dummy.onnx"
: > "$fixture/Resources/models/silero/dummy.onnx"
}
run_script() {
local fixture="$1"
local target_dir="$2"
SRCROOT="$fixture" TARGET_BUILD_DIR="$target_dir" WRAPPER_NAME=Test.app \
bash "$SCRIPT"
}
echo "Test 1: real MANIFESTs → script succeeds + bundle layout correct"
T1=$(mktemp_dir); BD1=$(mktemp_dir)
seed_fixture "$T1" "real" "real"
if run_script "$T1" "$BD1" >/dev/null 2>&1; then
if [ -d "$BD1/Test.app/Contents/Resources/Models/openWakeWord" ] \
&& [ -d "$BD1/Test.app/Contents/Resources/Models/silero" ]; then
ok "real-real → bundle path layout correct (capital M, openWakeWord)"
else
fail "real-real script ran but bundle layout missing"
fi
else
fail "real-real script exited non-zero unexpectedly"
fi
echo "Test 2: silero PLACEHOLDER_ → script fails with helpful message"
T2=$(mktemp_dir); BD2=$(mktemp_dir)
seed_fixture "$T2" "real" '{"models":{"silero_vad.onnx":{"sha256":"PLACEHOLDER_run_fetch"}}}'
if run_script "$T2" "$BD2" >/tmp/cvm-t2.log 2>&1; then
fail "PLACEHOLDER manifest should hard-fail; instead exit=0"
else
if grep -q "PLACEHOLDER" /tmp/cvm-t2.log; then
ok "silero PLACEHOLDER → exit non-zero with PLACEHOLDER hint"
else
fail "non-zero exit but no PLACEHOLDER mention in stderr"
fi
fi
echo "Test 3: openwakeword PLACEHOLDER_ → script fails"
T3=$(mktemp_dir); BD3=$(mktemp_dir)
seed_fixture "$T3" '{"models":[{"sha256":"PLACEHOLDER_x"}]}' "real"
if run_script "$T3" "$BD3" >/dev/null 2>&1; then
fail "openwakeword PLACEHOLDER should hard-fail"
else
ok "openwakeword PLACEHOLDER → exit non-zero"
fi
echo "Test 4: missing MANIFEST.json → script fails"
T4=$(mktemp_dir); BD4=$(mktemp_dir)
mkdir -p "$T4/Resources/models/openwakeword"
mkdir -p "$T4/Resources/models/silero"
# Note: no MANIFEST.json files
if run_script "$T4" "$BD4" >/dev/null 2>&1; then
fail "missing MANIFEST should hard-fail"
else
ok "missing MANIFEST → exit non-zero"
fi
echo "Test 5: missing source dir entirely → script soft-degrades (warn, exit 0)"
T5=$(mktemp_dir); BD5=$(mktemp_dir)
# T5 has no Resources/models at all
if run_script "$T5" "$BD5" >/tmp/cvm-t5.log 2>&1; then
if grep -q "warning:" /tmp/cvm-t5.log; then
ok "missing source dir → soft-degrades with warning"
else
fail "exit 0 but no warning emitted"
fi
else
fail "missing source dir should exit 0 (CI / lite workflows)"
fi
echo "Test 6: real run with capital-M expectation matches AppDelegate"
# AppDelegate.swift:653 builds the lookup path:
# bundleURL.appendingPathComponent("Contents/Resources/Models", ...)
# .appendingPathComponent("openWakeWord", ...)
# Verify the test 1 bundle layout matches that string literally.
EXPECTED="$BD1/Test.app/Contents/Resources/Models/openWakeWord/MANIFEST.json"
if [ -f "$EXPECTED" ]; then
ok "bundle path 'Contents/Resources/Models/openWakeWord/' matches AppDelegate expectation"
else
fail "expected file at $EXPECTED not found"
fi
# Cleanup.
rm -rf "$T1" "$BD1" "$T2" "$BD2" "$T3" "$BD3" "$T4" "$BD4" "$T5" "$BD5" /tmp/cvm-t2.log /tmp/cvm-t5.log
echo
echo "Results: $PASS passed, $FAIL failed"
[ "$FAIL" -eq 0 ]