fix bugs and add 3 players test

game_viz.log

@@ -1,2 +1,830 @@
 
 >>> Turn 0: a's turn
+✓ a built a settlement
+
+==================================================
+                    GAME STATE                    
+==================================================
+
+Turn: 0
+Current Player: ► a
+
+PLAYERS
+-------
+
+► a
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 0
+
+b
+  Victory Points: 0
+  Resources: None
+  Buildings: Settlements: 0, Cities: 0, Roads: 0
+
+c
+  Victory Points: 0
+  Resources: None
+  Buildings: Settlements: 0, Cities: 0, Roads: 0
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+✓ a built a road
+
+==================================================
+                    GAME STATE                    
+==================================================
+
+Turn: 0
+Current Player: ► a
+
+PLAYERS
+-------
+
+► a
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 1
+
+b
+  Victory Points: 0
+  Resources: None
+  Buildings: Settlements: 0, Cities: 0, Roads: 0
+
+c
+  Victory Points: 0
+  Resources: None
+  Buildings: Settlements: 0, Cities: 0, Roads: 0
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+
+>>> Turn 1: b's turn
+✓ b built a settlement
+
+==================================================
+                    GAME STATE                    
+==================================================
+
+Turn: 1
+Current Player: ► b
+
+PLAYERS
+-------
+
+a
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 1
+
+► b
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 0
+
+c
+  Victory Points: 0
+  Resources: None
+  Buildings: Settlements: 0, Cities: 0, Roads: 0
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+✓ b built a road
+
+==================================================
+                    GAME STATE                    
+==================================================
+
+Turn: 1
+Current Player: ► b
+
+PLAYERS
+-------
+
+a
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 1
+
+► b
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 1
+
+c
+  Victory Points: 0
+  Resources: None
+  Buildings: Settlements: 0, Cities: 0, Roads: 0
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+
+>>> Turn 2: c's turn
+✓ c built a settlement
+
+==================================================
+                    GAME STATE                    
+==================================================
+
+Turn: 2
+Current Player: ► c
+
+PLAYERS
+-------
+
+a
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 1
+
+b
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 1
+
+► c
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 0
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+✓ c built a road
+
+==================================================
+                    GAME STATE                    
+==================================================
+
+Turn: 2
+Current Player: ► c
+
+PLAYERS
+-------
+
+a
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 1
+
+b
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 1
+
+► c
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 1
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+
+>>> Turn 3: c's turn
+
+📦 Resources distributed:
+  c: Ore, Wheat, Wheat
+✓ c built a settlement
+
+==================================================
+                    GAME STATE                    
+==================================================
+
+Turn: 3
+Current Player: ► c
+
+PLAYERS
+-------
+
+a
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 1
+
+b
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 1
+
+► c
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 1
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+✓ c built a road
+
+==================================================
+                    GAME STATE                    
+==================================================
+
+Turn: 3
+Current Player: ► c
+
+PLAYERS
+-------
+
+a
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 1
+
+b
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 1
+
+► c
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+
+>>> Turn 4: b's turn
+
+📦 Resources distributed:
+  b: Sheep, Sheep, Ore
+✓ b built a settlement
+
+==================================================
+                    GAME STATE                    
+==================================================
+
+Turn: 4
+Current Player: ► b
+
+PLAYERS
+-------
+
+a
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 1
+
+► b
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 1
+
+c
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+✓ b built a road
+
+==================================================
+                    GAME STATE                    
+==================================================
+
+Turn: 4
+Current Player: ► b
+
+PLAYERS
+-------
+
+a
+  Victory Points: 1
+  Resources: None
+  Buildings: Settlements: 1, Cities: 0, Roads: 1
+
+► b
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+c
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+
+>>> Turn 5: a's turn
+
+📦 Resources distributed:
+  a: Ore, Brick
+✓ a built a settlement
+
+==================================================
+                    GAME STATE                    
+==================================================
+
+Turn: 5
+Current Player: ► a
+
+PLAYERS
+-------
+
+► a
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 1
+
+b
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+c
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+✓ a built a road
+
+==================================================
+                    GAME STATE                    
+==================================================
+
+Turn: 5
+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
+
+c
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+BOARD
+-----
+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
+
+==================================================
+                    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
+
+c
+  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
+
+c
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+
+>>> Turn 7: b's turn
+
+🎲 b rolled: 6 + 1 = 7
+✓ b rolled the dice
+
+==================================================
+                    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
+
+c
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+✓ b performed 13
+
+==================================================
+                    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
+
+c
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+BOARD
+-----
+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
+
+c
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+
+>>> Turn 8: c's turn
+
+🎲 c rolled: 3 + 5 = 8
+
+📦 Resources distributed:
+  Player 1: Sheep
+  Player 2: Wheat
+✓ c rolled the dice
+
+==================================================
+                    GAME STATE                    
+==================================================
+
+Turn: 8
+Current Player: ► c
+
+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
+
+► c
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+✓ c ended their turn
+
+==================================================
+                    GAME STATE                    
+==================================================
+
+Turn: 8
+Current Player: ► c
+
+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
+
+► c
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+
+>>> Turn 9: a's turn
+
+🎲 a rolled: 2 + 5 = 7
+✓ a rolled the dice
+
+==================================================
+                    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
+
+c
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+✓ a performed 13
+
+==================================================
+                    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
+
+c
+  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
+
+c
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+
+>>> Turn 10: b's turn
+
+🎲 b rolled: 4 + 1 = 5
+
+📦 Resources distributed:
+  Player 1: Ore
+✓ b rolled the dice
+
+==================================================
+                    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: 2
+
+c
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+✓ b bought a development card
+
+==================================================
+                    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
+  Dev Cards: 1
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+c
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+
+✗ b used road
+  Error: Action ActionType.USE_DEV_CARD not yet implemented
+
+==================================================
+                    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
+  Dev Cards: 1
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+c
+  Victory Points: 2
+  Resources: None
+  Buildings: Settlements: 2, Cities: 0, Roads: 2
+
+BOARD
+-----
+Board Tiles: 19 tiles configured
+

pycatan/game.py

@@ -83,6 +83,7 @@ class Game:
         self.players[player].add_dev_card(self.dev_deck[0])
         # removes that dev card from the deck
         del self.dev_deck[0]
+        return Statuses.ALL_GOOD
 
     # gives players the proper cards for a given roll
     def add_yield_for_roll(self, roll):

pycatan/game_manager.py

@@ -227,6 +227,12 @@ class GameManager:
                 return self._handle_roll_dice(action)
             elif action.action_type in [ActionType.TRADE_PROPOSE, ActionType.TRADE_ACCEPT, ActionType.TRADE_REJECT]:
                 return self._handle_trade_action(action)
+            elif action.action_type == ActionType.TRADE_BANK:
+                return self._execute_trade_bank(action)
+            elif action.action_type == ActionType.BUY_DEV_CARD:
+                return self._execute_buy_dev_card(action)
+            elif action.action_type == ActionType.USE_DEV_CARD:
+                return self._execute_use_dev_card(action)
             elif action.action_type == ActionType.DISCARD_CARDS:
                 return self._handle_discard_cards(action)
             elif action.action_type == ActionType.ROBBER_MOVE:
@@ -614,6 +620,129 @@ class GameManager:
         
         return resource_map.get(resource_name.lower())
     
+    def _execute_buy_dev_card(self, action: Action) -> ActionResult:
+        """Execute buying a development card."""
+        try:
+            player_id = action.player_id
+            
+            # Call the game's build_dev method
+            status = self.game.build_dev(player_id)
+            
+            if status == Statuses.ALL_GOOD:
+                # Get the card that was just added (last card in player's dev_cards list)
+                player = self.game.players[player_id]
+                if player.dev_cards:
+                    card_bought = player.dev_cards[-1]
+                    print(f"    ✓ Bought development card: {card_bought.name}")
+                else:
+                    print(f"    ✓ Bought development card")
+                
+                return ActionResult.success_result(
+                    self.get_full_state(),
+                    affected_players=[player_id]
+                )
+            else:
+                return self._map_status_to_result(status)
+                
+        except Exception as e:
+            return ActionResult.failure_result(
+                f"Error buying development card: {str(e)}",
+                "EXECUTION_ERROR"
+            )
+    
+    def _execute_use_dev_card(self, action: Action) -> ActionResult:
+        """Execute using a development card - interactive multi-step process."""
+        try:
+            player_id = action.player_id
+            card_type_str = action.parameters.get('card_type')
+            
+            if not card_type_str:
+                return ActionResult.failure_result(
+                    "Missing card_type parameter",
+                    "MISSING_PARAMETER"
+                )
+            
+            # Convert string to DevCard enum
+            from pycatan.card import DevCard
+            try:
+                card_type = DevCard[card_type_str]
+            except KeyError:
+                return ActionResult.failure_result(
+                    f"Invalid card type: {card_type_str}",
+                    "INVALID_CARD_TYPE"
+                )
+            
+            # Check if player has the card
+            if not self.game.players[player_id].has_dev_cards([card_type]):
+                return ActionResult.failure_result(
+                    f"You don't have a {card_type.name} card",
+                    "NO_CARD"
+                )
+            
+            # Route to specific card handler
+            if card_type == DevCard.Knight:
+                return self._use_knight_card(player_id, action)
+            elif card_type == DevCard.Road:
+                return self._use_road_building_card(player_id, action)
+            elif card_type == DevCard.Monopoly:
+                return self._use_monopoly_card(player_id, action)
+            elif card_type == DevCard.YearOfPlenty:
+                return self._use_year_of_plenty_card(player_id, action)
+            elif card_type == DevCard.VictoryPoint:
+                return ActionResult.failure_result(
+                    "Victory Point cards are counted automatically - don't use them!",
+                    "CANNOT_USE_VP"
+                )
+            else:
+                return ActionResult.failure_result(
+                    f"Unknown card type: {card_type}",
+                    "UNKNOWN_CARD"
+                )
+                
+        except Exception as e:
+            return ActionResult.failure_result(
+                f"Error using development card: {str(e)}",
+                "EXECUTION_ERROR"
+            )
+    
+    def _use_road_building_card(self, player_id: int, action: Action) -> ActionResult:
+        """Use Road Building card - needs 2 roads."""
+        # For now, return a message asking for input
+        # In the future, this will be an interactive multi-step process
+        return ActionResult.failure_result(
+            "Road Building card usage not yet fully implemented.\n"
+            "    💬 You need to specify 2 roads to build.\n"
+            "    Example format needed: rd [point1] [point2] and rd [point3] [point4]",
+            "NOT_IMPLEMENTED"
+        )
+    
+    def _use_knight_card(self, player_id: int, action: Action) -> ActionResult:
+        """Use Knight card - move robber and steal."""
+        return ActionResult.failure_result(
+            "Knight card usage not yet fully implemented.\n"
+            "    💬 You need to move the robber and optionally steal from a player.\n"
+            "    Use 'robber [tile_id]' first, then this card will work.",
+            "NOT_IMPLEMENTED"
+        )
+    
+    def _use_monopoly_card(self, player_id: int, action: Action) -> ActionResult:
+        """Use Monopoly card - take all of one resource."""
+        return ActionResult.failure_result(
+            "Monopoly card usage not yet fully implemented.\n"
+            "    💬 You need to specify which resource to monopolize.\n"
+            "    Example: All players give you their Wood/Brick/Sheep/Wheat/Ore",
+            "NOT_IMPLEMENTED"
+        )
+    
+    def _use_year_of_plenty_card(self, player_id: int, action: Action) -> ActionResult:
+        """Use Year of Plenty card - take 2 resources from bank."""
+        return ActionResult.failure_result(
+            "Year of Plenty card usage not yet fully implemented.\n"
+            "    💬 You need to specify 2 resource cards to take from the bank.\n"
+            "    Example: Take 1 Wood and 1 Brick (or 2 of the same)",
+            "NOT_IMPLEMENTED"
+        )
+    
     def start_game(self) -> bool:
         """
         Start the game session.

pycatan/game_moves_3Players.txt

@@ -0,0 +1,29 @@
+3
+a
+b
+c
+s 10
+rd 10 11
+s 14
+rd 14 13
+s 31
+rd 31 30
+s 44
+rd 44 43
+s 36
+rd 36 35
+s 40
+rd 40 41
+roll
+end
+roll
+robber 9
+end
+roll
+end
+roll
+robber 19
+end
+roll
+buy dev card
+use road

pycatan/human_user.py

@@ -445,21 +445,27 @@ class HumanUser(User):
             raise UserInputError("Use command requires card type. Example: 'use knight' or 'use road'")
         
         card_name = parts[1].lower()
-        params = {'card_type': card_name}
         
-        # Add specific parameters based on card type
-        if card_name == 'knight' and len(parts) >= 5:
-            try:
-                robber_row = int(parts[2])
-                robber_index = int(parts[3])
-                victim_player = int(parts[4]) if parts[4] != 'none' else None
-                
-                params.update({
-                    'tile_coords': [robber_row, robber_index],
-                    'victim': victim_player
-                })
-            except ValueError:
-                raise UserInputError("Knight card format: 'use knight [robber_row] [robber_index] [victim_player_or_none]'")
+        # Map user-friendly names to DevCard enum values
+        card_type_map = {
+            'knight': 'Knight',
+            'road': 'Road',
+            'roadbuilding': 'Road',
+            'monopoly': 'Monopoly',
+            'yearofplenty': 'YearOfPlenty',
+            'plenty': 'YearOfPlenty',
+            'year': 'YearOfPlenty'
+        }
+        
+        if card_name not in card_type_map:
+            raise UserInputError(f"Unknown card type '{card_name}'. Valid: knight, road, monopoly, yearofplenty")
+        
+        dev_card_type = card_type_map[card_name]
+        params = {'card_type': dev_card_type}
+        
+        # Each card type needs specific additional input
+        # For now, we'll create the action with just the card type
+        # The GameManager will request additional input as needed
         
         return Action(ActionType.USE_DEV_CARD, self.user_id, params)
     
@@ -642,9 +648,21 @@ class HumanUser(User):
         print("  Examples: 'trade bank wood 4 sheep 1' or 't player v wood sheep'")
         print()
         print("🃏 DEVELOPMENT CARDS:")
-        print("  buy                 - Buy development card (short: dev)")
+        print("  buy / dev           - Buy dev card (cost: 1 Ore + 1 Sheep + 1 Wheat)")
         print("  use <card_type>     - Use development card")
         print()
+        print("  📋 Card Types & Effects:")
+        print("     knight         - Move robber + steal card (gives +1 knight count)")
+        print("     road           - Build 2 free roads instantly")
+        print("     monopoly       - Take ALL cards of one resource from all players")
+        print("     yearofplenty   - Take any 2 resource cards from bank")
+        print("     victorypoint   - +1 VP (auto-counted, don't use manually)")
+        print()
+        print("  💡 Tips:")
+        print("     • 3+ knights = Largest Army (2 VP)")
+        print("     • Cards are interactive - game will ask for details after 'use'")
+        print("     • Example: 'use road' then follow prompts")
+        print()
         print("🎲 TURN ACTIONS:")
         print("  roll               - Roll dice (short: r, dice)")
         print("  end                - End turn (short: pass, done)")

pycatan/player.py

@@ -345,7 +345,7 @@ class Player:
         # adds VPs from developement cards
         if include_dev:
             for d in self.dev_cards:
-                if d == DevCard.VP:
+                if d == DevCard.VictoryPoint:
                     points += 1
 
         return points

pycatan/static/js/main.js

@@ -34,7 +34,7 @@ function toggleVertices() {
 function toggleBuildingCosts() {
     const modal = document.getElementById('buildingCostsModal');
     if (modal) {
-        modal.classList.toggle('show');
+        modal.classList.toggle('hidden');
     }
 }
 

pycatan/web_visualization.py

@@ -543,28 +543,31 @@ class WebVisualization(Visualization):
         """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 {action.player_id} built a settlement"
+                message = f"{player_name} built a settlement"
             elif action_name == 'BUILD_CITY':
-                message = f"Player {action.player_id} built a city"
+                message = f"{player_name} built a city"
             elif action_name == 'BUILD_ROAD':
-                message = f"Player {action.player_id} built a road"
+                message = f"{player_name} built a road"
             elif action_name == 'BUY_DEV_CARD':
-                message = f"Player {action.player_id} bought a development card"
+                message = f"{player_name} bought a development card"
             elif action_name == 'ROLL_DICE':
-                message = f"Player {action.player_id} rolled dice"
+                message = f"{player_name} rolled dice"
             elif action_name == 'END_TURN':
-                message = f"Player {action.player_id} ended turn"
+                message = f"{player_name} ended turn"
             elif action_name == 'TRADE_BANK':
-                message = f"Player {action.player_id} traded with bank"
+                message = f"{player_name} traded with bank"
             elif action_name == 'TRADE_PLAYER':
-                message = f"Player {action.player_id} traded with player"
+                message = f"{player_name} traded with player"
             else:
-                message = f"Player {action.player_id} performed {action_name}"
+                message = f"{player_name} performed {action_name}"
 
         action_data = {
             'timestamp': timestamp,

בלוג/INDEX.md

@@ -59,6 +59,22 @@
    - דוגמאות קוד: Statuses enum, Player.build_settlement, GameManager.execute_build_settlement, Testing
    - מצב: פורסם
 
+5. `פוסט בלוג 5 - דיבאג ב-Vibe Coding.md`
+   - נושא: דיבאג בעולם ה-Vibe Coding - כשהסוכן צריך לראות מה המחשב רואה
+   - נקודות עיקריות:
+     - האתגר: 3 מערכות קואורדינטות שלא מדברות (Game Logic, Axial, Pixels)
+     - כל הניסיונות שנכשלו: חישוב מתמטי, החלפת מודלים, בנייה מחדש
+     - התובנה: במקום לתקן - לבנות כלי שיראה מה המחשב "חושב"
+     - הפתרון: Manual Mapping Tool - ממשק לדיבאג משותף אדם-מכונה
+     - print_game_logic.py להדפסת הקשרים בין משושים לנקודות
+     - תהליך המיפוי הידני: הסתכלות + לחיצה = מיפוי מושלם
+     - התוצאה: PointMapper class שמתרגם ID פשוט לקואורדינטות
+     - המחשבה הגדולה: סוכנים צריכים ללמוד לבנות גשרים, לא רק לתקן קוד
+     - נגישות vibe coding: הסוכן צריך להציע עזרה דרך חוויית המשתמש
+     - מתי לעצור פיתוח ולבנות כלי דיבאג
+   - דוגמאות קוד: print_game_logic.py, manual_mapping.js, handlePointClick, PointMapper
+   - מצב: פורסם (8 דצמבר 2025)
+
 ---
 
 ## Next Posts (Planned)

בלוג/פוסט בלוג 5 - דיבאג ב-Vibe Coding.md

@@ -0,0 +1,438 @@
+# פוסט בלוג 5: דיבאג ב-Vibe Coding - כשהסוכן צריך לראות מה המחשב רואה
+
+*תאריך: 8 בדצמבר 2025*
+
+## פתיחה: הפואנטה לפני הכל
+
+אני בונה סביבה שבה סוכנים מבוססי LLM יכולים לשחק קטאן, ובדרך גיליתי משהו חשוב על העתיד של vibe coding:
+
+**כדי ש-vibe coding יהפוך באמת נגיש - לא רק למתכנתים או לאנשים סופר-טכניים - הסוכנים שאנחנו כותבים איתם חייבים לעשות קפיצת מדרגה משמעותית בכל מה שקשור לדיבאג.**
+
+לא מדובר רק ביכולת לקרוא לוגים או לבדוק שגיאות קומפילציה. מדובר באיך הסוכן "מסתכל" על בעיות, ואיך הוא מציע לפתור אותן יחד עם המשתמש.
+
+בואו נראה למה אני מתכוון דרך דוגמה אמיתית מהפרויקט.
+
+---
+
+## הבעיה: שלוש מערכות קואורדינטות שלא מדברות
+
+### מה יש בלוח קטאן?
+
+בלוח קטאן יש בגדול שתי ישויות בסיסיות:
+- **19 משושים (tiles)** - שם יושבים המספרים והמשאבים
+- **54 נקודות (vertices/points)** - הקודקודים שבהם אפשר לבנות התנחלויות
+
+```
+         🔵━━━🔵━━━🔵
+        ╱ ⬡ ╲ ╱ ⬡ ╲ ╱ ⬡ ╲
+       🔵━━━🔵━━━🔵━━━🔵
+      ╱ ⬡ ╲ ╱ ⬡ ╲ ╱ ⬡ ╲ ╱ ⬡ ╲
+     🔵━━━🔵━━━🔵━━━🔵━━━🔵
+        ╲ ⬡ ╱ ╲ ⬡ ╱ ╲ ⬡ ╱ ╲ ⬡ ╱
+         🔵━━━🔵━━━🔵━━━🔵
+            ╲ ⬡ ╱ ╲ ⬡ ╱ ╲ ⬡ ╱
+             🔵━━━🔵━━━🔵
+             
+       🔵 = נקודה (vertex) - מקום לבניית התנחלות
+       ⬡ = משושה (tile) - מייצר משאבים
+```
+
+כל ישות מוחזקת במבנה נתונים עצמאי, כך שלכל אחת יש קואורדינטות ומזהה ייחודי:
+
+```python
+# כך מנוע המשחק רואה משושה
+class Tile:
+    def __init__(self, type, token_num, position, points):
+        self.position = position  # [row, index] - למשל [0, 0]
+        self.points = points      # רשימת הנקודות שמקיפות אותו
+```
+
+```python
+# כך מנוע המשחק רואה נקודה
+class Point:
+    def __init__(self, tiles, position):
+        self.position = position  # [row, index] - למשל [2, 5]
+        self.tiles = tiles        # המשושים שהנקודה נוגעת בהם
+```
+
+### הסיבוך האמיתי: הויזואליזציה
+
+הבעיה מתחילה כשרוצים **להציג את זה על המסך**.
+
+יצרתי תצוגה ווב עם קלוד - שרת Flask מצד אחד, ו-JavaScript שמצייר את הלוח מצד שני:
+
+```javascript
+// board.js - ציור המשושים
+hexToPixel(q, r) {
+    // המרה ממערכת axial לפיקסלים
+    const x = this.hexRadius * (3/2 * q);
+    const y = this.hexRadius * (Math.sqrt(3)/2 * q + Math.sqrt(3) * r);
+    return {
+        x: this.centerX + x,
+        y: this.centerY + y
+    };
+}
+
+getHexagonVertices(q, r) {
+    const center = this.hexToPixel(q, r);
+    const vertices = [];
+    
+    for (let i = 0; i < 6; i++) {
+        const angle = (Math.PI / 3) * i;  // כל 60 מעלות
+        const x = center.x + this.hexRadius * Math.cos(angle);
+        const y = center.y + this.hexRadius * Math.sin(angle);
+        vertices.push({x: x, y: y});
+    }
+    
+    return vertices;
+}
+```
+
+עכשיו יש לי **שלוש מערכות קואורדינטות**:
+
+| מערכת | שימוש | דוגמה |
+|--------|-------|-------|
+| **Game Logic** | מנוע המשחק ב-Python | `[row, index]` = `[2, 5]` |
+| **Axial Coordinates** | ציור משושים ב-JS | `(q, r)` = `(-1, 2)` |
+| **Pixel Coordinates** | מיקום על המסך | `(x, y)` = `(342, 267)` |
+
+**והן צריכות להיות מסונכרנות!**
+
+כשמשתמש לוחץ על נקודה במסך, אני צריך:
+1. לתפוס את הפיקסלים `(342, 267)`
+2. לתרגם למערכת axial 
+3. לתרגם לקואורדינטות של המשחק `[2, 5]`
+4. להעביר לפייתון לביצוע הפעולה
+
+ואם יש טעות באחד השלבים? **הלוח נשבר.**
+
+---
+
+## כשום דבר לא עובד
+
+ניסיתי הכל:
+
+### ניסיון 1: לתת לקלוד לחשב
+```
+אני: "תחשב את התרגום בין מערכות הקואורדינטות"
+קלוד: *כותב פונקציה מתמטית מורכבת*
+תוצאה: הנקודות מוצגות, אבל לא במקום הנכון
+```
+
+### ניסיון 2: לפשט את המבנה
+```
+אני: "בוא ננסה גישה אחרת, נשתמש במזהים פשוטים 1-54"
+קלוד: *בונה מערכת מיפוי*
+תוצאה: המיפוי לא תואם את מה שעל המסך
+```
+
+### ניסיון 3: לבנות מחדש בנפרד
+```
+אני: "בוא נבנה רק את הלוח בפרויקט נפרד ונחבר חזרה"
+קלוד: *בונה פרויקט קטן*
+תוצאה: עובד בנפרד, נשבר בחיבור
+```
+
+### ניסיון 4: להחליף מודלים
+```
+קלוד -> Gemini -> GPT-4 -> חזרה לקלוד
+תוצאה: כולם נתקעים באותו מקום
+```
+
+**התסמינים היו תמיד זהים:**
+- או שהלוח עצמו נשבר ויזואלית
+- או שהתוצאה לא הייתה הגיונית - על המסך רואים שתי נקודות צמודות, אבל מנוע המשחק טוען שהן רחוקות ולכן אי אפשר לבנות דרך
+
+נשארתי עם זה קצת. שקלתי לכתוב את זה מחדש בעצמי, אבל זה סופר מורכב.
+
+**ואז בא לי רעיון.**
+
+---
+
+## התובנה: "מה אם הסוכן יראה לי איך הוא רואה את המשחק?"
+
+במקום לבקש מהסוכן לתקן את הבעיה, ביקשתי ממנו **לחשוף את מה שהוא "חושב"**.
+
+### שלב 1: הדפסת הלוגיקה הפנימית
+
+יצרנו סקריפט קטן שמדפיס בדיוק מה המשחק "רואה":
+
+```python
+# print_game_logic.py
+from pycatan import Game
+
+def print_game_expectations():
+    game = Game()
+    board = game.board
+    
+    print("GAME LOGIC EXPECTATIONS")
+    print("="*60)
+    print("Format: Hex [Row, Col] -> Connected Point Coordinates")
+    print("-"*60)
+
+    for r, row in enumerate(board.tiles):
+        for i, tile in enumerate(row):
+            point_coords = [list(p.position) for p in tile.points]
+            point_coords.sort()
+            print(f"Hex [{r}, {i}] connects to Points: {point_coords}")
+
+if __name__ == "__main__":
+    print_game_expectations()
+```
+
+הפלט נראה בערך ככה:
+
+```
+Hex [0, 0] connects to Points: [[0,0], [0,1], [1,0], [1,1], [1,2], [2,1]]
+Hex [0, 1] connects to Points: [[0,1], [0,2], [1,2], [1,3], [1,4], [2,3]]
+Hex [0, 2] connects to Points: [[0,2], [0,3], [1,4], [1,5], [1,6], [2,5]]
+...
+```
+
+עכשיו אני רואה **מה המחשב חושב** - איזה משושה מחובר לאילו נקודות.
+
+### שלב 2: כלי מיפוי אינטראקטיבי
+
+ביקשתי מקלוד ליצור ממשק וובי פשוט לדיבאג:
+
+```html
+<!-- manual_mapping.html -->
+<div class="mapping-controls">
+    <div class="mode-switch">
+        <label><input type="radio" name="mode" value="hex" checked> 
+            Map Hexes (1-19)</label>
+        <label><input type="radio" name="mode" value="point"> 
+            Map Points (1-54)</label>
+    </div>
+    
+    <div class="current-target">
+        Click to assign ID: <span id="nextId">1</span>
+        <div>
+            Target Game Coords: 
+            <span id="coordsHint">Row 0, Col 0</span>
+        </div>
+    </div>
+
+    <button onclick="exportMapping()">Export Mapping</button>
+    <textarea id="output" placeholder="Mapping will appear here..."></textarea>
+</div>
+```
+
+הרעיון פשוט:
+1. רואה על המסך "Target: Row 0, Col 0"
+2. מסתכל על ההדפסות - "זה הצומת השמאלי-עליון"
+3. לוחץ על הצומת המתאים בלוח הויזואלי
+4. הצומת הופך לירוק ומקבל מספר
+5. עובר לצומת הבא
+
+```javascript
+// manual_mapping.js
+handlePointClick(element) {
+    const visualId = parseInt(element.getAttribute('data-vertex-id'));
+    const vertex = this.vertices.find(v => v.id === visualId);
+
+    // שמירת המיפוי
+    this.mapping.points[this.currentId] = {
+        x: vertex.x,
+        y: vertex.y
+    };
+
+    // פידבק ויזואלי - הצומת הופך ירוק
+    element.classList.add('mapped');
+    
+    // מעבר לצומת הבא
+    this.currentId++;
+    this.updateUI();
+}
+```
+
+### שלב 3: התהליך בפועל
+
+```
+Target Game Coords: Row 0, Col 0
+Click to assign ID: 1
+
+[לוחץ על הצומת השמאלי העליון]
+
+✓ Mapped Point 1 -> (x: 287, y: 178)
+
+Target Game Coords: Row 0, Col 1  
+Click to assign ID: 2
+
+[לוחץ על הצומת הבא מימין]
+
+✓ Mapped Point 2 -> (x: 332, y: 152)
+
+...חוזר 52 פעמים נוספות...
+```
+
+### שלב 4: ייצוא המיפוי
+
+אחרי שסיימתי את כל 54 הנקודות, לחצתי "Export":
+
+```json
+{
+  "points": {
+    "1": {"x": 287, "y": 178},
+    "2": {"x": 332, "y": 152},
+    "3": {"x": 377, "y": 178},
+    ...
+    "54": {"x": 512, "y": 422}
+  }
+}
+```
+
+**וזה עבד.**
+
+זה לקח לי בערך חצי שעה. היה צריך רגע להבין מה זה כל משושה ואיזה נקודות מחוברות אליו, אבל ברגע שהדפוס ההגיוני התחיל להיווצר וראיתי את המיפוי **בעיניים** באופן ידני - זה פשוט רץ.
+
+---
+
+## הקוד שנוצר: PointMapper
+
+המיפוי הידני הפך לקלא�� `PointMapper` שמשמש את כל המערכת:
+
+```python
+# point_mapping.py
+class PointMapper:
+    """
+    Manages mapping between point IDs and coordinates.
+    
+    Point IDs are simple numbers (1, 2, 3...) that users can easily reference.
+    Coordinates are [row, index] pairs used internally by the game engine.
+    """
+    
+    def __init__(self):
+        self.point_to_coords: Dict[int, List[int]] = {}
+        self.coords_to_point: Dict[str, int] = {}
+        self._load_default_mapping()
+    
+    def _load_default_mapping(self):
+        """Load the default Catan board point mapping."""
+        # Standard Catan board layout - 54 intersection points
+        default_mapping = [
+            # Top row (7 points)
+            [0, 0], [0, 1], [0, 2], [0, 3], [0, 4], [0, 5], [0, 6],
+            # Second row (9 points) 
+            [1, 0], [1, 1], [1, 2], [1, 3], [1, 4], [1, 5], [1, 6], [1, 7], [1, 8],
+            # ... ממשיך עד 54 נקודות
+        ]
+        
+        for point_id, coords in enumerate(default_mapping, 1):
+            self.point_to_coords[point_id] = coords
+            self.coords_to_point[f"{coords[0]},{coords[1]}"] = point_id
+    
+    def point_to_coordinate(self, point_id: int) -> Optional[List[int]]:
+        """Convert point ID to coordinates."""
+        return self.point_to_coords.get(point_id)
+    
+    def coordinate_to_point(self, row: int, index: int) -> Optional[int]:
+        """Convert coordinates to point ID."""
+        return self.coords_to_point.get(f"{row},{index}")
+```
+
+עכשיו במקום:
+```python
+# לפני - צריך לדעת קואורדינטות פנימיות
+game.add_settlement(player=0, point=board.points[2][5])
+# מה זה [2][5]?? איפה זה על הלוח??
+```
+
+יש:
+```python
+# אחרי - מספר פשוט
+point_id = 23  # הצומת שאני רואה על המסך
+coords = mapper.point_to_coordinate(point_id)  # מחזיר [2, 5]
+game.add_settlement(player=0, point=board.points[coords[0]][coords[1]])
+```
+
+---
+
+## המחשבה הגדולה: מה זה אומר על עתיד ה-Vibe Coding
+
+### המצב היום
+
+אנחנו נמצאים בנקודה ש-vibe coding מרגיש לפעמים כמו לדבר עם הסוכן בטלפון:
+
+- אם הסוכן במוד נחמד, יש סיכוי לדיבאג אמיתי
+- אם אתם מספיק טכניים, אפשר לכוון אותו
+- לפעמים יש לוגים ובדיקה שלב-שלב כחלק מהשיחה
+
+**אבל אני מדבר על משהו מעבר לזה.**
+
+### מה חסר?
+
+סוכן טוב צריך להתחיל "לחשוב" ולהניח שהתוצר שלו הוא **כנראה לא אופטימלי**. במקום להמשיך לנסות לתקן את הקוד, הוא צריך:
+
+1. **לזהות שהוא תקוע** - לא רק להמשיך לנסות וריאציות על אותו פתרון
+2. **לחפש דרכים אחרות לאבחן** - דרך חוויית המשתמש, לא רק דרך הקוד
+3. **להציע כלי עזר** - אולי אפילו לעצור את הפיתוח ולבנות כלי דיבאג
+
+### הדוגמה מהפרויקט
+
+במקרה שלי, הפתרון לא היה "לתקן את הקוד". הפתרון היה:
+
+> **"בוא נבנה כלי שיעזור לי ולך לראות את אותו הדבר"**
+
+הסוכן לא יכול היה לפתור את בעיית המיפוי לבד. אבל הוא **כן יכול** לבנות כלי שיעזור **לנו יחד** לפתור אותה:
+
+```
+┌─────────────────────────────────────────────────────────────┐
+│  מצב נוכחי: סוכן מנסה לפתור לבד                            │
+│                                                             │
+│  User ──"תתקן את הבאג"──> Agent ──tries──> fails ──> tries  │
+│                                                             │
+└─────────────────────────────────────────────────────────────┘
+
+┌─────────────────────────────────────────────────────────────┐
+│  מצב רצוי: סוכן בונה גשר לשיתוף פעולה                      │
+│                                                             │
+│  User <──"בוא נראה יחד"──> Agent                           │
+│    │                          │                             │
+│    │    ┌────────────────┐    │                             │
+│    └───>│  Debug Tool    │<───┘                             │
+│         │  (visual)      │                                  │
+│         └───────────���────┘                                  │
+│                │                                            │
+│                v                                            │
+│         Problem Solved                                      │
+│                                                             │
+└─────────────────────────────────────────────────────────────┘
+```
+
+### למה זה חשוב לנגישות?
+
+כדי שבניית מוצרים תפסיק להיות משאב ששייך רק למי שטכני, הסוכנים צריכים ללמוד להציע עזרה בצורות שלא בהכרח באות מאיך שמתכנת רואה את הדברים:
+
+| גישה של מתכנת | גישה של משתמש |
+|---------------|---------------|
+| "תבדוק את הלוגים" | "תראה לי מה אתה רואה" |
+| "יש exception בשורה 47" | "למה הכפתור לא עובד?" |
+| "הקואורדינטות לא נכונות" | "זה לא נראה כמו שצריך" |
+
+סוכן שמבין את **שני** הצדדים יכול לגשר ביניהם.
+
+---
+
+## סיכום: כלי דיבאג כחלק מהפתרון
+
+הלקח מהניסיון הזה:
+
+> **לפעמים הפתרון לבעיה הוא לא לתקן את הקוד - אלא לבנות כלי שיעזור לנו להבין את הבעיה יחד.**
+
+בפרויקט PyCatan, הכלי הזה היה `manual_mapping.html` - ממשק פשוט שהראה לי **מה המחשב רואה** ואפשר לי **להגיד לו מה נכון**.
+
+זה לקח חצי שעה של עבודה ידנית, אבל:
+- פתר בעיה שסוכנים לא הצליחו לפתור שבועות
+- יצר artifact שימושי לפרויקט (קובץ המיפוי)
+- לימד אותי משהו על איך לעבוד עם סוכנים
+
+**ואולי הכי חשוב:** זה הראה לי שהעתיד של vibe coding הוא לא רק סוכנים שכותבים קוד טוב יותר - אלא סוכנים שיודעים **מתי לעצור ולבנות גשר** בין מה שהם רואים לבין מה שאנחנו רואים.
+
+---
+
+## קישורים
+
+- [הקוד של Manual Mapping Tool](../pycatan/static/js/manual_mapping.js)
+- [הקוד של PointMapper](../pycatan/point_mapping.py)
+- [פוסט קודם על מערכת הקואורדינטות](פוסט%20בלוג%203%20-%20קואורדינטות%20וקסם%20שחור.md)