changed voice and db mendatory points for appointment

.env

@@ -9,7 +9,7 @@ GOOGLE_API_KEY="AIzaSyA9sqz4YKQHKXR9TU1imw0DPOghzHOMiBo"
 
 
 ELEVENLABS_API_KEY="b3af3a938c8e15d5eae700ea47eea7d88dfe397f34fbd4b0c75c24f143b032b8"
-ELEVENLABS_VOICE_ID="rxvktZTNrsQlsGIpOQGz"
+ELEVENLABS_VOICE_ID="GrHQRXD136YZl3kbtri3"
 ELEVENLABS_MODEL_ID="eleven_v3"
 
 # TWILIO_ACCOUNT_SID="ACfafc0d2d007bdf14b21bb3e14a7a7b31"

core/backend.py

@@ -577,6 +577,8 @@ async def book_appointment(
     patient_age: str = "",
     patient_num: str = "",
     visiting_date: str = "",
+    visiting_day: str = "",
+    visiting_time: str = "",
     patient_mail: str = ""
 ) -> str:
     """
@@ -588,8 +590,10 @@ async def book_appointment(
         patient_name:  Full name of the patient.
         patient_age:   Age of the patient (e.g. "32").
         patient_num:   Contact phone number of the patient.
-        visiting_date: Date of visit in YYYY-MM-DD format or natural text.
-        patient_mail:  Optional mail address for confirmation mail.
+        visiting_date: Date of visit in YYYY-MM-DD format or natural text (optional if visiting_day is provided).
+        visiting_day:  Day of visit (e.g. "Sunday", "রবিবার", "today") — required if visiting_date is not provided.
+        visiting_time: Time of visit (e.g. "6pm-9pm") — required (can be auto-filled from doctor record if missing).
+        patient_mail:  Required email address for confirmation mail.
     """
     db_path     = get_db_path()
     patient_num = format_bd_number(patient_num)
@@ -598,16 +602,32 @@ async def book_appointment(
     doctor_name = _clean_text(doctor_name)
     category    = _clean_text(category)
     visiting_date = _clean_text(visiting_date)
+    visiting_day  = _clean_text(visiting_day)
+    visiting_time = _clean_text(visiting_time)
     patient_mail = _clean_text(patient_mail)
-    parsed_date = _parse_visit_date(visiting_date)
-    if parsed_date:
-        visiting_date = parsed_date
-
-    if not patient_name or not patient_age or not patient_num or not visiting_date:
-        return (
-            "Missing booking details. Need patient name, age, phone number, "
-            "and visit date."
-        )
+
+    if visiting_date:
+        parsed_date = _parse_visit_date(visiting_date)
+        if parsed_date:
+            visiting_date = parsed_date
+    elif visiting_day:
+        parsed_date = _parse_visit_date(visiting_day)
+        if parsed_date:
+            visiting_date = parsed_date
+
+    # Mandatory fields
+    if not patient_name:
+        return "Missing booking details. Need patient name."
+    if not patient_age:
+        return "Missing booking details. Need patient age."
+    if not patient_num:
+        return "Missing booking details. Need patient phone number."
+    if not (doctor_id or doctor_name):
+        return "Missing booking details. Need doctor name."
+    if not visiting_date:
+        return "Missing booking details. Need day/date to visit the doctor."
+    if not patient_mail:
+        return "Missing booking details. Need email address for confirmation."
 
     async with aiosqlite.connect(db_path) as db:
         db.row_factory = aiosqlite.Row
@@ -640,7 +660,23 @@ async def book_appointment(
         doctor_data     = dict(doctor)
         doctor_name     = doctor_data.get("doctor_name", "Unknown")
         doctor_category = doctor_data.get("category", "Unknown")
-        visiting_time = doctor_data.get("visiting_time", "Unknown")
+        doctor_visiting_days = doctor_data.get("visiting_days", "") or ""
+        doctor_visiting_time = doctor_data.get("visiting_time", "") or ""
+
+        # Auto-fill visiting_time from doctor record if caller didn't provide it
+        if not visiting_time:
+            visiting_time = doctor_visiting_time.strip()
+        if not visiting_time:
+            return "Missing booking details. Need time to visit the doctor."
+
+        # Keep visiting_day if provided; otherwise derive from date (English day)
+        if not visiting_day and visiting_date:
+            try:
+                import datetime as _dt
+                y, m, d = [int(x) for x in visiting_date.split("-")]
+                visiting_day = _dt.date(y, m, d).strftime("%A")
+            except Exception:
+                visiting_day = ""
 
         cursor = await db.execute(
             """SELECT id FROM patients
@@ -653,35 +689,33 @@ async def book_appointment(
                 f"on {visiting_date} already exists."
             )
 
+        # Create booking
         await db.execute(
             """INSERT INTO patients
-               (doctor_name, doctor_category, patient_name, patient_age, patient_num, visiting_date, patient_mail)
-               VALUES (?, ?, ?, ?, ?, ?, ?)""",
-            (doctor_name, doctor_category, patient_name, patient_age, patient_num, visiting_date, patient_mail),
+               (doctor_name, doctor_category, patient_name, patient_age, patient_num, visiting_date, visiting_day, visiting_time, patient_mail)
+               VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)""",
+            (doctor_name, doctor_category, patient_name, patient_age, patient_num, visiting_date, visiting_day, visiting_time, patient_mail),
         )
         await db.commit()
 
-    # Mail confirmation is optional. Book first, then try to send email if available.
+    # Mail confirmation is mandatory.
     mail_message = (
         f"Doctor     : {doctor_name}\n"
         f"Patient    : {patient_name}\n"
         f"Visit Date : {visiting_date}\n"
+        f"Visit Day  : {visiting_day}\n"
         f"Visit Time : {visiting_time}\n"
         f"Please arrive on time."
     )
-    mail_status = ""
-    if patient_mail:
-        try:
-            await send_mail(
-                to_mail=patient_mail,
-                subject="✅ Appointment Confirmed!",
-                body=mail_message,
-            )
-            mail_status = "\n📧 Mail confirmation sent."
-        except Exception as e:
-            mail_status = f"\n⚠️ Mail failed: {str(e)}"
-    else:
-        mail_status = "\n📧 No email provided, so email confirmation was skipped."
+    try:
+        await send_mail(
+            to_mail=patient_mail,
+            subject="✅ Appointment Confirmed!",
+            body=mail_message,
+        )
+        mail_status = "\n📧 Confirmation mail sent."
+    except Exception as e:
+        mail_status = f"\n⚠️ Mail failed: {str(e)}"
 
     return (
         f"✅ Appointment Booked!\n"
@@ -690,7 +724,10 @@ async def book_appointment(
         f"Patient      : {patient_name}\n"
         f"Age          : {patient_age}\n"
         f"Date         : {visiting_date}\n"
+        f"Day          : {visiting_day}\n"
+        f"Time         : {visiting_time}\n"
         f"Contact      : {patient_num}\n"
+        f"Email        : {patient_mail}\n"
         f"━━━━━━━━━━━━━━━━━━━━━━\n"
         f"Please arrive on time."
         f"{mail_status}"
@@ -800,7 +837,7 @@ TOOL RULES:
 - Use `book_appointment` only after identifying the doctor and required patient details.
 - Never invent `doctor_id`. Get it from tool results or resolve by doctor_name/category.
 - If the user gives a Bangla date like "আগামীকাল" or "পরশু", convert it to a real date before booking.
-- Do not block booking on email. If email is missing, book the appointment and skip email confirmation.
+- Email is REQUIRED for booking and must be used to send a confirmation mail.
 - If the user already provided name, age, phone, and date and then confirms, call `book_appointment` immediately.
 - If the user asks to cancel and only gives a phone number, cancel the single matching appointment if there is exactly one.
 
@@ -838,8 +875,9 @@ You must now choose the correct tool instead of answering in prose:
 - Use `delete_appointment` when the user is cancelling a booking.
 
 Important booking rules:
-- Email is optional. If the user did not provide email, book without it.
-- If the user already gave name, age, phone, and visit date, do not ask for those again.
+- Email is REQUIRED. Do not book without an email address.
+- Visiting time is REQUIRED. If the doctor record has a visiting_time, use it and confirm it with the user.
+- If the user already gave name, age, phone, doctor name, visit day/date, visit time, and email, do not ask again.
 - If the user has already confirmed the details, book immediately.
 
 Important cancellation rules:
@@ -923,11 +961,26 @@ class AIBackend:
                 patient_age TEXT,
                 patient_num TEXT,
                 visiting_date TEXT,
+                visiting_day TEXT,
+                visiting_time TEXT,
                 patient_mail TEXT
             )
         """)
         await self.conn.commit()
 
+        # Lightweight migrations for older DBs
+        async def _ensure_column(table: str, col: str, col_type: str) -> None:
+            async with self.conn.execute(f"PRAGMA table_info({table})") as cur:
+                rows = await cur.fetchall()
+            existing = {r[1] for r in rows}  # (cid,name,type,notnull,dflt,pk)
+            if col in existing:
+                return
+            await self.conn.execute(f"ALTER TABLE {table} ADD COLUMN {col} {col_type}")
+            await self.conn.commit()
+
+        await _ensure_column("patients", "visiting_day", "TEXT")
+        await _ensure_column("patients", "visiting_time", "TEXT")
+
     # ── Summarise node ─────────────────────────────────────────────────────────
     async def summarize_conversation(self, state: ChatState):
         existing = state.get("summary", "")

db_view/db.html

@@ -114,6 +114,8 @@
           <th>Doctor</th>
           <th>Category</th>
           <th>Visiting Date</th>
+          <th>Visiting Day</th>
+          <th>Visiting Time</th>
         </tr>
       </thead>
       <tbody></tbody>
@@ -186,6 +188,8 @@
             <td>${patient.doctor_name}</td>
             <td>${patient.doctor_category}</td>
             <td>${patient.visiting_date}</td>
+            <td>${patient.visiting_day || ''}</td>
+            <td>${patient.visiting_time || ''}</td>
           </tr>`).join('');
 
         loading.style.display = 'none';

db_view/dbapi.py

@@ -45,6 +45,8 @@ class Patient(BaseModel):
     patient_age: str
     patient_num: str
     visiting_date: str
+    visiting_day: str | None = ""
+    visiting_time: str | None = ""
     patient_mail: str
 
 class AllDataResponse(BaseModel):