Video Poker: flexible bid controls - #237
Merged
Merged
Conversation
Up/Down press doubles or halves the bet, Up/Down hold bets half the bank; Left/Right keep their +/-$10 steps with wraparound. Doubling is overflow-safe and caps at all-in, so the old Up = all-in remains reachable. A clamp_bet helper enforces minbet <= bet <= bank everywhere, fixing the +$10 step overshooting the bank for bets that are no longer multiples of $10 and the bet re-entering the betting screen below the table minimum after a low-bank win. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
github-actions
Bot
added
pack/catalog
Repository owner
locked as resolved and limited conversation to collaborators
Jul 25, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #163
Implements the requested bid strategy controls in the betting phase:
All examples from the issue behave as specified (bank $1000 bet $10 -> Left -> $1000 -> Down -> $500; bank $3000 bet $500 -> Up -> $1000; bank $10000 -> long Down -> $5000 -> Down -> $2500).
Implementation notes:
InputTypeShort/InputTypeLong(mutually exclusive per press), so they cannot collide with each other or with theInputTypePress-based deal action of Down in the hold phase; the stray vibration on Up press in every screen is goneclamp_bethelper enforcesminbet <= bet <= bankon every mutation. This also fixes the +$10 step overshooting the bank for bets that halving makes non-multiples of $10, and a state where the betting screen could be re-entered with the bet below the table minimum after winning from a sub-minimum all-in (which would have made "halve" raise the bet)<^v*> Place Bet; CHANGELOG.md added with history backfilled from git;fap_version1.3 -> 1.4Builds clean under ufbt (API 88.0),
ufbt formatapplied.🤖 Generated with Claude Code