fix(quickbooks): require current purchase payment type

Author: Bill Leoutsakos

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

@@ -13,7 +13,7 @@ import { BlockInfoCard } from "@/components/ui/block-info-card"
 {/* MANUAL-CONTENT-START:intro */}
 Connect one QuickBooks Online company per credential. During OAuth, choose the company that the workflow should access; Sim binds that company to the credential automatically, so you do not enter a realm ID or API host.
 
-Master Data and Sales transaction reads support **List** and **By ID** modes. List actions return at most one page. Use `nextStartPosition` in another workflow step when `hasMore` is true. Sim does not paginate, retry, or fetch related records automatically.
+Master Data, Sales, and Purchasing transaction reads support **List** and **By ID** modes. List actions return at most one page. Use `nextStartPosition` in another workflow step when `hasMore` is true. Sim does not paginate, retry, or fetch related records automatically.
 
 QuickBooks updates are sparse: provide the record ID, its current `SyncToken`, and only the fields you want to change. Use the latest `SyncToken` returned by a read or mutation. Voiding keeps the transaction in QuickBooks with a zeroed financial effect; it is not deletion and requires explicit confirmation. Create actions accept an optional `requestId` that QuickBooks uses for idempotency when the same request may be submitted again.
 

apps/sim/blocks/blocks/quickbooks.ts

@@ -746,7 +746,6 @@ export const QuickBooksBlock: BlockConfig<QuickBooksResponse> = {
       ],
       condition: { field: 'operation', value: 'quickbooks_update_purchase' },
       required: { field: 'operation', value: 'quickbooks_update_purchase' },
-      value: () => 'cash',
     },
     {
       id: 'paymentAccountId',

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

@@ -484,6 +484,18 @@ describe('QuickBooks purchasing block', () => {
     expect(operationIds).toContain('quickbooks_read_purchasing_transactions')
   })
 
+  it('requires the existing Purchase payment type without fabricating a default', () => {
+    const currentPaymentType = QuickBooksBlock.subBlocks.find(
+      (subBlock) => subBlock.id === 'currentPurchasePaymentType'
+    )
+
+    expect(currentPaymentType?.required).toEqual({
+      field: 'operation',
+      value: 'quickbooks_update_purchase',
+    })
+    expect(currentPaymentType?.value).toBeUndefined()
+  })
+
   it('keeps every subblock ID unique and purchasing updates header-only', () => {
     const ids = QuickBooksBlock.subBlocks.map((subBlock) => subBlock.id)
     expect(new Set(ids).size).toBe(ids.length)