Add resizable window support and refactor project structure

- Add LayoutManager for dynamic layout calculations based on window size
- Reorganize project into modular structure (rendering/, core/, engine/, input/)
- Rename constants to follow style guide with explicit names (e.g., CHESS_BOARD_SIZE_PIXELS)
- Window now supports free resizing with mouse, maintaining proportions
- Board stays square and properly positioned during resize
- Fonts scale proportionally with window size
- Bump version to 0.1.3

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: Bobain

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "python-chess-gui"
-version = "0.1.2"
+version = "0.1.3"
 description = "A Python chess GUI application using pygame and python-chess with Stockfish support"
 readme = "README.md"
 requires-python = ">=3.12"

src/python_chess_gui/__init__.py

@@ -1,3 +1,12 @@
 """Python Chess GUI - A chess application with pygame interface."""
 
-__version__ = "0.1.0"
+__version__ = "0.1.3"
+
+from python_chess_gui.main import main, ChessApplication
+from python_chess_gui.layout_manager import LayoutManager
+
+__all__ = [
+    "main",
+    "ChessApplication",
+    "LayoutManager",
+]

src/python_chess_gui/constants.py

@@ -1,38 +1,38 @@
 """Configuration constants for the chess GUI application."""
 
-# Window dimensions
-BOARD_SIZE = 640
-SQUARE_SIZE = BOARD_SIZE // 8
-EVAL_BAR_WIDTH = 40
-SIDEBAR_WIDTH = 160
-STATUS_BAR_HEIGHT = 50
-CONTROL_BAR_HEIGHT = 50
+# Window dimensions (in pixels)
+CHESS_BOARD_SIZE_PIXELS = 640
+BOARD_SQUARE_SIZE_PIXELS = CHESS_BOARD_SIZE_PIXELS // 8
+EVALUATION_BAR_WIDTH_PIXELS = 40
+SIDEBAR_WIDTH_PIXELS = 160
+STATUS_BAR_HEIGHT_PIXELS = 50
+CONTROL_BAR_HEIGHT_PIXELS = 50
 
-WINDOW_WIDTH = BOARD_SIZE + SIDEBAR_WIDTH
-WINDOW_HEIGHT = STATUS_BAR_HEIGHT + BOARD_SIZE + CONTROL_BAR_HEIGHT
+MAIN_WINDOW_WIDTH_PIXELS = CHESS_BOARD_SIZE_PIXELS + SIDEBAR_WIDTH_PIXELS
+MAIN_WINDOW_HEIGHT_PIXELS = STATUS_BAR_HEIGHT_PIXELS + CHESS_BOARD_SIZE_PIXELS + CONTROL_BAR_HEIGHT_PIXELS
 
-# Board position offset (from top-left of window)
-BOARD_OFFSET_X = 0
-BOARD_OFFSET_Y = STATUS_BAR_HEIGHT
+# Board position offset (from top-left of window, in pixels)
+BOARD_OFFSET_X_PIXELS = 0
+BOARD_OFFSET_Y_PIXELS = STATUS_BAR_HEIGHT_PIXELS
 
-# Colors (RGB)
-COLOR_LIGHT_SQUARE = (240, 217, 181)  # Tan
-COLOR_DARK_SQUARE = (181, 136, 99)    # Brown
-COLOR_BACKGROUND = (49, 46, 43)       # Dark gray
-COLOR_WHITE = (255, 255, 255)
-COLOR_BLACK = (0, 0, 0)
-COLOR_SELECTED = (186, 202, 68, 180)  # Yellow-green highlight
-COLOR_LEGAL_MOVE = (130, 151, 105)    # Green dot for legal moves
-COLOR_LAST_MOVE = (205, 210, 106, 128)  # Yellow highlight for last move
-COLOR_CHECK = (235, 97, 80)           # Red for check
-COLOR_HINT = (72, 150, 220, 200)      # Blue highlight for hint move
-COLOR_BUTTON = (70, 70, 70)           # Button background
-COLOR_BUTTON_SELECTED = (76, 154, 42) # Green for selected button
-COLOR_BUTTON_HOVER = (100, 100, 100)  # Button hover
-COLOR_TEXT = (255, 255, 255)          # White text
-COLOR_EVAL_WHITE = (255, 255, 255)    # White side of eval bar
-COLOR_EVAL_BLACK = (0, 0, 0)          # Black side of eval bar
-COLOR_MENU_BG = (39, 37, 34)          # Menu background
+# Colors (RGB/RGBA tuples)
+LIGHT_SQUARE_COLOR_RGB = (240, 217, 181)  # Tan
+DARK_SQUARE_COLOR_RGB = (181, 136, 99)    # Brown
+WINDOW_BACKGROUND_COLOR_RGB = (49, 46, 43)       # Dark gray
+WHITE_COLOR_RGB = (255, 255, 255)
+BLACK_COLOR_RGB = (0, 0, 0)
+SELECTED_SQUARE_HIGHLIGHT_RGBA = (186, 202, 68, 180)  # Yellow-green highlight
+LEGAL_MOVE_INDICATOR_COLOR_RGB = (130, 151, 105)    # Green dot for legal moves
+LAST_MOVE_HIGHLIGHT_RGBA = (205, 210, 106, 128)  # Yellow highlight for last move
+CHECK_HIGHLIGHT_COLOR_RGB = (235, 97, 80)           # Red for check
+HINT_MOVE_HIGHLIGHT_RGBA = (72, 150, 220, 200)      # Blue highlight for hint move
+BUTTON_BACKGROUND_COLOR_RGB = (70, 70, 70)           # Button background
+BUTTON_SELECTED_COLOR_RGB = (76, 154, 42) # Green for selected button
+BUTTON_HOVER_COLOR_RGB = (100, 100, 100)  # Button hover
+TEXT_COLOR_RGB = (255, 255, 255)          # White text
+EVAL_BAR_WHITE_COLOR_RGB = (255, 255, 255)    # White side of eval bar
+EVAL_BAR_BLACK_COLOR_RGB = (0, 0, 0)          # Black side of eval bar
+MENU_BACKGROUND_COLOR_RGB = (39, 37, 34)          # Menu background
 
 # Fonts
 FONT_NAME = None  # Use default system font
@@ -60,8 +60,8 @@
 }
 
 # Stockfish configuration
-STOCKFISH_DEPTH = 15  # Analysis depth
-STOCKFISH_MOVE_TIME = 1.0  # Seconds to think for AI moves
+STOCKFISH_ANALYSIS_DEPTH = 15  # Analysis depth
+STOCKFISH_MOVE_TIME_SECONDS = 1.0  # Seconds to think for AI moves
 
 # AI difficulty presets (Elo ratings)
 # Note: Stockfish 17+ requires minimum Elo of 1320
@@ -76,11 +76,11 @@
 DEFAULT_DIFFICULTY = "Level 3"
 
 # Evaluation bar scaling
-EVAL_MAX_CENTIPAWNS = 1000  # +/- 10 pawns = max bar
-EVAL_MATE_SCORE = 10000  # Score to use for mate positions
+EVALUATION_MAX_CENTIPAWNS = 1000  # +/- 10 pawns = max bar
+EVALUATION_MATE_SCORE = 10000  # Score to use for mate positions
 
 # Frame rate
-FPS = 60
+FRAMES_PER_SECOND = 60
 
 # Coordinate labels
 FILE_LABELS = ['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h']

src/python_chess_gui/core/__init__.py

@@ -0,0 +1,15 @@
+"""Core module for chess game logic and utilities."""
+
+from python_chess_gui.core.game_state_manager import GameStateManager
+from python_chess_gui.core.coordinate_converter import (
+    convert_screen_position_to_board_square,
+    convert_board_square_to_screen_position,
+    get_square_center,
+)
+
+__all__ = [
+    "GameStateManager",
+    "convert_screen_position_to_board_square",
+    "convert_board_square_to_screen_position",
+    "get_square_center",
+]

…python_chess_gui/coordinate_converter.py …n_chess_gui/core/coordinate_converter.pysrc/python_chess_gui/coordinate_converter.py renamed to src/python_chess_gui/core/coordinate_converter.py src/python_chess_gui/coordinate_converter.py renamed to src/python_chess_gui/core/coordinate_converter.py

@@ -2,62 +2,53 @@
 
 import chess
 
-from python_chess_gui.constants import (
-    BOARD_OFFSET_X,
-    BOARD_OFFSET_Y,
-    SQUARE_SIZE,
-)
+from python_chess_gui.layout_manager import LayoutManager
 
 
 def convert_screen_position_to_board_square(
-    screen_x: int, screen_y: int, player_is_white: bool
+    screen_x: int, screen_y: int, player_is_white: bool, layout: LayoutManager
 ) -> chess.Square | None:
     """Convert screen coordinates to a chess square.
 
     Args:
         screen_x: X coordinate on screen
         screen_y: Y coordinate on screen
         player_is_white: True if player is playing white (white at bottom)
+        layout: Layout manager for dimension calculations
 
     Returns:
         Chess square index (0-63) or None if outside board
     """
-    # Adjust for board offset
-    board_x = screen_x - BOARD_OFFSET_X
-    board_y = screen_y - BOARD_OFFSET_Y
+    board_x = screen_x - layout.board_offset_x
+    board_y = screen_y - layout.board_offset_y
 
-    # Check if within board bounds
-    if board_x < 0 or board_x >= SQUARE_SIZE * 8:
+    if board_x < 0 or board_x >= layout.square_size * 8:
         return None
-    if board_y < 0 or board_y >= SQUARE_SIZE * 8:
+    if board_y < 0 or board_y >= layout.square_size * 8:
         return None
 
-    # Convert to file and rank indices (0-7)
-    file_idx = board_x // SQUARE_SIZE
-    rank_idx = board_y // SQUARE_SIZE
+    file_idx = board_x // layout.square_size
+    rank_idx = board_y // layout.square_size
 
-    # If player is white, board is oriented with rank 8 at top (row 0)
-    # If player is black, board is flipped
     if player_is_white:
-        # White at bottom: rank 8 at top (rank_idx 0 = rank 7)
         file = file_idx
         rank = 7 - rank_idx
     else:
-        # Black at bottom: rank 1 at top (rank_idx 0 = rank 0)
         file = 7 - file_idx
         rank = rank_idx
 
     return chess.square(file, rank)
 
 
 def convert_board_square_to_screen_position(
-    square: chess.Square, player_is_white: bool
+    square: chess.Square, player_is_white: bool, layout: LayoutManager
 ) -> tuple[int, int]:
     """Convert a chess square to screen coordinates (top-left of square).
 
     Args:
         square: Chess square index (0-63)
         player_is_white: True if player is playing white (white at bottom)
+        layout: Layout manager for dimension calculations
 
     Returns:
         Tuple of (x, y) screen coordinates for top-left corner of square
@@ -66,29 +57,30 @@ def convert_board_square_to_screen_position(
     rank = chess.square_rank(square)
 
     if player_is_white:
-        # White at bottom: file a on left, rank 8 at top
         screen_col = file
         screen_row = 7 - rank
     else:
-        # Black at bottom: file h on left, rank 1 at top
         screen_col = 7 - file
         screen_row = rank
 
-    x = BOARD_OFFSET_X + screen_col * SQUARE_SIZE
-    y = BOARD_OFFSET_Y + screen_row * SQUARE_SIZE
+    x = layout.board_offset_x + screen_col * layout.square_size
+    y = layout.board_offset_y + screen_row * layout.square_size
 
     return (x, y)
 
 
-def get_square_center(square: chess.Square, player_is_white: bool) -> tuple[int, int]:
+def get_square_center(
+    square: chess.Square, player_is_white: bool, layout: LayoutManager
+) -> tuple[int, int]:
     """Get the center coordinates of a chess square on screen.
 
     Args:
         square: Chess square index (0-63)
         player_is_white: True if player is playing white (white at bottom)
+        layout: Layout manager for dimension calculations
 
     Returns:
         Tuple of (x, y) screen coordinates for center of square
     """
-    x, y = convert_board_square_to_screen_position(square, player_is_white)
-    return (x + SQUARE_SIZE // 2, y + SQUARE_SIZE // 2)
+    x, y = convert_board_square_to_screen_position(square, player_is_white, layout)
+    return (x + layout.square_size // 2, y + layout.square_size // 2)

…c/python_chess_gui/game_state_manager.py …hon_chess_gui/core/game_state_manager.pysrc/python_chess_gui/game_state_manager.py renamed to src/python_chess_gui/core/game_state_manager.py src/python_chess_gui/game_state_manager.py renamed to src/python_chess_gui/core/game_state_manager.py

@@ -31,7 +31,6 @@ def select_square(self, square: chess.Square) -> list[chess.Move]:
         """
         piece = self.board.piece_at(square)
 
-        # If there's a piece of the current turn's color, select it
         if piece is not None and piece.color == self.board.turn:
             self.selected_square = square
             return self.get_legal_moves_from_square(square)
@@ -59,16 +58,14 @@ def try_move(self, from_square: chess.Square, to_square: chess.Square) -> chess.
         Returns:
             The move if successful, None if illegal
         """
-        # Check for promotion
         piece = self.board.piece_at(from_square)
         promotion = None
 
         if piece is not None and piece.piece_type == chess.PAWN:
-            # Check if pawn is reaching the back rank
             if piece.color == chess.WHITE and chess.square_rank(to_square) == 7:
-                promotion = chess.QUEEN  # Auto-promote to queen
+                promotion = chess.QUEEN
             elif piece.color == chess.BLACK and chess.square_rank(to_square) == 0:
-                promotion = chess.QUEEN  # Auto-promote to queen
+                promotion = chess.QUEEN
 
         move = chess.Move(from_square, to_square, promotion=promotion)
 
@@ -79,7 +76,6 @@ def try_move(self, from_square: chess.Square, to_square: chess.Square) -> chess.
             self.selected_square = None
             return move
 
-        # Try without promotion (for non-pawn moves that might match)
         move_no_promo = chess.Move(from_square, to_square)
         if move_no_promo in self.board.legal_moves:
             self.board.push(move_no_promo)
@@ -129,10 +125,8 @@ def undo_move_pair(self) -> bool:
             True if moves were undone, False if not enough moves
         """
         if len(self.move_history) < 2:
-            # If only one move, undo just that one
             return self.undo_move()
 
-        # Undo two moves
         self.undo_move()
         self.undo_move()
         return True

src/python_chess_gui/engine/__init__.py

@@ -0,0 +1,19 @@
+"""Engine module for Stockfish integration."""
+
+from python_chess_gui.engine.stockfish_engine_controller import StockfishEngineController
+from python_chess_gui.engine.config_manager import (
+    find_stockfish,
+    get_stockfish_path,
+    set_stockfish_path,
+    load_config,
+    save_config,
+)
+
+__all__ = [
+    "StockfishEngineController",
+    "find_stockfish",
+    "get_stockfish_path",
+    "set_stockfish_path",
+    "load_config",
+    "save_config",
+]

src/python_chess_gui/config_manager.py …ython_chess_gui/engine/config_manager.pysrc/python_chess_gui/config_manager.py renamed to src/python_chess_gui/engine/config_manager.py src/python_chess_gui/config_manager.py renamed to src/python_chess_gui/engine/config_manager.py

@@ -95,25 +95,21 @@ def find_stockfish() -> str | None:
     Returns:
         Path to Stockfish executable, or None if not found
     """
-    # First, check saved config
     config_path = get_stockfish_path()
     if config_path:
         return config_path
 
-    # Second, check environment variable
     env_path = os.environ.get("STOCKFISH_PATH")
     if env_path and Path(env_path).exists():
         return env_path
 
-    # Third, try to find in PATH
     stockfish_in_path = shutil.which("stockfish")
     if stockfish_in_path:
         return stockfish_in_path
 
-    # Fourth, check platform-specific common locations
     system = platform.system()
 
-    if system == "Darwin":  # macOS
+    if system == "Darwin":
         common_paths = [
             "/opt/homebrew/bin/stockfish",
             "/usr/local/bin/stockfish",