fix(daemon): notify only when new mail (#1205)

EmilyxFox created 2 months ago

## What?

Fixes notifications every 5 minutes whether or not there are new emails

## Why?

notifications were sent consistently, when not intended

Change summary

daemon/daemon.go | 19 +++++++++++++++++--
1 file changed, 17 insertions(+), 2 deletions(-)

Detailed changes

daemon/daemon.go

@@ -348,6 +348,14 @@ func (d *Daemon) syncAllAccounts(ctx context.Context) {
 			continue
 		}
 
+		oldCached, _ := config.LoadFolderEmailCache("INBOX")
+		oldUIDs := make(map[uint32]struct{}, len(oldCached))
+		for _, e := range oldCached {
+			if e.AccountID == acct.ID {
+				oldUIDs[e.UID] = struct{}{}
+			}
+		}
+
 		// Cache the fetched emails to disk.
 		var cached []config.CachedEmail
 		for _, e := range emails {
@@ -372,14 +380,21 @@ func (d *Daemon) syncAllAccounts(ctx context.Context) {
 			EmailCount: len(emails),
 		})
 
+		newCount := 0
+		for _, e := range emails {
+			if _, seen := oldUIDs[e.UID]; !seen {
+				newCount++
+			}
+		}
+
 		// Send desktop notification if TUI not connected.
 		d.mu.RLock()
 		noClients := len(d.clients) == 0
 		d.mu.RUnlock()
 
-		if noClients && len(emails) > 0 {
+		if noClients && newCount > 0 {
 			if !d.config.DisableNotifications {
-				go notify.Send("Matcha", fmt.Sprintf("New mail for %s", acct.Email))
+				go notify.Send("Matcha", fmt.Sprintf("New mail for %s", acct.FetchEmail))
 			}
 		}
 	}