fix(quickbooks): normalize upload minor version

Author: Bill Leoutsakos

apps/sim/app/api/tools/quickbooks/upload-attachment/route.test.ts

@@ -135,6 +135,21 @@ describe('POST /api/tools/quickbooks/upload-attachment', () => {
     )
   })
 
+  it('defaults a blank minor version instead of rejecting the upload', async () => {
+    const response = await POST(
+      createMockRequest('POST', {
+        ...baseBody,
+        minorVersion: '   ',
+      })
+    )
+
+    expect(response.status).toBe(200)
+    expect(mockFetch).toHaveBeenCalledWith(
+      'https://quickbooks.api.intuit.com/v3/company/123145/upload?minorversion=75',
+      expect.any(Object)
+    )
+  })
+
   it('normalizes linked entity names before building attachment metadata', async () => {
     const response = await POST(
       createMockRequest('POST', {

apps/sim/lib/api/contracts/tools/quickbooks.ts

@@ -27,10 +27,17 @@ export const quickBooksUploadAttachmentBodySchema = z.object({
   note: z.string().max(2000, 'Attachment note must be 2000 characters or less').optional(),
   includeOnSend: z.boolean().optional(),
   apiEnvironment: z.enum(['production', 'sandbox']).optional(),
-  minorVersion: z
-    .string()
-    .regex(/^\d{1,5}$/, 'Minor version must contain one to five digits')
-    .optional(),
+  minorVersion: z.preprocess(
+    (value) => {
+      if (typeof value !== 'string') return value
+      const trimmed = value.trim()
+      return trimmed || undefined
+    },
+    z
+      .string()
+      .regex(/^\d{1,5}$/, 'Minor version must contain one to five digits')
+      .optional()
+  ),
 })
 
 const quickBooksUploadAttachmentResponseSchema = z.union([