Why

Slack API does not support mixing rich Block Kit blocks with file-upload media sends in our current send flow. We already enforce this lower in send.ts; this PR adds earlier guardrails in tool/plugin layers for faster, clearer failures.

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, feat(slack): support blocks in plugin send action #18268, feat(slack): support blocks in plugin edit action #18278, and feat(slack): centralize Block Kit input validation #18284.
• If reviewing before dependencies merge, review tip commit: 7fc427cd0.

What This PR Adds

• In /Users/solvely/Projects/OpenClaw/src/agents/tools/slack-actions.ts:

  • reject sendMessage when both mediaUrl and blocks are provided

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

  • reject plugin send when both media and blocks are provided

• Tests added:

  • /Users/solvely/Projects/OpenClaw/src/agents/tools/slack-actions.e2e.test.ts
  • /Users/solvely/Projects/OpenClaw/src/channels/plugins/actions/actions.test.ts

Behavior

• Valid text/media-only and text/blocks-only sends remain unchanged.
• Invalid mixed blocks + media now fails earlier with clear errors.

Validation

• pnpm test src/channels/plugins/actions/actions.test.ts
• pnpm exec vitest run --config vitest.e2e.config.ts src/agents/tools/slack-actions.e2e.test.ts
• pnpm check

Reviewer Focus

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

Greptile Summary

Despite the PR title referencing "blocks+media send guardrails," the actual diff in this PR is a single commit (eaf2ee66) that fixes viewClosed method binding in src/slack/monitor/events/interactions.ts. The other changes mentioned in the description belong to dependent PRs.

• Method binding fix: The viewClosed method on the Slack Bolt app was previously destructured into a standalone variable, losing its this context. The fix stores the cast object reference (appWithViewClosed) and calls the method on it directly (appWithViewClosed.viewClosed(...)), preserving proper method binding.
• No logic changes, no new features — this is a targeted correctness fix for method invocation.

Confidence Score: 5/5

• This PR is safe to merge — it's a minimal, correct fix that preserves method binding without changing any behavior.
• The change is a 2-line refactor that replaces destructured method extraction with a proper method call on an object reference. It preserves the existing type cast, guard check, and invocation pattern. No new logic is introduced, no existing behavior is altered for working cases, and the fix addresses a real potential runtime issue (lost this binding).
• No files require special attention.

_{Last reviewed commit: 1126db4}