From 5fe7041c8ddac2fa5d25580592a2a2bd96800406 Mon Sep 17 00:00:00 2001 From: Deepanshu Date: Tue, 28 Jul 2026 22:14:40 -0400 Subject: [PATCH] refactor(logging): hoist a safely-hoistable function-body import to module top CorrelationContextFilter.filter()'s `import litellm` was a function-body import; verified it can move to module top without a circular-import failure (litellm/__init__.py already imports from litellm._logging before setting request_correlation_in_logs, but a bare `import litellm` only binds the already-in-sys.modules module object - the attribute itself isn't read until filter() actually runs, by which point litellm is fully initialized). --- litellm/_logging.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/litellm/_logging.py b/litellm/_logging.py index 0ae06242dc6..204a58a99b3 100644 --- a/litellm/_logging.py +++ b/litellm/_logging.py @@ -3,6 +3,8 @@ import contextvars import logging import os import sys + +import litellm from datetime import datetime from logging import Formatter from typing import Any, Final @@ -113,8 +115,6 @@ class CorrelationContextFilter(logging.Filter): """ def filter(self, record: logging.LogRecord) -> bool: - import litellm - if not litellm.request_correlation_in_logs: return True trace_id = trace_id_var.get()