-
Notifications
You must be signed in to change notification settings - Fork 3.8k
chore: add support for grok 4.1 fast #8831
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 issues found across 3 files
Prompt for AI agents (all 2 issues)
Understand the root cause of the following 2 issues and fix them.
<file name="packages/llm-info/src/providers/xAI.ts">
<violation number="1" location="packages/llm-info/src/providers/xAI.ts:58">
`Grok 4.1 Fast Non Reasoning` reuses the reasoning model id, so consumers will never send requests to the non‑reasoning backend. The `model` string in this block should include `-non-reasoning`.</violation>
<violation number="2" location="packages/llm-info/src/providers/xAI.ts:72">
This addition duplicates the existing `grok-4` model entry defined a few lines above, creating two identical configurations for the same model id. Remove the new block or consolidate so each model id appears only once.</violation>
</file>
Reply to cubic to teach it or ask questions. Re-run a review with @cubic-dev-ai review this PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues found across 3 files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comments, also can we
- add to the add models form in GUI
- add it to recommended agent models i.e. make sure it gets multi edit tool
- add comments about how the model was tested e.g. does it do well in agent mode? both CLI and extensions?
- make sure it's added to auto tool support
- update the docs
| maxCompletionTokens: 4096, | ||
| description: | ||
| "Latest Grok model with improved reasoning and conversational abilities.", | ||
| "Latest Grok model series with improved reasoning and conversational abilities.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's a specific model
| "Latest Grok model series with improved reasoning and conversational abilities.", | |
| "Latest Grok model with improved reasoning and conversational abilities.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we skip to grok 4.1 for recommended?
Description
support for new grok 4.1
resolves CON-4961
AI Code Review
@continue-reviewChecklist
Screen recording or screenshot
[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]
Tests
[ What tests were added or updated to ensure the changes work as expected? ]
Summary by cubic
Add support for xAI Grok 4.1 Fast models and update docs to recommend Grok‑4 for chat, resolving CON‑4961. Also adds Grok 4.1 Fast to CometAPI and increases Grok 4 (07‑09) context to 256k.
Written for commit 698fa69. Summary will update automatically on new commits.