From 7f1c35b717ea55c054c72be2d1e17e51b2faa10b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jonas=20Bj=C3=B6rkert?= <jonas@bjorkert.se>
Date: Sat, 25 Apr 2026 11:00:45 +0200
Subject: [PATCH] Skip volume-button stop teardown when monitoring was never
 started

Previously alarmStopped() always logged "Alarm stop detected" under the
Volume Button Snooze category and ran the teardown, even when the
volume-button-snooze feature was disabled and the matching
alarmStarted() had early-returned. The result was a misleading log line
on every alarm event for users who don't use volume-button snooze.

Guard alarmStopped() on isMonitoring so it only runs when the volume
observer was actually started. The "Invalidating volume observer." log
inside stopMonitoring() now serves as the single, accurate teardown
entry.
---
 LoopFollow/Controllers/VolumeButtonHandler.swift | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/LoopFollow/Controllers/VolumeButtonHandler.swift b/LoopFollow/Controllers/VolumeButtonHandler.swift
index caa388d9f..e190eca54 100644
--- a/LoopFollow/Controllers/VolumeButtonHandler.swift
+++ b/LoopFollow/Controllers/VolumeButtonHandler.swift
@@ -106,7 +106,7 @@ class VolumeButtonHandler: NSObject {
     }
 
     private func alarmStopped() {
-        LogManager.shared.log(category: .volumeButtonSnooze, message: "Alarm stop detected")
+        guard isMonitoring else { return }
 
         alarmStartTime = nil
         stopMonitoring()