fyp

app/services/conversation_service.py

@@ -59,10 +59,11 @@ class ConversationService:
         
         # 2. Check if user is trying to message themselves
         if owner_id == current_user_id:
-            raise HTTPException(
-                status_code=status.HTTP_400_BAD_REQUEST,
-                detail="You cannot start a conversation with yourself"
-            )
+            return {
+                "success": False,
+                "error": "self_chat",
+                "message": "Oops! You can't chat with yourself. This is your own listing! 😊",
+            }
         
         # 3. Prepare property card data (always needed for frontend prompt)
         property_card = {
@@ -176,16 +177,14 @@ class ConversationService:
         if before_id and ObjectId.is_valid(before_id):
             query["_id"] = {"$lt": ObjectId(before_id)}
         
-        # Get messages
-        cursor = db.messages.find(query).sort("created_at", -1).limit(limit)
+        # Get messages - sort ascending (oldest first, newest last)
+        # This gives chronological order: oldest at top, newest at bottom
+        cursor = db.messages.find(query).sort("created_at", 1).limit(limit)
         
         messages = []
         async for doc in cursor:
             messages.append(Message.format_response(doc))
         
-        # Reverse to get chronological order
-        messages.reverse()
-        
         logger.info(f"Found {len(messages)} messages for conversation {conversation_id}")
         
         return messages