Compare commits

...

12 Commits

Author SHA1 Message Date
semantic-release-bot
1cb175982a chore(release): 2.7.0 [skip ci]
## [2.7.0](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.6.0...v2.7.0) (2025-10-08)

### Features

* extend GameModeContext to support room-based multiplayer ([ee6094d](ee6094d59d))
2025-10-08 15:14:28 +00:00
Thomas Hallock
ee6094d59d feat: extend GameModeContext to support room-based multiplayer
When a user is in a room, GameModeContext now merges players from all
room members to create a unified player set for gameplay. This enables
true multiplayer where all participants' active players participate
together in the game.

Key changes:
- Added useRoomData and useViewerId to GameModeContext
- Local players (from DB) are marked with isLocal: true
- Remote players (from other room members) are marked with isLocal: false
- Players map merges local + remote players when in a room
- activePlayers set includes all active players from all room members
- Edit operations (update/remove/setActive) only work on local players
- Socket broadcast when local players change to notify room members
- When not in a room, behavior is unchanged (solo/local multiplayer)

This implements the "players is the union of all active players for all
members of the room" requirement for room-based gameplay.

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

Co-Authored-By: Claude <noreply@anthropic.com>
2025-10-08 10:13:33 -05:00
semantic-release-bot
9d0c488f2b chore(release): 2.6.0 [skip ci]
## [2.6.0](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.5.0...v2.6.0) (2025-10-08)

### Features

* refactor room addressing to /arcade/room ([e7d2a73](e7d2a73ddf))
2025-10-08 15:08:52 +00:00
Thomas Hallock
e7d2a73ddf feat: refactor room addressing to /arcade/room
Simplify room URL structure so users access their room's game at
/arcade/room instead of /arcade/rooms/[roomId]/[game]. Since users can
only be in one room at a time (modal room enforcement), this provides a
cleaner addressing model.

Changes:
- useRoomData now fetches user's current room from /api/arcade/rooms/current
- Created /api/arcade/rooms/current endpoint to get user's active room
- Created /arcade/room page that renders the appropriate game for the room
- Removed URL parsing logic in favor of backend room lookup
- Socket connection and real-time updates still work with new structure

Next step: Extend GameModeContext to merge players from all room members
so gameplay uses the union of all active players in the room.

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

Co-Authored-By: Claude <noreply@anthropic.com>
2025-10-08 10:07:58 -05:00
semantic-release-bot
63517cf45d chore(release): 2.5.0 [skip ci]
## [2.5.0](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.4.6...v2.5.0) (2025-10-08)

### Features

* display room info and network players in mini app nav ([5e3261f](5e3261f3be))
2025-10-08 14:43:33 +00:00
Thomas Hallock
5e3261f3be feat: display room info and network players in mini app nav
When users are in a room (/arcade/rooms/[roomId]/*), the mini app nav now shows:
1. Room name and game type in RoomInfo component
2. Other members' player avatars with "network" indicators
3. Clear distinction between local players and network players

Implementation:
- Created useRoomData hook to fetch room data and listen to real-time updates
- Updated PageWithNav to use room data and compute network players
- Enhanced RoomInfo component to display room name when available
- Network players shown with special borders and connection indicators

The nav automatically detects room context from the URL and fetches:
- Room details (name, game, member count)
- All room members and their players
- Real-time updates via socket events (member-joined, member-left, players-updated)

Network players are filtered to exclude the current user and show each other
member's players with their display names for clear identification.

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

Co-Authored-By: Claude <noreply@anthropic.com>
2025-10-08 09:42:34 -05:00
semantic-release-bot
1e43e6945b chore(release): 2.4.6 [skip ci]
## [2.4.6](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.4.5...v2.4.6) (2025-10-08)

### Bug Fixes

* real-time room member updates via globalThis socket.io sharing ([94a1d9b](94a1d9b110))
2025-10-08 14:38:03 +00:00
Thomas Hallock
94a1d9b110 fix: real-time room member updates via globalThis socket.io sharing
The room member real-time update bug was caused by module isolation when API
routes dynamically imported socket-server.ts. Each import created a separate
module instance where the `io` variable was null, preventing broadcasts.

Root cause:
- API routes called getSocketIO() via dynamic import
- Dynamic imports created separate module instances
- The module-level `io` variable was never initialized in these instances
- Broadcasts from API routes never reached connected clients

The fix:
- Store socket.io instance in globalThis.__socketIO instead of module variable
- Ensures same instance accessible across all module boundaries
- API routes can now successfully broadcast to connected clients

Changes:
- socket-server.ts: Use globalThis.__socketIO for cross-module access
- src/lib/socket-io.ts: Clean up debug logging
- src/app/api/arcade/rooms/[roomId]/join/route.ts: Clean up debug logging
- __tests__/room-realtime-updates.e2e.test.ts: Add comprehensive e2e tests
- socket-server.js: DELETED (outdated, missing room handlers)

Tests verify:
1. member-joined broadcasts when users join via API
2. member-left broadcasts when users leave
3. Both members and players lists update correctly

All 3 e2e tests passing. User confirmed fix works in real app.

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

Co-Authored-By: Claude <noreply@anthropic.com>
2025-10-08 09:37:00 -05:00
semantic-release-bot
9fa5652173 chore(release): 2.4.5 [skip ci]
## [2.4.5](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.4.4...v2.4.5) (2025-10-08)

### Bug Fixes

* send all members (not just online) in socket broadcasts ([3fa6cce](3fa6cce17a))
2025-10-08 14:03:35 +00:00
Thomas Hallock
3fa6cce17a fix: send all members (not just online) in socket broadcasts
The root issue was that socket broadcasts were sending only online
members while the initial page load showed all members, causing
inconsistency.

Changes:
- Join/leave API routes now send `members` (all) instead of `onlineMembers`
- Socket server join/leave handlers now send `members` instead of `onlineMembers`
- Client event handlers updated to expect `data.members` instead of `data.onlineMembers`
- Removed unused `getOnlineRoomMembers` import from socket-server.ts

Now when a user joins, other users immediately see them in the members
list without needing to reload the page. Both members and players are
updated together in the same broadcast.

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

Co-Authored-By: Claude <noreply@anthropic.com>
2025-10-08 09:02:44 -05:00
semantic-release-bot
8f3dd9ec92 chore(release): 2.4.4 [skip ci]
## [2.4.4](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.4.3...v2.4.4) (2025-10-08)

### Bug Fixes

* correctly access getSocketIO from dynamic import ([30abf33](30abf33ee8))
2025-10-08 13:57:35 +00:00
Thomas Hallock
30abf33ee8 fix: correctly access getSocketIO from dynamic import
The dynamic import returns a module namespace object, so we need to
access socketServerModule.getSocketIO() rather than treating the
module itself as the function.

Simplified the wrapper to directly cache and use the module, checking
that getSocketIO exists before calling it.

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

Co-Authored-By: Claude <noreply@anthropic.com>
2025-10-08 08:56:47 -05:00
17 changed files with 934 additions and 110 deletions

View File

@@ -1,3 +1,45 @@
## [2.7.0](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.6.0...v2.7.0) (2025-10-08)
### Features
* extend GameModeContext to support room-based multiplayer ([ee6094d](https://github.com/antialias/soroban-abacus-flashcards/commit/ee6094d59d26a9e80ba5d023ca6dc13143bea308))
## [2.6.0](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.5.0...v2.6.0) (2025-10-08)
### Features
* refactor room addressing to /arcade/room ([e7d2a73](https://github.com/antialias/soroban-abacus-flashcards/commit/e7d2a73ddf2048691325a18e3d71a7ece444c131))
## [2.5.0](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.4.6...v2.5.0) (2025-10-08)
### Features
* display room info and network players in mini app nav ([5e3261f](https://github.com/antialias/soroban-abacus-flashcards/commit/5e3261f3bec8c19ec88c9a35a7e6ef8eda88a55e))
## [2.4.6](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.4.5...v2.4.6) (2025-10-08)
### Bug Fixes
* real-time room member updates via globalThis socket.io sharing ([94a1d9b](https://github.com/antialias/soroban-abacus-flashcards/commit/94a1d9b11058bfb4b54a4753e143cf85f215e913))
## [2.4.5](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.4.4...v2.4.5) (2025-10-08)
### Bug Fixes
* send all members (not just online) in socket broadcasts ([3fa6cce](https://github.com/antialias/soroban-abacus-flashcards/commit/3fa6cce17a7acd940cf5a9e6433bf6c4b497540c))
## [2.4.4](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.4.3...v2.4.4) (2025-10-08)
### Bug Fixes
* correctly access getSocketIO from dynamic import ([30abf33](https://github.com/antialias/soroban-abacus-flashcards/commit/30abf33ee86b36f2a98014e5b017fa8e466a2107))
## [2.4.3](https://github.com/antialias/soroban-abacus-flashcards/compare/v2.4.2...v2.4.3) (2025-10-08)

View File

@@ -34,7 +34,8 @@
"Bash(npm run pre-commit:*)",
"Bash(npm run:*)",
"Bash(git pull:*)",
"Bash(git stash:*)"
"Bash(git stash:*)",
"Bash(members of the room\" requirement for room-based gameplay.\n\n🤖 Generated with [Claude Code](https://claude.com/claude-code)\n\nCo-Authored-By: Claude <noreply@anthropic.com>\nEOF\n)\")"
],
"deny": [],
"ask": []

View File

@@ -0,0 +1,371 @@
/**
* @vitest-environment node
*/
import { createServer } from 'http'
import { eq } from 'drizzle-orm'
import { io as ioClient, type Socket } from 'socket.io-client'
import { afterEach, beforeEach, describe, expect, it, afterAll, beforeAll } from 'vitest'
import { db, schema } from '../src/db'
import { createRoom } from '../src/lib/arcade/room-manager'
import { addRoomMember } from '../src/lib/arcade/room-membership'
import { initializeSocketServer } from '../socket-server'
import type { Server as SocketIOServerType } from 'socket.io'
/**
* Real-time Room Updates E2E Tests
*
* Tests that socket broadcasts work correctly when users join/leave rooms.
* Simulates multiple connected users and verifies they receive real-time updates.
*/
describe('Room Real-time Updates', () => {
let testUserId1: string
let testUserId2: string
let testGuestId1: string
let testGuestId2: string
let testRoomId: string
let socket1: Socket
let httpServer: any
let io: SocketIOServerType
let serverPort: number
beforeAll(async () => {
// Create HTTP server and initialize Socket.IO for testing
httpServer = createServer()
io = initializeSocketServer(httpServer)
// Find an available port
await new Promise<void>((resolve) => {
httpServer.listen(0, () => {
serverPort = (httpServer.address() as any).port
console.log(`Test socket server listening on port ${serverPort}`)
resolve()
})
})
})
afterAll(async () => {
// Close all socket connections
if (io) {
io.close()
}
if (httpServer) {
await new Promise<void>((resolve) => {
httpServer.close(() => resolve())
})
}
})
beforeEach(async () => {
// Create test users
testGuestId1 = `test-guest-${Date.now()}-${Math.random().toString(36).slice(2)}`
testGuestId2 = `test-guest-${Date.now()}-${Math.random().toString(36).slice(2)}`
const [user1] = await db.insert(schema.users).values({ guestId: testGuestId1 }).returning()
const [user2] = await db.insert(schema.users).values({ guestId: testGuestId2 }).returning()
testUserId1 = user1.id
testUserId2 = user2.id
// Create a test room
const room = await createRoom({
name: 'Realtime Test Room',
createdBy: testGuestId1,
creatorName: 'User 1',
gameName: 'matching',
gameConfig: { difficulty: 6 },
ttlMinutes: 60,
})
testRoomId = room.id
})
afterEach(async () => {
// Disconnect sockets
if (socket1?.connected) {
socket1.disconnect()
}
// Clean up room members
await db.delete(schema.roomMembers).where(eq(schema.roomMembers.roomId, testRoomId))
// Clean up rooms
if (testRoomId) {
await db.delete(schema.arcadeRooms).where(eq(schema.arcadeRooms.id, testRoomId))
}
// Clean up users
await db.delete(schema.users).where(eq(schema.users.id, testUserId1))
await db.delete(schema.users).where(eq(schema.users.id, testUserId2))
})
it('should broadcast member-joined when a user joins via API', async () => {
// User 1 joins the room via API first (this is what happens when they click "Join Room")
await addRoomMember({
roomId: testRoomId,
userId: testGuestId1,
displayName: 'User 1',
isCreator: false,
})
// User 1 connects to socket
socket1 = ioClient(`http://localhost:${serverPort}`, {
path: '/api/socket',
transports: ['websocket'],
})
// Wait for socket to connect
await new Promise<void>((resolve, reject) => {
socket1.on('connect', () => resolve())
socket1.on('connect_error', (err) => reject(err))
setTimeout(() => reject(new Error('Connection timeout')), 2000)
})
// Small delay to ensure event handlers are set up
await new Promise((resolve) => setTimeout(resolve, 50))
// Set up listener for room-joined BEFORE emitting
const roomJoinedPromise = new Promise<void>((resolve, reject) => {
socket1.on('room-joined', () => resolve())
socket1.on('room-error', (err) => reject(new Error(err.error)))
setTimeout(() => reject(new Error('Room-joined timeout')), 3000)
})
// Now emit the join-room event
socket1.emit('join-room', { roomId: testRoomId, userId: testGuestId1 })
// Wait for confirmation
await roomJoinedPromise
// Set up listener for member-joined event BEFORE User 2 joins
const memberJoinedPromise = new Promise<any>((resolve, reject) => {
socket1.on('member-joined', (data) => {
resolve(data)
})
setTimeout(() => reject(new Error('Timeout waiting for member-joined event')), 3000)
})
// User 2 joins the room via addRoomMember
const { member: newMember } = await addRoomMember({
roomId: testRoomId,
userId: testGuestId2,
displayName: 'User 2',
isCreator: false,
})
// Manually trigger the broadcast (this is what the API route SHOULD do)
const { getRoomMembers } = await import('../src/lib/arcade/room-membership')
const { getRoomActivePlayers } = await import('../src/lib/arcade/player-manager')
const members = await getRoomMembers(testRoomId)
const memberPlayers = await getRoomActivePlayers(testRoomId)
const memberPlayersObj: Record<string, any[]> = {}
for (const [uid, players] of memberPlayers.entries()) {
memberPlayersObj[uid] = players
}
io.to(`room:${testRoomId}`).emit('member-joined', {
roomId: testRoomId,
userId: testGuestId2,
members,
memberPlayers: memberPlayersObj,
})
// Wait for the socket broadcast with timeout
const data = await memberJoinedPromise
// Verify the broadcast data
expect(data).toBeDefined()
expect(data.roomId).toBe(testRoomId)
expect(data.userId).toBe(testGuestId2)
expect(data.members).toBeDefined()
expect(Array.isArray(data.members)).toBe(true)
// Verify both users are in the members list
const memberUserIds = data.members.map((m: any) => m.userId)
expect(memberUserIds).toContain(testGuestId1)
expect(memberUserIds).toContain(testGuestId2)
// Verify the new member details
const addedMember = data.members.find((m: any) => m.userId === testGuestId2)
expect(addedMember).toBeDefined()
expect(addedMember.displayName).toBe('User 2')
expect(addedMember.roomId).toBe(testRoomId)
})
it('should broadcast member-left when a user leaves via API', async () => {
// User 1 joins the room first
await addRoomMember({
roomId: testRoomId,
userId: testGuestId1,
displayName: 'User 1',
isCreator: false,
})
// User 2 joins the room
await addRoomMember({
roomId: testRoomId,
userId: testGuestId2,
displayName: 'User 2',
isCreator: false,
})
// User 1 connects to socket
socket1 = ioClient(`http://localhost:${serverPort}`, {
path: '/api/socket',
transports: ['websocket'],
})
await new Promise<void>((resolve) => {
socket1.on('connect', () => resolve())
})
socket1.emit('join-room', { roomId: testRoomId, userId: testGuestId1 })
await new Promise<void>((resolve) => {
socket1.on('room-joined', () => resolve())
})
// Set up listener for member-left event
const memberLeftPromise = new Promise<any>((resolve) => {
socket1.on('member-left', (data) => {
resolve(data)
})
})
// User 2 leaves the room via API
await db.delete(schema.roomMembers).where(eq(schema.roomMembers.userId, testGuestId2))
// Manually trigger the leave broadcast (simulating what the API does)
const { getSocketIO } = await import('../src/lib/socket-io')
const io = await getSocketIO()
if (io) {
const { getRoomMembers } = await import('../src/lib/arcade/room-membership')
const { getRoomActivePlayers } = await import('../src/lib/arcade/player-manager')
const members = await getRoomMembers(testRoomId)
const memberPlayers = await getRoomActivePlayers(testRoomId)
const memberPlayersObj: Record<string, any[]> = {}
for (const [uid, players] of memberPlayers.entries()) {
memberPlayersObj[uid] = players
}
io.to(`room:${testRoomId}`).emit('member-left', {
roomId: testRoomId,
userId: testGuestId2,
members,
memberPlayers: memberPlayersObj,
})
}
// Wait for the socket broadcast with timeout
const data = await Promise.race([
memberLeftPromise,
new Promise((_, reject) =>
setTimeout(() => reject(new Error('Timeout waiting for member-left event')), 2000)
),
])
// Verify the broadcast data
expect(data).toBeDefined()
expect(data.roomId).toBe(testRoomId)
expect(data.userId).toBe(testGuestId2)
expect(data.members).toBeDefined()
expect(Array.isArray(data.members)).toBe(true)
// Verify User 2 is no longer in the members list
const memberUserIds = data.members.map((m: any) => m.userId)
expect(memberUserIds).toContain(testGuestId1)
expect(memberUserIds).not.toContain(testGuestId2)
})
it('should update both members and players lists in member-joined broadcast', async () => {
// Create an active player for User 2
const [player2] = await db
.insert(schema.players)
.values({
userId: testUserId2,
name: 'Player 2',
emoji: '🎮',
color: '#3b82f6',
isActive: true,
})
.returning()
// User 1 connects and joins room
socket1 = ioClient(`http://localhost:${serverPort}`, {
path: '/api/socket',
transports: ['websocket'],
})
await new Promise<void>((resolve) => {
socket1.on('connect', () => resolve())
})
socket1.emit('join-room', { roomId: testRoomId, userId: testGuestId1 })
await new Promise<void>((resolve) => {
socket1.on('room-joined', () => resolve())
})
const memberJoinedPromise = new Promise<any>((resolve) => {
socket1.on('member-joined', (data) => {
resolve(data)
})
})
// User 2 joins via API
await addRoomMember({
roomId: testRoomId,
userId: testGuestId2,
displayName: 'User 2',
isCreator: false,
})
// Manually trigger the broadcast (simulating what the API does)
const { getRoomMembers: getRoomMembers3 } = await import('../src/lib/arcade/room-membership')
const { getRoomActivePlayers: getRoomActivePlayers3 } = await import(
'../src/lib/arcade/player-manager'
)
const members2 = await getRoomMembers3(testRoomId)
const memberPlayers2 = await getRoomActivePlayers3(testRoomId)
const memberPlayersObj2: Record<string, any[]> = {}
for (const [uid, players] of memberPlayers2.entries()) {
memberPlayersObj2[uid] = players
}
io.to(`room:${testRoomId}`).emit('member-joined', {
roomId: testRoomId,
userId: testGuestId2,
members: members2,
memberPlayers: memberPlayersObj2,
})
const data = await Promise.race([
memberJoinedPromise,
new Promise((_, reject) => setTimeout(() => reject(new Error('Timeout')), 2000)),
])
// Verify members list is updated
expect(data.members).toBeDefined()
const memberUserIds = data.members.map((m: any) => m.userId)
expect(memberUserIds).toContain(testGuestId2)
// Verify players list is updated
expect(data.memberPlayers).toBeDefined()
expect(data.memberPlayers[testGuestId2]).toBeDefined()
expect(Array.isArray(data.memberPlayers[testGuestId2])).toBe(true)
// User 2's players should include the active player we created
const user2Players = data.memberPlayers[testGuestId2]
expect(user2Players.length).toBeGreaterThan(0)
expect(user2Players.some((p: any) => p.id === player2.id)).toBe(true)
// Clean up player
await db.delete(schema.players).where(eq(schema.players.id, player2.id))
})
})

View File

@@ -1,29 +0,0 @@
const { Server } = require('socket.io')
function initializeSocketServer(httpServer) {
const io = new Server(httpServer, {
path: '/api/socket',
cors: {
origin: process.env.NEXT_PUBLIC_URL || 'http://localhost:3000',
credentials: true,
},
})
io.on('connection', (socket) => {
console.log('🔌 Client connected:', socket.id)
socket.on('join-arcade-session', ({ userId }) => {
socket.join(`arcade:${userId}`)
console.log(`👤 User ${userId} joined arcade room`)
})
socket.on('disconnect', () => {
console.log('🔌 Client disconnected:', socket.id)
})
})
console.log('✅ Socket.IO initialized on /api/socket')
return io
}
module.exports = { initializeSocketServer }

View File

@@ -9,29 +9,27 @@ import {
updateSessionActivity,
} from './src/lib/arcade/session-manager'
import { createRoom, getRoomById } from './src/lib/arcade/room-manager'
import {
getOnlineRoomMembers,
getRoomMembers,
getUserRooms,
setMemberOnline,
} from './src/lib/arcade/room-membership'
import { getRoomMembers, getUserRooms, setMemberOnline } from './src/lib/arcade/room-membership'
import { getRoomActivePlayers } from './src/lib/arcade/player-manager'
import type { GameMove, GameName } from './src/lib/arcade/validation'
import { matchingGameValidator } from './src/lib/arcade/validation/MatchingGameValidator'
// Global socket.io server instance
let io: SocketIOServerType | null = null
// Use globalThis to store socket.io instance to avoid module isolation issues
// This ensures the same instance is accessible across dynamic imports
declare global {
var __socketIO: SocketIOServerType | undefined
}
/**
* Get the socket.io server instance
* Returns null if not initialized
*/
export function getSocketIO(): SocketIOServerType | null {
return io
return globalThis.__socketIO || null
}
export function initializeSocketServer(httpServer: HTTPServer) {
io = new SocketIOServer(httpServer, {
const io = new SocketIOServer(httpServer, {
path: '/api/socket',
cors: {
origin: process.env.NEXT_PUBLIC_URL || 'http://localhost:3000',
@@ -155,7 +153,7 @@ export function initializeSocketServer(httpServer: HTTPServer) {
// Notify all connected clients about the new session
const newSession = await getArcadeSession(data.userId)
if (newSession) {
io.to(`arcade:${data.userId}`).emit('session-state', {
io!.to(`arcade:${data.userId}`).emit('session-state', {
gameState: newSession.gameState,
currentGame: newSession.currentGame,
gameUrl: newSession.gameUrl,
@@ -171,7 +169,7 @@ export function initializeSocketServer(httpServer: HTTPServer) {
if (result.success && result.session) {
// Broadcast the updated state to all devices for this user
io.to(`arcade:${data.userId}`).emit('move-accepted', {
io!.to(`arcade:${data.userId}`).emit('move-accepted', {
gameState: result.session.gameState,
version: result.session.version,
move: data.move,
@@ -202,7 +200,7 @@ export function initializeSocketServer(httpServer: HTTPServer) {
try {
await deleteArcadeSession(userId)
io.to(`arcade:${userId}`).emit('session-ended')
io!.to(`arcade:${userId}`).emit('session-ended')
} catch (error) {
console.error('Error ending session:', error)
socket.emit('session-error', { error: 'Failed to end session' })
@@ -232,7 +230,6 @@ export function initializeSocketServer(httpServer: HTTPServer) {
// Get room data
const members = await getRoomMembers(roomId)
const onlineMembers = await getOnlineRoomMembers(roomId)
const memberPlayers = await getRoomActivePlayers(roomId)
// Convert memberPlayers Map to object for JSON serialization
@@ -245,7 +242,6 @@ export function initializeSocketServer(httpServer: HTTPServer) {
socket.emit('room-joined', {
roomId,
members,
onlineMembers,
memberPlayers: memberPlayersObj,
})
@@ -253,7 +249,7 @@ export function initializeSocketServer(httpServer: HTTPServer) {
socket.to(`room:${roomId}`).emit('member-joined', {
roomId,
userId,
onlineMembers,
members,
memberPlayers: memberPlayersObj,
})
@@ -275,8 +271,8 @@ export function initializeSocketServer(httpServer: HTTPServer) {
// Mark member as offline
await setMemberOnline(roomId, userId, false)
// Get updated online members
const onlineMembers = await getOnlineRoomMembers(roomId)
// Get updated members
const members = await getRoomMembers(roomId)
const memberPlayers = await getRoomActivePlayers(roomId)
// Convert memberPlayers Map to object
@@ -286,10 +282,10 @@ export function initializeSocketServer(httpServer: HTTPServer) {
}
// Notify remaining members
io.to(`room:${roomId}`).emit('member-left', {
io!.to(`room:${roomId}`).emit('member-left', {
roomId,
userId,
onlineMembers,
members,
memberPlayers: memberPlayersObj,
})
@@ -314,7 +310,7 @@ export function initializeSocketServer(httpServer: HTTPServer) {
}
// Broadcast to all members in the room (including sender)
io.to(`room:${roomId}`).emit('room-players-updated', {
io!.to(`room:${roomId}`).emit('room-players-updated', {
roomId,
memberPlayers: memberPlayersObj,
})
@@ -335,6 +331,8 @@ export function initializeSocketServer(httpServer: HTTPServer) {
})
})
// Store in globalThis to make accessible across module boundaries
globalThis.__socketIO = io
console.log('✅ Socket.IO initialized on /api/socket')
return io
}

View File

@@ -1,6 +1,6 @@
import { type NextRequest, NextResponse } from 'next/server'
import { getRoomById, touchRoom } from '@/lib/arcade/room-manager'
import { addRoomMember, getOnlineRoomMembers } from '@/lib/arcade/room-membership'
import { addRoomMember, getRoomMembers } from '@/lib/arcade/room-membership'
import { getActivePlayers, getRoomActivePlayers } from '@/lib/arcade/player-manager'
import { getViewerId } from '@/lib/viewer'
import { getSocketIO } from '@/lib/socket-io'
@@ -61,7 +61,7 @@ export async function POST(req: NextRequest, context: RouteContext) {
const io = await getSocketIO()
if (io) {
try {
const onlineMembers = await getOnlineRoomMembers(roomId)
const members = await getRoomMembers(roomId)
const memberPlayers = await getRoomActivePlayers(roomId)
// Convert memberPlayers Map to object for JSON serialization
@@ -74,7 +74,7 @@ export async function POST(req: NextRequest, context: RouteContext) {
io.to(`room:${roomId}`).emit('member-joined', {
roomId,
userId: viewerId,
onlineMembers,
members,
memberPlayers: memberPlayersObj,
})

View File

@@ -1,6 +1,6 @@
import { type NextRequest, NextResponse } from 'next/server'
import { getRoomById } from '@/lib/arcade/room-manager'
import { getOnlineRoomMembers, isMember, removeMember } from '@/lib/arcade/room-membership'
import { getRoomMembers, isMember, removeMember } from '@/lib/arcade/room-membership'
import { getRoomActivePlayers } from '@/lib/arcade/player-manager'
import { getViewerId } from '@/lib/viewer'
import { getSocketIO } from '@/lib/socket-io'
@@ -37,7 +37,7 @@ export async function POST(_req: NextRequest, context: RouteContext) {
const io = await getSocketIO()
if (io) {
try {
const onlineMembers = await getOnlineRoomMembers(roomId)
const members = await getRoomMembers(roomId)
const memberPlayers = await getRoomActivePlayers(roomId)
// Convert memberPlayers Map to object for JSON serialization
@@ -50,7 +50,7 @@ export async function POST(_req: NextRequest, context: RouteContext) {
io.to(`room:${roomId}`).emit('member-left', {
roomId,
userId: viewerId,
onlineMembers,
members,
memberPlayers: memberPlayersObj,
})

View File

@@ -0,0 +1,52 @@
import { NextResponse } from 'next/server'
import { getUserRooms } from '@/lib/arcade/room-membership'
import { getRoomById } from '@/lib/arcade/room-manager'
import { getRoomMembers } from '@/lib/arcade/room-membership'
import { getRoomActivePlayers } from '@/lib/arcade/player-manager'
import { getViewerId } from '@/lib/viewer'
/**
* GET /api/arcade/rooms/current
* Returns the user's current room (if any)
*/
export async function GET() {
try {
const userId = await getViewerId()
// Get all rooms user is in (should be at most 1 due to modal room enforcement)
const roomIds = await getUserRooms(userId)
if (roomIds.length === 0) {
return NextResponse.json({ room: null }, { status: 200 })
}
const roomId = roomIds[0]
// Get room data
const room = await getRoomById(roomId)
if (!room) {
return NextResponse.json({ error: 'Room not found' }, { status: 404 })
}
// Get members
const members = await getRoomMembers(roomId)
// Get active players for all members
const memberPlayers = await getRoomActivePlayers(roomId)
// Convert Map to object for JSON serialization
const memberPlayersObj: Record<string, any[]> = {}
for (const [uid, players] of memberPlayers.entries()) {
memberPlayersObj[uid] = players
}
return NextResponse.json({
room,
members,
memberPlayers: memberPlayersObj,
})
} catch (error) {
console.error('[Current Room API] Error:', error)
return NextResponse.json({ error: 'Failed to fetch current room' }, { status: 500 })
}
}

View File

@@ -0,0 +1,77 @@
'use client'
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'
/**
* /arcade/room - Renders the game for the user's current room
* Since users can only be in one room at a time, this is a simple singular route
*/
export default function RoomPage() {
const router = useRouter()
const { roomData, isLoading } = useRoomData()
// Redirect to arcade if no room
useEffect(() => {
if (!isLoading && !roomData) {
console.log('[RoomPage] No active room, redirecting to /arcade')
router.push('/arcade')
}
}, [isLoading, roomData, router])
// Show loading state
if (isLoading) {
return (
<div
style={{
display: 'flex',
alignItems: 'center',
justifyContent: 'center',
height: '100vh',
fontSize: '18px',
color: '#666',
}}
>
Loading room...
</div>
)
}
// Show nothing while redirecting
if (!roomData) {
return null
}
// Render the appropriate game based on room's gameName
switch (roomData.gameName) {
case 'matching':
return (
<ArcadeGuardedPage>
<ArcadeMemoryPairsProvider>
<MemoryPairsGame />
</ArcadeMemoryPairsProvider>
</ArcadeGuardedPage>
)
// TODO: Add other games (complement-race, memory-quiz, etc.)
default:
return (
<div
style={{
display: 'flex',
alignItems: 'center',
justifyContent: 'center',
height: '100vh',
fontSize: '18px',
color: '#666',
}}
>
Game "{roomData.gameName}" not yet supported
</div>
)
}
}

View File

@@ -83,8 +83,8 @@ export default function RoomDetailPage() {
sock.on('member-joined', (data) => {
console.log('Member joined:', data)
if (data.onlineMembers) {
setMembers(data.onlineMembers)
if (data.members) {
setMembers(data.members)
}
if (data.memberPlayers) {
setMemberPlayers(data.memberPlayers)
@@ -93,8 +93,8 @@ export default function RoomDetailPage() {
sock.on('member-left', (data) => {
console.log('Member left:', data)
if (data.onlineMembers) {
setMembers(data.onlineMembers)
if (data.members) {
setMembers(data.members)
}
if (data.memberPlayers) {
setMemberPlayers(data.memberPlayers)

View File

@@ -3,6 +3,8 @@
import React from 'react'
import { useGameMode } from '../contexts/GameModeContext'
import { useArcadeGuard } from '../hooks/useArcadeGuard'
import { useRoomData } from '../hooks/useRoomData'
import { useViewerId } from '../hooks/useViewerId'
import { AppNavBar } from './AppNavBar'
import { GameContextNav } from './nav/GameContextNav'
import { PlayerConfigDialog } from './nav/PlayerConfigDialog'
@@ -30,6 +32,8 @@ export function PageWithNav({
}: PageWithNavProps) {
const { players, activePlayers, setActive, activePlayerCount } = useGameMode()
const { hasActiveSession, activeSession } = useArcadeGuard({ enabled: false }) // Don't redirect, just get info
const { roomData, isInRoom } = useRoomData()
const { data: viewerId } = useViewerId()
const [mounted, setMounted] = React.useState(false)
const [configurePlayerId, setConfigurePlayerId] = React.useState<string | null>(null)
@@ -80,17 +84,33 @@ export function PageWithNav({
// Compute arcade session info for display
const roomInfo =
hasActiveSession && activeSession
isInRoom && roomData
? {
gameName: activeSession.currentGame,
playerCount: activePlayerCount, // TODO: Get actual player count from session when available
roomName: roomData.name,
gameName: roomData.gameName,
playerCount: roomData.members.length,
}
: undefined
: hasActiveSession && activeSession
? {
gameName: activeSession.currentGame,
playerCount: activePlayerCount,
}
: undefined
// Compute network players (other players in the arcade session)
// For now, we don't have this info in activeSession, so return empty array
// TODO: When arcade room system is implemented, fetch other players from session
const networkPlayers: Array<{ id: string; emoji?: string; name?: string }> = []
// Compute network players (other players in the room, excluding current user)
const networkPlayers: Array<{ id: string; emoji?: string; name?: string }> =
isInRoom && roomData
? roomData.members
.filter((member) => member.userId !== viewerId)
.flatMap((member) => {
const memberPlayerList = roomData.memberPlayers[member.userId] || []
return memberPlayerList.map((player) => ({
id: player.id,
emoji: player.emoji,
name: `${player.name} (${member.displayName})`,
}))
})
: []
// Create nav content if title is provided
const navContent = navTitle ? (

View File

@@ -22,6 +22,7 @@ interface NetworkPlayer {
}
interface ArcadeRoomInfo {
roomName?: string
gameName: string
playerCount: number
}
@@ -134,6 +135,7 @@ export function GameContextNav({
{/* Room Info - show when in arcade session */}
{roomInfo && !showFullscreenSelection && (
<RoomInfo
roomName={roomInfo.roomName}
gameName={roomInfo.gameName}
playerCount={roomInfo.playerCount}
shouldEmphasize={shouldEmphasize}

View File

@@ -1,4 +1,5 @@
interface RoomInfoProps {
roomName?: string
gameName: string
playerCount: number
shouldEmphasize: boolean
@@ -7,7 +8,7 @@ interface RoomInfoProps {
/**
* Displays current arcade room/session information
*/
export function RoomInfo({ gameName, playerCount, shouldEmphasize }: RoomInfoProps) {
export function RoomInfo({ roomName, gameName, playerCount, shouldEmphasize }: RoomInfoProps) {
return (
<div
style={{
@@ -53,7 +54,7 @@ export function RoomInfo({ gameName, playerCount, shouldEmphasize }: RoomInfoPro
letterSpacing: '0.5px',
}}
>
Arcade Session
{roomName ? 'Room' : 'Arcade Session'}
</div>
<div
style={{
@@ -61,7 +62,7 @@ export function RoomInfo({ gameName, playerCount, shouldEmphasize }: RoomInfoPro
fontWeight: 'bold',
}}
>
{gameName}
{roomName || gameName}
</div>
</div>

View File

@@ -1,6 +1,7 @@
'use client'
import { createContext, type ReactNode, useContext, useEffect, useMemo, useState } from 'react'
import { io } from 'socket.io-client'
import type { Player as DBPlayer } from '@/db/schema/players'
import {
useCreatePlayer,
@@ -8,6 +9,8 @@ import {
useUpdatePlayer,
useUserPlayers,
} from '@/hooks/useUserPlayers'
import { useRoomData } from '@/hooks/useRoomData'
import { useViewerId } from '@/hooks/useViewerId'
import { getNextPlayerColor } from '../types/player'
// Client-side Player type (compatible with old type)
@@ -66,28 +69,72 @@ export function GameModeProvider({ children }: { children: ReactNode }) {
const { mutate: createPlayer } = useCreatePlayer()
const { mutate: updatePlayerMutation } = useUpdatePlayer()
const { mutate: deletePlayer } = useDeletePlayer()
const { roomData } = useRoomData()
const { data: viewerId } = useViewerId()
const [isInitialized, setIsInitialized] = useState(false)
// Convert DB players to Map
const players = useMemo(() => {
// Convert DB players to Map (local players)
const localPlayers = useMemo(() => {
const map = new Map<string, Player>()
dbPlayers.forEach((dbPlayer) => {
map.set(dbPlayer.id, toClientPlayer(dbPlayer))
map.set(dbPlayer.id, {
...toClientPlayer(dbPlayer),
isLocal: true,
})
})
return map
}, [dbPlayers])
// Track active players from DB isActive status
// When in a room, merge all players from all room members
const players = useMemo(() => {
const map = new Map<string, Player>(localPlayers)
if (roomData) {
// Add players from other room members (marked as remote)
Object.entries(roomData.memberPlayers).forEach(([userId, memberPlayers]) => {
// Skip the current user's players (already in localPlayers)
if (userId === viewerId) return
memberPlayers.forEach((roomPlayer) => {
map.set(roomPlayer.id, {
id: roomPlayer.id,
name: roomPlayer.name,
emoji: roomPlayer.emoji,
color: roomPlayer.color,
createdAt: Date.now(),
isActive: true, // Players in memberPlayers are active
isLocal: false, // Remote player
})
})
})
}
return map
}, [localPlayers, roomData, viewerId])
// Track active players (local + room members when in a room)
const activePlayers = useMemo(() => {
const set = new Set<string>()
dbPlayers.forEach((player) => {
if (player.isActive) {
set.add(player.id)
}
})
if (roomData) {
// In room mode: all players from all members are active
Object.values(roomData.memberPlayers).forEach((memberPlayers) => {
memberPlayers.forEach((player) => {
set.add(player.id)
})
})
} else {
// Solo mode: only local active players
dbPlayers.forEach((player) => {
if (player.isActive) {
set.add(player.id)
}
})
}
return set
}, [dbPlayers])
}, [dbPlayers, roomData])
// Initialize with default players if none exist
useEffect(() => {
@@ -111,6 +158,26 @@ export function GameModeProvider({ children }: { children: ReactNode }) {
}
}, [dbPlayers, isLoading, isInitialized, createPlayer])
// When in a room, broadcast player updates to other members
useEffect(() => {
if (!roomData || !viewerId || !isInitialized) return
const socket = io({ path: '/api/socket' })
// Wait for connection before emitting
socket.on('connect', () => {
console.log('[GameModeContext] Emitting players-updated for room:', roomData.id)
socket.emit('players-updated', {
roomId: roomData.id,
userId: viewerId,
})
})
return () => {
socket.disconnect()
}
}, [dbPlayers, roomData, viewerId, isInitialized])
const addPlayer = (playerData?: Partial<Player>) => {
const playerList = Array.from(players.values())
@@ -125,15 +192,33 @@ export function GameModeProvider({ children }: { children: ReactNode }) {
}
const updatePlayer = (id: string, updates: Partial<Player>) => {
updatePlayerMutation({ id, updates })
const player = players.get(id)
// Only allow updating local players
if (player?.isLocal) {
updatePlayerMutation({ id, updates })
} else {
console.warn('[GameModeContext] Cannot update remote player:', id)
}
}
const removePlayer = (id: string) => {
deletePlayer(id)
const player = players.get(id)
// Only allow removing local players
if (player?.isLocal) {
deletePlayer(id)
} else {
console.warn('[GameModeContext] Cannot remove remote player:', id)
}
}
const setActive = (id: string, active: boolean) => {
updatePlayerMutation({ id, updates: { isActive: active } })
const player = players.get(id)
// Only allow changing active status of local players
if (player?.isLocal) {
updatePlayerMutation({ id, updates: { isActive: active } })
} else {
console.warn('[GameModeContext] Cannot change active status of remote player:', id)
}
}
const getActivePlayers = (): Player[] => {

View File

@@ -0,0 +1,203 @@
import { useEffect, useState } from 'react'
import { io, type Socket } from 'socket.io-client'
import { useViewerId } from './useViewerId'
export interface RoomMember {
id: string
userId: string
displayName: string
isOnline: boolean
isCreator: boolean
}
export interface RoomPlayer {
id: string
name: string
emoji: string
color: string
}
export interface RoomData {
id: string
name: string
code: string
gameName: string
members: RoomMember[]
memberPlayers: Record<string, RoomPlayer[]> // userId -> players
}
/**
* Hook to fetch and subscribe to the user's current room data
* Returns null if user is not in any room
*/
export function useRoomData() {
const { data: userId } = useViewerId()
const [socket, setSocket] = useState<Socket | null>(null)
const [roomData, setRoomData] = useState<RoomData | null>(null)
const [isLoading, setIsLoading] = useState(false)
// Fetch the user's current room
useEffect(() => {
if (!userId) {
setRoomData(null)
return
}
setIsLoading(true)
// Fetch current room data
fetch('/api/arcade/rooms/current')
.then((res) => {
if (!res.ok) throw new Error('Failed to fetch current room')
return res.json()
})
.then((data) => {
if (data.room) {
setRoomData({
id: data.room.id,
name: data.room.name,
code: data.room.code,
gameName: data.room.gameName,
members: data.members || [],
memberPlayers: data.memberPlayers || {},
})
} else {
setRoomData(null)
}
setIsLoading(false)
})
.catch((error) => {
console.error('Failed to fetch room data:', error)
setRoomData(null)
setIsLoading(false)
})
}, [userId])
// Initialize socket connection when user has a room
useEffect(() => {
if (!roomData?.id || !userId) {
if (socket) {
socket.disconnect()
setSocket(null)
}
return
}
const sock = io({ path: '/api/socket' })
sock.on('connect', () => {
console.log('[useRoomData] Socket connected, joining room:', roomData.id)
// Join the room to receive updates
sock.emit('join-room', { roomId: roomData.id, userId })
})
sock.on('disconnect', () => {
console.log('[useRoomData] Socket disconnected')
})
setSocket(sock)
return () => {
if (sock.connected) {
// Leave the room before disconnecting
sock.emit('leave-room', { roomId: roomData.id, userId })
sock.disconnect()
}
}
}, [roomData?.id, userId])
// Subscribe to real-time updates via socket
useEffect(() => {
if (!socket || !roomData?.id) return
const handleRoomJoined = (data: {
roomId: string
members: RoomMember[]
memberPlayers: Record<string, RoomPlayer[]>
}) => {
console.log('[useRoomData] Received room-joined event:', data)
if (data.roomId === roomData.id) {
setRoomData((prev) => {
if (!prev) return null
return {
...prev,
members: data.members,
memberPlayers: data.memberPlayers,
}
})
}
}
const handleMemberJoined = (data: {
roomId: string
userId: string
members: RoomMember[]
memberPlayers: Record<string, RoomPlayer[]>
}) => {
console.log('[useRoomData] Received member-joined event:', data)
if (data.roomId === roomData.id) {
setRoomData((prev) => {
if (!prev) return null
return {
...prev,
members: data.members,
memberPlayers: data.memberPlayers,
}
})
}
}
const handleMemberLeft = (data: {
roomId: string
userId: string
members: RoomMember[]
memberPlayers: Record<string, RoomPlayer[]>
}) => {
console.log('[useRoomData] Received member-left event:', data)
if (data.roomId === roomData.id) {
setRoomData((prev) => {
if (!prev) return null
return {
...prev,
members: data.members,
memberPlayers: data.memberPlayers,
}
})
}
}
const handleRoomPlayersUpdated = (data: {
roomId: string
memberPlayers: Record<string, RoomPlayer[]>
}) => {
console.log('[useRoomData] Received room-players-updated event:', data)
if (data.roomId === roomData.id) {
setRoomData((prev) => {
if (!prev) return null
return {
...prev,
memberPlayers: data.memberPlayers,
}
})
}
}
socket.on('room-joined', handleRoomJoined)
socket.on('member-joined', handleMemberJoined)
socket.on('member-left', handleMemberLeft)
socket.on('room-players-updated', handleRoomPlayersUpdated)
return () => {
socket.off('room-joined', handleRoomJoined)
socket.off('member-joined', handleMemberJoined)
socket.off('member-left', handleMemberLeft)
socket.off('room-players-updated', handleRoomPlayersUpdated)
}
}, [socket, roomData?.id])
return {
roomData,
isLoading,
isInRoom: !!roomData,
}
}

View File

@@ -6,34 +6,35 @@
import type { Server as SocketIOServerType } from 'socket.io'
// Import the socket server module (this is safe because it's only used in Node.js context)
let socketServer: { getSocketIO: () => SocketIOServerType | null } | null = null
// Lazy-load the socket server module (only works on server-side)
async function loadSocketServer() {
if (typeof window !== 'undefined') {
// Client-side: return null
return null
}
if (!socketServer) {
try {
// Dynamic import to avoid bundling issues
socketServer = await import('../../socket-server')
} catch (error) {
console.error('[Socket IO] Failed to load socket server:', error)
return null
}
}
return socketServer
}
// Cache for the socket server module
let socketServerModule: any = null
/**
* Get the socket.io server instance
* Returns null if not initialized or if called on client-side
*/
export async function getSocketIO(): Promise<SocketIOServerType | null> {
const server = await loadSocketServer()
return server ? server.getSocketIO() : null
// Client-side: return null
if (typeof window !== 'undefined') {
return null
}
// Lazy-load the socket server module on first call
if (!socketServerModule) {
try {
// Dynamic import to avoid bundling issues
socketServerModule = await import('../../socket-server')
} catch (error) {
console.error('[Socket IO] Failed to load socket server:', error)
return null
}
}
// Call the exported getSocketIO function from the module
if (socketServerModule && typeof socketServerModule.getSocketIO === 'function') {
return socketServerModule.getSocketIO()
}
console.warn('[Socket IO] getSocketIO function not found in socket-server module')
return null
}

View File

@@ -1,6 +1,6 @@
{
"name": "soroban-monorepo",
"version": "2.4.3",
"version": "2.7.0",
"private": true,
"description": "Beautiful Soroban Flashcard Generator - Monorepo",
"workspaces": [