Why

Modal private_metadata parsing is currently embedded in interaction handler logic. A shared utility makes routing metadata consistent and gives us an explicit producer-facing encoder for future modal-open flows.

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, feat(slack): centralize Block Kit input validation #18284, fix(slack): add blocks+media send guardrails #18290, and test(slack): cover sendMessage Block Kit paths #18372.
• If reviewing before dependencies merge, review tip commit: f7aafce81.

What This PR Adds

• New utility module:

  • /Users/solvely/Projects/OpenClaw/src/slack/modal-metadata.ts
  • parseSlackModalPrivateMetadata(raw)
  • encodeSlackModalPrivateMetadata(input)

• Utility behavior:

  • parse: safe JSON parse with known fields (sessionKey, channelId, channelType)
  • encode: normalized payload and Slack size guard (private_metadata <= 3000 chars)

• Interaction handler wiring:

  • /Users/solvely/Projects/OpenClaw/src/slack/monitor/events/interactions.ts now uses shared parse utility.

• Tests:

  • /Users/solvely/Projects/OpenClaw/src/slack/modal-metadata.test.ts
  • existing interactions tests still pass.

Behavior

• No behavioral regression intended; parsing logic is centralized.
• Adds reusable encoder for upcoming modal-open helper work.

Validation

• pnpm test src/slack/modal-metadata.test.ts src/slack/monitor/events/interactions.test.ts
• pnpm check

Reviewer Focus

• src/slack/modal-metadata.ts
• src/slack/modal-metadata.test.ts
• src/slack/monitor/events/interactions.ts

Greptile Summary

This PR preserves the method binding for viewClosed in the Slack interaction handler. The change extracts the typed app object reference instead of just the method, ensuring the correct this context is maintained when calling viewClosed. This prevents potential runtime errors from losing the method's binding to its parent object.

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• The change is a small, focused refactor that fixes a method binding issue. It's well-tested (existing tests cover viewClosed functionality), follows established patterns, and has no side effects on other code paths.
• No files require special attention

_{Last reviewed commit: 86996a3}