Prhub

sgl-project/sglang · 标签视图

标签列表

聚合结果

function-call 相关 PR

2026-09-13

#37015 [Test] Add unit test for muse_glimmer_format

原始 PR · 作者 Hritik003 · 合并时间 2026-09-13 18:40

测试 重要性 7.25 洞察度 4.00

为 muse_glimmer_format 流式助手新增 31 个 CPU 单元测试

值得快速浏览,重点是 TestPartialMarkerLen 与 TestCouldStartHeader 的边界用例设计——它们清晰呈现了流式解析“暂缓输出 vs 立即输出”的判定规则,对后续编写其他流式解析器的单元测试有直接借鉴价值。PR 本身技术含量不高,无需精读完整实现。

缺陷修复 重要性 8.60 洞察度 7.00

修复 GLM 工具参数类型在 Schema 联合中被过早定型

值得精读,尤其是负责 function calling / 结构化输出与流式一致性的同学。建议重点关注三处设计决策:(a) `_argument_types` 用「约束集求交 + union 求并」替代单向推断,以及 `const`/`enum` 只在与已完成取值相容时参与收窄的条件;(b) `_needs_argument_context` 用「是否需要缓冲」把延迟定型的影响限制在真正含 union 的工具上,而非全局缓冲;(c) `_coerce_numeric_string` 刻意保留 `number`/`integer` 的非对称(`"1.5"` 对 integer 保持字符串)。阅读顺序建议先看 `glm47_moe_detector.py` 的类型解析函数,再看 `test_glm47_schema_types.py` 的 `check_arguments` 断言方式(值 + 原生类型双重校验、多分块粒度)。后续行动建议:推动 `glm4_moe_detector`/`ling3_detector` 同步修复,并在 #35625 合并前确认两者的方案不会冲突。

2026-08-28
缺陷修复 重要性 7.81 洞察度 6.00

修复顶层 anyOf/oneOf/allOf 工具参数解析,覆盖 13 个解析器

值得精读。建议关注三点:(1) get_schema_properties 的递归合并与 setdefault 首分支优先策略,可作为"兼容组合 schema"的通用样板;(2) glm 流式收尾从字符串启发式到状态判断的修复思路,类似的 endswith 误判在流式解析中很常见;(3) 测试裁剪策略——只保留未修复代码上必失败的用例,避免回归测试"假绿"。对正在维护工具调用解析器或接入新模型的工程师有直接参考价值。