Fix stale roomCodeInput and autoJoinedRef when navigating between rooms

[Copilot]

PeerView reuses its component instance when navigating between /listen/:roomId routes, leaving roomCodeInput showing the previous room code and autoJoinedRef permanently true, preventing auto-join from firing for subsequent rooms.

Changes

PeerView.jsx

• Renamed initialRoomCode → roomId (live value from useParams)
• Added a useEffect on roomId that resets all relevant state and closes any active RTCPeerConnection on navigation:

  useEffect(() => {
    setRoomCodeInput(roomId || '');
    setJoinedRoom(null);
    setConnectionState('idle');
    setError(null);
    autoJoinedRef.current = false;
    pcRef.current?.close();
    pcRef.current = null;
  }, [roomId]);

• Auto-join effect now reads roomId directly instead of the stale captured value

PeerView.test.jsx

• Consolidated duplicated route definitions into a shared SHARED_ROUTES constant
• Added renderPeerViewWithNav helper (injects a NavButtons component into MemoryRouter to drive navigation without createMemoryRouter, which has an AbortSignal incompatibility with jsdom in RR v7)
• Two new tests: navigating to a new roomId resets the input and fires a fresh auto-join; navigating to /join-room clears the input

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.