fix(quickbooks): mark sales pagination outputs optional

Author: Bill Leoutsakos

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

@@ -524,11 +524,6 @@ List or read one estimate, invoice, sales receipt, payment, credit memo, or refu
 
 | Parameter | Type | Description |
 | --------- | ---- | ----------- |
-| `startPosition` | number | One-based position of the first item in this response |
-| `maxResults` | number | Actual number of items reported for this response |
-| `nextStartPosition` | number | Position to use when explicitly requesting the next page |
-| `hasMore` | boolean | Conservative indication that another page may exist |
-| `time` | string | QuickBooks response timestamp |
 | `transactionType` | string | Sales transaction type returned |
 | `item` | json | Single native QuickBooks sales transaction |
 |   ↳ `Id` | string | QuickBooks sales transaction ID |
@@ -594,6 +589,12 @@ List or read one estimate, invoice, sales receipt, payment, credit memo, or refu
 |   ↳ `MetaData` | json | Transaction creation and update timestamps |
 |     ↳ `CreateTime` | string | Entity creation timestamp |
 |     ↳ `LastUpdatedTime` | string | Entity last-updated timestamp |
+| `startPosition` | number | One-based position of the first item in this response |
+| `maxResults` | number | Actual number of items reported for this response |
+| `nextStartPosition` | number | Position to use when explicitly requesting the next page |
+| `hasMore` | boolean | Conservative indication that another page may exist |
+| `time` | string | QuickBooks response timestamp |
+
 ### `quickbooks_create_estimate`
 
 Create an estimate with bounded item and description lines

apps/sim/tools/quickbooks/read_sales_transactions.ts

@@ -5,10 +5,7 @@ import type {
   QuickBooksReadSalesTransactionsResponse,
   QuickBooksSalesTransaction,
 } from '@/tools/quickbooks/types'
-import {
-  QUICKBOOKS_LIST_OUTPUTS,
-  QUICKBOOKS_SALES_TRANSACTION_PROPERTIES,
-} from '@/tools/quickbooks/types'
+import { QUICKBOOKS_SALES_TRANSACTION_PROPERTIES } from '@/tools/quickbooks/types'
 import {
   buildQuickBooksEntityUrl,
   buildQuickBooksQueryUrl,
@@ -152,6 +149,31 @@ export const quickbooksReadSalesTransactionsTool: ToolConfig<
       optional: true,
       items: { type: 'json', properties: QUICKBOOKS_SALES_TRANSACTION_PROPERTIES },
     },
-    ...QUICKBOOKS_LIST_OUTPUTS,
+    startPosition: {
+      type: 'number',
+      description: 'One-based position of the first item in this response',
+      optional: true,
+    },
+    maxResults: {
+      type: 'number',
+      description: 'Actual number of items reported for this response',
+      optional: true,
+    },
+    nextStartPosition: {
+      type: 'number',
+      description: 'Position to use when explicitly requesting the next page',
+      optional: true,
+    },
+    hasMore: {
+      type: 'boolean',
+      description: 'Conservative indication that another page may exist',
+      optional: true,
+    },
+    time: {
+      type: 'string',
+      description: 'QuickBooks response timestamp',
+      optional: true,
+      nullable: true,
+    },
   },
 }

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

@@ -129,6 +129,14 @@ describe('QuickBooks sales reader', () => {
     })
   })
 
+  it('marks mode-specific sales read outputs as optional', () => {
+    expect(quickbooksReadSalesTransactionsTool.outputs.item?.optional).toBe(true)
+    expect(quickbooksReadSalesTransactionsTool.outputs.items?.optional).toBe(true)
+    for (const output of ['startPosition', 'maxResults', 'nextStartPosition', 'hasMore']) {
+      expect(quickbooksReadSalesTransactionsTool.outputs[output]?.optional).toBe(true)
+    }
+  })
+
   it('rejects malformed records without usable QuickBooks IDs', async () => {
     await expect(
       quickbooksReadSalesTransactionsTool.transformResponse!(