Synchronize real-time audio API names#692
Merged
kunal-vaishnavi merged 15 commits intomainfrom May 4, 2026
Merged
Conversation
…TranscriptionTypes to LiveAudioTypes
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
There was a problem hiding this comment.
Pull request overview
Standardizes real-time audio transcription streaming API names across the Foundry Local SDK language bindings to provide a consistent developer experience, and aligns file/module naming around “live audio session”.
Changes:
- Renames live audio stream retrieval methods to
GetStream/get_stream(and updates call sites across tests/samples/docs). - Renames/refactors live-audio client/session modules and types files for consistency (e.g.,
live_audio_session,liveAudioSession,LiveAudioTypes). - Removes the JS
IModel.createLiveTranscriptionSessionsurface from the interface.
Reviewed changes
Copilot reviewed 30 out of 34 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| sdk/rust/tests/integration/live_audio_test.rs | Updates integration test to use get_stream() |
| sdk/rust/src/openai/mod.rs | Renames module export to live_audio_session |
| sdk/rust/src/openai/live_audio_session.rs | Renames API/docs to get_stream() |
| sdk/rust/src/openai/audio_client.rs | Updates import path for live audio session |
| sdk/python/test/openai/test_live_audio_transcription.py | Updates test imports and get_stream() usage |
| sdk/python/src/openai/live_audio_types.py | Introduces consolidated live-audio types module |
| sdk/python/src/openai/live_audio_session.py | Renames generator API to get_stream() and updates docs |
| sdk/python/src/openai/audio_client.py | Updates session import and sample usage to get_stream() |
| sdk/python/src/openai/init.py | Re-exports from renamed live-audio modules |
| sdk/js/test/openai/liveAudioTranscription.test.ts | Updates imports to renamed JS live-audio modules |
| sdk/js/src/openai/liveAudioTypes.ts | Introduces consolidated live-audio types + parsing/helpers |
| sdk/js/src/openai/liveAudioSession.ts | Updates types import to renamed module |
| sdk/js/src/openai/audioClient.ts | Updates session import path |
| sdk/js/src/index.ts | Updates public exports to renamed JS live-audio modules |
| sdk/js/src/imodel.ts | Removes createLiveTranscriptionSession from IModel |
| sdk/js/src/detail/modelVariant.ts | Updates live-audio session import path |
| sdk/js/src/detail/model.ts | Updates live-audio session import path |
| sdk/cs/test/FoundryLocal.Tests/LiveAudioTranscriptionTests.cs | Renames test usage to GetStream() |
| sdk/cs/src/OpenAI/LiveAudioTypes.cs | Adds consolidated C# live-audio types implementation |
| sdk/cs/src/OpenAI/LiveAudioSession.cs | Renames session API to GetStream() and updates docs |
| sdk/cs/README.md | Updates docs to GetStream() |
| sdk/cpp/test/live_audio_test.cpp | Updates include to openai_live_audio_session.h |
| sdk/cpp/src/openai_live_audio_session.cpp | Updates include to session header |
| sdk/cpp/src/openai_audio_client.cpp | Updates include to session header |
| sdk/cpp/include/openai/openai_live_audio_session.h | Adds/renames C++ session header |
| sdk/cpp/include/foundry_local.h | Updates public include to session header |
| sdk/cpp/CMakeLists.txt | Updates build sources to openai_live_audio_session.cpp |
| samples/rust/live-audio-transcription/src/main.rs | Updates sample to get_stream() |
| samples/python/live-audio-transcription/src/app.py | Updates sample to get_stream() |
| samples/python/live-audio-transcription/README.md | Updates sample docs to get_stream() |
| samples/js/live-audio-transcription/app.js | Updates sample to getStream() |
| samples/js/live-audio-transcription/README.md | Updates sample docs to getStream() |
| samples/cs/live-audio-transcription/README.md | Updates sample docs to GetStream() |
| samples/cs/live-audio-transcription/Program.cs | Updates sample to GetStream() |
Comments suppressed due to low confidence (1)
sdk/js/src/openai/liveAudioSession.ts:3
- LiveAudioTranscriptionSession still exposes
getTranscriptionStream()(and related docs/error text), but the samples and PR goal indicate the API should be renamed togetStream(). As-is,session.getStream()calls in samples will fail at runtime/compile time. Please rename the method (and its references/messages) togetStream(), and consider keepinggetTranscriptionStream()as a backwards-compatible alias/deprecation shim if this is a public API.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: kunal-vaishnavi <115581922+kunal-vaishnavi@users.noreply.github.com>
baijumeswani
reviewed
May 4, 2026
baijumeswani
previously approved these changes
May 4, 2026
baijumeswani
approved these changes
May 4, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR standardizes the names of several real-time audio APIs across the Foundry Local SDK language bindings. It also removes an extra unused API in the JS SDK.
Motivation and Context
The API names and their usage should be similar across the language bindings. This ensures one language binding is not favored over another language binding due to API usage.
Copilot Summary
This pull request standardizes the naming of the real-time transcription stream method across all SDKs by renaming methods like
GetTranscriptionStream,getTranscriptionStream, andget_transcription_streamtoGetStream,getStream, andget_streamrespectively. It also renames related files and updates all references in documentation, tests, and code to match the new naming. Additionally, C++ and JS client files are renamed for consistency. These changes improve API clarity and make the SDKs more consistent across languages.SDK method and API renaming:
GetStream/getStream/get_streamin C#, JS, Python, and Rust SDKs, replacing previous names likeGetTranscriptionStreamandgetTranscriptionStream. All usages and documentation references are updated accordingly.File and import renaming for consistency:
openai_live_audio_client.*andliveAudioTranscriptionClient.jstoopenai_live_audio_session.*andliveAudioSession.js, updating all includes/imports throughout the codebase.Interface and API cleanup:
createLiveTranscriptionSessionmethod from the JSIModelinterface, reflecting the new naming and usage patterns.These changes ensure a consistent and predictable developer experience across all supported languages.