Skip to content

Fix Slack thread transfer context sync#758

Closed
gugu91 wants to merge 1 commit into
mainfrom
fix/slack-thread-ownership-transfer-756
Closed

Fix Slack thread transfer context sync#758
gugu91 wants to merge 1 commit into
mainfrom
fix/slack-thread-ownership-transfer-756

Conversation

@gugu91
Copy link
Copy Markdown
Owner

@gugu91 gugu91 commented May 18, 2026

Summary

  • include Slack source/channel context in explicit Pinet thread-transfer metadata
  • sync transferred Slack thread context into follower thread cache before delivering A2A prompts
  • keep safe semantics: only broker-explicit Slack transfers with channel context are cached; non-Slack/missing-channel cases remain rejected/ignored

Fixes #756

Root cause

transfer_thread_id updated broker ownership state, but the A2A assignment only carried the Slack thread timestamp. Follower runtime classifies A2A messages separately from normal Slack inbox entries, so workers never persisted the Slack channel/source context needed by slack_send/Pinet Slack delivery. Once ownership was explicitly transferred, broker-side relays correctly failed with "already owned by another agent", leaving broker and worker without a usable visible-thread reply path.

Validation

  • pnpm vitest run slack-bridge/helpers.test.ts slack-bridge/pinet-mesh-ops.test.ts
  • pnpm lint
  • pnpm typecheck
  • pnpm test
  • pre-push hook reran pnpm lint && pnpm typecheck && pnpm test successfully

PR author/account: gugu91

@gugu91
Copy link
Copy Markdown
Owner Author

gugu91 commented May 18, 2026

Closing as duplicate/superseded by #757.

I compared #757 against this backup PR. #757 covers the material fix here (broker transfer metadata includes Slack source/channel, follower runtime hydrates transferred Slack context before A2A delivery) and adds useful visible recovery guidance / slack_send hints that this PR does not. I do not see a material gap that needs to be carried from #758.

Validation for #758 was clean, but #757 should be the coordinated lane for #756.

@gugu91 gugu91 closed this May 18, 2026
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.

Bug: Slack thread ownership transfer can leave broker and workers unable to post

1 participant