Skip to content

Add new Grok 4.20 models with price history#57

Open
damdafayton wants to merge 1 commit into
simonw:mainfrom
damdafayton:patch-1
Open

Add new Grok 4.20 models with price history#57
damdafayton wants to merge 1 commit into
simonw:mainfrom
damdafayton:patch-1

Conversation

@damdafayton
Copy link
Copy Markdown

No description provided.

Copilot AI review requested due to automatic review settings March 26, 2026 15:50
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds entries for new xAI Grok 4.20 models to the pricing data so they appear in generated current/historical price outputs.

Changes:

  • Added grok-4.20-0309-reasoning model with a current price record.
  • Added grok-4.20-0309-non-reasoning model with a current price record.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread data/xai.json
"id": "grok-4.20-0309-non-reasoning",
"name": "Grok 4.20 Non-Reasoning",
"price_history": [
{
Copy link

Copilot AI Mar 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There appears to be trailing whitespace after the { on this line in the price_history entry. Please remove the extra spaces to keep the JSON formatting consistent (and avoid noisy diffs / formatter churn).

Suggested change
{
{

Copilot uses AI. Check for mistakes.
Comment thread data/xai.json
}
]
},
{
Copy link

Copilot AI Mar 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The opening { for the second new model entry is mis-indented compared to the other items in the models array, which makes the JSON hard to scan and suggests a formatting mistake. Please align it to the same indentation level as the surrounding model objects.

Suggested change
{
{

Copilot uses AI. Check for mistakes.
@hooch
Copy link
Copy Markdown

hooch commented Apr 28, 2026

@simonw the value of this repo diminishes pretty fast if pricing can't be kept fresh?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants