Skip to content

feat(app): pulse the status dot while the event stream reconnects - #47574

Merged
Hona merged 1 commit into
anomalyco:v2from
Hona:status-dot-connecting
Sep 6, 2026
Merged

Hona merged 1 commit into
anomalyco:v2from
Hona:status-dot-connecting

Conversation

@Hona

@Hona Hona commented Sep 6, 2026

Copy link
Copy Markdown
Member

While the event stream is connecting or reconnecting, nothing in the UI shows it. With the stall detection in #47571 a phone that wakes from sleep now reconnects and resyncs on its own, but for that second or two the page looks frozen with a green dot.

  • The existing server status dot keeps its neutral colour while connection.status() !== "connected" and gains animate-pulse, so it visibly breathes while waiting on the server.
  • A server that is known to be down stays critical red; the pulse never masks a real outage.
  • Health-only states (ready, attention, issue) are unchanged once the stream is connected.

Both clips: stream healthy → event stream cut for ~5 s (server still answering /api/health) → stream restored.

Before — the dot stays solid green while the stream is dead:

before.mp4

After — the dot turns neutral and pulses until the stream reconnects:

after.mp4
if (input.serverHealth === false) return "bg-icon-critical-base"
if (input.connecting) return "bg-border-weak-base animate-pulse"

Copilot AI lite review requested due to automatic review settings September 6, 2026 03:55
@Hona
Hona requested a review from Brendonovich as a code owner September 6, 2026 03:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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.

@Hona
Hona merged commit bfcb388 into anomalyco:v2 Sep 6, 2026
14 checks passed
@Hona
Hona deleted the status-dot-connecting branch September 6, 2026 05:15
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