✏️ more review fixes

Author: leeandher

src/sentry/integrations/slack/requests/event.py

@@ -75,7 +75,7 @@ def user_id(self) -> str:
     def thread_ts(self) -> str:
         if self.is_assistant_thread_event:
             return self.dm_data.get("assistant_thread", {}).get("thread_ts", "")
-        return self.dm_data.get("ts", "")
+        return self.dm_data.get("thread_ts") or self.dm_data.get("ts", "")
 
     @property
     def has_assistant_scope(self):

src/sentry/integrations/slack/webhooks/event.py

@@ -46,6 +46,24 @@
 
 _logger = logging.getLogger(__name__)
 
+_SEER_STARTING_PROMPTS = [
+    {
+        "title": "Summarize recent issues",
+        "message": "What are the most important unresolved issues in my projects right now?",
+    },
+    {
+        "title": "Investigate an error",
+        "message": "Help me investigate what's causing errors in my project.",
+    },
+    {
+        "title": "Explain a stack trace",
+        "message": "Can you explain the root cause of this stack trace?",
+    },
+    {
+        "title": "Find performance bottlenecks",
+        "message": "What are the slowest endpoints or pages in my projects?",
+    },
+]
 _SEER_LOADING_MESSAGES = [
     "Digging through your errors...",
     "Sifting through stack traces...",
@@ -544,24 +562,7 @@ def on_assistant_thread_started(self, slack_request: SlackEventRequest) -> Respo
                     channel_id=channel_id,
                     thread_ts=thread_ts,
                     title="Hi there! I'm Seer, Sentry's AI assistant. How can I help?",
-                    prompts=[
-                        {
-                            "title": "Summarize recent issues",
-                            "message": "What are the most important unresolved issues in my projects right now?",
-                        },
-                        {
-                            "title": "Investigate an error",
-                            "message": "Help me investigate what's causing errors in my project.",
-                        },
-                        {
-                            "title": "Explain a stack trace",
-                            "message": "Can you explain the root cause of this stack trace?",
-                        },
-                        {
-                            "title": "Find performance bottlenecks",
-                            "message": "What are the slowest endpoints or pages in my projects?",
-                        },
-                    ],
+                    prompts=_SEER_STARTING_PROMPTS,
                 )
             except Exception:
                 _logger.exception(

src/sentry/seer/entrypoints/slack/messaging.py

@@ -278,7 +278,7 @@ def send_identity_link_prompt(
     from sentry.integrations.slack.views.link_identity import build_linking_url
 
     # TODO(leander): We'll need to revisit the UX around linking. We can't pass threads here so while
-    # the linking start message is correctly located and ephemeral, the success message afterwards is not.abs
+    # the linking start message is correctly located and ephemeral, the success message afterwards is not.
     # By omitting the response_url here, it will arrive as a DM, but it doesn't accept threads so this is the best we can do for now.
     associate_url = build_linking_url(
         integration=integration,