feat: show all hole cards and bot names in hand-history mode (#33)

* feat: show all hole cards and bot names in hand-history mode

Improves hand-history mode for bot testing and debugging by showing all
players' hole cards from the start of each hand, and displaying bot names
instead of IDs throughout.

Changes:
- Add HoleCards field to HandPlayer struct for passing cards to monitors
- Update hand runner to pass hole cards and display names in OnHandStart
- Modify pretty monitor to display all players' hole cards in the
  "HOLE CARDS" section
- Use bot.DisplayName() instead of bot.ID for consistent naming

This makes it much easier to understand bot decision-making when reviewing
hands, as you can see what everyone was holding from the start rather than
only at showdown.

Example output:
  *** HOLE CARDS ***
  Dealt to aragorn-dev Q♣ K♠
  Dealt to complex-bot-1 7♣ J♥
  Dealt to complex-bot-2 9♦ K♦

All names (aragorn-dev, complex-bot-1, etc.) now appear consistently in
headers, actions, and summary.

* fix: preserve bot ID in HandPlayer.Name for stats tracking

Addresses P1 feedback: using DisplayName as HandPlayer.Name broke stats
tracking because StatsMonitor uses player.Name as the stable key for
indexing bot statistics.

Changes:
- Add DisplayName field to HandPlayer struct for human-readable display
- Keep Name field as bot.ID for stable stats tracking (prevents stats
  key mismatches and duplicate display name conflicts)
- Update pretty monitor to use DisplayName for all display purposes
  (seat listings, hole cards, actions)

This ensures stats tracking works correctly while still showing friendly
bot names (aragorn-dev, complex-bot-1, etc.) consistently in hand-history
output.

Co-authored-by: chatgpt-codex-connector[bot]

Author: lox

internal/server/hand_runner.go

@@ -320,10 +320,20 @@ func (hr *HandRunner) broadcastHandStart() {
 		monitor := hr.pool.GetHandMonitor()
 		players := make([]HandPlayer, len(hr.bots))
 		for i, bot := range hr.bots {
+			player := hr.handState.Players[i]
+			displayName := bot.DisplayName()
+			if displayName == "" {
+				displayName = bot.ID // Fallback to ID if no display name
+			}
 			players[i] = HandPlayer{
-				Seat:  i,
-				Name:  bot.ID, // Use bot ID for stats tracking
-				Chips: hr.handState.Players[i].Chips,
+				Seat:        i,
+				Name:        bot.ID,      // Stable bot ID for stats tracking
+				DisplayName: displayName, // Human-readable name for display
+				Chips:       player.Chips,
+				HoleCards: []string{
+					player.HoleCards.GetCard(0).String(),
+					player.HoleCards.GetCard(1).String(),
+				},
 			}
 		}
 		blinds := Blinds{

internal/server/monitor.go

@@ -25,9 +25,11 @@ type HandMonitor interface {
 
 // HandPlayer represents a player at the start of a hand.
 type HandPlayer struct {
-	Seat  int
-	Name  string
-	Chips int
+	Seat        int
+	Name        string // Bot ID for stable stats tracking
+	DisplayName string // Human-readable name for display
+	Chips       int
+	HoleCards   []string // Hole cards for debugging/testing (not sent to other players)
 }
 
 // Blinds represents the blind structure.

internal/server/pretty_monitor.go

@@ -21,19 +21,20 @@ const (
 
 // handState tracks the current hand for pretty printing
 type prettyHandState struct {
-	handID         string
-	players        []HandPlayer
-	button         int
-	smallBlind     int
-	bigBlind       int
-	board          []string
-	currentStreet  string
-	playerStacks   map[int]int
-	playerFolded   map[int]bool
-	playerAllIn    map[int]bool
-	roles          map[int][]string
-	printedStreets map[string]bool
-	printedHole    bool
+	handID          string
+	players         []HandPlayer
+	button          int
+	smallBlind      int
+	bigBlind        int
+	board           []string
+	currentStreet   string
+	playerStacks    map[int]int
+	playerFolded    map[int]bool
+	playerAllIn     map[int]bool
+	playerHoleCards map[int][]string // Hole cards for each seat
+	roles           map[int][]string
+	printedStreets  map[string]bool
+	printedHole     bool
 }
 
 // PrettyPrintMonitor implements HandMonitor for formatted hand display
@@ -82,24 +83,26 @@ func (p *PrettyPrintMonitor) OnHandStart(handID string, players []HandPlayer, bu
 
 	// Initialize hand state
 	p.currentHand = &prettyHandState{
-		handID:         handID,
-		players:        players,
-		button:         button,
-		smallBlind:     blinds.Small,
-		bigBlind:       blinds.Big,
-		board:          []string{},
-		currentStreet:  "preflop",
-		playerStacks:   make(map[int]int),
-		playerFolded:   make(map[int]bool),
-		playerAllIn:    make(map[int]bool),
-		roles:          make(map[int][]string),
-		printedStreets: make(map[string]bool),
-		printedHole:    false,
-	}
-
-	// Initialize player stacks
+		handID:          handID,
+		players:         players,
+		button:          button,
+		smallBlind:      blinds.Small,
+		bigBlind:        blinds.Big,
+		board:           []string{},
+		currentStreet:   "preflop",
+		playerStacks:    make(map[int]int),
+		playerFolded:    make(map[int]bool),
+		playerAllIn:     make(map[int]bool),
+		playerHoleCards: make(map[int][]string),
+		roles:           make(map[int][]string),
+		printedStreets:  make(map[string]bool),
+		printedHole:     false,
+	}
+
+	// Initialize player stacks and hole cards
 	for _, player := range players {
 		p.currentHand.playerStacks[player.Seat] = player.Chips
+		p.currentHand.playerHoleCards[player.Seat] = player.HoleCards
 	}
 
 	// Assign roles
@@ -112,7 +115,7 @@ func (p *PrettyPrintMonitor) OnHandStart(handID string, players []HandPlayer, bu
 	// Print players
 	for _, player := range players {
 		seatNum := player.Seat + 1
-		name := p.formatPlayerName(player.Seat, player.Name, false, false)
+		name := p.formatPlayerName(player.Seat, player.DisplayName, false, false)
 		line := fmt.Sprintf("Seat %d: %s", seatNum, name)
 		line += fmt.Sprintf(" (%s in chips)", formatAmountPlain(player.Chips))
 		fmt.Fprintln(p.writer, line)
@@ -150,13 +153,23 @@ func (p *PrettyPrintMonitor) OnPlayerAction(handID string, seat int, action stri
 		p.currentHand.printedHole = true
 		fmt.Fprintln(p.writer)
 		fmt.Fprintln(p.writer, colorize("*** HOLE CARDS ***", colorBold+colorBlue))
+
+		// Print hole cards for all players (for testing/debugging)
+		for _, player := range p.currentHand.players {
+			if cards, ok := p.currentHand.playerHoleCards[player.Seat]; ok && len(cards) > 0 {
+				name := fallbackName(player.DisplayName, player.Seat)
+				fmt.Fprintf(p.writer, "Dealt to %s %s\n",
+					colorize(name, colorBold),
+					formatCards(cards))
+			}
+		}
 	}
 
 	// Get player name
 	playerName := "Unknown"
 	for _, player := range p.currentHand.players {
 		if player.Seat == seat {
-			playerName = player.Name
+			playerName = player.DisplayName
 			break
 		}
 	}