AntFleet

Disagreement · 5bbcd2d3-anthropic-3

sendReaction does not record outbound channel activity

solo Opus
repo 5149da9d·PR #3·reviewed 2 days ago

Opus finding

sendReaction does not record outbound channel activity

lowmaintainabilitymedium
  • src/web/inbound/send-api.ts:71-89
sendMessage and sendPoll both call recordChannelActivity to update the outbound activity heartbeat, but sendReaction (which also sends a message via the same socket) does not. This causes channel-activity observability/heartbeats to under-count outbound traffic, which can produce false 'idle channel' signals when the bot only reacts. Since the comment on getWebAuthAgeMs explicitly mentions observability for heartbeats, the omission is likely unintentional.

Recommendation

Call recordChannelActivity({channel:'whatsapp', accountId: params.defaultAccountId, direction:'outbound'}) after sendReaction (and consider sendComposingTo).

Other reviewer

The other reviewer flagged nothing in this file/line range.

Why this didn't post

This finding didn't meet AntFleet's unanimous agreement threshold. Both frontier models review every PR independently; only findings they both flag with the same severity and category are posted to the PR. This one fell through.

read the methodology →

From the same review

These findings passed the unanimous gate on the same PR review. The disagreement above was filtered out; the findings below were posted.

← back to all disagreementsview public receiptssee unanimous findings + anatomies →
Tweet ↗