Skip to content

fix(server): raise provider status probe timeouts to 10s - #13327

Closed
Kamkmgamer wants to merge 1 commit into
pingdotgg:mainfrom
Kamkmgamer:fix/provider-status-timeout-10s
Closed

Kamkmgamer wants to merge 1 commit into
pingdotgg:mainfrom
Kamkmgamer:fix/provider-status-timeout-10s

Conversation

@Kamkmgamer

Copy link
Copy Markdown
Contributor

Provider status checks fail on slower machines. Probes capped at 3-4s time out before cold CLIs answer, so providers show Unavailable when they work. This change raises the status probe timeouts to 10s across the shared default, Codex rate limits, Grok version and init, Cursor about, and OpenCode version probes, and updates the OpenCode timeout test. Fixes #13326.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Sep 24, 2026
@juliusmarminge

Copy link
Copy Markdown
Member

Superseded: #13326 was closed as a duplicate of #11080 (same hardcoded ~3–4s provider status/version probe timeouts). Please continue any fix work against #11080 instead of this PR.

@juliusmarminge

Copy link
Copy Markdown
Member

Closing as superseded — target issue #13326 was closed duplicate of #11080.

@macroscopeapp

macroscopeapp Bot commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR changes production provider health-check and enrichment deadlines from 3–8 seconds to 10 seconds across several providers, including a shared Claude timeout default, which can increase status-probe latency and alter unavailable/error timing. The accompanying test only updates the expected OpenCode timeout behavior.

You can add or adjust custom eligibility rules. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Provider status checks fail on weaker machines

2 participants