From 915d8a5343e70a30c7a82bed645e6628fcc08a86 Mon Sep 17 00:00:00 2001
From: Thomas Hallock <thomas@evolutioniq.com>
Date: Wed, 22 Oct 2025 11:14:33 -0500
Subject: [PATCH] fix(complement-race): use local player instead of first
 player for train display
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Previously used `firstActivePlayer` which could show the wrong player's
name/emoji on the local train in multiplayer sessions. Now explicitly
finds the local player using `isLocal` flag.

Also updated passenger filtering to only show passengers claimed by
the local player, not the first player in the list.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 .../RaceTrack/SteamTrainJourney.tsx           | 23 +++++++++----------
 1 file changed, 11 insertions(+), 12 deletions(-)

diff --git a/apps/web/src/app/arcade/complement-race/components/RaceTrack/SteamTrainJourney.tsx b/apps/web/src/app/arcade/complement-race/components/RaceTrack/SteamTrainJourney.tsx
index b4c90e51..85940e02 100644
--- a/apps/web/src/app/arcade/complement-race/components/RaceTrack/SteamTrainJourney.tsx
+++ b/apps/web/src/app/arcade/complement-race/components/RaceTrack/SteamTrainJourney.tsx
@@ -101,20 +101,20 @@ export function SteamTrainJourney({
   const { players } = useGameMode()
   const { profile: _profile } = useUserProfile()
 
-  // Get the first active player's emoji
+  // Get the LOCAL player's emoji (not just the first player!)
   const activePlayers = Array.from(players.values()).filter((p) => p.isActive)
-  const firstActivePlayer = activePlayers[0]
-  const playerEmoji = firstActivePlayer?.emoji ?? '👤'
+  const localPlayer = activePlayers.find((p) => p.isLocal)
+  const playerEmoji = localPlayer?.emoji ?? '👤'
 
-  // Log only when firstActivePlayer changes
+  // Log only when localPlayer changes
   useEffect(() => {
     console.log(
-      '[SteamTrainJourney] firstActivePlayer:',
-      firstActivePlayer?.name,
+      '[SteamTrainJourney] localPlayer:',
+      localPlayer?.name,
       'isLocal:',
-      firstActivePlayer?.isLocal
+      localPlayer?.isLocal
     )
-  }, [firstActivePlayer?.id, firstActivePlayer?.name, firstActivePlayer?.isLocal])
+  }, [localPlayer?.id, localPlayer?.name, localPlayer?.isLocal])
 
   const svgRef = useRef<SVGSVGElement>(null)
   const pathRef = useRef<SVGPathElement>(null)
@@ -175,14 +175,13 @@ export function SteamTrainJourney({
 
   // Memoize filtered passenger lists to avoid recalculating on every render
   // Arcade room multiplayer uses claimedBy/deliveredBy instead of isBoarded/isDelivered
-  // Only show passengers claimed by the first active player
+  // Only show passengers claimed by the LOCAL player
   const boardedPassengers = useMemo(
     () =>
       displayPassengers.filter(
-        (p) =>
-          p.claimedBy === firstActivePlayer?.id && p.claimedBy !== null && p.deliveredBy === null
+        (p) => p.claimedBy === localPlayer?.id && p.claimedBy !== null && p.deliveredBy === null
       ),
-    [displayPassengers, firstActivePlayer?.id]
+    [displayPassengers, localPlayer?.id]
   )
 
   const nonDeliveredPassengers = useMemo(