-
Notifications
You must be signed in to change notification settings - Fork 3.4k
fix: add explicit fallback model and prevent direct opencode provider calls #4653
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
Merged
Conversation
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
Contributor
no1wudi
commented
Nov 23, 2025
- Add fallbackModel parameter to getSmallModel() to provide explicit fallback
- Check opencode provider availability before using gpt-5-nano model
- Update callers in prompt.ts and summary.ts to use current model as fallback
- Prevent errors when opencode provider is disabled
Contributor
Author
|
fix title gen issue if opencode provider disabled |
… calls - Add fallbackModel parameter to getSmallModel() to provide explicit fallback - Check opencode provider availability before using gpt-5-nano model - Update callers in prompt.ts and summary.ts to use current model as fallback - Prevent errors when opencode provider is disabled
Collaborator
|
good catch |
shuv1337
pushed a commit
to Latitudes-Dev/shuvcode
that referenced
this pull request
Nov 24, 2025
…r calls (sst#4653) Co-authored-by: Aiden Cline <[email protected]>
shuv1337
added a commit
to Latitudes-Dev/shuvcode
that referenced
this pull request
Nov 24, 2025
* ci: fix action * feature: optional selectedListItemText element in themes and luminance-based fallback to solve 4369 (sst#4572) Co-authored-by: knanao <[email protected]> Co-authored-by: knanao <[email protected]> * ignore: update download stats 2025-11-23 * ignore: update @solidjs/start dependency and fix console redirect handling Updates the @solidjs/start dependency to latest version and removes deprecated getResponseHeaders usage from auth callback. Also adds error handling for workspace ID lookup to prevent redirect failures. * Update Nix flake.lock and hashes * tweak: fix bool * sync * ci: stuff * Update Nix flake.lock and hashes * ci: stuff * Update Nix flake.lock and hashes * fix bash tool wsl * ci: ignore * release: v1.0.106 * Update Nix flake.lock and hashes * ci: ignore * token * ci: secret * chore: format code * ci: ignore * ci: ignore * ci: ignore * ci: remove log * tweak: slight improvements to title gen * wip: zen * wip: zen * chore: format code * fix: add explicit fallback model and prevent direct opencode provider calls (sst#4653) Co-authored-by: Aiden Cline <[email protected]> * nix: bundle js dist with bun and patch tree-sitter wasm paths (sst#4644) Co-authored-by: Aiden Cline <[email protected]> Co-authored-by: Github Action <[email protected]> * chore: format code * fix: fatal: undefined is not an object (evaluating 'color.buffer') * bump copilot plugin, give better error message for copilot (sst#4678) * release: v1.0.107 * tweak: modified files sidebar * fix: persist light vs dark mode * chore: format code * tweak: display MCPs in alphabetic order in the sidebar. (sst#4680) Co-authored-by: Github Action <[email protected]> * TUI: fix: add null check for user.time in duration calculation (sst#4679) * ignore: update download stats 2025-11-24 * docs: add Ollama Cloud provider setup to providers.mdx (sst#4693) * non-corpo loading spinner * chore: format code * release: v1.0.108 * textarea highlight cursor color * chore: format code * wip: zen * ci: nix hash * add autoupdate: notify * zen: add opus 4.5 * wip(share): more styling * wip(share): more styling * wip(share): more styling * wip(share): enterprise favicon * fix(console): favicon issues * chore: format code * fix(console): remove extra favicon * adjust bundled provider logic, fix tree shaking stuff (sst#4708) * wip: zen * zen: display reasoning tokens * fix(desktop): layout * Add --refresh flag to models command (sst#4707) * zen: byok for gemini * remove hardcoded openrouter provider * release: v1.0.109 * chore: format code * Update Nix flake.lock and hashes --------- Co-authored-by: Aiden Cline <[email protected]> Co-authored-by: Ariane Emory <[email protected]> Co-authored-by: knanao <[email protected]> Co-authored-by: knanao <[email protected]> Co-authored-by: GitHub Action <[email protected]> Co-authored-by: Dax Raad <[email protected]> Co-authored-by: opencode <[email protected]> Co-authored-by: Frank <[email protected]> Co-authored-by: Huang Qi <[email protected]> Co-authored-by: Albert O'Shea <[email protected]> Co-authored-by: Aiden Cline <[email protected]> Co-authored-by: Carli Samuele <[email protected]> Co-authored-by: Frankie Seabrook <[email protected]> Co-authored-by: Sebastian Herrlinger <[email protected]> Co-authored-by: Adam <[email protected]> Co-authored-by: Adam Hosker <[email protected]>
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.