Stabilize dev checks and local IPC

Author: nakorndev

.editorconfig

@@ -2,7 +2,7 @@ root = true
 
 [*]
 charset = utf-8
-end_of_line = crlf
+end_of_line = lf
 insert_final_newline = true
 trim_trailing_whitespace = true
 

.gitattributes

@@ -0,0 +1,2 @@
+* text=auto eol=lf
+

.github/workflows/ci.yml

@@ -2,7 +2,7 @@ name: CI
 
 on:
   push:
-    branches: [main]
+    branches: [main, develop]
   pull_request:
 
 jobs:

scripts/run-dev.ps1

@@ -37,6 +37,10 @@ try {
 
     switch ($Mode) {
         "app" {
+            Get-Process -Name "ToastDeck.App" -ErrorAction SilentlyContinue |
+                Where-Object { $_.Path -like (Join-Path $repoRoot "src\ToastDeck.App\bin\x64\*\ToastDeck.App.exe") } |
+                Stop-Process -Force
+
             if (-not $NoBuild) {
                 Invoke-Checked { dotnet build ToastDeck.slnx --configuration $Configuration --no-restore }
             }

src/ToastDeck.App/App.xaml.cs

@@ -8,7 +8,9 @@ public partial class App : Application
 
     public App()
     {
+#if DEBUG
         UnhandledException += App_UnhandledException;
+#endif
         InitializeComponent();
     }
 
@@ -18,6 +20,7 @@ protected override void OnLaunched(LaunchActivatedEventArgs args)
         window.Activate();
     }
 
+#if DEBUG
     private static void App_UnhandledException(object sender, Microsoft.UI.Xaml.UnhandledExceptionEventArgs e)
     {
         try
@@ -34,4 +37,5 @@ private static void App_UnhandledException(object sender, Microsoft.UI.Xaml.Unha
         {
         }
     }
+#endif
 }

src/ToastDeck.Core/Integrations/ToastDeckPipeClient.cs

@@ -8,7 +8,7 @@ public sealed class ToastDeckPipeClient
     private static readonly JsonSerializerOptions JsonOptions = new(JsonSerializerDefaults.Web)
     {
         PropertyNameCaseInsensitive = true,
-        WriteIndented = true
+        WriteIndented = false
     };
 
     public async Task<ToastDeckPipeResponse> SendAsync(
@@ -34,14 +34,25 @@ public async Task<ToastDeckPipeResponse> SendAsync(
             return ToastDeckPipeResponse.Fail("ToastDeck is not running or did not respond. Start the ToastDeck app and try again.");
         }
 
-        await using var writer = new StreamWriter(pipe) { AutoFlush = true };
-        using var reader = new StreamReader(pipe);
-        await writer.WriteLineAsync(JsonSerializer.Serialize(request, JsonOptions));
-        var responseJson = await reader.ReadLineAsync(linked.Token);
+        try
+        {
+            await using var writer = new StreamWriter(pipe, leaveOpen: true) { AutoFlush = true };
+            using var reader = new StreamReader(pipe, leaveOpen: true);
+            await writer.WriteLineAsync(JsonSerializer.Serialize(request, JsonOptions));
+            var responseJson = await reader.ReadLineAsync(linked.Token);
 
-        return string.IsNullOrWhiteSpace(responseJson)
-            ? ToastDeckPipeResponse.Fail("ToastDeck returned an empty IPC response.")
-            : JsonSerializer.Deserialize<ToastDeckPipeResponse>(responseJson, JsonOptions) ??
-                ToastDeckPipeResponse.Fail("ToastDeck returned an invalid IPC response.");
+            return string.IsNullOrWhiteSpace(responseJson)
+                ? ToastDeckPipeResponse.Fail("ToastDeck returned an empty IPC response.")
+                : JsonSerializer.Deserialize<ToastDeckPipeResponse>(responseJson, JsonOptions) ??
+                    ToastDeckPipeResponse.Fail("ToastDeck returned an invalid IPC response.");
+        }
+        catch (IOException ex)
+        {
+            return ToastDeckPipeResponse.Fail($"ToastDeck IPC failed: {ex.Message}");
+        }
+        catch (JsonException ex)
+        {
+            return ToastDeckPipeResponse.Fail($"ToastDeck returned invalid IPC JSON: {ex.Message}");
+        }
     }
 }

src/ToastDeck.Core/Integrations/ToastDeckPipeServer.cs

@@ -8,7 +8,7 @@ public sealed class ToastDeckPipeServer
     private static readonly JsonSerializerOptions JsonOptions = new(JsonSerializerDefaults.Web)
     {
         PropertyNameCaseInsensitive = true,
-        WriteIndented = true
+        WriteIndented = false
     };
 
     private readonly Func<ToastDeckPipeRequest, CancellationToken, Task<ToastDeckPipeResponse>> handler;
@@ -32,8 +32,8 @@ public async Task RunAsync(CancellationToken cancellationToken)
             try
             {
                 await pipe.WaitForConnectionAsync(cancellationToken);
-                using var reader = new StreamReader(pipe);
-                await using var writer = new StreamWriter(pipe) { AutoFlush = true };
+                using var reader = new StreamReader(pipe, leaveOpen: true);
+                await using var writer = new StreamWriter(pipe, leaveOpen: true) { AutoFlush = true };
                 var requestJson = await reader.ReadLineAsync(cancellationToken);
 
                 var response = await HandleAsync(requestJson, cancellationToken);

tests/ToastDeck.Tests/NotificationStateMachineTests.cs

@@ -32,10 +32,10 @@ public void MarkRead_ClearsSnoozeAndSetsReadTimestamp()
         var now = DateTimeOffset.UtcNow;
         var notification = NotificationFactory.Create(new NotificationDraft("Calendar", "Reminder", "Starts soon"))
             with
-            {
-                State = NotificationState.Snoozed,
-                SnoozedUntil = now.AddMinutes(15)
-            };
+        {
+            State = NotificationState.Snoozed,
+            SnoozedUntil = now.AddMinutes(15)
+        };
 
         var read = NotificationStateMachine.MarkRead(notification, now);