Updated sources

Author: tisnik

src/app/endpoints/health.py

@@ -75,15 +75,14 @@ async def get_providers_health_statuses() -> list[ProviderHealthStatus]:
         providers = await client.providers.list()
         logger.debug("Found %d providers", len(providers))
 
-        health_results = [
+        return [
             ProviderHealthStatus(
                 provider_id=provider.provider_id,
                 status=str(provider.health.get("status", "unknown")),
                 message=str(provider.health.get("message", "")),
             )
             for provider in providers
         ]
-        return health_results
 
     except APIConnectionError as e:
         logger.error("Failed to check providers health: %s", e)

src/app/endpoints/models.py

@@ -49,7 +49,7 @@ def parse_llama_stack_model(model: Any) -> dict[str, Any]:
         if k not in ("provider_id", "provider_resource_id", "model_type")
     }
 
-    legacy_model = {
+    return {
         "identifier": getattr(model, "id", ""),
         "metadata": metadata,
         "api_model_type": model_type,
@@ -58,7 +58,6 @@ def parse_llama_stack_model(model: Any) -> dict[str, Any]:
         "provider_resource_id": str(custom_metadata.get("provider_resource_id", "")),
         "model_type": model_type,
     }
-    return legacy_model
 
 
 models_responses: dict[int | str, dict[str, Any]] = {

src/authorization/resolvers.py

@@ -148,8 +148,7 @@ def _get_claims(auth: AuthTuple) -> dict[str, Any]:
             # No claims for guests
             return {}
 
-        jwt_claims = unsafe_get_claims(token)
-        return jwt_claims
+        return unsafe_get_claims(token)
 
     @staticmethod
     def _evaluate_operator(

src/models/requests.py

@@ -503,8 +503,7 @@ def validate_categories(
         if len(value) == 0:
             return None  # Convert empty list to None for consistency
 
-        unique_categories = list(dict.fromkeys(value))  # don't lose ordering
-        return unique_categories
+        return list(dict.fromkeys(value))  # don't lose ordering
 
     @model_validator(mode="after")
     def check_feedback_provided(self) -> Self:

src/utils/responses.py

@@ -997,8 +997,7 @@ async def select_model_for_responses(
         and user_conversation.last_used_model
         and user_conversation.last_used_provider
     ):
-        model_id = f"{user_conversation.last_used_provider}/{user_conversation.last_used_model}"
-        return model_id
+        return f"{user_conversation.last_used_provider}/{user_conversation.last_used_model}"
 
     # 2. Select default model from configuration
     if configuration.inference is not None:

src/utils/tool_formatter.py

@@ -36,7 +36,7 @@ def format_tool_response(tool_dict: dict[str, Any]) -> dict[str, Any]:
         description = clean_description
 
     # Extract only the required fields
-    formatted_tool = {
+    return {
         "identifier": tool_dict.get("identifier", ""),
         "description": description,
         "parameters": tool_dict.get("parameters", []),
@@ -46,8 +46,6 @@ def format_tool_response(tool_dict: dict[str, Any]) -> dict[str, Any]:
         "type": tool_dict.get("type", ""),
     }
 
-    return formatted_tool
-
 
 def extract_clean_description(description: str) -> str:
     """