修复 streaming 中 tool_choice=none 仍调用工具解析器的 bug
该 PR 是重要的正确性修复,值得所有使用工具解析功能的开发者关注。守卫位置和条件范围的设计决策(集中到 `_extract_tool_calls_streaming`、仅检查 `"none"`)值得在类似问题中借鉴。建议后续补充 Responses API 的回归测试,确保完全覆盖。
标签列表
聚合结果
修复 streaming 中 tool_choice=none 仍调用工具解析器的 bug
该 PR 是重要的正确性修复,值得所有使用工具解析功能的开发者关注。守卫位置和条件范围的设计决策(集中到 `_extract_tool_calls_streaming`、仅检查 `"none"`)值得在类似问题中借鉴。建议后续补充 Responses API 的回归测试,确保完全覆盖。
抑制工具解析器中 ast.literal_eval 的 SyntaxWarning
推荐默认合并。这是一次低风险、高收益的清理操作,能改善 Python 3.12+ 用户的日志可读性,且逻辑简单。
修复 Responses API 流式工具调用参数丢失
值得快速合并。修复明确,改动量小,风险低。可关注后续是否还有类似遗漏的调用点。
修复 HF chat template 渲染的数字精度和字段影子问题
本 PR 值得精读,尤其关注以下要点:(1)如何在不修改 MiniJinja 核心的前提下,通过自定义 Object 和 pycompat 后门解决字段与方法冲突的设计模式;(2)全局 `serde_json` 特性调整时的依赖影响评估思路;(3)测试用例设计中对渲染精度妥协的明确标注。对于同样使用 MiniJinja 渲染 HF 模板的其他项目,此方案具有直接参考价值。
原始 PR · 作者 william-rom · 合并时间 2026-06-03 13:17
修复 tool_call arguments 为 "null" 字符串时的崩溃
此 PR 值得合并,问题定位清晰,修复方式简单安全,测试充分。对于关注工具调用稳定性的团队,可直接参考此修复。
支持递归工具参数类型转换
建议合并,但需关注空字符串行为变化,考虑补充对应测试或文档说明。设计模式值得学习:使用 `Into<ParamInput>` 桥接新旧输入,以及递归模式处理策略。
原始 PR · 作者 albertoperdomo2 · 合并时间 2026-06-02 16:50
迁移 ResponsesParser 到统一 Parser 接口
该 PR 值得精读,展示了将遗留组件迁移到统一接口的典型模式。重点关注 `ResponsesParser.process()` 的简化和流式处理的缓存设计(未来改进方向)。建议合并后尽快跟进流式问题的 CI 测试。
统一推理与工具调用解析到 Parser.parse()
值得精读,因为统一解析入口是前端架构重构的关键步骤,为后续支持更多解析组合打下基础。需关注作者关于“匹配 streaming”的设计决策及其潜在的兼容性影响。