refactor: avoid adding to context in this changeset

Author: zimeg

slack_bolt/context/base_context.py

@@ -18,7 +18,6 @@ class BaseContext(dict):
         "actor_team_id",
         "actor_user_id",
         "channel_id",
-        "ts",
         "thread_ts",
         "response_url",
         "matches",
@@ -111,11 +110,6 @@ def channel_id(self) -> Optional[str]:
         """The conversation ID associated with this request."""
         return self.get("channel_id")
 
-    @property
-    def ts(self) -> Optional[str]:
-        """The message timestamp associated with this request."""
-        return self.get("ts")
-
     @property
     def thread_ts(self) -> Optional[str]:
         """The conversation thread's ID associated with this request."""

slack_bolt/kwargs_injection/async_utils.py

@@ -89,13 +89,9 @@ def build_async_required_kwargs(
     if "agent" in required_arg_names:
         from slack_bolt.agent.async_agent import AsyncBoltAgent
 
-        # For thread_ts, we check multiple sources:
-        # 1. context.thread_ts - populated for assistant events
-        # 2. event.thread_ts - for non-assistant events in a thread (e.g., app_mention in thread)
-        # 3. context.ts - fallback to the message timestamp
-        # We read from event directly to avoid changing context.thread_ts which would affect say() behavior
+        # Resolve thread_ts: assistant events set context.thread_ts, otherwise read from event
         event = request.body.get("event", {})
-        thread_ts = request.context.thread_ts or event.get("thread_ts") or request.context.ts
+        thread_ts = request.context.thread_ts or event.get("thread_ts") or event.get("ts")
 
         all_available_args["agent"] = AsyncBoltAgent(
             client=request.context.client,

slack_bolt/kwargs_injection/utils.py

@@ -88,13 +88,9 @@ def build_required_kwargs(
     if "agent" in required_arg_names:
         from slack_bolt.agent.agent import BoltAgent
 
-        # For thread_ts, we check multiple sources:
-        # 1. context.thread_ts - populated for assistant events
-        # 2. event.thread_ts - for non-assistant events in a thread (e.g., app_mention in thread)
-        # 3. context.ts - fallback to the message timestamp
-        # We read from event directly to avoid changing context.thread_ts which would affect say() behavior
+        # Resolve thread_ts: assistant events set context.thread_ts, otherwise read from event
         event = request.body.get("event", {})
-        thread_ts = request.context.thread_ts or event.get("thread_ts") or request.context.ts
+        thread_ts = request.context.thread_ts or event.get("thread_ts") or event.get("ts")
 
         all_available_args["agent"] = BoltAgent(
             client=request.context.client,

slack_bolt/request/async_internals.py

@@ -15,7 +15,6 @@
     extract_actor_team_id,
     extract_actor_user_id,
     extract_thread_ts,
-    extract_ts,
 )
 
 
@@ -46,9 +45,6 @@ def build_async_context(
     channel_id = extract_channel_id(body)
     if channel_id:
         context["channel_id"] = channel_id
-    ts = extract_ts(body)
-    if ts:
-        context["ts"] = ts
     thread_ts = extract_thread_ts(body)
     if thread_ts:
         context["thread_ts"] = thread_ts

slack_bolt/request/internals.py

@@ -245,18 +245,6 @@ def extract_thread_ts(payload: Dict[str, Any]) -> Optional[str]:
     return None
 
 
-def extract_ts(payload: Dict[str, Any]) -> Optional[str]:
-    """Extract the message timestamp from an event payload."""
-    event = payload.get("event", {})
-    # Direct ts on the event (e.g., app_mention, message)
-    if event.get("ts") is not None:
-        return event["ts"]
-    # message_changed events have ts in the message
-    if event.get("message", {}).get("ts") is not None:
-        return event["message"]["ts"]
-    return None
-
-
 def extract_function_execution_id(payload: Dict[str, Any]) -> Optional[str]:
     if payload.get("function_execution_id") is not None:
         return payload.get("function_execution_id")
@@ -307,9 +295,6 @@ def build_context(context: BoltContext, body: Dict[str, Any]) -> BoltContext:
     channel_id = extract_channel_id(body)
     if channel_id:
         context["channel_id"] = channel_id
-    ts = extract_ts(body)
-    if ts:
-        context["ts"] = ts
     thread_ts = extract_thread_ts(body)
     if thread_ts:
         context["thread_ts"] = thread_ts