Skip to content

Commit 38d0c42

Browse files
committed
style: keep original comment with possibilities of revisiting
1 parent 2cbb1af commit 38d0c42

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

slack_bolt/request/internals.py

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -215,11 +215,9 @@ def extract_channel_id(payload: Dict[str, Any]) -> Optional[str]:
215215

216216

217217
def extract_thread_ts(payload: Dict[str, Any]) -> Optional[str]:
218-
# This utility only extracts thread_ts for assistant events to avoid breaking existing say() behavior.
219-
# For non-assistant events, thread_ts is intentionally NOT extracted into context because:
220-
# - say() uses context.thread_ts to decide where to post messages
221-
# - Existing apps may expect say() to post to the channel, not the thread
222-
# - Changing this would be a breaking change for existing apps
218+
# This utility initially supports only the use cases for AI assistants, but it may be fine to add more patterns.
219+
# That said, note that thread_ts is always required for assistant threads, but it's not for channels.
220+
# Thus, blindly setting this thread_ts to say utility can break existing apps' behaviors.
223221
#
224222
# The BoltAgent class handles non-assistant thread_ts separately by reading from the event directly,
225223
# allowing it to work correctly without affecting say() behavior.

0 commit comments

Comments
 (0)