Skip to content

Recover from daemon hello timeout#1179

Open
Jinwoo-H wants to merge 1 commit intomainfrom
Jinwoo-H/daemon-hello-timeout
Open

Recover from daemon hello timeout#1179
Jinwoo-H wants to merge 1 commit intomainfrom
Jinwoo-H/daemon-hello-timeout

Conversation

@Jinwoo-H
Copy link
Copy Markdown
Contributor

Summary

  • add a timeout for daemon hello handshakes after socket connect
  • classify hello timeout as a wedged-daemon condition so the existing respawn/retry path runs
  • cover the no-hello handshake case in daemon client tests

Why

A daemon can accept the socket but fail to complete the protocol hello. Before this, terminal spawn could wait forever, leaving a mounted pane with no live transport and a blank blinking cursor.

Test plan

  • pnpm exec oxlint src/main/daemon/client.ts src/main/daemon/client.test.ts src/main/daemon/daemon-pty-adapter.ts --format github
  • pnpm test src/main/daemon/client.test.ts src/main/daemon/daemon-pty-adapter.test.ts
  • git diff --check

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