PraisonAI项目中MCP工具签名参数排序问题的分析与解决

在PraisonAI项目的开发过程中，我们遇到了一个关于Model Context Protocol(MCP)工具函数签名创建的典型问题。本文将深入分析该问题的技术背景、产生原因以及解决方案。

问题现象

开发人员在尝试使用GitHub MCP服务器时，遇到了Python解释器抛出的ValueError: non-default argument follows default argument异常。这一错误导致MCP工具无法正常初始化，影响了整个功能的运行。

技术背景

Python函数签名有一个基本规则：所有非默认参数（即必选参数）必须出现在默认参数（即可选参数）之前。这一规则源于Python函数调用时的参数绑定机制，它确保了参数解析的一致性和可预测性。

在PraisonAI的MCP实现中，系统需要动态地为各种工具创建函数签名。这些工具可能来自不同的MCP服务器（如GitHub、MongoDB等），每个工具都有自己独特的参数集。

问题根源分析

通过代码审查，我们发现问题的核心在于_create_tool_wrapper方法中参数列表的生成方式。原始实现简单地按照工具定义中参数的原始顺序创建参数列表，而没有考虑Python对参数顺序的约束。

具体来说，当工具定义中存在以下情况时就会触发错误：

1. 一个可选参数（带有默认值）出现在参数列表的前部
2. 而一个必选参数（无默认值）出现在其后

这种参数顺序虽然在某些接口定义中很常见，但直接转换为Python函数签名时就会违反语言规范。

解决方案设计

我们采用了以下策略来解决这个问题：

1. 参数分类处理：首先将参数分为必选参数和可选参数两类
2. 顺序重组：确保所有必选参数排在可选参数之前
3. 签名创建：使用重组后的参数列表创建函数签名

这种方法具有以下优点：

• 完全符合Python语言规范
• 保持与原始工具定义的功能一致性
• 对上层调用透明，无需修改现有代码
• 适用于所有类型的MCP工具

实现细节

在具体实现中，我们改进了参数收集逻辑：

# 分离必选和可选参数
required_params = []
optional_params = []

for param_name in param_names:
    param_info = parameters[param_name]
    if "default" not in param_info or param_info["default"] is None:
        required_params.append((param_name, param_info))
    else:
        optional_params.append((param_name, param_info))

# 合并参数，必选参数在前
ordered_params = required_params + optional_params

这种处理方式确保了最终生成的函数签名总是合法的，同时保留了原始参数的所有元信息。

影响范围

该修复不仅解决了GitHub MCP服务器的问题，同时也修复了其他MCP服务器的类似问题，如MongoDB MCP等。这体现了该解决方案的通用性和鲁棒性。

最佳实践建议

基于这一问题的解决经验，我们建议开发者在处理动态函数签名时：

1. 始终验证参数顺序是否符合语言规范
2. 考虑使用参数分类和重排序技术
3. 编写针对参数顺序的单元测试
4. 在文档中明确参数顺序要求

总结

PraisonAI项目中MCP工具签名问题的解决展示了如何处理动态代码生成中的语言规范约束。通过参数分类和顺序重组的技术，我们既遵守了Python语言规则，又保持了接口的灵活性。这一解决方案为类似动态接口生成场景提供了有价值的参考。