Assorted small fixes in the replace-litellm branch (#1082)

* Make condition more pythonic

* Silence warning in the copilot provider

* Remove unused variable

Author: jhrozek

Diff for: src/codegate/pipeline/base.py

@@ -231,7 +231,7 @@ def get_last_user_message_block(
                 user_messages.append(txt)
                 last_idx = idx
 
-        if user_messages == []:
+        if not user_messages:
             return None
         return "\n".join(reversed(user_messages)), last_idx
 

Diff for: src/codegate/pipeline/codegate_context_retriever/codegate.py

@@ -136,9 +136,6 @@ async def process(  # noqa: C901
 
         logger.info(f"Adding {len(all_bad_packages)} bad packages to the context.")
 
-        # Generate context string using the searched objects
-        context_str = "CodeGate did not find any malicious or archived packages."
-
         # Nothing to do if no bad packages are found
         if len(all_bad_packages) == 0:
             return PipelineResult(request=request, context=context)

Diff for: src/codegate/providers/copilot/provider.py

@@ -24,11 +24,6 @@
     CopilotPipeline,
 )
 from codegate.providers.copilot.streaming import SSEProcessor
-from codegate.types.common import (
-    Delta,
-    ModelResponse,
-    StreamingChoices,
-)
 from codegate.types.openai import StreamingChatCompletion
 
 setup_logging()
@@ -239,7 +234,7 @@ async def _body_through_pipeline(
         path: str,
         headers: list[str],
         body: bytes,
-    ) -> Tuple[bytes, PipelineContext]:
+    ) -> Tuple[bytes, PipelineContext | None]:
         strategy = self._select_pipeline(method, path)
         if len(body) == 0 or strategy is None:
             # if we didn't select any strategy that would change the request