Phidata项目中MCPTool工具包含问题的分析与修复

问题背景

在Phidata项目的1.4.2版本中，开发人员发现当使用MCPTool工具集时，如果尝试通过include_tools参数指定要包含的工具，会导致系统抛出异常。这个问题主要出现在使用MCPStdioParams配置工具集时，特别是当用户希望从可用工具列表中筛选特定工具时。

问题现象

当开发人员按照以下方式配置MCPTool时：

server_params = MCPStdioParams(
    command="docker",
    args=[
        "run",
        "-i",
        "--rm",
        "--mount",
        f"type=bind,src={tmp_path},dst=/projects",
        "mcp/filesystem",
        "/projects",
    ],
    tools=[
        "write_file",
        "read_file"
    ],
)
MCPTools(
    command=server_params,
    include_tools=["write_file"],
    env={**os.environ},
)

系统会抛出ValueError异常，提示"write_file"工具不存在于工具包中，而实际上这个工具确实已经在参数中明确定义了。

技术分析

根本原因

经过深入分析，发现问题出在MCPTools类的初始化过程中。当MCPTools调用父类的__init__方法时，没有正确传递tools参数。具体表现为：

1. MCPTools类在初始化时调用了父类的构造函数
2. 父类构造函数接收include_tools参数，用于检查指定的工具是否存在于可用工具列表中
3. 但由于tools参数没有被传递，父类构造函数中的可用工具列表为空
4. 因此，任何在include_tools中指定的工具都会被判定为不存在

影响范围

这个问题影响了所有需要使用MCPTool工具集并希望筛选特定工具的场景。特别是在以下情况下尤为明显：

• 当工具集中包含多个工具但只需要使用其中一部分时
• 当开发人员希望动态控制可用工具列表时
• 在自动化测试环境中需要精确控制工具可用性时

解决方案

修复方法

正确的实现应该是在MCPTools类的初始化过程中，将tools参数从server_params中提取出来并传递给父类构造函数。具体修改应包括：

1. 从server_params中获取已定义的工具列表
2. 在调用父类构造函数时，同时传递tools参数和include_tools参数
3. 确保工具可用性检查基于实际定义的工具列表

验证方法

为了验证修复效果，可以编写单元测试用例，模拟以下场景：

1. 定义包含多个工具的工具集
2. 尝试通过include_tools筛选部分工具
3. 验证工具集是否只包含指定的工具
4. 验证当指定不存在的工具时是否抛出正确异常

最佳实践建议

在使用Phidata的MCPTool工具集时，建议开发人员注意以下几点：

1. 始终明确指定工具列表，即使使用include_tools参数
2. 在复杂配置场景下，先验证工具集的可用性
3. 考虑编写单元测试来验证工具筛选逻辑
4. 当升级Phidata版本时，特别注意工具集相关功能的变更

总结

这个问题的修复不仅解决了工具筛选功能的基本可用性问题，也为开发人员提供了更可靠的工具管理能力。通过正确处理工具参数的传递，确保了MCPTool工具集在各种使用场景下的稳定性和一致性。对于依赖Phidata进行开发的项目来说，理解并正确使用工具筛选功能将大大提高开发效率和系统可靠性。