fix(tools): bound internal tool calls by the plan deadline, not Bun's fetch default - #6547

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Aug 11, 2026 6:59pm

PR Summary

Medium Risk
Touches the core internal tool HTTP path and timeout behavior for all internal routes on Bun; incorrect deadline mapping could cause hangs or premature failures, but behavior is covered by unit tests and Node is documented as ignoring the option.

Overview
Fixes long internal tool runs (e.g. Function blocks) dying at ~5 minutes even when the executor already computed a much larger plan timeout. Bun’s fetch idle timer defaults to 300s and is not raised by AbortSignal alone; only the in-process abort was wired before.

Adds withFetchDeadline to set Bun’s timeout on RequestInit to match the same deadlineMs used for the local AbortController (or timeout: false when there is no valid positive deadline, so the runtime does not silently keep the 300s default). Adds isTransportTimeoutError to recognize Bun’s opaque TimeoutError / fetch failed cases.

Internal route requests in tools/index.ts wrap fetch with withFetchDeadline(..., timeout) and, on transport timeout, rethrow with tool id, elapsed time, and deadline while preserving the original message for retry classification.

^{Reviewed by Cursor Bugbot for commit 0f421b1. Configure here.}

Greptile Summary

The PR prevents Bun’s default transport timeout from undercutting the plan-aware deadline on internal tool requests.

• Adds a request-init helper that disables Bun’s transport timer while preserving the caller-owned abort signal.
• Annotates recognized transport failures with the tool ID, elapsed time, and configured deadline.
• Adds unit coverage for request-init preservation and transport-timeout recognition.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

sequenceDiagram
  participant Executor as Tool executor
  participant Timer as Plan deadline timer
  participant Fetch as Bun fetch
  participant Route as Internal route
  Executor->>Timer: Arm AbortController with plan-aware timeout
  Executor->>Fetch: "Request with signal and timeout=false"
  Fetch->>Route: Internal tool request
  alt Route responds before deadline
    Route-->>Fetch: Response
    Fetch-->>Executor: Tool response
  else Plan deadline expires
    Timer->>Fetch: Abort signal
    Fetch-->>Executor: AbortError
  else Transport fails
    Fetch-->>Executor: TimeoutError or fetch failed
    Executor->>Executor: Add tool, elapsed-time, and deadline context
  end

_{Reviews (2): Last reviewed commit: "fix(tools): disarm Bun's fetch idle time..." | Re-trigger Greptile}

@greptile

@cursor review