mirror of
https://github.com/BerriAI/litellm.git
synced 2026-09-19 00:01:29 +00:00
chore(interactions): keep the text param build inside the LIT002 budget
The rebase onto current staging picked up a tighter type-discipline ceiling, and the six mutable-collection builds this change added put LIT002 at 26723 against a limit of 26715. Wrap the single-entry fallback in a tuple, and mark the text param builds mutable-ok: litellm_completion_transformation reads both the param and its format field back with isinstance(..., dict), so a MappingProxyType there would drop the format silently. Request output is unchanged for every response_format shape the tests cover.
This commit is contained in:
parent
0967f284b0
commit
c5f8d1ea8b
1 changed files with 5 additions and 5 deletions
|
|
@ -129,7 +129,7 @@ class LiteLLMResponsesInteractionsConfig:
|
|||
schema=response_format,
|
||||
)
|
||||
|
||||
entries: Final = response_format if isinstance(response_format, list) else [response_format]
|
||||
entries: Final = response_format if isinstance(response_format, list) else (response_format,)
|
||||
text_entry: Final = next(
|
||||
(entry for entry in entries if isinstance(entry, Mapping) and entry.get("type") == "text"),
|
||||
None,
|
||||
|
|
@ -146,16 +146,16 @@ class LiteLLMResponsesInteractionsConfig:
|
|||
if mime_type is not None and mime_type != _JSON_MIME_TYPE:
|
||||
return None
|
||||
if isinstance(schema, Mapping) and schema:
|
||||
return {
|
||||
"format": {
|
||||
return { # mutable-ok: litellm_completion_transformation reads this back with isinstance(..., dict)
|
||||
"format": { # mutable-ok: and checks this field the same way
|
||||
"type": "json_schema",
|
||||
"name": "response_schema",
|
||||
"schema": dict(schema),
|
||||
"schema": dict(schema), # mutable-ok: shallow copy of the caller's top-level keys
|
||||
"strict": False,
|
||||
}
|
||||
}
|
||||
if mime_type == _JSON_MIME_TYPE:
|
||||
return {"format": {"type": "json_object"}}
|
||||
return {"format": {"type": "json_object"}} # mutable-ok: same isinstance(..., dict) round trip
|
||||
return None
|
||||
|
||||
@staticmethod
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue