Read model prices and context windows from LiteLLM instead of hand-kept tables - #40
Merged
Conversation
Contributor
Author
|
6e190b8 removes model o4-mini-2025-04-16 from models json, upon merge we can test workflow to see if we updated the models json correctly |
luhe19001
reviewed
Aug 5, 2026
luhe19001
left a comment
Collaborator
There was a problem hiding this comment.
This backend migration regresses cost and context-window handling for model IDs that were supported by the previous pricing table but are absent from the generated LiteLLM subset. For sessions using previously supported IDs such as claude-mythos-5, gpt-5.5-pro, or gpt-5.5-cyber, the lookup now fails after only removing a date suffix, so usage becomes unpriced and contributes $0. claude-mythos-5 also loses its context window. Please preserve local aliases or fallback entries for these IDs when moving pricing to the backend.
luhe19001
reviewed
Aug 5, 2026
calvintvu
force-pushed
the
cvu/auto-pull-model-data
branch
from
August 5, 2026 23:30
6e190b8 to
1d22057
Compare
luhe19001
reviewed
Aug 6, 2026
calvintvu
force-pushed
the
cvu/auto-pull-model-data
branch
from
August 6, 2026 23:06
c633c22 to
e61909b
Compare
calvintvu
force-pushed
the
cvu/auto-pull-model-data
branch
from
August 7, 2026 20:32
b2dc78b to
3f5832c
Compare
calvintvu
force-pushed
the
cvu/auto-pull-model-data
branch
from
August 10, 2026 17:44
3f5832c to
78bcbc5
Compare
luhe19001
approved these changes
Aug 10, 2026
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.
Context
we dont want to manually sync model information, we'll follow
ccusagepattern and pull info from litellmChanges
make modelsrewrites the table. The Models workflow runs it daily and opens a pull request, so a person reads every price change before it merges.Test
make check- passed. The collector carries no test files, somake testruns nothing.npm run format:check,npx tsc -b,npm run lint,npm test(8 tests),npm run build- all passed/api/sessions. 191 sessions, total cost $1166.56, no unpriced model, subagent costs present, claude-sonnet-5 window 1,000,000.claude-opus-5deleted reported the drop and exited 1.o4-mini-2025-04-16from the table on purpose, to watch the workflow report a model as added. Revert it before merge.Screenshots