feat(quickbooks): add observable PO-to-bill linking - #6194
feat(quickbooks): add observable PO-to-bill linking#6194BillLeoutsakosvl346 wants to merge 4 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryMedium Risk Overview After creation, the action exposes observable linkage results: Docs and integration metadata are updated to describe optional PO-linked bill creation, the new outputs, and expanded native purchasing Reviewed by Cursor Bugbot for commit afdc799. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThe PR extends QuickBooks Create Bill with optional line-level Purchase Order references and reports whether QuickBooks returned each requested link.
Confidence Score: 5/5The PR appears safe to merge, although the user explicitly requested that it not be merged automatically. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| apps/sim/tools/quickbooks/purchasing_utils.ts | Adds Bill-specific line parsing, PO linkage payload construction, and post-create linkage verification while retaining the shared purchasing-line path. |
| apps/sim/tools/quickbooks/create_bill.ts | Extends Create Bill response transformation and output metadata with observable linkage results. |
| apps/sim/tools/quickbooks/types.ts | Defines linked-transaction shapes, Bill-link inputs and outputs, and richer purchasing transaction line metadata. |
| apps/sim/blocks/blocks/quickbooks.ts | Maps Create Bill through the Bill-specific parser and exposes linkage status fields to workflows. |
| apps/sim/tools/quickbooks/purchasing.test.ts | Covers standalone, linked, partial, missing, mixed, duplicate, and operation-scoped PO-link behavior. |
Sequence Diagram
sequenceDiagram
participant Workflow
participant Tool as Create Bill Tool
participant QBO as QuickBooks
Workflow->>Tool: Bill lines with optional PO and PO-line IDs
Tool->>Tool: Validate paired IDs and build LinkedTxn fields
Tool->>QBO: POST Bill
QBO-->>Tool: Created native Bill
Tool->>Tool: Compare requested links with Bill Line[].LinkedTxn
Tool-->>Workflow: Bill plus linkingSucceeded, linkedLines, missingLinks, warning
Reviews (2): Last reviewed commit: "fix(quickbooks): keep shared line exampl..." | 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 702149c. Configure here.
8d44381 to
c7d27e3
Compare
68cbb1d to
ccc6701
Compare
c7d27e3 to
da45ed3
Compare
ccc6701 to
2b6202c
Compare
da45ed3 to
d15b309
Compare
2b6202c to
afdc799
Compare

What changed
linkingRequested,linkingSucceeded,linkedLines,missingLinks, andlinkingWarning.Why
QuickBooks requires line-level
TxnLineIdvalues to establish PO-to-Bill relationships. It may also create the Bill while silently omitting an invalid or unavailable requested link. The tool therefore verifies the returned Bill and reports exactly which requested links QuickBooks established instead of treating HTTP success as proof of linkage.Validation
staging: approved with no actionable findings.git diff --checkand memory-load audit.Stack
feat/quickbooks-04-accountingat8d443814c4f29f3a8f4acb36fabed7d0df44d182feat/quickbooks-04a-po-bill-linkingThis PR must not be merged automatically.