Skip to content

Slack: attribute interaction confirmations and structured selects#18474

Closed
Solvely-Colin wants to merge 35 commits into
openclaw:mainfrom
Solvely-Colin:codex/slack-confirmation-actor-metadata
Closed

Slack: attribute interaction confirmations and structured selects#18474
Solvely-Colin wants to merge 35 commits into
openclaw:mainfrom
Solvely-Colin:codex/slack-confirmation-actor-metadata

Conversation

@Solvely-Colin

@Solvely-Colin Solvely-Colin commented Feb 16, 2026

Copy link
Copy Markdown
Contributor

Summary

Improves interaction clarity by attributing confirmation rows to the acting user and exposing structured select entity arrays.

  • append actor mention (<@user>) to confirmation row messages
  • preserve existing selected-value labels while adding identity context in-channel
  • add structured select entity arrays in payload normalization:
    • selectedUsers
    • selectedChannels
    • selectedConversations
  • extend interaction tests for actor-attributed confirmation text and structured selection fields

Scope

  • src/slack/monitor/events/interactions.ts
  • src/slack/monitor/events/interactions.test.ts

Validation

  • pnpm test src/slack/monitor/events/interactions.test.ts src/slack/monitor/slash.test.ts
  • pnpm check

Dependency

Depends on:

If reviewing before dependencies merge, review tip commit: 6dd377449.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

17 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

Comment thread src/slack/monitor/events/interactions.ts Outdated
Comment thread src/slack/monitor/slash.ts Outdated
@openclaw-barnacle openclaw-barnacle Bot added channel: slack Channel integration: slack agents Agent runtime and tooling size: XL labels Feb 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agents Agent runtime and tooling channel: slack Channel integration: slack size: XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants