Skip to content

fix: keep interactive controls live without WebSockets - #41

Merged
sppidy merged 3 commits into
mainfrom
codex/http-control-fallback
Aug 23, 2026
Merged

fix: keep interactive controls live without WebSockets#41
sppidy merged 3 commits into
mainfrom
codex/http-control-fallback

Conversation

@sppidy

@sppidy sppidy commented Aug 23, 2026

Copy link
Copy Markdown
Owner

Summary

  • preserve WebSocket as the primary interactive control transport
  • fall back to an authenticated, CSRF-protected HTTPS long-poll tunnel when upgrades are blocked
  • share ACL, RBAC, approval, ownership, and recording enforcement across both transports
  • add browser journey coverage for an executable terminal with WebSockets forcibly blocked

Validation

  • cargo test -p server --locked (67 passed)
  • cargo clippy -p server --all-targets --locked -- -D warnings
  • npm test (98 passed)
  • npm run typecheck
  • npm run lint (0 errors; 63 existing warnings)
  • npm run build
  • local container journey unavailable because this host has no Docker Compose provider; production browser verification will force WebSockets off after deployment

Copilot AI lite review requested due to automatic review settings August 23, 2026 17:37

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 08b8d5f into main Aug 23, 2026
8 checks passed
@sppidy
sppidy deleted the codex/http-control-fallback branch August 23, 2026 18:17
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