Skip to content

feat(gateway): scope restart notifications to channel allowlists - #56015

Closed
walshg3 wants to merge 1 commit into
NousResearch:mainfrom
walshg3:greg/gateway-restart-notification-allowlist
Closed

feat(gateway): scope restart notifications to channel allowlists#56015
walshg3 wants to merge 1 commit into
NousResearch:mainfrom
walshg3:greg/gateway-restart-notification-allowlist

Conversation

@walshg3

@walshg3 walshg3 commented Jul 1, 2026

Copy link
Copy Markdown

Summary

  • Add optional per-platform gateway_restart_notification_channels allowlists for gateway lifecycle/restart notifications.
  • Apply the allowlist anywhere restart lifecycle messages are emitted: active-session shutdown notices, home-channel shutdown notices, planned restart confirmations, and startup notifications.
  • Bridge the key from both top-level platform config and nested gateway.platforms.<platform> config, and document the config example.

Default behavior is unchanged: if gateway_restart_notification_channels is unset, the platform behaves exactly as it did before. gateway_restart_notification: false remains the coarse per-platform kill switch.

Why

Multi-platform gateways often have several active chats, but restart lifecycle pings are operator noise. Users should be able to keep the signal in one ops/home channel without spraying Telegram, Discord, Slack, etc. chats that are not meant to receive operational lifecycle messages.

This is a narrower alternative/complement to the existing open restart-notification PRs:

This PR keeps the existing delivery targets, but gates them through an explicit chat/channel ID allowlist. That makes the behavior easy to reason about and keeps current behavior as the unset default.

Test plan

python -m pytest tests/gateway/test_restart_notification.py tests/gateway/test_config.py -q -o 'addopts='

Result: 109 passed, 2 warnings.

@walshg3

walshg3 commented Jul 1, 2026

Copy link
Copy Markdown
Author

Closing this draft because the related open PRs #52584 and #31046 look sufficient for this need. I’ll track those instead.

@walshg3 walshg3 closed this Jul 1, 2026
@walshg3
walshg3 deleted the greg/gateway-restart-notification-allowlist branch July 1, 2026 02:16
@alt-glitch alt-glitch added type/feature New feature or request P3 Low — cosmetic, nice to have comp/gateway Gateway runner, session dispatch, delivery sweeper:risk-message-delivery Sweeper risk: may drop, duplicate, misroute, or suppress messages labels Jul 1, 2026
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 sweeper:risk-message-delivery Sweeper risk: may drop, duplicate, misroute, or suppress messages type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants