OpenCode/Flight-Ops
2
0
Fork 0
Code Issues Pull Requests Actions Activity

Error messages #8

New Issue
Closed
opened 2026-09-18 15:16:57 +02:00 by kamilkrzywda · 1 comment
kamilkrzywda commented 2026-09-18 15:16:57 +02:00
Owner
Copy Link
Copy Source

Error messages like "insufficient funds: a ATR 72-600 costs 200000, you have 62721" should be shown on bottom-right corner as toast message.
Also add more messages on different events, like plane landing (for now).
Enable browser notifications for them if window is not focused.

Error messages like "insufficient funds: a ATR 72-600 costs 200000, you have 62721" should be shown on bottom-right corner as toast message. Also add more messages on different events, like plane landing (for now). Enable browser notifications for them if window is not focused.
kamilkrzywda commented 2026-09-19 17:46:25 +02:00
Author
Owner
Copy Link
Copy Source

Fixed in v0.17.0 (PR #19, merge commit 0467b54c), deployed to https://flight.dev.kkhost.pl and verified (/api/version → 0.17.0).

  • Error toasts: action errors ("insufficient funds: …", sim-control failures) now appear as bottom-right toasts (new useToasts hook + <ToastStack>, 5 s auto-dismiss, capped queue); the inline right-pane error is kept too
  • Event messages: the client diffs consecutive snapshots (lib/deriveEvents.ts) and toasts info events — plane landed at X, package delivered (+€), expired, new package available, bought/sold
  • Browser notifications: "Enable notifications" button in the Time section (explicit permission request); when the window is unfocused and permission granted, events also fire new Notification
  • Bonus robustness found while testing: sim-control POSTs (/api/sim) now retry up to 3× with backoff — a lost control change on a flaky connection no longer leaves UI and server out of sync (this was the recurring e2e flake in runs #44/#47)

New e2e spec toasts.spec.ts covers the insufficient-funds toast path + notifications opt-in; 12 new unit tests for the event diffing and retry logic.

Fixed in **v0.17.0** (PR #19, merge commit `0467b54c`), deployed to https://flight.dev.kkhost.pl and verified (`/api/version` → 0.17.0). - **Error toasts**: action errors ("insufficient funds: …", sim-control failures) now appear as bottom-right toasts (new `useToasts` hook + `<ToastStack>`, 5 s auto-dismiss, capped queue); the inline right-pane error is kept too - **Event messages**: the client diffs consecutive snapshots (`lib/deriveEvents.ts`) and toasts info events — plane landed at X, package delivered (+€), expired, new package available, bought/sold - **Browser notifications**: "Enable notifications" button in the Time section (explicit permission request); when the window is unfocused and permission granted, events also fire `new Notification` - Bonus robustness found while testing: sim-control POSTs (`/api/sim`) now retry up to 3× with backoff — a lost control change on a flaky connection no longer leaves UI and server out of sync (this was the recurring e2e flake in runs #44/#47) New e2e spec `toasts.spec.ts` covers the insufficient-funds toast path + notifications opt-in; 12 new unit tests for the event diffing and retry logic.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No labels
Milestone
No items
No Milestone
Assignees
Clear assignees
kamilkrzywda (Kamil Krzywda) smysakowski
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: OpenCode/Flight-Ops#8