feat: introduce ToolsConfiguration on AgentMcpResourceConfig#1655
Open
radugheo wants to merge 1 commit into
Open
feat: introduce ToolsConfiguration on AgentMcpResourceConfig#1655radugheo wants to merge 1 commit into
radugheo wants to merge 1 commit into
Conversation
github-actions
Bot
added
test:uipath-langchain
radugheo
force-pushed
the
feat/tools-configuration
branch
from
1f7ff11 to
083b7d5
Compare
There was a problem hiding this comment.
Pull request overview
This PR updates the agent definition models to replace the legacy dynamic_tools/dynamicTools setting on MCP resources with a new ToolsConfiguration model that represents “cached” vs “dynamic” tool sourcing, while keeping the old enum as a deprecated compatibility artifact.
Changes:
- Deprecates
DynamicToolsModein favor of a newToolsConfigurationmodel. - Introduces a discriminated union (
Cached/Dynamic) underToolsConfiguration.cached_behaviour. - Replaces
AgentMcpResourceConfig.dynamic_toolswithAgentMcpResourceConfig.tools_configuration.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
2 tasks
radugheo
force-pushed
the
feat/tools-configuration
branch
from
083b7d5 to
18a5756
Compare
radugheo
force-pushed
the
feat/tools-configuration
branch
7 times, most recently
from
6d21593 to
49b9720
Compare
Adds Cached / Dynamic discriminated union under a new ToolsConfiguration model accessed via tools_configuration.cached_behaviour on the MCP resource. Cached corresponds to the old dynamic_tools=none; Dynamic corresponds to dynamic_tools=all. Schema mode is dropped from the new model for now and will be reintroduced in a later sprint. The old dynamic_tools field is removed from the model. Existing agent.json files that still serialize dynamicTools (any value) continue to parse unchanged because BaseCfg keeps extra="allow"; the legacy DynamicToolsMode enum stays exported as a deprecated alias. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
radugheo
force-pushed
the
feat/tools-configuration
branch
from
49b9720 to
5bde334
Compare
🚨 Heads up:
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Replaces the legacy
dynamic_toolsenum onAgentMcpResourceConfigwith a newtools_configuration: Optional[ToolsConfiguration]field.ToolsConfiguration.cached_behaviouris a discriminated union of:CachedToolsConfig(the oldnone) — snapshot onlySchemaToolsConfig(the oldschema) — server discovery filtered by snapshot allowlist; not currently exposed in the frontend but kept in the runtime so the option can be re-enabled later without further model changesDynamicToolsConfig(the oldall) — full server-driven discoveryThe legacy
DynamicToolsModeenum stays exported as a deprecated alias. Thedynamic_toolsfield is removed from the model, but oldagent.jsonfiles that still serializedynamicToolscontinue to validate becauseBaseCfgkeepsextra="allow".Patch version bumped:
2.10.67→2.10.68.Related PRs (must land together)
Test plan
packages/uipath/tests/agent/models/test_agent.py— 65 tests passingDevelopment Packages
uipath