AntFleet

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.

← back to all receipts