Summary

Delivers an advanced modal-controls bundle by adding richer structured select payloads and native confirm dialogs for slash arg menus.

• add structured selection arrays in interaction payloads:
  • selectedUsers
  • selectedChannels
  • selectedConversations
• keep existing unified selectedValues while preserving per-entity detail
• add native Slack confirm dialogs to slash arg menu elements:
  • button
  • overflow
  • static_select
• extend interactions/slash tests to validate both structured payloads and confirm-dialog emission

Scope

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

Validation

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

Dependency

Depends on:

• Slack: improve Block Kit interaction handling (foundation) #18180 Slack: improve Block Kit interaction handling (foundation)
• feat(slack): capture Block Kit view_closed lifecycle events #18234 feat(slack): capture Block Kit view_closed lifecycle events
• feat(slack): use static_select for large slash arg menus #18242 feat(slack): use static_select for large slash arg menus
• feat(slack): route modal interactions with private metadata #18252 feat(slack): route modal interactions with private metadata
• feat(slack): support Block Kit blocks in sendMessage #18257 feat(slack): support Block Kit blocks in sendMessage
• feat(slack): support Block Kit blocks in editMessage #18262 feat(slack): support Block Kit blocks in editMessage
• feat(slack): support blocks in plugin send action #18268 feat(slack): support blocks in plugin send action
• feat(slack): support blocks in plugin edit action #18278 feat(slack): support blocks in plugin edit action
• feat(slack): centralize Block Kit input validation #18284 feat(slack): centralize Block Kit input validation
• fix(slack): add blocks+media send guardrails #18290 fix(slack): add blocks+media send guardrails
• test(slack): cover sendMessage Block Kit paths #18372 test(slack): cover sendMessage Block Kit paths
• refactor(slack): centralize modal private_metadata utilities #18409 refactor(slack): centralize modal private_metadata utilities
• Slack: expand interaction payload normalization coverage #18413 Slack: expand interaction payload normalization coverage
• Slack: validate runtime blocks in send and edit paths #18416 Slack: validate runtime blocks in send and edit paths
• Slack: dedupe normalized interaction selections #18423 Slack: dedupe normalized interaction selections
• Slack: enrich block action context payloads #18431 Slack: enrich block action context payloads
• Slack: add overflow menus for slash arg choices #18439 Slack: add overflow menus for slash arg choices
• Slack: update action rows for select interactions #18448 Slack: update action rows for select interactions
• Slack: enrich modal input payload normalization #18455 Slack: enrich modal input payload normalization
• Slack: add header and context blocks to arg menus #18458 Slack: add header and context blocks to arg menus
• Slack: show picker values in interaction confirmations #18465 Slack: show picker values in interaction confirmations

If reviewing before dependencies merge, review tip commit: 4d343a427.

Greptile Summary

Fixes a this-binding bug in the viewClosed handler registration. The old code destructured viewClosed from the type-cast ctx.app object, which detaches the method from its receiver and loses the this context when invoked. The new code retains a reference to the cast object (appWithViewClosed) and calls viewClosed as a method on it, preserving the correct this binding for the Slack Bolt App instance.

• Replaces destructured method extraction with method-call-on-object pattern for viewClosed
• No functional changes to handler logic, type signatures, or event routing
• Single focused commit that matches the actual code change

Confidence Score: 5/5

• This PR is safe to merge — it's a small, focused fix that preserves method binding with no behavioral changes.
• The change is a single-commit, minimal refactor that fixes a potential this-binding issue. The transformation is mechanical (destructured method → method call on object), the type signatures are unchanged, and existing tests cover the viewClosed handler registration path. No new logic, no removed safety checks, no risk of regression.
• No files require special attention.

_{Last reviewed commit: c028c42}