feat(triggers): add Gong webhook triggers for call events#3984
feat(triggers): add Gong webhook triggers for call events#3984waleedlatif1 merged 4 commits intostagingfrom
Conversation
PR SummaryMedium Risk Overview Wires Gong into the webhook provider registry via a new Reviewed by Cursor Bugbot for commit a79bb4b. Configure here. |
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Greptile SummaryThis PR adds Gong webhook trigger support to the Sim platform, introducing two trigger types (
Confidence Score: 5/5Safe to merge — implementation is correct, follows established patterns, and all prior review concerns are resolved All findings from previous review threads (auth design, triggerSave pattern, buildGenericOutputs delegation, dropdown ordering) were addressed by the developer with clear justifications. The Gong trigger code is structurally identical to the circleback and fathom integrations. No new P0 or P1 issues were found in this review pass. No files require special attention Important Files Changed
Sequence DiagramsequenceDiagram
participant Gong as Gong
participant API as Webhook Endpoint
participant Registry as providers/registry
participant Handler as gongHandler
participant Executor as Workflow Executor
Gong->>API: POST /api/webhooks/trigger/{path}
Note over Gong,API: {isTest, callData:{metaData,<br/>parties, context, content}}
API->>Registry: getProviderHandler("gong")
Registry-->>API: gongHandler (no verifyAuth)
Note over API: defaultHandler checks bearer token<br/>if providerConfig.token is set
API->>Handler: formatInput({ body })
Note over Handler: Extracts: isTest, callData,<br/>metaData (from callData.metaData),<br/>parties, context, trackers<br/>(from callData.content.trackers)
Handler-->>API: { input: { isTest, callData,<br/>metaData, parties, context, trackers } }
API->>Executor: executeWorkflowCore({ input })
Executor-->>Gong: 200 OK
Reviews (3): Last reviewed commit: "json" | Re-trigger Greptile |
|
@greptile |
|
@cursor review |
|
@greptile |
|
@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 a79bb4b. Configure here.
Summary
gong_call_completed,gong_webhook) for receiving call data from Gong automation rulestriggerAllowedand trigger subBlocksType of Change
Testing
Tested manually
Checklist