fix: preserve llm_base_url when saving MCP server config (#13225)

This commit is contained in:
Chris Bagwell
2026-03-05 19:39:58 -06:00
committed by GitHub
parent 6c394cc415
commit 41d8bd28e9
2 changed files with 70 additions and 3 deletions

View File

@@ -141,9 +141,12 @@ async def store_llm_settings(
settings.llm_api_key = existing_settings.llm_api_key
if settings.llm_model is None:
settings.llm_model = existing_settings.llm_model
# if llm_base_url is missing or empty, try to determine appropriate URL
# if llm_base_url is missing or empty, try to preserve existing or determine appropriate URL
if not settings.llm_base_url:
if is_openhands_model(settings.llm_model):
if settings.llm_base_url is None and existing_settings.llm_base_url:
# Not provided at all (e.g. MCP config save) - preserve existing
settings.llm_base_url = existing_settings.llm_base_url
elif is_openhands_model(settings.llm_model):
# OpenHands models use the LiteLLM proxy
settings.llm_base_url = LITE_LLM_API_URL
elif settings.llm_model: