Skip to content

[#249] Fix terminal WebSocket ready state check#250

Merged
realproject7 merged 1 commit into
mainfrom
codex/249-terminal-websocket-open
May 29, 2026
Merged

[#249] Fix terminal WebSocket ready state check#250
realproject7 merged 1 commit into
mainfrom
codex/249-terminal-websocket-open

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

  • replace server-side WebSocket.OPEN usage with a runtime-safe numeric readyState helper
  • add a regression test covering open/connecting/closing/closed socket states
  • keep browser-side terminal code unchanged, where WebSocket.OPEN is valid

Verification

Closes #249

@realproject7 realproject7 merged commit 9ea16e0 into main May 29, 2026
1 check passed
@realproject7 realproject7 deleted the codex/249-terminal-websocket-open branch May 29, 2026 22:54
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.

Fix terminal reconnect crash from server-side WebSocket.OPEN usage

1 participant