GOBIGnINTERRUPT

v1.1.6 (2026-05-01)

Full Changelog Previous Releases

• v1.1.6 - red "?" RCA: cross-realm name mismatch + SendQuery guard
  User reported red "?" addon-presence badge always shows. Codex audit +
  my own grep converged on the same root cause and a sibling bug.
  ** Bug 1 (high) - cross-realm name mismatch **
  peerSeen is keyed on the SHORT name. The CHAT_MSG_ADDON handler strips
  realm before storing:
  local short = sender and (sender:match("^([%-]+)") or sender)
  if short then peerSeen[short] = GetTime() end
  But two sites looked up against the LONG name:
  * senderToUnit: compared n == short where n was raw UnitName(u).
  For cross-realm party members (Connected Realms / LFG cross-server
  groups), UnitName(partyN) returns "Name-Realm". Short was already
  "Name". The compare missed; senderToUnit returned nil; messages
  were dropped.
  * PeerHasAddon: looked up peerSeen[UnitName(unit)]. Same realm-suffix
  miss, fell through to the false branch after PEER_GRACE expired.
  Net effect: any cross-realm peer running GOBIGnINTERRUPT was silently
  flagged "no addon" -> red "?" badge on every cross-realm row, on every
  0.5s tick, forever. Same-realm peers worked. Made the badge appear to
  "always show" in any LFG / Battle.net cross-server group.
  Fix: shared shortName(n) helper, applied at every UnitName lookup site
  in CDComm.lua (senderToUnit, PeerHasAddon, DumpPeerPresence).
  ** Bug 2 (medium) - SendQuery stamps lastQueryAt even when send() bailed **
  SendQuery did lastQueryAt = GetTime() BEFORE calling send(). send()
  silently returns when comm is disabled (DB.comm.enabled = false) or the
  player is not in a group, so lastQueryAt could be set to "now" without
  any Q actually broadcasting.
  PeerHasAddon's false branch reads:
  if lastQueryAt > 0 and (GetTime() - lastQueryAt) > PEER_GRACE
  then return false end
  After 5 seconds, that returns false for every party row even though no
  Q ever went out. Painted red "?" everywhere even with comm-disabled or
  during the brief solo window before joining a group.
  Fix: guard enabled()/IsInGroup() before stamping lastQueryAt.
  Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com