Skip to content

Voice: Jabra fallback when the car is away + owner's trailing @eclass - #71

Merged
ThinkOffApp merged 2 commits into
mainfrom
fix/speak-fallback-and-owner-mention
Sep 26, 2026
Merged

ThinkOffApp merged 2 commits into
mainfrom
fix/speak-fallback-and-owner-mention

Conversation

@ThinkOffApp

Copy link
Copy Markdown
Owner

Measured on vadelma 26 Sep 2026 (main d5736cd, Jabra Speak2 40 = ALSA card 0, car away):

  1. Voice answers were silent. ~/.carwatch/car-bt-mac holds the car's MAC, so listen._speak aimed every answer at the car's bluealsa A2DP sink even with the car not connected, and never reached the USB fallback. The 14:46 "Hello, do you hear me?" was HEARD and answered in the room, but nothing played. Now the car is used only when its A2DP link is actually up after the connect attempt; otherwise USB, then headset, as before.
  2. Same bug in scripts/car-speak.sh (dash speak button, brief): falls back to the first USB Audio card when the car is not connected. Known limit: while the voice listener holds the Jabra (mic keepalive from Voice: keep a silent output open on USB speakerphones so the mic streams #70), this path can get "device busy"; it then reports the failure to the dash instead of claiming success.
  3. "can you speak using the jabra @eclass" was ignored: the Car persona promises actions it cannot perform ("I'll set it to Europe/Berlin now"); answers third-person mentions #32 gate only counts a leading handle. The owner typing the literal @handle anywhere now addresses the car. Agents' third-person mentions and the owner's spoken names mid-sentence stay quiet (tested).

Tests: new tests/test_speak_route.py (5), which fail 2/5 on main and pass with the fix; full suite 165/165 OK.

Not deployed. Deploy on petrus's word (update.sh / carwatch-update.timer).

🤖 Generated with Claude Code

…ailing @handle addresses the car

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Sep 26, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
carwatch-dev Ready Ready Preview Sep 26, 2026 3:02pm UTC

Request Review

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

…y -l + car BT state)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ThinkOffApp
ThinkOffApp merged commit 301ba44 into main Sep 26, 2026
3 checks passed

This branch was successfully deployed

1 active deployment
Preview — 0557c65e Deployed Sep 26, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant