some fixes for players syncing

common/GameCommon.js

@@ -11,6 +11,10 @@ function setGame(gameId, game) {
   GAMES[gameId] = game
 }
 
+function playerExists(gameId, playerId) {
+  return !!GAMES[gameId].players[playerId]
+}
+
 function addPlayer(gameId, playerId) {
   const ts = Util.timestamp()
   if (!GAMES[gameId].players[playerId]) {
@@ -27,17 +31,22 @@ function addPlayer(gameId, playerId) {
   } else {
     changePlayer(gameId, playerId, { ts })
   }
-  GAMES[gameId].evtInfos[playerId] = {
+  if (!GAMES[gameId].evtInfos[playerId]) {
-    _last_mouse: null,
+    GAMES[gameId].evtInfos[playerId] = {
-    _last_mouse_down: null,
+      _last_mouse: null,
+      _last_mouse_down: null,
+    }
   }
 }
 
-function addSocket(gameId, socket) {
+function socketExists(gameId, socket) {
-  const sockets = GAMES[gameId].sockets
+  return GAMES[gameId].sockets.includes(socket)
+}
 
-  if (!sockets.includes(socket)) {
+function addSocket(gameId, socket) {
-    sockets.push(socket)
+  if (!GAMES[gameId].sockets.includes(socket)) {
+    console.log('adding socket: ', gameId, socket.protocol)
+    GAMES[gameId].sockets.push(socket)
   }
 }
 
@@ -447,7 +456,9 @@ function handleInput(gameId, playerId, input) {
 export default {
   setGame,
   exists,
+  playerExists,
   addPlayer,
+  socketExists,
   addSocket,
   removeSocket,
   get,

game/WsWrapper.js

@@ -46,6 +46,11 @@ export default class WsWrapper {
     ws.onmessage = (e) => {
       this.onmessage(e)
     }
+    ws.onerror = (e) => {
+      this.handle = null
+      this.reconnectTimeout = setTimeout(() => { this.connect() }, 1000)
+      this.onclose(e)
+    }
     ws.onclose = (e) => {
       this.handle = null
       this.reconnectTimeout = setTimeout(() => { this.connect() }, 1000)

server/Game.js

@@ -45,7 +45,9 @@ export default {
   createGame,
   exists: GameCommon.exists,
   addPlayer: GameCommon.addPlayer,
+  playerExists: GameCommon.playerExists,
   addSocket: GameCommon.addSocket,
+  socketExists: GameCommon.socketExists,
   removeSocket: GameCommon.removeSocket,
   get: GameCommon.get,
   getSockets: GameCommon.getSockets,

server/WebSocketServer.js

@@ -50,43 +50,19 @@ class WebSocketServer {
         return
       }
 
-      socket.isAlive = true
-      socket.on('pong', function () {
-        this.isAlive = true;
-      })
-
       socket.on('message', (data) => {
-        console.log(`ws| `, data)
+        console.log(`ws`, socket.protocol, data)
         this.evt.dispatch('message', {socket, data})
       })
     })
 
-    this._interval = setInterval(() => {
-      this._websocketserver.clients.forEach((socket) => {
-        if (socket.isAlive === false) {
-          return socket.terminate()
-        }
-        socket.isAlive = false
-        this.evt.dispatch('close', {socket})
-        socket.ping(() => { })
-      })
-    }, 30000)
-
     this._websocketserver.on('close', () => {
       clearInterval(this._interval)
     })
   }
 
   notifyOne(data, socket) {
-    if (socket.isAlive) {
+    socket.send(JSON.stringify(data))
-      socket.send(JSON.stringify(data))
-    }
-  }
-
-  notifyAll(data) {
-    this._websocketserver.clients.forEach((socket) => {
-      this.notifyOne(data, socket)
-    })
   }
 }
 

server/index.js

@@ -109,6 +109,30 @@ wss.on('close', async ({socket}) => {
   }
 })
 
+const ensurePlayerGame = (gameId, clientId, socket) => {
+  Game.addPlayer(gameId, clientId)
+  Game.addSocket(gameId, socket)
+  Game.store(gameId)
+  const game = Game.get(gameId)
+  notify(
+    [Protocol.EV_SERVER_INIT, {
+      puzzle: game.puzzle,
+      players: game.players,
+      evtInfos: game.evtInfos,
+    }],
+    [socket]
+  )
+}
+
+const handlePlayerInput = (gameId, clientId, clientSeq, clientEvtData) => {
+  const changes = Game.handleInput(gameId, clientId, clientEvtData)
+  Game.store(gameId)
+  notify(
+    [Protocol.EV_SERVER_EVENT, clientId, clientSeq, changes],
+    Game.getSockets(gameId)
+  )
+}
+
 wss.on('message', async ({socket, data}) => {
   try {
     const proto = socket.protocol.split('|')
@@ -121,24 +145,14 @@ wss.on('message', async ({socket, data}) => {
         if (!Game.exists(gameId)) {
           throw `[game ${gameId} does not exist... ]`
         }
-        Game.addPlayer(gameId, clientId)
+        ensurePlayerGame(gameId, clientId, socket)
-        Game.addSocket(gameId, socket)
-        Game.store(gameId)
-        notify(
-          [Protocol.EV_SERVER_INIT, Game.get(gameId)],
-          [socket]
-        )
       } break;
 
       case Protocol.EV_CLIENT_EVENT: {
         const clientSeq = msg[1]
         const clientEvtData = msg[2]
-        const changes = Game.handleInput(gameId, clientId, clientEvtData)
+        ensurePlayerGame(gameId, clientId, socket)
-        Game.store(gameId)
+        handlePlayerInput(gameId, clientId, clientSeq, clientEvtData)
-        notify(
-          [Protocol.EV_SERVER_EVENT, clientId, clientSeq, changes],
-          Game.getSockets(gameId)
-        )
       } break;
     }
   } catch (e) {