Skip to content

feat(gateway): always send home-channel startup notification on init - #32147

Closed
Gerkinfeltser wants to merge 1 commit into
NousResearch:mainfrom
Gerkinfeltser:feat/gateway-startup-notification
Closed

feat(gateway): always send home-channel startup notification on init#32147
Gerkinfeltser wants to merge 1 commit into
NousResearch:mainfrom
Gerkinfeltser:feat/gateway-startup-notification

Conversation

@Gerkinfeltser

Copy link
Copy Markdown

Previously _send_home_channel_startup_notifications() was gated behind restart_notification_pending, so '♻️ Gateway online' only fired after a /restart command. Now it fires on every gateway startup — including crashes and systemctl restart — so home channels are notified whenever the gateway comes back up.

Changes:

  • Removes the guard that restricted notifications to restart-only scenarios
  • Preserves skip_home_targets deduplication for restart requester's direct completion notice
  • Startup notification now fires on every gateway init, not just after /restart

Previously _send_home_channel_startup_notifications() was gated behind
restart_notification_pending, so '♻️ Gateway online' only fired after
a /restart command. Now it fires on every gateway startup — including
crashes and systemctl restart — so home channels are notified whenever
the gateway comes back up.
@alt-glitch

Copy link
Copy Markdown
Collaborator

Overlaps with open PR #27278 (same unconditional approach) and #24271 (opt-in approach). Related feature request: #27870.

@alt-glitch alt-glitch added type/feature New feature or request P3 Low — cosmetic, nice to have comp/gateway Gateway runner, session dispatch, delivery labels May 25, 2026
@Gerkinfeltser

Copy link
Copy Markdown
Author

Closing � this change already landed on main in commit c9e6c1d (feat(gateway): always send home-channel startup notification on init). The PR will conflict if merged since both sides make the same change. Not yet in a tagged release (post v0.16.0), so the fix will ship in v0.17.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/gateway Gateway runner, session dispatch, delivery P3 Low — cosmetic, nice to have type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants