模型排序 - #18
Merged
Merged
Conversation
BigStrongSun
added a commit
that referenced
this pull request
Aug 17, 2026
PR #18 合入后本地门禁发现两处问题，均已根修： 1. 编译错误（E0063）：CodexCatalogModelSpec 新增 sort_index 字段后，PR 漏改了 3 处 测试夹具初始化（codex_config.rs 8196/8261/10841 行附近， codex_subagent_v2_real_sync_rewrites_managed_output_for_changed_settings、 codex_subagent_v2_same_retained_settings_survive_v1_cleanup_and_v2_restore、 以及 deepseek-v4-flash spec 用例），按 PR 自身对其它夹具的统一模式补 sort_index: None。 2. Prettier 格式：CodexRouterWorkspacePage.tsx 的 hasChanges 链式调用缩进不符合 仓库 Prettier 配置，prettier --write 修正（纯缩进，无逻辑变化）。 门禁结果（worktree .worktrees/pr18-model-sort，main + PR #18）： - pnpm typecheck 通过 - pnpm format:check 通过 - pnpm vitest run CodexRouterWorkspacePage.test.ts 54/54 通过 - cargo test --lib codex_config:: 178/178 通过（含新增 codex_model_catalog_uses_user_model_sort_index_for_picker_order） - cargo test --lib 全量 3108 passed / 0 failed / 5 ignored - cargo fmt --check 通过 本次提交由BigStrongsSun完成
BigStrongSun
added a commit
that referenced
this pull request
Aug 17, 2026
- 记录用户 PR #18（模型排序）审查、合入与门禁：merge b40d914 + 修复 962d42d 已 push fork main，GitHub MERGED。 - 记录审查发现的两处 PR 缺陷根修（3 处测试夹具缺 sort_index 的 E0063 编译错误、Prettier 缩进）与完整门禁数据（typecheck/format:check/vitest 54+43/codex_config 178/全量 lib 3108/fmt）。 - 记录行为边界：用户 sortIndex 优先于 spawn_agent_model_priority，仅影响 picker 展示顺序，需重启 Codex Desktop 生效。 本次提交由BigStrongsSun完成
BigStrongSun
added a commit
that referenced
this pull request
Aug 17, 2026
- 合入 bigstrongsun/fix-responses-chat-turn-coalescing：Stage A 扩围（第三方中转父模型也剥离 message.encrypted，修复第三方父→第三方子 V2 加密任务失败）+ Responses→Chat commentary 与工具调用合并（修复 Qwen 纯进度 stop 短轮）。 - 合入 bigstrongsun/release-v3.19.2-6：补 Qwen 英文 raw reasoning 边界 memory 记录。 - 主干已含 PR #18 模型排序与 CI Clippy 三连根修（0224cc4e/338f69de/acb32709）。 - 版本号 3.19.2-5 → 3.19.2-6（package.json / Cargo.toml / Cargo.lock / tauri.conf.json）。 - 回归：Rust lib 3109 passed / 0 failed / 5 ignored，cargo fmt --check 与 git diff --check 通过。 本次提交由BigStrongsSun完成
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
2 participants
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 / 概述
本次 PR 主要是针对 Codex 多模型路由进行模型排序，使 Codex 中的自定义模型按照指定顺序展示
Related Issue / 关联 Issue
Fixes #
Screenshots / 截图
旧版 ccsm
pr 新增顶部模型排序 tab，拖动即可进行模型排序
Checklist / 检查清单
pnpm typecheckpasses / 通过 TypeScript 类型检查pnpm format:checkpasses / 通过代码格式检查cargo clippypasses (if Rust code changed) / 通过 Clippy 检查（如修改了 Rust 代码）