Fix all ESLint errors and missing return types in shared workspace

Co-authored-by: chrisreddington <[email protected]>

Author: Copilot

shared/src/constants/game-constants.ts

@@ -81,6 +81,6 @@ export function isValidGameStatus(status: string): status is GameStatus {
 /**
  * Get difficulty display configuration
  */
-export function getDifficultyDisplay(difficulty: Difficulty) {
+export function getDifficultyDisplay(difficulty: Difficulty): { emoji: string; label: string } {
   return DIFFICULTY_DISPLAY[difficulty]
 }

shared/src/games/rock-paper-scissors.ts

@@ -95,7 +95,7 @@ export class RockPaperScissorsGame implements Game<RPSGameState, RPSMove> {
     newRounds[gameState.currentRound] = currentRound;
 
     let newCurrentRound = gameState.currentRound;
-    let newScores = { ...gameState.scores };
+    const newScores = { ...gameState.scores };
     let newCurrentPlayerId = gameState.currentPlayerId;
 
     // If both players have made their choices, resolve the round

shared/src/storage/game-storage.test.ts

@@ -1,7 +1,7 @@
 import { vi } from 'vitest'
 import * as gameStorage from './game-storage';
 import * as sqliteStorage from './sqlite-storage';
-import type { GameSession, Player } from '../types/game';
+import type { GameSession } from '../types/game';
 import type { TicTacToeGameState, RPSGameState } from '../types/games';
 
 // Mock sqlite-storage module

shared/src/storage/sqlite-storage.ts

@@ -1,5 +1,4 @@
 import sqlite3 from 'sqlite3'
-import { promisify } from 'util'
 import path from 'path'
 import type { GameSession } from '../types/game'
 import type { TicTacToeGameState, RPSGameState } from '../types/games'

shared/src/testing/api-test-utils.ts

@@ -13,7 +13,13 @@ import type { TicTacToeGameState, RPSGameState, TicTacToeMove, RPSMove } from '.
  * Generic game mock factory
  * Creates a mock game instance with all required methods
  */
-export function createGameMock() {
+export function createGameMock(): {
+  getInitialState: ReturnType<typeof vi.fn>;
+  validateMove: ReturnType<typeof vi.fn>;
+  applyMove: ReturnType<typeof vi.fn>;
+  checkGameEnd: ReturnType<typeof vi.fn>;
+  getValidMoves: ReturnType<typeof vi.fn>;
+} {
   return {
     getInitialState: vi.fn(),
     validateMove: vi.fn(),
@@ -95,7 +101,10 @@ export function createMockGameSession<T extends BaseGameState>(gameState: T, gam
  * Vitest mock configuration for shared package games
  * Use this to create consistent mocks across API route tests
  */
-export function createSharedGameMocks(gameClass: string) {
+export function createSharedGameMocks(gameClass: string): {
+  mockImplementation: any;
+  mockGame: ReturnType<typeof createGameMock>;
+} {
   const mockGame = createGameMock()
 
   return {
@@ -110,7 +119,7 @@ export function createSharedGameMocks(gameClass: string) {
 /**
  * Create storage function mocks for a specific game type
  */
-export function createStorageMocks(gameType: GameType) {
+export function createStorageMocks(gameType: GameType): Record<string, ReturnType<typeof vi.fn>> {
   if (gameType === 'tic-tac-toe') {
     return {
       getTicTacToeGame: vi.fn(),

shared/src/testing/test-database.ts

@@ -51,7 +51,7 @@ export async function setupTestDatabase(
       }
 
       // Create tables synchronously to ensure they exist before tests run
-      const createTables = async () => {
+      const createTables = async (): Promise<void> => {
         try {
           await createTicTacToeTable()
           await createRPSTable()

shared/src/testing/vitest-setup.ts

@@ -12,7 +12,7 @@ import { setupTestDatabase, teardownTestDatabase } from './test-database'
  * Standard database setup for all packages
  * Call this in vitest.setup.ts files to ensure consistent database setup
  */
-export function setupStandardTestDatabase() {
+export function setupStandardTestDatabase(): void {
   // Setup test database before all tests
   beforeAll(async () => {
     try {

shared/src/types/games.ts

@@ -1,4 +1,4 @@
-import { BaseGameState, GameMove } from './game';
+import { BaseGameState } from './game';
 
 // Tic-tac-toe specific types
 export type CellValue = 'X' | 'O' | null;