scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-09-10 22:41:41 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 129

Merge pull request #24282 from kimimgo/fix/openrouter-prefix-strip-24234

Browse source 
fix: strip 'openrouter/' prefix from model names (#24234)
This commit is contained in:
Cesar Garcia 2026-03-21 21:39:33 -03:00 • committed by GitHub
commit 6cd3896fa6
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 24 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
litellm/litellm_core_utils/get_llm_provider_logic.py
View file

@ -158,12 +158,15 @@ def get_llm_provider( # noqa: PLR0915
): # handle scenario where model="azure/*" and custom_llm_provider="azure"
model = custom_llm_provider + "/" + model
# Native OpenRouter models have IDs like "openrouter/free" where the
# Native OpenRouter models have IDs like "openrouter/auto" where the
# "openrouter/" prefix is part of the actual model name on the API.
# When called from a bridge (e.g. anthropic_messages adapter),
# custom_llm_provider is already resolved, so return early to prevent
# the provider-list stripping below from removing the prefix.
if custom_llm_provider == "openrouter" and model.startswith("openrouter/"):
# Only preserve prefix for single-segment native models (no extra '/'),
# not for provider-routed models like "openrouter/anthropic/claude-3.5-sonnet".
if (
custom_llm_provider == "openrouter"
and model.startswith("openrouter/")
and "/" not in model[len("openrouter/") :]
):
return model, custom_llm_provider, dynamic_api_key, api_base
if api_key and api_key.startswith("os.environ/"):

16
tests/local_testing/test_get_llm_provider.py
View file

@ -420,3 +420,19 @@ def test_get_llm_provider_use_proxy_arg_true_with_direct_args():
assert provider == "litellm_proxy"
assert key == arg_api_key # Should use the argument key
assert base == arg_api_base # Should use the argument base
def test_openrouter_model_prefix_stripped():
"""
GH#24234: OpenRouter models like "openrouter/anthropic/claude-3.5-sonnet"
should have the "openrouter/" prefix stripped before being sent to the API.
The API expects "anthropic/claude-3.5-sonnet", not the full prefixed name.
"""
model, provider, _, _ = litellm.get_llm_provider(
model="openrouter/anthropic/claude-3.5-sonnet"
)
assert provider == "openrouter"
assert model == "anthropic/claude-3.5-sonnet", (
f"Expected 'anthropic/claude-3.5-sonnet' but got '{model}'. "
"The 'openrouter/' prefix was not stripped."
)