fix(slack): thread agent identity through channel reply path by hou-rong · Pull Request #27134 · openclaw/openclaw

hou-rong · 2026-02-26T03:52:33Z

Summary

Thread agent outbound identity (chat:write.customize overrides) through the Slack channel reply delivery path so per-agent username, icon URL, and icon emoji are applied to all Slack replies including media messages.

Fixes #27080

Problem

When an agent has outbound.identity configured (name, avatarUrl, emoji), the Slack monitor reply path (deliverReplies) ignores it. All replies appear as the default bot identity, even though sendMessageSlack already supports an identity parameter via chat:write.customize.

Solution

Add optional identity?: SlackSendIdentity param to deliverReplies() in src/slack/monitor/replies.ts
Pass identity through to both text-only and media sendMessageSlack calls
In dispatch.ts, resolve the agent's outbound identity via resolveAgentOutboundIdentity() and convert to SlackSendIdentity format
Pass the resolved identity to deliverNormally() → deliverReplies()

Test plan

Added src/slack/monitor/replies.test.ts with 3 tests:
- Verifies identity object is passed to sendMessageSlack for text replies
- Verifies identity object is passed to sendMessageSlack for media replies
- Verifies identity key is omitted when no identity is provided
pnpm test src/slack/monitor/replies.test.ts — all 3 tests pass

Changelog

Added entry under ### Fixes in CHANGELOG.md.

greptile-apps · 2026-02-26T03:55:33Z

Greptile Summary

Added agent identity customization support to the Slack channel reply path, allowing multi-agent setups to display distinct usernames and avatars when posting replies. The implementation correctly threads the agent's outbound.identity configuration through deliverReplies() → sendMessageSlack() by:

Resolving agent identity via resolveAgentOutboundIdentity() in dispatch.ts
Mapping OutboundIdentity format to Slack's SlackSendIdentity format (name→username, avatarUrl→iconUrl, emoji→iconEmoji)
Passing identity through to both text-only and media reply paths
Using conditional spread syntax to avoid passing undefined values

The changes are minimal, focused, and properly typed. Identity support is correctly implemented for the standard reply path that uses chat.postMessage with the chat:write.customize scope.

Confidence Score: 5/5

This PR is safe to merge with minimal risk
The implementation is straightforward and correct: it properly resolves agent identity, maps types correctly, and threads the identity parameter through all necessary code paths. The changes are minimal (2 files, ~15 lines), well-scoped, and use existing infrastructure (resolveAgentOutboundIdentity, sendMessageSlack already supports identity). Type mappings are accurate, and conditional spread syntax prevents passing undefined. No breaking changes or risky operations.
No files require special attention

_{Last reviewed commit: 6a18aa4}

hou-rong · 2026-02-26T03:59:15Z

Fixed the oxfmt import ordering issue — CI should be green on the next run.

This threads the agent's outbound.identity config through the Slack monitor reply path so multi-agent setups can post with distinct display names/avatars via chat:write.customize. Ready for review.

hou-rong · 2026-02-26T06:52:13Z

Ping for review — CI green (Windows test failure is the known session.test.ts path issue, same as merged #27140). Ready to go whenever you have a moment.

hou-rong · 2026-02-28T03:48:06Z

Friendly ping — CI is all green now (including the TypeScript fix for the test mock). This PR adds tests, changelog entry, and follows conventional commit format. Ready for review when you have a moment. Thanks!

sendMessageSlack() correctly accepts opts.identity for chat:write.customize overrides, but the inbound channel reply path (deliverReplies) never passed it. Resolve the agent's outbound identity from config and forward it so replies display the configured name/avatar instead of the default bot profile. Closes openclaw#27080 Co-Authored-By: Claude Opus 4.6 <[email protected]>

Add unit tests verifying agent identity (username, icon URL, emoji) is correctly threaded through deliverReplies to sendMessageSlack for both text and media replies. Co-Authored-By: Claude Opus 4.6 <[email protected]>

Provide required log/error/exit stubs so tsgo passes. Co-Authored-By: Claude Opus 4.6 <[email protected]>

Takhoffman · 2026-03-01T15:25:53Z

PR #27134 - fix(slack): thread agent identity through channel reply path (#27134)

Merged via squash.

Merge commit: b3f60a6
Verified: pnpm install --frozen-lockfile, pnpm build, pnpm check, pnpm test:macmini
Changes made:
M\textensions/slack/src/channel.test.ts
Why these changes were made:
Rebase onto latest main surfaced a typecheck regression in the Slack extension tests because buildAccountSnapshot now expects cfg and may return asynchronously; the test was updated to pass cfg and await the snapshot.
Changelog: CHANGELOG.md updated=true required=true opt_out=false

Thanks @hou-rong!

@hou-rong

…w#27134) thanks @hou-rong Cherry-pick of upstream b3f60a6.

@hou-rong