suppress LLM text reply after search_trips sends trip cards

- Add suppress_llm_reply flag to ToolResult (not serialized to LLM)
- search_trips handler now sends a 'choose a card' prompt and sets
suppress_llm_reply=True when trips are found
- Orchestrator returns empty string immediately when flag is set,
avoiding a second LLM call
- Guard conversation service against sending empty replies
- Update system prompt and tool schema to reflect new behavior

app/ai/orchestrator.py

@@ -7,7 +7,7 @@ from app.ai.providers import (
     InvalidToolCallGenerationError,
     RetryableProviderError,
 )
-from app.models.domain import AIProviderResponse, ToolCall
+from app.models.domain import AIProviderResponse, ToolCall, ToolResult
 from app.tools.registry import ToolRegistry
 
 logger = logging.getLogger(__name__)
@@ -93,12 +93,14 @@ class AIOrchestrator:
                 logger.warning("++++++++"+str(tool_call)+ "&&&"+ str(registry))
                 result = await _execute_tool_call(registry, tool_call)
                 logger.warning("+++++++++++++"+str(result))
+                if result.suppress_llm_reply:
+                    return ""
                 working_messages.append(
                     {
                         "role": "tool",
                         "tool_call_id": tool_call.id,
                         "name": tool_call.name,
-                        "content": json.dumps(result, ensure_ascii=False),
+                        "content": json.dumps(result.to_payload(), ensure_ascii=False),
                     }
                 )
 
@@ -125,12 +127,12 @@ def _assistant_tool_message(response: AIProviderResponse) -> dict[str, Any]:
     }
 
 
-async def _execute_tool_call(registry: ToolRegistry, tool_call: ToolCall) -> dict[str, Any]:
+async def _execute_tool_call(registry: ToolRegistry, tool_call: ToolCall) -> ToolResult:
     try:
         arguments = json.loads(tool_call.arguments or "{}")
         if not isinstance(arguments, dict):
             raise ValueError("Tool arguments must be a JSON object")
     except (json.JSONDecodeError, ValueError) as exc:
-        return {"ok": False, "error": f"Invalid tool arguments: {exc}", "data": {}}
+        return ToolResult(ok=False, data={}, error=f"Invalid tool arguments: {exc}")
 
-    return (await registry.execute(tool_call.name, arguments)).to_payload()
+    return await registry.execute(tool_call.name, arguments)

app/ai/tool_schemas.py

@@ -37,8 +37,8 @@ _SEARCH_TRIPS = {
             "Use when the customer asks for travel options. "
             "You can search with only departure or only destination"
             " — the other will match any location. "
-            "Each matching trip is sent as a separate WhatsApp message. "
-            "The customer can reply to a trip card to select it."
+            "Matching trips are sent as WhatsApp cards and user is prompted to pick one. "
+            "Do NOT add follow-up text — response is handled automatically."
         ),
         "parameters": {
             "type": "object",

app/models/domain.py

@@ -39,6 +39,7 @@ class ToolResult:
     ok: bool
     data: dict[str, Any]
     error: str | None = None
+    suppress_llm_reply: bool = False
 
     def to_payload(self) -> dict[str, Any]:
         payload: dict[str, Any] = {"ok": self.ok, "data": self.data}

app/services/conversation_service.py

@@ -115,6 +115,9 @@ class ConversationService:
             registry=registry,
         )
 
+        if not reply:
+            return reply
+
         await self.whatsapp.send_text(inbound.remoteJid, reply)
 
         await self.repository.create_message(

app/tools/handlers.py

@@ -182,19 +182,35 @@ class FalsaToolHandlers:
                 except WhatsAppClientError:
                     logger.warning("Failed to send trip card for trip %s", trip_id)
 
+            prompt = "يرجى الرد على إحدى بطاقات الرحلات أعلاه لاختيار رحلتك"
+            await self.whatsapp.send_text(self.remoteJid, prompt)
+            await self.repository.create_message(
+                customer_id=str(self.customer["id"]),
+                sender_type="assistant",
+                message=prompt,
+                metadata={"type": "trip_selection_prompt"},
+            )
+
+            return ToolResult(
+                ok=True,
+                data={
+                    "count": len(top_trips),
+                    "matches": top_trips,
+                    "alternate_alert": alternate_alert,
+                    "sent_as_messages": True,
+                    "note": "Cards sent. No text reply needed.",
+                },
+                suppress_llm_reply=True,
+            )
+
         return ToolResult(
             ok=True,
             data={
-                "count": len(top_trips),
-                "matches": top_trips,
+                "count": 0,
+                "matches": [],
                 "alternate_alert": alternate_alert,
-                "sent_as_messages": bool(top_trips),
-                "note": (
-                    "No active matching trips were found."
-                    if not top_trips
-                    else "Trips were sent as separate WhatsApp messages. "
-                          "Ask the user to reply to a trip card to select it."
-                ),
+                "sent_as_messages": False,
+                "note": "No active matching trips were found.",
             },
         )
 

prompts/system_passenger.md

@@ -1,5 +1,5 @@
 - search_trips as soon as the user mentions a departure or destination. You do not need all details — the tool accepts partial info. Only ask a follow-up if neither departure nor destination was mentioned.
-- When search_trips returns matches, trips are sent as separate WhatsApp messages. The user can reply to a trip card to select it — the system will detect which trip they meant automatically, without needing a trip_id or short_id.
+- When search_trips finds matches, trip cards are sent to the user automatically along with a prompt to choose one. Do NOT add any text after calling search_trips — the tool will handle the response.
 - When the user replies to a trip card, immediately call create_booking_lead — trip_id is auto-detected and seats default to 1. Do not ask for seat count or other details.
 - Booking leads are pending — seats not reserved.
 - After successful create_booking_lead, tell the passenger the driver's phone number from driver_phone so they can contact the driver directly.