Improve SmartHomeMonitorWatcherTask: CancellationToken propagation, sync, and logging (#60)

* Initial plan

* Apply PR review feedback: CancellationToken, Unknown state, semaphore, logging improvements

Co-authored-by: thomasneuberger <23504477+thomasneuberger@users.noreply.github.com>
Agent-Logs-Url: https://github.com/thomasneuberger/TgHomeBot/sessions/02041d09-0799-491d-ac3a-ccd36acdea63

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: thomasneuberger <23504477+thomasneuberger@users.noreply.github.com>

Author: Copilot

TgHomeBot.Scheduling/Tasks/SmartHomeMonitorWatcherTask.cs

@@ -14,7 +14,17 @@ public class SmartHomeMonitorWatcherTask(ILogger<SmartHomeMonitorWatcherTask> lo
 
     public async Task ExecuteAsync(CancellationToken cancellationToken)
     {
-        var isRunning = await smartHomeConnector.EnsureMonitorIsRunning();
-        logger.LogInformation("Smart Home Monitor is running: {IsRunning}", isRunning);
+        cancellationToken.ThrowIfCancellationRequested();
+
+        var isRunning = await smartHomeConnector.EnsureMonitorIsRunning(cancellationToken).ConfigureAwait(false);
+
+        if (!isRunning)
+        {
+            logger.LogWarning("Smart Home Monitor is not running and could not be started.");
+        }
+        else
+        {
+            logger.LogDebug("Smart Home Monitor is running.");
+        }
     }
 }

TgHomeBot.SmartHome.Contract/ISmartHomeConnector.cs

@@ -8,5 +8,5 @@ public interface ISmartHomeConnector
     Task<IReadOnlyList<SmartDevice>> GetDevices(IReadOnlyList<MonitoredDevice> requestedDevices);
     Task<ISmartHomeMonitor> CreateMonitorAsync(IReadOnlyList<MonitoredDevice> devices,
         CancellationToken cancellationToken);
-    Task<bool> EnsureMonitorIsRunning();
+    Task<bool> EnsureMonitorIsRunning(CancellationToken cancellationToken);
 }

TgHomeBot.SmartHome.HomeAssistant/HomeAssistantConnector.cs

@@ -67,19 +67,28 @@ public async Task<ISmartHomeMonitor> CreateMonitorAsync(IReadOnlyList<MonitoredD
         }
     }
 
-    public async Task<bool> EnsureMonitorIsRunning()
+    public async Task<bool> EnsureMonitorIsRunning(CancellationToken cancellationToken)
     {
         try
         {
-            switch (_smartHomeMonitor?.State)
+            await _semaphore.WaitAsync(cancellationToken);
+            try
+            {
+                switch (_smartHomeMonitor?.State)
+                {
+                    case MonitorState.Idle:
+                    case MonitorState.Unknown:
+                        await _smartHomeMonitor!.StartMonitoringAsync(cancellationToken);
+                        return true;
+                    case MonitorState.Listening:
+                        return true;
+                    default:
+                        return false;
+                }
+            }
+            finally
             {
-                case MonitorState.Idle:
-                    await _smartHomeMonitor.StartMonitoringAsync(CancellationToken.None);
-                    return true;
-                case MonitorState.Listening:
-                    return true;
-                default:
-                    return false;
+                _semaphore.Release();
             }
         }
         catch (Exception e)