Add a toggle to mute sounds (audio cues)

Author: tstromberg

cmd/goose/main.go

@@ -132,6 +132,7 @@ type App struct {
 	hideStaleIncoming   bool
 	loadingTurnData     bool
 	enableReminders     bool // Whether to send daily reminder notifications
+	enableAudioCues     bool // Whether to play audio cues for notifications
 }
 
 func main() {
@@ -183,8 +184,12 @@ func main() {
 		updateInterval:      updateInterval,
 		pendingTurnResults:  make([]TurnResult, 0),
 		enableReminders:     true,
+		enableAudioCues:     true,
 	}
 
+	// Load saved settings
+	app.loadSettings()
+
 	log.Println("Initializing GitHub clients...")
 	err = app.initClients(ctx)
 	if err != nil {

cmd/goose/sound.go

@@ -52,6 +52,16 @@ func (app *App) initSoundCache() {
 
 // playSound plays a cached sound file using platform-specific commands.
 func (app *App) playSound(ctx context.Context, soundType string) {
+	// Check if audio cues are enabled
+	app.mu.RLock()
+	audioEnabled := app.enableAudioCues
+	app.mu.RUnlock()
+
+	if !audioEnabled {
+		log.Printf("[SOUND] Sound playback skipped (audio cues disabled): %s", soundType)
+		return
+	}
+
 	log.Printf("[SOUND] Playing %s sound", soundType)
 	// Ensure sounds are cached
 	app.initSoundCache()

cmd/goose/ui.go

@@ -287,18 +287,24 @@ func (app *App) addStaticMenuItems(ctx context.Context) {
 		hideStaleItem.Check()
 	}
 	hideStaleItem.Click(func() {
+		app.mu.Lock()
 		app.hideStaleIncoming = !app.hideStaleIncoming
-		if app.hideStaleIncoming {
+		hideStale := app.hideStaleIncoming
+		// Clear menu state to force rebuild
+		app.lastMenuState = nil
+		app.mu.Unlock()
+
+		if hideStale {
 			hideStaleItem.Check()
 		} else {
 			hideStaleItem.Uncheck()
 		}
+
+		// Save settings to disk
+		app.saveSettings()
+
 		// Toggle hide stale PRs setting
 		// Force menu rebuild since hideStaleIncoming changed
-		app.mu.Lock()
-		// Clear menu state to force rebuild
-		app.lastMenuState = nil
-		app.mu.Unlock()
 		app.rebuildMenu(ctx)
 	})
 
@@ -323,11 +329,40 @@ func (app *App) addStaticMenuItems(ctx context.Context) {
 			reminderItem.Uncheck()
 			log.Println("[SETTINGS] Daily reminders disabled")
 		}
+
+		// Save settings to disk
+		app.saveSettings()
 	})
 
 	// Add login item option (macOS only)
 	addLoginItemUI(ctx, app)
 
+	// Audio cues
+	// Add 'Audio cues' option
+	audioItem := systray.AddMenuItem("Audio cues", "Play sounds for notifications")
+	app.mu.RLock()
+	if app.enableAudioCues {
+		audioItem.Check()
+	}
+	app.mu.RUnlock()
+	audioItem.Click(func() {
+		app.mu.Lock()
+		app.enableAudioCues = !app.enableAudioCues
+		enabled := app.enableAudioCues
+		app.mu.Unlock()
+
+		if enabled {
+			audioItem.Check()
+			log.Println("[SETTINGS] Audio cues enabled")
+		} else {
+			audioItem.Uncheck()
+			log.Println("[SETTINGS] Audio cues disabled")
+		}
+
+		// Save settings to disk
+		app.saveSettings()
+	})
+
 	// Quit
 	// Add 'Quit' option
 	quitItem := systray.AddMenuItem("Quit", "")