scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-09-14 23:21:35 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 102

fix(lint): suppress genuine LIT002 hits instead of padding the ceiling

Browse source 
My earlier rebase conflict resolution for type-discipline-budget.json's
LIT002 limit was too low, then overcorrected by padding it well above the
actual measured count. Root-caused instead: _independent_snapshot and the
PATCH-endpoint rollback path legitimately construct plain, mutable
request-payload/config dicts (matching this file's existing precedent for
the same shape), so suppress those four sites with `# mutable-ok:` rather
than reshaping code that must stay a plain dict by contract. Set the limit
to the exact current measured total; the small remaining gap vs upstream's
own committed ceiling is pre-existing drift in litellm_internal_staging
itself (its own tree already measures over its committed limit), not
attributable to this PR.
This commit is contained in:
Deepanshu 2026-08-27 19:11:00 -04:00
parent 2e0ec3c262
commit 71ec3ca6b3
3 changed files with 14 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
litellm/proxy/guardrails/guardrail_endpoints.py
View file

@ -1231,7 +1231,10 @@ async def patch_guardrail(
guardrail_id=guardrail_id,
guardrail_name=existing_guardrail.get("guardrail_name") or "",
litellm_params=LitellmParams(**existing_litellm_params),
guardrail_info=existing_guardrail.get("guardrail_info", {}),
guardrail_info=existing_guardrail.get(
"guardrail_info",
{}, # mutable-ok: Guardrail's own constructor takes a plain dict
),
),
prisma_client=prisma_client,
)

13
litellm/proxy/utils.py
View file

@ -476,9 +476,9 @@ def _independent_snapshot(
need to be genuinely independent, and those are plain, cleanly-copyable
structures.
"""
sanitized: Final = {
sanitized: Final = { # mutable-ok: same request-payload shape as data
key: (
{
{ # mutable-ok: same request-payload shape as data
inner_key: ("placeholder" if inner_key == "litellm_parent_otel_span" else inner_value)
for inner_key, inner_value in value.items()
}
@ -500,10 +500,15 @@ def _independent_snapshot(
and isinstance(original_value, dict)
and "litellm_parent_otel_span" in original_value
):
return {**copied_value, "litellm_parent_otel_span": original_value["litellm_parent_otel_span"]}
return { # mutable-ok: same request-payload shape as data
**copied_value,
"litellm_parent_otel_span": original_value["litellm_parent_otel_span"],
}
return copied_value
return {key: _copied_value(key, value) for key, value in sanitized.items()}
return { # mutable-ok: same request-payload shape as data
key: _copied_value(key, value) for key, value in sanitized.items()
}
def _prompt_block_text(block: object) -> str:

2
type-discipline-budget.json
View file

@ -3,7 +3,7 @@
"limit": 22727
},
"LIT002": {
"limit": 26900
"limit": 26873
},
"LIT003": {
"limit": 269