Skip to content

feat(voice): preserve voice session end detail#1764

Merged
JoshParkSJ merged 6 commits into
mainfrom
josh/flow-part-1
Jul 6, 2026
Merged

feat(voice): preserve voice session end detail#1764
JoshParkSJ merged 6 commits into
mainfrom
josh/flow-part-1

Conversation

@JoshParkSJ

Copy link
Copy Markdown
Contributor

Draft support for AgentInterfaces flow-part-2 voice session outcome plumbing. Not required for AgentInterfaces #1013. This preserves the voice_session_ended payload opaquely on the voice bridge so downstream runtimes can surface callEnded, endedBy, reason, and future CAS fields.

@github-actions github-actions Bot added test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-integrations labels Jun 25, 2026
@JoshParkSJ JoshParkSJ marked this pull request as ready for review July 5, 2026 19:43
Copilot AI review requested due to automatic review settings July 5, 2026 19:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 plumbing in the CLI voice bridge to retain the voice_session_ended payload (CAS “end detail”) so downstream runtimes can surface richer call outcome information (e.g., endedBy, callEnded, reason, and future fields).

Changes:

  • Store the voice_session_ended payload on VoiceToolCallSession and expose it via a new end_detail property.
  • Improve session-ended logging to include key outcome fields when present.
  • Add tests to ensure dict payloads are preserved and non-dict payloads result in an empty detail.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
packages/uipath/src/uipath/_cli/_chat/_voice_bridge.py Preserves and exposes voice_session_ended payload and logs key fields.
packages/uipath/tests/cli/chat/test_voice_bridge.py Adds coverage for end_detail preservation and non-dict handling.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/uipath/src/uipath/_cli/_chat/_voice_bridge.py Outdated
Comment thread packages/uipath/src/uipath/_cli/_chat/_voice_bridge.py
Comment thread packages/uipath/src/uipath/_cli/_chat/_voice_bridge.py Outdated
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

🚨 Heads up: uipath-integrations cross-tests are FAILING 🚨

Your changes may break one or more integrations in uipath-integrations-python:

  • uipath-openai-agents
  • uipath-google-adk
  • uipath-agent-framework
  • uipath-llamaindex
  • uipath-pydantic-ai

⚠️ These checks are NOT enforced by branch protection rules. Please review the failures before merging.

🔍 Inspect the failed run →

@sonarqubecloud

sonarqubecloud Bot commented Jul 6, 2026

Copy link
Copy Markdown

@JoshParkSJ JoshParkSJ merged commit 0c3eeb9 into main Jul 6, 2026
154 of 164 checks passed
@JoshParkSJ JoshParkSJ deleted the josh/flow-part-1 branch July 6, 2026 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:uipath-integrations test:uipath-langchain Triggers tests in the uipath-langchain-python repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants