-
Notifications
You must be signed in to change notification settings - Fork 188
feat: added xai provider in ui and docs #1236
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
Conversation
|
Caution Review failedThe pull request is closed. 📝 WalkthroughSummary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings. WalkthroughAdds the xAI (Grok) provider across documentation, OpenAPI schema, and UI constants: navigation entry and provider docs, Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
📜 Recent review detailsConfiguration used: Organization UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (8)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
🧪 Test Suite AvailableThis PR can be tested by a repository admin. |
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.
Actionable comments posted: 1
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (7)
docs/docs.jsondocs/openapi/schemas/inference/common.yamldocs/providers/supported-providers/overview.mdxdocs/providers/supported-providers/xai.mdxui/lib/constants/config.tsui/lib/constants/icons.tsxui/lib/constants/logs.ts
👮 Files not reviewed due to content moderation or server errors (1)
- ui/lib/constants/icons.tsx
🧰 Additional context used
📓 Path-based instructions (1)
**
⚙️ CodeRabbit configuration file
always check the stack if there is one for the current PR. do not give localized reviews for the PR, always see all changes in the light of the whole stack of PRs (if there is a stack, if there is no stack you can continue to make localized suggestions/reviews)
Files:
docs/providers/supported-providers/overview.mdxdocs/docs.jsondocs/providers/supported-providers/xai.mdxui/lib/constants/icons.tsxui/lib/constants/config.tsdocs/openapi/schemas/inference/common.yamlui/lib/constants/logs.ts
🧠 Learnings (2)
📚 Learning: 2025-12-30T05:37:48.365Z
Learnt from: Pratham-Mishra04
Repo: maximhq/bifrost PR: 1180
File: docs/features/mcp/connecting-to-servers.mdx:452-458
Timestamp: 2025-12-30T05:37:48.365Z
Learning: When reviewing documentation PRs in a Graphite-managed stack, first check related or previous PRs in the stack for feature implementations before flagging documentation as incorrect or unsupported. Documentation MDX files often reference features implemented in earlier stack PRs; verify that the documented behavior exists in earlier changes and that the docs accurately reflect the implemented state before requesting edits.
Applied to files:
docs/providers/supported-providers/overview.mdxdocs/providers/supported-providers/xai.mdx
📚 Learning: 2025-12-31T09:23:15.709Z
Learnt from: Pratham-Mishra04
Repo: maximhq/bifrost PR: 1180
File: docs/openapi/schemas/integrations/bedrock/batch.yaml:81-81
Timestamp: 2025-12-31T09:23:15.709Z
Learning: In OpenAPI schemas, keep definitions modular by placing root-level schemas in separate YAML files under docs/openapi/schemas. Use local references in the form $ref: '#/SchemaName' (do not use $ref: '#/components/schemas/SchemaName'). These files are resolved by the bundler script at docs/openapi/bundle.py. Apply this convention to all YAML files under docs/openapi/schemas (e.g., docs/openapi/schemas/**/*.yaml).
Applied to files:
docs/openapi/schemas/inference/common.yaml
🔇 Additional comments (9)
docs/providers/supported-providers/overview.mdx (1)
38-38: LGTM!The xAI provider row is correctly added to the support matrix following the established pattern. The support indicators accurately reflect a text/chat-focused provider with OpenAI-compatible operations.
docs/openapi/schemas/inference/common.yaml (1)
25-25: LGTM!The
xaienum value is correctly added to theModelProviderdefinition, expanding the set of allowed providers. The placement at the end is consistent with the existing non-alphabetical ordering.docs/docs.json (1)
110-111: LGTM!The xAI documentation page is correctly added to the navigation structure. The JSON remains valid and the placement maintains the existing organization pattern.
ui/lib/constants/logs.ts (2)
21-21: LGTM!The
xaiprovider is correctly added toKnownProvidersNames, which automatically extends the derivedProviderNametype throughout the UI.
65-65: LGTM!The provider label mapping is correctly added with proper brand capitalization ("xAI"), maintaining consistency with the
ProviderLabelsrecord type.ui/lib/constants/config.ts (2)
51-51: LGTM!The xAI model placeholder provides clear examples that align with the models mentioned in the PR description, helping users configure the provider correctly.
73-73: LGTM!The API key requirement is correctly set to
truefor xAI, consistent with other external provider services.docs/providers/supported-providers/xai.mdx (2)
91-134: Remaining sections are clear and well-documented.The Responses API, Text Completions, List Models, and Unsupported Features sections provide appropriate detail and clarity. The unsupported features table effectively communicates limitations with clear reasoning.
1-33: Documentation is accurate and well-aligned with implementation.The front matter, overview, and supported operations table are accurate. All claimed endpoints are confirmed:
- ✅
/v1/responses— Responses API (supported and tested)- ✅
/v1/chat/completions— Chat Completions (supported and tested)- ✅
/v1/completions— Text Completions (supported and tested)- ✅
/v1/models— List Models (supported and tested)- ✅ Vision support — Confirmed in tests (image URLs and base64)
- ✅ Reasoning support — Confirmed in tests
- ✅ Tool calling — Confirmed in tests
- ✅ Embeddings marked unsupported — Correctly matches implementation
All unsupported operations (Speech, Transcription, Batch, Files) are accurately marked and properly explained.
2a4deb0 to
c2a0031
Compare
c2a0031 to
af35195
Compare
Merge activity
|

Add xAI Provider Support
Added support for the xAI provider, which powers the Grok family of models. This integration enables users to access Grok models through Bifrost with full OpenAI compatibility.
Changes
Type of change
Affected areas
How to test
Breaking changes
Related issues
Implements provider support requested in issue tracker
Checklist