Skip to content

Commit 49ac4fe

Browse files
authored
Fix break reminder showing malformed "Next:" output (#47)
- Check meeting is in future (> now_ms) not just non-zero - Omit "Next:" suffix entirely when no upcoming meeting - Fallback title to "meeting" if calendar entry has empty title - Clean output: "☕ 771m focus. Break?" or with meeting: "☕ 771m focus. Break? Next: Standup in 45m" Fixes issue where stale/past meetings caused "Next: nothing in ∞m"
1 parent 1668e4f commit 49ac4fe

1 file changed

Lines changed: 14 additions & 7 deletions

File tree

plugins/claude-status-hub/bin/refresh-daemon.sh

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -61,18 +61,25 @@ check_focus_break() {
6161

6262
# Check for gap before next meeting (need at least 15 min)
6363
local next_meeting_start=$(jq -r '.calendar.lastSeen[0].startTime // 0' "$CONFIG" 2>/dev/null)
64-
if [ "$next_meeting_start" -gt 0 ]; then
64+
local next_suffix=""
65+
66+
if [ "$next_meeting_start" -gt "$now_ms" ]; then
67+
# Meeting is in the future
6568
local min_until_meeting=$(( (next_meeting_start - now_ms) / 60000 ))
66-
[ "$min_until_meeting" -ge 15 ] || return 0
69+
if [ "$min_until_meeting" -ge 15 ]; then
70+
local next_title=$(jq -r '.calendar.lastSeen[0].title // ""' "$CONFIG" 2>/dev/null)
71+
[ -z "$next_title" ] && next_title="meeting"
72+
next_suffix=" Next: ${next_title} in ${min_until_meeting}m"
73+
else
74+
# Meeting too soon - skip break reminder
75+
return 0
76+
fi
6777
fi
68-
69-
# Trigger break reminder alert
70-
local next_title=$(jq -r '.calendar.lastSeen[0].title // "nothing"' "$CONFIG" 2>/dev/null)
71-
local min_until=${min_until_meeting:-""}
78+
# No meeting or meeting is far enough away - show break reminder
7279

7380
# Update bridge with focus break alert
7481
if [ -f "$BRIDGE" ]; then
75-
local alert_text="${duration_min}m focus. Break? Next: ${next_title} in ${min_until}m"
82+
local alert_text="${duration_min}m focus. Break?${next_suffix}"
7683
jq --arg alert "$alert_text" \
7784
'.foreground = [{"service": "focus", "icon": "☕", "title": "Break reminder", "detail": $alert, "hasAlert": true}] + .foreground' \
7885
"$BRIDGE" > "${BRIDGE}.tmp" && mv "${BRIDGE}.tmp" "$BRIDGE"

0 commit comments

Comments
 (0)