fix(VendorConfig): 修复原生 Anthropic 兼容供应商共享字段误报跨供应商警告#138
Merged
Conversation
_warn_irrelevant_fields 验证器仅跳过同名供应商，未处理共享字段集的供应商组， 导致 zhipu 配置 api_key 时误报"属于 minimax/kimi/doubao/xiaomi/alibaba 供应商"。 新增 field_name in exclusive 判断，跳过与当前 vendor 共享的字段； 同时修正测试 logger 名称并新增共享字段测试用例。 🤖 Generated with [Claude Code](https://github.com/claude), [CodeX](https://openai.com), [Gemini](https://github.com/apps/gemini-code-assist) Co-Authored-By: Aurelius Huang<threefish.ai@gmail.com>
ThreeFish-AI
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
_warn_irrelevant_fields验证器的共享字段误报问题：当配置vendor=zhipu并设置api_key后，启动服务会输出多条误报警告（字段 api_key 属于 minimax/kimi/doubao/xiaomi/alibaba 供应商，当前值将被忽略）_VENDOR_EXCLUSIVE_FIELDS中 zhipu/minimax/kimi/doubao/xiaomi/alibaba 均映射到同一个_NATIVE_ANTHROPIC_FIELDS（{"api_key"}），但验证器仅跳过同名供应商（vendor_type == self.vendor），未跳过共享相同字段集的其他供应商coding.proxy.config.schema→coding.proxy.config.routing，与实际 logger 模块路径一致Changes
src/coding/proxy/config/routing.pyif field_name in exclusive: continue，跳过与当前 vendor 共享的合法字段tests/test_schema.pytest_no_warning_for_native_anthropic_shared_fields测试用例Test plan
uv run pytest tests/test_schema.py -v— 16 tests passed（含新增用例）uv run pytest tests/ -v— 1127 tests passed，零回归uv run coding-proxy start— 启动时无误报警告输出🤖 Generated with Claude Code, CodeX, Gemini