Repository navigation
Add post-call delivery configuration and related models to VoiceAgent… - #49813
Howie Leung (howieleung) wants to merge 6 commits into
Conversation
|
Azure Pipelines: Successfully started running 1 pipeline(s). 9 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
There was a problem hiding this comment.
🟡 Changes recommended
The new post-call delivery feature lacks a required changelog entry covering VoiceAgentDefinition.post_call, Event Grid/routine destinations, and the preview restriction on audio delivery, leaving the capability and limitation out of release notes.
1 open issue
What changed in this PR
Adds post-call delivery configuration to Voice Agents in azure-ai-projects, allowing callers to configure Event Grid and routine destinations.
Changes:
- Adds
VoiceAgentDefinition.post_call, destination models, and related enums; documents that audio delivery is unsupported in this preview. - Exports the new types and refreshes API snapshots, cross-language mappings, and the TypeSpec revision.
- Adds Pylint suppressions to the sample CI runner.
| File | Description |
|---|---|
sdk/ai/azure-ai-projects/tsp-location.yaml |
Updates the source specification commit. |
sdk/ai/azure-ai-projects/tests/samples/ci/run_samples.py |
Suppresses line-length and unused-suppression warnings. |
sdk/ai/azure-ai-projects/azure/ai/projects/models/_models.py |
Adds post-call configuration and destination models. |
sdk/ai/azure-ai-projects/azure/ai/projects/models/_enums.py |
Adds destination and delivery-content enums. |
sdk/ai/azure-ai-projects/azure/ai/projects/models/__init__.py |
Exports the new models and enums. |
sdk/ai/azure-ai-projects/apiview-properties.json |
Updates cross-language API mappings and version. |
sdk/ai/azure-ai-projects/api.metadata.yml |
Refreshes the API hash. |
sdk/ai/azure-ai-projects/api.md |
Records the expanded public API. |
Review effort: Balanced
| post_call: Optional["_models.VoiceAgentPostCallConfig"] = rest_field( | ||
| visibility=["read", "create", "update", "delete", "query"] | ||
| ) |
This comment has been minimized.
This comment has been minimized.
| destinations: list["_models.VoiceAgentPostCallDestination"] = rest_field( | ||
| visibility=["read", "create", "update", "delete", "query"] | ||
| ) |
- Improved the clarity of SSE frame documentation in AgentsOperations by formatting the description of the `data` field. - Expanded the list of known values for the `provider` keyword in BetaVoiceAgentsTelephonyOperations to include "sip" for both bindings and calls. - Updated the commit reference in tsp-location.yaml to reflect the latest changes in the Azure REST API specs repository.
| VoiceAgentServerEventSessionHandoffAborted, | ||
| VoiceAgentServerEventSessionHandoffCompleted, | ||
| VoiceAgentServerEventSessionHandoffStarted, |
| """The number of unique reserved handoff attempts. Required.""" | ||
| available_edge_ids: list[str] = rest_field(visibility=["read", "create", "update", "delete", "query"]) | ||
| """The edge identifiers currently available from the active node. Required.""" | ||
| transfer_tool: "_models.VoiceAgentFunctionTool" = rest_field( |
- Introduced new methods for managing live conversations in the BetaVoiceAgentsLiveConversationsOperations class, including listing, retrieving, deleting, and streaming audio for live conversations. - Updated public methods documentation to reflect the addition of 9 new methods under the `beta.voice_agents.live_conversations` namespace. - Updated the TSP location YAML file to point to the latest commit in the Azure REST API specs repository.
There was a problem hiding this comment.
🟡 Changes recommended
Renamed model exports break existing imports, pagination ignores has_more=false, and the new live-conversation operations lack dedicated sync and async test coverage.
7 open issues
Preserve compatibility exports for renamed tool models · New Add sync and async tests for live conversation operations · New Stop pagination when has_more is false · New Preserve nullable transfer_tool annotations Register handoff events in both realtime event unions Test post_call destination serialization and polymorphic decoding Add changelog entry for post-call delivery feature
1 file not reviewed
eng/emitter-package-lock.json: Generated file
Review effort: Balanced
| return deserialized # type: ignore | ||
|
|
||
|
|
||
| class BetaVoiceAgentsLiveConversationsOperations: # pylint: disable=docstring-missing-param,name-too-long |
| ) | ||
| if cls: | ||
| list_of_elem = cls(list_of_elem) # type: ignore | ||
| return deserialized.get("last_id") or None, iter(list_of_elem) |
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
🟡 Changes recommended
Pagination ignores has_more and triggers extra requests, new realtime events lack typed dispatch registration, and the operation group is missing public exports and annotations.
8 open issues
Fix async pagination for terminal pages · New Export and annotate the new operations group · New Stop pagination when has_more is false Add sync and async tests for live conversation operations Preserve nullable transfer_tool annotations Register handoff events in both realtime event unions Test post_call destination serialization and polymorphic decoding Add changelog entry for post-call delivery feature
1 resolved since last review
1 file not reviewed
eng/emitter-package-lock.json: Generated file
Review effort: Balanced
| ) | ||
| if cls: | ||
| list_of_elem = cls(list_of_elem) # type: ignore | ||
| return deserialized.get("last_id") or None, AsyncList(list_of_elem) |
| return deserialized # type: ignore | ||
|
|
||
|
|
||
| class BetaVoiceAgentsLiveConversationsOperations: # pylint: disable=docstring-missing-param,name-too-long |
[Pilot] PR Pipeline Failure AnalysisWhat failedAzure Pipelines build 7039262 ( Root cause 1 — Spelling/lint validation failure (Build Analyze job): Root cause 2 — Playback/test-recording mismatch (azure-ai-projects, all 8 platforms): Unrelated, pre-existing failure (azure-core, ubuntu2404_311_ib1 and windows2022_312_ib1 only): Relevant pipeline outputRecommended next steps
Automated fix: Requested
|



… (#49805)
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