Skip to content

feat(slackbotv2): start workflows from Slack shortcuts - #1594

Open
decofe wants to merge 2 commits into
mainfrom
centaur/add-slack-workflow-shortcuts-1788471166
Open

feat(slackbotv2): start workflows from Slack shortcuts#1594
decofe wants to merge 2 commits into
mainfrom
centaur/add-slack-workflow-shortcuts-1788471166

Conversation

@decofe

@decofe decofe commented Sep 3, 2026

Copy link
Copy Markdown
Member

Summary

  • accept signed Slack message_action shortcuts and start idempotent durable workflow runs for workflow.<name> callbacks
  • pass only the invoking Slack user's identity metadata to API-rs, resolve the matching requester principal server-side, and bind it to the workflow sandbox proxy
  • require the actor's Slack team to match the app's home team, preventing Slack Connect users from borrowing home-workspace OAuth credentials
  • persist only the opaque requester principal ID in the durable task; reject credential-shaped requester fields and omit Slack response/trigger URLs
  • inherit the requester principal for child workflows while scheduled and requester-less runs remain unchanged

Validation

  • bun test in services/slackbotv2 (255 passed, 1 skipped)
  • focused centaur-iron-control requester tests (2 passed)
  • focused centaur-workflows tests (19 passed)
  • cargo +nightly fmt --all -- --check
  • cargo +nightly clippy -p centaur-iron-control -p centaur-workflows -p centaur-api-server --all-targets -- -D warnings
  • bun run check:types remains blocked by three pre-existing fetch.preconnect typing errors in test/slack-user.test.ts

Prompted by: @vinaykantharia

Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
@decofe
decofe force-pushed the centaur/add-slack-workflow-shortcuts-1788471166 branch from a604d40 to 14ac48f Compare September 3, 2026 22:25
Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
@decofe
decofe force-pushed the centaur/add-slack-workflow-shortcuts-1788471166 branch from 14ac48f to d3d4fd7 Compare September 3, 2026 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants