Repository navigation
fix(settings): make gateway auth cleanup configurable with compatible defaults - #464
Conversation
|
审查结论:实现完成,建议进入 review。 这次改动的方向是把
当前本地验证:
行为回归覆盖:
完整 service lib 串行验证结果为 |
|
审查结论:暂缓合并,保留配置的能力有价值,但需要适配当前 main 并验证图片扩展行为。 本次依据 head 目前有两个合入前需要解决的问题:
这里并不是说该字段与 bearer token 必然冲突:官方 auth.rs 优先解析 建议修复路径:
建议验证: cargo fmt --all -- --check
cargo test -p codexmanager-service --lib codex_profile::tests -- --test-threads=1
cargo test -p codexmanager-service --test app_settings -- --test-threads=1
pnpm -C apps run test:runtime
pnpm -C apps run build:desktop以上兼容风险来自源码与官方版本对照,本次未运行该 PR 的真实图片链路。解决冲突、补齐实际链路证据并让最新 head 的 CI 通过后,可以重新评估合入。 |
|
已补齐修复并解决与当前 main 的冲突,提交
审查通过,可以合并。本次没有调用真实图片模型;选择保留旧 |
托管网关此前总是清理
cmprovider 的requires_openai_auth,无法满足需要保留旧配置的用户。本 PR 增加持久化开关,并保持当前主线的兼容默认值:默认开启清理,关闭时只保留已有值,不恢复曾删除的字段。true可能禁用 Codex 图片/actor 扩展。验证:profile 43 项、app_settings 35 项、前端 runtime 250 项通过;build:desktop 静态构建及浏览器开关、RPC patch、刷新后状态保留回归通过;格式和最终 PR diff 检查通过。最新 CI 结果见维护者评论。
没有调用真实图片模型;默认行为与当前 main 一致,主动保留旧 true 的兼容限制已在界面和文档中说明。