Fix openrouter custom base url Issue [#2181] #2188
Conversation
|
cobra91
changed the title
…obra91/kilocode into fix-openrouter-custom-base-url
cobra91
changed the title
cobra91
changed the title
|
#2385 it's not the same problem |
| provider === "openrouter" | ||
| ? apiConfiguration?.apiKey | ||
| : provider === "kilocode" | ||
| ? apiConfiguration?.kilocodeToken |
There was a problem hiding this comment.
Why was this change made? I don't think it is correct, because the Kilo Code provider doesn't have this endpoint (yet at least).
There was a problem hiding this comment.
yes want to reuse max code but i guess it's not good because kilo code provider don't need custom ! sorry
| getModels({ provider: "openrouter" }), | ||
| getModels({ | ||
| provider: "openrouter", | ||
| baseUrl: this.options.openRouterBaseUrl, // kilocode_change |
There was a problem hiding this comment.
Should the api key also be specified here?
There was a problem hiding this comment.
not for openrouter who don't need apikey to fetch model list => https://openrouter.ai/api/v1/models work on browser
There was a problem hiding this comment.
ok, but the same is true for getModelEndpoints isn't it?
There was a problem hiding this comment.
yes ! https://openrouter.ai/api/v1/models/deepseek/deepseek-r1/endpoints answer without apikey
…thub.com//issues/2181)
Context
Implementation
Screenshots
How to Test
Get in Touch