Address pylint failures

Author: samdoran

src/lightspeed_stack/utils/agents/streaming.py

@@ -141,7 +141,7 @@ async def retrieve_agent_response_generator(
         raise HTTPException(**response.model_dump()) from exc
 
 
-async def generate_agent_response(
+async def generate_agent_response(  # pylint: disable=too-many-arguments,too-many-positional-arguments,too-many-locals
     generator: AsyncIterator[str],
     context: ResponseGeneratorContext,
     responses_params: ResponsesApiParams,
@@ -389,7 +389,7 @@ def _process_token(
 
 
 @singledispatch
-def dispatch_stream_event(
+def dispatch_stream_event(  # pylint: disable=useless-return
     event: AgentDispatchEvent,
     _state: AgentTurnAccumulator,
 ) -> Optional[StreamEventPayload]:

src/lightspeed_stack/utils/agents/tool_processor.py

@@ -101,7 +101,7 @@ def summarize_native_tool_call(
                 type="mcp_call",
             )
         case _:
-            logger.warning(f"Unknown tool name: {part.tool_name}")
+            logger.warning("Unknown tool name: %s", part.tool_name)
             return None
 
 
@@ -184,7 +184,7 @@ def process_native_tool_result(
         case tool_name if tool_name.startswith(_MCP_SERVER_TOOL_PREFIX):
             tool_result = summarize_mcp_tool_result(part, state.tool_round)
         case _:
-            logger.warning(f"Unknown tool name: {part.tool_name}")
+            logger.warning("Unknown tool name: %s", part.tool_name)
             return None
 
     state.emitted_tool_result_ids.add(tool_result.id)

src/lightspeed_stack/utils/reranker.py

@@ -41,7 +41,7 @@ async def _get_cross_encoder(model_name: str) -> Any:
         if model_name in _cross_encoder_models:
             return _cross_encoder_models[model_name]
         try:
-            from sentence_transformers import (  # pylint: disable=import-outside-toplevel
+            from sentence_transformers import (  # pylint: disable=import-outside-toplevel,import-error
                 CrossEncoder,
             )
 

tests/unit/app/endpoints/test_conversations_v2.py

@@ -1,4 +1,4 @@
-# pylint: disable=redefined-outer-name
+# pylint: disable=redefined-outer-name,too-many-lines
 
 """Unit tests for the /conversations REST API endpoints."""