redweb 0.7.1 → 0.7.3

package/README.md

@@ -217,7 +217,6 @@ Useful for managing players, NPCs, chat members, rooms, etc.
 ### 🔧 Basic Usage
 
 ```js
-const { SocketRegistry } = require('redweb');
 
 class Player {
     constructor(socket, id) {
@@ -351,4 +350,4 @@ Here’s the updated `CHANGELOG.md` entry for **RedWeb v0.7.1**, written profess
 
 ## 🪪 License
 
-MIT
+MIT

package/index.d.ts

@@ -113,26 +113,27 @@ declare module 'redweb' {
     };
 
     /** Generic event‑driven registry for socket objects */
-    export class SocketRegistry<T extends SocketWrapper = SocketWrapper> {
-        protected clients: T[];
-
-        onCreate?: (client: T) => boolean | void;
-        onRemove?: (client: T) => boolean | void;
+    /** Generic event-driven registry for socket objects */
+    export class SocketRegistry<T extends SocketWrapper = SocketWrapper> extends EventEmitter {
+        protected items: T[];
 
         constructor();
 
-        setCreateValidator(fn: (client: T) => boolean | void): void;
-        setRemoveValidator(fn: (client: T) => boolean | void): void;
+        /** Adds a socket-bound object to the registry */
+        add(item: T): void;
 
-        add(client: T): boolean;
-        removeById(id: string): boolean;
+        /**
+         * Removes a socket-bound object by reference or id (default key: 'id')
+         * @param itemOrId Object or ID string
+         * @param by Key name to match against (default is 'id')
+         */
+        remove(itemOrId: T | string, by?: keyof T): boolean;
 
-        getById(id: string): T | undefined;
-        getBySocket(socket: WebSocket): T | undefined;
-        allWithout(socket: WebSocket): T[];
+        /** Returns a shallow copy of all registered items */
+        all(): T[];
 
-        broadcast(message: SocketMessage, excludeSocket?: WebSocket | null): void;
-        getSanitizedList(): Record<string, any>[];
+        /** Returns the number of registered items */
+        count(): number;
     }
 
     /** ─────────────────── CONCRETE SERVERS ─────────────────── */

package/package.json

@@ -1,10 +1,10 @@
 {
   "name": "redweb",
-  "version": "0.7.1",
+  "version": "0.7.3",
   "description": "A way to quickly set up an express server",
   "main": "index.js",
   "scripts": {
-    "test": "node tester.js"
+    "test": "npx jest"
   },
   "files": [
     "src/*",

package/src/ws/BaseSocketServer.js

@@ -34,7 +34,21 @@ class BaseSocketServer {
   }
 
   handleUpgrade(req, sock, head) {
-    const route = this.routes.find(r => r.path === req.url);
+    // Some websocket clients (e.g., certain UE plugins) are finicky about the
+    // HTTP upgrade path they send. Normalise the path and fall back to a default
+    // route so we can still complete the upgrade instead of tearing the socket down.
+    const path = (() => {
+      try {
+        return new URL(req.url, `http://${req.headers.host || 'localhost'}`).pathname;
+      } catch {
+        return req.url;
+      }
+    })();
+
+    const route =
+      this.routes.find(r => r.path === path) ||
+      this.routes.find(r => r.path === '/');
+
     if (!route) return sock.destroy();
 
     route.server.handleUpgrade(req, sock, head, (s, r) =>