Skip to content

fix(examples): update grok example to use grok-4.3#2846

Merged
dgageot merged 1 commit into
docker:mainfrom
dgageot:fix-grok-example
May 21, 2026
Merged

fix(examples): update grok example to use grok-4.3#2846
dgageot merged 1 commit into
docker:mainfrom
dgageot:fix-grok-example

Conversation

@dgageot
Copy link
Copy Markdown
Member

@dgageot dgageot commented May 21, 2026

Problem

TestParseExamples/../../examples/grok.yaml fails in CI:

model "grok-2-latest" not found in provider "xai"

The xai/grok-2-latest model is no longer published in the models.dev API — the xai provider now only exposes the grok-4.x family. The failure is masked locally for anyone whose ~/.cagent/models_dev.json cache still contains the old grok-2-* entries (the cache is valid for 24h before refresh).

Fix

Switch examples/grok.yaml to xai/grok-4.3, which is currently published, and adjust max_tokens to match the model's 30k output limit.

Validation

rm ~/.cagent/models_dev.json
go test ./pkg/config/...

Now passes.

The xai/grok-2-latest model is no longer listed in the models.dev API,
which causes TestParseExamples to fail in CI (or locally once the
~/.cagent/models_dev.json cache refreshes).

Switch the example to grok-4.3, which is currently published, and
adjust max_tokens to match the model's 30k output limit.
@dgageot dgageot requested a review from a team as a code owner May 21, 2026 07:41
Copy link
Copy Markdown

@docker-agent docker-agent left a comment

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

This PR fixes a CI test failure by updating examples/grok.yaml from the deprecated xai/grok-2-latest model to xai/grok-4.3, and adjusting max_tokens from 131072 to 30000 to match the new model's output limit.

Changes reviewed:

  • agents.root.model: xai/grok-2-latestxai/grok-4.3
  • models.grok-model.model: grok-2-latestgrok-4.3
  • models.grok-model.max_tokens: 131072 → 30000 ✅ (matches grok-4.3's 30k output limit)

Both references are updated consistently. No bugs found in the changed code.

@dgageot dgageot merged commit 87bbe80 into docker:main May 21, 2026
8 checks passed
@aheritier aheritier added area/providers For features/issues/fixes related to LLM providers (Bedrock, LiteLLM, Qwen, custom, etc.) kind/fix PR fixes a bug (maps to fix: commit prefix) labels May 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/providers For features/issues/fixes related to LLM providers (Bedrock, LiteLLM, Qwen, custom, etc.) kind/fix PR fixes a bug (maps to fix: commit prefix)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants