Skip to content

fix(a2a-tool): prefer UiPath proxy url over cached agent card url#928

Open
radugheo wants to merge 1 commit into
mainfrom
fix/remote-a2a-proxy-url
Open

fix(a2a-tool): prefer UiPath proxy url over cached agent card url#928
radugheo wants to merge 1 commit into
mainfrom
fix/remote-a2a-proxy-url

Conversation

@radugheo

@radugheo radugheo commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

Copilot AI review requested due to automatic review settings June 23, 2026 14:28
@radugheo radugheo force-pushed the fix/remote-a2a-proxy-url branch 2 times, most recently from f2f23ad to b6cb2a0 Compare June 23, 2026 14:30

Copilot AI left a comment

Copy link
Copy Markdown

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 updates the A2A tool’s URL resolution to prefer a UiPath-provided proxy URL (a2a_url) over the cached agent card URL, aligning endpoint selection with agent configuration.

Changes:

  • Update _resolve_a2a_url() to return config.a2a_url when present before falling back to cached_agent_card["url"].
  • Adjust the URL-missing error message to no longer reference cachedAgentCard specifically.

Comment on lines 158 to 163
def _resolve_a2a_url(config: AgentA2aResourceConfig) -> str:
"""Resolve the A2A endpoint URL from the cached agent card."""
"""Resolve the A2A endpoint URL, preferring the UiPath-hosted proxy URL."""
if config.a2a_url:
return config.a2a_url
if config.cached_agent_card and "url" in config.cached_agent_card:
return config.cached_agent_card["url"]
Comment thread src/uipath_langchain/agent/tools/a2a/a2a_tool.py
@radugheo radugheo force-pushed the fix/remote-a2a-proxy-url branch from b6cb2a0 to 0e6bf57 Compare June 23, 2026 15:38
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 90%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants