feat(xai): GUI opt-in switch for the Grok Responses lane (doc 130) - #2266
Conversation
|
✅ Deterministic PR hygiene checks passed. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (35)
📝 WalkthroughWalkthroughAdds an xAI Responses opt-in control for Grok 4.5 and 4.6. The backend manages both model adapters atomically and reports enabled, disabled, or mixed state. The GUI renders and updates the control. Tests and documentation cover the behavior. ChangesxAI Responses opt-in
Estimated code review effort: 4 (Complex) | ~45 minutes Sequence Diagram(s)sequenceDiagram
participant Dashboard
participant ProviderAuthPanel
participant useProvidersCrud
participant providerRoutes
participant ProviderConfig
Dashboard->>ProviderAuthPanel: Render xAI opt-in state
ProviderAuthPanel->>useProvidersCrud: Update xaiResponsesOptIn
useProvidersCrud->>providerRoutes: PATCH provider update
providerRoutes->>ProviderConfig: Set or clear Grok 4.5 and 4.6 adapters
ProviderConfig-->>providerRoutes: Persisted configuration
providerRoutes-->>useProvidersCrud: xaiResponsesOptInState
useProvidersCrud-->>ProviderAuthPanel: Effective state
ProviderAuthPanel-->>Dashboard: Updated switch state
Suggested reviewers: ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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 |
Summary
The doc-130 unit: the xAI Responses implementation stays reachable through an explicit GUI opt-in switch (like the x_search opt-in), now that #2255 made Chat the OAuth default.
xaiResponsesOptIn: booleansets/clears BOTH grok-4.6 and grok-4.5modelAdaptersentries in one config transaction, preserving unrelated overrides; READxaiResponsesOptInState: true | false | "mixed"(partial pre-existing state reads mixed; the first boolean write normalizes). xai-only; other providers rejected.Verification
Checklist
Summary by CodeRabbit