chore(release): 2.8.4 [skip ci]

## [2.8.4](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.8.3...v2.8.4) (2025-10-09) ### Bug Fixes * prevent redirect loops by checking if already at target URL ([c5268b7](c5268b79de))
fix: prevent redirect loops by checking if already at target URL
2025-10-09 01:11:33 +00:00 · 2025-10-08 20:10:40 -05:00 · 2025-10-08 19:25:16 +00:00 · 2025-10-08 14:24:29 -05:00
5 changed files with 32 additions and 10 deletions
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,17 @@
+## [2.8.4](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.8.3...v2.8.4) (2025-10-09)
+
+
+### Bug Fixes
+
+* prevent redirect loops by checking if already at target URL ([c5268b7](https://github.com/antialias/soroban-abacus-flashcards/commit/c5268b79dee66aa02e14e2024fe1c6242a172ed3))
+
+## [2.8.3](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.8.2...v2.8.3) (2025-10-08)
+
+
+### Bug Fixes
+
+* remove ArcadeGuardedPage from room page to prevent redirect loop ([4686f59](https://github.com/antialias/soroban-abacus-flashcards/commit/4686f59d245b2b502dc0764c223a5ce84bf1af44))
+
 ## [2.8.2](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.8.1...v2.8.2) (2025-10-08)


--- a/apps/web/src/app/arcade/room/page.tsx
+++ b/apps/web/src/app/arcade/room/page.tsx
@@ -2,7 +2,6 @@

 import { useRouter } from 'next/navigation'
 import { useEffect } from 'react'
-import { ArcadeGuardedPage } from '@/components/ArcadeGuardedPage'
 import { useRoomData } from '@/hooks/useRoomData'
 import { MemoryPairsGame } from '../matching/components/MemoryPairsGame'
 import { ArcadeMemoryPairsProvider } from '../matching/context/ArcadeMemoryPairsContext'
@@ -46,14 +45,14 @@ export default function RoomPage() {
  }

  // Render the appropriate game based on room's gameName
+  // Note: We don't use ArcadeGuardedPage here because room-based games
+  // have their own navigation logic via useRoomData
  switch (roomData.gameName) {
    case 'matching':
      return (
-        <ArcadeGuardedPage>
-          <ArcadeMemoryPairsProvider>
-            <MemoryPairsGame />
-          </ArcadeMemoryPairsProvider>
-        </ArcadeGuardedPage>
+        <ArcadeMemoryPairsProvider>
+          <MemoryPairsGame />
+        </ArcadeMemoryPairsProvider>
      )

    // TODO: Add other games (complement-race, memory-quiz, etc.)
--- a/apps/web/src/hooks/useArcadeGuard.ts
+++ b/apps/web/src/hooks/useArcadeGuard.ts
@@ -74,10 +74,13 @@ export function useArcadeGuard(options: UseArcadeGuardOptions = {}): UseArcadeGu
      })

      // Redirect if we're not already on the active game page (only if enabled)
-      if (enabled && pathname !== data.gameUrl) {
+      const isAlreadyAtTarget = pathname === data.gameUrl
+      if (enabled && !isAlreadyAtTarget) {
        console.log('[ArcadeGuard] Redirecting to active session:', data.gameUrl)
        onRedirect?.(data.gameUrl)
        router.push(data.gameUrl)
+      } else if (isAlreadyAtTarget) {
+        console.log('[ArcadeGuard] Already at target URL, no redirect needed')
      }
    },

@@ -127,10 +130,13 @@ export function useArcadeGuard(options: UseArcadeGuardOptions = {}): UseArcadeGu
          })

          // Redirect if we're not already on the active game page (only if enabled)
-          if (enabled && pathname !== session.gameUrl) {
+          const isAlreadyAtTarget = pathname === session.gameUrl
+          if (enabled && !isAlreadyAtTarget) {
            console.log('[ArcadeGuard] Redirecting to active session:', session.gameUrl)
            onRedirect?.(session.gameUrl)
            router.push(session.gameUrl)
+          } else if (isAlreadyAtTarget) {
+            console.log('[ArcadeGuard] Already at target URL, no redirect needed')
          }
        } else if (response.status === 404) {
          // No active session
--- a/apps/web/src/hooks/useArcadeRedirect.ts
+++ b/apps/web/src/hooks/useArcadeRedirect.ts
@@ -71,10 +71,13 @@ export function useArcadeRedirect(options: UseArcadeRedirectOptions = {}): UseAr
      // Determine if we need to redirect
      const isArcadeLobby = currentGame === null || currentGame === undefined
      const isWrongGame = currentGame && currentGame !== data.currentGame
+      const isAlreadyAtTarget = _pathname === data.gameUrl

-      if (isArcadeLobby || isWrongGame) {
+      if ((isArcadeLobby || isWrongGame) && !isAlreadyAtTarget) {
        console.log('[ArcadeRedirect] Redirecting to active game:', data.gameUrl)
        router.push(data.gameUrl)
+      } else if (isAlreadyAtTarget) {
+        console.log('[ArcadeRedirect] Already at target URL, no redirect needed')
      }
    },

--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
  "name": "soroban-monorepo",
-  "version": "2.8.2",
+  "version": "2.8.4",
  "private": true,
  "description": "Beautiful Soroban Flashcard Generator - Monorepo",
  "workspaces": [
Author	SHA1	Message	Date
semantic-release-bot	678f4423b6	chore(release): 2.8.4 [skip ci] ## [2.8.4](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.8.3...v2.8.4) (2025-10-09) ### Bug Fixes * prevent redirect loops by checking if already at target URL ([`c5268b7`](`c5268b79de`))	2025-10-09 01:11:33 +00:00
Thomas Hallock	c5268b79de	fix: prevent redirect loops by checking if already at target URL Both useArcadeRedirect and useArcadeGuard were causing infinite loops: 1. User navigates to /arcade/room 2. Room page redirects to /arcade (if roomData null during loading) 3. /arcade sees active session at /arcade/room, redirects back 4. Loop repeats Fix: Check if pathname already matches target URL before redirecting. If already at target, skip the redirect and log that we're already there. This fixes the infinite redirect loop when navigating to /arcade/room. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>	2025-10-08 20:10:40 -05:00
semantic-release-bot	d9aadd1f81	chore(release): 2.8.3 [skip ci] ## [2.8.3](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.8.2...v2.8.3) (2025-10-08) ### Bug Fixes * remove ArcadeGuardedPage from room page to prevent redirect loop ([`4686f59`](`4686f59d24`))	2025-10-08 19:25:16 +00:00
Thomas Hallock	4686f59d24	fix: remove ArcadeGuardedPage from room page to prevent redirect loop ArcadeGuardedPage's redirect logic was conflicting with the room page's own redirect logic, causing an infinite loop: 1. Room page would redirect to /arcade if no roomData 2. /arcade would see active session and redirect back to /arcade/room 3. Loop repeats Room-based games don't need ArcadeGuardedPage because: - They have their own navigation logic via useRoomData - Arcade sessions are created dynamically when starting games in rooms - We don't want to redirect away from the room page This fixes the infinite redirect loop when navigating to /arcade/room. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>	2025-10-08 14:24:29 -05:00