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

remove dead allowed_tools field from BlockedToolsResult

Browse source 
The field was computed but never read after construction — _check_tool_calls
only uses blocked.explanation. The local variable remains for the
all-tools-allowed comparison.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Krrish Dholakia 2026-05-13 14:23:17 -07:00
parent b9a5ca6efb
commit 9f7a4fe1a5
1 changed files with 0 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
litellm/integrations/rubrik.py
View file

@ -43,7 +43,6 @@ _WEBHOOK_PATH_LOGGING_BATCH = "/v1/litellm/batch"
class BlockedToolsResult:
"""Returned by _extract_blocked_tools when at least one tool was blocked."""
allowed_tools: list
explanation: str
@ -446,6 +445,5 @@ class RubrikLogger(CustomGuardrail, CustomBatchLogger):
explanation = blocking_explanation or "Tool call blocked by policy."
return BlockedToolsResult(
allowed_tools=allowed_tools,
explanation=f"\n\n{explanation}",
)