fix(zulip-monitor): stream alert body carries real content; WARN on delivery failure #68

Merged

abiba-bot merged 3 commits from fm/zulip-monitor-stream-body-20260909 into master 2026-09-09 04:21:37 +00:00

Conversation 0 Commits 3 Files Changed 3

+7 -5

3 Commits

Author	SHA1	Message	Date
root	7ff7ce5b33	no-mistakes(document): docs: fix stale replaced-claim and Telegram header comment	2026-09-09 03:27:09 +00:00
root	36f218e255	chore(agents): replace CLAUDE.md symlink with canonical @AGENTS.md pointer file ... fm-ensure-agents-md.sh converted the tracked AGENTS.md symlink into the real-file pointer form, removing the dangling-symlink hazard.	2026-09-09 02:48:17 +00:00
root	947e8b24e0	fix(zulip-monitor): stream alert body now carries real content, plain & params, WARN on delivery failure ... The #agent-hub / zulip-health stream post in notify() encoded content with python quote(str()) (always empty), so every stream alert posted empty content and Zulip rejected it silently behind '|| true'. The merged body also used backslash-escaped ampersands inside double quotes, which curl transmits literally (type=stream\ -> Zulip 400 'Invalid type'). Stream post now percent-encodes the real message text by piping it through the encoder (locale-proof: quote_from_bytes on stdin.buffer), uses plain & separators, and on failure appends one WARN line to the monitor log with the curl exit status instead of silently swallowing it. Delivery failure stays non-fatal and unretried. DM path, probes, alive rule, exit codes and log format unchanged.	2026-09-09 02:47:44 +00:00