scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-10-07 02:59:05 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 103

fix(logging): keep logging empty string system prompts

Browse source 
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
This commit is contained in:
Devin AI 2026-08-10 14:51:01 +00:00
parent 8bdd61bdea
commit 1356309ee9
2 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
litellm/litellm_core_utils/litellm_logging.py
View file

@ -4549,7 +4549,7 @@ class StandardLoggingPayloadSetup:
if kwargs is None:
return messages
system: Final = kwargs.get("system")
if not system or not isinstance(system, (str, list)):
if not isinstance(system, (str, list)) or (isinstance(system, list) and not system):
return messages
system_message: Final = {"role": "system", "content": system}
if messages is None:

6
tests/test_litellm/litellm_core_utils/test_litellm_logging.py
View file

@ -2261,16 +2261,18 @@ def test_append_system_prompt_messages_content_blocks():
)
assert result == [{"role": "system", "content": system_blocks}, *messages]
# already-prepended system blocks are not duplicated
result = StandardLoggingPayloadSetup.append_system_prompt_messages(
kwargs={"system": system_blocks}, messages=result
)
assert len(result) == 2
# empty system is not logged as an empty system turn
assert (
StandardLoggingPayloadSetup.append_system_prompt_messages(kwargs={"system": []}, messages=messages) == messages
)
assert StandardLoggingPayloadSetup.append_system_prompt_messages(kwargs={"system": ""}, messages=messages) == [
{"role": "system", "content": ""},
*messages,
]
@pytest.mark.asyncio