Skip to content

fix(slack): restore immediate status in existing threads - #3612

Open
just-cameron wants to merge 1 commit into
mainfrom
overlord/fix-slack-immediate-status
Open

fix(slack): restore immediate status in existing threads#3612
just-cameron wants to merge 1 commit into
mainfrom
overlord/fix-slack-immediate-status

Conversation

@just-cameron

Copy link
Copy Markdown
Contributor

Overlord (agent-c2adbf5c-8419-4211-8cd8-3740db164974)

Summary

  • show the native Slack assistant status as soon as a follow-up turn is queued in an existing thread, rather than waiting for the first tool or final reply
  • activate generic thinking copy only while status is idle, preserving concrete tool titles when another message queues mid-turn
  • keep known root-shaped retry/continuation events quiet after the bot has already replied

Test plan

  • Regression test fails on main and passes with this patch
  • bun test src/channels/slack/status-controller.test.ts (16 passed)
  • bun test src/channels/slack/media.test.ts (18 passed; isolated control for the Slack mock boundary)
  • bun run check (12/12 passed)

👾 Generated with Letta Code

Letta Code (agent-c2adbf5c-8419-4211-8cd8-3740db164974)

Queue established Slack thread turns with an idle-only native thinking status so the thread shows immediate feedback without clobbering active tool titles. Keep known root-shaped retries quiet after a visible reply while concrete progress and terminal clears retain ownership.

Co-authored-by: Cameron Pfiffer <cameron@pfiffer.org>

👾 Generated with [Letta Code](https://letta.com)

Co-Authored-By: Letta Code <noreply@letta.com>
@just-cameron
just-cameron requested a review from cpacker as a code owner July 31, 2026 23:10
@cursor

cursor Bot commented Jul 31, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

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