chore: rename methods for posterity

Author: kevinjosethomas

src/api/lichess/streaming.ts

@@ -67,13 +67,11 @@ export const getLichessGameInfo = async (gameId: string) => {
 
 export const streamLichessGame = async (
   gameId: string,
-  onGameStart: (data: StreamedGame) => void,
+  onGameInfo: (data: StreamedGame) => void,
   onMove: (data: StreamedMove) => void,
   onComplete: () => void,
   abortSignal?: AbortSignal,
 ) => {
-  console.log(`Starting stream for game ${gameId}`)
-
   const stream = fetch(`https://lichess.org/api/stream/game/${gameId}`, {
     signal: abortSignal,
     headers: {
@@ -83,10 +81,8 @@ export const streamLichessGame = async (
 
   const onMessage = (message: any) => {
     if (message.id) {
-      console.log('Game start message:', message)
-      onGameStart(message as StreamedGame)
+      onGameInfo(message as StreamedGame)
     } else if (message.uci || message.lm) {
-      console.log('Move message:', message)
       onMove({
         fen: message.fen,
         uci: message.uci || message.lm,

src/components/Home/LiveChessBoard.tsx

@@ -37,7 +37,6 @@ export const LiveChessBoard: React.FC = () => {
   const abortController = useRef<AbortController | null>(null)
 
   const handleGameStart = useCallback((gameData: StreamedGame) => {
-    console.log('Live board - Game started:', gameData)
     if (gameData.fen) {
       setCurrentFen(gameData.fen)
     }
@@ -51,15 +50,13 @@ export const LiveChessBoard: React.FC = () => {
   }, [])
 
   const handleMove = useCallback((moveData: StreamedMove) => {
-    console.log('Live board - New move:', moveData)
     if (moveData.fen) {
       setCurrentFen(moveData.fen)
     }
   }, [])
 
   const handleStreamComplete = useCallback(() => {
     console.log('Live board - Stream completed')
-    // Try to get a new live game
     fetchNewGame()
   }, [])
 
@@ -83,7 +80,6 @@ export const LiveChessBoard: React.FC = () => {
         isLive: true,
       })
 
-      // Start streaming the new game
       streamLichessGame(
         tvGame.gameId,
         handleGameStart,

src/hooks/useLichessStreamController.ts

@@ -84,14 +84,11 @@ export const useLichessStreamController = (): LichessStreamController => {
     })
   }, [clearReconnectTimeout])
 
-  const handleGameStart = useCallback(
+  const handleStreamedGameInfo = useCallback(
     (gameData: StreamedGame) => {
       setGame((prev) => {
-        console.log('CHECKING FOR GAME END', prev?.id, gameData.id)
         // if the game is already loaded, this is a game termination message
         if (prev?.id === gameData.id) {
-          console.log('GAME ENDED')
-
           setStreamState((prev) => ({
             ...prev,
             gameEnded: true,
@@ -123,8 +120,6 @@ export const useLichessStreamController = (): LichessStreamController => {
 
   const handleMove = useCallback(
     (moveData: StreamedMove) => {
-      console.log('HANDLE MOVE:', moveData)
-
       streamMoves.current.push(moveData)
 
       if (moveData.wc !== undefined && moveData.bc !== undefined) {
@@ -154,12 +149,6 @@ export const useLichessStreamController = (): LichessStreamController => {
         try {
           const newGame = parseLichessStreamMove(moveData, prev)
 
-          if (!newGame.loaded) {
-            if (newGame.loadedFen === moveData.fen) {
-              console.log('LOADED GAME')
-            }
-          }
-
           return {
             ...newGame,
             loaded: newGame.loaded
@@ -176,8 +165,6 @@ export const useLichessStreamController = (): LichessStreamController => {
   )
 
   const handleStreamComplete = useCallback(() => {
-    console.log('Stream completed')
-
     setStreamState((prev) => ({
       ...prev,
       isConnected: false,
@@ -207,16 +194,14 @@ export const useLichessStreamController = (): LichessStreamController => {
 
       try {
         // Start streaming directly - the stream API will handle invalid game IDs
-        // Note: isConnected will be set when we receive the first data (in handleGameStart or handleMove)
+        // Note: isConnected will be set when we receive the first data (in handleStreamedGameInfo or handleMove)
         await streamLichessGame(
           gameId,
-          handleGameStart,
+          handleStreamedGameInfo,
           handleMove,
           handleStreamComplete,
           abortController.current.signal,
         )
-
-        console.log('Stream completed')
       } catch (error) {
         console.error('Stream error:', error)
 
@@ -235,7 +220,7 @@ export const useLichessStreamController = (): LichessStreamController => {
         abortController.current = null
       }
     },
-    [handleGameStart, handleMove, handleStreamComplete],
+    [handleStreamedGameInfo, handleMove, handleStreamComplete],
   )
 
   useEffect(() => {