Conversation
b66da4a to
2995e3f
Compare
2995e3f to
46cd517
Compare
andrii-novikov
left a comment
andrii-novikov
left a comment
There was a problem hiding this comment.
Dependency resolution I think we could do as separate PR if it is not fast fix
There was a problem hiding this comment.
check for intended changes
There was a problem hiding this comment.
Confirmed and reverted — the whole file had a bunch of unrelated line-rewrapping mixed into the diff. Restored the original formatting everywhere except the genuinely new content.
| from quickapp.mcp_tooling._mcp_eager_resource import MCPEagerTextResource | ||
| from quickapp.mcp_tooling._mcp_resource_meta import MCPResourceMeta | ||
| from quickapp.mcp_tooling._mcp_server_capabilities import MCPServerCapabilities | ||
| from quickapp.tool_discovery._deferred_tools_context import ( |
There was a problem hiding this comment.
Cross module dependency
There was a problem hiding this comment.
Fixed — DeferredToolsContext/is_toolset_deferred moved to shared/deferred_tools/, imported from there now instead of the tool_discovery module.
| from quickapp.config.tools.rest_api import RestApiTool | ||
| from quickapp.config.toolsets.rest_api import RestApiToolSet | ||
| from quickapp.tool_discovery._deferred_tools_context import ( | ||
| DeferredToolsContext, |
There was a problem hiding this comment.
The same cross module dependency
There was a problem hiding this comment.
Same fix applied here — now imports DeferredToolsContext/is_toolset_deferred from quickapp.shared.deferred_tools.
andrii-novikov
changed the title
VinShurik
changed the title
VinShurik
changed the title
# Conflicts: # README.md
4 participants
Applicable issues
Description of changes
Introduce new mechanism under PREVIEW flag.
Now REST and MCP toolsets would not pass whole set of their tools to the orchestrator (unless it's small number <MIN_TOOLS_FOR_DEFERRAL - default 5)
There is a new internal tool with name "tool_search" and proper description to select only tools, that are required to fullfill the user's request.
Checklist
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.