Skip to content

Slack: improve Block Kit interaction handling (foundation)#18180

Closed
Solvely-Colin wants to merge 0 commit into
openclaw:mainfrom
Solvely-Colin:codex/slack-blockkit-dev-foundation-clean
Closed

Slack: improve Block Kit interaction handling (foundation)#18180
Solvely-Colin wants to merge 0 commit into
openclaw:mainfrom
Solvely-Colin:codex/slack-blockkit-dev-foundation-clean

Conversation

@Solvely-Colin

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

Copy link
Copy Markdown
Contributor

Why

Slack interactions in OpenClaw need a reliable foundation before broader Block Kit expansion. This PR makes interaction routing and payload capture robust while keeping scope Slack-only.

What This PR Adds

  • Registers a dedicated Slack interaction event handler in src/slack/monitor/events.ts.
  • Implements action handling in src/slack/monitor/events/interactions.ts.
  • Adds focused tests in src/slack/monitor/events/interactions.test.ts.

Behavior

  • Handles openclaw:* Block Kit interactions.
  • Emits structured system events with action metadata (actionId, actionType, selection/date/time values, actor/channel/message IDs).
  • Preserves button confirmation UX via row-level replacement.
  • Skips message rewrites for non-button interactions.
  • Handles modal submissions (view_submission) and emits normalized input summaries.
  • Routes session keys correctly for DM/group/channel flows.

Scope

  • In scope: Slack monitor interaction plumbing and tests.
  • Out of scope: shared cross-channel component abstraction, full Slack renderer expansion, modal close lifecycle follow-ups.

Dependency

Foundation PR for this Slack stack.
Depends on: none.

Validation

  • pnpm test src/slack/monitor/events/interactions.test.ts
  • pnpm test src/slack/monitor/events/interactions.test.ts src/slack/monitor/slash.test.ts
  • pnpm check
  • Manual dev verification on Slack dev gateway (18790): button, static select, overflow, datepicker.

Reviewer Focus

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

@openclaw-barnacle openclaw-barnacle Bot added channel: slack Channel integration: slack size: M labels Feb 16, 2026
@Solvely-Colin
Solvely-Colin marked this pull request as ready for review February 16, 2026 16:18

@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.

3 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

Comment thread src/slack/monitor/events/interactions.ts Outdated
This was referenced Feb 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

channel: slack Channel integration: slack size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants