Skip to content

feat(slack): post a durable-host message as a named author - #28

Merged
1 commit merged into
mainfrom
devin/1792436576-slack-post-as-sender
Oct 19, 2026
Merged

1 commit merged into
mainfrom
devin/1792436576-slack-post-as-sender

Conversation

@ghost

@ghost ghost commented Oct 19, 2026

Copy link
Copy Markdown

Summary

Hosts that mirror a user's input from another surface (for example Moyai's web UI) can now post it in Slack under that user's name and avatar, the way Devin does, instead of under a bot header.

  • message_request(..., username=None, icon_url=None) passes through to MessageRequest::new(..., author: Option<&SlackAuthor>, ...). Both fields go only on chat.postMessage; chat.update ignores them. The username is whitespace-collapsed and must be 1-80 chars. icon_url must be https:// with no userinfo and at most 2048 chars. An icon_url without a username is rejected.
  • Slack only honors these fields when the bot has chat:write.customize (see docs/slack.md).
  • thread_context previously attributed our own bot's history messages by sender id. It now uses the custom username they were posted with, so the model sees "Ishaan Jaffer: continue" and not a bot message.

Link to Devin session: https://app.devin.ai/sessions/86a7a2fbcc4b422ea615fa8f7c82f1a8
Open in Devin Desktop: https://app.devin.ai/desktop/session/86a7a2fbcc4b422ea615fa8f7c82f1a8?variant=devin
Requested by: @ishaan-berri

Co-Authored-By: Ishaan Jaffer <155045088+ishaan-berri@users.noreply.github.com>
@devin-ai-integration devin-ai-integration Bot assigned ghost Oct 19, 2026
@devin-ai-integration

Copy link
Copy Markdown
Contributor

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@ghost
ghost merged commit 5bffaf1 into main Oct 19, 2026
11 of 12 checks passed
@ghost
ghost deleted the devin/1792436576-slack-post-as-sender branch October 19, 2026 19:17
This pull request was closed.
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.

0 participants