-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[VoiceLive] Add AgentConfig model and agent field to ResponseSession #43671
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…e.py Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…e.py Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds support for agent configuration in the Azure AI VoiceLive SDK. The changes introduce a new AgentConfig model and integrate it with the ResponseSession model to enable agent-based conversations.
- Introduced
AgentConfigmodel with properties for agent type, name, description, agent_id, and thread_id - Added
agentfield toResponseSessionto support agent configuration in session responses - Cleaned up documentation formatting by removing extraneous blank lines in docstrings
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| sdk/ai/azure-ai-voicelive/azure/ai/voicelive/models/_models.py | Added AgentConfig model, integrated agent field into ResponseSession, and cleaned up docstring formatting |
| sdk/ai/azure-ai-voicelive/azure/ai/voicelive/models/init.py | Exported AgentConfig in the public API |
| sdk/ai/azure-ai-voicelive/apiview-properties.json | Added cross-language mapping for AgentConfig |
| sdk/ai/azure-ai-voicelive/CHANGELOG.md | Documented the new agent configuration feature |
Comments suppressed due to low confidence (2)
sdk/ai/azure-ai-voicelive/azure/ai/voicelive/models/_models.py:1
- The test
test_response_session_with_agentdoesn't actually test the newagentfield. It should create and verify anAgentConfiginstance with theResponseSessionto properly test the new functionality. Consider adding assertions like:from azure.ai.voicelive.models import AgentConfig; agent = AgentConfig(name='test-agent', agent_id='agent-123', thread_id='thread-456'); session = ResponseSession(model='gpt-4o-realtime-preview', agent=agent, id='session-789'); assert session.agent == agent; assert session.agent.name == 'test-agent'
# pylint: disable=line-too-long,useless-suppression,too-many-lines
sdk/ai/azure-ai-voicelive/CHANGELOG.md:1
- The release date appears to be in the future. The current date in October 2025 hasn't occurred yet. Please verify the intended release date format is correct.
# Release History
…zure#43671) * [VoiceLive] Add async function-calling agent sample * add phrase list * fix typo * Update sdk/ai/azure-ai-voicelive/samples/async_function_calling_sample.py Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update sdk/ai/azure-ai-voicelive/samples/async_function_calling_sample.py Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * update * fix typo * update changelog * update * remove breaking change section * update changelog * fix change log * revert changelog I lost * update version and change log * enable type verification * update * [VoiceLive] Relase 1.0.0b4 * [VoiceLive] Add AgentConfig model and agent field to ResponseSession * add release date * update change log --------- Co-authored-by: Xiting Zhang <xitzhang@microsoft.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new API spec, a link to the pull request containing these API spec changes should be included above.
All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines