Skip to content

Cherry-pick 2c5b898ee: feat(slack): add download-file action#1432

Merged
alexey-pelykh merged 1 commit intomainfrom
cherry-pick/2c5b898ee-slack-download-file
Mar 15, 2026
Merged

Cherry-pick 2c5b898ee: feat(slack): add download-file action#1432
alexey-pelykh merged 1 commit intomainfrom
cherry-pick/2c5b898ee-slack-download-file

Conversation

@alexey-pelykh
Copy link
Copy Markdown

Cherry-pick of upstream commit 2c5b898ee — "feat(slack): add download-file action for on-demand file attachment access (openclaw#24723)"

Conflicts resolved:

  • Fixed OpenClawConfigRemoteClawConfig in test files

Part of #677

…ccess (openclaw#24723)

* feat(slack): add download-file action for on-demand file attachment access

Adds a new `download-file` message tool action that allows the agent to
download Slack file attachments by file ID on demand. This is a prerequisite
for accessing images posted in thread history, where file attachments are
not automatically resolved.

Changes:
- Add `files` field to `SlackMessageSummary` type so file IDs are
  visible in message read results
- Add `downloadSlackFile()` to fetch a file by ID via `files.info`
  and resolve it through the existing `resolveSlackMedia()` pipeline
- Register `download-file` in `CHANNEL_MESSAGE_ACTION_NAMES`,
  `MESSAGE_ACTION_TARGET_MODE`, and `listSlackMessageActions`
- Add `downloadFile` dispatch case in `handleSlackAction`
- Wire agent-facing `download-file` → internal `downloadFile` in
  `handleSlackMessageAction`

Closes openclaw#24681

* style: fix formatting in slack-actions and actions

* test(slack): cover download-file action path

---------

Co-authored-by: Tak Hoffman <[email protected]>
(cherry picked from commit 2c5b898)
@alexey-pelykh alexey-pelykh merged commit 6ff63c3 into main Mar 15, 2026
7 checks passed
@alexey-pelykh alexey-pelykh deleted the cherry-pick/2c5b898ee-slack-download-file branch March 15, 2026 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants