fix: update alert type formatting to handle enum names in Slack messages

Author: mjmendo

litellm/integrations/SlackAlerting/slack_alerting.py

@@ -1367,7 +1367,9 @@ async def send_alert(
         # Get the current timestamp
         current_time = datetime.now().strftime("%H:%M:%S")
         _proxy_base_url = os.getenv("PROXY_BASE_URL", None)
-        alert_type_formatted = f"Alert type: `{alert_type}`\n"
+        # Use .name if it's an enum, otherwise use as is
+        alert_type_name = getattr(alert_type, 'name', alert_type)
+        alert_type_formatted = f"Alert type: `{alert_type_name}`"
         if alert_type == "daily_reports" or alert_type == "new_model_added":
             formatted_message = alert_type_formatted + message
         else:

tests/test_litellm/integrations/SlackAlerting/test_slack_alerting.py

@@ -188,11 +188,11 @@ def test_alert_type_in_formatted_message(self, mock_datetime):
         current_time = "12:34:56"
 
         # Test the specific formatting logic we're interested in
-        alert_type_formatted = f"Alert type: `{alert_type}`\n"
+        alert_type_formatted = f"Alert type: `{alert_type.name}`\n"
         formatted_message = f"{alert_type_formatted}\n Level: `{level}`\nTimestamp: `{current_time}`\n\nMessage: {message}"
 
         # Verify alert_type is in the formatted message as expected
-        self.assertIn("Alert type: `AlertType.llm_exceptions`", formatted_message)
+        self.assertIn("Alert type: `llm_exceptions`", formatted_message)
         self.assertIn("Level: `Medium`", formatted_message)
         self.assertIn("Timestamp: `12:34:56`", formatted_message)
         self.assertIn("Message: Test alert message", formatted_message)