fix(pi): restore firecrawl's server-side budget after the timeout rename

Pi search set params.timeout, which served as both the transport deadline
and Firecrawl's own body timeout -- the collision this PR exists to remove.
Renaming the tool param dropped the body field from the host path while the
sandbox path still hardcoded it, so the two diverged. Setting
firecrawlTimeout restores the documented intent that both paths send the
same effective deadline.

Author: waleedlatif1

apps/sim/executor/handlers/pi/search/normalize.test.ts

@@ -13,6 +13,7 @@ import {
   PI_SEARCH_MAX_RESULTS,
   PI_SEARCH_MAX_SNIPPET_LENGTH,
   PI_SEARCH_MAX_TITLE_LENGTH,
+  PI_SEARCH_TIMEOUT_MS,
   PI_SEARCH_TOOL_PARAMETERS,
   PI_SEARCH_TRUNCATED_MESSAGE,
   parsePiSearchArgs,
@@ -90,6 +91,10 @@ describe('buildPiSearchProviderArgs', () => {
     expect(buildPiSearchProviderArgs('firecrawl', query)).toEqual({
       query: 'ts 5.9 release notes',
       limit: 3,
+      // Firecrawl's own server-side budget, carried under a name that does not collide with the
+      // transport's reserved `timeout`. The sandbox path sends the same value, and
+      // parity.test.ts is what holds the two paths together.
+      firecrawlTimeout: PI_SEARCH_TIMEOUT_MS,
     })
   })
 })

apps/sim/executor/handlers/pi/search/normalize.ts

@@ -156,7 +156,8 @@ export function parsePiSearchArgs(args: Record<string, unknown>): PiSearchQuery
  * default — so this is the one place the mapping is written down.
  *
  * Note two declared-type traps that make this look impossible from the tool definitions:
- * `firecrawl_search` declares only `query`/`apiKey` yet its request body reads `params.limit`, and
+ * `firecrawl_search` declares only `query`/`apiKey` yet its request body reads `params.limit` and
+ * `params.firecrawlTimeout`, and
  * `exa_search` declares `text` as a boolean yet the object form is what requests page text. Both
  * work because `executeTool` hands the raw parameter bag to `body()`.
  */
@@ -172,7 +173,12 @@ export function buildPiSearchProviderArgs(
     case 'parallel':
       return { objective: query, max_results: numResults }
     case 'firecrawl':
-      return { query, limit: numResults }
+      /**
+       * `firecrawlTimeout` becomes Firecrawl's own server-side `timeout` body field. It is named
+       * apart from `timeout` because the transport reads `params.timeout` as the outbound fetch
+       * deadline, so a single key cannot mean both. The sandbox path hardcodes the same value.
+       */
+      return { query, limit: numResults, firecrawlTimeout: PI_SEARCH_TIMEOUT_MS }
   }
 }