feat(slack): launch v2 triggers and backfill custom bots - #6873
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryHigh Risk Overview Migrates pasted bot tokens into reusable Webhook lookup also caps targets per path/routing key. Docs and integration availability now treat Slack as Reviewed by Cursor Bugbot for commit 5f2a0bb. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThe PR launches the Slack v2 trigger, adds reusable custom-bot ingress and legacy credential backfill, and gates native Slack OAuth triggers behind extended scopes.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains; the previously reported Slack acknowledgment and retry issues are resolved across native, custom-bot, and migrated legacy ingress paths.
|
| Filename | Overview |
|---|---|
| apps/sim/lib/webhooks/slack-dispatch.ts | Centralizes bounded Slack fan-out and consistently reduces dispatch outcomes into acknowledgment or retry responses. |
| apps/sim/app/api/webhooks/trigger/[path]/route.ts | Adds credential-based handling for migrated legacy Slack webhook paths while preserving direct webhook delivery. |
| apps/sim/app/api/webhooks/slack/custom/[credentialId]/route.ts | Delegates custom-bot verification and dispatch to shared ingress helpers and returns the reduced dispatch result. |
| apps/sim/lib/webhooks/slack-custom-ingress.ts | Implements reusable custom-bot credential verification, routing-key lookup, and dispatch. |
| packages/db/scripts/migrate-slack-custom-bots.ts | Adds bounded, resumable migration behavior for legacy Slack custom-bot credentials and webhook routing metadata. |
| apps/sim/lib/webhooks/providers/slack.ts | Makes queue-admission failures retryable by returning a non-success provider response. |
| apps/sim/blocks/blocks/slack.ts | Releases the Slack v2 block and aligns its trigger and authentication metadata. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart LR
S[Slack delivery] --> I{Ingress}
I -->|Native app| N[Team-based lookup]
I -->|Custom bot| C[Credential-based lookup]
I -->|Migrated legacy URL| L[Verify credential aliases]
N --> F[Bounded webhook fan-out]
C --> F
L --> F
F --> R{Dispatch results}
R -->|Queued or intentionally ignored| A[HTTP 200 acknowledgment]
R -->|All targets fail| E[Non-2xx retry response]
Reviews (7): Last reviewed commit: "fix(slack): retry failed webhook deliver..." | Re-trigger Greptile
83137a3 to
e421379
Compare
|
@cursor review |
e421379 to
8c78e18
Compare
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 8c78e18. Configure here.
8c78e18 to
12513cb
Compare
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 12513cb. Configure here.
12513cb to
5f2a0bb
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 5f2a0bb. Configure here.
5f2a0bb to
808473b
Compare
|
@cursor review |
|
Skipping Bugbot: Bugbot is disabled for this repository. Visit the Bugbot dashboard to update your settings. |
|
@cursor review |
|
Skipping Bugbot: Bugbot is disabled for this repository. Visit the Bugbot dashboard to update your settings. |
|
@cursor review |
|
Skipping Bugbot: Bugbot is disabled for this repository. Visit the Bugbot dashboard to update your settings. |

Summary
Type of Change
Testing
Checklist