Receipt · eb14fff2-0
Slack bot-message filter is inverted: BOT_ID = "null" string check rejects real users
bughighclosed in 4b9b492closed in 1 hour
repo 6f7fc663·PR #2·reviewed 2 days ago·2 days ago
The finding
- .github/workflows/messages.yml:105-117
`jq -r '.bot_id // empty'` outputs the empty string when `bot_id` is absent (i.e., the message is from a real user), and outputs the actual bot id string when it is a bot. The code then checks `[ "$BOT_ID" = "null" ]` to admit the message. This condition is never true for user messages (where BOT_ID is empty) and never true for bot messages (where BOT_ID is a non-null id), so the Slack branch effectively never dispatches anything. The intent (per comments) is to skip bot messages, so the check should be `[ -z "$BOT_ID" ]`. The Telegram and Discord branches are correctly written, so this is an inconsistency that silently disables Slack ingestion.
Fix
Change the guard to `[ -z "$BOT_ID" ]` (consistent with the Telegram chat-id check pattern). Also use `jq -r '.bot_id // ""'` consistently and verify with a unit-style shell test.
Agent attribution
The agents that produced this receipt — both reviewer models had to flag this independently for the agreement gate to emit it.
anthropic
gpt-5
76.3s · error
openai
claude-opus-4-7
110.8s · error
Total
wall-clock review time · est. inference cost
110.8s · $0.40
Sweeper
closed at SHA 4b9b492
closed in 1 hour
internal review id · eb14fff2
Third-party witnesses
Everything below lives on GitHub's event log, not ours. Click any link to verify the SHA, the timestamp, and the surrounding context for yourself.
Closure receipt comment
https://github.com/AntFleet/aeon-bench/pull/2#issuecomment-4476009572Original review comment
https://github.com/AntFleet/aeon-bench/pull/2#issuecomment-4475356633The pull request
https://github.com/AntFleet/aeon-bench/pull/2