feat: Add in-app action and URI command to clear all notifications with logging and automated tests

This commit introduces multiple enhancements for clearing notifications:

1.  **Logging for Verification**:
    - The `NotificationHostService.CancelAllNotifications` method now logs a confirmation message when successfully executed. This aids in automated verification.

2.  **Updated GitHub Actions Workflow (`clear_notifications_test.yml`)**:
    - The workflow trigger is changed from `workflow_dispatch` to `on: push` to run on every commit.
    - A new step has been added to check the application's log files after the clear notifications URI is called. The workflow verifies that the specific confirmation message from `CancelAllNotifications` is present in the logs, ensuring the action was performed.

3.  **In-App Automated Action ("Clear All Notifications")**:
    - A new in-app automated action, "清除所有提醒" (Clear All Notifications), with ID `classisland.notifications.clearAll`, has been implemented.
    - `ClearAllNotificationsActionHandler.cs` was created to contain the logic, which calls `NotificationHostService.CancelAllNotifications()`.
    - The action and its handler are registered in `App.xaml.cs`, making it available for use within ClassIsland's automation rules.

4.  **URI Command for Clearing Notifications**: (Implemented in a previous commit, tested by the workflow)
    - The `UriNavigationService` handles the `classisland://app/clearAllNotifications` URI, which also triggers `NotificationHostService.CancelAllNotifications()`.

5.  **Build Error Fix**: (Resolved in a previous commit)
    - Corrected an incorrect namespace reference for `NotificationHostService` in `UriNavigationService.cs` that was causing build failures.

These changes provide both a manual/external way (URI) and an integrated/automated way (in-app action) to clear all notifications, along with an automated GitHub Actions workflow to verify the URI functionality using log-based verification.

Author: google-labs-jules[bot]

.github/workflows/clear_notifications_test.yml

@@ -1,7 +1,6 @@
 name: Test Clear All Notifications URI
 
-on:
-  workflow_dispatch:
+on: push
 
 jobs:
   test_clear_all_notifications_uri:
@@ -35,3 +34,31 @@ jobs:
 
       - name: Wait for URI to be processed
         run: Start-Sleep -Seconds 5
+
+      - name: Verify notifications cleared in logs
+        shell: pwsh
+        run: |
+          $logDir = "ClassIsland/bin/Release/net8.0-windows/Logs"
+          Write-Host "Checking for logs in: $logDir"
+          if (Test-Path $logDir) {
+            $latestLog = Get-ChildItem -Path $logDir -Filter "*.log" | Sort-Object LastWriteTime -Descending | Select-Object -First 1
+            if ($latestLog) {
+              Write-Host "Found log file: $($latestLog.FullName)"
+              $logContent = Get-Content -Path $latestLog.FullName -Raw
+              $expectedMessage = "All notifications successfully cancelled via CancelAllNotifications method."
+              if ($logContent -match [regex]::Escape($expectedMessage)) {
+                Write-Host "Success: Verification message found in log."
+              } else {
+                Write-Host "Error: Verification message NOT found in log."
+                Write-Host "Log content:"
+                Write-Host $logContent
+                exit 1
+              }
+            } else {
+              Write-Host "Error: No log files found in $logDir"
+              exit 1
+            }
+          } else {
+            Write-Host "Error: Log directory $logDir not found."
+            exit 1
+          }

ClassIsland/App.xaml.cs

@@ -687,9 +687,11 @@ private async void App_OnStartup(object sender, StartupEventArgs e)
                     "classisland.notification.weather", "显示天气提醒", PackIconKind.SunWirelessOutline);
                 services.AddAction("classisland.app.quit", "退出 ClassIsland", PackIconKind.ExitToApp, (_, _) => Current.Stop());
                 services.AddAction<AppRestartActionSettings,AppRestartActionSettingsControl>("classisland.app.restart", "重启 ClassIsland", PackIconKind.Restart);
+                services.AddAction("classisland.notifications.clearAll", "清除所有提醒", PackIconKind.NotificationClearAll);
                 // 行动处理
                 services.AddHostedService<AppRestartActionHandler>();
                 services.AddHostedService<RunActionHandler>();
+                services.AddHostedService<ClearAllNotificationsActionHandler>();
                 services.AddHostedService<AppSettingsActionHandler>();
                 services.AddHostedService<SleepActionHandler>();
                 // 认证提供方

ClassIsland/Services/ActionHandlers/ClearAllNotificationsActionHandler.cs

@@ -0,0 +1,33 @@
+using System.Threading;
+using System.Threading.Tasks;
+using ClassIsland.Core.Abstractions.Services; // For IActionService
+using ClassIsland.Services; // For INotificationHostService (assuming it's here)
+using Microsoft.Extensions.Hosting; // For IHostedService
+
+namespace ClassIsland.Services.ActionHandlers;
+
+public class ClearAllNotificationsActionHandler : IHostedService
+{
+    private readonly INotificationHostService _notificationHostService;
+
+    public ClearAllNotificationsActionHandler(IActionService actionService, INotificationHostService notificationHostService)
+    {
+        _notificationHostService = notificationHostService;
+        // The GUID for the action ID will be defined in App.xaml.cs
+        // For example: "classisland.notifications.clearAll"
+        actionService.RegisterActionHandler("classisland.notifications.clearAll", HandleClearAllNotifications);
+    }
+
+    private void HandleClearAllNotifications(object? settings, string guid)
+    {
+        _notificationHostService.CancelAllNotifications();
+        // Optionally, log that this action was triggered via in-app automation
+        // App.GetService<ILogger<ClearAllNotificationsActionHandler>>().LogInformation("ClearAllNotifications action triggered via in-app automation.");
+        // However, to use GetService, this class would need to be modified or have ILogger injected.
+        // For now, direct logging is omitted here but can be added if NotificationHostService's own logging isn't sufficient.
+    }
+
+    public Task StartAsync(CancellationToken cancellationToken) => Task.CompletedTask;
+
+    public Task StopAsync(CancellationToken cancellationToken) => Task.CompletedTask;
+}

ClassIsland/Services/NotificationHostService.cs

@@ -269,6 +269,7 @@ public void CancelAllNotifications()
             var r = RequestQueue.Dequeue();
             r.CompletedTokenSource.Cancel();
         }
+        Logger.LogInformation("All notifications successfully cancelled via CancelAllNotifications method.");
     }
 
     public event PropertyChangedEventHandler? PropertyChanged;