feat(quickbooks): add general accounting operations - #6185
feat(quickbooks): add general accounting operations#6185BillLeoutsakosvl346 wants to merge 6 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryMedium Risk Overview New capabilities include Read Accounting Transactions (list or by-ID for journal entries, deposits, and read-only transfers) and create/update tools for journal entries and deposits. The block adds matching operation dropdown entries, an accounting transaction-type selector, JSON fields for journal and deposit lines (with wand prompts), and a Confirm Posting gate on journal create/update. Parameter mapping uses Descriptions in Reviewed by Cursor Bugbot for commit d15b309. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThe PR adds bounded QuickBooks journal-entry, deposit, and transfer reading plus create/update operations for journal entries and deposits.
Confidence Score: 5/5The PR appears safe to merge from a code-review perspective, with no actionable changed-code defects identified. The accounting operations use fixed entity mappings, bounded validated inputs, exact journal balancing, explicit posting confirmation, header-only sparse updates, and registry/block parity without a concrete reachable failure remaining.
|
| Filename | Overview |
|---|---|
| apps/sim/blocks/blocks/quickbooks.ts | Adds accounting operation UI, conditional inputs, parameter coercion, tool access, and output visibility with registry parity. |
| apps/sim/tools/quickbooks/accounting_utils.ts | Implements bounded line parsing, exact journal balancing, posting confirmation, and create/sparse-update payload builders. |
| apps/sim/tools/quickbooks/read_accounting_transactions.ts | Adds fixed list and by-ID reads for journal entries, deposits, and transfers using shared response transformers. |
| apps/sim/tools/quickbooks/create_journal_entry.ts | Adds a non-retrying authenticated journal-entry creation tool with bounded response handling. |
| apps/sim/tools/quickbooks/create_deposit.ts | Adds a non-retrying authenticated deposit creation tool with validated account lines. |
| apps/sim/tools/quickbooks/update_journal_entry.ts | Adds confirmed sparse journal-entry header updates using ID and sync token. |
| apps/sim/tools/quickbooks/update_deposit.ts | Adds sparse deposit header updates using ID and sync token. |
| apps/sim/tools/quickbooks/accounting.test.ts | Covers entity mappings, response transformations, line validation, payload construction, confirmation, sparse updates, and block parity. |
| apps/sim/tools/quickbooks/types.ts | Extends QuickBooks transaction types and output schemas for general-accounting records. |
| apps/sim/tools/registry.ts | Registers all five new accounting tools consistently with block access and barrel exports. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart LR
Block[QuickBooks block] --> Params[Resolve and validate parameters]
Params --> Reader[Read accounting transactions]
Params --> Journal[Create or update journal entry]
Params --> Deposit[Create or update deposit]
Reader --> Types{Transaction type}
Types --> JE[JournalEntry]
Types --> DP[Deposit]
Types --> TR[Transfer - read only]
Journal --> QBO[QuickBooks Online API]
Deposit --> QBO
JE --> QBO
DP --> QBO
TR --> QBO
Reviews (1): Last reviewed commit: "fix(quickbooks): balance journal entries..." | Re-trigger Greptile
178b3a2 to
76e502b
Compare
8d44381 to
c7d27e3
Compare
76e502b to
7ea91de
Compare
c7d27e3 to
da45ed3
Compare
da45ed3 to
d15b309
Compare
Summary
Safety and validation
Manual acceptance
Live authenticated sandbox execution could not be automated because this environment had no controllable signed-in browser. Request contracts and transformations are covered by sanitized fixtures and the complete local suite.
Stacked on feat/quickbooks-03-payables. Never merge automatically.