Skip to content

chore(settings): 删除未被引用的 sortModelsBySelection#277

Merged
su-fen merged 1 commit into
mainfrom
chore/remove-dead-sort-models-by-selection
Jul 26, 2026
Merged

chore(settings): 删除未被引用的 sortModelsBySelection#277
su-fen merged 1 commit into
mainfrom
chore/remove-dead-sort-models-by-selection

Conversation

@su-fen

@su-fen su-fen commented Jul 26, 2026

Copy link
Copy Markdown
Member

改动说明

删除 providerUtils.ts 中未被任何代码引用的 sortModelsBySelection,GUI 与 Gateway WebUI 各一份。

该函数自 initial commit 起就没有调用点。供应商模型列表的排序职责现在由 lib/providers/modelVendor 承担(sortModelsByActiveStateAndVendor 负责启用态+厂商排序,createModelOrderSnapshot / applyModelOrderSnapshot 负责模态框内的显示顺序快照),这份实现属于历史遗留。

纯删除,无行为变更。

缘起:审核 #265 时顺带发现。当时刻意没有夹带进那个功能 PR —— providerUtils.ts 不在 #265 的改动范围内,也不在 scripts/mirror-manifest.json 中(两端靠手工同步,CI 查不出漂移),故单开此 PR。

改动类型

  • 新功能
  • Bug 修复
  • 重构
  • 文档更新
  • 其他

测试说明

基于 main dd519470(即 #265 合入后)验证:

  • GUI tsc --noEmit 通过(0 error)
  • Gateway WebUI tsc --noEmit 通过(0 error)
  • GUI 前端测试 1255/1255 通过
  • Gateway WebUI 全量测试 440/440 通过
  • GUI 与 Gateway WebUI build 均通过
  • Mirror Check 通过(105 文件)
  • 改动文件 Biome 检查零警告
  • git diff --check 干净
  • 全仓检索 sortModelsBySelection 已无残留引用(含 re-export 与动态引用检查)
  • 删除后两份 providerUtils.ts 仍逐字节一致

关联 Issue

该函数自 initial commit 起就没有任何调用点,两端前端各存一份。模型列表的
排序现由 lib/providers/modelVendor 的 sortModelsByActiveStateAndVendor 与
createModelOrderSnapshot 负责,这份实现属于历史遗留。

纯删除,无行为变更。GUI 与 WebUI 两份镜像同步修改,删除后两文件仍逐字节一致。
@su-fen
su-fen merged commit 961a87a into main Jul 26, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant