Fix DeepSeek v4 Claude pricing through stale runtime cache#367
Open
ozymandiashh wants to merge 1 commit into
Open
Fix DeepSeek v4 Claude pricing through stale runtime cache#367ozymandiashh wants to merge 1 commit into
ozymandiashh wants to merge 1 commit into
Conversation
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
Addresses #365 by pricing DeepSeek v4 Claude-compatible sessions instead of reporting them as $0.
Root cause
CodeBurn relies on LiteLLM pricing data, but
deepseek-v4-proanddeepseek-v4-flashare not yet available in the bundled/runtime LiteLLM data used by current CodeBurn. A fresh 24-hour runtime pricing cache can also replace the bundled snapshot with data that still lacks these models, so adding snapshot entries alone is not enough.What changed
Validation
I ran the same Claude JSONL fixture against
origin/mainand this branch. The fixture contains two Claude-compatible assistant calls using the reported DeepSeek v4 model ids, plus a fresh runtime LiteLLM cache that does not contain those models.Input shape:
Before, on
origin/main, CodeBurn parsed the calls and tokens but priced both models at zero:After, on this branch, the same input produces non-zero costs and the expected display names:
That before/after path is now covered by
tests/cli-deepseek-v4-pricing.test.ts, which runs the CLI against the realistic Claude fixture and stale runtime cache.Notes