Why

Plugin-level Slack edit actions should have the same Block Kit capability as tool-level editMessage.

Dependency

• Depends on Slack: improve Block Kit interaction handling (foundation) #18180, feat(slack): capture Block Kit view_closed lifecycle events #18234, feat(slack): use static_select for large slash arg menus #18242, feat(slack): route modal interactions with private metadata #18252, feat(slack): support Block Kit blocks in sendMessage #18257, feat(slack): support Block Kit blocks in editMessage #18262, and feat(slack): support blocks in plugin send action #18268.
• If reviewing before dependencies merge, review tip commit: 7548391de.

What This PR Adds

• In /Users/solvely/Projects/OpenClaw/src/plugin-sdk/slack-message-actions.ts:

  • edit now accepts optional blocks (JSON string or array)
  • validates at least one of message or blocks is present
  • forwards normalized payload to editMessage

• In /Users/solvely/Projects/OpenClaw/src/channels/plugins/actions/actions.test.ts:

  • adds Slack adapter tests for:
    • edit with blocks JSON
    • edit with blocks array
    • reject edit when both message and blocks are missing

Behavior

• Existing text-only edits still work.
• Plugin callers can now update messages with Block Kit blocks.

Validation

• pnpm test src/channels/plugins/actions/actions.test.ts
• pnpm check

Reviewer Focus

• src/plugin-sdk/slack-message-actions.ts
• src/channels/plugins/actions/actions.test.ts

Greptile Summary

This PR makes two changes to src/slack/monitor/events/interactions.ts:

• Fixes viewClosed method binding: The previous code extracted viewClosed from ctx.app into a standalone variable, which loses the this context when invoked. The fix keeps a reference to the cast object (appWithViewClosed) and calls the method on it directly (appWithViewClosed.viewClosed(...)), preserving binding — consistent with how ctx.app.action() and ctx.app.view() are already called.
• Inlines InteractionSelectionFields type: Removes the shared InteractionSelectionFields type and duplicates its fields directly into InteractionSummary and ModalInputSummary. Also reorders import statements (type imports first). This is a purely structural refactor with no behavioral change.

Both changes are low-risk. The binding fix addresses a real bug where Slack Bolt's internal this context could be lost during viewClosed handler registration.

Confidence Score: 5/5

• This PR is safe to merge — it fixes a method binding bug and performs a low-risk type refactor.
• The viewClosed binding fix is a clear correctness improvement that aligns with existing patterns in the same file. The type inlining is mechanically correct — all fields from the removed InteractionSelectionFields are present in both destination types. No logic changes, no new runtime behavior beyond the binding fix.
• No files require special attention.

_{Last reviewed commit: 4949782}