improve log system

.github/instructions/BUILD_PLAN.md

@@ -61,7 +61,7 @@
 
 ### משימה 3.3: End-to-End Testing
 **סטטוס:** 🟢 בעבודה פעילה!
-**עדכון אחרון:** 15 דצמבר 2025
+**עדכון אחרון:** 20 דצמבר 2025
 
 - [x] הרצת משחק מלא עם HumanUser
 - [x] בדיקת כל סוגי הפעולות (בנייה, גלגול קוביות)
@@ -81,15 +81,23 @@
   - מימוש מלא של `_execute_build_city` ב-`game_manager.py`
   - המרת הודעות שגיאה למספרי נקודות (point 44) במקום קואורדינטות
   - הצגת הודעות שגיאה בקונסול הראשי של המשתמש
+- ✅ **מערכת לוגים מובנית מלאה (20 דצמבר 2025)**:
+  - יצירת `log_events.py` עם EventType enum, LogEntry dataclass
+  - עדכון `web_visualization.py` לתמוך בלוגים מפורטים עם כל הנתונים
+  - עדכון `console_visualization.py` לתמוך בלוגים מפורטים
+  - עדכון `game_manager.py` להוסיף פרטים מלאים לכל action (point numbers, dice values, costs)
+  - תיקון הצגת קוביות - עכשיו מוצגים הערכים האמיתיים במקום [0,0]
+  - כל אירוע עכשיו כולל: timestamp, turn number, player info, detailed data, status, errors
 
 **מה שעובד כרגע:**
 - ✨ משחק מלא עם 3 שחקנים
 - ✨ בנייה: יישובים, כבישים, **ערים** ✅ (תוקן 15 דצמבר!)
-- ✨ גלגול קוביות וחלוקת משאבים
+- ✨ גלגול קוביות וחלוקת משאבים ✅ (לוגים מפורטים 20 דצמבר!)
 - ✨ חוק ה-7: השלכת קלפים, העברת רובר, גניבת קלף
 - ✨ מסחר עם הבנק
 - ✨ קניית קלפי פיתוח
-- ✨ ויזואליזציה בזמן אמת (Console + Web)
+- ✨ ויזואליזציה בזמן אמת (Console + Web) עם לוגים מובנים
+- ✨ מערכת לוגים parsable לניתוח מידע
 
 ---
 
@@ -136,23 +144,37 @@
 **מטרה:** טיפול במסחר ואינטראקציות מורכבות
 
 ### משימה 5.1: Trading System
-**סטטוס:** ⭕ לא התחיל  
+**סטטוס:** 🟢 הושלם חלקית!
+**תאריך עדכון:** 20 דצמבר 2025
 **זמן משוער:** 4-5 שעות
 
-- [ ] הצעות מסחר בין שחקנים
-- [ ] מנגנון אישור/דחייה
+- [x] הצעות מסחר בין שחקנים
+- [x] מנגנון אישור/דחייה
 - [ ] counter-offers
 - [ ] timeout למסחר
-- [ ] mסחר עם הבנק (4:1, harbors)
+- [x] מסחר עם הבנק (4:1, harbors)
+
+**מימוש:**
+- `_execute_trade_propose()` - הצעת מסחר בין שחקנים
+- `_execute_trade_bank()` - מסחר עם הבנק
+- תמיכה בפורמט: `trade player <name> offer <cards> for <cards>`
+- מנגנון אישור/דחייה: `y`/`n` או `yes`/`no`
 
 ### משימה 5.2: Robber Interactions
-**סטטוס:** ⭕ לא התחיל  
+**סטטוס:** ✅ הושלם!
+**תאריך השלמה:** נובמבר 2025
 **זמן משוער:** 2-3 שעות
 
-- [ ] בחירת מיקום רובר
-- [ ] בחירת שחקן לגניבה
-- [ ] Knight card interactions
-- [ ] ויזואליזציה של רובר
+- [x] בחירת מיקום רובר
+- [x] בחירת שחקן לגניבה
+- [x] Knight card interactions
+- [x] ויזואליזציה של רובר
+
+**מימוש:**
+- `_handle_robber_move()` - העברת רובר
+- `_handle_steal_card()` - גניבת קלף
+- תמיכה בפורמט: `robber <tile_id>`
+- אינטגרציה עם Knight card ו-Rule of 7
 
 ### משימה 5.3: Development Cards & Special Actions
 **סטטוס:** 🟢 בעבודה פעילה!  

game_viz.log

@@ -22,7 +22,7 @@ Current Player: ► a
   Resources: None
   Buildings: Settlements: 0, Cities: 0, Roads: 0
 
-s
+c
   Victory Points: 0
   Resources: None
   Buildings: Settlements: 0, Cities: 0, Roads: 0
@@ -53,7 +53,7 @@ Current Player: ► a
   Resources: None
   Buildings: Settlements: 0, Cities: 0, Roads: 0
 
-s
+c
   Victory Points: 0
   Resources: None
   Buildings: Settlements: 0, Cities: 0, Roads: 0
@@ -86,7 +86,7 @@ Current Player: ► b
   Resources: None
   Buildings: Settlements: 1, Cities: 0, Roads: 0
 
-s
+c
   Victory Points: 0
   Resources: None
   Buildings: Settlements: 0, Cities: 0, Roads: 0
@@ -117,7 +117,7 @@ Current Player: ► b
   Resources: None
   Buildings: Settlements: 1, Cities: 0, Roads: 1
 
-s
+c
   Victory Points: 0
   Resources: None
   Buildings: Settlements: 0, Cities: 0, Roads: 0
@@ -127,15 +127,15 @@ Current Player: ► b
 Board Tiles: 19 tiles configured
 
 
->>> Turn 2: s's turn
-✓ s built a settlement
+>>> Turn 2: c's turn
+✓ c built a settlement
 
 ==================================================
                     GAME STATE                    
 ==================================================
 
 Turn: 2
-Current Player: ► s
+Current Player: ► c
 
 PLAYERS
 -------
@@ -150,7 +150,7 @@ Current Player: ► s
   Resources: None
   Buildings: Settlements: 1, Cities: 0, Roads: 1
 
-► s
+► c
   Victory Points: 1
   Resources: None
   Buildings: Settlements: 1, Cities: 0, Roads: 0
@@ -159,14 +159,14 @@ Current Player: ► s
 -----
 Board Tiles: 19 tiles configured
 
-✓ s built a road
+✓ c built a road
 
 ==================================================
                     GAME STATE                    
 ==================================================
 
 Turn: 2
-Current Player: ► s
+Current Player: ► c
 
 PLAYERS
 -------
@@ -181,7 +181,7 @@ Current Player: ► s
   Resources: None
   Buildings: Settlements: 1, Cities: 0, Roads: 1
 
-► s
+► c
   Victory Points: 1
   Resources: None
   Buildings: Settlements: 1, Cities: 0, Roads: 1
@@ -191,18 +191,18 @@ Current Player: ► s
 Board Tiles: 19 tiles configured
 
 
->>> Turn 3: s's turn
+>>> Turn 3: c's turn
 
 📦 Resources distributed:
-  s: Ore, Wheat, Wheat
-✓ s built a settlement
+  c: Ore, Wheat, Wheat
+✓ c built a settlement
 
 ==================================================
                     GAME STATE                    
 ==================================================
 
 Turn: 3
-Current Player: ► s
+Current Player: ► c
 
 PLAYERS
 -------
@@ -217,7 +217,7 @@ Current Player: ► s
   Resources: None
   Buildings: Settlements: 1, Cities: 0, Roads: 1
 
-► s
+► c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 1
@@ -226,14 +226,14 @@ Current Player: ► s
 -----
 Board Tiles: 19 tiles configured
 
-✓ s built a road
+✓ c built a road
 
 ==================================================
                     GAME STATE                    
 ==================================================
 
 Turn: 3
-Current Player: ► s
+Current Player: ► c
 
 PLAYERS
 -------
@@ -248,7 +248,7 @@ Current Player: ► s
   Resources: None
   Buildings: Settlements: 1, Cities: 0, Roads: 1
 
-► s
+► c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
@@ -284,7 +284,7 @@ Current Player: ► b
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 1
 
-s
+c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
@@ -315,7 +315,7 @@ Current Player: ► b
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
 
-s
+c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
@@ -351,7 +351,7 @@ Current Player: ► a
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
 
-s
+c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
@@ -382,7 +382,7 @@ Current Player: ► a
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
 
-s
+c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
@@ -393,12 +393,6 @@ Board Tiles: 19 tiles configured
 
 
 >>> Turn 6: a's turn
-
-🎲 a rolled: 5 + 3 = 8
-
-📦 Resources distributed:
-  Player 1: Sheep
-  Player 2: Wheat
 ✓ a rolled the dice
 
 ==================================================
@@ -421,38 +415,7 @@ Current Player: ► a
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
 
-s
-  Victory Points: 2
-  Resources: None
-  Buildings: Settlements: 2, Cities: 0, Roads: 2
-
-BOARD
------
-Board Tiles: 19 tiles configured
-
-✓ a ended their turn
-
-==================================================
-                    GAME STATE                    
-==================================================
-
-Turn: 6
-Current Player: ► a
-
-PLAYERS
--------
-
-► a
-  Victory Points: 2
-  Resources: None
-  Buildings: Settlements: 2, Cities: 0, Roads: 2
-
-b
-  Victory Points: 2
-  Resources: None
-  Buildings: Settlements: 2, Cities: 0, Roads: 2
-
-s
+c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
@@ -463,8 +426,6 @@ Board Tiles: 19 tiles configured
 
 
 >>> Turn 7: b's turn
-
-🎲 b rolled: 6 + 1 = 7
 ✓ b rolled the dice
 
 ==================================================
@@ -487,7 +448,7 @@ Current Player: ► b
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
 
-s
+c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
@@ -518,7 +479,7 @@ Current Player: ► b
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
 
-s
+c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
@@ -527,84 +488,16 @@ Current Player: ► b
 -----
 Board Tiles: 19 tiles configured
 
-✓ b ended their turn
-
-==================================================
-                    GAME STATE                    
-==================================================
-
-Turn: 7
-Current Player: ► b
-
-PLAYERS
--------
-
-a
-  Victory Points: 2
-  Resources: None
-  Buildings: Settlements: 2, Cities: 0, Roads: 2
-
-► b
-  Victory Points: 2
-  Resources: None
-  Buildings: Settlements: 2, Cities: 0, Roads: 2
-
-s
-  Victory Points: 2
-  Resources: None
-  Buildings: Settlements: 2, Cities: 0, Roads: 2
-
-BOARD
------
-Board Tiles: 19 tiles configured
-
-
->>> Turn 8: s's turn
-
-🎲 s rolled: 3 + 5 = 8
-
-📦 Resources distributed:
-  Player 1: Sheep
-  Player 2: Wheat
-✓ s rolled the dice
-
-==================================================
-                    GAME STATE                    
-==================================================
-
-Turn: 8
-Current Player: ► s
-
-PLAYERS
--------
-
-a
-  Victory Points: 2
-  Resources: None
-  Buildings: Settlements: 2, Cities: 0, Roads: 2
-
-b
-  Victory Points: 2
-  Resources: None
-  Buildings: Settlements: 2, Cities: 0, Roads: 2
-
-► s
-  Victory Points: 2
-  Resources: None
-  Buildings: Settlements: 2, Cities: 0, Roads: 2
-
-BOARD
------
-Board Tiles: 19 tiles configured
 
-✓ s ended their turn
+>>> Turn 8: c's turn
+✓ c rolled the dice
 
 ==================================================
                     GAME STATE                    
 ==================================================
 
 Turn: 8
-Current Player: ► s
+Current Player: ► c
 
 PLAYERS
 -------
@@ -619,7 +512,7 @@ Current Player: ► s
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
 
-► s
+► c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
@@ -630,8 +523,6 @@ Board Tiles: 19 tiles configured
 
 
 >>> Turn 9: a's turn
-
-🎲 a rolled: 2 + 5 = 7
 ✓ a rolled the dice
 
 ==================================================
@@ -654,7 +545,7 @@ Current Player: ► a
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
 
-s
+c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
@@ -685,38 +576,7 @@ Current Player: ► a
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
 
-s
-  Victory Points: 2
-  Resources: None
-  Buildings: Settlements: 2, Cities: 0, Roads: 2
-
-BOARD
------
-Board Tiles: 19 tiles configured
-
-✓ a ended their turn
-
-==================================================
-                    GAME STATE                    
-==================================================
-
-Turn: 9
-Current Player: ► a
-
-PLAYERS
--------
-
-► a
-  Victory Points: 2
-  Resources: None
-  Buildings: Settlements: 2, Cities: 0, Roads: 2
-
-b
-  Victory Points: 2
-  Resources: None
-  Buildings: Settlements: 2, Cities: 0, Roads: 2
-
-s
+c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
@@ -727,11 +587,6 @@ Board Tiles: 19 tiles configured
 
 
 >>> Turn 10: b's turn
-
-🎲 b rolled: 4 + 1 = 5
-
-📦 Resources distributed:
-  Player 1: Ore
 ✓ b rolled the dice
 
 ==================================================
@@ -754,7 +609,7 @@ Current Player: ► b
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
 
-s
+c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
@@ -786,7 +641,7 @@ Current Player: ► b
   Dev Cards: 1
   Buildings: Settlements: 2, Cities: 0, Roads: 2
 
-s
+c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
@@ -817,7 +672,7 @@ Current Player: ► b
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 4
 
-s
+c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
@@ -826,52 +681,16 @@ Current Player: ► b
 -----
 Board Tiles: 19 tiles configured
 
-✓ b ended their turn
 
-==================================================
-                    GAME STATE                    
-==================================================
-
-Turn: 10
-Current Player: ► b
-
-PLAYERS
--------
-
-a
-  Victory Points: 2
-  Resources: None
-  Buildings: Settlements: 2, Cities: 0, Roads: 2
-
-► b
-  Victory Points: 2
-  Resources: None
-  Buildings: Settlements: 2, Cities: 0, Roads: 4
-
-s
-  Victory Points: 2
-  Resources: None
-  Buildings: Settlements: 2, Cities: 0, Roads: 2
-
-BOARD
------
-Board Tiles: 19 tiles configured
-
-
->>> Turn 11: s's turn
-
-🎲 s rolled: 5 + 4 = 9
-
-📦 Resources distributed:
-  Player 3: Wheat, Wheat
-✓ s rolled the dice
+>>> Turn 11: c's turn
+✓ c rolled the dice
 
 ==================================================
                     GAME STATE                    
 ==================================================
 
 Turn: 11
-Current Player: ► s
+Current Player: ► c
 
 PLAYERS
 -------
@@ -886,7 +705,7 @@ Current Player: ► s
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 4
 
-► s
+► c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
@@ -895,14 +714,14 @@ Current Player: ► s
 -----
 Board Tiles: 19 tiles configured
 
-✓ s proposed a trade
+✓ c proposed a trade
 
 ==================================================
                     GAME STATE                    
 ==================================================
 
 Turn: 11
-Current Player: ► s
+Current Player: ► c
 
 PLAYERS
 -------
@@ -917,7 +736,7 @@ Current Player: ► s
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 4
 
-► s
+► c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
@@ -926,7 +745,7 @@ Current Player: ► s
 -----
 Board Tiles: 19 tiles configured
 
-✗ s built a city
+✗ c built a city
   Error: You don't own the settlement at point 40
 
 ==================================================
@@ -934,7 +753,7 @@ Board Tiles: 19 tiles configured
 ==================================================
 
 Turn: 11
-Current Player: ► s
+Current Player: ► c
 
 PLAYERS
 -------
@@ -949,54 +768,23 @@ Current Player: ► s
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 4
 
-► s
-  Victory Points: 2
-  Resources: None
-  Buildings: Settlements: 2, Cities: 0, Roads: 2
-
-BOARD
------
-Board Tiles: 19 tiles configured
-
-✓ s built a city
-
-==================================================
-                    GAME STATE                    
-==================================================
-
-Turn: 11
-Current Player: ► s
-
-PLAYERS
--------
-
-a
+► c
   Victory Points: 2
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 2
 
-b
-  Victory Points: 2
-  Resources: None
-  Buildings: Settlements: 2, Cities: 0, Roads: 4
-
-► s
-  Victory Points: 3
-  Resources: None
-  Buildings: Settlements: 1, Cities: 1, Roads: 2
-
 BOARD
 -----
 Board Tiles: 19 tiles configured
 
-✓ s ended their turn
+✓ c built a city
 
 ==================================================
                     GAME STATE                    
 ==================================================
 
 Turn: 11
-Current Player: ► s
+Current Player: ► c
 
 PLAYERS
 -------
@@ -1011,7 +799,7 @@ Current Player: ► s
   Resources: None
   Buildings: Settlements: 2, Cities: 0, Roads: 4
 
-► s
+► c
   Victory Points: 3
   Resources: None
   Buildings: Settlements: 1, Cities: 1, Roads: 2

pycatan/__init__.py

@@ -15,6 +15,7 @@ from pycatan.actions import (
     GamePhase, TurnPhase, create_build_settlement_action, create_build_road_action,
     create_trade_action
 )
+from pycatan.log_events import EventType, LogEntry, create_log_entry
 from pycatan.user import User, UserInputError, validate_user_list, create_test_user
 from pycatan.human_user import HumanUser
 from pycatan.game_manager import GameManager

pycatan/console_visualization.py

@@ -10,6 +10,7 @@ from typing import Dict, Any, List, Optional
 from .visualization import Visualization
 from .actions import Action, ActionResult, ActionType, GameState
 from .card import ResCard, DevCard
+from .log_events import EventType, LogEntry, create_log_entry
 
 
 class ConsoleVisualization(Visualization):
@@ -642,4 +643,43 @@ class ConsoleVisualization(Visualization):
         print("  'board'     - Show detailed board layout") 
         print("  'robber'    - Show robber information")
         print("  'help'      - Show all available commands")
-        print()
+        print()
+    
+    def log_event(self, log_entry: LogEntry):
+        """
+        Log a structured event directly to console.
+        This allows GameManager to send detailed log entries.
+        """
+        if not self.enabled:
+            return
+        
+        # Display human-readable version
+        message = log_entry.to_human_string()
+        
+        # Color based on status
+        if log_entry.status == "SUCCESS":
+            color = 'green'
+            symbol = '✓'
+        elif log_entry.status == "FAIL":
+            color = 'red'
+            symbol = '✗'
+        else:
+            color = 'yellow'
+            symbol = '⏳'
+        
+        self._print(f"{self.colors[color]}{symbol}{self.colors['reset']} {message}")
+        
+        # If there's an error, display it
+        if log_entry.error:
+            self._print(f"  {self.colors['red']}└─ Error: {log_entry.error}{self.colors['reset']}")
+    
+    def log_structured(self, log_entry: LogEntry):
+        """
+        Log a structured event in machine-readable format.
+        Useful for debugging and data analysis.
+        """
+        if not self.enabled:
+            return
+        
+        # Display structured log string
+        self._print(log_entry.to_log_string())

pycatan/game_manager.py

@@ -1153,6 +1153,109 @@ class GameManager:
                 "ACTION_PROCESSING_ERROR"
             )
     
+    def _enrich_action_parameters(self, action: Action, result: ActionResult) -> None:
+        """
+        Enrich action parameters with detailed information for visualization and logging.
+        Adds all relevant details like point numbers, costs, card types, etc.
+        """
+        from .card import ResCard, DevCard
+        from .board_definition import board_definition
+        
+        # Ensure parameters dict exists
+        if not hasattr(action, 'parameters') or action.parameters is None:
+            action.parameters = {}
+        
+        params = action.parameters
+        
+        # Add turn number
+        params['turn_number'] = self._current_game_state.turn_number
+        
+        # Building actions - add point number and costs
+        if action.action_type in [ActionType.BUILD_SETTLEMENT, ActionType.PLACE_STARTING_SETTLEMENT]:
+            if 'point_coords' in params:
+                coords = params['point_coords']
+                try:
+                    point_id = board_definition.game_coords_to_point_id(coords[0], coords[1])
+                    params['point'] = point_id if point_id else f"[{coords[0]},{coords[1]}]"
+                except:
+                    params['point'] = f"[{coords[0]},{coords[1]}]"
+            
+            if result.success and action.action_type == ActionType.BUILD_SETTLEMENT:
+                params['cost'] = ['WOOD', 'BRICK', 'WHEAT', 'SHEEP']
+        
+        elif action.action_type == ActionType.BUILD_CITY:
+            if 'point_coords' in params:
+                coords = params['point_coords']
+                try:
+                    point_id = board_definition.game_coords_to_point_id(coords[0], coords[1])
+                    params['point'] = point_id if point_id else f"[{coords[0]},{coords[1]}]"
+                except:
+                    params['point'] = f"[{coords[0]},{coords[1]}]"
+            
+            if result.success:
+                params['cost'] = ['ORE', 'ORE', 'ORE', 'WHEAT', 'WHEAT']
+        
+        elif action.action_type in [ActionType.BUILD_ROAD, ActionType.PLACE_STARTING_ROAD]:
+            if 'start_coords' in params and 'end_coords' in params:
+                start_coords = params['start_coords']
+                end_coords = params['end_coords']
+                try:
+                    start_id = board_definition.game_coords_to_point_id(start_coords[0], start_coords[1])
+                    end_id = board_definition.game_coords_to_point_id(end_coords[0], end_coords[1])
+                    params['points'] = [start_id, end_id] if (start_id and end_id) else [start_coords, end_coords]
+                except:
+                    params['points'] = [start_coords, end_coords]
+            
+            if result.success and action.action_type == ActionType.BUILD_ROAD:
+                params['cost'] = ['WOOD', 'BRICK']
+        
+        # Dev card actions
+        elif action.action_type == ActionType.BUY_DEV_CARD:
+            if result.success:
+                # Try to get the actual card that was drawn
+                player = self.game.players[action.player_id]
+                if player.dev_cards:
+                    last_card = player.dev_cards[-1]
+                    params['card'] = last_card.name if hasattr(last_card, 'name') else str(last_card)
+                params['cost'] = ['ORE', 'SHEEP', 'WHEAT']
+        
+        elif action.action_type == ActionType.USE_DEV_CARD:
+            if 'card_type' in params:
+                card_type = params['card_type']
+                params['card'] = card_type.name if hasattr(card_type, 'name') else str(card_type)
+        
+        # Dice roll
+        elif action.action_type == ActionType.ROLL_DICE:
+            if 'dice' in params:
+                params['total'] = sum(params['dice'])
+        
+        # Robber actions
+        elif action.action_type == ActionType.ROBBER_MOVE:
+            if 'tile_coords' in params:
+                params['tile'] = str(params['tile_coords'])
+        
+        # Trading
+        elif action.action_type == ActionType.TRADE_BANK:
+            # params should already have 'give' and 'receive'
+            pass
+        
+        elif action.action_type == ActionType.TRADE_PROPOSE:
+            if 'target_player' in params:
+                target_id = params['target_player']
+                target_name = self.users[target_id].name if hasattr(self.users[target_id], 'name') else f"Player {target_id}"
+                params['to_player'] = target_name
+        
+        # End turn - add player state
+        elif action.action_type == ActionType.END_TURN:
+            player = self.game.players[action.player_id]
+            params['player_state'] = {
+                'victory_points': player.victory_points,
+                'card_count': len(player.cards),
+                'roads': len(player.roads),
+                'settlements': len(player.settlements),
+                'cities': len(player.cities)
+            }
+    
     def _update_all_systems(self, action: Action, result: ActionResult) -> None:
         """
         Updates all systems after an action has been executed.
@@ -1199,6 +1302,9 @@ class GameManager:
                     player_name = self.users[action.player_id].name if hasattr(self.users[action.player_id], 'name') else f"Player {action.player_id}"
                     action.parameters['player_name'] = player_name
                 
+                # Enrich action parameters with detailed information for logging
+                self._enrich_action_parameters(action, result)
+                
                 # Display the action result (success or failure)
                 self.visualization_manager.display_action(action, result)
                 
@@ -1429,28 +1535,26 @@ class GameManager:
         die2 = random.randint(1, 6)
         total = die1 + die2
         
+        # Update action parameters for logging/visualization
+        if not hasattr(action, 'parameters') or action.parameters is None:
+            action.parameters = {}
+        action.parameters['dice'] = [die1, die2]
+        action.parameters['total'] = total
+        
         # Update state
         self._current_game_state.dice_rolled = (die1, die2)
         
-        # Notify visualization about dice roll
-        if self.visualization_manager:
-            player_name = self.users[action.player_id].name if hasattr(self.users[action.player_id], 'name') else f"Player {action.player_id + 1}"
-            self.visualization_manager.display_dice_roll(player_name, [die1, die2], total)
-        
         # Distribute resources or handle robber
         if total != 7:
             distribution = self.game.add_yield_for_roll(total)
             
-            # Notify visualization about resources (even if empty)
-            if self.visualization_manager:
-                if distribution:
-                    self.visualization_manager.display_resource_distribution(distribution)
-                    message = f"Rolled {total} ({die1}+{die2}). Resources distributed."
-                else:
-                    # No resources were distributed (no settlements on this number)
-                    message = f"Rolled {total} ({die1}+{die2}). No settlements on this number - no resources distributed."
-            else:
+            # Add distribution to action parameters for logging
+            action.parameters['distribution'] = distribution
+            
+            if distribution:
                 message = f"Rolled {total} ({die1}+{die2}). Resources distributed."
+            else:
+                message = f"Rolled {total} ({die1}+{die2}). No settlements on this number."
         else:
             # Rolled 7! Handle robber sequence
             message = f"Rolled 7 ({die1}+{die2})! 🏴‍☠️ Robber activated!"

pycatan/log_events.py

@@ -0,0 +1,228 @@
+"""
+Log Events - Structured logging for game actions
+Provides EventType enum and LogEntry dataclass for detailed game logging
+"""
+
+from enum import Enum
+from dataclasses import dataclass, field
+from datetime import datetime
+from typing import Dict, Any, Optional, List
+
+
+class EventType(Enum):
+    """All possible game events for structured logging"""
+    # Game flow
+    GAME_START = "GAME_START"
+    GAME_END = "GAME_END"
+    TURN_START = "TURN_START"
+    TURN_END = "TURN_END"
+    PHASE_CHANGE = "PHASE_CHANGE"
+    
+    # Dice and resources
+    DICE_ROLL = "DICE_ROLL"
+    RESOURCE_DIST = "RESOURCE_DIST"
+    RESOURCE_LOSS = "RESOURCE_LOSS"
+    
+    # Building actions
+    BUILD_SETTLEMENT = "BUILD_SETTLEMENT"
+    BUILD_CITY = "BUILD_CITY"
+    BUILD_ROAD = "BUILD_ROAD"
+    
+    # Development cards
+    BUY_DEV_CARD = "BUY_DEV_CARD"
+    USE_DEV_CARD = "USE_DEV_CARD"
+    
+    # Trading
+    TRADE_PROPOSE = "TRADE_PROPOSE"
+    TRADE_RESPONSE = "TRADE_RESPONSE"
+    TRADE_EXECUTE = "TRADE_EXECUTE"
+    TRADE_BANK = "TRADE_BANK"
+    
+    # Robber
+    ROBBER_MOVE = "ROBBER_MOVE"
+    ROBBER_STEAL = "ROBBER_STEAL"
+    DISCARD_CARDS = "DISCARD_CARDS"
+    
+    # Special
+    LONGEST_ROAD = "LONGEST_ROAD"
+    LARGEST_ARMY = "LARGEST_ARMY"
+    VICTORY = "VICTORY"
+    
+    # Errors
+    ACTION_FAILED = "ACTION_FAILED"
+
+
+@dataclass
+class LogEntry:
+    """Structured log entry for a game event"""
+    timestamp: datetime
+    event_type: EventType
+    turn: int
+    player_id: Optional[int] = None
+    player_name: Optional[str] = None
+    data: Dict[str, Any] = field(default_factory=dict)
+    status: str = "SUCCESS"  # SUCCESS, FAIL, WAITING, PENDING
+    error: Optional[str] = None
+    
+    def to_dict(self) -> Dict[str, Any]:
+        """Convert to dictionary for JSON serialization"""
+        return {
+            'timestamp': self.timestamp.isoformat(),
+            'event_type': self.event_type.value,
+            'turn': self.turn,
+            'player_id': self.player_id,
+            'player_name': self.player_name,
+            'data': self.data,
+            'status': self.status,
+            'error': self.error
+        }
+    
+    def to_log_string(self) -> str:
+        """
+        Convert to structured log string format:
+        [timestamp] EVENT_TYPE | key=value | key=value | ...
+        """
+        parts = [
+            f"[{self.timestamp.strftime('%Y-%m-%d %H:%M:%S.%f')[:-3]}]",
+            self.event_type.value
+        ]
+        
+        # Add turn
+        parts.append(f"turn={self.turn}")
+        
+        # Add player info
+        if self.player_id is not None and self.player_name:
+            parts.append(f"player={self.player_id}({self.player_name})")
+        elif self.player_id is not None:
+            parts.append(f"player={self.player_id}")
+        
+        # Add all data fields
+        for key, value in self.data.items():
+            if isinstance(value, list):
+                value_str = f"[{','.join(str(v) for v in value)}]"
+            elif isinstance(value, dict):
+                items = [f"{k}:{v}" for k, v in value.items()]
+                value_str = f"{{{','.join(items)}}}"
+            else:
+                value_str = str(value)
+            parts.append(f"{key}={value_str}")
+        
+        # Add status if not SUCCESS
+        if self.status != "SUCCESS":
+            parts.append(f"status={self.status}")
+        
+        # Add error if present
+        if self.error:
+            parts.append(f"error={self.error}")
+        
+        return " | ".join(parts)
+    
+    def to_human_string(self) -> str:
+        """Convert to human-readable string for display"""
+        # Player identifier
+        if self.player_name:
+            player_str = self.player_name
+        elif self.player_id is not None:
+            player_str = f"Player {self.player_id}"
+        else:
+            player_str = "System"
+        
+        # Build message based on event type
+        if self.event_type == EventType.DICE_ROLL:
+            dice = self.data.get('dice', [])
+            total = self.data.get('total', sum(dice))
+            return f"🎲 {player_str} rolled {dice} = {total}"
+        
+        elif self.event_type == EventType.RESOURCE_DIST:
+            resource = self.data.get('resource', '?')
+            recipients = self.data.get('recipients', [])
+            amounts = self.data.get('amounts', [])
+            distrib = ', '.join([f"Player {r}: {a}×{resource}" for r, a in zip(recipients, amounts)])
+            return f"📦 {distrib}"
+        
+        elif self.event_type == EventType.BUILD_SETTLEMENT:
+            point = self.data.get('point', '?')
+            if self.status == "SUCCESS":
+                return f"🏠 {player_str} built settlement at point {point}"
+            else:
+                return f"❌ {player_str} failed to build settlement at point {point}: {self.error}"
+        
+        elif self.event_type == EventType.BUILD_CITY:
+            point = self.data.get('point', '?')
+            if self.status == "SUCCESS":
+                return f"🏛️ {player_str} built city at point {point}"
+            else:
+                return f"❌ {player_str} failed to build city at point {point}: {self.error}"
+        
+        elif self.event_type == EventType.BUILD_ROAD:
+            points = self.data.get('points', [])
+            if self.status == "SUCCESS":
+                return f"🛤️ {player_str} built road {points[0]}→{points[1]}"
+            else:
+                return f"❌ {player_str} failed to build road {points}: {self.error}"
+        
+        elif self.event_type == EventType.BUY_DEV_CARD:
+            card = self.data.get('card', '?')
+            return f"🎴 {player_str} bought development card: {card}"
+        
+        elif self.event_type == EventType.USE_DEV_CARD:
+            card = self.data.get('card', '?')
+            return f"✨ {player_str} used {card}"
+        
+        elif self.event_type == EventType.TRADE_PROPOSE:
+            to_player = self.data.get('to_player', '?')
+            offer = self.data.get('offer', {})
+            request = self.data.get('request', {})
+            offer_str = ', '.join([f"{v}×{k}" for k, v in offer.items()])
+            request_str = ', '.join([f"{v}×{k}" for k, v in request.items()])
+            return f"💱 {player_str} proposed trade to {to_player}: [{offer_str}] for [{request_str}]"
+        
+        elif self.event_type == EventType.TRADE_RESPONSE:
+            response = self.data.get('response', '?')
+            return f"💬 {player_str} {response} trade"
+        
+        elif self.event_type == EventType.TRADE_EXECUTE:
+            details = self.data.get('details', '')
+            return f"✅ Trade executed: {details}"
+        
+        elif self.event_type == EventType.ROBBER_MOVE:
+            tile = self.data.get('tile', '?')
+            return f"🦹 {player_str} moved robber to tile {tile}"
+        
+        elif self.event_type == EventType.ROBBER_STEAL:
+            victim = self.data.get('victim', '?')
+            card = self.data.get('card', 'a card')
+            return f"🦹 {player_str} stole {card} from {victim}"
+        
+        elif self.event_type == EventType.TURN_START:
+            phase = self.data.get('phase', 'MAIN')
+            return f"➤ Turn {self.turn}: {player_str}'s turn begins ({phase})"
+        
+        elif self.event_type == EventType.TURN_END:
+            vp = self.data.get('vp', '?')
+            cards = self.data.get('cards', '?')
+            return f"⏹️ {player_str} ended turn [VP: {vp}, Cards: {cards}]"
+        
+        elif self.event_type == EventType.VICTORY:
+            vp = self.data.get('vp', 10)
+            return f"🏆 {player_str} WON with {vp} victory points!"
+        
+        else:
+            # Generic fallback
+            return f"• {player_str} performed {self.event_type.value}"
+
+
+def create_log_entry(event_type: EventType, turn: int, player_id: Optional[int] = None,
+                    player_name: Optional[str] = None, data: Optional[Dict[str, Any]] = None,
+                    status: str = "SUCCESS", error: Optional[str] = None) -> LogEntry:
+    """Helper function to create a log entry"""
+    return LogEntry(
+        timestamp=datetime.now(),
+        event_type=event_type,
+        turn=turn,
+        player_id=player_id,
+        player_name=player_name,
+        data=data or {},
+        status=status,
+        error=error
+    )

pycatan/web_visualization.py

@@ -22,6 +22,7 @@ except ImportError:
 from .visualization import Visualization
 from .actions import Action, ActionResult, GameState
 from .board_definition import board_definition
+from .log_events import EventType, LogEntry, create_log_entry
 
 
 class WebVisualization(Visualization):
@@ -65,6 +66,7 @@ class WebVisualization(Visualization):
         self.current_game_state = None
         self.action_history: List[Dict[str, Any]] = []
         self.event_history: List[Dict[str, Any]] = []  # Track all events (turn starts, dice rolls, etc.)
+        self.log_entries: List[LogEntry] = []  # Structured log entries
         
         # SSE (Server-Sent Events) for real-time updates
         self.sse_clients: List[Queue] = []
@@ -539,48 +541,141 @@ class WebVisualization(Visualization):
             if client in self.sse_clients:
                 self.sse_clients.remove(client)
     
+    def _map_action_to_event(self, action: Action, result: ActionResult) -> tuple:
+        """
+        Map ActionType to EventType and extract relevant data.
+        Returns: (EventType, data_dict)
+        """
+        from .actions import ActionType as AT
+        
+        event_data = {}
+        
+        # Extract common parameters
+        params = action.parameters if hasattr(action, 'parameters') else {}
+        
+        if action.action_type == AT.BUILD_SETTLEMENT or action.action_type == AT.PLACE_STARTING_SETTLEMENT:
+            event_type = EventType.BUILD_SETTLEMENT
+            event_data['point'] = params.get('point', '?')
+            event_data['is_starting'] = action.action_type == AT.PLACE_STARTING_SETTLEMENT
+            if result.success and 'cost' in params:
+                event_data['cost'] = params['cost']
+        
+        elif action.action_type == AT.BUILD_CITY:
+            event_type = EventType.BUILD_CITY
+            event_data['point'] = params.get('point', '?')
+            if result.success and 'cost' in params:
+                event_data['cost'] = params['cost']
+        
+        elif action.action_type == AT.BUILD_ROAD or action.action_type == AT.PLACE_STARTING_ROAD:
+            event_type = EventType.BUILD_ROAD
+            event_data['points'] = params.get('points', ['?', '?'])
+            event_data['is_starting'] = action.action_type == AT.PLACE_STARTING_ROAD
+            if result.success and 'cost' in params:
+                event_data['cost'] = params['cost']
+        
+        elif action.action_type == AT.BUY_DEV_CARD:
+            event_type = EventType.BUY_DEV_CARD
+            event_data['card'] = params.get('card', 'Unknown')
+            if result.success:
+                event_data['cost'] = ['ORE', 'SHEEP', 'WHEAT']
+        
+        elif action.action_type == AT.USE_DEV_CARD:
+            event_type = EventType.USE_DEV_CARD
+            event_data['card'] = params.get('card_type', 'Unknown')
+            if 'details' in params:
+                event_data['details'] = params['details']
+        
+        elif action.action_type == AT.ROLL_DICE:
+            event_type = EventType.DICE_ROLL
+            event_data['dice'] = params.get('dice', [0, 0])
+            event_data['total'] = sum(event_data['dice'])
+        
+        elif action.action_type == AT.ROBBER_MOVE:
+            event_type = EventType.ROBBER_MOVE
+            event_data['tile'] = params.get('tile', '?')
+            if 'victim' in params:
+                event_data['victim'] = params['victim']
+        
+        elif action.action_type == AT.TRADE_BANK:
+            event_type = EventType.TRADE_BANK
+            event_data['give'] = params.get('give', {})
+            event_data['receive'] = params.get('receive', {})
+        
+        elif action.action_type == AT.TRADE_PROPOSE:
+            event_type = EventType.TRADE_PROPOSE
+            event_data['to_player'] = params.get('to_player', '?')
+            event_data['offer'] = params.get('offer', {})
+            event_data['request'] = params.get('request', {})
+        
+        elif action.action_type == AT.TRADE_RESPOND:
+            event_type = EventType.TRADE_RESPONSE
+            event_data['response'] = params.get('response', 'UNKNOWN')
+        
+        elif action.action_type == AT.DISCARD_CARDS:
+            event_type = EventType.DISCARD_CARDS
+            event_data['cards'] = params.get('cards', [])
+            event_data['count'] = len(event_data['cards'])
+        
+        elif action.action_type == AT.END_TURN:
+            event_type = EventType.TURN_END
+            # Add player state at end of turn
+            if 'player_state' in params:
+                ps = params['player_state']
+                event_data['vp'] = ps.get('victory_points', 0)
+                event_data['cards'] = ps.get('card_count', 0)
+                event_data['roads'] = ps.get('roads', 0)
+                event_data['settlements'] = ps.get('settlements', 0)
+                event_data['cities'] = ps.get('cities', 0)
+        
+        else:
+            # Generic action
+            event_type = EventType.ACTION_FAILED if not result.success else EventType.TURN_START
+            event_data['action'] = action.action_type.name
+        
+        return event_type, event_data
+    
     def notify_action(self, action: Action, result: ActionResult):
         """Notify web clients of action execution."""
-        timestamp = datetime.now().strftime("%H:%M:%S")
-        
         # Get player name from action parameters (added by GameManager)
         player_name = action.parameters.get('player_name', f'Player {action.player_id + 1}') if hasattr(action, 'parameters') and action.parameters else f'Player {action.player_id + 1}'
         
-        # Generate a better message for the web log
-        message = result.error_message
-        if result.success:
-            action_name = action.action_type.name
-            if action_name == 'BUILD_SETTLEMENT':
-                message = f"{player_name} built a settlement"
-            elif action_name == 'BUILD_CITY':
-                message = f"{player_name} built a city"
-            elif action_name == 'BUILD_ROAD':
-                message = f"{player_name} built a road"
-            elif action_name == 'BUY_DEV_CARD':
-                message = f"{player_name} bought a development card"
-            elif action_name == 'ROLL_DICE':
-                message = f"{player_name} rolled dice"
-            elif action_name == 'END_TURN':
-                message = f"{player_name} ended turn"
-            elif action_name == 'TRADE_BANK':
-                message = f"{player_name} traded with bank"
-            elif action_name == 'TRADE_PLAYER':
-                message = f"{player_name} traded with player"
-            else:
-                message = f"{player_name} performed {action_name}"
-
+        # Map ActionType to EventType and extract relevant data
+        event_type, event_data = self._map_action_to_event(action, result)
+        
+        # Get turn number from action parameters (added by GameManager)
+        turn_number = action.parameters.get('turn_number', 0) if hasattr(action, 'parameters') and action.parameters else 0
+        
+        # Create structured log entry
+        log_entry = create_log_entry(
+            event_type=event_type,
+            turn=turn_number,
+            player_id=action.player_id,
+            player_name=player_name,
+            data=event_data,
+            status="SUCCESS" if result.success else "FAIL",
+            error=result.error_message if not result.success else None
+        )
+        
+        # Store log entry
+        self.log_entries.append(log_entry)
+        if len(self.log_entries) > 100:
+            self.log_entries = self.log_entries[-100:]
+        
+        # Create display data from log entry
         action_data = {
-            'timestamp': timestamp,
+            'timestamp': log_entry.timestamp.strftime("%H:%M:%S.%f")[:-3],
             'action_type': action.action_type.name,
+            'event_type': event_type.value,
             'player': action.player_id,
+            'player_name': player_name,
             'success': result.success,
-            'message': message
+            'message': log_entry.to_human_string(),
+            'structured': log_entry.to_log_string(),
+            'data': event_data
         }
         
         # Add to history
         self.action_history.append(action_data)
-        
-        # Keep only last 100 actions
         if len(self.action_history) > 100:
             self.action_history = self.action_history[-100:]
         
@@ -603,6 +698,42 @@ class WebVisualization(Visualization):
             'type': 'game_update',
             'payload': web_state
         })
+    
+    def log_event(self, log_entry: LogEntry):
+        """
+        Log a structured event directly.
+        This allows GameManager to send detailed log entries.
+        """
+        # Store log entry
+        self.log_entries.append(log_entry)
+        if len(self.log_entries) > 100:
+            self.log_entries = self.log_entries[-100:]
+        
+        # Create display data
+        event_data = {
+            'timestamp': log_entry.timestamp.strftime("%H:%M:%S.%f")[:-3],
+            'event_type': log_entry.event_type.value,
+            'turn': log_entry.turn,
+            'player_id': log_entry.player_id,
+            'player_name': log_entry.player_name,
+            'success': log_entry.status == "SUCCESS",
+            'status': log_entry.status,
+            'message': log_entry.to_human_string(),
+            'structured': log_entry.to_log_string(),
+            'data': log_entry.data,
+            'error': log_entry.error
+        }
+        
+        # Add to history
+        self.event_history.append(event_data)
+        if len(self.event_history) > 100:
+            self.event_history = self.event_history[-100:]
+        
+        # Broadcast to web clients
+        self._broadcast_to_clients({
+            'type': 'log_event',
+            'payload': event_data
+        })
 
     # ===== ConsoleVisualization Interface Compatibility =====
     # Adding methods to match ConsoleVisualization interface
@@ -632,62 +763,55 @@ class WebVisualization(Visualization):
     
     def display_turn_start(self, player_name: str, turn_number: int) -> None:
         """Display turn start notification (ConsoleVisualization interface)."""
-        timestamp = datetime.now().strftime("%H:%M:%S")
-        
-        message_data = {
-            'timestamp': timestamp,
-            'type': 'turn_start',
-            'player_name': player_name,
-            'turn_number': turn_number,
-            'message': f"Turn {turn_number}: {player_name}'s turn begins"
-        }
+        # Create structured log entry
+        log_entry = create_log_entry(
+            event_type=EventType.TURN_START,
+            turn=turn_number,
+            player_name=player_name,
+            data={'phase': 'MAIN'}
+        )
         
-        # Add to event history
-        self.event_history.append(message_data)
-        # Keep only last 100 events
-        if len(self.event_history) > 100:
-            self.event_history = self.event_history[-100:]
-        
-        # Broadcast to web clients
-        self._broadcast_to_clients({
-            'type': 'turn_start',
-            'payload': message_data
-        })
+        # Use the log_event method
+        self.log_event(log_entry)
     
     def display_dice_roll(self, player_name: str, dice_values: List[int], total: int) -> None:
         """Display dice roll results (ConsoleVisualization interface)."""
-        timestamp = datetime.now().strftime("%H:%M:%S")
-        
-        dice_data = {
-            'timestamp': timestamp,
-            'player_name': player_name,
-            'dice_values': dice_values,
-            'total': total,
-            'message': f"{player_name} rolled {dice_values} = {total}"
-        }
-        
-        # Add to event history
-        self.event_history.append(dice_data)
-        if len(self.event_history) > 100:
-            self.event_history = self.event_history[-100:]
+        # Create structured log entry
+        log_entry = create_log_entry(
+            event_type=EventType.DICE_ROLL,
+            turn=0,  # Turn number will be set by GameManager
+            player_name=player_name,
+            data={
+                'dice': dice_values,
+                'total': total
+            }
+        )
         
-        # Broadcast to web clients
-        self._broadcast_to_clients({
-            'type': 'dice_roll',
-            'payload': dice_data
-        })
+        # Use the log_event method
+        self.log_event(log_entry)
     
     def display_resource_distribution(self, distributions: Dict[str, List[str]]) -> None:
         """Display resource distribution (ConsoleVisualization interface)."""
-        timestamp = datetime.now().strftime("%H:%M:%S")
+        # Create structured log entry for each resource distribution
+        for resource, players in distributions.items():
+            log_entry = create_log_entry(
+                event_type=EventType.RESOURCE_DIST,
+                turn=0,  # Turn number will be set by GameManager
+                data={
+                    'resource': resource,
+                    'recipients': list(range(len(players))),  # Player IDs
+                    'amounts': [1] * len(players)  # Assuming 1 card each
+                }
+            )
+            self.log_event(log_entry)
         
+        # Keep compatibility with old event history format
+        timestamp = datetime.now().strftime("%H:%M:%S")
         distribution_data = {
             'timestamp': timestamp,
             'distributions': distributions,
             'message': "Resources distributed to players"
         }
-        
-        # Add to event history
         self.event_history.append(distribution_data)
         if len(self.event_history) > 100:
             self.event_history = self.event_history[-100:]

test_dice_log.py

@@ -0,0 +1,27 @@
+"""Quick test to check dice logging"""
+
+from pycatan import Game, GameManager, HumanUser, WebVisualization, ConsoleVisualization
+from pycatan.actions import ActionType, Action
+
+# Create a simple game
+users = [HumanUser("Alice"), HumanUser("Bob"), HumanUser("Charlie")]
+viz = ConsoleVisualization(use_colors=False)
+manager = GameManager(users, [viz])
+
+# Start game
+manager.start_game()
+
+# Create a roll dice action
+action = Action(
+    action_type=ActionType.ROLL_DICE,
+    player_id=0,
+    parameters={}
+)
+
+# Execute it
+result = manager.execute_action(action)
+
+print("\n=== Action Parameters After Execution ===")
+print(f"Parameters: {action.parameters}")
+print(f"Dice: {action.parameters.get('dice', 'NOT FOUND')}")
+print(f"Total: {action.parameters.get('total', 'NOT FOUND')}")