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

fix(transformation): prevent 'reasoning_effort' promotion for non-Claude models

Browse source
This commit is contained in:
Timothy Oluwatobi Ojebiyi 2026-05-16 08:53:53 +01:00
parent 21be2785ae
commit ee237054ed
2 changed files with 18 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
litellm/llms/github_copilot/chat/transformation.py
View file

@ -174,7 +174,10 @@ class GithubCopilotConfig(OpenAIConfig):
else:
reasoning_effort = "minimal"
optional_params["reasoning_effort"] = reasoning_effort
elif non_default_params.get("reasoning_effort") is not None:
elif (
"claude" in model.lower()
and non_default_params.get("reasoning_effort") is not None
):
optional_params["reasoning_effort"] = non_default_params.pop(
"reasoning_effort"
)

14
tests/test_litellm/llms/github_copilot/test_github_copilot_transformation.py
View file

@ -563,6 +563,20 @@ def test_map_openai_params_thinking_not_popped_for_non_claude_model():
assert "reasoning_effort" not in optional_params
def test_map_openai_params_reasoning_effort_not_promoted_for_non_claude_model():
"""For non-Claude models, ``reasoning_effort`` must not be force-promoted
to ``optional_params``; the parent's supported-params filtering must run."""
config = GithubCopilotConfig()
optional_params = config.map_openai_params(
non_default_params={"reasoning_effort": "high"},
optional_params={},
model="gpt-4",
drop_params=True,
)
assert "reasoning_effort" not in optional_params
def test_copilot_vision_request_header_with_image():
"""Test that Copilot-Vision-Request header is added when messages contain images"""
config = GithubCopilotConfig()