[model-inventory] Add missing MAI-Code model entry and correct Copilot cache-read pricing - #52191
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
🎉 Great work on the model inventory update, Copilot! This PR is well-executed and looks ready for review. Your approach to keep both ✅ What's working well:
This PR is ready for maintainer review and merge.
|
There was a problem hiding this comment.
Pull request overview
Synchronizes Copilot model pricing catalogs with the latest inventory.
Changes:
- Adds
mai-code-1.1-flashpricing. - Corrects five cache-read prices.
- Adds regression tests for MAI-Code and Grok pricing.
Show a summary per file
| File | Description |
|---|---|
pkg/cli/model_costs_test.go |
Tests new and corrected pricing. |
pkg/cli/data/models.json |
Updates the CLI pricing catalog. |
actions/setup/js/models.json |
Mirrors the catalog updates. |
Review details
Tip
Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
- Files reviewed: 3/3 changed files
- Comments generated: 0
- Review effort level: Balanced
Daily inventory identified one missing
github-copilotmodel inmodels.jsonand severalcache_readmismatches, including a confirmed 10x underpricedgrok-4.5cache-read value. Alias coverage was already complete, so this PR is scoped to pricing/catalog correctness only.Model catalog sync (
pkg/cli/data/models.json,actions/setup/js/models.json)mai-code-1.1-flashwith Copilot SDK-aligned pricing (input,output,cache_read,cache_write).Pricing corrections (
github-copilotentries)cache_readfor:grok-4.5(5e-08→5e-07)gpt-5.4-mini(7.5e-08→7e-08)gpt-5.3-codex(1.75e-07→1.7e-07)gpt-5-mini(2.5000000000000002e-08→2e-08)mai-code-1-flash-picker(7.5e-08→7e-08)Regression coverage (
pkg/cli/model_costs_test.go)mai-code-1.1-flashpricing lookup (includingcache_write)grok-4.5pricing lookup with corrected cache-read value