test(quickbooks): extend contract parity coverage to the file operations

The download body is a discriminated union and the add-attachment body carries a
superRefine, so neither exposes a flat shape - but their declared keys are still
introspectable, so all three file tools are now held to the same parity rule as
the twelve JSON operations.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RB5M2gk7zZgGomnQqr7WRB

Author: waleedlatif1

apps/sim/lib/internal/quickbooks/contract-param-parity.test.ts

@@ -3,7 +3,9 @@
  */
 import { describe, expect, it } from 'vitest'
 import {
+  quickBooksAddAttachmentContract,
   quickBooksCreateBillPaymentContract,
+  quickBooksDownloadDocumentContract,
   quickBooksUpdateBillContract,
   quickBooksUpdateBillPaymentContract,
   quickBooksUpdateCreditMemoContract,
@@ -16,7 +18,10 @@ import {
   quickBooksUpdateVendorContract,
   quickBooksUpdateVendorCreditContract,
 } from '@/lib/api/contracts/tools/quickbooks'
+import { quickbooksAddAttachmentTool } from '@/tools/quickbooks/add_attachment'
 import { quickbooksCreateBillPaymentTool } from '@/tools/quickbooks/create_bill_payment'
+import { quickbooksDownloadAttachmentTool } from '@/tools/quickbooks/download_attachment'
+import { quickbooksDownloadTransactionPdfTool } from '@/tools/quickbooks/download_transaction_pdf'
 import { quickbooksUpdateBillTool } from '@/tools/quickbooks/update_bill'
 import { quickbooksUpdateBillPaymentTool } from '@/tools/quickbooks/update_bill_payment'
 import { quickbooksUpdateCreditMemoTool } from '@/tools/quickbooks/update_credit_memo'
@@ -62,6 +67,44 @@ const CONTRACT_BOUND_OPERATIONS = [
   ['update_vendor_credit', quickbooksUpdateVendorCreditTool, quickBooksUpdateVendorCreditContract],
 ] as const
 
+/**
+ * The file operations do not expose a flat `shape`: the download body is a
+ * discriminated union (one option per `documentKind`) and the add-attachment
+ * body carries a `superRefine`. Their declared keys are still introspectable,
+ * so they are held to the same parity rule as the JSON operations.
+ */
+const FILE_OPERATIONS = [
+  [
+    'download_attachment',
+    quickbooksDownloadAttachmentTool,
+    unionOptionKeys(quickBooksDownloadDocumentContract.body, 'attachment'),
+  ],
+  [
+    'download_transaction_pdf',
+    quickbooksDownloadTransactionPdfTool,
+    unionOptionKeys(quickBooksDownloadDocumentContract.body, 'transaction_pdf'),
+  ],
+  [
+    'add_attachment',
+    quickbooksAddAttachmentTool,
+    new Set(
+      Object.keys(
+        (quickBooksAddAttachmentContract.body as unknown as { shape: Record<string, unknown> })
+          .shape
+      )
+    ),
+  ],
+] as const
+
+/** Keys declared by the union option whose `documentKind` literal matches. */
+function unionOptionKeys(body: unknown, documentKind: string): Set<string> {
+  const options = (body as { options: Array<{ shape: Record<string, { value?: string }> }> })
+    .options
+  const option = options.find((candidate) => candidate.shape.documentKind?.value === documentKind)
+  if (!option) throw new Error(`No download contract option for documentKind ${documentKind}`)
+  return new Set(Object.keys(option.shape))
+}
+
 describe('QuickBooks contract/tool param parity', () => {
   it.each(CONTRACT_BOUND_OPERATIONS)(
     '%s declares every tool param in its contract body',
@@ -72,4 +115,12 @@ describe('QuickBooks contract/tool param parity', () => {
       expect(dropped).toEqual([])
     }
   )
+
+  it.each(FILE_OPERATIONS)(
+    '%s declares every tool param in its contract body',
+    (_n, tool, declared) => {
+      const dropped = Object.keys(tool.params).filter((param) => !declared.has(param))
+      expect(dropped).toEqual([])
+    }
+  )
 })