Refactor property listing summary in chat_with_session_memory

- Introduced a direct summary response for property listings, bypassing the LLM when properties are available, to enhance user experience.
- Consolidated the logic for building the response text, including search status and property details, improving readability and maintainability.
- Removed redundant code related to listing details in the system message, ensuring clarity in the response structure.
- Updated instructions for handling unavailable property details to improve user communication.

ai_chat.py

@@ -37,6 +37,37 @@ def chat_with_session_memory(state):
             response = vid if vid else "There isn't a video tour for that listing."
             return {"response": response}
     
+    # Direct listing summary (bypass LLM when properties are found)
+    props = state.get("properties", [])
+    if props:
+        # a) Build a direct summary string
+        response_text = ""
+        # include any search status
+        status = state.get("search_status_message")
+        if status:
+            response_text += status + "\n\n"
+        response_text += "Here are some listings I found:\n"
+        for p in props[:5]:
+            response_text += (
+                f"- {p.get('title')} in {p.get('location')}, {p.get('city')}: "
+                f"{p.get('size_sqm')} sqm, {p.get('price')} ({p.get('price_type')})\n"
+            )
+            url = p.get("listing_url")
+            if url:
+                response_text += f"  Link: {url}\n"
+        # b) Suggest extras based on available fields
+        extras = []
+        if any(p.get("images") for p in props): extras.append("images")
+        if any(p.get("features") for p in props): extras.append("feature list")
+        if any(p.get("floorplan_pdf") for p in props): extras.append("floorplan PDF")
+        if any(p.get("video_url") for p in props): extras.append("video tour")
+        if extras:
+            response_text += (
+                "\nI can also share " + ", ".join(extras) +
+                " for any listing—just let me know what you'd like."
+            )
+        return {"response": response_text}
+    
     # Get conversation history from database
     session_messages = []
     if session_id:
@@ -66,46 +97,7 @@ def chat_with_session_memory(state):
         f"and must-haves: {', '.join(must_have_list) if must_have_list else '[none]'}. "
     )
     
-    props = state.get("properties", [])
-    search_status = state.get("search_status_message", "")
-    
-    if search_status:
-        system_message += f"\n{search_status}\n"
-    
-    if props:
-        system_message += "\nHere are some listings I found:\n"
-        for p in props[:5]:
-            title = p.get("title", "[no title]")
-            loc = p.get("location", "")
-            city = p.get("city", "")
-            size = p.get("size_sqm", "")
-            price = p.get("price", "")
-            ptype = p.get("price_type", "")
-            features = ", ".join(p.get("features", [])) or "[no features]"
-            system_message += (
-                f"- {title} in {loc}, {city}: {size} sqm, "
-                f"{price} ({ptype}); features: {features}\n"
-            )
-            url = p.get("listing_url")
-            if url:
-                system_message += f"  Link: {url}\n"
-    
-    # After the listings loop, before building messages[]
-    extras = set()
-    for p in props[:5]:
-        if p.get("images"):
-            extras.add("images")
-        if p.get("features"):
-            extras.add("feature list")
-        if p.get("floorplan_pdf"):
-            extras.add("floorplan PDF")
-        if p.get("video_url"):
-            extras.add("video tour")
-    if extras:
-        opts = ", ".join(sorted(extras))
-        system_message += (
-            f"\nI can also share {opts} for any listing—just let me know what you'd like."
-        )
+    # Note: Listing details are now handled by direct summary above, not included in system_message
     
     system_message += "\n\nIMPORTANT: When users ask about specific details for a listing, check if that information is available in the property data. If not available, respond with 'For this listing, I don't have [specific detail] available right now' instead of saying you can't provide URLs or external information."