Blaze Livekit plugin integration#1112
Conversation
🦋 Changeset detectedLatest commit: 4c70551 The changes in this PR will be included in the next version bump. This PR includes changesets to release 22 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 |
|
Have you ran the tests on those plugins? |
Yes — I re-ran the plugin tests and checks after the latest fixes. What I validated:
I also added a regression test for the stream options race-condition and fixed the STT/TTS typed-array buffer window handling from review feedback. |
…nts and update llm/tts
…t/unhandled rejection
|
@toubatbrian Validation:
Devin Review is clean now. Please take another look when you have a chance. |
f39cb7d to
14cb5a2
Compare
| /** | ||
| * Update TTS options at runtime. | ||
| */ | ||
| updateOptions(opts: Partial<Omit<TTSOptions, 'config'>>): void { |
There was a problem hiding this comment.
🟡 TTS updateOptions type accepts sampleRate but silently ignores it
The updateOptions method's type signature is Partial<Omit<TTSOptions, 'config'>>, which includes sampleRate. However, the implementation handles every other field from TTSOptions (language, speakerId, authToken, model, audioFormat, audioSpeed, audioQuality, timeout, normalizationRules, batchMinChars, batchTargetChars, batchMaxChars, batchMaxWaitMs, interSentenceSilenceMs, apiUrl) except sampleRate. A caller invoking tts.updateOptions({ sampleRate: 48000 }) would have the change silently dropped — this.#opts.sampleRate remains at the old value, and the base class TTS.sampleRate getter (backed by a private field set in the constructor at plugins/blaze/src/tts.ts:925) also remains unchanged. The type should exclude sampleRate to prevent the misleading contract.
| updateOptions(opts: Partial<Omit<TTSOptions, 'config'>>): void { | |
| updateOptions(opts: Partial<Omit<TTSOptions, 'config' | 'sampleRate'>>): void { |
Was this helpful? React with 👍 or 👎 to provide feedback.
Summary
Add
@livekit/agents-plugin-blaze, a new Blaze provider plugin foragents-jswith support for:The implementation is scoped under
plugins/blazeand follows the existing provider/plugin structure used by other integrations in this repo.What changed
plugins/blazepackage with build/package wiringNotes
Changeset
@livekit/agents-plugin-blazeTest plan
pnpm vitest run plugins/blaze/src/*.test.tspnpm --filter @livekit/agents-plugin-blaze buildpnpm --filter @livekit/agents-plugin-blaze lint