scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-11-01 07:03:02 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 128

fix(anthropic): never name an empty role for another field's failure

Browse source
This commit is contained in:
mateo-berri 2026-10-30 23:35:39 -07:00
parent 769c1086d1
commit 301b519a67
2 changed files with 14 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
litellm/llms/anthropic/chat/transformation.py
View file

@ -195,7 +195,7 @@ def _tool_call_ids_failing(
def _non_string_roles(messages: Sequence[object]) -> Iterator[tuple[str, object]]:
roles: Final = ((index, message_field(message, "role")) for index, message in enumerate(messages))
return ((f"messages[{index}].role", role) for index, role in roles if is_present_non_string(role))
return ((f"messages[{index}].role", role) for index, role in roles if role and is_present_non_string(role))
def _assistant_tool_call_ids_failing(

13
tests/unit/llms/anthropic/chat/test_anthropic_chat_transformation.py
View file

@ -10169,6 +10169,19 @@ def test_chat_transform_request_never_blames_a_numeric_assistant_tool_call_id_fo
assert raised.value.message.startswith("'str' object has no attribute 'get'")
@pytest.mark.parametrize("empty_role", [[], 0, False])
def test_chat_transform_request_never_blames_an_empty_role_for_another_failure(local_model_cost_map, empty_role):
list_part_type_after_an_empty_role: Final = [
_ASK,
{"role": empty_role, "content": "ok"},
{"role": "user", "content": [{"type": ["text"], "text": "again"}]},
{"role": "system", "content": "Be terse."},
]
with pytest.raises(TypeError):
_chat_request(AnthropicConfig(), FLAGGED_CLAUDE, list_part_type_after_an_empty_role)
def test_chat_transform_request_names_no_field_for_a_failure_outside_roles_and_tool_call_ids(local_model_cost_map):
list_part_type: Final = [
_ASK,