fix(usage): add manual backfill for missing historical costs#2586
Open
Ghibli1024 wants to merge 1 commit intofarion1231:mainfrom
Open
fix(usage): add manual backfill for missing historical costs#2586Ghibli1024 wants to merge 1 commit intofarion1231:mainfrom
Ghibli1024 wants to merge 1 commit intofarion1231:mainfrom
Conversation
Owner
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
|
Codex Review: Didn't find any major issues. What shall we delve into next? ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
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.
Summary / 概述
为“用量统计 / 模型定价”面板增加一个手动“补全历史成本”操作。
当用户在已有用量记录之后才补充某个模型的定价时,历史记录里的
0成本不会自动重新计算。这个 PR 把已有的历史成本回填能力暴露到前端,让用户在补充或更新模型定价后,可以主动把历史上未计费的用量重新计算出来。Problem / 问题背景
用量成本是在请求日志写入时计算的。如果当时模型还没有价格配置,这些日志会以
0成本保存。之后用户虽然可以在模型定价里手动添加价格,但这只更新了价格表,并不会自动改写已经存在的历史用量记录。因此会出现“新价格已经配置了,但历史统计金额仍然缺失”的情况。
What Changed / 变更内容
backfill_missing_usage_costs数据库回填逻辑。PricingConfigPanel组件测试,覆盖点击按钮、触发回填和成功提示。Related Issue / 关联 Issue
Fixes #2532
Screenshots / 截图
Checklist / 检查清单
pnpm typecheckpasses / 通过 TypeScript 类型检查pnpm format:checkpasses / 通过代码格式检查cargo clippypasses (if Rust code changed) / 通过 Clippy 检查(如修改了 Rust 代码)Validation / 验证
pnpm typecheckpnpm format:checkpnpm vitest run tests/components/PricingConfigPanel.test.tsx --maxWorkers=1 --minWorkers=1cargo fmt --checkcargo clippy -- -D warningsTZ=UTC cargo testFrontend Checks,Backend Checks)