fix(timeout): honor timeout config for internal routes and fix type coercion

- Add AbortController support for internal routes (/api/*) to honor timeout
- Fix type coercion: convert string timeout from short-input to number
- Handle NaN gracefully by falling back to undefined (default timeout)

Fixes #2786
Fixes #2242

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: waleedlatif1

apps/sim/tools/index.ts

@@ -625,11 +625,29 @@ async function executeToolRequest(
     let response: Response
 
     if (isInternalRoute) {
-      response = await fetch(fullUrl, {
-        method: requestParams.method,
-        headers: headers,
-        body: requestParams.body,
-      })
+      // Set up AbortController for timeout support on internal routes
+      const controller = new AbortController()
+      const timeoutId = requestParams.timeout
+        ? setTimeout(() => controller.abort(), requestParams.timeout)
+        : undefined
+
+      try {
+        response = await fetch(fullUrl, {
+          method: requestParams.method,
+          headers: headers,
+          body: requestParams.body,
+          signal: controller.signal,
+        })
+      } catch (error) {
+        if (timeoutId) clearTimeout(timeoutId)
+        // Convert AbortError to a timeout error message
+        if (error instanceof Error && error.name === 'AbortError') {
+          throw new Error(`Request timed out after ${requestParams.timeout}ms`)
+        }
+        throw error
+      } finally {
+        if (timeoutId) clearTimeout(timeoutId)
+      }
     } else {
       const urlValidation = await validateUrlWithDNS(fullUrl, 'toolUrl')
       if (!urlValidation.isValid) {

apps/sim/tools/utils.ts

@@ -123,10 +123,12 @@ export function formatRequestParams(tool: ToolConfig, params: Record<string, any
     }
   }
 
-  // Get timeout from params (if specified)
-  const timeout = params.timeout as number | undefined
+  // Get timeout from params (if specified) and ensure it's a number
+  // The short-input subBlock returns a string, so we need to convert it
+  const rawTimeout = params.timeout
+  const timeout = rawTimeout != null ? Number(rawTimeout) : undefined
 
-  return { url, method, headers, body, timeout }
+  return { url, method, headers, body, timeout: Number.isNaN(timeout) ? undefined : timeout }
 }
 
 /**