chore: add more known model prices to ChatModelAdminPanel - #27839
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the ChatModelAdminPanel’s generated known-model catalog by expanding the curated input set for the price generator, and regenerating the frontend’s knownModelsGenerated.json so more providers/models have known token pricing metadata available.
Changes:
- Expanded
scripts/aibridgepricesgen/curation.jsonwith additional curated models across more providers (Azure, Bedrock, Google, OpenRouter, Vercel). - Regenerated
knownModelsGenerated.jsonto include the newly curated providers/models and updated some existing costs. - Attempted to mark
knownModelsGenerated.jsonas generated in.gitattributes(currently missing thelinguist-generated=trueattribute).
Frontend Review (FE1-FE10)
FE1 PASS
FE2 PASS
FE3 PASS
FE4 PASS
FE5 PASS
FE6 PASS
FE7 PASS
FE8 PASS
FE9 PASS
FE10 PASS
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| site/src/pages/AgentsPage/components/ChatModelAdminPanel/knownModels/knownModelsGenerated.json | Regenerated known-model metadata to include more providers/models and updated some price fields. |
| scripts/aibridgepricesgen/curation.json | Expanded curated model list that drives the generated known-model pricing artifact. |
| .gitattributes | Intended to mark the generated known-model artifact as generated for GitHub linguist. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "cacheWriteCost": 3.75 | ||
| } | ||
| ], | ||
| "azure": [ |
There was a problem hiding this comment.
Hum, so we have the same data stored in 2 files:
- prices.json
- and this one
Would it make sense to merge them into one? We would probably need to adjust the units, though. Not now, but as a potential follow-up
There was a problem hiding this comment.
We should definitely only have one source of truth for these. It doesn't make sense to have two
There was a problem hiding this comment.
FWIW it's the same source of truth, but two different views.
a6c1998 to
c542855
Compare
|
AIGOV-571 |
Depends on #27837
Expand
curation.jsonto include more known models.Also marked
knownModelsGenerated.jsonas generated.