Add primary and non-primary fact block builders

- Introduced `PrimaryFactBlock()` and `NonPrimaryFactBlock()` functions to create fact blocks with configurable primary status
- Simplified fact block creation by extracting common logic into a shared implementation
- Supports creating fact blocks with flexible title and value configurations

Author: MikaKerman

elementary/messages/block_builders.py

@@ -87,6 +87,24 @@ def SummaryLineBlock(
     return LineBlock(inlines=text_blocks)
 
 
+def NonPrimaryFactBlock(fact: Tuple[LineBlock, LineBlock]) -> FactBlock:
+    title, value = fact
+    return FactBlock(
+        title=title,
+        value=value,
+        primary=False,
+    )
+
+
+def PrimaryFactBlock(fact: Tuple[LineBlock, LineBlock]) -> FactBlock:
+    title, value = fact
+    return FactBlock(
+        title=title,
+        value=value,
+        primary=True,
+    )
+
+
 def FactsBlock(
     *,
     facts: Sequence[

elementary/monitor/alerts/alert_messages/builder.py

@@ -5,14 +5,19 @@
     BoldTextLineBlock,
     BulletListBlock,
     FactsBlock,
+    ItalicTextLineBlock,
     JsonCodeBlock,
     LinkLineBlock,
+    NonPrimaryFactBlock,
+    PrimaryFactBlock,
     SummaryLineBlock,
+    TextLineBlock,
 )
 from elementary.messages.blocks import (
     CodeBlock,
     DividerBlock,
     ExpandableBlock,
+    FactListBlock,
     HeaderBlock,
     Icon,
     InlineBlock,
@@ -217,23 +222,54 @@ def _get_details_blocks(
                 ]
             )
         )
-        facts = []
+        fact_blocks = []
         if table:
-            facts.append(("Table", table))
+            fact_blocks.append(
+                PrimaryFactBlock(
+                    (TextLineBlock(text="Table"), TextLineBlock(text=table))
+                )
+            )
         if column:
-            facts.append(("Column", column))
+            fact_blocks.append(
+                NonPrimaryFactBlock(
+                    (TextLineBlock(text="Column"), TextLineBlock(text=column))
+                )
+            )
 
-        facts.append(("Tags", ", ".join(tags) if tags else "No tags"))
-        facts.append(("Owners", ", ".join(owners) if owners else "No owners"))
-        facts.append(
-            ("Subscribers", ", ".join(subscribers) if subscribers else "No subscribers")
+        tags_line = (
+            TextLineBlock(text=", ".join(tags))
+            if tags
+            else ItalicTextLineBlock(text="No tags")
+        )
+        owners_line = (
+            TextLineBlock(text=", ".join(owners))
+            if owners
+            else ItalicTextLineBlock(text="No owners")
+        )
+        subscribers_line = (
+            TextLineBlock(text=", ".join(subscribers))
+            if subscribers
+            else ItalicTextLineBlock(text="No subscribers")
+        )
+        fact_blocks.append(NonPrimaryFactBlock((TextLineBlock(text="Tags"), tags_line)))
+        fact_blocks.append(
+            NonPrimaryFactBlock((TextLineBlock(text="Owners"), owners_line))
+        )
+        fact_blocks.append(
+            NonPrimaryFactBlock((TextLineBlock(text="Subscribers"), subscribers_line))
         )
 
         if description:
-            facts.append(("Description", description))
+            fact_blocks.append(
+                PrimaryFactBlock(
+                    (TextLineBlock(text="Description"), TextLineBlock(text=description))
+                )
+            )
         if path:
-            facts.append(("Path", path))
-        blocks.append(FactsBlock(facts=facts))
+            fact_blocks.append(
+                PrimaryFactBlock((TextLineBlock(text="Path"), TextLineBlock(text=path)))
+            )
+        blocks.append(FactListBlock(facts=fact_blocks))
         return blocks
 
     def _get_result_blocks(
@@ -333,7 +369,7 @@ def _get_model_alert_config_blocks(
         facts = []
         if materialization:
             facts.append(("Materialization", materialization))
-        if full_refresh is not None:
+        if full_refresh:
             facts.append(("Full Refresh", "Yes" if full_refresh else "No"))
         return [FactsBlock(facts=facts)]
 
@@ -368,6 +404,7 @@ def _get_alert_list_line(
             if len(owners) == 1:
                 inlines.append(TextBlock(text=f"Owner: {owners.pop()}"))
             else:
+                # order owners by alphabetical order
                 owners.sort()
                 inlines.append(TextBlock(text=f"Owners: {', '.join(owners)}"))
 

tests/unit/alerts/alert_messages/fixtures/adaptive_card_dbt_test_alert_status-None_link-False_description-False_tags-False_owners-False_table-True_error-False_sample-False.json

@@ -43,15 +43,15 @@
             },
             {
               "title": "Tags",
-              "value": "No tags"
+              "value": "_No tags_"
             },
             {
               "title": "Owners",
-              "value": "No owners"
+              "value": "_No owners_"
             },
             {
               "title": "Subscribers",
-              "value": "No subscribers"
+              "value": "_No subscribers_"
             }
           ]
         }

tests/unit/alerts/alert_messages/fixtures/adaptive_card_dbt_test_alert_status-None_link-False_description-False_tags-False_owners-False_table-True_error-False_sample-True.json

@@ -43,15 +43,15 @@
             },
             {
               "title": "Tags",
-              "value": "No tags"
+              "value": "_No tags_"
             },
             {
               "title": "Owners",
-              "value": "No owners"
+              "value": "_No owners_"
             },
             {
               "title": "Subscribers",
-              "value": "No subscribers"
+              "value": "_No subscribers_"
             }
           ]
         }

tests/unit/alerts/alert_messages/fixtures/adaptive_card_dbt_test_alert_status-None_link-False_description-False_tags-False_owners-False_table-True_error-True_sample-False.json

@@ -43,15 +43,15 @@
             },
             {
               "title": "Tags",
-              "value": "No tags"
+              "value": "_No tags_"
             },
             {
               "title": "Owners",
-              "value": "No owners"
+              "value": "_No owners_"
             },
             {
               "title": "Subscribers",
-              "value": "No subscribers"
+              "value": "_No subscribers_"
             }
           ]
         }

tests/unit/alerts/alert_messages/fixtures/adaptive_card_dbt_test_alert_status-None_link-False_description-False_tags-False_owners-False_table-True_error-True_sample-True.json

@@ -43,15 +43,15 @@
             },
             {
               "title": "Tags",
-              "value": "No tags"
+              "value": "_No tags_"
             },
             {
               "title": "Owners",
-              "value": "No owners"
+              "value": "_No owners_"
             },
             {
               "title": "Subscribers",
-              "value": "No subscribers"
+              "value": "_No subscribers_"
             }
           ]
         }

tests/unit/alerts/alert_messages/fixtures/adaptive_card_dbt_test_alert_status-None_link-False_description-False_tags-False_owners-True_table-False_error-False_sample-False.json

@@ -39,15 +39,15 @@
           "facts": [
             {
               "title": "Tags",
-              "value": "No tags"
+              "value": "_No tags_"
             },
             {
               "title": "Owners",
               "value": "owner1, owner2"
             },
             {
               "title": "Subscribers",
-              "value": "No subscribers"
+              "value": "_No subscribers_"
             }
           ]
         }

tests/unit/alerts/alert_messages/fixtures/adaptive_card_dbt_test_alert_status-None_link-False_description-False_tags-False_owners-True_table-False_error-False_sample-True.json

@@ -39,15 +39,15 @@
           "facts": [
             {
               "title": "Tags",
-              "value": "No tags"
+              "value": "_No tags_"
             },
             {
               "title": "Owners",
               "value": "owner1, owner2"
             },
             {
               "title": "Subscribers",
-              "value": "No subscribers"
+              "value": "_No subscribers_"
             }
           ]
         }

tests/unit/alerts/alert_messages/fixtures/adaptive_card_dbt_test_alert_status-None_link-False_description-False_tags-False_owners-True_table-False_error-True_sample-False.json

@@ -39,15 +39,15 @@
           "facts": [
             {
               "title": "Tags",
-              "value": "No tags"
+              "value": "_No tags_"
             },
             {
               "title": "Owners",
               "value": "owner1, owner2"
             },
             {
               "title": "Subscribers",
-              "value": "No subscribers"
+              "value": "_No subscribers_"
             }
           ]
         }

tests/unit/alerts/alert_messages/fixtures/adaptive_card_dbt_test_alert_status-None_link-False_description-False_tags-False_owners-True_table-False_error-True_sample-True.json

@@ -39,15 +39,15 @@
           "facts": [
             {
               "title": "Tags",
-              "value": "No tags"
+              "value": "_No tags_"
             },
             {
               "title": "Owners",
               "value": "owner1, owner2"
             },
             {
               "title": "Subscribers",
-              "value": "No subscribers"
+              "value": "_No subscribers_"
             }
           ]
         }