Skip to content

Fix broker-managed Pinet follower startup mode#749

Merged
tmustier merged 3 commits into
mainfrom
fix/pinet-follower-startup-mode-748
May 18, 2026
Merged

Fix broker-managed Pinet follower startup mode#749
tmustier merged 3 commits into
mainfrom
fix/pinet-follower-startup-mode-748

Conversation

@tmustier
Copy link
Copy Markdown
Collaborator

Summary

  • detect broker-managed tmux follower launches and keep startup runtime off instead of honoring a persistent broker/single mode before /pinet follow
  • preserve explicit follower/autoFollow behavior when the launch is intentionally configured to auto-follow
  • hide recently disconnected/ghost agents from pinet agents by default, with include_ghosts: true for diagnostics

Closes #748.

Tests

  • pnpm vitest run slack-bridge/runtime-mode.test.ts slack-bridge/pinet-tools.test.ts
  • pnpm typecheck
  • pnpm lint
  • pnpm test
  • pre-push hook re-ran pnpm lint && pnpm typecheck && pnpm test

Notes

  • RCA confirmed: broker-managed tmux workers inherit persistent slack-bridge settings; when those settings request broker mode, session startup attempts transitionToRuntimeMode(..., "broker") before the broker sends /pinet follow, causing the duplicate-broker error even though follow subsequently succeeds.

@tmustier tmustier merged commit b75c4b7 into main May 18, 2026
1 check passed
@tmustier tmustier deleted the fix/pinet-follower-startup-mode-748 branch May 18, 2026 18:37
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.

Pinet follower startup emits duplicate-broker failure before /pinet follow

1 participant