Skip to content

feat(holdem): add a recap of the last five hands - #106

Merged
JonasFocus merged 1 commit into
mainfrom
claude/project-thread-w4ihw3
Oct 21, 2026
Merged

JonasFocus merged 1 commit into
mainfrom
claude/project-thread-w4ihw3

Conversation

@JonasFocus

Copy link
Copy Markdown
Owner

Requested by Jonas · project thread

Closes #58. Players can now look back at what someone had and how a pot was split after the next hand starts.

  • Keep the last 5 finished hands (history) in table state, recorded when the table clears after the showdown beat: board (both boards when run twice), pot paid out, and for each dealt-in player their cards, hand name, winnings and net.
  • Redact recap cards per viewer in redactTx: only hands shown down or turned face up after a fold win are public, and everyone still sees their own cards. Mucked and folded hands stay hidden.
  • Show the recap as "Last hands" under the standings in the Net sheet.
  • New tests/texasholdem.recap.test.ts covers the recap contents, redaction for every viewer, fold wins with and without Show, and the 5-hand cap.

Checked: lint, typecheck, test (419 pass) and build locally, plus a browser run against the local server with two bots (recap rendered with hidden bot hands and visible own hands).


Generated by Claude Code

- Keep the last 5 finished hands in table state: board (both boards when run twice), pot, and each dealt-in player's cards, hand, winnings and net
- Redact recap cards per viewer: only hands shown down or turned face up are public, everyone still sees their own
- Show the recap under the standings in the Net sheet
- Cover the recap contents, redaction, fold wins and the 5-hand cap in tests

Closes #58

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0166pZu7YE6Ei4yq58F3K1vk
@vercel

vercel Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
game-time Ready Ready Preview Oct 21, 2026 2:18pm UTC

@JonasFocus JonasFocus self-assigned this Oct 21, 2026
@JonasFocus
JonasFocus marked this pull request as ready for review October 21, 2026 14:20
@JonasFocus
JonasFocus merged commit 9f34936 into main Oct 21, 2026
4 checks passed

This branch was successfully deployed

1 active deployment
Preview — b1aea2bc Deployed Oct 21, 2026 by vercel[bot]
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.

Add a last-hand recap

2 participants