slack: light working pill immediately when a Slack human relays a prompt#85
Open
rogeriochaves wants to merge 1 commit into
Open
slack: light working pill immediately when a Slack human relays a prompt#85rogeriochaves wants to merge 1 commit into
rogeriochaves wants to merge 1 commit into
Conversation
…a prompt The bridge already suppresses the daemon's UserPromptSubmit echo for Slack-relayed prompts (they're already in the channel as the human's message), which leaves nothing bot-authored for assistant.threads.setStatus to attach to. Result: the channel looks dead for the 10-20s between the human typing and the agent's first text post, which is the natural pill anchor for the existing rollout-tail flow. Post a single eyes emoji as a minimal bot-authored anchor right after we paste into tmux, then setStatus the working pill onto it. The agent's first real text post lands as a new thread root via the existing post loop, draining any accumulated tool calls under the eyes and moving the pill onto the narrative reply — same shape as the existing scheduled-nudge path. setStatus only works on app-authored thread roots (Slack rejects event.ts of a user-channel message with invalid_thread_ts), so pinning the pill directly to the human's own message is not an option. The single- emoji anchor is the lowest-noise placeholder that still gives the pill something valid to anchor to.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
assistant.threads.setStatuson that post.invalid_thread_tson user-channel ts, see slack-setstatus-app-authored-threads-only memory from the earlier revert), so pinning the pill to the human's own message is not viable. The single-emoji anchor is the lowest-noise placeholder that still gives setStatus something valid to attach to.Test plan
agents-slack-bridge.serviceon this branch: typing a prompt in any agent channel surfaces an eyes-emoji bot post + working pill within a second, and the pill moves onto the agent's first real text post when it lands.invalid_thread_tserrors injournalctl -u agents-slack-bridge.service.