[codex] Add multi-provider chat switching - #103
Draft
Star-Star66 wants to merge 2 commits into
Draft
Conversation
Collaborator
|
👋 Thanks for the PR @Star-Star66! This is marked as draft — I'll review it when it's ready for merge. No action needed now. — Carbon Code |
Collaborator
Review SummaryDraft PR reviewed — no action needed yet. This PR adds multi-provider chat switching with /provider slash command, ChatProviderConfig, and runtime config reload. The scope and compatibility story look solid. Will perform a full review and merge when the draft is marked ready. Thanks @Star-Star66 for tackling the multi-provider feature! -- Carbon Code |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
apiKey/baseUrlconfiguration/provider [name]to list and switch providers without restarting the active TUI sessionWhy
Issue #95 item 3 asks for multiple provider configuration and in-conversation model/provider switching. Carbon Code previously stored only one chat connection, so changing providers required overwriting the current credentials and endpoint.
Compatibility
Existing single-provider DeepSeek configurations continue to resolve as the implicit
deepseekprovider.DEEPSEEK_API_KEYandDEEPSEEK_BASE_URLremain supported as explicit environment overrides.Validation
npm run verifyAddresses #95 item 3.