Fix suggestions

Author: erikdubbelboer

internal/signaling/stores/postgres.go

@@ -354,7 +354,7 @@ func (s *PostgresStore) ListLobbies(ctx context.Context, game string, latency []
 
 	var lobbies []Lobby
 	rows, err := s.DB.Query(ctx, `
-		WITH lobbies AS (
+		WITH game_lobbies AS (
 			SELECT
 				code,
 				COALESCE(ARRAY_LENGTH(peers, 1), 0) AS "playerCount",
@@ -382,7 +382,7 @@ func (s *PostgresStore) ListLobbies(ctx context.Context, game string, latency []
 			  AND public = true
 		)
 		SELECT *
-		FROM lobbies
+		FROM game_lobbies
 		WHERE `+where+`
 		ORDER BY `+order+`
 		LIMIT $3
@@ -428,28 +428,20 @@ func (s *PostgresStore) CreatePeer(ctx context.Context, peerID, secret, gameID s
 func (s *PostgresStore) UpdatePeerLatency(ctx context.Context, peerID string, latency []float32) error {
 	now := util.NowUTC(ctx)
 
-	if len(latency) == 0 {
-		_, err := s.DB.Exec(ctx, `
-			UPDATE peers
-			SET
-				latency_vector = NULL,
-				updated_at = $1
-			WHERE peer = $2
-		`, now, peerID)
-		if err != nil {
-			return err
-		}
-	} else {
-		_, err := s.DB.Exec(ctx, `
-			UPDATE peers
-			SET
-				latency_vector = $1,
-				updated_at = $2
-			WHERE peer = $3
-		`, pgvector.NewVector(latency), now, peerID)
-		if err != nil {
-			return err
-		}
+	var vec *pgvector.Vector
+	if len(latency) > 0 {
+		v := pgvector.NewVector(latency)
+		vec = &v
+	}
+	_, err := s.DB.Exec(ctx, `
+		UPDATE peers
+		SET
+			latency_vector = $1,
+			updated_at = $2
+		WHERE peer = $3
+	`, vec, now, peerID)
+	if err != nil {
+		return err
 	}
 
 	return nil

lib/network-latency.ts

@@ -30,7 +30,7 @@ export async function getLatencyVector (max: number, pings: number): Promise<num
       latency += Math.round((performance.now() - start) / 2) // Divide by 2 to estimate one-way latency.
     }
 
-    return Math.round(latency / pings) // Average of two measurements.
+    return Math.round(latency / pings) // Average of multiple measurements.
   }))
   return measurements
 }