Skip to content

adding loading-indicator audio support#2

Merged
tigranbs merged 2 commits into
masterfrom
loading-indicator
May 25, 2026
Merged

adding loading-indicator audio support#2
tigranbs merged 2 commits into
masterfrom
loading-indicator

Conversation

@tigranbs

Copy link
Copy Markdown
Contributor

Adds loading_audio config plus loading_start / loading_stop fire-and-forget WebSocket commands to node-sdk and python-sdk, mirroring the server addition in SaynaAI/sayna#18. Failures continue to surface through the existing registerOnError / register_on_error callbacks; no new error types are added.

See ../sayna/docs/websocket.md#loading-indicator for the protocol contract.

tigranbs added 2 commits May 25, 2026 02:07
Adds loading_audio config plus loading_start / loading_stop fire-and-forget
WebSocket commands to node-sdk and python-sdk, mirroring the server addition
in SaynaAI/sayna#18. Failures continue to surface through the existing
registerOnError / register_on_error callbacks; no new error types are added.

See ../sayna/docs/websocket.md#loading-indicator for the protocol contract.
The wrapped-exception branches are exercised by
test_loading_start_wraps_send_failure and test_loading_stop_wraps_send_failure
in python-sdk/tests/test_client.py. The pragma was copy-pasted from
sip_transfer (which has no equivalent test) and incorrectly masked covered
code from coverage tracking.
@tigranbs tigranbs merged commit 3b6b2db into master May 25, 2026
@tigranbs tigranbs deleted the loading-indicator branch May 25, 2026 23:15
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