fix(core): keep a v1 provider's baseURL when api is also set

This commit is contained in:
Dan Smolsky
2026-11-07 06:43:08 -05:00
Unverified
parent c72ce6cf82
commit 222f0d5dee
2 changed files with 6 additions and 1 deletions
+1 -1
View File
@@ -125,7 +125,7 @@ function migrateStandardProvider(info: ConfigProviderV1.Info) {
env: info.env,
package: info.npm ? packageSpecifier(info.npm) : undefined,
compatibility: setCacheKey === undefined ? undefined : { supportsPromptCacheKey: setCacheKey },
settings: info.api ? { ...options.settings, baseURL: info.api } : info.options ? options.settings : undefined,
settings: info.api ? { baseURL: info.api, ...options.settings } : info.options ? options.settings : undefined,
headers: info.options && options.headers,
body: info.options && options.body,
models:
@@ -641,6 +641,11 @@ describe("ConfigProviderPlugin.Plugin", () => {
models: { chat: {} },
},
},
{
name: "provider baseURL alongside api",
legacy: { api: "openai", options: { baseURL: "http://127.0.0.1:4000/v1" }, models: { chat: {} } },
native: { settings: { baseURL: "http://127.0.0.1:4000/v1" }, models: { chat: {} } },
},
{
name: "model package, limits, costs, and variant overrides",
legacy: {