执行摘要
本PR修复了Ernie工具解析器在流式解析中错误过滤空白字符的bug，并重构核心逻辑为基于标签计数的状态机方案，显著提升解析鲁棒性；同时单元测试从约120行大幅扩展到840行，确保各种边界case得到覆盖，是APIServer模块的重要改进。
功能与动机
主要解决两个问题：首先，ErnieX1ToolParser和Ernie45VLThinkingToolParser的流式解析方法extract_tool_calls_streaming中，对delta_text.strip()的空白检查会错误丢弃仅包含\n或空格的有效token，导致流式输出内容缺失或tool call arguments丢失空白字符；其次，原ErnieX1ToolParser基于buffer累积和手动括号匹配的逻辑复杂，存在边界问题如partial JSON回退和unclosed tool_call block处理。目标是通过重构使解析逻辑更清晰、鲁棒性更强，并完善测试以预防回归。
实现拆解
ernie_x1_tool_parser.py：重写extract_tool_calls方法，改用正则表达式<tool_call>\s*(\{.*?\})\s*</tool_call>提取JSON并直接json.loads；重写extract_tool_calls_streaming方法，引入基于<tool_call>/</tool_call>标签计数的状态机，借助partial_json_parser进行增量解析，移除原复杂括号匹配逻辑。ernie_45_vl_thinking_tool_parser.py：删除if len(delta_text.strip()) == 0: return None的空检查逻辑，避免有效token被过滤。test_ernie_x1_tool_parser.py：从基础测试扩展为覆盖初始化、批量提取、流式解析的各分支路径，包括多tool call、异常处理和边界case，代码行数从约120行增加到840行。
评论区精华
Review讨论中，Copilot提出了多个关键洞察：
- 正则风险：指出正则
\{.*?\}可能在arguments含嵌套对象时提前截断，导致JSON解析失败。作者回复“无需修复”，暗示设计权衡。
- 流式逻辑错误：发现
full_text = current_text + delta_text重复拼接，造成解析错位。作者回复“已修复”，修正了逻辑。
- 测试覆盖：建议补充单测以避免回归。作者回复“已增加单测”，大幅扩展测试文件。
- 文档规范：指出标题缺少标签和描述不全。作者回复“已补充”，完善了文档。
风险与影响
- 技术风险：正则表达式可能无法处理嵌套JSON，如
{"arguments": { ... }}场景，导致解析失败；流式解析逻辑改动可能引入新bug，如输出错位；尽管测试扩展，但嵌套JSON覆盖需验证；核心路径变更可能影响上游APIServer组件。
- 影响范围：用户侧修复了流式输出中空白字符丢失问题，提升Tool Call准确性；系统侧解析更鲁棒，但需真实场景测试；团队侧需确保测试通过并关注相关代码更新。
关联脉络
从近期历史PR看，PR 7054同样涉及APIServer模块的改动，扩展了/config-info端点，显示团队在持续完善OpenAI协议支持。本PR作为Tool解析器的关键重构，可能为后续APIServer功能演进奠定基础，但无直接相同文件修改的关联PR。
参与讨论