Fixes some typing, updates attachments content

Author: GabeVillalobos

src/sentry/notifications/platform/api/endpoints/internal_registered_templates.py

@@ -12,7 +12,7 @@
 from sentry.notifications.platform.email.provider import EmailRenderer
 from sentry.notifications.platform.msteams.provider import MSTeamsRenderable, MSTeamsRenderer
 from sentry.notifications.platform.registry import template_registry
-from sentry.notifications.platform.slack.provider import SlackRenderer
+from sentry.notifications.platform.slack.provider import SlackNotificationProvider
 from sentry.notifications.platform.types import (
     NotificationData,
     NotificationProviderKey,
@@ -92,13 +92,14 @@ def serialize_slack_preview[T: NotificationData](
 ) -> dict[str, Any]:
     data = template.example_data
     rendered_template = template.render_example()
-    message = SlackRenderer.render(data=data, rendered_template=rendered_template)
+    renderer = SlackNotificationProvider.get_renderer(data=data, category=template.category)
+    message = renderer.render(data=data, rendered_template=rendered_template)
 
     serialized_blocks = []
     for block in message.get("blocks", []):
         serialized_blocks.append(block.to_dict())
 
-    return {"blocks": serialized_blocks}
+    return {"blocks": serialized_blocks, "attachments": message.get("attachments", None)}
 
 
 def serialize_discord_preview[T: NotificationData](

src/sentry/notifications/platform/slack/provider.py

@@ -1,7 +1,7 @@
 from __future__ import annotations
 
 from dataclasses import dataclass
-from typing import TYPE_CHECKING, NotRequired, TypedDict
+from typing import TYPE_CHECKING, Any, NotRequired, TypedDict
 
 from slack_sdk.models.blocks import (
     ActionsBlock,
@@ -58,9 +58,8 @@ class SlackProviderThreadingContext(ProviderThreadingContext):
 
 class SlackRenderable(TypedDict):
     blocks: list[Block]
-    attachments: NotRequired[list[Block]]
+    attachments: NotRequired[list[dict[str, Any]]]
     text: str
-    color: NotRequired[str]
 
 
 class SlackRenderer(NotificationRenderer[SlackRenderable]):

src/sentry/notifications/platform/slack/renderers/metric_alert.py

@@ -43,12 +43,14 @@ def render[DataT: NotificationData](
             *blocks, fallback_text=fallback_text, color=color
         )
 
+        attachment_blocks = [
+            {"blocks": slack_body.get("blocks", []), "color": slack_body.get("color", "")}
+        ]
+
         renderable = SlackRenderable(
-            blocks=slack_body.get("blocks", []),
-            attachments=slack_body.get("attachments", []),
+            blocks=[],
+            attachments=attachment_blocks,
             text=slack_body.get("text", ""),
         )
-        if (color := slack_body.get("color")) is not None:
-            renderable["color"] = color
 
         return renderable

src/sentry/notifications/platform/templates/metric_alert.py

@@ -42,7 +42,6 @@ class MetricAlertNotificationData(NotificationData):
 @template_registry.register(NotificationSource.METRIC_ALERT)
 class MetricAlertNotificationTemplate(NotificationTemplate[MetricAlertNotificationData]):
     category = NotificationCategory.METRIC_ALERT
-    hide_from_debugger = True
     example_data = MetricAlertNotificationData(
         group_id=1,
         organization_id=1,