fix(remote-a2a): fall back to folder context when folder_path is missing#1749
Conversation
RemoteA2aService._retrieve_spec now resolves the folder via a _resolve_folder_key helper (mirroring McpService): when no folder_path is supplied it falls back to the FolderContext instead of raising "Cannot obtain folder_key without providing folder_path". This lets local runs (e.g. uipath debug without UIPATH_FOLDER_PATH) resolve the agent the same way MCP servers already do. Bumps uipath-platform to 0.1.75 and uipath to 2.11.12. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
There was a problem hiding this comment.
Pull request overview
This PR fixes Remote A2A agent retrieval so that when folder_path is not provided it no longer raises, and instead falls back to the folder context (mirroring the existing McpService behavior). This unblocks local A2A runs (e.g., uipath debug) where UIPATH_FOLDER_PATH is unset and no explicit resource override is provided.
Changes:
- Add
RemoteA2aService._resolve_folder_key()and use it in_retrieve_spec()to fall back toFolderContextwhenfolder_pathis missing. - Add unit tests covering both the fallback-to-context behavior and the explicit
folder_pathresolution path. - Bump package versions:
uipath-platform→0.1.75,uipath→2.11.12, and update bothuv.lockfiles accordingly.
Reviewed changes
Copilot reviewed 4 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| packages/uipath/uv.lock | Updates lockfile to reflect the new uipath / uipath-platform versions and refreshed resolution metadata. |
| packages/uipath/pyproject.toml | Bumps uipath version and raises minimum uipath-platform dependency to >=0.1.75. |
| packages/uipath-platform/uv.lock | Updates uipath-platform lockfile for the new version and refreshed resolution metadata. |
| packages/uipath-platform/tests/services/test_remote_a2a_service.py | Adds regression coverage for _retrieve_spec() folder key resolution (fallback vs explicit). |
| packages/uipath-platform/src/uipath/platform/agenthub/_remote_a2a_service.py | Implements _resolve_folder_key() and uses it to avoid raising when folder_path is missing. |
| packages/uipath-platform/pyproject.toml | Bumps uipath-platform version to 0.1.75. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
radu-mocanu
left a comment
There was a problem hiding this comment.
Golden-rule review
Reviewed the remote-a2a folder fallback change against the golden rules. The version bumps, lock files, and tests are consistent with the rules. No golden-rule violations found.
No golden-rule violations found.
RemoteA2aService._retrieve_specnow resolves the folder through a_resolve_folder_keyhelper that mirrorsMcpService: when nofolder_pathis provided it falls back to theFolderContextinstead of raisingCannot obtain folder_key without providing folder_path.UIPATH_FOLDER_PATHenv var (e.g.uipath debugwith noUIPATH_FOLDER_PATHand no resource override)uipath-platformto 0.1.75 anduipathto 2.11.12.