From 397b5c8bd621338dbb49d8571d4b4c5562a560cc Mon Sep 17 00:00:00 2001 From: Emerson Gomes Date: Wed, 12 Aug 2026 12:50:48 -0500 Subject: [PATCH] style(router): explain mutable budget queue --- litellm/router_strategy/budget_limiter.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/litellm/router_strategy/budget_limiter.py b/litellm/router_strategy/budget_limiter.py index c52040c8e3d..e14ecb58b68 100644 --- a/litellm/router_strategy/budget_limiter.py +++ b/litellm/router_strategy/budget_limiter.py @@ -532,14 +532,18 @@ class RouterBudgetLimiting(CustomLogger): Only runs if Redis is initialized """ - increment_operations_to_flush: list[RedisPipelineIncrementOperation] = [] + increment_operations_to_flush: list[RedisPipelineIncrementOperation] = ( # mutable-ok: Redis pipeline contract requires a list batch + [] # mutable-ok: Redis pipeline batches are lists + ) try: if not self.dual_cache.redis_cache: return # Redis is not initialized async with self._get_redis_increment_queue_lock(): increment_operations_to_flush = self.redis_increment_operation_queue - self.redis_increment_operation_queue = [] + self.redis_increment_operation_queue = ( + [] # mutable-ok: the emptied queue must remain appendable by logging callbacks + ) verbose_router_logger.debug( "Pushing Redis Increment Pipeline for queue: %s",