chore(settings): 删除未被引用的 sortModelsBySelection#277
Merged
Conversation
该函数自 initial commit 起就没有任何调用点,两端前端各存一份。模型列表的 排序现由 lib/providers/modelVendor 的 sortModelsByActiveStateAndVendor 与 createModelOrderSnapshot 负责,这份实现属于历史遗留。 纯删除,无行为变更。GUI 与 WebUI 两份镜像同步修改,删除后两文件仍逐字节一致。
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
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.
改动说明
删除
providerUtils.ts中未被任何代码引用的sortModelsBySelection,GUI 与 Gateway WebUI 各一份。该函数自 initial commit 起就没有调用点。供应商模型列表的排序职责现在由
lib/providers/modelVendor承担(sortModelsByActiveStateAndVendor负责启用态+厂商排序,createModelOrderSnapshot/applyModelOrderSnapshot负责模态框内的显示顺序快照),这份实现属于历史遗留。纯删除,无行为变更。
改动类型
测试说明
基于 main
dd519470(即 #265 合入后)验证:tsc --noEmit通过(0 error)tsc --noEmit通过(0 error)git diff --check干净sortModelsBySelection已无残留引用(含 re-export 与动态引用检查)providerUtils.ts仍逐字节一致关联 Issue
无