Skip to content

feat(asr): add Deepgram streaming transport - #46

Merged
quanru merged 2 commits into
mainfrom
feat/deepgram-transport
Sep 25, 2026
Merged

quanru merged 2 commits into
mainfrom
feat/deepgram-transport

Conversation

@quanru

@quanru quanru commented Sep 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • add owner-only Deepgram API-key storage with validation and safe-file checks
  • add a session-owned Nova-3 WebSocket transport for 16 kHz mono PCM
  • stream interim snapshots, accumulate finalized segments, and close with Deepgram Finalize/CloseStream messages
  • classify handshake and protocol authentication failures separately from network failures

Dependency

Depends on #45. This PR deliberately does not register Deepgram in the product UI.

Validation

  • PYTHONPATH=src /Users/bytedance/personal/doubao-say/.venv/bin/python -m unittest tests.unit.test_deepgram_credentials tests.unit.test_deepgram_asr_client
  • /Users/bytedance/personal/doubao-say/.venv/bin/python -m ruff check src tests packaging
  • PYTHONPATH=src /Users/bytedance/personal/doubao-say/.venv/bin/python -m compileall -q src/doubao_input/deepgram tests/unit/test_deepgram_asr_client.py tests/unit/test_deepgram_credentials.py
  • git diff --check

Validation gap

No live Deepgram credential was used. Protocol behavior is covered with a fake bidirectional WebSocket; Linux CI provides the full repository test matrix.

Review order

Review after #45 and before the Deepgram product-integration PR.

@quanru
quanru changed the base branch from feat/recognition-provider-foundation to main September 25, 2026 11:49
@quanru
quanru merged commit 9165786 into main Sep 25, 2026
9 of 19 checks passed
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