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

chore(reasoning): remove narrative comments flagged in review

Browse source
This commit is contained in:
jibanez-staticduo 2026-09-30 16:01:09 +02:00
parent 8897879028
commit e5098f6080
No known key found for this signature in database
2 changed files with 0 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
litellm/litellm_core_utils/get_litellm_params.py
View file

@ -32,9 +32,6 @@ AWS_CREDENTIAL_KWARGS_KEYS: Final = frozenset(
PROVIDER_AFFINITY_HEADER_KWARG_KEY: Final = "provider_affinity_header"
# Internal reasoning-history options that `completion()` forwards from its own
# kwargs into `get_litellm_params`, which are otherwise invisible to it because
# that call site passes explicit named arguments rather than `**kwargs`.
REASONING_TRANSPORT_KWARGS_KEYS: Final = frozenset({"forward_reasoning_content", "reasoning_content_field"})
# Pre-define optional kwargs keys as frozenset for O(1) lookups

2
tests/test_litellm/proxy/management_endpoints/test_model_management_endpoints.py
View file

@ -1425,8 +1425,6 @@ class TestUpdateModel:
mock_prisma.db.litellm_proxymodeltable.update.assert_awaited_once()
mock_clear_cache.assert_awaited_once_with()
stored = json.loads(mock_prisma.db.litellm_proxymodeltable.update.call_args.kwargs["data"]["litellm_params"])
# The endpoint encrypts every incoming litellm_param, so the stored reasoning field is
# read back through the real decrypt path instead of a stubbed encryption helper.
assert decrypt_value_helper(stored["reasoning_content_field"], key="reasoning_content_field") == (reasoning_field or "reasoning")
if reasoning_field is None:
assert stored["reasoning_content_field"] == existing_row.litellm_params["reasoning_content_field"]