Merge pull request #141 from jonsagara/feature/chat

Instead of making alert level a separate text paragraph widget, prepe…

Author: jonsagara

Directory.Build.props

@@ -4,7 +4,7 @@
 		<TargetFrameworks>net8.0;net9.0;net10.0</TargetFrameworks>
 
 		<!-- NuGet -->
-		<Version>5.9.2</Version>
+		<Version>5.9.3</Version>
 		<AssemblyVersion>5.9.0</AssemblyVersion>
 		<FileVersion>5.9.0</FileVersion>
 		<Authors>Jon Sagara</Authors>

src/Sagara.Core.Google.Tests/Chat/GoogleChatServiceTests.cs

@@ -25,7 +25,7 @@ public async Task SendMessageAsync_BodyOnly_SendsTextOnlyPayload()
     }
 
     [Fact]
-    public async Task SendMessageAsync_Cards_EmitsAlertTextParagraphAndButtonWidgets()
+    public async Task SendMessageAsync_Cards_EmitsAlertPrefixedTitleTextParagraphAndButtonWidgets()
     {
         var handler = new CapturingHttpMessageHandler(HttpStatusCode.OK);
         var service = CreateService(handler);
@@ -47,16 +47,15 @@ await service.SendMessageAsync(
         var json = await handler.GetRequestJsonAsync();
 
         var cardElement = json.GetProperty("cardsV2").EnumerateArray().First().GetProperty("card");
-        Assert.Equal("Deploy failed", cardElement.GetProperty("header").GetProperty("title").GetString());
+        Assert.Equal("🔴 Deploy failed", cardElement.GetProperty("header").GetProperty("title").GetString());
         Assert.Equal("prod", cardElement.GetProperty("header").GetProperty("subtitle").GetString());
 
         var widgets = cardElement.GetProperty("sections").EnumerateArray().First()
             .GetProperty("widgets").EnumerateArray().ToList();
 
-        Assert.Equal(3, widgets.Count); // alert accent widget + text paragraph widget + button list
-        Assert.Contains("ERROR", widgets[0].GetProperty("textParagraph").GetProperty("text").GetString(), StringComparison.Ordinal);
-        Assert.Contains("more **info**", widgets[1].GetProperty("textParagraph").GetProperty("text").GetString(), StringComparison.Ordinal);
-        Assert.True(widgets[2].TryGetProperty("buttonList", out _));
+        Assert.Equal(2, widgets.Count); // text paragraph widget + button list
+        Assert.Contains("more **info**", widgets[0].GetProperty("textParagraph").GetProperty("text").GetString(), StringComparison.Ordinal);
+        Assert.True(widgets[1].TryGetProperty("buttonList", out _));
     }
 
     [Fact]

src/Sagara.Core.Google/Chat/GoogleChatAlertLevelInfo.cs

@@ -10,7 +10,7 @@ public static GoogleChatAlertLevelInfo For(GoogleChatAlertLevel level)
         {
             GoogleChatAlertLevel.Info => new GoogleChatAlertLevelInfo("ℹ️", "INFO", "#1a73e8"),
             GoogleChatAlertLevel.Warning => new GoogleChatAlertLevelInfo("⚠️", "WARNING", "#f9ab00"),
-            GoogleChatAlertLevel.Error => new GoogleChatAlertLevelInfo("❌", "ERROR", "#d93025"),
+            GoogleChatAlertLevel.Error => new GoogleChatAlertLevelInfo("🔴", "ERROR", "#d93025"),
             GoogleChatAlertLevel.Success => new GoogleChatAlertLevelInfo("✅", "SUCCESS", "#188038"),
             _ => throw new ArgumentOutOfRangeException(nameof(level), level, message: null),
         };

src/Sagara.Core.Google/Chat/GoogleChatService.cs

@@ -487,22 +487,17 @@ private static ChatCardWrapper BuildCard(
 
 
         //
-        // First widget: Alert Level, if any.
+        // If the caller specified an alert level and a title, prepend the emoji to the card title.
         //
 
         if (alertLevel is { } level)
         {
-            var alertLevelInfo = GoogleChatAlertLevelInfo.For(level);
-
-            widgets.Add(new ChatCardWidget
+            if (title is not null)
             {
-                // This is our content, not the user's, so we don't need to worry about escaping it.
-                TextParagraph = new ChatTextParagraph
-                {
-                    Text = $"<font color=\"{alertLevelInfo.HexColor}\">{alertLevelInfo.Emoji} {alertLevelInfo.Label}</font>",
-                    TextSyntax = "HTML",
-                },
-            });
+                var alertLevelInfo = GoogleChatAlertLevelInfo.For(level);
+
+                title = $"{alertLevelInfo.Emoji} {title}";
+            }
         }