-
-
Notifications
You must be signed in to change notification settings - Fork 80.7k
[Bug]: runLinkUnderstanding resolves HTTP fetch timeout from the first model entry only #100660
Copy link
Copy link
Closed
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:fix-shape-clearClawSweeper found a clear likely implementation shape for this issue.ClawSweeper found a clear likely implementation shape for this issue.clawsweeper:queueable-fixClawSweeper marked this issue as an existing queue_fix_pr work candidate.ClawSweeper marked this issue as an existing queue_fix_pr work candidate.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:otherThis issue has meaningful maintainer-visible impact outside the owned taxonomy.This issue has meaningful maintainer-visible impact outside the owned taxonomy.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.no-staleExclude from stale automationExclude from stale automation
Description
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:fix-shape-clearClawSweeper found a clear likely implementation shape for this issue.ClawSweeper found a clear likely implementation shape for this issue.clawsweeper:queueable-fixClawSweeper marked this issue as an existing queue_fix_pr work candidate.ClawSweeper marked this issue as an existing queue_fix_pr work candidate.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:otherThis issue has meaningful maintainer-visible impact outside the owned taxonomy.This issue has meaningful maintainer-visible impact outside the owned taxonomy.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.no-staleExclude from stale automationExclude from stale automation
Type
Fields
Priority
None yet
What Problem This Solves
In
src/link-understanding/runner.ts, the timeout for the guarded HTTP fetch operation (fetchLinkContent) is resolved from the first entry of the link models list:const timeoutMs = resolveTimeoutMsFromConfig({ config, entry: entries[0] });. If there are multiple entries with different execution timeout values, the fetch operation uses the first one. Fetching content is independent of the CLI execution and should use a dedicated global fetch timeout configuration or default timeout rather than capping it dynamically on the first command entry.Expected vs Actual Behavior
entries[0].Code Link
src/link-understanding/runner.ts#L223-L233
Proposed Solution
Resolve the fetch timeout using a fallback hierarchy starting from
config.timeoutSecondsbefore falling back to the individual model entry timeouts.