Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

fix(ui): surface websocket send failures#180

Open
LiuShenXi wants to merge 1 commit into
OpenBMB:main from
LiuShenXi:fix/websocket-send-failed-feedback
Open

fix(ui): surface websocket send failures #180
LiuShenXi wants to merge 1 commit into
OpenBMB:main from
LiuShenXi:fix/websocket-send-failed-feedback

Conversation

@LiuShenXi

@LiuShenXi LiuShenXi commented Jun 8, 2026

Copy link
Copy Markdown

Summary

  • Make the shared WebSocket sendMessage helper report whether a frame was actually sent.
  • Propagate that result through startSessionCommand so the chat composer can detect failed sends.
  • When a chat command cannot be sent because the WebSocket is not connected, keep the draft input, avoid entering the optimistic Processing state, and show a visible error message instead of only logging to the console.
  • Add regression coverage for successful/failed send reporting and the visible failure message.

Test Plan

  • pnpm --filter pilotdeck-ui test
  • pnpm --filter pilotdeck-ui build

Notes

  • pnpm --filter pilotdeck-ui typecheck currently fails on the base branch with existing React type-version conflicts and an existing handleCommandInputChange signature error; no new sendMessage/startSessionCommand type errors were found when filtering the output for touched APIs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

AltStyle によって変換されたページ (->オリジナル) /