fix(firecrawl): treat a blank maxDiscoveryDepth as unset

A cleared field arrived as '' and was sent as 0, a crawl of the start page
only. It now follows the blank maxDepth handling.

Author: waleedlatif1

apps/sim/tools/firecrawl/crawl.test.ts

@@ -106,10 +106,12 @@ describe('Firecrawl legacy crawl depth', () => {
       apiKey: 'test',
       url: 'https://example.com/docs',
       maxDepth: '',
+      maxDiscoveryDepth: ' ',
     }
     const body = crawlTool.request.body(blankDepth)
     if (!isPlainRecord(body)) throw new Error('Expected a JSON request body')
     expect(body.includePaths).toBeUndefined()
     expect(body.regexOnFullURL).toBeUndefined()
+    expect(body).not.toHaveProperty('maxDiscoveryDepth')
   })
 })

apps/sim/tools/firecrawl/crawl.ts

@@ -154,7 +154,7 @@ export const crawlTool: ToolConfig<FirecrawlCrawlParams, FirecrawlCrawlResponse>
       }
 
       if (params.prompt) body.prompt = params.prompt
-      if (params.maxDiscoveryDepth != null)
+      if (params.maxDiscoveryDepth != null && String(params.maxDiscoveryDepth).trim() !== '')
         body.maxDiscoveryDepth = Number(params.maxDiscoveryDepth)
       if (params.sitemap) body.sitemap = params.sitemap
       if (typeof params.crawlEntireDomain === 'boolean')

apps/sim/tools/firecrawl/types.ts

@@ -290,7 +290,8 @@ export interface FirecrawlCrawlParams {
   formats?: FirecrawlFormat[]
   onlyMainContent?: boolean
   prompt?: string
-  maxDiscoveryDepth?: number
+  /** Raw field text like {@link maxDepth}: a cleared field arrives as `''`. */
+  maxDiscoveryDepth?: number | string
   sitemap?: 'skip' | 'include'
   crawlEntireDomain?: boolean
   allowExternalLinks?: boolean