anomalyco/opencode
1
0
Fork 0
mirror of https://github.com/anomalyco/opencode.git synced 2026-07-22 02:05:26 -04:00
Code Issues 5.9k Packages Projects Releases 19 Wiki Activity

[PR #7468] fix(tui): add retry backoff for event subscription to prevent CPU spin #12410

New Issue
Open
opened 2026-02-16 18:17:19 -05:00 by yindo · 0 comments
yindo commented 2026-02-16 18:17:19 -05:00
Owner
Copy Link
Copy Source

Original Pull Request: https://github.com/anomalyco/opencode/pull/7468

State: open
Merged: No

Problem

When the server shuts down unexpectedly, the TUI attach client enters a tight retry loop causing 100% CPU usage. This happens because the event subscription loop in sdk.tsx lacks error handling and backoff.

Reproduction:

  1. Run opencode spawn
  2. Kill the spawn process (Ctrl+C or kill)
  3. The attach subprocess continues running at 100% CPU, trying to reconnect indefinitely

Solution

  • Add try-catch around sdk.event.subscribe()
  • Implement exponential backoff: 1s → 2s → 4s → ... → 30s max
  • Add random jitter to prevent thundering herd
  • Give up after 10 consecutive failures with error message

Testing

Manually tested by killing the server process and observing the attach client behavior - it now backs off exponentially and eventually exits gracefully instead of spinning the CPU.

Fixes #7645

**Original Pull Request:** https://github.com/anomalyco/opencode/pull/7468 **State:** open **Merged:** No --- ## Problem When the server shuts down unexpectedly, the TUI attach client enters a tight retry loop causing 100% CPU usage. This happens because the event subscription loop in `sdk.tsx` lacks error handling and backoff. **Reproduction**: 1. Run `opencode spawn` 2. Kill the spawn process (Ctrl+C or `kill`) 3. The attach subprocess continues running at 100% CPU, trying to reconnect indefinitely ## Solution - Add try-catch around `sdk.event.subscribe()` - Implement exponential backoff: 1s → 2s → 4s → ... → 30s max - Add random jitter to prevent thundering herd - Give up after 10 consecutive failures with error message ## Testing Manually tested by killing the server process and observing the attach client behavior - it now backs off exponentially and eventually exits gracefully instead of spinning the CPU. Fixes #7645
yindo added the pull-request label 2026-02-16 18:17:19 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
acp
bad
bug
discussion
docs
documentation
enhancement
extension
good first issue
growth
help-wanted
mcp
model-problem
needs:compliance
nix
opentui
perf
pull-request
Mirrored from GitHub Pull Request
 tui
web
windows
zen
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: anomalyco/opencode#12410
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?