resolve pylint issue

Author: Abdul-Microsoft

src/api/api/api_routes.py

@@ -103,17 +103,17 @@ async def conversation(request: Request):
         request_json = await request.json()
         conversation_id = request_json.get("conversation_id")
         query = request_json.get("query")
-        
+
         # Track chat request initiation
         track_event_if_configured("ChatRequestReceived", {
             "conversation_id": conversation_id
         })
-        
+
         # Attach conversation_id to current span for Application Insights correlation
         span = trace.get_current_span()
         if span and conversation_id:
             span.set_attribute("conversation_id", conversation_id)
-        
+
         chat_service = ChatService()
         result = await chat_service.stream_chat_request(conversation_id, query)
         track_event_if_configured(
@@ -124,14 +124,14 @@ async def conversation(request: Request):
 
     except Exception as ex:
         logger.exception("Error in conversation endpoint: %s", str(ex))
-        
+
         # Track specific error type
         track_event_if_configured("ChatRequestError", {
             "conversation_id": request_json.get("conversation_id") if 'request_json' in locals() else "",
             "error": str(ex),
             "error_type": type(ex).__name__
         })
-        
+
         span = trace.get_current_span()
         if span is not None:
             span.record_exception(ex)

src/api/api/history_routes.py

@@ -1,5 +1,4 @@
 import logging
-import os
 from fastapi import APIRouter, HTTPException, Query, Request
 from fastapi.responses import JSONResponse
 from auth.auth_utils import get_authenticated_user_details

src/api/app.py

@@ -48,8 +48,8 @@
 logging.getLogger("azure.monitor.opentelemetry.exporter.export._base").setLevel(logging.WARNING)
 
 # Configure Azure Monitor and OpenTelemetry before importing routes
-from azure.monitor.opentelemetry import configure_azure_monitor
-from opentelemetry.instrumentation.fastapi import FastAPIInstrumentor
+from azure.monitor.opentelemetry import configure_azure_monitor  # noqa: E402
+from opentelemetry.instrumentation.fastapi import FastAPIInstrumentor  # noqa: E402
 
 
 def build_app() -> FastAPI:
@@ -89,7 +89,7 @@ async def health_check():
             enable_live_metrics=True,
             disable_offline_storage=True  # Reduces "Storing events" logs
         )
-        
+
         # Instrument FastAPI app to automatically trace all requests
         FastAPIInstrumentor.instrument_app(fastapi_app)
         logging.info("Application Insights configured with live metrics and FastAPI instrumentation enabled")

src/api/services/history_service.py

@@ -236,7 +236,7 @@ async def delete_conversation(self, user_id: str, conversation_id: str) -> bool:
             logger.info("Successfully deleted conversation %s for user %s", conversation_id, user_id)
             return True
 
-        except Exception as e:
+        except Exception:
             logger.exception("Error deleting conversation %s for user %s", conversation_id, user_id)
             return False
 
@@ -288,7 +288,7 @@ async def get_messages(self, user_id: str, conversation_id: str):
             messages = await cosmos_conversation_client.get_messages(conversation_id)
             return messages
 
-        except Exception as e:
+        except Exception:
             logger.exception(
                 "Error retrieving messages for conversation %s", conversation_id)
             return []
@@ -371,7 +371,7 @@ async def clear_messages(self, user_id: str, conversation_id: str) -> bool:
                 "Successfully cleared messages in conversation %s for user %s", conversation_id, user_id)
             return True
 
-        except Exception as e:
+        except Exception:
             logger.exception(
                 "Error clearing messages for conversation %s", conversation_id)
             return False