Skip to content

Improve debugging#114

Merged
jasonvarga merged 5 commits intomasterfrom
ws-debug
Apr 24, 2026
Merged

Improve debugging#114
jasonvarga merged 5 commits intomasterfrom
ws-debug

Conversation

@jasonvarga
Copy link
Copy Markdown
Member

@jasonvarga jasonvarga commented Apr 24, 2026

Adds visibility into why the Collaboration addon's websocket isn't connecting. Previously, a misconfigured broadcast setup (wrong Pusher cluster, unreachable Reverb server, etc.) would leave the "Attempting websocket connection…" banner stuck with zero signal in the console or UI.

  • Logs channel-level events: echo.join, channel.subscribed, channel.error.
  • Logs pusher-js connection state transitions and errors (feature-gated on Echo.echo.connector.pusher.connection, so non-pusher-js drivers are unaffected).
  • After 5 seconds of waiting, logs a "still waiting — is your broadcast server running and reachable?" hint, and swaps the status bar banner to "Taking longer than expected. Check the browser console for details." so non-devs get a signal to go look.
  • Uses console.error for the error paths so they stand out.
  • Extracts a small logger.js module (debug / error) so the [Collaboration] prefix lives in one place.

jasonvarga and others added 5 commits April 24, 2026 11:29
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@jasonvarga jasonvarga changed the title Improve websocket connection debugging Improve debugging Apr 24, 2026
@jasonvarga jasonvarga merged commit 122b7de into master Apr 24, 2026
@jasonvarga jasonvarga deleted the ws-debug branch April 24, 2026 15:51
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