-
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.
Review complete. The constructor parameter order was fixed, but
Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues. |
| </> | ||
| )} | ||
|
|
||
| {currentSettings.codebaseIndexEmbedderProvider === "openrouter" && ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new UI block for OpenRouter base URL reuses the codebaseIndexEmbedderBaseUrl state, which is also used by Ollama. Ensure that sharing this field between providers is intentional to prevent accidental overwrites when switching providers.
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>
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 (
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:
taltas1