Repository navigation
Conversation
GPT-6 Astra 自动代码审查审查提交: 由 Codex 定时任务使用 GPT-6 Astra 子 agent 审查,并经主 agent 复核。 本次审查未发现可确认的问题。 验证与范围已对精确 BASE...HEAD 的全部五个变更文件进行独立审查,并覆盖关键生产调用链与新增测试。生产变更仅新增 DeepSeek V4.1 Flash 定价种子及调整列表排序;补种保留已有价格,模型清洗可命中新条目,列表排序不改变计费查询。5 项隔离离线测试全部通过,未发现具有足够证据、由本次改动新增且可发布的真实 bug、安全缺陷或行为回归。coverage_complete 表示 diff 与关键上下文已完整读取,不表示运行了完整测试套件。
未验证或受限范围:
本评论仅针对以上提交的代码审查结果,不构成合并批准。 |
1 similar comment
|
@farion1231 已同步变基至最新 v4.0.4 主线。修复 DeepSeek V4.1 Flash 定价表匹配与排序异常,分支无冲突且 CI 全绿,方便时麻烦审阅一下,谢谢! |
|
@farion1231 这边已经顺手跟最新的 main 对齐过了,依然可以 clean merge,等空下来看下哈,不着急~ |
…_calculation signature PR farion1231#7632 的测试按旧签名写(含已移除的 cost_multiplier 参数,缺 cache_creation_1h_tokens), merge-tree 干净但编译不过;补齐字段、去掉多余参数,行为与断言不变。
|
感谢修复和补的测试! 排序这部分在当前 main 上依然有效:定价列表现在还是按 另外有两处变化需要处理一下:
麻烦 rebase 到最新 main 后我们再看,谢谢! |
Seed `deepseek-v4.1-flash` so provider-prefixed dotted IDs such as `deepseek-ai/DeepSeek-V4.1-Flash` resolve to the V4.1 Flash price instead of silently costing zero. Add regressions for dotted IDs, provider prefixes, mixed case, and the existing legacy aliases. Refs farion1231#7512 Signed-off-by: Xingqiu Xu <hilbertanjou83@gmail.com>
Sort non-empty display names case-insensitively with model_id fallback and tie-breaking so custom provider-prefixed rows stay in their expected group. Add logger-level regressions for the raw V4.1 ID, recommended name, and legacy aliases. Refs farion1231#7512 Signed-off-by: Xingqiu Xu <hilbertanjou83@gmail.com>
8fd7b01 to
2f92b55
Compare
|
@farion1231 感谢指导! 已完成变基并处理好冲突与测试适配:
|
Summary
deepseek-v4.1-flashso provider-prefixed dotted model IDs such asdeepseek-ai/DeepSeek-V4.1-Flashresolve to the DeepSeek V4.1 Flash price instead of being billed at zero.displayNameusing case-insensitive ordering, withmodel_idas the fallback for empty names and the stable tie-breaker for duplicate display names.Verification
cargo test --lib deepseek_v41 -- --nocapture— 4 passedcargo test --lib model_pricing -- --nocapture— 19 passedcargo fmt --all -- --check— passedcargo clippy --lib --tests— passed with one pre-existing unrelated warning intransform_codex_chat.rsCloses #7512