Skip to content

feat: webrtc and AI plugins #90

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 243 commits into
base: main
Choose a base branch
from
Open

feat: webrtc and AI plugins #90

wants to merge 243 commits into from

Conversation

vivekd22
Copy link
Contributor

@vivekd22 vivekd22 commented Jul 7, 2025

No description provided.

maxkahan and others added 30 commits August 20, 2025 19:25
- Fix VADPartialEvent and VADAudioEvent constructor calls with invalid parameters
- Update event handlers in tests to expect event objects instead of PcmData
- Fix numpy array truthiness issue in event logging
- Improve event filtering robustness in global event system tests
- Fix event handler signatures in VAD, TTS, and STS tests
- Resolve all internal plugin test failures (117 tests now passing)

The remaining 24 test failures are due to missing external dependencies
(deepgram and elevenlabs packages), not code issues.
- Fix event handler signatures across all plugins to expect event objects
- Fix ElevenLabs TTS async/await issue in synthesize() method
- Fix Deepgram STT test metadata expectations
- Fix FAL STT error event assertions
- Fix Cartesia TTS event handler signatures
- Install missing external dependencies
- Move event system examples to proper location

All plugins now have full test coverage and are production-ready.
feat: Add new global event system and models
Allow for optionally passing in STT clients
- Fixes NameError: name 'provider_name' is not defined in STS constructor
- Adds provider_name: Optional[str] = None parameter to __init__ method
- Updates docstring to include the new parameter
- Fixes ElevenLabs TTS test mocking issues
- Fixes Kokoro TTS client attribute syntax error
- All plugin tests now pass (169 passed, 0 failed)

This resolves the bug introduced in commit 0b64c14 where the provider_name
parameter was implemented in the constructor body but missing from the
method signature.
refactor: location and core methods
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.

5 participants