Skip to content

fix(codex): 官方认证编辑保存时保留模型目录 - #113

Open
MattLYT wants to merge 1 commit into
BigStrongSun:mainfrom
MattLYT:codex/fix-official-auth-catalog-save
Open

MattLYT wants to merge 1 commit into
BigStrongSun:mainfrom
MattLYT:codex/fix-official-auth-catalog-save

Conversation

@MattLYT

@MattLYT MattLYT commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

概述

修复编辑 OpenAI Official 认证方式时,已有模型目录被表单丢弃、保存被后端目录必填校验拒绝的问题。

原因

表单此前将 modelCatalog 的保存条件绑定到 shouldPersistCodexLocalConfig。官方供应商未启用本地路由时不需要改写原生配置,但仍需要保存数据库中的模型目录。旧逻辑会从提交数据中删除目录,触发 codex_provider_set_model_catalog_required,使认证切换无法保存。此路径不依赖本机路径、真实账号或网络状态。

变更

  • 始终保存已有的标准化模型目录及子代理模型选择。
  • 默认模型自动补齐仍限定在需要本地配置投影的路径,保持官方原生 TOML 不变。
  • 增加使用合成账号和模型数据的认证切换回归测试,检查目录保留、认证选择保存、原生配置不变且不引入路由配置。

行为验证

  • 回归场景:官方供应商已有模型目录,原生配置中的默认模型处于禁用状态,从 Desktop 当前登录切换到 CCSM OAuth 固定账号后保存。原实现丢弃目录;修复后目录和认证选择均保留,原生 TOML 原样保存。
  • 最终独立提交的 5 个相关前端测试文件共 41 项通过,覆盖供应商预设、目录校验与顺序、官方认证状态和认证选择组件。
  • 认证保存修复已单独在安装态核对:设置保存成功,路由运行时采用 OAuth,并保留 8 个模型。此核对不代表过期令牌本身已由本 PR 修复。

关联 Issue

无,来自实际使用反馈。

截图

不适用。变更为表单提交数据修复,没有新增用户可见文本;不上传包含认证账号信息的现场截图。

检查清单

  • 通过 TypeScript 类型检查(直接运行已安装的 TypeScript,等价于 pnpm typecheck)
  • 通过全 src 格式检查及修改的测试文件格式检查
  • 前端生产构建通过
  • Clippy 检查:不适用,本 PR 未修改 Rust 代码
  • 国际化更新:不适用,本 PR 未修改用户可见文本
  • 提交范围仅为供应商表单和相关回归测试,不包含本地配置、数据库、凭证或安装产物

官方 Provider 编辑表单保存标准化模型目录,避免切换认证被目录必填校验拒绝;仍仅在需要本地配置投影时补齐默认模型,不改写官方原生 TOML。

验证:目录丢失回归先复现失败后通过;相关前端测试、类型检查与生产前端构建通过。该保存修复已单独部署并核对认证采用 OAuth。
@github-actions github-actions Bot added documentation Improvements or additions to documentation dependencies backend frontend labels Oct 8, 2026
@MattLYT
MattLYT force-pushed the codex/fix-official-auth-catalog-save branch from 7602e43 to d3751f4 Compare October 8, 2026 15:04
@github-actions github-actions Bot removed documentation Improvements or additions to documentation dependencies backend labels Oct 8, 2026
@MattLYT MattLYT changed the title fix(codex): 修复官方认证保存并防止状态刷新连带退出 fix(codex): 官方认证编辑保存时保留模型目录 Oct 8, 2026
@MattLYT
MattLYT marked this pull request as ready for review October 9, 2026 03:38

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant