Skip to content

fix(socket): improve connection reliability and surface errors#16

Merged
rajshekharbind merged 1 commit intoOPCODE-Open-Spring-Fest:mainfrom
ayushhh101:feat/navigation-responsive
Oct 14, 2025
Merged

fix(socket): improve connection reliability and surface errors#16
rajshekharbind merged 1 commit intoOPCODE-Open-Spring-Fest:mainfrom
ayushhh101:feat/navigation-responsive

Conversation

@ayushhh101
Copy link
Contributor

Description

  • Improve Socket.IO reliability and visibility of errors in development.
  • Use connect event (instead of connection) in client.
  • Add reconnection strategy and toasts for connect_error, reconnect_attempt, reconnect, reconnect_failed.
  • Subscribe to server notification and command_ack events.
  • Memoize notification actions (addNotification, removeNotification, clearNotifications) to stabilize effect dependencies.
  • Allow Vite dev origin (http://localhost:5173) in mock server CORS.

Affected files:

  • src/context/SocketContext.jsx
  • src/context/NotificationContext.jsx
  • server/mock-socket-server.js

Semver Changes

  • Patch (bug fix, no new features)
  • Minor (new features, no breaking changes)
  • Major (breaking changes)

Issues

List any issues that this pull request closes.

Checklist

@ayushhh101
Copy link
Contributor Author

Hi @rajshekharbind , the label checker is blocking the PR. Could you please add:
Type: Easy
Semver: Patch
PR:Accept
Thanks!

@rajshekharbind rajshekharbind merged commit 3784721 into OPCODE-Open-Spring-Fest:main Oct 14, 2025
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants