[bug] SkillToolSet.get_tools() 在 is_include_all_tools=False 时仍忽略 tool_filter #352

问题描述

SkillToolSet 接收 tool_filter 和 is_include_all_tools 参数，但在显式设置 is_include_all_tools=False 后，get_tools() 仍返回全部工具，没有应用过滤器。

该问题同时存在于首次构建工具列表和后续缓存返回路径。工具名称列表过滤和谓词过滤均不生效。

复现环境

• Python：3.12.14
• SDK：1.2.0，源码安装
• Commit：02509b20ef83f287c65d0a6e5aee803477e9d9a4

最小复现

无需模型 API Key，也无需实际执行 Skill：

import asyncio
import tempfile

from trpc_agent_sdk.skills import SkillToolSet


async def main():
    with tempfile.TemporaryDirectory() as path:
        toolset = SkillToolSet(
            paths=[path],
            tool_filter=["skill_load"],
            is_include_all_tools=False,
        )

        # 分别验证首次构建和缓存返回路径
        for label in ("first", "cached"):
            names = [tool.name for tool in await toolset.get_tools()]
            print(label, names)


asyncio.run(main())

预期行为

两次调用均只返回：

['skill_load']

实际行为

两次调用均返回全部 10 个工具：

['skill_load', 'skill_run', 'skill_exec',
 'workspace_save_artifact', 'workspace_exec',
 'workspace_write_stdin', 'workspace_kill_session',
 'skill_list', 'skill_list_docs', 'skill_select_docs']

另外，传入始终返回 False 的谓词，并设置 is_include_all_tools=False 时，仍返回全部工具，谓词调用次数为 0。

原因分析

构造函数已将过滤参数传入 ToolSetABC，但 SkillToolSet.get_tools() 没有调用基类的 _is_tool_selected()：

• 缓存路径直接返回 self._default_tools.copy()。
• 首次构建路径直接返回 tools。

相关代码：

影响

调用方无法通过该配置限制暴露给 Agent 的工具集合。本地进一步通过 ToolsProcessor.process_llm_request() 验证，全部 10 个工具也会进入模型请求的工具声明，而不是在后续流程中被过滤。

建议修复

保留完整的工具缓存，并在每次返回前调用 _is_tool_selected(tool, invocation_context)，同时覆盖首次构建和缓存路径，以支持基于当前上下文的谓词过滤。

补充说明：默认 is_include_all_tools=True 时忽略过滤器符合基类现有逻辑，本 issue 针对的是显式设置为 False 后仍不生效的情况。