Skip to content

fix(client): evict unresponsive service - #40896

Merged
thdxr merged 1 commit into
v2from
service-timeout-eviction
Aug 6, 2026
Merged

thdxr merged 1 commit into
v2from
service-timeout-eviction

Conversation

@thdxr

@thdxr thdxr commented Aug 6, 2026

Copy link
Copy Markdown
Member

Summary

  • distinguish timed-out health probes from immediate connection failures
  • evict the registered process after three consecutive timeouts before spawning a replacement
  • preserve registration ownership checks before SIGTERM and SIGKILL
  • keep Effect and Promise service clients in parity

Testing

  • bun typecheck in packages/client
  • bun typecheck in packages/cli
  • bun test --timeout 30000 test/service.test.ts in packages/client
  • bun test --timeout 30000 test/promise-service.test.ts in packages/client

@thdxr
thdxr merged commit 1539bd6 into v2 Aug 6, 2026
1 check passed
@thdxr
thdxr deleted the service-timeout-eviction branch August 6, 2026 17:37
jinhuang712 pushed a commit to jinhuang712/opencode that referenced this pull request Sep 26, 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.

1 participant