fix(settings): custom providers UI, URL detection, and fallback model - #414
Closed
sebuh-infsol wants to merge 2 commits into
Closed
fix(settings): custom providers UI, URL detection, and fallback model#414sebuh-infsol wants to merge 2 commits into
sebuh-infsol wants to merge 2 commits into
Conversation
…odel - Render a persistent endpoints table for custom_providers with apply/remove. - Show active primary and manifest URLs when missing from custom_providers, with apply and add-to-list actions. - Resolve custom base URL from manifest, custom_providers (case-insensitive name match), or top-level base_url; align configured status with that. - Add optional fallback model fields and persist fallback_model on save. - Merge manifest saves into existing model config instead of replacing it. - Add helpers for deduplication, manifest URL read, and named row persist. Co-authored-by: Cursor <cursoragent@cursor.com>
- Add custom provider form: title (e.g. Model.Host), optional provider id, base URL, optional per-row api_key; slugify id when omitted. - Endpoints table adds Title column; suggest title from model + URL for quick-save and active/manifest rows. - Manifest base URL state reads providers.manifest only; primary URL no longer appears in that control. Clearer placeholders and save validation. - Manifest CUSTOM_API_KEY row: optional copy, save empty clears env key. - Helpers: suggestCustomProviderTitle, slugifyCustomProviderId; persist supports title and yamlApiKey options.
Author
|
Additional commits on this branch:
|
Owner
|
Closing as superseded by #432. The validated fix was folded into the consolidated batch branch |
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
2 participants
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
This updates Settings → Model & Provider (
src/routes/settings/index.tsx) so custom and OpenAI-compatible setups are visible, editable, and harder to confuse with unrelated providers. It addresses the behaviour described in #413.Custom Providers
custom_providerswith Apply and Remove.base_urlor manifest URL is in use but not yet stored undercustom_providers, with Apply / Add to list.CUSTOM_API_KEY, manifest/custom blocks,custom_providers(case-insensitive name match), or top-levelbase_url.base_url) show as configured when they are actually in use.Model & Provider
fallback_model.modelobject instead of replacing it with only{ provider: 'manifest' }.Testing
Fixes #413.
Made with Cursor