decentraland / agent-server / 23814227455

26%

main: 34%

Pull #63

github

fix: wire extractEventText into fetchThread and formatMessages

Both fetchThread (slack.ts) and formatMessages (read-slack-thread.ts)
were using `m.text || ""` directly, which silently drops message
content posted exclusively via attachments or blocks (e.g. GitHub
Slack app notifications).

The extractEventText helper already existed and correctly merged
text + attachments + blocks, but was never called in the thread-
fetching path.

Changes:
- Extract extractBlockText and extractEventText into a new shared
  src/slack-utils.ts to avoid a circular import
  (slack.ts -> read-slack-thread.ts -> slack.ts)
- Re-export extractEventText from slack.ts to preserve existing
  public API (tests import it from there)
- Replace m.text || "" with extractEventText(m) in fetchThread
- Replace m.text || "" with extractEventText(m) in formatMessages

Requested by Ignacio Mazzara via Slack

121 of 501 branches covered (24.15%)

Branch coverage included in aggregate %.

28 of 29 new or added lines in 3 files covered. (96.55%)

236 of 860 relevant lines covered (27.44%)

1.92 hits per line