Update app.py

app.py

@@ -82,35 +82,28 @@ def fetch_todays_messages(slack: WebClient, log: list) -> list:
 
 
 # ─── STEP 2B: CHECK LEAVE STATUS VIA SLACK PROFILE ──────────────────────────
-def is_on_leave_today(name: str, status_text: str, status_emoji: str, claude_client) -> bool:
-    """Use Claude to intelligently determine if an intern is on leave today."""
-    if not status_text and not status_emoji:
+def is_on_leave_today(name: str, full_status: str, claude_client) -> bool:
+    """Use Claude to determine if intern is on leave today from their full Slack status string."""
+    if not full_status or full_status.strip() == "":
         return False
 
     today = datetime.datetime.now(IST).strftime("%d-%m-%Y")
 
-    prompt = f"""Today's date is {today}.
+    prompt = f"""Today is {today}.
 
-Intern name: {name}
-Slack status text: "{status_text}"
-Slack status emoji: "{status_emoji}"
+Intern Slack status: "{full_status}"
 
-Is this intern on leave TODAY? Consider:
-- Date ranges like "on leave 27-04-2026 to 09-05-2026"
-- Simple messages like "on leave", "off today", "absent"
-- Red circle emoji or similar leave indicators
-- Any combination of emoji + text suggesting absence
-
-Reply with ONLY "yes" or "no"."""
+Is this intern on leave TODAY?
+Consider: date ranges, "on leave", "off", "absent", :red_circle: emoji, away messages.
+Reply ONLY with "yes" or "no"."""
 
     try:
         resp = claude_client.messages.create(
             model="claude-sonnet-4-20250514",
-            max_tokens=10,
+            max_tokens=5,
             messages=[{"role": "user", "content": prompt}]
         )
-        answer = resp.content[0].text.strip().lower()
-        return answer.startswith("yes")
+        return resp.content[0].text.strip().lower().startswith("yes")
     except Exception:
         return False
 
@@ -123,22 +116,22 @@ def get_interns_on_leave(slack: WebClient, claude_client, interns: list, log: li
 
     for intern in interns:
         try:
-            resp = slack.users_profile_get(user=intern["userId"])
-            profile = resp.get("profile", {})
+            # Use users.info for reliable status reading
+            resp = slack.users_info(user=intern["userId"])
+            profile = resp.get("user", {}).get("profile", {})
             status_text  = profile.get("status_text", "")
             status_emoji = profile.get("status_emoji", "")
+            full_status  = f"{status_emoji} {status_text}".strip()
 
-            if not status_text and not status_emoji:
+            if not full_status:
                 continue
 
-            on_leave_today = is_on_leave_today(
-                intern["name"], status_text, status_emoji, claude_client
-            )
+            on_leave_today = is_on_leave_today(intern["name"], full_status, claude_client)
             if on_leave_today:
                 on_leave.add(intern["userId"])
-                log.append(f"   {intern['name']} is on leave — status: '{status_emoji} {status_text}'")
+                log.append(f"   {intern['name']} — on leave ({full_status})")
         except Exception as e:
-            log.append(f"   Could not check status for {intern['name']}: {e}")
+            log.append(f"   Could not check {intern['name']}: {e}")
 
     return on_leave
 
@@ -277,49 +270,59 @@ def get_alert_channel_id(slack: WebClient, log: list) -> str:
 
 # ─── STEP 5: BUILD SLACK MESSAGE ──────────────────────────────────────────────
 def build_slack_message(results: dict, evals: dict, run_time: str) -> str:
-    submitted = [r for r in results.values() if r["submitted"]]
-    missed    = [r for r in results.values() if not r["submitted"]]
-    good      = [r for uid, r in results.items() if r["submitted"] and evals.get(uid, {}).get("quality") == "good"]
-    weak      = [r for uid, r in results.items() if r["submitted"] and evals.get(uid, {}).get("quality") == "weak"]
-    invalid   = [r for uid, r in results.items() if r["submitted"] and evals.get(uid, {}).get("quality") == "invalid"]
+    active   = {uid: r for uid, r in results.items() if not r.get("on_leave")}
+    submitted = [r for r in active.values() if r["submitted"]]
+    missed    = [r for r in active.values() if not r["submitted"]]
+    on_leave_list = [r for r in results.values() if r.get("on_leave")]
+    good    = [r for uid, r in active.items() if r["submitted"] and evals.get(uid, {}).get("quality") == "good"]
+    weak    = [r for uid, r in active.items() if r["submitted"] and evals.get(uid, {}).get("quality") == "weak"]
+    invalid = [r for uid, r in active.items() if r["submitted"] and evals.get(uid, {}).get("quality") == "invalid"]
 
     total = len(results)
+    active_count = len(active)
     date_str = datetime.datetime.now(IST).strftime("%d %b %Y")
+
     lines = []
-    lines.append(f"<!channel> *EOD Report Summary — {date_str}*")
-    lines.append(f"_{total} interns | Deadline: 11:00 PM IST_")
+    lines.append(f"<!channel>")
+    lines.append(f"*Daily Report Check — {date_str}*")
+    lines.append(f"{'─' * 32}")
+    lines.append(f":busts_in_silhouette: *{active_count} active* · :spiral_calendar_pad: Deadline 11:00 PM IST")
     lines.append("")
 
-    on_leave_list = [r for r in results.values() if r.get("on_leave")]
-    if on_leave_list:
-        lines.append(f"*On Leave ({len(on_leave_list)}/{total})*")
-        for r in on_leave_list:
-            lines.append(f"  :red_circle: *{r['name']}* — on approved leave")
-        lines.append("")
-
+    # Submitted
     if submitted:
-        lines.append(f"*Submitted ({len(submitted)}/{total})*")
-        for uid, r in results.items():
-            if not r["submitted"] or r.get("on_leave"):
+        lines.append(f":notepad_spiral: *Reports Received — {len(submitted)}/{active_count}*")
+        for uid, r in active.items():
+            if not r["submitted"]:
                 continue
-            ev = evals.get(uid, {})
-            q  = ev.get("quality", "?")
-            sc = ev.get("score", "?")
-            label = {"good": ":large_green_circle:", "weak": ":large_yellow_circle:", "invalid": ":red_circle:"}.get(q, ":white_circle:")
+            ev  = evals.get(uid, {})
+            q   = ev.get("quality", "?")
+            sc  = ev.get("score", "?")
+            icon = {"good": ":large_green_circle:", "weak": ":large_yellow_circle:", "invalid": ":red_circle:"}.get(q, ":white_circle:")
             flags = ev.get("flags", [])
-            line = f"  {label} *{r['name']}* — {sc}/10 — {ev.get('reason', '')}"
+            line = f"  {icon} *{r['name']}*  ·  {sc}/10  ·  {ev.get('reason', '')}"
             if flags:
-                line += f" | flags: {', '.join(flags)}"
+                line += "\n       _flags: " + ", ".join(flags) + "_"
             lines.append(line)
         lines.append("")
 
+    # Missed
     if missed:
-        lines.append(f"*Did Not Submit ({len(missed)}/{total})*")
+        lines.append(f":x: *No Report — {len(missed)}/{active_count}*")
         for r in missed:
-            lines.append(f"  :x: *{r['name']}* — no report received")
+            lines.append(f"  :red_circle: *{r['name']}*")
+        lines.append("")
+
+    # On Leave
+    if on_leave_list:
+        lines.append(f":palm_tree: *On Leave — {len(on_leave_list)}/{total}*")
+        for r in on_leave_list:
+            lines.append(f"  :white_circle: *{r['name']}*")
         lines.append("")
 
-    lines.append(f"Good: {len(good)}  |  Weak: {len(weak)}  |  Invalid: {len(invalid)}  |  Missed: {len(missed)}")
+    # Footer
+    lines.append(f"{'─' * 32}")
+    lines.append(f":large_green_circle: {len(good)}  :large_yellow_circle: {len(weak)}  :red_circle: {len(invalid)}  :x: {len(missed)}  :palm_tree: {len(on_leave_list)}")
 
     return "\n".join(lines)