执行摘要
- 一句话:更新 Mergify 工具调用标签规则
- 推荐动作:值得合并,属于低成本的流程改进,提高标签一致性。
功能与动机
PR body 指出,涉及 vllm/parser/、vllm/reasoning/、tests/parser/ 或 tests/reasoning/ 的 PR 与 tool-calling 功能密切相关,但未自动获得 tool-calling 标签,因此需要更新 Mergify 配置以统一打标。
实现拆解
- 修改 Mergify 配置文件:在
.github/mergify.yml 的 tool-calling 标签规则中,向 files 条件列表添加四个新模式——^tests/parser/、^tests/reasoning/、^vllm/parser/、^vllm/reasoning/。
- 保持现有规则不变:原有
tests/tool_use/、tests/tool_parsers/、vllm/tool_parsers/ 等目录仍保留,仅做追加。
- 仅修改基础设施配置:变更完全限于 CI/CD 自动打标配置,无源码或测试逻辑改动。
关键文件:
.github/mergify.yml(模块 部署脚本;类别 infra;类型 infrastructure): 本 PR 唯一修改的文件,更新了 tool-calling 标签的匹配规则,新增四个目录模式。
关键符号:未识别
评论区精华
Review 仅由 yewentao256 批准,评论为“LGTM, thanks for the work!”,无实质讨论或争议。
风险与影响
- 风险:风险极低。变更仅影响 Mergify 自动标记规则,不涉及代码逻辑、性能或功能行为。唯一风险是误标,但新增目录确实与 tool-calling 关联(解析器和推理逻辑),故误标概率小。
- 影响:影响范围小,仅限于 CI/CD 标签流程。此后涉及
parser、reasoning 模块的 PR 会自动获得 tool-calling 标签,便于分类和审查。对用户无影响。
- 风险标记:元配置文件变更
关联脉络
- PR #45701 [Frontend] Add Streaming Parser Engine and new MinimaxM2 Parser: 新增了 parser 模块,正属于本 PR 标签规则覆盖范围,本 PR 可确保此类 PR 自动标记 tool-calling。
- PR #45763 [Bugfix] Fix Qwen3 prompt tool-call reasoning false positive: 修改了 qwen3 parser 和 reasoning 测试,属于本 PR 新增标签规则覆盖的路径。
参与讨论