fix linter

Author: Radovan Fuchs

src/app/endpoints/query.py

@@ -314,14 +314,16 @@ async def query_endpoint_handler_base(  # pylint: disable=R0914
             if not existing_conversation:
                 # Check if topic summary should be generated (default: True)
                 should_generate = query_request.generate_topic_summary
-                
+
                 if should_generate:
                     logger.debug("Generating topic summary for new conversation")
                     topic_summary = await get_topic_summary_func(
                         query_request.query, client, llama_stack_model_id
                     )
                 else:
-                    logger.debug("Topic summary generation disabled by request parameter")
+                    logger.debug(
+                        "Topic summary generation disabled by request parameter"
+                    )
                     topic_summary = None
         # Convert RAG chunks to dictionary format once for reuse
         logger.info("Processing RAG chunks...")

src/app/endpoints/streaming_query.py

@@ -864,14 +864,15 @@ async def response_generator(
                 if not existing_conversation:
                     # Check if topic summary should be generated (default: True)
                     should_generate = query_request.generate_topic_summary
-                    
                     if should_generate:
                         logger.debug("Generating topic summary for new conversation")
                         topic_summary = await get_topic_summary(
                             query_request.query, client, model_id
                         )
                     else:
-                        logger.debug("Topic summary generation disabled by request parameter")
+                        logger.debug(
+                            "Topic summary generation disabled by request parameter"
+                        )
                         topic_summary = None
 
             completed_at = datetime.now(UTC).strftime("%Y-%m-%dT%H:%M:%SZ")

src/models/requests.py

@@ -81,7 +81,7 @@ class QueryRequest(BaseModel):
         system_prompt: The optional system prompt.
         attachments: The optional attachments.
         no_tools: Whether to bypass all tools and MCP servers (default: False).
-        generate_topic_summary: Whether to generate topic summary for new conversations (default: None = use server config).
+        generate_topic_summary: Whether to generate topic summary for new conversations.
         media_type: The optional media type for response format (application/json or text/plain).
 
     Example:
@@ -149,7 +149,7 @@ class QueryRequest(BaseModel):
 
     generate_topic_summary: Optional[bool] = Field(
         True,
-        description="Whether to generate topic summary for new conversations (None = use server config default)",
+        description="Whether to generate topic summary for new conversations",
         examples=[True, False],
     )
 

tests/unit/app/endpoints/test_query.py

@@ -2295,16 +2295,12 @@ async def test_query_endpoint_generate_topic_summary_default_true(
         return_value=("test_model", "test_model", "test_provider"),
     )
     mocker.patch("app.endpoints.query.is_transcripts_enabled", return_value=False)
-    
-    # Mock get_topic_summary - this SHOULD be called
+
     mock_get_topic_summary = mocker.patch(
         "app.endpoints.query.get_topic_summary", return_value="Generated topic"
     )
-    
-    # Mock database operations
     mock_database_operations(mocker)
 
-    # Call without specifying generate_topic_summary (defaults to True)
     await query_endpoint_handler(
         request=dummy_request,
         query_request=QueryRequest(query="test query"),
@@ -2347,22 +2343,18 @@ async def test_query_endpoint_generate_topic_summary_explicit_false(
         return_value=("test_model", "test_model", "test_provider"),
     )
     mocker.patch("app.endpoints.query.is_transcripts_enabled", return_value=False)
-    
-    # Mock get_topic_summary - this should NOT be called
+
     mock_get_topic_summary = mocker.patch(
         "app.endpoints.query.get_topic_summary", return_value="Generated topic"
     )
-    
-    # Mock database operations
+
     mock_database_operations(mocker)
 
-    # Call with generate_topic_summary=False
     await query_endpoint_handler(
         request=dummy_request,
         query_request=QueryRequest(query="test query", generate_topic_summary=False),
         auth=("user123", "username", False, "auth_token_123"),
         mcp_headers={},
     )
 
-    # Verify get_topic_summary was NOT called
     mock_get_topic_summary.assert_not_called()

tests/unit/models/requests/test_query_request.py

@@ -155,7 +155,6 @@ def test_validate_media_type(self, mocker: MockerFixture) -> None:
         # Media type is now fully supported, no warning expected
         mock_logger.warning.assert_not_called()
 
-
     def test_generate_topic_summary_explicit_false(self) -> None:
         """Test that generate_topic_summary can be explicitly set to False."""
         qr = QueryRequest(