fix(zoho-desk): let an unsupported receivedInDays reach the tool's validation

The block mapper filtered on shape before forwarding, so a fractional or
non-numeric value was dropped and List Tickets then ran with no window at all —
returning the whole queue as though the requested filter had applied. The tool
owns that validation, so the mapper now passes the value straight through.

Adds a block-to-tool seam test: neither side's own tests could catch a value
lost between them.

Author: waleedlatif1

apps/sim/blocks/blocks/zoho-desk.ts

@@ -621,13 +621,17 @@ export const ZohoDeskBlock: BlockConfig<ZohoDeskResponse> = {
           if (typeof rawChannelFilter === 'string' && rawChannelFilter.trim()) {
             result.channel = rawChannelFilter.trim()
           }
+          // Forward whatever was supplied and let the tool judge it. Filtering
+          // here on shape would swallow 30.5 or a non-numeric value, and the
+          // tool would then run without the filter and return the entire queue
+          // as though the requested window had applied. Only the empty
+          // "Any time" option is dropped, because that genuinely means no filter.
           if (
             rawReceivedInDays !== undefined &&
             rawReceivedInDays !== null &&
             rawReceivedInDays !== ''
           ) {
-            const receivedInDays = Number(rawReceivedInDays)
-            if (Number.isInteger(receivedInDays)) result.receivedInDays = receivedInDays
+            result.receivedInDays = Number(rawReceivedInDays)
           }
         }
 

apps/sim/tools/zoho_desk/list_tickets.test.ts

@@ -2,6 +2,7 @@
  * @vitest-environment node
  */
 import { describe, expect, it } from 'vitest'
+import { ZohoDeskBlock } from '@/blocks/blocks/zoho-desk'
 import { zohoDeskListTicketsTool } from '@/tools/zoho_desk/list_tickets'
 
 describe('zohoDeskListTicketsTool request url', () => {
@@ -50,3 +51,36 @@ describe('zohoDeskListTicketsTool request url', () => {
     expect(query.get('status')).toBe('Open,On Hold')
   })
 })
+
+/**
+ * The block maps subBlock values onto tool params, so a value the mapper drops
+ * never reaches the tool's validation and the request silently runs unfiltered.
+ * These cover the seam rather than either side of it.
+ */
+describe('ZohoDeskBlock receivedInDays reaches the tool intact', () => {
+  const buildParams = ZohoDeskBlock.tools.config?.params
+  const buildUrl = zohoDeskListTicketsTool.request.url as (p: Record<string, unknown>) => string
+
+  const throughBlock = (receivedInDays: unknown) => {
+    if (!buildParams) throw new Error('ZohoDeskBlock is missing tools.config.params')
+    const mapped = buildParams({ operation: 'list_tickets', receivedInDays }) as Record<
+      string,
+      unknown
+    >
+    return buildUrl({ accessToken: 'tok', orgId: '700', ...mapped })
+  }
+
+  it('carries a supported window through to the query', () => {
+    expect(new URL(throughBlock('30')).searchParams.get('receivedInDays')).toBe('30')
+  })
+
+  it('lets an unsupported value reach the tool and throw instead of silently unfiltering', () => {
+    for (const value of [7, '7', 30.5, 'abc']) {
+      expect(() => throughBlock(value)).toThrow(/must be 15, 30, or 90/)
+    }
+  })
+
+  it('treats the "Any time" option as no filter at all', () => {
+    expect(new URL(throughBlock('')).searchParams.has('receivedInDays')).toBe(false)
+  })
+})