Skip to content

feat(chat): Hazel channels as alert destinations through the bot - #2027

Merged
Makisuo merged 1 commit into
mainfrom
feat/hazel-bot-destinations
Oct 31, 2026
Merged

Makisuo merged 1 commit into
mainfrom
feat/hazel-bot-destinations

feat(chat): Hazel channels as alert destinations through the bot

5b4006d
Select commit
Loading
Failed to load commit list.
Maple Review Bot / Maple / review succeeded Oct 31, 2026 in 1m 0s

Confidence 10/10 · No issues found

🟢 Confidence 10/10 · safe to merge. A contained connector change: one new reader through the existing client-span and error-mapping helpers, pinned by three transport tests.
no findings · tests covered · risk medium · 1/1 new units observable

Hazel's transport now lists the bot's channels from GET /organizations/{id}/bot-channels, so Hazel channels become ordinary chat alert destinations; the manifest description and connector README follow. The change reuses the existing request, decode and failure-mapping helpers and is safe to merge.

  • hazelOutbound.destinations reads Hazel's bot-channels endpoint instead of answering empty
  • Manifest description and README now cover alerts and the channels:read scope

Before merge

  • Manual · Ship Hazel's bot-channels endpoint (MapleTechLabs/hazel#372) before this deploys; without it the picker fails
  • Manual · Add channels:read to the US and EU Hazel bots, or every channel listing answers auth

Production impact

Telemetry this change adds and removes (1)
  • ➕ span name Hazel.destinations · packages/chat-platform/src/connectors/hazel/outbound.ts:365
What was checked
  • New call goes through attempt, keeping the client span, peer.service and route template (connectors/hazel/outbound.ts:366)
  • Unknown reply fields are ignored, matching the connector's existing decode pattern (outbound.ts:89 vs :72)
  • Host paths are connector-agnostic, so no Hazel-specific alert wiring is missing (ChatAlertPoster.ts:257, ChatWorkspaceService.ts:680)
Observability coverage: 1 of 1 changes observable
Change Kind Observable Evidence
Hazel bot-channels listing (destinations) outbound HTTP call yes connectors/hazel/outbound.ts:366 goes through attempt, which sets peer.service, server.address and the route template on a client span

5b4006d · Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.