fix(slack): pass thread_ts through approval buttons and standalone sends

[shivasymbl]

Summary

Thread-aware interactive buttons for all Slack user-input prompts. Responses stay in-thread instead of creating new top-level messages.

• Add thread_id to _send_slack() — the only platform missing it
• Encode thread_ts in approval button values for thread propagation
• New: send_update_prompt() — Yes/No Block Kit buttons for Slack (was Discord/Telegram only)
• New: send_choice_prompt() — Generic multi-option prompt (up to 25 options)

Changes

File	Change
tools/send_message_tool.py	Add thread_id to _send_slack(), forward from _send_to_platform()
gateway/platforms/slack.py	_encode/_decode_button_value helpers, thread_ts in approval buttons, new send_update_prompt(), new send_choice_prompt() with action handlers
gateway/run.py	Wire send_update_prompt with metadata for thread context
tests/gateway/test_slack_approval_buttons.py	346 lines added — encoding tests, update prompt tests, choice prompt tests
tests/tools/test_send_message_tool.py	103 lines added — _send_slack thread_id tests

5 files changed, 865 insertions(+), 10 deletions(-)

New capabilities

Method	Description
send_exec_approval()	Fixed — thread_ts encoded in button values
send_update_prompt()	New — Yes/No Block Kit buttons, thread-aware
send_choice_prompt()	New — generic multi-option prompt, up to 25 choices, file-based resolution
_send_slack()	Fixed — thread_ts for all standalone deliveries

send_choice_prompt is fully generic — cron confirmations, routing decisions, model picks, topic selection:

await adapter.send_choice_prompt(
    chat_id=channel,
    prompt="Which blog topic?",
    choices=[("Digital Labor ROI", "dl_roi"), ("Customer Zero", "cz")],
    session_key="choice-001",
    metadata={"thread_id": thread_ts}
)

Test plan

• 38 targeted tests pass (0 failures)
• Button value encoding round-trips with session keys containing :
• Legacy button values (no ||) decode correctly — backward compatible
• _send_slack includes thread_ts when provided, omits when None
• Update prompt Yes/No buttons render and resolve correctly
• Choice prompt with multiple options renders and resolves correctly
• All handlers registered with Slack app action listener

Fixes #9394

🤖 Generated with Claude Code

[houko]

Good fix — properly encoding thread_ts in button values to preserve thread routing is important. Backward compatibility with split logic is a nice touch. 👍

[teknium1]

Thanks for the detailed Slack investigation. This is an automated hermes-sweeper review; the reported thread-routing behavior is now implemented on current main.

• Commit 74e845c00 (fix(slack): pass thread_ts in standalone send_message tool path) fixed the standalone-send defect.
• The current migrated sender preserves that behavior at plugins/platforms/slack/adapter.py:4351-4352, adding Slack thread_ts from thread_id.
• Slack approval prompts already post in-thread at plugins/platforms/slack/adapter.py:3257-3318. Their click handler updates that same prompt and resolves the existing session at plugins/platforms/slack/adapter.py:3655-3682, so it has no separate follow-up message that needs thread data encoded in the button value.
• The Slack adapter moved from gateway/platforms/slack.py to plugins/platforms/slack/adapter.py in 1a3806605, so the remaining PR implementation targets a superseded layout.