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

Copilot AI left a comment

Copy link
Copy Markdown

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": [
{

Copilot AI Mar 26, 2026

Copy link

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
}
]
},
{

Copilot AI Mar 26, 2026

Copy link

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.
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.

2 participants