decentraland / agent-server

26%

main: 34%

LAST BUILD ON BRANCH fix/fetch-thread-extract-event-text

branch: fix/fetch-thread-extract-event-text

CHANGE BRANCH

x

Reset

Sync Branches

• fix/fetch-thread-extract-event-text
• MauroCicerchia/slack-issue-bot
• chore/add-davide-ab-reconversion
• chore/aws-infra-skill-pulumi-reference
• chore/aws-infra-skill-readme-testing
• chore/update-readme
• chore/use-correct-model
• dependabot/npm_and_yarn/brace-expansion-1.1.13
• dependabot/npm_and_yarn/file-type-21.3.3
• dependabot/npm_and_yarn/handlebars-4.7.9
• dependabot/npm_and_yarn/multi-88c8261ef1
• dependabot/npm_and_yarn/path-to-regexp-8.4.0
• dependabot/npm_and_yarn/picomatch-2.3.2
• dependabot/npm_and_yarn/undici-7.24.4
• dependabot/npm_and_yarn/yaml-2.8.3
• feat/ab-conversion-skill
• feat/ab-reconvert-skill
• feat/add-dependency-management-skill
• feat/add-error-details-responses
• feat/aws-infra-improve-active-regions
• feat/aws-infra-skill-implementation
• feat/compound-plugin
• feat/csv-file-upload-skill
• feat/dapps-design-system-skill
• feat/data-query-improvements
• feat/data-query-skill
• feat/data-query-skill-improvements
• feat/dms
• feat/explorer-pr-workflow
• feat/explorer-skills
• feat/explorer-ticket-status-check
• feat/gitlab-mr-reviews
• feat/html-visualizations
• feat/improve-consistency
• feat/incident-excessive-resource-severity
• feat/memory
• feat/memory-improvements
• feat/pipeline-troubleshooting-skill
• feat/private-conv
• feat/read-messages-in-slack
• feat/shape-qa-release-buffer
• feat/skill-unban-credits
• feat/status
• feat/subscribe-to-channel
• feat/support-attachments-and-blocks
• feat/two-agents-account
• feat/unity-explorer-nullable-reference-types
• feat/usage-skill
• fix/auto-reply
• fix/auto-reply-nacho-headache
• fix/aws-infra-improve-assumptions
• fix/aws-infra-missing-pagination
• fix/correct-regenesis-lab-slack-user-ids
• fix/delete-schedule
• fix/fetchthread-use-extracteventtext
• fix/image-attachments-in-threads
• fix/moderator
• fix/no-logs-dms
• fix/regenesis-lab-slack-ids
• fix/schedule
• fix/schedule-s3-sync
• fix/status
• fix/vulnerability_hardening
• frontend
• gh-action
• main
• pipeline
• pr-39

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