Skip to content

fix: auto-reconnect WebSocket and show connection state#2

Merged
dimakis merged 1 commit into
mainfrom
fix/ws-reconnect
Mar 31, 2026
Merged

fix: auto-reconnect WebSocket and show connection state#2
dimakis merged 1 commit into
mainfrom
fix/ws-reconnect

Conversation

@dimakis
Copy link
Copy Markdown
Owner

@dimakis dimakis commented Mar 31, 2026

Summary

Fixes the bug where messages sent after a WebSocket disconnect silently disappear. Common on mobile when the phone sleeps, Tailscale reconnects, or the tab is backgrounded.

  • WebSocket auto-reconnects on disconnect with jittered backoff (2-5s)
  • Reconnects immediately on visibilitychange (phone waking up)
  • Pulsing red ! indicator in the chat header when disconnected
  • Sending on a dead connection shows "Connection lost. Reconnecting..." instead of silently failing
  • Intentional navigation away skips reconnect (no zombie connections)

Test plan

  • Open chat, send a message, verify normal flow works
  • Kill the server (pkill -f tsx), verify red indicator appears
  • Restart server, verify indicator clears and chat resumes
  • Lock phone for 30s, unlock, verify reconnect happens
  • Try sending while disconnected — verify error message appears

Made with Cursor

- WebSocket reconnects automatically on disconnect with jittered backoff
- Reconnects on tab/app visibility change (phone coming back from sleep)
- Pulsing red indicator in header when disconnected
- Send attempts on dead connection show error message instead of silently failing
- Intentional close (navigation) skips reconnect

Made-with: Cursor
@dimakis dimakis merged commit 3c2760f into main Mar 31, 2026
1 check passed
@dimakis dimakis deleted the fix/ws-reconnect branch March 31, 2026 23:39
dimakis added a commit that referenced this pull request Apr 10, 2026
- Add script existence check before exec (was #1 critical)
- Add Zod CalendarResponse schema for stdout validation (#2)
- Check r.ok before parsing fetch response in hook (#3)
- Compute today fresh in handleToday to avoid stale date (#6)
- Extract EventCard + SprintBar to separate component files (#7)
- Extract calendar CSS to dedicated calendar.css (#8)
- Add tests: Day/Week toggle, Today button, EventCard expand,
  milestone expand, non-ok HTTP response handling (#4, #5)

832 tests passing (5 new).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
dimakis added a commit that referenced this pull request Apr 10, 2026
- Add script existence check before exec (was #1 critical)
- Add Zod CalendarResponse schema for stdout validation (#2)
- Check r.ok before parsing fetch response in hook (#3)
- Compute today fresh in handleToday to avoid stale date (#6)
- Extract EventCard + SprintBar to separate component files (#7)
- Extract calendar CSS to dedicated calendar.css (#8)
- Add tests: Day/Week toggle, Today button, EventCard expand,
  milestone expand, non-ok HTTP response handling (#4, #5)

832 tests passing (5 new).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
dimakis added a commit that referenced this pull request Apr 10, 2026
…sprints (#164)

* feat(calendar): add calendar view with meetings, milestones, and sprints

Unified calendar page that merges Google Calendar events, release
milestones, and sprint boundaries via the mgmt calendar_api.py backend.
Mobile-first day/week view with responsive desktop grid layout.

- GET /api/calendar endpoint shells out to calendar_api.py
- useCalendarData hook with date navigation
- CalendarView page with EventCard expand/collapse
- Sprint bars, milestone badges, attendee chips
- Day/Week toggle, prev/next/today navigation
- 17 tests (5 server + 5 hook + 7 page)
- Quick action added to .mitzo.json
- Route at /calendar behind ProtectedRoute

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(calendar): address all Centaur review findings

- Add script existence check before exec (was #1 critical)
- Add Zod CalendarResponse schema for stdout validation (#2)
- Check r.ok before parsing fetch response in hook (#3)
- Compute today fresh in handleToday to avoid stale date (#6)
- Extract EventCard + SprintBar to separate component files (#7)
- Extract calendar CSS to dedicated calendar.css (#8)
- Add tests: Day/Week toggle, Today button, EventCard expand,
  milestone expand, non-ok HTTP response handling (#4, #5)

832 tests passing (5 new).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(calendar): resolve lint errors and review findings

- Remove unused imports (beforeEach, writeFileSync, result)
- Fix toLocalDate to use local timezone instead of UTC
- Remove dead 'sprint' variant from CalendarEvent.type union

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* style: format global.css with prettier

Also add .claude/worktrees/ to .prettierignore to prevent
worktree artifacts from failing format checks.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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.

1 participant