Repository navigation
feat: bridge audio inputs for text-only models - #9807
Conversation
Final stacked-head validation for 689e1e0
No code blocker remains in this stacked diff. The single external gate remains the explicitly documented live Deepgram call, which cannot be run until one usable local Deepgram connection is exposed. No VPS deployment and no merge were performed. |
|
|
|
|
1 similar comment
|
|
689e1e0 to
cdea9af
Compare
Post-merge Deepgram live gate — passed on VPS .15The required product gate was completed on the Local VPS against release
The gate initially exposed a real runtime defect: Node native Freeze evidence on VPS .15: 23/23 focused Node tests, 4/4 Audio Bridge UI Vitest tests, focused ESLint, |
* feat(modality-bridge): resolve audio input capability * feat(modality-bridge): resolve audio runtime settings * feat(modality-bridge): add audio transcription helpers * feat(modality-bridge): add Audio Bridge guardrail * feat(dashboard): make Modality Bridge audio tab functional * docs(guardrails): document Audio Bridge runtime * fix(modality-bridge): harden audio catalog and response header * chore(changelog): record audio modality bridge --------- Co-authored-by: diegosouzapw <diegosouzapw@users.noreply.github.com>
Summary
Integration state
Parent PR #9782 is merged. This PR is now based directly on
release/v3.8.50and was rebuilt as the same eight audio-only commits on release SHA4fc04a56045e0afb9b81b38ab8dc74fa9983e89d.cdea9afd4223b8fc54c12e1601b8fe239706fbf8Verification
Passing on the final retargeted head:
npm run test:vitest: 37 files, 344/344npm run typecheck:corenpm run lintnpm run check:cycles: 406 files, no cyclesnpm run i18n:sync-ui:dry -- --translate-markers: 0 additions, 0 translations11542/11542)git diff --checkBase-red discrimination
The manual
workflow_dispatchquality run exposed three inherited release reds. Each was reproduced on a disposable worktree at the pure base SHA4fc04a56045e0afb9b81b38ab8dc74fa9983e89d; none of the affected paths is in this PR:.env.examplecontainsOMNIROUTE_STRICT_SYSTEM_PROVIDERS, whiledocs/reference/ENVIRONMENT.mddoes not.open-sse/handlers/videoGeneration.ts,open-sse/mcp-server/schemas/tools.ts,open-sse/mcp-server/server.ts, andopen-sse/utils/stream.ts.TS2339errors inopen-sse/handlers/videoGeneration/falHandler.ts.check-env-doc-syncandissue-7793-env-doc-sync-reprofail on the same missing env documentation. The two additional failures instream-numeric-ids.test.tsandstream-utils.test.tswere run in isolation on both the PR head and the pure release SHA; both produced exactly 57/59 with the same replaced numeric ID and missingtotal_tokensassertion.The file-size job used its documented absolute fallback because this was a manual dispatch and therefore had no pull-request base SHA. These reds are tracked as release-base health, including #9985; they are not assigned to the Audio Bridge diff.
Security and review reconciliation
The Standards and Spec reviews were reconciled before publication. Two findings were fixed: response-header CRLF hardening for both modality bridges and transcription-model discovery through the authenticated unified catalog. No remaining diff-owned code blocker was found.
Live Deepgram gate — merge blocked
Discovery was non-destructive and did not print credentials:
DEEPGRAM_*environment variable or repository Actions secret nameThe required real external
input_audio -> text-only chatproof cannot be executed in the current environment. Per Hard Rule #18, this PR must remain open and must not be merged until that live gate is recorded.The single unblock is to add or expose one usable Deepgram connection locally. Then send a known
input_audioclip to a text-only chat target and record the HTTP status,x-omniroute-modality-bridgeheader, and a non-sensitive response excerpt. No VPS deploy was performed.Related