From 9f7a4fe1a58ef9fcec46ff4151a0a1822081a5c0 Mon Sep 17 00:00:00 2001 From: Krrish Dholakia Date: Wed, 13 May 2026 14:23:17 -0700 Subject: [PATCH] remove dead allowed_tools field from BlockedToolsResult MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 --- litellm/integrations/rubrik.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/litellm/integrations/rubrik.py b/litellm/integrations/rubrik.py index a988c0b3604..1dae40474d7 100644 --- a/litellm/integrations/rubrik.py +++ b/litellm/integrations/rubrik.py @@ -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}", )