fix(web): 统一协议显示规范 - #498
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: QUIET Plan: Pro Plus Run ID: 📒 Files selected for processing (9)
💤 Files with no reviewable changes (4)
Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review. 📝 WalkthroughWalkthrough前端各处的协议选项和结果字段现在直接显示原始协议值。代码移除了 Merge Risk: ⚪ Minimal · up to This change standardizes protocol labels in the web interface without changing protocol values, APIs, routing, or persistence. No actionable merge-blocking risk remains beyond normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (5 skipped: 5 unsupported.) ✨ Finishing Touches✨ Simplify code
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
关联 Issue / Related Issue
无
变更内容 / Change Content
Bug 修复 / Bug fix
新功能 / New feature
其他改动 / Other changes
移除
openai-embeddings在 AccessKey、路由检查、日志筛选和凭据测试中的专用友好名称与端点说明。所有协议统一展示 canonical value,不新增统一展示层或其他抽象。
删除不再使用的三语专用协议文案,并同步更新正式 Embeddings 实施方案。
兼容性:仅调整前端展示;协议值、管理 API、数据面路由和持久化均不变,无数据迁移。
自查清单 / Checklist
make check,或在说明中写明无法运行的原因和未验证范围。 / I ranmake check, or documented why it could not run and what remains unverified.Summary by CodeRabbit