Skip to content

fix(multi-review): 放宽 opencode server-start 超时到 30s (env 可配)#185

Merged
Svtter merged 1 commit into
sun-praise:mainfrom
sunhongliang0817:fix/server-start-timeout
Jun 4, 2026
Merged

fix(multi-review): 放宽 opencode server-start 超时到 30s (env 可配)#185
Svtter merged 1 commit into
sun-praise:mainfrom
sunhongliang0817:fix/server-start-timeout

Conversation

@sunhongliang0817

Copy link
Copy Markdown
Contributor

问题

multi-review 在 latex-agent CI 上稳定失败:

Starting opencode server...
Fatal error: Error: Timeout waiting for server to start after 5000ms

SDK createOpencode 默认 server-start 超时仅 5000ms。繁忙 self-hosted runner 上 opencode binary 启动常 >5s,导致稳定超时(重跑多次都同样 fail,非偶发)。

修复

createOpencodetimeout(SDK ServerOptions.timeout),默认放宽到 30000ms,并支持 MULTI_REVIEW_SERVER_TIMEOUT_MS env 覆盖:

const serverTimeoutMs = intEnv("MULTI_REVIEW_SERVER_TIMEOUT_MS", 30000);
const { client, server } = await createOpencode({ config: sdkConfig as any, timeout: serverTimeoutMs });

验证

  • npm run check(tsc)通过
  • npm run build(tsup)通过,dist/index.cjs 已重新生成
  • npm test 14 passed
  • bump version 3.3.1 → 3.3.2

发版

合并后需移动 @v3 floating tag 到新 commit(或发 v3.3.2 release),latex-agent CI 才会用到修复。

注:latex-agent PR #3622(#3619 撤销按钮)正卡在这个 Multi-Review 超时,合并发版后即可恢复。

…nfigurable)

SDK createOpencode default server-start timeout is only 5000ms; on busy self-hosted
runners the opencode binary often takes >5s to start, causing stable failures with
'Timeout waiting for server to start after 5000ms'. Pass timeout to createOpencode
(default 30000ms, overridable via MULTI_REVIEW_SERVER_TIMEOUT_MS). bump 3.3.1->3.3.2.
@Svtter

Svtter commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

provider 问题导致;deepseek provider 不出现此问题。可以再次试试。

经过验证,deepseek provider 也会出现此问题。

@Svtter Svtter merged commit 9c7643b into sun-praise:main Jun 4, 2026
3 of 4 checks passed
@github-actions github-actions Bot mentioned this pull request Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants