johno/gastown
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix(mail): use visible banner for tmux notifications

Browse Source 
Replace tmux display-message (subtle status bar notification) with
send-keys to echo a visible banner to the terminal. This ensures
mail notifications are actually seen by agents.

Closes gt-vnp9

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Steve Yegge
2025-12-19 12:00:22 -08:00
parent 95ba8fcb6b
commit d371f60e03
2 changed files with 19 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
internal/mail/router.go
View File

@@ -83,7 +83,7 @@ func (r *Router) GetMailbox(address string) (*Mailbox, error) {
}
// notifyRecipient sends a notification to a recipient's tmux session.
// Uses display-message for non-disruptive notification.
// Uses send-keys to echo a visible banner to ensure notification is seen.
// Supports mayor/, rig/polecat, and rig/refinery addresses.
func (r *Router) notifyRecipient(msg *Message) error {
sessionID := addressToSessionID(msg.To)
@@ -97,9 +97,8 @@ func (r *Router) notifyRecipient(msg *Message) error {
return nil // No active session, skip notification
}
// Display notification in status line (non-disruptive)
notification := fmt.Sprintf("[MAIL] From %s: %s", msg.From, msg.Subject)
return r.tmux.DisplayMessageDefault(sessionID, notification)
// Send visible notification banner to the terminal
return r.tmux.SendNotificationBanner(sessionID, msg.From, msg.Subject)
}
// addressToSessionID converts a mail address to a tmux session ID.