Skip to content

Move Soniqo live streaming into owhisper client#5859

Merged
ComputelessComputer merged 1 commit into
mainfrom
refactor/soniqo-owhisper-live
Jul 1, 2026
Merged

Move Soniqo live streaming into owhisper client#5859
ComputelessComputer merged 1 commit into
mainfrom
refactor/soniqo-owhisper-live

Conversation

@ComputelessComputer

@ComputelessComputer ComputelessComputer commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Route local Soniqo Parakeet live audio through an owhisper-client local live client and reuse the shared listener stream finalization path.


Note

Medium Risk
Touches the realtime transcription pipeline (buffering, finalize, dual-mic echo gate) with a large logic move; regressions could affect live session timing or transcript metadata even though behavior is intended to match via shared process_stream.

Overview
Moves local Soniqo Parakeet live transcription out of listener-core and into owhisper-client behind the local feature, so Soniqo follows the same live-client pattern as remote STT adapters.

listener-core now enables owhisper-client with features = ["local"] and spawn_soniqo_rx_task wires mic/speaker audio through LocalSoniqoLiveClient (from_realtime_audio_single / dual) and the existing process_stream + FinalizeHandle path, replacing hundreds of lines of inline buffering, flush timers, session lifecycle, and echo gating in adapters.rs.

The new local_soniqo_live module owns session start/stop on a blocking pool, 250ms buffer flushes, i16→f32 conversion, dual-channel echo suppression, and streams Result<StreamResponse, LocalSoniqoLiveError>; echo-gate unit tests moved with it. Cargo.lock picks up the transcribe-soniqo dependency on the consumer that enables local.

Reviewed by Cursor Bugbot for commit f6de2ed. Bugbot is set up for automated code reviews on this repo. Configure here.

@netlify

netlify Bot commented Jul 1, 2026

Copy link
Copy Markdown

Deploy Preview for old-char canceled.

Name Link
🔨 Latest commit f6de2ed
🔍 Latest deploy log https://app.netlify.com/projects/old-char/deploys/6a4555dab0bf1100085ff6d5

Comment thread crates/listener-core/src/actors/listener/adapters.rs
Comment thread crates/owhisper-client/src/local_soniqo_live.rs
@ComputelessComputer ComputelessComputer force-pushed the refactor/soniqo-owhisper-live branch from 49dd718 to 480048d Compare July 1, 2026 16:40
Comment thread crates/owhisper-client/src/local_soniqo_live.rs
@ComputelessComputer ComputelessComputer force-pushed the refactor/soniqo-owhisper-live branch from 480048d to add9a0a Compare July 1, 2026 17:14

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

There are 3 total unresolved issues (including 2 from previous reviews).

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit add9a0a. Configure here.

Comment thread crates/listener-core/src/actors/listener/adapters.rs
Route local Soniqo Parakeet live audio through an owhisper-client local live client and reuse the shared listener stream finalization path.
@ComputelessComputer ComputelessComputer force-pushed the refactor/soniqo-owhisper-live branch from add9a0a to f6de2ed Compare July 1, 2026 18:00
@ComputelessComputer ComputelessComputer merged commit be6fa72 into main Jul 1, 2026
12 checks passed
@ComputelessComputer ComputelessComputer deleted the refactor/soniqo-owhisper-live branch July 1, 2026 18:28
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