feat: add Modelink endpoint#30
Conversation
There was a problem hiding this comment.
Code Review
此拉取提交主要更新了项目的 API 线路配置,将默认国内线路更改为 https://openai.qiniu.com,原国内线路 https://api.qnaigc.com 降级为备用线路,并新增了 Modelink 线路(https://api.modelink.ai)。相关修改同步更新到了文档、多语言本地化文件以及端点定义中。审查意见指出,在 src/lib/tools/codex-tool.ts 中硬编码默认端点地址可能会导致维护问题,建议使用 configManager.baseUrl 动态获取当前配置的端点。
| content = upsertTopLevelModelProvider(content); | ||
|
|
||
| const providerBaseUrl = `${(baseUrl || 'https://api.qnaigc.com').replace(/\/+$/, '')}/bypass/openai/v1`; | ||
| const providerBaseUrl = `${(baseUrl || 'https://openai.qiniu.com').replace(/\/+$/, '')}/bypass/openai/v1`; |
There was a problem hiding this comment.
[P2] 避免硬编码默认端点地址
触发条件:在 buildCodexConfig 中,当 baseUrl 未传入时,代码硬编码回退到 'https://openai.qiniu.com'。
实际影响:如果未来默认端点发生变化,或者用户配置了其他端点,这里硬编码的默认值会导致不一致,增加维护成本。
改进建议:由于 configManager 已经导入,可以直接使用 configManager.baseUrl 作为回退值,这样可以动态获取当前配置的端点,避免硬编码。
| const providerBaseUrl = `${(baseUrl || 'https://openai.qiniu.com').replace(/\/+$/, '')}/bypass/openai/v1`; | |
| const providerBaseUrl = (baseUrl || configManager.baseUrl).replace(/\/+$/, '') + '/bypass/openai/v1'; |
References
- 避免硬编码默认端点地址,应使用配置管理器动态获取。 (link)
|
Review completed across code quality, performance, security, and documentation accuracy passes. No noteworthy issues found in the changed code. |
Codex 评审结论:通过 问题未发现需要阻塞或关注的问题。 本次 PR 主要影响 验证说明
|
c977502 to
bce2d4d
Compare
Codex 评审结论:通过 问题 未发现需要关注的 bug、行为回归、安全风险、数据丢失风险或 CLI 交互回归。 评审说明 本 PR 主要影响线路定义与默认线路、线路选择菜单展示、配置面板中的线路文案,以及 Codex 未显式传入 baseUrl 时的默认写入地址。新增 交互报告覆盖了主菜单、工具选择器,以及动态发现的 Claude Code、Codex、CodeBuddy、WorkBuddy、Hermes Agent 工具菜单键盘导航;方向键、回车确认、返回/退出路径均通过,未看到菜单闪烁失控、卡住或输出不可读。交互脚本使用临时 HOME/USERPROFILE,未发现污染真实 HOME 的证据。 测试/验证 已读取测试产物: |
Summary
Testing