fix: vision(#623): WebSocket streaming for chat UI to replace one-shot claude --print (#1026) #1076

Merged

dev-qwen merged 2 commits from fix/issue-1026 into main 2026-04-20 11:48:11 +00:00

Conversation 3 Commits 2 Files changed 3 +603 -7

2 commits

Author	SHA1	Message	Date
Agent	01f7d061bc	fix: WebSocket streaming - address all AI review findings (#1076) ... Fixes identified in AI review: - Blocker #1: Server now handles chat_request WebSocket frames and invokes Claude - Blocker #2: accept_connection() uses self.headers from BaseHTTPRequestHandler - Blocker #3: handle_websocket_upgrade() uses asyncio.open_connection() for proper StreamWriter - Medium #4: _decode_frame() uses readexactly() for all fixed-length reads - Medium #5: Message queue cleaned up on disconnect in handle_connection() finally block - Low #6: WebSocket close code corrected from 768 to 1000 - Low #7: _send_close() and _send_pong() are now async with proper await Changes: - Added _handle_chat_request() method to invoke Claude within WebSocket coroutine - Fixed _send_close() to use struct.pack for correct close code (1000) - Made _send_pong() async with proper await - Updated handle_connection() to call async close/pong methods and cleanup queue - Fixed handle_websocket_upgrade() to pass Sec-WebSocket-Key from HTTP headers - Replaced create_connection() with open_connection() for proper reader/writer	2026-04-20 11:36:27 +00:00
Agent	17e745376d	fix: vision(#623): WebSocket streaming for chat UI to replace one-shot claude --print (#1026)	2026-04-20 11:36:27 +00:00