Skip to content

feat: add Slack thread mention-only routing - #56890

Closed
RoySRose wants to merge 1 commit into
NousResearch:mainfrom
RoySRose:feat/slack-thread-mention-only-clean
Closed

feat: add Slack thread mention-only routing#56890
RoySRose wants to merge 1 commit into
NousResearch:mainfrom
RoySRose:feat/slack-thread-mention-only-clean

Conversation

@RoySRose

@RoySRose RoySRose commented Jul 2, 2026

Copy link
Copy Markdown

Summary

  • add Slack thread-local mention-only flags controlled by natural-language on/off phrases
  • block active-session, mentioned-thread, bot-reply, free-response, and mention-pattern wakeups while the flag is active
  • prune flags by session idle TTL/default 24h and cap stored entries

Verification

  • uv run --extra dev python -m pytest tests/gateway/test_slack_thread_mention_only.py tests/gateway/test_slack_mention.py -q -o 'addopts=' → 72 passed
  • uv run --extra dev python -m pytest tests/gateway/test_slack.py tests/gateway/test_slack_thread_mention_only.py tests/gateway/test_slack_mention.py -q -o 'addopts=' → 288 passed
  • uv run --extra dev ruff check plugins/platforms/slack/adapter.py tests/gateway/test_slack_thread_mention_only.py → All checks passed
  • git diff --check -- plugins/platforms/slack/adapter.py tests/gateway/test_slack_thread_mention_only.py → clean

@alt-glitch alt-glitch added type/feature New feature or request platform/slack Slack app adapter comp/plugins Plugin system and bundled plugins P3 Low — cosmetic, nice to have labels Jul 2, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

This was generated by AI during triage.

Related: #24848 (open — routes plain replies from mentioned thread parents). Same Slack thread-mention-routing family but a different mechanism: this PR adds an opt-in per-thread "mention-only" silencing flag toggled by natural-language phrases, whereas #24848 fixes reply routing for already-mentioned parents. Prior strict_mention attempts (#12258, #14117, #44800) are closed. Not a duplicate — flagging the cluster so a maintainer can decide how these compose.

@RoySRose

RoySRose commented Jul 2, 2026

Copy link
Copy Markdown
Author

Closed: opened from a misinterpreted continuation. The Slack adapter change is not the active Waka approval task.

@RoySRose RoySRose closed this Jul 2, 2026
@RoySRose
RoySRose deleted the feat/slack-thread-mention-only-clean branch July 2, 2026 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/plugins Plugin system and bundled plugins P3 Low — cosmetic, nice to have platform/slack Slack app adapter type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants