fix(elevenlabs): include stream request flags#1770
fix(elevenlabs): include stream request flags#1770rosetta-livekit-bot[bot] wants to merge 1 commit into
Conversation
🦋 Changeset detectedLatest commit: 50a4d43 The changes in this PR will be included in the next version bump. This PR includes changesets to release 34 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
| text: this.inputText, | ||
| model_id: this.#opts.model, | ||
| voice_settings: voiceSettings, | ||
| apply_text_normalization: this.#opts.applyTextNormalization, | ||
| ...extraParams, |
There was a problem hiding this comment.
🚩 ChunkedStream body params are a subset of what multiStreamUrl sends
The multiStreamUrl function (line 147-168) sends several parameters that are not included in the ChunkedStream body: enable_ssml_parsing, inactivity_timeout, sync_alignment, auto_mode, and pronunciation_dictionary_locators. Some of these are WebSocket-specific (e.g. inactivity_timeout), but others like enable_ssml_parsing and pronunciation_dictionary_locators could potentially apply to the REST endpoint too. This is pre-existing behavior not introduced by this PR, but worth noting since the PR's intent is to add "missing" flags — there may still be additional ones missing for the chunked stream path.
(Refers to lines 858-863)
Was this helpful? React with 👍 or 👎 to provide feedback.
Summary
Testing
Ported from livekit/agents#6031
Original PR description
These flags is being used in livekit multi-turn-input but was not passed on to stream api. Fixes livekit/agents#6027