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

fix(anthropic): explicit supports_reasoning=False wins over per-level flags

Browse source 
Greptile P1: the implicit-True branch bypassed the operator's explicit
supports_reasoning: false escape hatch when per-level flags were present
or inherited through the bare-twin lookup. Return () first on explicit
False, then apply the per-level implication only when the flag is unset.

Also drops a test comment that restated the test name (P2).

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Tin Chi Lo 2026-08-27 20:58:24 -07:00
parent 5e651d8052
commit 460361923c
2 changed files with 19 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
litellm/router_utils/reasoning_effort_capability.py
View file

@ -149,17 +149,19 @@ def resolve_supported_reasoning_efforts(
on. deployment_is_mapped is that provenance, and an operator who wants either answer for an
off-map deployment gets it by setting supports_reasoning explicitly.
If no explicit supports_reasoning flag is set but at least one per-level flag (e.g.
supports_minimal_reasoning_effort) is present, treat supports_reasoning as implicitly True,
since the per-level flags are evidence the model supports reasoning.
If supports_reasoning is unset but at least one per-level flag (e.g.
supports_minimal_reasoning_effort) is present, treat it as implicitly True, since the
per-level flags are evidence the model supports reasoning. An explicit False always wins:
it is the operator's escape hatch and must not be overridden by inherited per-level flags.
"""
supports_reasoning: Final = model_info.get("supports_reasoning")
if supports_reasoning is False:
return ()
flags: Final = _declared_effort_flags(model_info)
has_per_level_flag: Final = any(value is not None for value in flags.values())
if supports_reasoning is not True:
if not has_per_level_flag:
return () if supports_reasoning is False or deployment_is_mapped else None
if supports_reasoning is not True and not has_per_level_flag:
return () if deployment_is_mapped else None
declared: Final = declared_reasoning_efforts(model_info)
if declared is not None:

13
tests/test_litellm/router_utils/test_reasoning_effort_capability.py
View file

@ -86,9 +86,6 @@ class TestResolveSupportedReasoningEfforts:
assert resolved == ("none", "minimal", "low", "medium", "high")
def test_per_level_flag_without_supports_reasoning_treats_as_implicit_true(self):
# A model with only a per-level flag (e.g. supports_minimal_reasoning_effort) but no
# explicit supports_reasoning should be treated as reasoning-capable, since the per-level
# flag is evidence of reasoning support.
resolved = resolve_supported_reasoning_efforts(
{
"supports_minimal_reasoning_effort": True,
@ -97,6 +94,16 @@ class TestResolveSupportedReasoningEfforts:
)
assert resolved == ("none", "minimal", "low", "medium", "high")
def test_explicit_supports_reasoning_false_wins_over_per_level_flags(self):
resolved = resolve_supported_reasoning_efforts(
{
"supports_reasoning": False,
"supports_minimal_reasoning_effort": True,
},
deployment_is_mapped=True,
)
assert resolved == ()
class TestBareModelNameFallback:
def test_a_prefixed_entry_inherits_the_flags_of_its_unprefixed_twin(self):