Refactor playerlist to state-driven rendering with renderPlayerList

handlePR and handlePU now only update client.playerlist state,
and renderPlayerList handles all DOM rendering from that state.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

1 files changed, 3 insertions, 2 deletions

diff --git a/webAO/client.ts b/webAO/client.ts
index 95df67a..05a40c9 100644
--- a/webAO/client.ts
+++ b/webAO/client.ts
@@ -150,7 +150,8 @@ class Client extends EventEmitter {
   connect: () => void;
   loadResources: () => void;
   isLowMemory: () => void;
-  players: Map<number, { charId: number; area: number }>;
+  /** Maps player ID to player data */
+  playerlist: Map<number, { charId: number; charName: string; showName: string; name: string; area: number }>;
   charicon_extensions: string[];
   emote_extensions: string[];
   emotions_extensions: string[];
@@ -212,7 +213,7 @@ class Client extends EventEmitter {
     this.temp_packet = "";
     loadResources;
     isLowMemory;
-    this.players = new Map();
+    this.playerlist = new Map();
     this.charicon_extensions = [".png", ".webp"];
     this.emote_extensions = [".gif", ".png", ".apng", ".webp", ".webp.static"];
     this.emotions_extensions = [".png", ".webp"];