Update src/api/agent.py

src/api/agent.py

@@ -519,78 +519,96 @@ def _rule_based_classify(question: str) -> Dict[str, str]:
 
 def _hours_answer() -> str:
     """
-    Return a rich, day-aware library hours answer.
+    Return a rich, day-aware library hours answer with correct Normal and Semester Break schedules.
     Always links to https://library.ku.ac.ae/hours for live schedule.
     """
     from datetime import datetime, timezone, timedelta
     # UAE is UTC+4
     uae_now = datetime.now(timezone(timedelta(hours=4)))
-    day_num = uae_now.weekday()  # 0=Mon … 6=Sun
-    # Map to day name
     day_names = ["Monday", "Tuesday", "Wednesday", "Thursday", "Friday", "Saturday", "Sunday"]
-    today = day_names[day_num]
-
-    # Hours by day (0=Mon … 6=Sun)
-    _HOURS = {
-        "Sunday":    ("Open", "8:00 AM – 7:00 PM"),
-        "Monday":    ("Open", "8:00 AM – 7:00 PM"),
-        "Tuesday":   ("Open", "8:00 AM – 7:00 PM"),
-        "Wednesday": ("Open", "8:00 AM – 7:00 PM"),
-        "Thursday":  ("Open", "8:00 AM – 4:00 PM"),
-        "Friday":    ("Closed", ""),
-        "Saturday":  ("Open", "12:00 PM – 8:00 PM"),
+    today = day_names[uae_now.weekday()]
+
+    # Normal hours (classes in session)
+    _NORMAL = {
+        "Monday":    ("Open",   "8:00 AM – 9:00 PM"),
+        "Tuesday":   ("Open",   "8:00 AM – 9:00 PM"),
+        "Wednesday": ("Open",   "8:00 AM – 9:00 PM"),
+        "Thursday":  ("Open",   "8:00 AM – 5:00 PM"),
+        "Friday":    ("Open",   "7:30 AM – 12:30 PM"),
+        "Saturday":  ("Closed", ""),
+        "Sunday":    ("Open",   "12:00 PM – 8:00 PM"),
     }
-    status, hours_str = _HOURS.get(today, ("Unknown", ""))
-
-    if status == "Closed":
-        today_line = f'📅 <strong>Today ({today}):</strong> <span style="color:#dc2626">Closed</span>'
-    elif status == "Open":
-        today_line = (
-            f'📅 <strong>Today ({today}):</strong> '
-            f'<span style="color:#16a34a">Open</span> · {hours_str}'
+    # Semester break hours (classes not in session)
+    _BREAK = {
+        "Monday":    ("Open",   "8:00 AM – 4:00 PM"),
+        "Tuesday":   ("Open",   "8:00 AM – 4:00 PM"),
+        "Wednesday": ("Open",   "8:00 AM – 4:00 PM"),
+        "Thursday":  ("Open",   "8:00 AM – 4:00 PM"),
+        "Friday":    ("Open",   "7:30 AM – 12:00 PM"),
+        "Saturday":  ("Closed", ""),
+        "Sunday":    ("Closed", ""),
+    }
+
+    def _day_row(day: str, status: str, hrs: str) -> str:
+        if status == "Closed":
+            color = "#dc2626"
+            label = "Closed"
+        elif "12:00 PM" in hrs or "12:30 PM" in hrs:
+            color = "#d97706"
+            label = hrs
+        else:
+            color = "#16a34a"
+            label = hrs
+        return (
+            f"<tr><td style='padding:3px 12px 3px 0;color:#374151'>{day}</td>"
+            f"<td style='padding:3px 0;font-weight:600;color:{color}'>{label}</td></tr>"
+        )
+
+    def _today_line(schedule: dict) -> str:
+        status, hrs = schedule.get(today, ("Unknown", ""))
+        if status == "Closed":
+            return (f'📅 <strong>Today ({today}):</strong> '
+                    f'<span style="color:#dc2626;font-weight:700">Closed</span>')
+        elif status == "Open":
+            return (f'📅 <strong>Today ({today}):</strong> '
+                    f'<span style="color:#16a34a;font-weight:700">Open</span> · {hrs}')
+        return f'📅 <strong>Today ({today}):</strong> Check live hours page.'
+
+    def _schedule_table(schedule: dict) -> str:
+        rows = "".join(
+            _day_row(d, *schedule[d])
+            for d in ["Monday","Tuesday","Wednesday","Thursday","Friday","Saturday","Sunday"]
+        )
+        return (
+            "<table style='font-size:.85rem;border-collapse:collapse;width:100%;max-width:380px'>"
+            + rows + "</table>"
         )
-    else:
-        today_line = f'📅 <strong>Today ({today}):</strong> Check the live hours page.'
 
     return (
-        f"Here are the <strong>Khalifa University Library</strong> hours:<br><br>"
-        f"{today_line}<br><br>"
-
-        # Main Campus
-        "<strong>📍 Main Campus Library</strong><br>"
-        "<span style='font-size:.82rem;color:#6b7280'>E Building, Main Campus, Abu Dhabi · "
-        '<a href="tel:+97123124604">+971 2 312 4604</a></span><br><br>'
-        "<table style='font-size:.85rem;border-collapse:collapse;width:100%;max-width:380px'>"
-        "<tr><td style='padding:3px 12px 3px 0;color:#374151'>Sunday – Wednesday</td>"
-        "<td style='padding:3px 0;font-weight:600;color:#16a34a'>8:00 AM – 7:00 PM</td></tr>"
-        "<tr><td style='padding:3px 12px 3px 0;color:#374151'>Thursday</td>"
-        "<td style='padding:3px 0;font-weight:600;color:#d97706'>8:00 AM – 4:00 PM</td></tr>"
-        "<tr><td style='padding:3px 12px 3px 0;color:#374151'>Friday</td>"
-        "<td style='padding:3px 0;font-weight:600;color:#dc2626'>Closed</td></tr>"
-        "<tr><td style='padding:3px 12px 3px 0;color:#374151'>Saturday</td>"
-        "<td style='padding:3px 0;font-weight:600;color:#16a34a'>12:00 PM – 8:00 PM</td></tr>"
-        "</table><br>"
-
-        # Habshan / SAN
-        "<strong>📍 Habshan Library (SAN Campus)</strong><br>"
-        "<span style='font-size:.82rem;color:#6b7280'>Building 5, Sas Al Nakhl Campus, Abu Dhabi · "
-        '<a href="tel:+97123123160">+971 2 312 3160</a></span><br><br>'
-        "<table style='font-size:.85rem;border-collapse:collapse;width:100%;max-width:380px'>"
-        "<tr><td style='padding:3px 12px 3px 0;color:#374151'>Sunday – Wednesday</td>"
-        "<td style='padding:3px 0;font-weight:600;color:#16a34a'>8:00 AM – 7:00 PM</td></tr>"
-        "<tr><td style='padding:3px 12px 3px 0;color:#374151'>Thursday</td>"
-        "<td style='padding:3px 0;font-weight:600;color:#d97706'>8:00 AM – 4:00 PM</td></tr>"
-        "<tr><td style='padding:3px 12px 3px 0;color:#374151'>Friday</td>"
-        "<td style='padding:3px 0;font-weight:600;color:#dc2626'>Closed</td></tr>"
-        "<tr><td style='padding:3px 12px 3px 0;color:#374151'>Saturday</td>"
-        "<td style='padding:3px 0;font-weight:600;color:#16a34a'>12:00 PM – 8:00 PM</td></tr>"
-        "</table><br>"
-
-        # Warning + live link
-        "⚠️ <strong>Hours change during Ramadan, exam periods, and public holidays.</strong><br>"
-        f'Always check the live schedule: '
-        f'<a href="{LIBRARY_HOURS_URL}" target="_blank" style="font-weight:700;color:#003366">'
-        f'{LIBRARY_HOURS_URL}</a>'
+        "Here are the <strong>Khalifa University Library</strong> hours:<br><br>"
+        f"{_today_line(_NORMAL)}<br>"
+        "<span style='font-size:.75rem;color:#6b7280'>(based on normal semester schedule — see note below)</span><br><br>"
+
+        "<strong>📚 Normal Hours</strong> "
+        "<span style='font-size:.78rem;color:#6b7280'>(when classes are in session)</span><br><br>"
+        + _schedule_table(_NORMAL) +
+
+        "<br><br><strong>🏖️ Semester Break Hours</strong> "
+        "<span style='font-size:.78rem;color:#6b7280'>(when classes are not in session)</span><br><br>"
+        + _schedule_table(_BREAK) +
+
+        "<br><br><strong>📍 Both campuses follow the same schedule:</strong><br>"
+        "<span style='font-size:.82rem;color:#6b7280'>"
+        "Main Campus Library · E Building · "
+        '<a href="tel:+97123124604">+971 2 312 4604</a><br>'
+        "Habshan Library · Building 5, SAN Campus · "
+        '<a href="tel:+97123123160">+971 2 312 3160</a>'
+        "</span><br><br>"
+
+        "⚠️ <strong>Hours change during Ramadan, public holidays, and exam periods.</strong> "
+        "Always check the live schedule for the most current times:<br>"
+        f'<a href="{LIBRARY_HOURS_URL}" target="_blank" '
+        f'style="font-weight:700;color:#003366">{LIBRARY_HOURS_URL}</a>'
     )