Repository navigation
Conversation
…s switched off July 9)
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (6)
Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review. WalkthroughThe ElevenLabs STT model reference changes from ChangesElevenLabs STT model update
Suggested reviewers: Priority: ⬆️ High Change: Bug fix Merge Risk: ⚪ Minimal · up to Both language implementations now use the documented 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. A rabbit checks the model name, Comment |
langwatch-agent
left a comment
There was a problem hiding this comment.
No actionable findings established. Both runtimes now use scribe_v2. The provider changelog confirms the replacement for retired scribe_v1. The test still asserts the exact model sent through the adapter; auth, endpoint, WAV conversion and response-text extraction are unchanged. No live transcription or accuracy comparison was performed, and metadata checks are not runtime proof.
Static review using the existing worktree and upstream references. No contributor code, installs, package scripts or secret-bearing calls executed. Not an approval.
LangWatch-Review: verdict=clean sha=cf5ebcd5faa9912259ca979e02bea80fd756e27c p0=0 p1=0 p2=0 p3=0
Human Review BriefCaution ElevenLabs speech-to-text in scenario has been failing since July, if the PR is right, and nobody has run this fix. The provider retired Same contributor and tool as scenario#1009 (Cartesia TTS). Both are plausible and both are unverified against the live API.
🗺️ Never opened this module? Start here · ~2 minThe scenario SDK transcribes voice turns through pluggable STT providers; each has a default model id. ❓ Worth asking the author · ~1 min
✅ Why the evidence is good, and what each file is · ~1 minUpdated string assertions only.
Note Two mentions of the old id remain in |
|
On the question above: we don't have an ElevenLabs key, so this hasn't been run against the live API. What it rests on is ElevenLabs' own changelog for June 8, 2026 (https://elevenlabs.io/docs/changelog/2026/6/8): "The If someone with a key can run one transcription, that settles it, and I'll adjust anything it turns up. |
|
Following up on the question above: I've now run it with a real ElevenLabs key, and scribe_v2 transcribes a test clip correctly. A correction too: scribe_v1 also still works today, with the same output, so the description was wrong to say this code has been failing. It's deprecated (ElevenLabs asked integrations to move by July 9), and this moves it to the supported id before ElevenLabs stops accepting it. I've corrected the title and description; sorry for the wrong claim. |
Why
ElevenLabs deprecated
scribe_v1and asked integrations to move toscribe_v2by July 9, 2026 (https://elevenlabs.io/docs/changelog/2026/6/8). A live check on October 9 shows requests namingscribe_v1are still accepted (they appear to be served byscribe_v2), so nothing is failing yet, but the deprecated id can stop being accepted without notice.What changed
scribe_v1toscribe_v2injavascript/src/voice/voice-models.tsandpython/scenario/voice/stt.py, including their doc comments.javascript/src/voice/__tests__/stt.test.tsandjavascript/src/voice/adapters/__tests__/elevenlabs.test.ts.javascript/src/voice/stt/stt-provider.tsand the doc comment injavascript/src/voice/stt/elevenlabs-stt.tsso they referencescribe_v2instead of the deprecated model.Test plan
javascript/src/voice/__tests__/stt.test.ts,javascript/src/voice/adapters/__tests__/elevenlabs.test.ts); confirm the tests still exercise the right behavior.How I can prove I was successful
No playable artifact: this change only swaps a model id string across the JS and Python ElevenLabs STT code and their tests/docs.
Anything surprising?
The following files also mention
scribe_v1but were not changed in this PR:docs/adr/003-voice-internal-design.md(lines 242, 373).Prepared with YakBarber, a tool I'm building that updates code when an API changes, and reviewed by me before sending. It was sent once, to this repository only; if it isn't useful, feel free to close it.
Check the whole repository yourself:
npx yakbarber checklists every AI model and API it calls that is being retired, with the date and the company's own words. Free, read-only, runs locally (source).