Skip to content

fix(slack): preserve interactive context and surface delegated profiles - #82519

Open
rooroosu wants to merge 7 commits into
NousResearch:mainfrom
rooroosu:fix/2026-08-09-slack-reconnect-race
Open

fix(slack): preserve interactive context and surface delegated profiles#82519
rooroosu wants to merge 7 commits into
NousResearch:mainfrom
rooroosu:fix/2026-08-09-slack-reconnect-race

Conversation

@rooroosu

@rooroosu rooroosu commented Aug 9, 2026

Copy link
Copy Markdown

Summary

  • bind approval and clarification callbacks to the originating Slack requester
  • allow delegate_task to select an existing Hermes profile without loading profile credentials or memory
  • relay profiled child start/final messages into the originating Slack thread with that profile's configured persona
  • preserve parent session and workspace isolation while hiding internal child tool/reasoning events

Verification

  • 278 focused delegation, async-delivery, profile-persona, and Slack adapter tests passed
  • ruff passed on all changed runtime and test files
  • live Slack smoke test: Luffy-only mention produced Nami and Chopper messages in the same thread with their configured names/icons

rooroosu and others added 7 commits August 9, 2026 23:08
@rooroosu Minwoo Park
feat(slack): route crew invocations to multiplex profiles
febdf3a 
One-app Slack crew routing with profile-scoped sessions, persisted thread affinity, customized user-invoked replies, and legacy /hermes crew fallback.
@rooroosu Minwoo Park
fix(slack): load crew profiles from YAML (#4)
195534a 
Co-authored-by: Minwoo Park <6597646+rooroosu@users.noreply.github.com>
@rooroosu Minwoo Park
fix(slack): switch crew profile in bound threads (#5)
7df9b67 
* fix(slack): switch crew profile in bound threads

* chore: map contributor email

---------

Co-authored-by: Minwoo Park <minwoopark@talentedmusicians.co>
Minwoo Park
fix(slack): avoid racing native socket reconnect
f802c66
Minwoo Park
fix(slack): authorize approval prompt requester
3bb864f
Minwoo Park
fix(slack): bind clarify prompts to requester
0b59841
Minwoo Park
feat(slack): surface specialist delegation in threads
3a4ac55
@rooroosu
rooroosu force-pushed the fix/2026-08-09-slack-reconnect-race branch from ccbff0c to 3a4ac55 Compare August 9, 2026 14:10
@alt-glitch alt-glitch added type/feature New feature or request comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint comp/cli CLI entry point, hermes_cli/, setup wizard comp/gateway Gateway runner, session dispatch, delivery comp/plugins Plugin system and bundled plugins tool/delegate Subagent delegation platform/slack Slack app adapter P3 Low — cosmetic, nice to have needs-decision Awaiting maintainer decision before any implementation sweeper:risk-message-delivery Sweeper risk: may drop, duplicate, misroute, or suppress messages sweeper:risk-session-state Sweeper risk: may lose/corrupt/mis-associate session or context state labels Aug 9, 2026
@Xipong Xipong mentioned this pull request Aug 13, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint comp/cli CLI entry point, hermes_cli/, setup wizard comp/gateway Gateway runner, session dispatch, delivery comp/plugins Plugin system and bundled plugins needs-decision Awaiting maintainer decision before any implementation P3 Low — cosmetic, nice to have platform/slack Slack app adapter sweeper:risk-message-delivery Sweeper risk: may drop, duplicate, misroute, or suppress messages sweeper:risk-session-state Sweeper risk: may lose/corrupt/mis-associate session or context state tool/delegate Subagent delegation type/feature New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rooroosu @alt-glitch