-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Enhancement: Adds OpenRouter Base URL for OpenRouter embedder provider #10342
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
base: main
Are you sure you want to change the base?
Enhancement: Adds OpenRouter Base URL for OpenRouter embedder provider #10342
Conversation
- Adds UI with standard text input within CodeIndexPopover - Modifies interface accordingly - Adds UI locales for english (other languages to come) - Updates the config-manager with boiler plate - Update openrouter embedder code to use baseURL if available, otherwise use default.
…y that no longer exists.
All issues have been resolved. The PR is ready for merge.
Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues. |
Review complete. Found 1 issue that needs to be addressed before merging.
Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues. |
Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com>
… be seperate from Ollama. - Mimic the way codebaseIndexOpenRouterSpecificProvider is done, but for the base url setting
…ved) - Remove unused variable from config manager
…tings storage, etc
webview-ui/src/components/chat/__tests__/CodeIndexPopover.auto-populate.spec.tsx
Outdated
Show resolved
Hide resolved
|
@roomote are you still seeing the following 2 issues
|
Fixed the reported issue. Added |
Review complete. No issues found. The PR is ready for merge. All previously flagged issues have been resolved:
Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues. |
Related GitHub Issue
Closes: #9053
Description
Allow users to specify a custom OpenRouter API URL when selecting OpenRouter as the embedder provider in Code Index settings.
I attempted to follow the way oLlama did it's baseUrl config within the code indexing popover.
My current PR shares the same baseUrlsetting (I realised that this is infact not a good idea, and separated the settings.codebaseIndexEmbedderBaseUrl) as oLlama, so they both share the same state. The reason why I did it this way, was the way the setting was namedcodebaseIndexEmbedderBaseUrl, it gave me the impression that this should be shared state, or at least that was initial intention. I'm not sure if this is desired, but I'm betting most users aren't really switching between providers for code indexing like they might be for prompting. Willing to hear people out if they feel differently. You can see a demo of this below.Screen.Recording.2025-12-25.at.7.10.12.pm.mov
High level code changes:
TODO:
Test Procedure
Tests TBD
Pre-Submission Checklist
Screenshots / Videos
Before:

After:

Documentation Updates
Get in Touch
Discord:
taltas1Important
Adds support for configuring a custom OpenRouter Base URL for the OpenRouter embedder provider, with UI updates and tests.
codebaseIndexOpenRouterEmbedderBaseUrltocodebase-index.tsfor custom OpenRouter API URL configuration.ClineProvider.tsto handlecodebaseIndexOpenRouterEmbedderBaseUrlin state management.webview-ui/src/components/chat/CodeIndexPopover.tsxto include a text input for the OpenRouter Base URL.config-manager.spec.tsandorchestrator.spec.tsto validate OpenRouter Base URL handling and configuration changes.CodeIndexPopover.auto-populate.spec.tsxfor the new input field.webview-ui/src/i18n/localesto include labels and placeholders for the OpenRouter Base URL in multiple languages.This description was created by
for 316a77e. You can customize this summary. It will automatically update as commits are pushed.