feat(quickbooks): complete webhook trigger matrix - #6248
feat(quickbooks): complete webhook trigger matrix#6248BillLeoutsakosvl346 wants to merge 3 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryMedium Risk Overview Each new entity gets a thin trigger module (shared sub-blocks, outputs, and webhook headers), registration in the trigger registry, entries on the QuickBooks block’s Matching and UX tweaks: Tests assert parity across definitions, block access, and registry, and validate all 101 supported CloudEvent patterns plus rejection of unsupported actions. Reviewed by Cursor Bugbot for commit bed4b32. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThe PR expands QuickBooks webhook support from seven representative entities to the complete 29-entity, 101-action matrix while retaining the existing webhook ingestion and dispatch architecture.
Confidence Score: 5/5The PR appears safe to merge, with the expanded trigger matrix consistently represented across runtime, registry, block, metadata, test, and documentation surfaces. The new triggers reuse the established ingestion path, their identifiers and conditional subblocks remain aligned across consumers, and the matching tests cover every declared entity/action combination without revealing a concrete reachable failure.
|
| Filename | Overview |
|---|---|
| apps/sim/triggers/quickbooks/utils.ts | Defines the centralized 29-entity webhook matrix, shared trigger subblocks, setup instructions, and entity/action matching logic without an identified defect. |
| apps/sim/triggers/quickbooks/quickbooks.test.ts | Adds table-driven parity and matching checks for all 29 triggers and 101 supported entity/action combinations. |
| apps/sim/triggers/quickbooks/index.ts | Exports the complete QuickBooks trigger set consistently with the registry and block catalogs. |
| apps/sim/triggers/registry.ts | Registers all newly supported QuickBooks trigger IDs with no missing or duplicate registration found. |
| apps/sim/blocks/blocks/quickbooks.ts | Exposes all 29 triggers and their conditional subblocks through the QuickBooks block using the established shared-field pattern. |
| apps/sim/lib/integrations/integrations.json | Updates integration metadata to advertise the complete trigger catalog and matching trigger count. |
| apps/docs/content/docs/en/integrations/quickbooks.mdx | Documents the expanded trigger matrix and the common normalized webhook output contract. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart LR
QB[QuickBooks webhook] --> Parser[Parse CloudEvent type]
Parser --> Match[Match entity and configured action]
Match --> Registry[Resolve one of 29 trigger IDs]
Registry --> Dispatch[Existing realm-aware dispatcher]
Dispatch --> Workflow[Execute configured workflow]
Reviews (1): Last reviewed commit: "fix(quickbooks): clarify webhook setup" | Re-trigger Greptile
Summary
Validation
Notes
feat/quickbooks-08-webhook-core)QUICKBOOKS_WEBHOOK_VERIFIER_TOKEN; signed fixtures cover the delivery contract locally