From 46273a55a860995a6e326c905f7f06dfa980b642 Mon Sep 17 00:00:00 2001 From: kshitijk4poor <82637225+kshitijk4poor@users.noreply.github.com> Date: Thu, 2 Jul 2026 13:04:23 +0530 Subject: [PATCH] docs(toolsets): clarify get_toolset static-view returns None for registry-derived aliases too Follow-up on the #56480 salvage: the include_registry=False docstring said None is returned only for registry/MCP-only toolsets; it also applies to registry-derived aliases, which have no static TOOLSETS counterpart. --- toolsets.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/toolsets.py b/toolsets.py index af1ee4a92a2..03e64fdba4c 100644 --- a/toolsets.py +++ b/toolsets.py @@ -599,8 +599,10 @@ def get_toolset(name: str, *, include_registry: bool = True) -> Optional[Dict[st Returns: Dict: Toolset definition with description, tools, and includes - None: If toolset not found (registry-only toolsets have no static view, - so they return None when include_registry=False) + None: If toolset not found. With include_registry=False the static + view only recognizes names literally present in ``TOOLSETS``, so + registry/MCP-only toolsets AND registry-derived aliases return None + (they have no static counterpart). """ toolset = TOOLSETS.get(name)