feat(tui): add backup models array#30852
Open
jjdubski wants to merge 1 commit into
Open
Conversation
Contributor
|
The following comment was made by an LLM, it may be inaccurate: Potential Duplicate PRs FoundI found two related PRs that address similar functionality:
These PRs may be duplicates or overlapping implementations of the backup/fallback model feature. You should review PRs #27939 and #26292 to understand if:
|
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.
Issue for this PR
Closes #25150
Closes #7602
Closes #20100
Closes #16868
Type of change
What does this PR do?
This improves the user experience, especially for those who like to use the free models. It sucks when the session breaks when the rate limit is hit, and it continues to retry or shows "Retrying in 4h 31m...". This way it will continue on error, switching to the backup when the provider is down, or usage is hit.
Format for backup model is an array with "model/provider", same as model. Example:
How did you verify your code works?
bun testinside the packages/opencode directorybun typecheckScreenshots / recordings
How it looks in my commit-storyteller.md agent file in .config/opencode/agents directory:
Checklist
Any feedback would be greatly appreciated!