Skip to content

Fix browser control WebSocket routing - #39

Merged
sppidy merged 1 commit into
mainfrom
codex/browser-control-ws-fix
Aug 23, 2026
Merged

Fix browser control WebSocket routing#39
sppidy merged 1 commit into
mainfrom
codex/browser-control-ws-fix

Conversation

@sppidy

@sppidy sppidy commented Aug 23, 2026

Copy link
Copy Markdown
Owner

Summary

  • resolve /ui/ws from the browser current origin at connection time
  • remove the misleading runtime NEXT_PUBLIC_WS_URL compose setting
  • log abnormal close codes and add a stale-build-URL regression test

Root cause

Durable REST/SSE and the raw production WebSocket synthetic were healthy, but browsers never reached the server. The client retained a Next.js public-environment branch whose build/runtime semantics could select a stale URL rather than the dashboard origin.

Validation

  • WebSocketProvider tests: 8 passed
  • TypeScript: passed
  • ESLint: 0 errors (existing warnings only)
  • Next.js production build: passed
  • built client contains no NEXT_PUBLIC_WS_URL or placeholder WebSocket hostname

Copilot AI lite review requested due to automatic review settings August 23, 2026 16:56

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@sppidy
sppidy merged commit 10d43fc into main Aug 23, 2026
6 checks passed
@sppidy
sppidy deleted the codex/browser-control-ws-fix branch August 23, 2026 16:57
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