From 5f5dd82b3edd53cf472925c3f979335a934ff6fa Mon Sep 17 00:00:00 2001 From: Maximilian Rink Date: Sat, 10 Oct 2026 11:23:02 +0200 Subject: [PATCH] style: satisfy cache scope type gates --- litellm/caching/caching.py | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/litellm/caching/caching.py b/litellm/caching/caching.py index 8635bfe2d9a..c7e21f3e04f 100644 --- a/litellm/caching/caching.py +++ b/litellm/caching/caching.py @@ -392,17 +392,20 @@ class Cache: def as_mapping(value: object) -> Mapping[str, object]: return value if isinstance(value, Mapping) else {} - litellm_params = as_mapping(kwargs.get("litellm_params")) - metadata_sources: list[Mapping[str, object]] = [ + litellm_params: Final = as_mapping(kwargs.get("litellm_params")) + metadata_sources: Final = tuple( # comprehension-ok: flatten trusted metadata containers as_mapping(source.get(key)) for source in (kwargs, litellm_params) for key in ("metadata", "litellm_metadata") - ] + ) - scope_values: list[tuple[str, object | None]] = [] - for field in self._PROXY_CACHE_SCOPE_FIELDS: - value = next((source[field] for source in metadata_sources if source.get(field) is not None), None) - scope_values.append((field, value)) + scope_values: Final = tuple( + ( + field, + next((source[field] for source in metadata_sources if source.get(field) is not None), None), + ) + for field in self._PROXY_CACHE_SCOPE_FIELDS + ) return "".join(f"{field}: {value}" for field, value in scope_values if value is not None) def get_cache_key(self, **kwargs: object) -> str: