执行摘要

• 一句话：工具初始化集中化并移除MCP支持
• 推荐动作：建议阅读该 PR 以了解工具统一加载的设计模式，重点考察 load_all_tools 函数的接口和碰撞处理逻辑。同时应关注两个遗留问题并考虑修复。设计上值得学习的是将工具初始化提升到 worker 层，使 dataset 和 agent loop 共享同一视图。

功能与动机

PR描述指出：将所有工具初始化（native + @function_tool）从ToolAgentLoop移到AgentLoopWorker，RLHFDataset现在可以正确识别function_tool过滤超长提示。此外，MCP工具支持已弃用，决定移除。这解决了工具初始化职责分散、过滤准确性不足、以及技术债务问题。

实现拆解

1. 删除 MCP 工具模块：删除 verl/tools/mcp_base_tool.py、verl/tools/mcp_search_tool.py、verl/tools/utils/mcp_clients/ 整个目录（包括 MCPClientManager.py、utils.py、__init__.py）。移除令牌桶限速、MCP 转 OpenAI schema 等废弃代码。
2. 重构工具注册表（verl/tools/utils/tool_registry.py）：移除 ToolType.MCP 枚举及 initialize_mcp_tool、get_mcp_event_loop 等异步辅助函数。简化 initialize_tools_from_config 仅支持 NATIVE。新增 load_all_tools(tool_config_path, function_tool_path) 统一加载 native 和 function 工具，并执行跨源名称碰撞检测。
3. 迁移工具初始化到 AgentLoopWorker（verl/experimental/agent_loop/agent_loop.py）：AgentLoopWorker.__init__ 新增参数 tool_config_path 和 function_tool_path，调用 load_all_tools 初始化工具。ToolAgentLoop.__init__ 移除加载逻辑，改为接收已初始化的工具列表。更新 FunctionToolListWrap 和 ToolListWrap 以使 Hydra 实例化兼容。
4. 更新数据集过滤（verl/utils/dataset/rl_dataset.py）：RLHFDataset.__init__ 新增工具配置参数并加载工具，maybe_filter_out_long_prompts 利用加载后的 schema 正确计算 function_tool 长度。
5. 测试重写和文档更新：tests/tools/test_mixed_tools_on_cpu.py 重构为验证 load_all_tools 函数，新增碰撞检测、重复安全调用测试。tests/tools/test_function_tool_on_cpu.py 适配 Hydra 实例化。docs/sglang_multiturn/multiturn.rst 移除 MCP 工具示例代码。

关键文件：

• verl/tools/mcp_base_tool.py（模块 工具库；类别 source；类型 deletion；符号 MCPBaseTool, init, get_openai_tool_schema, create）: MCP 基础工具类，被完全删除
• verl/tools/utils/mcp_clients/McpClientManager.py（模块 工具库；类别 source；类型 deletion；符号 MCPClientManager, initialize, call_tool, fetch_tool_schemas）: MCP 客户端管理器，被完全删除
• verl/tools/utils/tool_registry.py（模块 工具库；类别 source；类型 dependency-wiring；符号 ToolType, initialize_tools_from_config, load_all_tools）: 核心工具注册表重构，新增统一加载函数
• verl/tools/utils/mcp_clients/utils.py（模块 工具库；类别 source；类型 deletion；符号 TokenBucket, init, acquire, mcp2openai）: MCP 工具辅助函数，被完全删除
• verl/tools/mcp_search_tool.py（模块 工具库；类别 source；类型 deletion；符号 MCPSearchTool, init, _parse_tool_result）: MCP 搜索工具类，被完全删除
• verl/experimental/agent_loop/agent_loop.py（模块 代理循环；类别 source；类型 core-logic；符号 AgentLoopWorker, init, FunctionToolListWrap, ToolListWrap）: AgentLoopWorker 新增工具初始化逻辑
• verl/experimental/agent_loop/tool_agent_loop.py（模块 代理循环；类别 source；类型 core-logic；符号 ToolAgentLoop, init）: ToolAgentLoop 移除工具初始化，改为接收参数
• verl/utils/dataset/rl_dataset.py（模块 数据集；类别 source；类型 dependency-wiring；符号 RLHFDataset, init, maybe_filter_out_long_prompts）: 数据集添加工具加载以支持正确过滤
• tests/tools/test_mixed_tools_on_cpu.py（模块 测试；类别 test；类型 test-coverage；符号 _merge_like_tool_agent_loop, _load_as_dict, test_no_paths_returns_empty, test_loader_merge_is_safe_to_call_per_trajectory）: 重构测试以验证 load_all_tools 函数
• tests/tools/test_function_tool_on_cpu.py（模块 测试；类别 test；类型 test-coverage；符号 test_function_tool_list_wrap_survives_hydra_instantiate, test_tool_list_wrap_survives_hydra_instantiate）: 适配新的 ListWrap 类
• verl/tools/utils/mcp_clients/__init__.py（模块 工具库；类别 source；类型 deletion）: MCP 客户端包初始化文件，被删除
• docs/sglang_multiturn/multiturn.rst（模块 文档；类别 docs；类型 documentation；符号 MCPYourTool, init, _parse_tool_result）: 移除 MCP 工具示例文档

关键符号：load_all_tools, initialize_tools_from_config, AgentLoopWorker.init, ToolAgentLoop.init, RLHFDataset.init

关键源码片段

verl/tools/utils/tool_registry.py

核心工具注册表重构，新增统一加载函数

def load_all_tools(
    tool_config_path: Optional[str],
    function_tool_path: Optional[str],
) -> list[BaseTool | FunctionTool]:
    """Load native + function tools, check for name collisions, return merged list."""
    # 从 YAML 配置文件加载 native 工具
    native_tools: list = initialize_tools_from_config(tool_config_path) if tool_config_path else []
    # 从 Python 文件加载带有 @function_tool 装饰器的函数工具
    function_tools: list[FunctionTool] = load_function_tools_from_path(function_tool_path) if function_tool_path else []
​
    # 当两种工具都存在时，检查名称是否冲突
    if function_tools and native_tools:
        existing = {t.name for t in native_tools}
        collisions = sorted(t.name for t in function_tools if t.name in existing)
        if collisions:
            raise ValueError(
                f'Function tool name(s) {collisions} collide with tools already declared in '
                f"'{tool_config_path}'. Each tool name must be unique across `tool_config_path` '
                f'and `function_tool_path`; rename one of them.'
            )
    # 注意：未检查 native_tools 或 function_tools 内部是否存在同名工具
    return native_tools + function_tools

评论区精华

审查中 gemini-code-assist[bot] 提出两个关键问题：

1. 碰撞检测范围过窄：load_all_tools 仅检查跨源冲突（native vs function），但未检查源内重复。若 YAML 或函数工具文件定义同名工具，后续字典构建将静默覆盖。建议改为对全量工具集检查唯一性。
2. RLHFDataset 异常吞噬：工具初始化异常被捕获并设 tool_schemas=None，可能导致过滤不准确，与 AgentLoopWorker 的严格失败模式不一致。建议传播异常。
   两个问题均标记为高优先级，但截至合并未收到作者修改，PR 已合并，可能作为后续改进项。

• 碰撞检测应扩展至源内重复 (design): 建议添加全工具集唯一性检查，但未见修改。
• RLHFDataset 异常吞噬 (correctness): 建议传播异常，但未见修改。

风险与影响

• 风险：
  • 兼容性风险：删除 MCP 工具模块是 breaking change，使用这些接口的代码无法继续工作。
  • 碰撞检测不完善：源内重复可能导致工具静默覆盖，影响训练逻辑。
  • 异常处理风险：RLHFDataset 的工具加载异常被吞噬，可能导致超长提示过滤失效，潜在 OOM。
  • 依赖遗留：fastmcp 等 MCP 依赖未清理，可能造成混淆。
• 影响：
  • 用户：使用 MCP 工具的用户需迁移到 native 或 function_tool，否则中断。
  • 系统：工具初始化集中到 AgentLoopWorker，代码更简洁统一。RLHFDataset 正确过滤含 function_tool 的提示，提高稳定性。
  • 团队：删除约 693 行代码，减少技术债务。但遗留两个已知问题（碰撞检测、异常处理），需后续跟进。
  • 风险标记：MCP 移除破坏兼容, 碰撞检测不完善, 异常吞噬风险

关联脉络

• PR #6189 [tool] feat: simpler function-based tool registration: 引入 @function_tool 装饰器和注册机制，本 PR 将其集成到统一加载函数中。