Skip to content

Conversation

@lox
Copy link
Owner

@lox lox commented Nov 14, 2025

Summary

  • Adds new list output mode to pokerforbots spawn
  • Shows one line per hand with hand ID, winner(s), and BB won/lost
  • Color-codes custom bot names (green for wins, bold green for custom bot wins)
  • Suppresses logs in list mode for cleaner output

Format

hand-1       aragorn-dev                    +15.5 BB
hand-2       complex-bot-3                  +8.0 BB
hand-3       aragorn-dev, random-bot-1      +12.3 BB

Test plan

  • Built successfully
  • Tested with built-in bots only
  • Verified color coding for winners
  • All tests passing

lox added 2 commits November 13, 2025 15:12
Adds a new --output mode "dots" that shows minimal progress feedback
during long test runs. Each hand is represented by a colored dot:
- Green (●) for custom bot wins
- Red (●) for custom bot losses
- Gray (●) for neutral hands (no custom bot or break-even)

Custom bots are detected by checking if their display name matches
built-in bot patterns (calling-bot-, random-bot-, aggressive-bot-,
complex-bot-). Any bot that doesn't match these patterns is considered
custom and will trigger colored dots.

Perfect for long test runs where you want to see progress without
overwhelming log output. Dots wrap at 80 characters per line.

Example:
  pokerforbots spawn \
    --spec "complex:5" \
    --bot-cmd "./my-bot" \
    --hand-limit 1000 \
    --output dots

Output:
  ●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●●
  ●●●●●●●●●●●●●●●●●●●●●●●●●●●●
  Completed 1000 hands (hand_limit_reached)
Add new 'list' output mode that shows one line per hand with:
- Hand ID
- Winner(s) with color-coded custom bot names (green for wins)
- BB won/lost with +/- sign and color

This provides a compact middle ground between dots (minimal) and
hand-history (verbose) modes for bot testing.
@lox lox merged commit c7263c8 into main Nov 14, 2025
4 checks passed
@lox lox deleted the feat/list-output-mode branch November 14, 2025 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants