fix(quickbooks): include account in deposit updates

Author: Bill Leoutsakos

apps/docs/content/docs/en/integrations/quickbooks.mdx

@@ -2290,14 +2290,15 @@ Create a deposit with bounded account lines
 
 ### `quickbooks_update_deposit`
 
-Sparse-update deposit header fields using the current sync token
+Sparse-update deposit header fields using the current sync token and destination account
 
 #### Input
 
 | Parameter | Type | Required | Description |
 | --------- | ---- | -------- | ----------- |
 | `depositId` | string | Yes | Deposit ID to update |
 | `syncToken` | string | Yes | Current deposit sync token |
+| `depositAccountId` | string | Yes | Current QuickBooks account receiving the deposit |
 | `transactionDate` | string | No | Replacement date in YYYY-MM-DD format |
 | `privateNote` | string | No | Replacement internal note |
 

apps/sim/blocks/blocks/quickbooks.ts

@@ -1029,11 +1029,16 @@ export const QuickBooksBlock: BlockConfig<QuickBooksResponse> = {
           'quickbooks_update_refund_receipt',
           ...PAYMENT_OPERATIONS,
           'quickbooks_create_deposit',
+          'quickbooks_update_deposit',
         ],
       },
       required: {
         field: 'operation',
-        value: ['quickbooks_create_refund_receipt', 'quickbooks_create_deposit'],
+        value: [
+          'quickbooks_create_refund_receipt',
+          'quickbooks_create_deposit',
+          'quickbooks_update_deposit',
+        ],
       },
     },
     {
@@ -1385,8 +1390,7 @@ export const QuickBooksBlock: BlockConfig<QuickBooksResponse> = {
             confirmPosting: isJournalEntry
               ? parseConfirmation(params.confirmPosting, 'confirmPosting')
               : undefined,
-            depositAccountId:
-              isCreate && !isJournalEntry ? optionalValue(params.depositAccountId) : undefined,
+            depositAccountId: !isJournalEntry ? optionalValue(params.depositAccountId) : undefined,
             transactionDate: optionalValue(params.transactionDate),
             documentNumber: isJournalEntry ? optionalValue(params.documentNumber) : undefined,
             privateNote: optionalValue(params.privateNote),

apps/sim/lib/integrations/integrations.json

@@ -14724,7 +14724,7 @@
         },
         {
           "name": "Update Deposit",
-          "description": "Sparse-update deposit header fields using the current sync token"
+          "description": "Sparse-update deposit header fields using the current sync token and destination account"
         }
       ],
       "operationCount": 39,

apps/sim/tools/quickbooks/accounting.test.ts

@@ -257,16 +257,25 @@ describe('QuickBooks accounting mutations', () => {
         ...authParams,
         depositId: '13',
         syncToken: '2',
+        depositAccountId: '35',
         transactionDate: '2026-08-01',
       })
-    ).toEqual({ Id: '13', SyncToken: '2', sparse: true, TxnDate: '2026-08-01' })
+    ).toEqual({
+      Id: '13',
+      SyncToken: '2',
+      sparse: true,
+      DepositToAccountRef: { value: '35' },
+      TxnDate: '2026-08-01',
+    })
     expect(() =>
       buildQuickBooksUpdateDepositBody({
         ...authParams,
         depositId: '13',
         syncToken: '2',
+        depositAccountId: '35',
       })
     ).toThrow('at least one field')
+    expect(quickbooksUpdateDepositTool.params.depositAccountId).toMatchObject({ required: true })
   })
 
   it.each([
@@ -324,6 +333,22 @@ describe('QuickBooks accounting block', () => {
       depositAccountId: '35',
       lines: depositLines,
     })
+    expect(
+      QuickBooksBlock.tools.config!.params!({
+        operation: 'quickbooks_update_deposit',
+        oauthCredential: 'credential-id',
+        transactionId: '204',
+        syncToken: '0',
+        depositAccountId: '35',
+        privateNote: 'Updated through Sim',
+      })
+    ).toMatchObject({
+      credential: 'credential-id',
+      depositId: '204',
+      syncToken: '0',
+      depositAccountId: '35',
+      privateNote: 'Updated through Sim',
+    })
   })
 
   it('exposes exactly 39 operations with tool/access parity', () => {
@@ -344,6 +369,7 @@ describe('QuickBooks accounting block', () => {
         oauthCredential: 'credential-id',
         transactionId: '12',
         syncToken: '1',
+        depositAccountId: '35',
         confirmPosting: 'yes',
         privateNote: 'Updated',
         journalLines: JSON.stringify(journalLines),

apps/sim/tools/quickbooks/accounting_utils.ts

@@ -295,8 +295,9 @@ export function buildQuickBooksUpdateDepositBody(
     Id: requiredQuickBooksString(params.depositId, 'depositId'),
     SyncToken: requiredQuickBooksString(params.syncToken, 'syncToken'),
     sparse: true,
+    DepositToAccountRef: quickBooksReference(params.depositAccountId, 'depositAccountId'),
     ...transactionHeader(params),
   }
-  assertQuickBooksSparseUpdate(body)
+  assertQuickBooksSparseUpdate(body, 4)
   return body
 }

apps/sim/tools/quickbooks/types.ts

@@ -341,6 +341,7 @@ export interface QuickBooksCreateDepositParams extends QuickBooksAuthParams {
 export interface QuickBooksUpdateDepositParams extends QuickBooksAuthParams {
   depositId: string
   syncToken: string
+  depositAccountId: string
   transactionDate?: string
   privateNote?: string
 }

apps/sim/tools/quickbooks/update_deposit.ts

@@ -23,7 +23,8 @@ export const quickbooksUpdateDepositTool: ToolConfig<
 > = {
   id: 'quickbooks_update_deposit',
   name: 'QuickBooks Update Deposit',
-  description: 'Sparse-update deposit header fields using the current sync token',
+  description:
+    'Sparse-update deposit header fields using the current sync token and destination account',
   version: '1.0.0',
   params: {
     accessToken: {
@@ -50,6 +51,12 @@ export const quickbooksUpdateDepositTool: ToolConfig<
       visibility: 'user-or-llm',
       description: 'Current deposit sync token',
     },
+    depositAccountId: {
+      type: 'string',
+      required: true,
+      visibility: 'user-or-llm',
+      description: 'Current QuickBooks account receiving the deposit',
+    },
     transactionDate: {
       type: 'string',
       required: false,