-
Notifications
You must be signed in to change notification settings - Fork 1.1k
add independent fast-apply provider setup decoupled from the main provider #3020
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?
Conversation
|
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.
All reviewed, but I don't know the general process of translation, is it necessary for PR submitters to translate all by themselves? Or will the translation be done by the kilo official themselves?
I think this may duplicate #2813 |
OK,can you answer a few questions for me? I want to know how to correctly create a new LLM client in Kilo code. A while ago, after I updated, I noticed that the option to configure the LLM model for autocomplete disappeared. According to the contributor who deleted the code, Kilo currently does not actually use a mechanism for independent clients. The API directly points to the main API configuration, causing a bug that results in massive token consumption. Additionally, I want to know why each provider's API key uses a different name instead of adhering to a unified interface. This makes it difficult for us to build an LLM API client based on a non-currently enabled API configuration. thinks |
Probably, but its worth mentioning I'm not officially part of Kilo Code - just an interested constributor.
Here's an excellent sample PR the includes a bunch of best practices (like dynamic model fetching, good tests, etc) from @b3nw #2825
In that particular case, the code was such that it simply defaulted to the currently selected profile, which meant that people were using expensive, slow models for autocomplete if they didn't change the config.
Not quite. Kilo does have mechanisms for different clients, but only 1
Yes, this is an artifact of history and a fast growing/changing codebase. I have a proposal for what I believe to be a better solution (#2670) but it has not yet gained traction or review from the Kilo team. |
We can do it for you, it's mostly AI generated. |
Context
The current Fast Apply function is only applicable in two specific scenarios: when the current Provider is Kilo Code API / OpenRouter, or when Morph key is specified as the Fast Apply.
I hope to decouple the Fast Apply function's Provider from the current Provider. Regardless of what the current Provider is, it should allow independent selection of Kilo Code API / OpenRouter / Morph as the Fast Apply Provider.
Implementation
Main Changes
Screenshots
How to Test
Get in Touch
Wait