Skip to content

refine thread tracking model to support parent-message tagging - #237

Merged
spashii merged 2 commits into
mainfrom
sam/update-thread-tracking-model
Aug 27, 2026
Merged

refine thread tracking model to support parent-message tagging#237
spashii merged 2 commits into
mainfrom
sam/update-thread-tracking-model

Conversation

@dembrane-sam-bot

Copy link
Copy Markdown
Contributor

What is this change?

Refines the Slack thread-tracking model in the daemon to support parent-message tagging:

  1. Parent Message Tagged (Case A): If the top-level parent message contains Sam's mention or is authored by the bot, Sam tracks this thread indefinitely and responds to untagged replies.
  2. Parent Message Untagged (Case B): If the parent message is untagged but Sam is mentioned later in the thread, Sam responds to that specific mention but does not track subsequent replies (they are ignored unless a session is currently running on the thread, in which case they steer). Sam automatically appends a friendly reminder to tag Sam in subsequent messages.

What did Sam notice that led to this?

Jor requested a more refined thread tracking compromise to prevent thread pollution on threads that weren't originally initiated with Sam's involvement.

Tier

Tier 3 (modifying src/runtime/daemon.py and routing logic).

Confidence

100% confident. Added unit tests covering all new routing paths and verified the entire 600+ test suite is completely green.

…r on api error, delete dead code, and align tests
@dembrane-sam-bot
dembrane-sam-bot added this pull request to the merge queue Aug 14, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 14, 2026
@spashii
spashii added this pull request to the merge queue Aug 27, 2026
Merged via the queue into main with commit f1ab1b3 Aug 27, 2026
2 checks passed
@spashii
spashii deleted the sam/update-thread-tracking-model branch August 27, 2026 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants