Bubbles/function_call_review.md

Function Call 改造代码审核

阻塞问题（必须修复）

• [已完成] FunctionResult 结果封装统一

  • 处理位置：function_calls/spec.py:9、function_calls/router.py:195、commands/context.py:66、robot.py:296
  • 现状：结果模型已改为 at: str 并提供 dispatch 方法，路由器不再手动拼接 @ 列表。python3 -m compileall function_calls 验证通过，群聊场景使用新的 at 字段。

• [已完成] 提醒功能使用结构化参数

  • 处理位置：function_calls/models.py:19、function_calls/services/reminder.py:36、function_calls/handlers.py:70、function_calls/router.py:70
  • 将 ReminderArgs 改为 type/time/content/weekday 等字段，移除了对旧 commands.handlers 的依赖，并删除路由层对提醒的直接自然语言拼装。

重要改进项

• [已完成] Handler 逻辑脱离旧命令体系

  • 现状：所有 handler 均迁移到 function_calls/services（天气、新闻、提醒、Perplexity、骂人、群工具等），不再篡改 ctx.msg.content 或调用旧命令模块。

• [已完成] 直接命令路径参数校验

  • 现状：function_calls/router.py:102-119 对直接匹配的函数调用 validate_arguments，与 LLM 分支保持一致。

• [已完成] FunctionSpec 类型标注同步

  • 现状：function_calls/spec.py:27-34 中的 handler 类型现为 Callable[[MessageContext, Any], FunctionResult]。

架构一致性评估

• 当前所有功能均通过 Function Call 服务层完成，提醒/骂人/搜索等不再依赖自然语言解析。
• LLM 适配层维持兼容，必要时可扩展 jsonschema 校验和重试策略。
• robot.py:163-231 仅初始化和调用 FunctionCallRouter，旧的命令/AI 路由器已移除，配置项也同步精简。

建议的下一步

1. 扩充 function_calls/services 层的单元测试（例如提醒设置、Perplexity fallback 等），确保服务纯函数行为稳定。
2. 若后续新增工具函数，遵循 FunctionResult + service 的模式，并及时更新 FUNCTION_CALL_USAGE.md。
3. 观察线上日志，确认精简后的路由无遗漏场景；如需更多指令，优先在 direct-match 表中补充结构化参数生成逻辑。

如按以上步骤推进，可逐步达到“标准 Function Call 模式”预期：所有工具能力通过结构化 schema 暴露，handler 仅消费结构化参数，无需再回退自然语言解析。