ref(np): Dispatch provider renderers through a registry

src/sentry/notifications/apps.py

@@ -7,7 +7,13 @@ class Config(AppConfig):
     def ready(self) -> None:
         # Imports to populate registries
         import sentry.notifications.platform.discord.provider  # noqa: F401
+        import sentry.notifications.platform.discord.renderers.issue  # noqa: F401
+        import sentry.notifications.platform.discord.renderers.metric_alert  # noqa: F401
         import sentry.notifications.platform.email.provider  # noqa: F401
         import sentry.notifications.platform.msteams.provider  # noqa: F401
         import sentry.notifications.platform.slack.provider  # noqa: F401
+        import sentry.notifications.platform.slack.renderers.issue  # noqa: F401
+        import sentry.notifications.platform.slack.renderers.metric_alert  # noqa: F401
+        import sentry.notifications.platform.slack.renderers.seer  # noqa: F401
+        import sentry.notifications.platform.slack.renderers.seer_agent_write_approval  # noqa: F401
         import sentry.notifications.platform.templates  # noqa: F401

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

@@ -104,7 +104,7 @@ def serialize_slack_preview[T: NotificationData](
 ) -> dict[str, Any]:
     data = template.example_data
     rendered_template = template.render_example()
-    renderer = SlackNotificationProvider.get_renderer(data=data, category=template.category)
+    renderer = SlackNotificationProvider.get_renderer(data=data)
     message = renderer.render(data=data, rendered_template=rendered_template)
 
     serialized_blocks = []

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

@@ -18,7 +18,6 @@
 from sentry.notifications.platform.threading import ThreadContext
 from sentry.notifications.platform.types import (
     LinkTextBlock,
-    NotificationCategory,
     NotificationData,
     NotificationProviderKey,
     NotificationRenderedTemplate,
@@ -40,8 +39,6 @@
 
 
 class DiscordRenderer(NotificationRenderer[DiscordRenderable]):
-    provider_key = NotificationProviderKey.DISCORD
-
     @classmethod
     def render[DataT: NotificationData](
         cls, *, data: DataT, rendered_template: NotificationRenderedTemplate
@@ -152,21 +149,6 @@ def is_available(cls, *, organization: RpcOrganizationSummary | None = None) ->
         # TODO(ecosystem): Check for the integration, maybe a feature as well
         return False
 
-    @classmethod
-    def get_renderer(
-        cls, *, data: NotificationData, category: NotificationCategory
-    ) -> type[NotificationRenderer[DiscordRenderable]]:
-        from sentry.notifications.platform.discord.renderers.issue import IssueDiscordRenderer
-        from sentry.notifications.platform.discord.renderers.metric_alert import (
-            DiscordMetricAlertRenderer,
-        )
-
-        if category == NotificationCategory.ISSUE:
-            return IssueDiscordRenderer
-        if category == NotificationCategory.METRIC_ALERT:
-            return DiscordMetricAlertRenderer
-        return cls.default_renderer
-
     @classmethod
     def send(
         cls,

src/sentry/notifications/platform/discord/renderers/issue.py

@@ -4,20 +4,21 @@
 from sentry.integrations.discord.message_builder.issues import DiscordIssuesMessageBuilder
 from sentry.models.group import Group
 from sentry.notifications.platform.discord.provider import DiscordRenderable
+from sentry.notifications.platform.registry import renderer_registry
 from sentry.notifications.platform.renderer import NotificationRenderer
 from sentry.notifications.platform.service import NotificationRenderError
 from sentry.notifications.platform.templates.issue import IssueNotificationData
 from sentry.notifications.platform.types import (
     NotificationData,
     NotificationProviderKey,
     NotificationRenderedTemplate,
+    NotificationSource,
 )
 from sentry.services.eventstore.models import Event
 
 
+@renderer_registry.register(NotificationProviderKey.DISCORD, NotificationSource.ISSUE)
 class IssueDiscordRenderer(NotificationRenderer[DiscordRenderable]):
-    provider_key = NotificationProviderKey.DISCORD
-
     @classmethod
     def render[DataT: NotificationData](
         cls, *, data: DataT, rendered_template: NotificationRenderedTemplate

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

@@ -8,18 +8,19 @@
 from sentry.integrations.discord.message_builder.metric_alerts import get_started_at
 from sentry.integrations.metric_alerts import get_status_text
 from sentry.notifications.platform.discord.provider import DiscordRenderable
+from sentry.notifications.platform.registry import renderer_registry
 from sentry.notifications.platform.renderer import NotificationRenderer
 from sentry.notifications.platform.templates.metric_alert import MetricAlertNotificationData
 from sentry.notifications.platform.types import (
     NotificationData,
     NotificationProviderKey,
     NotificationRenderedTemplate,
+    NotificationSource,
 )
 
 
+@renderer_registry.register(NotificationProviderKey.DISCORD, NotificationSource.METRIC_ALERT)
 class DiscordMetricAlertRenderer(NotificationRenderer[DiscordRenderable]):
-    provider_key = NotificationProviderKey.DISCORD
-
     @classmethod
     def render[DataT: NotificationData](
         cls, *, data: DataT, rendered_template: NotificationRenderedTemplate

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

@@ -38,8 +38,6 @@
 
 
 class EmailRenderer(NotificationRenderer[EmailRenderable]):
-    provider_key = NotificationProviderKey.EMAIL
-
     @classmethod
     def render[DataT: NotificationData](
         cls, *, data: DataT, rendered_template: NotificationRenderedTemplate

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

@@ -38,8 +38,6 @@
 
 
 class MSTeamsRenderer(NotificationRenderer[MSTeamsRenderable]):
-    provider_key = NotificationProviderKey.MSTEAMS
-
     @classmethod
     def render[DataT: NotificationData](
         cls, *, data: DataT, rendered_template: NotificationRenderedTemplate

src/sentry/notifications/platform/provider.py

@@ -8,7 +8,6 @@
 from sentry.notifications.platform.target import IntegrationNotificationTarget
 from sentry.notifications.platform.threading import ThreadContext
 from sentry.notifications.platform.types import (
-    NotificationCategory,
     NotificationData,
     NotificationProviderKey,
     NotificationTarget,
@@ -133,15 +132,17 @@
         return
 
     @classmethod
-    def get_renderer(
-        cls, *, data: NotificationData, category: NotificationCategory
-    ) -> type[NotificationRenderer[RenderableT]]:
+    def get_renderer(cls, *, data: NotificationData) -> type[NotificationRenderer[RenderableT]]:
         """
         Returns an instance of a renderer for a given notification, falling back to the default renderer.
         Override this to method to permit different renderers for the provider, though keep in mind
         that this may produce inconsistencies between notifications.
         """
-        return cls.default_renderer
+        # Imported here since the registry imports this module to type its registrations.
+        from sentry.notifications.platform.registry import renderer_registry
+
+        renderer = renderer_registry.get(provider_key=cls.key, source=data.source)
+        return renderer or cls.default_renderer
 
     @classmethod
     def is_available(cls, *, organization: RpcOrganizationSummary | None = None) -> bool:

src/sentry/notifications/platform/registry.py

@@ -1,11 +1,17 @@
 from __future__ import annotations
 
+from collections.abc import Callable
 from typing import Any
 
 from sentry.notifications.platform.provider import NotificationProvider
-from sentry.notifications.platform.types import NotificationTemplate
+from sentry.notifications.platform.renderer import NotificationRenderer
+from sentry.notifications.platform.types import (
+    NotificationProviderKey,
+    NotificationSource,
+    NotificationTemplate,
+)
 from sentry.organizations.services.organization.model import RpcOrganizationSummary
-from sentry.utils.registry import Registry
+from sentry.utils.registry import AlreadyRegisteredError, Registry
 
 
 class NotificationProviderRegistry(Registry[type[NotificationProvider[Any]]]):
@@ -33,5 +39,49 @@ def get_available(
         ]
 
 
+class NotificationRendererRegistry:
+    """
+    A registry for renderers which override a provider's default renderer. Renderers are keyed by
+    the provider they produce output for, and the notification sources they render.
+    """
+
+    def __init__(self) -> None:
+        self.registrations: dict[
+            tuple[NotificationProviderKey, NotificationSource], type[NotificationRenderer[Any]]
+        ] = {}
+
+    def register[RenderableT](
+        self, provider_key: NotificationProviderKey, *sources: NotificationSource
+    ) -> Callable[
+        [type[NotificationRenderer[RenderableT]]], type[NotificationRenderer[RenderableT]]
+    ]:
+        if not sources:
+            raise ValueError("At least one notification source is required to register a renderer")
+
+        def inner(
+            renderer: type[NotificationRenderer[RenderableT]],
+        ) -> type[NotificationRenderer[RenderableT]]:
+            for source in sources:
+                key = (provider_key, source)
+                if key in self.registrations:
+                    raise AlreadyRegisteredError(
+                        f"A registration already exists for {key}: {self.registrations[key]}"
+                    )
+                self.registrations[key] = renderer
+            return renderer
+
+        return inner
+
+    def get(
+        self, *, provider_key: NotificationProviderKey, source: NotificationSource
+    ) -> type[NotificationRenderer[Any]] | None:
+        """
+        Returns the registered renderer for the provider/source pair, or `None` if the provider has
+        no override and should fall back to its default renderer.
+        """
+        return self.registrations.get((provider_key, source))
+
+
 provider_registry = NotificationProviderRegistry()
 template_registry = Registry[type[NotificationTemplate[Any]]]()
+renderer_registry = NotificationRendererRegistry()

src/sentry/notifications/platform/renderer.py

@@ -2,7 +2,6 @@
 
 from sentry.notifications.platform.types import (
     NotificationData,
-    NotificationProviderKey,
     NotificationRenderedTemplate,
 )
 
@@ -17,8 +16,6 @@ class NotificationRenderer[RenderableT](Protocol):
     RenderableT is a type that matches the connected provider.
     """
 
-    provider_key: NotificationProviderKey
-
     @classmethod
     def render[DataT: NotificationData](
         cls, *, data: DataT, rendered_template: NotificationRenderedTemplate

src/sentry/notifications/platform/service.py

@@ -171,7 +171,7 @@ def render_template[RenderableT](
         provider: type[NotificationProvider[RenderableT]],
     ) -> RenderableT:
         rendered_template = template.render(data=data)
-        renderer = provider.get_renderer(data=data, category=template.category)
+        renderer = provider.get_renderer(data=data)
         return renderer.render(data=data, rendered_template=rendered_template)
 
     @staticmethod

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

@@ -32,13 +32,11 @@
 from sentry.notifications.platform.threading import ThreadContext
 from sentry.notifications.platform.types import (
     LinkTextBlock,
-    NotificationCategory,
     NotificationData,
     NotificationProviderKey,
     NotificationRenderedTemplate,
     NotificationSection,
     NotificationSectionType,
-    NotificationSource,
     NotificationTarget,
     NotificationTargetResourceType,
     NotificationTextBlock,
@@ -65,8 +63,6 @@ class SlackRenderable(TypedDict):
 
 
 class SlackRenderer(NotificationRenderer[SlackRenderable]):
-    provider_key = NotificationProviderKey.SLACK
-
     @classmethod
     def render[DataT: NotificationData](
         cls, *, data: DataT, rendered_template: NotificationRenderedTemplate
@@ -144,31 +140,6 @@ def is_available(cls, *, organization: RpcOrganizationSummary | None = None) ->
         # I currently view this as akin to a rollout or feature flag for the registry
         return False
 
-    @classmethod
-    def get_renderer(
-        cls, *, data: NotificationData, category: NotificationCategory
-    ) -> type[NotificationRenderer[SlackRenderable]]:
-        from sentry.notifications.platform.slack.renderers.issue import (
-            IssueSlackRenderer,
-        )
-        from sentry.notifications.platform.slack.renderers.metric_alert import (
-            SlackMetricAlertRenderer,
-        )
-        from sentry.notifications.platform.slack.renderers.seer import SeerSlackRenderer
-        from sentry.notifications.platform.slack.renderers.seer_agent_write_approval import (
-            SeerAgentWriteApprovalSlackRenderer,
-        )
-
-        if category == NotificationCategory.SEER:
-            if data.source == NotificationSource.SEER_AGENT_WRITE_APPROVAL:
-                return SeerAgentWriteApprovalSlackRenderer
-            return SeerSlackRenderer
-        if category == NotificationCategory.ISSUE:
-            return IssueSlackRenderer
-        if category == NotificationCategory.METRIC_ALERT:
-            return SlackMetricAlertRenderer
-        return cls.default_renderer
-
     @classmethod
     def send(
         cls,

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

@@ -2,19 +2,20 @@
 
 from sentry import eventstore
 from sentry.models.group import Group
+from sentry.notifications.platform.registry import renderer_registry
 from sentry.notifications.platform.renderer import NotificationRenderer
 from sentry.notifications.platform.slack.provider import SlackRenderable
 from sentry.notifications.platform.templates.issue import IssueNotificationData
 from sentry.notifications.platform.types import (
     NotificationData,
     NotificationProviderKey,
     NotificationRenderedTemplate,
+    NotificationSource,
 )
 
 
+@renderer_registry.register(NotificationProviderKey.SLACK, NotificationSource.ISSUE)
 class IssueSlackRenderer(NotificationRenderer[SlackRenderable]):
-    provider_key = NotificationProviderKey.SLACK
-
     @classmethod
     def render[DataT: NotificationData](
         cls, *, data: DataT, rendered_template: NotificationRenderedTemplate

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

@@ -7,19 +7,20 @@
 from sentry.integrations.slack.message_builder.incidents import get_started_at
 from sentry.integrations.slack.message_builder.types import INCIDENT_COLOR_MAPPING
 from sentry.integrations.slack.utils.escape import escape_slack_text
+from sentry.notifications.platform.registry import renderer_registry
 from sentry.notifications.platform.renderer import NotificationRenderer
 from sentry.notifications.platform.slack.provider import SlackRenderable
 from sentry.notifications.platform.templates.metric_alert import MetricAlertNotificationData
 from sentry.notifications.platform.types import (
     NotificationData,
     NotificationProviderKey,
     NotificationRenderedTemplate,
+    NotificationSource,
 )
 
 
+@renderer_registry.register(NotificationProviderKey.SLACK, NotificationSource.METRIC_ALERT)
 class SlackMetricAlertRenderer(NotificationRenderer[SlackRenderable]):
-    provider_key = NotificationProviderKey.SLACK
-
     @classmethod
     def render[DataT: NotificationData](
         cls, *, data: DataT, rendered_template: NotificationRenderedTemplate

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

@@ -21,6 +21,7 @@
     SectionBlock,
 )
 
+from sentry.notifications.platform.registry import renderer_registry
 from sentry.notifications.platform.renderer import NotificationRenderer
 from sentry.notifications.platform.slack.provider import SlackRenderable
 from sentry.notifications.platform.templates.seer import (
@@ -32,7 +33,9 @@
 )
 from sentry.notifications.platform.types import (
     NotificationData,
+    NotificationProviderKey,
     NotificationRenderedTemplate,
+    NotificationSource,
 )
 from sentry.seer.autofix.utils import AutofixStoppingPoint, CodingAgentProviderType
 
@@ -85,6 +88,15 @@ class AutofixStageConfig(TypedDict):
 }
 
 
+@renderer_registry.register(
+    NotificationProviderKey.SLACK,
+    NotificationSource.SEER_AUTOFIX_TRIGGER,
+    NotificationSource.SEER_AUTOFIX_ERROR,
+    NotificationSource.SEER_AUTOFIX_SUCCESS,
+    NotificationSource.SEER_AUTOFIX_UPDATE,
+    NotificationSource.SEER_AGENT_RESPONSE,
+    NotificationSource.SEER_AGENT_ERROR,
+)
 class SeerSlackRenderer(NotificationRenderer[SlackRenderable]):
     @classmethod
     def render[DataT: NotificationData](