From 340ffd19f438567cd54171c6c4d58ace2ca90fea Mon Sep 17 00:00:00 2001 From: stevejaker Date: Tue, 10 Mar 2026 14:27:58 -0600 Subject: [PATCH] style: minor cleanup from Greptile review - Add noqa comment for unused model/is_async params (interface compatibility) - Filter empty strings in tool content flattening for consistency Co-Authored-By: Claude Opus 4.6 --- litellm/llms/snowflake/chat/transformation.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/litellm/llms/snowflake/chat/transformation.py b/litellm/llms/snowflake/chat/transformation.py index 0f166d7f85a..c4fc203a7f1 100644 --- a/litellm/llms/snowflake/chat/transformation.py +++ b/litellm/llms/snowflake/chat/transformation.py @@ -156,7 +156,7 @@ class SnowflakeConfig(SnowflakeBaseConfig, OpenAIGPTConfig): return f"{api_base}/cortex/inference:complete" - def _transform_messages( + def _transform_messages( # noqa: ARG002 - model, is_async unused; accepted for interface compatibility self, messages: List[AllMessageValues], model: str, is_async: bool = False ) -> List[AllMessageValues]: """ @@ -335,9 +335,12 @@ class SnowflakeConfig(SnowflakeBaseConfig, OpenAIGPTConfig): if content is None: content = "null" elif isinstance(content, list): - # Flatten OpenAI multipart tool content to a plain string + # Flatten OpenAI multipart tool content to a plain string. + # Filter out empty strings from non-text parts (e.g., images). content = " ".join( - part.get("text", "") for part in content if isinstance(part, dict) + part.get("text", "") + for part in content + if isinstance(part, dict) and part.get("text") ) elif not isinstance(content, str): content = str(content)