Skip to content

Slack: include stacked modal lifecycle context#18487

Closed
Solvely-Colin wants to merge 37 commits into
openclaw:mainfrom
Solvely-Colin:codex/slack-modal-lifecycle-context
Closed

Slack: include stacked modal lifecycle context#18487
Solvely-Colin wants to merge 37 commits into
openclaw:mainfrom
Solvely-Colin:codex/slack-modal-lifecycle-context

Conversation

@Solvely-Colin

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

Copy link
Copy Markdown
Contributor

Summary

Adds richer modal lifecycle context for stacked Slack views in interaction events.

  • include stacked view metadata on view_submission and view_closed payloads:
    • rootViewId
    • previousViewId
    • externalId
    • viewHash
    • isStackedView
  • keep existing routing/session metadata and input summaries intact
  • add lifecycle-context assertions to view submission/close tests

Scope

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

Validation

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

Dependency

Depends on:

If reviewing before dependencies merge, review tip commit: c1dce56ff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agents Agent runtime and tooling channel: slack Channel integration: slack size: XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants