Refactor BotController to improve logging and code formatting

src/controllers/_bot_controller.py

@@ -1,7 +1,7 @@
-from io import BytesIO
 import os
 import re
 import traceback
+from io import BytesIO
 
 import aiohttp
 from fastapi import APIRouter, BackgroundTasks, Form, HTTPException, Request, UploadFile
@@ -11,7 +11,7 @@ from slack_sdk.web.async_client import AsyncWebClient
 
 from src.models import ContentDeliveryFrequency, User
 from src.services import FileService, SlackBotService, SlackTeamService, UserService
-from src.utils import logger, Helper
+from src.utils import Helper, logger
 
 file_set = set()
 
@@ -63,7 +63,8 @@ class BotController:
                     if not user:
                         raise Exception("Failed to register user")
                 if not user:
-                    raise Exception("User not found")
+                    logger.warning(f"User not found with slack id: {slack_user_id}")
+                    return
 
                 if "files" in actual_event or "file" in actual_event:
                     if send_message:
@@ -332,8 +333,8 @@ class BotController:
                 "response_type": "ephemeral",
                 "text": "Please provide a valid file URL with the /refresh command.",
             }
-        
-        async with Helper as helper:    
+
+        async with Helper as helper:
             if not helper.is_valid_url(text):
                 return {
                     "response_type": "ephemeral",
@@ -342,7 +343,7 @@ class BotController:
         background_tasks.add_task(self.refreshUserContent, payload)
         return {
             "response_type": "ephemeral",
-            "text": "You're very welcome! 🎉 We'll be sharing these insights with you gradually over the next few weeks. 😊"
+            "text": "You're very welcome! 🎉 We'll be sharing these insights with you gradually over the next few weeks. 😊",
         }
 
     async def refreshUserContent(self, payload: dict):
@@ -374,7 +375,7 @@ class BotController:
                                         )
                                         await user_service.send_message(
                                             user_id=user.id,
-                                            message="Failed to download file. Please try again."
+                                            message="Failed to download file. Please try again.",
                                         )
                                         return
 
@@ -397,7 +398,7 @@ class BotController:
                                         )
                                         await user_service.send_message(
                                             user_id=user.id,
-                                            message="Unsupported content type. Please provide a valid file URL."
+                                            message="Unsupported content type. Please provide a valid file URL.",
                                         )
                                         return
 
@@ -409,11 +410,15 @@ class BotController:
                                             message="Failed to download file. Please try again.",
                                         )
                                         return
-                                    
-                                    content_disposition = response.headers.get("content-disposition")
+
+                                    content_disposition = response.headers.get(
+                                        "content-disposition"
+                                    )
                                     if content_disposition:
                                         # Try to extract filename from content-disposition
-                                        match = re.search(r'filename="(.+?)"', content_disposition)
+                                        match = re.search(
+                                            r'filename="(.+?)"', content_disposition
+                                        )
                                         if match:
                                             file_name = match.group(1)
                                         else: