Skip to content

fix(providers): sync xai defaultModel with grok-4.5 recommended flag#5512

Merged
waleedlatif1 merged 1 commit into
stagingfrom
fix-xai-default-model
Jul 8, 2026
Merged

fix(providers): sync xai defaultModel with grok-4.5 recommended flag#5512
waleedlatif1 merged 1 commit into
stagingfrom
fix-xai-default-model

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Follow-up to feat(providers): add xAI grok-4.5 model #5511 (already merged) — that PR moved recommended: true to grok-4.5 but left defaultModel: 'grok-4.3' unsynced, flagged by Greptile as a valid P1 after merge
  • Updates xai's defaultModel to grok-4.5 to match the recommended flagship, consistent with how anthropic/google already keep defaultModel in sync with their recommended model

Type of Change

  • Bug fix

Testing

Tested manually (biome-clean, no test assertions reference the old default)

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Jul 8, 2026 6:50pm

Request Review

@cursor

cursor Bot commented Jul 8, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Single metadata field in the provider catalog; no auth, persistence, or runtime logic changes beyond the default model id for xAI.

Overview
Aligns xAI’s defaultModel with grok-4.5, which is already marked recommended: true after a prior catalog update left the default stuck on grok-4.3.

New xAI agent blocks and any code that calls getProviderDefaultModel('xai') will pre-select Grok 4.5 instead of 4.3, matching how other providers keep defaults in sync with their flagship model.

Reviewed by Cursor Bugbot for commit 20765c6. Configure here.

@greptile-apps

greptile-apps Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR fixes a stale defaultModel on the xAI provider that was left pointing at grok-4.3 after recommended: true was moved to grok-4.5 in the preceding PR #5511.

  • Updates defaultModel from 'grok-4.3' to 'grok-4.5' in apps/sim/providers/models.ts so that the default model aligns with the recommended flagship, matching the convention followed by other providers (Anthropic, Google).

Confidence Score: 5/5

Safe to merge — one-line configuration change with no logic impact.

The change is a single field update in a static configuration object, bringing defaultModel into alignment with the model already marked recommended: true. Both values exist in the models array, no downstream logic is altered, and the pattern mirrors how other providers in the same file are structured.

No files require special attention.

Important Files Changed

Filename Overview
apps/sim/providers/models.ts Single-field change updating defaultModel from grok-4.3 to grok-4.5 to match the model already carrying recommended: true; consistent with the convention used by other providers.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[xAI Provider Definition] --> B[defaultModel]
    A --> C[models array]
    B -->|was grok-4.3| D[grok-4.3 - no recommended flag]
    B -->|now grok-4.5| E[grok-4.5 - recommended: true]
    C --> E
    C --> D
    C --> F[grok-4-latest]
    style E fill:#4ade80,stroke:#16a34a
    style D fill:#f87171,stroke:#dc2626
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    A[xAI Provider Definition] --> B[defaultModel]
    A --> C[models array]
    B -->|was grok-4.3| D[grok-4.3 - no recommended flag]
    B -->|now grok-4.5| E[grok-4.5 - recommended: true]
    C --> E
    C --> D
    C --> F[grok-4-latest]
    style E fill:#4ade80,stroke:#16a34a
    style D fill:#f87171,stroke:#dc2626
Loading

Reviews (1): Last reviewed commit: "fix(providers): sync xai defaultModel wi..." | Re-trigger Greptile

@waleedlatif1 waleedlatif1 merged commit 117fa66 into staging Jul 8, 2026
13 checks passed
@waleedlatif1 waleedlatif1 deleted the fix-xai-default-model branch July 8, 2026 18:52
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.

1 participant