feat(quickbooks): add core webhook triggers - #6245
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryMedium Risk Overview Introduces a fixed The QuickBooks block is enabled for triggers; integration metadata and docs describe the seven triggers and setup (Intuit callback URL + verifier token). Reviewed by Cursor Bugbot for commit 28c04b1. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThe PR adds authenticated QuickBooks webhook ingestion and routes company-scoped events through durable workflow dispatch.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains; the previously reported batch-termination issue is resolved because target and lookup failures are isolated until all later events have been processed.
|
| Filename | Overview |
|---|---|
| apps/sim/app/api/webhooks/quickbooks/route.ts | Adds bounded, signature-authenticated app-level webhook ingress that acknowledges deliveries only after durable enqueue. |
| apps/sim/background/quickbooks-webhook-ingress.ts | Routes every accepted event by QuickBooks company, isolates target failures, and retries after processing the complete batch. |
| apps/sim/lib/webhooks/providers/quickbooks.ts | Adds QuickBooks signature verification, event matching, normalization, and event-ID extraction for idempotency. |
| apps/sim/lib/webhooks/deploy.ts | Integrates QuickBooks with the app-level trigger deployment lifecycle. |
| apps/sim/triggers/quickbooks/index.ts | Exports the new QuickBooks event trigger catalog for registry consumption. |
| apps/sim/blocks/blocks/quickbooks.ts | Enables trigger mode and exposes the registered QuickBooks trigger options on the existing block. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart LR
QB[QuickBooks] --> Route[Signed webhook route]
Route --> Validate[Verify signature and validate bounded envelope]
Validate --> Ingress[Durable ingress job]
Ingress --> Lookup[Find deployed targets by company realm]
Lookup --> Dispatch[Dispatch each event to matching workflows]
Dispatch --> Idempotency[Webhook execution idempotency]
Idempotency --> Execute[Execute workflow]
Reviews (4): Last reviewed commit: "refactor(quickbooks): align webhook trig..." | Re-trigger Greptile
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 937ec78. Configure here.
* feat(quickbooks): complete webhook trigger matrix * fix(quickbooks): document webhook event selectors * fix(quickbooks): clarify webhook setup --------- Co-authored-by: Bill Leoutsakos <billleoutsakos@Mac.localdomain>

Summary
Validation
Live Intuit delivery remains dependent on configuring the public callback and QUICKBOOKS_WEBHOOK_VERIFIER_TOKEN; signed raw-body fixtures cover ingress locally.