Let the table tell the dealer who won

An option — off by default, and off means the tool is exactly what it was.

Turn it on and the community cards go on the felt. Each street asks for the cards as
it is dealt: the picker opens on the flop the moment the flop button is pressed, and
again on the turn and the river, because the moment a dealer is holding those cards is
the only moment they know what they are without turning the board round. Three taps,
then one, then one. The picker is the same sheet as the hole cards, which now knows
about the middle of the table too — the seven of clubs cannot be on the flop and in
somebody's hand, and until now it could.

Turning the players' cards over is unchanged. That already worked the way it should.

What is new is what happens next. Resolve Pot stops asking who won and says so:

    Alexandra wins $50 with three of a kind, nines.

with every hand at the table read out under its player's name, the winner already
ticked, and the button reading "Pay Alexandra". A chop comes out as "Christopher and
Mike from work split it, $22.50 each, with a pair of aces", both ticked. The dealer
still confirms — the tool reads the cards, a person still says the pot is good, and a
hand entered wrong is corrected by unticking rather than by starting over.

It declines rather than guesses. A board that isn't five cards, a player who hasn't
turned over, the same card entered in two places: each says which of those it is and
hands the question back, and the wizard is the one it always was. Who is *ahead* and
who *won* are different claims and only the second one can pay somebody.

Side pots are read per round, not once. The best hand takes the main pot and is then
out of the question; the side pot goes to the best of who is left, which is a different
answer and the one the tool has to ask for again.

handRank.js is the only module here that can pay the wrong person without anybody
noticing, so it is built to be checkable rather than clever: rank five cards, then take
the best of the twenty-one five-card hands in seven. Beside the written cases — every
category against every other, the wheel and the steel wheel, three kickers deep, the
counterfeited two pair, the board that plays — it is checked against a second evaluator
written a different way, on twenty thousand random seven-card hands and four thousand
random pairs of them. An ace that keeps its value in the wheel, a dropped two-pair
kicker and a flush read only by its top card were each introduced on purpose and each
caught.

The board takes the middle of the felt, sized from the room the seat ring actually
leaves — measured against every seat rather than the ring's widest point, which is only
the answer for a seat sitting exactly on the horizontal axis and had two pucks on top
of the board while claiming 77px of clearance either side. Where there is no middle to
take, it moves to a strip above the table: a phone on its side has a felt 528x83 with
55px pucks, so the ring closes over the centre entirely and a board in it is a board
with players painted on top.

That strip found an older bug. It costs the felt height, and a felt that changes size
had only ever been re-measured, never laid out again: the seats follow it on their own,
being placed in percentages, but a bet chip anchored inside a seat is placed in pixels
off that seat's centre. Measured at 640x360, the chips were positioned for a felt 83px
tall and painted on one of 60 — seat 3's $5 came to rest on seat 4's puck, and two
pairs of seat pucks were overlapping. The bar growing mid-hand does the same thing: a
side pot appearing is 55px off the felt. Both fixed.

The sweep grew a third table — the same five players in the same nine chairs as the
second, with the board on, so the only thing that differs between those two rows is the
board itself. It earned its place on the first run.

728 unit tests. 216 audits: what is left is the seat ring hanging past the rail on a
landscape phone, the known residual, plus five or six pixels more of it on the layouts
where the strip takes the felt's height, and a blind badge over a stack on a packed
twelve-seat portrait table.

Co-Authored-By: Claude Opus 5 <[email protected]>
Claude-Session: https://claude.ai/code/session_015bnp7xKUPFFSpwkYNFaPgb

JoshHeaps.Net/Pages/Casino/TexasHoldem.cshtml

@@ -7,28 +7,37 @@
 
 <div id="holdemApp">
     <header id="holdemHeader">
+        @* Six equally-weighted buttons used to sit here in two rows, and on a 390px
+           phone the felt did not start until 225px down — a quarter of the screen spent
+           before the dealer saw the table. Only two of the six are per-hand controls.
+           The rest have gone where they belong rather than into a "more" menu, which
+           would have relocated the crowding rather than removed it: Reset Table to the
+           Session panel beside Clear history and the save/load pair, Quick Split down
+           with Common rulings where the other "open something else" link already lives,
+           and the way back to the site up top where a back link goes. *@
         <div id="holdemHeaderLeft">
+            <a href="/casino" class="holdemBackLink">← Casino</a>
             <h1>Texas Hold'em Dealer</h1>
-            <p class="holdemSubtitle">Seat your players, track the action, split the pot.</p>
             <span id="handCounterBadge" class="handCounterBadge">Hand #1</span>
         </div>
         <div id="holdemHeaderActions">
-            <a href="/casino/texasholdem/quicksplit" class="holdemBtn" title="Chop a pot without setting up a table">Quick Split</a>
             <button id="misdealBtn" class="holdemBtn" title="Take this hand back — every chip returns to the player who put it in, and the same button deals again" hidden>Misdeal</button>
             <button id="nextHandBtn" class="holdemBtn" title="Move the button, small blind and big blind to the next occupied seat">Next Hand ⟳</button>
             <button id="resolvePotBtn" class="holdemBtn holdemBtnAccent">Resolve Pot</button>
-            <button id="resetTableBtn" class="holdemBtn holdemBtnDanger">Reset Table</button>
-            <a href="/casino" class="holdemBtn holdemBtnGhost">← Casino</a>
         </div>
     </header>
 
     <div id="holdemLayout">
         <section id="tableSection">
+            <div id="boardStrip" hidden></div>
             <div id="pokerTableWrap">
                 <div id="pokerTable">
-                    <div id="potBadge">
-                        <span id="potBadgeLabel">Pot</span>
-                        <span id="potBadgeAmount">$0</span>
+                    <div id="tableCentre">
+                        <div id="boardRow" hidden aria-label="Community cards"></div>
+                        <div id="potBadge">
+                            <span id="potBadgeLabel">Pot</span>
+                            <span id="potBadgeAmount">$0</span>
+                        </div>
                     </div>
                 </div>
                 <div id="betsLayer" aria-hidden="true"></div>
@@ -37,6 +46,7 @@
             <p id="tableHint" class="tableHint"></p>
             <p class="tableHintActions">
                 <button type="button" id="openRulingsBtn" class="linkBtn">Common rulings</button>
+                <a href="/casino/texasholdem/quicksplit" class="linkBtn" title="Chop a pot without setting up a table">Quick Split</a>
             </p>
         </section>
 
@@ -52,7 +62,11 @@
                 </div>
             </section>
 
-            <details class="sidebarPanel" open>
+            @* Not `open` in the markup any more. Whether this panel starts open is a
+               question about the table, so main.js answers it after the state has
+               loaded: open on a table nobody has set up yet, shut on one already
+               half-way through a night. *@
+            <details class="sidebarPanel" id="tableSetupPanel">
                 <summary>Table Setup</summary>
                 <div class="panelBody">
                     <label class="fieldRow">
@@ -71,6 +85,11 @@
                         <input type="checkbox" id="dedicatedDealerToggle" />
                     </label>
                     <p class="fieldHint">Turn this on if a person outside the game deals — no seat shows the dealer button, but blinds still rotate automatically each hand.</p>
+                    <label class="fieldRow">
+                        <span>Track the board and read the showdown</span>
+                        <input type="checkbox" id="trackBoardToggle" />
+                    </label>
+                    <p class="fieldHint">Puts the community cards on the felt and asks for them as each street is dealt. Turn the players&rsquo; cards over at showdown and Resolve Pot works out who won and how much, instead of asking you.</p>
                     <div class="blindsRow">
                         <label class="fieldRow">
                             <span>Small blind</span>
@@ -209,6 +228,11 @@
                         <button id="saveTableBtn" class="holdemBtn holdemBtnSmall">Save table file</button>
                         <button id="loadTableBtn" class="holdemBtn holdemBtnSmall">Load table file</button>
                         <input id="loadTableInput" type="file" accept="application/json,.json" hidden />
+                        @* Down from the header, where it sat in the same weight as Resolve Pot
+                           — the most-pressed control in a session next to the one that throws
+                           the night away. Here it is beside Clear history, which is the other
+                           thing that destroys records, and behind a panel you have to open. *@
+                        <button id="resetTableBtn" class="holdemBtn holdemBtnSmall holdemBtnDanger">Reset table</button>
                     </div>
                 </div>
             </details>
@@ -565,7 +589,17 @@
     </div>
 </div>
 
-<div id="actionBar" aria-live="polite" hidden></div>
+@* No aria-live here any more. This element's entire contents are rebuilt on every
+   action, so a live region on it announced the whole bar every time — measured, after
+   one tap: "PRE-FLOP Pot $15 9 players Bob · $250 behind 0:00 ↶ dealing the hand U
+   Fold Call $10 Raise". That is what a screen reader read out after every single fold.
+   A live region has to be a stable element whose *text* changes; the one below is. *@
+<div id="actionBar" hidden></div>
+
+@* Whose turn it is, in a sentence. Empty in the markup and only ever written to, so
+   assistive tech is already watching it when the first update lands — a live region
+   created at the moment it has something to say is one the screen reader misses. *@
+<div id="actionAnnouncer" role="status" aria-live="polite" class="visuallyHidden"></div>
 
 <div id="toastHost" aria-live="polite"></div>
 
@@ -587,13 +621,43 @@
             <button id="closePotWizardBtn" data-modal-close class="holdemBtn holdemBtnGhost">✕</button>
         </div>
         <div id="potWizardBody"></div>
+        @* The buttons live outside the scrolling body on purpose. This is the one
+           dialog that divides money, and its Confirm sat below a scroll at every screen
+           size — 244px off the bottom on an iPhone, 583 on a phone held sideways. A
+           sticky footer cannot fix that from inside: the actions were the last child of
+           the scrolling content, so their containing block ended exactly where they did
+           and `bottom` had no room to lift them. Out here they simply don't scroll. *@
+        <div id="potWizardFooter"></div>
     </div>
 </div>
 
 @section Styles {
     <link href="/css/casino/holdem.css?v=@ViewData["cssVersion"]" rel="stylesheet" type="text/css" />
+
+    @* Installable, and works with no signal — see the service worker registration at the
+       foot of main.js. A dealer running a night in somebody's back room is on the worst
+       network there is, and the tool being a *website* was the last thing that could
+       take it away from them mid-game. *@
+    <link rel="manifest" href="/casino/holdem.webmanifest" />
+    @* iOS reads none of the manifest's icons and only this. *@
+    <link rel="apple-touch-icon" href="/images/casino/holdem-apple-touch.png" />
+    <meta name="apple-mobile-web-app-capable" content="yes" />
+    <meta name="apple-mobile-web-app-status-bar-style" content="black-translucent" />
+    <meta name="apple-mobile-web-app-title" content="Hold'em" />
+    @* Paints the phone's chrome the same near-black as the felt's ground, so an
+       installed copy doesn't sit under a white status bar. *@
+    <meta name="theme-color" content="#141414" />
 }
 
+@* A note for whoever reads the ?v= below and assumes it works: it doesn't. ViewData
+   ["cssVersion"] is assigned in _Layout's own code block, which runs after these
+   sections are handed their values, so every one of these renders as a bare "?v=" —
+   on this page and on Chess, Particles and Memory Lane too. It is inert rather than
+   broken: Program.cs sends `Cache-Control: no-store` on every static file, so nothing
+   is being browser-cached for a version string to bust in the first place.
+   Left alone deliberately. Fixing it is a one-line change to a file every page on the
+   site shares, and the service worker was the reason to care — that now uses stale-
+   while-revalidate, which needs no version to stay current. *@
 @section Scripts {
     <script src="~/js/casino/holdem/money.js?v=@ViewData["cssVersion"]"></script>
     <script src="~/js/casino/holdem/feedback.js?v=@ViewData["cssVersion"]"></script>
@@ -601,6 +665,7 @@
     <script src="~/js/casino/holdem/state.js?v=@ViewData["cssVersion"]"></script>
     <script src="~/js/casino/holdem/settleUp.js?v=@ViewData["cssVersion"]"></script>
     <script src="~/js/casino/holdem/render.js?v=@ViewData["cssVersion"]"></script>
+    <script src="~/js/casino/holdem/handRank.js?v=@ViewData["cssVersion"]"></script>
     <script src="~/js/casino/holdem/cardPicker.js?v=@ViewData["cssVersion"]"></script>
     <script src="~/js/casino/holdem/seatMenu.js?v=@ViewData["cssVersion"]"></script>
     <script src="~/js/casino/holdem/addChips.js?v=@ViewData["cssVersion"]"></script>

JoshHeaps.Net/Program.cs

@@ -1,6 +1,7 @@
 using JoshHeaps.Net.Hubs;
 using JoshHeaps.Net.Services.Implementations;
 using JoshHeaps.Net.Services.Interfaces;
+using Microsoft.AspNetCore.StaticFiles;
 
 var builder = WebApplication.CreateBuilder(args);
 
@@ -52,8 +53,16 @@ if (!app.Environment.IsDevelopment())
 
 app.UseHttpsRedirection();
 
+// .webmanifest isn't in the default MIME map, and an unknown extension isn't served
+// with the wrong type — it isn't served at all. Without this the Hold'em tool's
+// manifest 404s, and a 404 manifest is exactly the silent failure that stops a browser
+// ever offering "Add to Home Screen": nothing is logged and no prompt appears.
+var contentTypes = new FileExtensionContentTypeProvider();
+contentTypes.Mappings[".webmanifest"] = "application/manifest+json";
+
 app.UseStaticFiles(new StaticFileOptions
 {
+    ContentTypeProvider = contentTypes,
     OnPrepareResponse = ctx =>
     {
         ctx.Context.Response.Headers.Append("Cache-Control", "no-cache, no-store, must-revalidate");

JoshHeaps.Net/wwwroot/casino/holdem.webmanifest

@@ -0,0 +1,32 @@
+{
+  "name": "Texas Hold'em Dealer",
+  "short_name": "Hold'em",
+  "description": "Seat your players, track the action, split the pot. Works with no signal.",
+  "start_url": "/casino/texasholdem",
+  "scope": "/casino/",
+  "display": "standalone",
+  "orientation": "any",
+  "background_color": "#141414",
+  "theme_color": "#141414",
+  "categories": ["utilities"],
+  "icons": [
+    {
+      "src": "/images/casino/holdem-192.png",
+      "sizes": "192x192",
+      "type": "image/png",
+      "purpose": "any"
+    },
+    {
+      "src": "/images/casino/holdem-512.png",
+      "sizes": "512x512",
+      "type": "image/png",
+      "purpose": "any"
+    },
+    {
+      "src": "/images/casino/holdem-maskable-512.png",
+      "sizes": "512x512",
+      "type": "image/png",
+      "purpose": "maskable"
+    }
+  ]
+}

JoshHeaps.Net/wwwroot/casino/sw.js

@@ -0,0 +1,125 @@
+// Keeps the dealer tool working when the network doesn't.
+//
+// A home game is the worst network in the world: somebody's back room, one bar of
+// signal, a router two floors away. The tool already keeps the whole night in
+// localStorage, so the *table* survives anything — but the page didn't. Reload with no
+// signal, or open it fresh when the wifi has dropped, and a dealer holding a phone in
+// one hand and a deck in the other got a browser error page with nine players waiting.
+//
+// This is the one thing standing between "a website about poker" and "a tool you can
+// rely on at a table", so it is worth the moving part.
+//
+// Scope is /casino/ rather than /casino/texasholdem/, because a scope has to be a
+// prefix of the page it controls and "/casino/texasholdem" does not start with
+// "/casino/texasholdem/". Anything in that scope this worker has not been told about
+// goes straight to the network, so the other casino pages behave exactly as they did.
+
+// Bumped whenever the shape of what is cached changes. The asset *version* rides in
+// the URLs themselves (?v=1.0.8 from _Layout), so a normal release makes new keys
+// without touching this.
+const CACHE = "holdem-shell-v1";
+
+// What this worker will keep. Anything else in scope — a form post, an API call, a
+// page it was never told about — goes straight to the network and behaves exactly as
+// it did before the worker existed.
+const CACHEABLE = new Set(["script", "style", "image", "font", "manifest"]);
+
+// The page hands over the exact list of URLs it was built from — see registerWorker()
+// in main.js. Deriving it from the document instead of keeping a list here is the
+// whole trick: a hand-written manifest of files drifts from the markup the first time
+// somebody adds a script tag, and the failure mode is a tool that is offline-capable
+// right up until the release that quietly broke it.
+async function precache(urls) {
+    const cache = await caches.open(CACHE);
+    // Individually rather than cache.addAll, which rejects the whole batch if any one
+    // URL 404s — one stale entry in the list would leave the dealer with nothing.
+    await Promise.all(urls.map(async url => {
+        try {
+            const res = await fetch(url, { cache: "reload" });
+            if (res.ok) await cache.put(url, res);
+        } catch {
+            // Offline while installing. The runtime handler below picks these up on
+            // the next online visit.
+        }
+    }));
+}
+
+// No install handler, and deliberately no skipWaiting(). A new worker taking over a
+// live page can hand it assets from a different release than the ones it started with,
+// and this page loads seventeen separate scripts that expect to agree with each other.
+// Waiting costs a dealer nothing — the update lands the next time they open the tool —
+// and it cannot leave them with a half-swapped app in the middle of a hand.
+
+self.addEventListener("activate", event => {
+    event.waitUntil((async () => {
+        const keys = await caches.keys();
+        await Promise.all(keys
+            .filter(key => key.startsWith("holdem-shell-") && key !== CACHE)
+            .map(key => caches.delete(key)));
+        await self.clients.claim();
+    })());
+});
+
+self.addEventListener("message", event => {
+    if (event.data && event.data.type === "precache" && Array.isArray(event.data.urls)) {
+        event.waitUntil(precache(event.data.urls));
+    }
+});
+
+// Only GETs, and only same-origin. A cross-origin request has opaque responses that
+// can't be inspected, and caching one would mean serving a failure forever.
+function handles(request) {
+    if (request.method !== "GET") return false;
+    const url = new URL(request.url);
+    return url.origin === self.location.origin;
+}
+
+self.addEventListener("fetch", event => {
+    const { request } = event;
+    if (!handles(request)) return;
+
+    // The page itself: network first, so an online dealer always gets the current
+    // build, and the cached copy is there the moment they aren't.
+    if (request.mode === "navigate") {
+        event.respondWith((async () => {
+            try {
+                const fresh = await fetch(request);
+                if (fresh.ok) (await caches.open(CACHE)).put(request, fresh.clone());
+                return fresh;
+            } catch {
+                const cached = await caches.match(request, { ignoreSearch: true });
+                if (cached) return cached;
+                throw new Error("offline and this page was never cached");
+            }
+        })());
+        return;
+    }
+
+    // Assets: stale while revalidate. The cached copy answers immediately — which is
+    // the whole point, a dealer opening this between hands should not wait for a
+    // network — and a fresh copy is fetched in the background for next time.
+    //
+    // Cache-first would be the obvious choice, on the reasoning that these URLs carry
+    // ?v= from the layout and so a hit is always the right file. They don't: the
+    // version renders empty (see the note in TexasHoldem.cshtml), so every release
+    // reuses the same URLs. Cache-first against unversioned URLs is how a tool gets
+    // frozen on a build from three months ago with no way for anyone to tell. This
+    // strategy needs no version to be correct: worst case a dealer is one launch
+    // behind, which for a tool that runs entirely in the browser is nothing.
+    if (!CACHEABLE.has(request.destination)) return;
+
+    event.respondWith((async () => {
+        const cache = await caches.open(CACHE);
+        const cached = await cache.match(request);
+
+        const fresh = fetch(request).then(res => {
+            if (res.ok) cache.put(request, res.clone());
+            return res;
+        }).catch(() => null);
+
+        if (cached) return cached;
+        const res = await fresh;
+        if (res) return res;
+        throw new Error("offline and not cached: " + request.url);
+    })());
+});

JoshHeaps.Net/wwwroot/js/casino/holdem/actionBar.js

@@ -37,6 +37,10 @@ const ActionBar = {
             this.publishFeltRoom(height);
         };
 
+        // Kept so anything that moves the table without touching the bar can ask for
+        // the felt to be measured again — the board strip appearing above it does
+        // exactly that, and the felt's height is worked out from where the table starts.
+        this.publishHeight = publish;
         if (typeof ResizeObserver === "function") {
             this.heightWatcher = new ResizeObserver(publish);
             this.heightWatcher.observe(host);
@@ -86,13 +90,27 @@ const ActionBar = {
         // header in normal flow, so its own height never moves its top and this can't
         // chase itself around the screen.
         const topInDocument = wrap.getBoundingClientRect().top + window.scrollY;
-        document.documentElement.style.setProperty(
-            "--feltRoomPx",
-            `${this.feltRoomFor(window.innerHeight, barHeight, topInDocument)}px`);
+        const root = document.documentElement;
+        const was = root.style.getPropertyValue("--feltRoomPx");
+        const now = `${this.feltRoomFor(window.innerHeight, barHeight, topInDocument)}px`;
+        root.style.setProperty("--feltRoomPx", now);
         // The felt has just changed size, so how much rim each seat gets has changed
         // with it. Told rather than left to be noticed on the next render, which on a
         // bar that grew mid-hand could be several actions away.
-        if (typeof Render !== "undefined" && Render.updateSeatDensity) Render.updateSeatDensity();
+        if (typeof Render === "undefined" || !Render.state) return;
+        if (Render.updateSeatDensity) Render.updateSeatDensity();
+
+        // And when it really did change size, the seats are laid out again rather than
+        // only re-measured.
+        //
+        // The seats follow a felt on their own — they are placed in percentages of it —
+        // but a bet chip anchored inside a seat is placed in pixels off that seat's
+        // centre, worked out against the felt as it was. Nothing re-ran that. Measured
+        // at 640x360 with five players: the chips were positioned for a felt 83px tall,
+        // painted on one of 60, and seat 3's $5 came to rest on seat 4's puck. The bar
+        // growing mid-hand does this too — a side pot appearing is 55px off the felt —
+        // so it is not only the board strip that could reach it.
+        if (now !== was && Render.renderSeats) Render.renderSeats();
     },
 
     // Anything to act on? A hand with nobody dealt in doesn't get a bar — and that
@@ -113,12 +131,19 @@ const ActionBar = {
         if (!host) return;
         host.innerHTML = "";
 
+        // A table with nobody at it used to hide this bar altogether, which meant the
+        // one moment a dealer has never used the tool before is the one moment it says
+        // nothing at all. What they got instead was a felt printing EMPTY ten times and
+        // a green Resolve Pot button that would resolve nothing.
+        //
+        // An empty table is a state with an obvious next step, so it says what it is.
         if (!this.isRunning()) {
-            host.hidden = true;
-            document.body.classList.remove("hasActionBar");
-            document.documentElement.style.setProperty("--actionBarHeight", "0px");
+            host.hidden = false;
+            document.body.classList.add("hasActionBar");
+            host.appendChild(this.renderEmptyTable());
             this.stopTankTimer();
             this.tankSeatIndex = null;
+            this.announce();
             return;
         }
         host.hidden = false;
@@ -142,6 +167,79 @@ const ActionBar = {
         }
 
         this.syncTankTimer();
+        this.announce();
+    },
+
+    // The empty table. One sentence and one button, in the place every other prompt in
+    // this tool appears, so a dealer who has never opened it before is looking at the
+    // next step rather than at ten chairs labelled EMPTY.
+    renderEmptyTable() {
+        const wrap = document.createElement("div");
+        wrap.className = "dealPrompt";
+
+        const line = document.createElement("p");
+        line.className = "dealPromptLine";
+        line.textContent = "Nobody's at the table yet.";
+        wrap.appendChild(line);
+
+        const buttons = document.createElement("div");
+        buttons.className = "dealPromptButtons";
+
+        const seat = document.createElement("button");
+        seat.type = "button";
+        seat.className = "actionBtn actionDeal";
+        seat.textContent = "Seat a player";
+        seat.addEventListener("click", () => this.hooks.seatFirstPlayer());
+        buttons.appendChild(seat);
+
+        wrap.appendChild(buttons);
+        return wrap;
+    },
+
+    // One sentence, for anyone running this by ear.
+    //
+    // The bar used to carry aria-live itself, and because render() rebuilds the whole
+    // thing every action, a screen reader read the lot out after every tap — street,
+    // pot, player count, who's short, the clock, the undo label and all three buttons.
+    // Nothing in that is wrong; it is just unusable at the speed a hand is dealt.
+    //
+    // What a dealer actually needs from the tool between taps is who is on the clock
+    // and what it costs them. That is what this says, into a small element that exists
+    // from page load and only ever has its text replaced.
+    //
+    // Guarded on the sentence itself rather than on the state, because render() is
+    // called for a great many reasons that don't change whose turn it is — resizing the
+    // window, opening a panel, editing a note — and a live region that repeats itself
+    // is read out again every time.
+    announce() {
+        const target = document.getElementById("actionAnnouncer");
+        if (!target) return;
+
+        const said = this.announcement();
+        if (said === this.lastAnnouncement) return;
+        this.lastAnnouncement = said;
+        target.textContent = said;
+    },
+
+    announcement() {
+        const state = this.state;
+        if (!this.isRunning()) return "Nobody's at the table yet.";
+        if (!state.bettingOpen) {
+            const preview = Betting.blindPreview(state);
+            return preview ? "Ready to deal." : "";
+        }
+        if (Betting.isHandOver(state)) return "Hand over. Resolve the pot.";
+
+        const index = state.actingSeatIndex;
+        if (index === null || index === undefined) {
+            return `${Betting.streetLabel(state.street)} betting is done.`;
+        }
+
+        const player = state.seats[index];
+        const name = HoldemState.displayName(player, index);
+        const owed = Betting.toCall(state, player);
+        const cost = owed > 0 ? `${Money.format(owed)} to call` : "checked to them";
+        return `${name} to act, ${cost}. Pot ${Money.format(Betting.potSoFar(state))}.`;
     },
 
     renderStatus() {
@@ -231,7 +329,17 @@ const ActionBar = {
             undo.type = "button";
             undo.className = "actionUndo";
             undo.title = `Undo ${undoLabel}`;
-            undo.textContent = `↶ ${undoLabel}`;
+            // The label goes in its own element rather than straight onto the button.
+            // The pill is a fixed width — it names the last action, so its text changes
+            // on every tap, and a control that resizes under the dealer's thumb reads
+            // as the interface twitching. Fixed width means long labels have to
+            // ellipsize, and only a real element can: as a bare text node the label
+            // truncated *through* the keycap below it, which then sat squashed against
+            // the cut-off text looking like a rendering fault.
+            const label = document.createElement("span");
+            label.className = "actionUndoLabel";
+            label.textContent = `↶ ${undoLabel}`;
+            undo.appendChild(label);
             // U undoes, and nothing said so — this control was the only place a dealer
             // could have learned it, and it was the one action bar control without its
             // key printed on it.
@@ -521,6 +629,19 @@ const ActionBar = {
     // recorded so it can be taken back. A misclick on Fold is the one mistake in this
     // tool a dealer can't talk their way out of.
     commit(label, mutate) {
+        // A toast is news about the last thing that happened, and this is the next
+        // thing: whatever it says is now out of date. Two reasons it goes rather than
+        // waiting out its eight seconds. It lies — "Next hand — button on seat 2.
+        // [Undo]" reads as an offer to move the button back, but the snapshot behind
+        // that Undo predates the hand you have since dealt, so taking it would throw
+        // the hand away without saying so. And it is in the way: measured across the
+        // sweep, a toast left up over a dealt table covered seats, the clock button
+        // or the bar's own Undo on thirty of the small screens it was checked at,
+        // because there is nowhere on a 360x640 phone for a 560px strip to go. The
+        // bar's own Undo control takes over from here, and it names the step it
+        // would actually take back.
+        if (typeof Toast !== "undefined" && Toast.dismiss) Toast.dismiss();
+
         this.hooks.beginAction(label);
         mutate();
         this.hooks.save(this.state);
@@ -1027,6 +1148,10 @@ const ActionBar = {
                 return;
             }
             this.commit(`dealing the ${Betting.streetLabel(Betting.STREETS[Betting.STREETS.indexOf(this.state.street) + 1]).toLowerCase()}`, () => Betting.closeStreet(this.state));
+            // The cards that just came off the deck. Asked for here rather than left
+            // for the dealer to remember, because the moment they are holding them is
+            // the only moment they know what they are without turning the board round.
+            if (this.state.trackBoard && typeof Render !== "undefined") Render.openBoardPicker();
         }));
         wrap.appendChild(row);
 

JoshHeaps.Net/wwwroot/js/casino/holdem/betting.js

@@ -13,6 +13,15 @@
 const Betting = {
     STREETS: ["preflop", "flop", "turn", "river"],
 
+    // How many community cards are face up once a street has been dealt. The flop is
+    // three at once and everything after it is one at a time, which is why this is a
+    // table rather than an index.
+    BOARD_CARDS: { preflop: 0, flop: 3, turn: 4, river: 5 },
+
+    boardCardsFor(street) {
+        return this.BOARD_CARDS[street] ?? 0;
+    },
+
     streetLabel(street) {
         return { preflop: "Pre-flop", flop: "Flop", turn: "Turn", river: "River" }[street] || "Pre-flop";
     },
@@ -557,6 +566,7 @@ const Betting = {
         state.actingSeatIndex = null;
         state.streetAggressorIndex = null;
         state.handAggressorIndex = null;
+        HoldemState.clearBoard(state);
         // Nothing was posted, as far as the next rotation is concerned: the hand this
         // would have counted as is being dealt again from the same button.
         state.postedBlinds = null;
@@ -593,6 +603,10 @@ const Betting = {
         state.street = "preflop";
         state.currentBet = 0;
         state.lastRaiseSize = 0;
+        // A fresh deal starts on an empty felt. Dealing without moving the button —
+        // after a misdeal, or a table set up and dealt twice — would otherwise leave
+        // the last hand's board sitting there being ranked against the new cards.
+        HoldemState.clearBoard(state);
         // Posting a blind isn't aggression — a hand limped and checked all the way
         // down has no aggressor at all, which is exactly the case the showdown rule
         // sends to the seat left of the button.

JoshHeaps.Net/wwwroot/js/casino/holdem/cardPicker.js

@@ -16,45 +16,97 @@ const CardPicker = {
 
     state: null,
     hooks: null,
+    // Which set of slots is being filled: a seat's two hole cards, or the five
+    // community cards. The grid, the duplicate check and the tap-to-fill behaviour are
+    // the same job either way — what changes is where the card is written and how many
+    // slots there are to walk along.
+    target: null,
     seatIndex: null,
     cardIndex: 0,
 
     open(state, hooks, seatIndex, cardIndex) {
         this.state = state;
         this.hooks = hooks;
+        this.target = "seat";
         this.seatIndex = seatIndex;
         this.cardIndex = cardIndex;
         this.render();
         document.getElementById("cardPickerOverlay").hidden = false;
     },
 
+    // The board. `limit` is how many of the five are actually out — three on the flop,
+    // four on the turn — so the sheet never offers a slot for a card the dealer has not
+    // turned over yet.
+    openBoard(state, hooks, cardIndex, limit = 5) {
+        this.state = state;
+        this.hooks = hooks;
+        this.target = "board";
+        this.seatIndex = null;
+        this.boardLimit = Math.max(1, Math.min(5, limit));
+        this.cardIndex = Math.min(cardIndex, this.boardLimit - 1);
+        this.render();
+        document.getElementById("cardPickerOverlay").hidden = false;
+    },
+
+    onBoard() {
+        return this.target === "board";
+    },
+
+    // The slots the sheet is filling, whichever set that is.
+    slotValues() {
+        return this.onBoard()
+            ? (this.state.board || []).slice(0, this.boardLimit)
+            : this.state.seats[this.seatIndex].cards;
+    },
+
+    write(index, card) {
+        if (this.onBoard()) {
+            if (!Array.isArray(this.state.board)) this.state.board = ["", "", "", "", ""];
+            this.state.board[index] = card;
+            return;
+        }
+        this.state.seats[this.seatIndex].cards[index] = card;
+    },
+
     close() {
         document.getElementById("cardPickerOverlay").hidden = true;
     },
 
-    // Every card sitting in front of anyone else. The slot being edited is excluded,
-    // so re-picking the card that's already there isn't treated as a duplicate.
+    // Every card already dealt somewhere — in front of a player or face up in the
+    // middle. The slot being edited is excluded, so re-picking the card that's already
+    // there isn't treated as a duplicate.
+    //
+    // The board belongs in here as much as the hands do, and in both directions: the
+    // seven of clubs cannot be on the flop and in somebody's hand, whichever of the two
+    // the dealer is filling in at the time.
     cardsInUse() {
         const used = new Map();
         this.state.seats.forEach((player, seat) => {
             (player.cards || []).forEach((card, slot) => {
                 if (!card) return;
-                if (seat === this.seatIndex && slot === this.cardIndex) return;
+                if (!this.onBoard() && seat === this.seatIndex && slot === this.cardIndex) return;
                 used.set(card.toUpperCase(), HoldemState.displayName(player, seat));
             });
         });
+        (this.state.board || []).forEach((card, slot) => {
+            if (!card) return;
+            if (this.onBoard() && slot === this.cardIndex) return;
+            used.set(card.toUpperCase(), "the board");
+        });
         return used;
     },
 
     assign(card) {
-        const player = this.state.seats[this.seatIndex];
-        player.cards[this.cardIndex] = card;
+        this.write(this.cardIndex, card);
         this.hooks.onPick(this.seatIndex, this.cardIndex, card);
 
-        // Straight on to the second card — a dealer picking hole cards is always
-        // picking two, and making them re-open the picker doubles the taps.
-        if (this.cardIndex === 0 && !player.cards[1]) {
-            this.cardIndex = 1;
+        // Straight on to the next empty slot — a dealer picking hole cards is always
+        // picking two and a flop is always three, and making them re-open the picker
+        // between each one multiplies the taps by the only thing the sheet is for.
+        const values = this.slotValues();
+        const next = values.findIndex((value, i) => i > this.cardIndex && !value);
+        if (next !== -1) {
+            this.cardIndex = next;
             this.render();
             return;
         }
@@ -62,29 +114,37 @@ const CardPicker = {
     },
 
     clearSlot() {
-        const player = this.state.seats[this.seatIndex];
-        player.cards[this.cardIndex] = "";
+        this.write(this.cardIndex, "");
         this.hooks.onPick(this.seatIndex, this.cardIndex, "");
         this.render();
     },
 
-    render() {
-        const player = this.state.seats[this.seatIndex];
-        const used = this.cardsInUse();
+    // What this slot is called out loud: the flop's three cards, then the turn, then
+    // the river — which is what a dealer is holding when they open this, rather than
+    // "card 4".
+    boardSlotLabel(slot) {
+        return ["Flop 1", "Flop 2", "Flop 3", "Turn", "River"][slot] || `Card ${slot + 1}`;
+    },
 
-        document.getElementById("cardPickerTitle").textContent =
-            `${HoldemState.displayName(player, this.seatIndex)} — card ${this.cardIndex + 1}`;
+    render() {
+        const used = this.cardsInUse();
+        const values = this.slotValues();
+
+        document.getElementById("cardPickerTitle").textContent = this.onBoard()
+            ? `Board — ${this.boardSlotLabel(this.cardIndex).toLowerCase()}`
+            : `${HoldemState.displayName(this.state.seats[this.seatIndex], this.seatIndex)} — card ${this.cardIndex + 1}`;
 
         // Which slot the next tap fills, switchable without closing the sheet.
         const slots = document.getElementById("cardPickerSlots");
         slots.innerHTML = "";
-        [0, 1].forEach(slot => {
+        values.forEach((_, slot) => {
             const btn = document.createElement("button");
             btn.type = "button";
             btn.className = "pickerSlot";
             btn.classList.toggle("active", slot === this.cardIndex);
-            const value = player.cards[slot] || "";
+            const value = values[slot] || "";
             btn.textContent = value || "—";
+            if (this.onBoard()) btn.title = this.boardSlotLabel(slot);
             btn.classList.toggle("cardRed", /[HD]$/i.test(value));
             btn.addEventListener("click", () => {
                 this.cardIndex = slot;
@@ -105,8 +165,12 @@ const CardPicker = {
                 btn.className = "pickerCard";
                 btn.classList.toggle("cardRed", suit.red);
                 btn.disabled = !!takenBy;
-                if (takenBy) btn.title = `Already dealt to ${takenBy}`;
-                if ((player.cards[this.cardIndex] || "").toUpperCase() === card) {
+                if (takenBy) {
+                    btn.title = takenBy === "the board"
+                        ? "Already face up on the board"
+                        : `Already dealt to ${takenBy}`;
+                }
+                if ((values[this.cardIndex] || "").toUpperCase() === card) {
                     btn.classList.add("selected");
                 }
 

JoshHeaps.Net/wwwroot/js/casino/holdem/handRank.js

@@ -0,0 +1,285 @@
+// What beats what.
+//
+// The dealer has always been the one who decides who won: the pot wizard asks "who won
+// the hand?" and pays whoever gets ticked. That is the right default — most hands are
+// won without a showdown, and at a home game the table reads the cards out loud between
+// them. But when the board is on the felt and the hands are on their backs, the tool
+// knows the answer, and reading it off a screen beats three people counting kickers.
+//
+// Cards are the same two-character strings the rest of the tool has always stored:
+// a rank from `RANKS` and a suit from S, H, D, C. Case is not significant.
+//
+// Nothing in here touches the DOM, the table, or money. It compares cards, and the pot
+// wizard decides what to do about it.
+const HandRank = {
+    RANKS: "23456789TJQKA",
+
+    // High to low. The number is the hand's category and nothing else — kickers and
+    // the rest of the tie-breaking live in the `tiebreak` array beside it.
+    STRAIGHT_FLUSH: 8,
+    FOUR_OF_A_KIND: 7,
+    FULL_HOUSE: 6,
+    FLUSH: 5,
+    STRAIGHT: 4,
+    THREE_OF_A_KIND: 3,
+    TWO_PAIR: 2,
+    ONE_PAIR: 1,
+    HIGH_CARD: 0,
+
+    // 2 through 14. Null for anything that isn't a card, which is how a half-filled
+    // slot gets told apart from a bad one.
+    rankValue(card) {
+        if (typeof card !== "string" || card.length < 2) return null;
+        const index = this.RANKS.indexOf(card[0].toUpperCase());
+        return index < 0 ? null : index + 2;
+    },
+
+    suitOf(card) {
+        if (typeof card !== "string" || card.length < 2) return null;
+        const suit = card[card.length - 1].toUpperCase();
+        return "SHDC".includes(suit) ? suit : null;
+    },
+
+    isCard(card) {
+        return this.rankValue(card) !== null && this.suitOf(card) !== null;
+    },
+
+    // Singular and plural, for reading a hand out loud.
+    NAMES: {
+        14: ["ace", "aces"], 13: ["king", "kings"], 12: ["queen", "queens"],
+        11: ["jack", "jacks"], 10: ["ten", "tens"], 9: ["nine", "nines"],
+        8: ["eight", "eights"], 7: ["seven", "sevens"], 6: ["six", "sixes"],
+        5: ["five", "fives"], 4: ["four", "fours"], 3: ["three", "threes"],
+        2: ["two", "twos"],
+    },
+
+    name(rank, plural = false) {
+        const pair = this.NAMES[rank];
+        return pair ? pair[plural ? 1 : 0] : "?";
+    },
+
+    // ---- The five-card hand -------------------------------------------------------
+
+    // Exactly five cards, ranked. Everything else is built on this.
+    //
+    // Five rather than seven because five is the hand a player actually holds, and it
+    // is small enough to check by hand — which is the whole argument for doing it this
+    // way. The seven-card version below tries all twenty-one five-card hands and keeps
+    // the best, which is a hand of poker's own definition of "your best five" written
+    // out literally. A dealer resolves one pot a hand with at most a dozen players in
+    // it; two hundred and fifty comparisons of five cards is not a cost worth trading
+    // a subtler algorithm's correctness for.
+    evaluate5(cards) {
+        if (!Array.isArray(cards) || cards.length !== 5) return null;
+        if (!cards.every(card => this.isCard(card))) return null;
+
+        const ranks = cards.map(card => this.rankValue(card)).sort((a, b) => b - a);
+        const suits = cards.map(card => this.suitOf(card));
+        const flush = suits.every(suit => suit === suits[0]);
+
+        // How many of each rank, ordered by how many first and by rank second — so
+        // the front of this list is always the rank that decides the hand.
+        const counts = new Map();
+        ranks.forEach(rank => counts.set(rank, (counts.get(rank) || 0) + 1));
+        const groups = [...counts.entries()]
+            .sort((a, b) => (b[1] - a[1]) || (b[0] - a[0]));
+        const shape = groups.map(([, count]) => count);
+        const byGroup = groups.map(([rank]) => rank);
+
+        const straightHigh = this.straightHigh(ranks);
+
+        const made = (category, tiebreak) => ({ category, tiebreak, cards: [...cards] });
+
+        if (flush && straightHigh) return made(this.STRAIGHT_FLUSH, [straightHigh]);
+        if (shape[0] === 4) return made(this.FOUR_OF_A_KIND, byGroup);
+        if (shape[0] === 3 && shape[1] === 2) return made(this.FULL_HOUSE, byGroup);
+        if (flush) return made(this.FLUSH, ranks);
+        if (straightHigh) return made(this.STRAIGHT, [straightHigh]);
+        if (shape[0] === 3) return made(this.THREE_OF_A_KIND, byGroup);
+        if (shape[0] === 2 && shape[1] === 2) return made(this.TWO_PAIR, byGroup);
+        if (shape[0] === 2) return made(this.ONE_PAIR, byGroup);
+        return made(this.HIGH_CARD, ranks);
+    },
+
+    // The top card of the straight these five ranks make, or null.
+    //
+    // The wheel is the exception the rest of the ordering has to be told about: A-2-3-4-5
+    // is a straight, and it is the *lowest* one, so its ace counts as five rather than
+    // as fourteen. Getting that wrong is how A-2-3-4-5 comes to beat 10-J-Q-K-A.
+    straightHigh(sortedDesc) {
+        const distinct = [...new Set(sortedDesc)];
+        if (distinct.length !== 5) return null;
+        if (distinct[0] - distinct[4] === 4) return distinct[0];
+        // Ace, five, four, three, two.
+        if (distinct[0] === 14 && distinct[1] === 5 && distinct[4] === 2) return 5;
+        return null;
+    },
+
+    // ---- The best five out of however many ----------------------------------------
+
+    // The best five-card hand available from five, six or seven cards — hole cards and
+    // board together, which is what Hold'em asks for. Returns null if there aren't five
+    // real cards, which is the ordinary state of a hand still being dealt.
+    best(cards) {
+        const real = (cards || []).filter(card => this.isCard(card));
+        if (real.length < 5) return null;
+        // A card in two places at once is a mis-entry, not a hand — say so rather than
+        // ranking a board that has the same king on it twice.
+        if (new Set(real.map(card => card.toUpperCase())).size !== real.length) return null;
+
+        let best = null;
+        this.eachFive(real, five => {
+            const hand = this.evaluate5(five);
+            if (!best || this.compare(hand, best) > 0) best = hand;
+        });
+        return best;
+    },
+
+    // Every five-card subset, without building an array of them.
+    eachFive(cards, visit) {
+        const n = cards.length;
+        for (let a = 0; a < n - 4; a++) {
+            for (let b = a + 1; b < n - 3; b++) {
+                for (let c = b + 1; c < n - 2; c++) {
+                    for (let d = c + 1; d < n - 1; d++) {
+                        for (let e = d + 1; e < n; e++) {
+                            visit([cards[a], cards[b], cards[c], cards[d], cards[e]]);
+                        }
+                    }
+                }
+            }
+        }
+    },
+
+    // Positive if `a` wins, negative if `b` does, zero for a chop. A chop is a real
+    // answer here rather than a rounding of one: two players playing the same board
+    // split the pot, and the fifth card decides it far more often than a table expects.
+    compare(a, b) {
+        if (!a && !b) return 0;
+        if (!a) return -1;
+        if (!b) return 1;
+        if (a.category !== b.category) return a.category - b.category;
+        const length = Math.max(a.tiebreak.length, b.tiebreak.length);
+        for (let i = 0; i < length; i++) {
+            const diff = (a.tiebreak[i] || 0) - (b.tiebreak[i] || 0);
+            if (diff !== 0) return diff;
+        }
+        return 0;
+    },
+
+    // ---- Saying it out loud -------------------------------------------------------
+
+    // What a dealer says when they turn the hand over. Named to the level that settles
+    // it and no further: "two pair, aces and eights" is the call, and the kicker only
+    // gets a mention where the tool has had to use one to break a tie.
+    describe(hand) {
+        if (!hand) return "";
+        const [first, second] = hand.tiebreak;
+        switch (hand.category) {
+            case this.STRAIGHT_FLUSH:
+                return first === 14 ? "Royal flush" : `Straight flush, ${this.name(first)} high`;
+            case this.FOUR_OF_A_KIND:
+                return `Four of a kind, ${this.name(first, true)}`;
+            case this.FULL_HOUSE:
+                return `Full house, ${this.name(first, true)} full of ${this.name(second, true)}`;
+            case this.FLUSH:
+                return `Flush, ${this.name(first)} high`;
+            case this.STRAIGHT:
+                return `Straight, ${this.name(first)} high`;
+            case this.THREE_OF_A_KIND:
+                return `Three of a kind, ${this.name(first, true)}`;
+            case this.TWO_PAIR:
+                return `Two pair, ${this.name(first, true)} and ${this.name(second, true)}`;
+            case this.ONE_PAIR:
+                return `Pair of ${this.name(first, true)}`;
+            default:
+                return `${this.name(first).charAt(0).toUpperCase()}${this.name(first).slice(1)} high`;
+        }
+    },
+
+    // The same call with the article a table would put in front of it. Not derivable
+    // from the first letter: it is "a full house" and "a flush", but "four of a kind"
+    // and "two pair" take none at all, and "an ace high" is not something anybody says.
+    // So it belongs to the category, like the name does.
+    ARTICLED: {
+        8: hand => (hand.tiebreak[0] === 14 ? "a royal flush" : "a"),
+        7: () => null,
+        6: () => "a",
+        5: () => "a",
+        4: () => "a",
+        3: () => null,
+        2: () => null,
+        1: () => "a",
+        0: () => null,
+    },
+
+    // "wins with X" — the phrase, ready to drop into a sentence.
+    phrase(hand) {
+        if (!hand) return "the best hand";
+        const described = this.describe(hand);
+        const article = (this.ARTICLED[hand.category] || (() => null))(hand);
+        if (article === null) return described.toLowerCase();
+        if (article !== "a") return article;
+        return `a ${described.toLowerCase()}`;
+    },
+
+    // ---- The showdown -------------------------------------------------------------
+
+    // Rank a set of players against a board.
+    //
+    // `players` is [{ id, cards }, ...] — the two in front of each of them. Every player
+    // is returned whether or not they could be ranked, in the order they were given, so
+    // the caller can show a row for each and say which ones it could not read. `place`
+    // is 1 for the winner and shared by a chop; a player whose cards aren't known has a
+    // null hand and no place.
+    //
+    // `complete` is the question the pot wizard actually asks: can this be trusted to
+    // pay the table. It is only true when the board is all five cards and every single
+    // player handed in has two of their own — one unknown hand and the tool cannot say
+    // who won, only who is winning so far, which is not the same thing and must not be
+    // dressed up as it.
+    showdown(board, players) {
+        const boardCards = (board || []).filter(card => this.isCard(card));
+        const boardComplete = boardCards.length === 5;
+
+        const ranked = (players || []).map(player => {
+            const hole = (player.cards || []).filter(card => this.isCard(card));
+            const hand = hole.length === 2 && boardComplete
+                ? this.best([...hole, ...boardCards])
+                : null;
+            return { id: player.id, cards: hole, hand, place: null };
+        });
+
+        const readable = ranked.filter(row => row.hand);
+        // A duplicated card anywhere means somebody has been mis-entered, and the
+        // ranking would be of a deck that doesn't exist.
+        const all = [...boardCards, ...ranked.flatMap(row => row.cards)].map(c => c.toUpperCase());
+        const duplicated = new Set(all).size !== all.length;
+
+        const sorted = [...readable].sort((a, b) => this.compare(b.hand, a.hand));
+        let place = 0;
+        sorted.forEach((row, i) => {
+            if (i === 0 || this.compare(row.hand, sorted[i - 1].hand) !== 0) place = i + 1;
+            row.place = place;
+        });
+
+        const winners = sorted.filter(row => row.place === 1);
+        const complete = boardComplete && !duplicated &&
+            ranked.length > 0 && readable.length === ranked.length;
+
+        return {
+            players: ranked,
+            winners: complete ? winners.map(row => row.id) : [],
+            boardComplete,
+            duplicated,
+            complete,
+            // Who the tool could not read, so the dialog can name them rather than
+            // just declining to help.
+            unknown: ranked.filter(row => !row.hand).map(row => row.id),
+        };
+    },
+};
+
+if (typeof module !== "undefined" && module.exports) {
+    module.exports = HandRank;
+}

JoshHeaps.Net/wwwroot/js/casino/holdem/main.js

@@ -257,6 +257,22 @@ document.addEventListener("DOMContentLoaded", () => {
 
     Render.mount(state, hooks);
 
+    // Table Setup used to ship open, so the tool's first screen was its most crowded
+    // one: seat count, blind direction, dedicated dealer, both blinds, starting stack
+    // and two sub-panels, all under a felt the dealer had come here to look at. It is
+    // the right panel to have open on a table nobody has touched, and the wrong one to
+    // have open on the fourth hand of a night.
+    //
+    // Decided once, here, rather than on every render — a panel that reopened itself
+    // whenever the table emptied would fight the dealer who had just shut it.
+    (() => {
+        const panel = document.getElementById("tableSetupPanel");
+        if (!panel) return;
+        const fresh = (Number(state.handNumber) || 1) === 1 &&
+            !state.seats.some(seat => HoldemState.seatInPlay(seat));
+        panel.open = fresh;
+    })();
+
     const openPotWizard = () => {
         // Chips still in front of players belong in the pot being priced, so bank them
         // before the wizard reads anyone's contribution.
@@ -301,6 +317,16 @@ document.addEventListener("DOMContentLoaded", () => {
         rerender: hooks.rerender,
         resolvePot: openPotWizard,
 
+        // The first thing a dealer does on a table nobody is sitting at. Opens the
+        // seat editor on the first free chair — the same panel tapping the seat gives
+        // them, so the button teaches the gesture rather than replacing it.
+        seatFirstPlayer: () => {
+            const index = state.seats.findIndex(seat => !HoldemState.seatInPlay(seat));
+            Render.toggleSeatEditor(index < 0 ? 0 : index);
+            const panel = document.getElementById("seatEditorPanel");
+            if (panel) panel.scrollIntoView({ block: "nearest", behavior: "smooth" });
+        },
+
         beginAction: label => {
             actionHistory.push({ label, snapshot: snapshotState() });
             if (actionHistory.length > MAX_UNDO_STEPS) actionHistory.shift();
@@ -429,6 +455,15 @@ document.addEventListener("DOMContentLoaded", () => {
         Render.renderTableHint();
     });
 
+    document.getElementById("trackBoardToggle").addEventListener("change", e => {
+        state.trackBoard = e.target.checked;
+        // Turning it off leaves whatever was entered in place rather than wiping it —
+        // a dealer who taps this by mistake mid-hand can tap it back and the flop is
+        // still there. The next hand clears it the way it clears everything else.
+        hooks.save(state);
+        Render.renderAll();
+    });
+
     document.getElementById("tableNotes").addEventListener("input", e => {
         state.notes = e.target.value;
         hooks.save(state);
@@ -847,4 +882,51 @@ document.addEventListener("DOMContentLoaded", () => {
             "The saved table couldn't be read, so this one started empty.",
             { duration: 12000 });
     }
+
+    // ---- Working with no signal ----
+    //
+    // The night already survives anything — it lives in localStorage — but the page
+    // did not. Reload in a back room with one bar of signal and a dealer with nine
+    // players waiting got a browser error page. The worker below keeps the tool itself
+    // on the phone; see wwwroot/casino/sw.js for what it does and doesn't cache.
+    //
+    // The list of files to keep is read off this document rather than written down in
+    // the worker. Every stylesheet and script the page actually loaded is right here in
+    // the DOM, already carrying the ?v= release string from _Layout — so the list
+    // cannot drift from the markup, which is the way a hand-kept one always fails: it
+    // stays right until somebody adds a script tag, and then the tool is quietly
+    // offline-broken until a dealer finds out the hard way.
+    const registerWorker = () => {
+        if (!("serviceWorker" in navigator)) return;
+
+        const shell = [
+            location.pathname,
+            "/casino/holdem.webmanifest",
+            ...[...document.querySelectorAll('link[rel="stylesheet"][href]')].map(el => el.href),
+            ...[...document.querySelectorAll("script[src]")].map(el => el.src),
+        ].filter(url => new URL(url, location.origin).origin === location.origin);
+
+        navigator.serviceWorker.register("/casino/sw.js", { scope: "/casino/" })
+            .then(reg => {
+                // The worker that will answer for this page — `controller` is still null
+                // on the very first visit, before activate has claimed it.
+                const worker = reg.active || navigator.serviceWorker.controller;
+                if (worker) {
+                    worker.postMessage({ type: "precache", urls: shell });
+                    return;
+                }
+                navigator.serviceWorker.ready.then(ready => {
+                    if (ready.active) ready.active.postMessage({ type: "precache", urls: shell });
+                });
+            })
+            .catch(() => {
+                // No worker means no offline, and nothing else. Every other part of the
+                // tool works exactly as it did, so this is not worth a toast at a table.
+            });
+    };
+
+    // After load, not during it: registering competes with the page's own scripts for
+    // the network, and the first paint matters more than the second visit does.
+    if (document.readyState === "complete") registerWorker();
+    else window.addEventListener("load", registerWorker, { once: true });
 });

JoshHeaps.Net/wwwroot/js/casino/holdem/potWizard.js

@@ -76,6 +76,39 @@ const PotWizard = {
     },
 
     renderStep() {
+        this.renderStepInto();
+        this.markOverflow();
+    },
+
+    // Say when the list runs on past the bottom of the dialog.
+    //
+    // The body scrolls and the footer stays put, which is the right shape and was as far
+    // as it went: nothing on screen said there was anything below. Measured on a Galaxy
+    // S8, a three-way pot showed two names and cut the third off on a row boundary, just
+    // under a hairline that reads as the end of the list. A dealer would have ticked the
+    // two they could see. On the dialog that pays the table, a list that silently
+    // continues is not a small thing.
+    markOverflow() {
+        const body = document.getElementById("potWizardBody");
+        const modal = document.getElementById("potWizardModal");
+        if (!body || !modal) return;
+
+        const update = () => {
+            const below = body.scrollHeight - body.clientHeight - body.scrollTop;
+            modal.classList.toggle("potWizardMoreBelow", below > 1);
+        };
+        if (!this.watchingOverflow) {
+            body.addEventListener("scroll", update, { passive: true });
+            window.addEventListener("resize", update);
+            this.watchingOverflow = true;
+        }
+        update();
+        // Again once the browser has laid the step out — on the first open the body has
+        // no height yet, and an answer taken then is always "nothing below".
+        requestAnimationFrame(update);
+    },
+
+    renderStepInto() {
         // Auto-resolve any round where only one player is left standing — nothing to ask.
         let contenders = PotCalculator.getContenders(this.entries);
         while (contenders.length === 1) {
@@ -87,6 +120,11 @@ const PotWizard = {
 
         const body = document.getElementById("potWizardBody");
         body.innerHTML = "";
+        // The footer is a sibling of the body, outside the scroll — see the note in
+        // TexasHoldem.cshtml. Cleared alongside it so a step can never inherit the
+        // previous step's buttons.
+        const footer = this.footer();
+        footer.innerHTML = "";
         if (this.log.length === 0) body.appendChild(this.renderBreakdown());
         body.appendChild(this.renderLog());
 
@@ -119,13 +157,25 @@ const PotWizard = {
         const round = document.createElement("div");
         round.className = "potWizardRound";
 
+        const showdown = this.readShowdown(contenders);
+        const decided = showdown && showdown.complete ? new Set(showdown.winners) : null;
+        const handById = new Map((showdown ? showdown.players : []).map(p => [p.id, p.hand]));
+
         const prompt = document.createElement("p");
         prompt.className = "potWizardPrompt";
-        prompt.innerHTML = this.log.length === 0
-            ? `Who won the hand? <strong>${Money.format(remaining)}</strong> is in play.`
-            : `That side pot still leaves <strong>${Money.format(remaining)}</strong> unassigned — who's next best?`;
+        prompt.innerHTML = decided
+            // It has read the hands, so it stops asking a question it knows the answer
+            // to. The list stays — the dealer confirms, and a hand entered wrong is
+            // corrected by unticking rather than by starting over.
+            ? `<strong>${Money.format(remaining)}</strong> in play.`
+            : this.log.length === 0
+                ? `Who won the hand? <strong>${Money.format(remaining)}</strong> is in play.`
+                : `That side pot still leaves <strong>${Money.format(remaining)}</strong> unassigned — who's next best?`;
         round.appendChild(prompt);
 
+        const verdict = this.renderVerdict(showdown, contenders);
+        if (verdict) round.appendChild(verdict);
+
         const choices = document.createElement("div");
         choices.className = "potWizardChoices";
         contenders.forEach(c => {
@@ -135,6 +185,10 @@ const PotWizard = {
             const checkbox = document.createElement("input");
             checkbox.type = "checkbox";
             checkbox.value = c.id;
+            if (decided && decided.has(c.id)) {
+                checkbox.checked = true;
+                label.classList.add("chosen");
+            }
             // The row is the target, not the box — so the row has to look chosen too.
             checkbox.addEventListener("change", () => {
                 label.classList.toggle("chosen", checkbox.checked);
@@ -149,11 +203,25 @@ const PotWizard = {
             name.textContent = this.nameById.get(c.id);
             text.appendChild(name);
 
+            // How this hand was read, under the name — on every row rather than only
+            // the winning one, because a dealer checking the tool's answer is reading
+            // the losers' hands, and a row that says nothing is a row they have to work
+            // out for themselves. It takes the line the ceiling used to have; the
+            // ceiling moves out beside the name, which is where the phone layout had
+            // already put it to save exactly this line.
+            const read = handById.get(c.id);
+            if (read) {
+                const ranked = document.createElement("span");
+                ranked.className = "potWizardChoiceHand";
+                ranked.textContent = HandRank.describe(read);
+                text.appendChild(ranked);
+            }
+            label.appendChild(text);
+
             const upTo = document.createElement("span");
             upTo.className = "potWizardChoiceUpTo";
             upTo.textContent = `up to ${Money.format(c.remaining)}`;
-            text.appendChild(upTo);
-            label.appendChild(text);
+            label.appendChild(upTo);
 
             // Cards sit apart from the name rather than inside it, so a dealer reading
             // a row across the table sees the hand, not a sentence.
@@ -182,7 +250,11 @@ const PotWizard = {
 
         const confirmBtn = document.createElement("button");
         confirmBtn.className = "holdemBtn holdemBtnAccent";
-        confirmBtn.textContent = "Confirm Winner(s)";
+        confirmBtn.textContent = decided
+            ? (showdown.winners.length === 1
+                ? `Pay ${this.nameById.get(showdown.winners[0])}`
+                : "Split the pot")
+            : "Confirm Winner(s)";
         confirmBtn.addEventListener("click", () => {
             const selected = Array.from(choices.querySelectorAll("input:checked")).map(i => i.value);
             if (selected.length === 0) return;
@@ -191,9 +263,91 @@ const PotWizard = {
             this.entries = result.updatedEntries;
             this.renderStep();
         });
-        round.appendChild(confirmBtn);
-
         body.appendChild(round);
+        footer.appendChild(confirmBtn);
+    },
+
+    // How the tool reads the hands still in for this round of the pot.
+    //
+    // Asked per round rather than once for the whole hand, because a side pot has its
+    // own contenders: the player who won the main pot is out of the question for the
+    // next one, and the best hand among who is left is a different answer.
+    //
+    // Only when the dealer asked to track the board — everything here is off by default
+    // and the wizard behaves exactly as it always has.
+    readShowdown(contenders) {
+        if (!this.state.trackBoard || typeof HandRank === "undefined") return null;
+        return HandRank.showdown(this.state.board, contenders.map(c => ({
+            id: c.id,
+            cards: this.cardsById.get(c.id) || [],
+        })));
+    },
+
+    // What the winners would actually be paid if this round were confirmed now.
+    //
+    // Worked out from the same function that does the paying rather than from a second
+    // one that agrees with it today — a preview that can drift from the payout is worse
+    // than no preview, because it is the number the dealer says out loud.
+    previewPayout(winnerIds) {
+        try {
+            const result = PotCalculator.resolveRound(this.entries, this.orderFromButton(winnerIds));
+            return result.perWinner || [];
+        } catch {
+            return [];
+        }
+    },
+
+    // "Alexandra wins $25 with a full house, kings full of threes."
+    //
+    // The whole point of the feature, in one line: who won and how much. It goes above
+    // the list rather than replacing it, because the dealer still confirms — the tool
+    // reads the cards, a person still says the pot is good.
+    renderVerdict(showdown, contenders) {
+        if (!showdown) return null;
+
+        const line = document.createElement("p");
+        line.className = "potWizardVerdict";
+
+        if (!showdown.boardComplete) {
+            line.classList.add("potWizardVerdictUnsure");
+            line.textContent = "The board isn't all five cards yet, so pick the winner yourself — or fill it in on the felt and this will read it.";
+            return line;
+        }
+        if (showdown.duplicated) {
+            line.classList.add("potWizardVerdictUnsure");
+            line.textContent = "The same card is in two places, so this can't be read. Check the cards on the felt.";
+            return line;
+        }
+        if (!showdown.complete) {
+            const missing = showdown.unknown.map(id => this.nameById.get(id)).filter(Boolean);
+            line.classList.add("potWizardVerdictUnsure");
+            line.textContent = missing.length
+                ? `${this.listOf(missing)} ${missing.length === 1 ? "hasn't" : "haven't"} turned over, so who won is still yours to say.`
+                : "Not enough cards to read this one.";
+            return line;
+        }
+
+        const paid = this.previewPayout(showdown.winners);
+        const amount = paid.length ? Money.format(paid[0].amount) : "";
+        const names = showdown.winners.map(id => this.nameById.get(id)).filter(Boolean);
+        const hand = showdown.players.find(p => p.id === showdown.winners[0]);
+        const described = HandRank.phrase(hand && hand.hand);
+
+        line.classList.add("potWizardVerdictSure");
+        line.textContent = names.length === 1
+            ? `${names[0]} wins ${amount} with ${described}.`
+            : `${this.listOf(names)} split it, ${amount} each, with ${described}.`;
+        return line;
+    },
+
+    listOf(names) {
+        if (names.length <= 1) return names[0] || "";
+        return `${names.slice(0, -1).join(", ")} and ${names[names.length - 1]}`;
+    },
+
+    // The row of buttons that never scrolls away.
+    footer() {
+        return document.getElementById("potWizardFooter");
     },
 
     recordRound(result) {
@@ -331,12 +485,10 @@ const PotWizard = {
             const note = document.createElement("p");
             note.className = "fieldHint potWizardPendingNote";
             note.textContent = "Nothing has been paid out yet. Start Next Hand pushes the pot and logs the hand.";
-            wrap.appendChild(actions);
             wrap.appendChild(note);
-            return wrap;
         }
 
-        wrap.appendChild(actions);
+        this.footer().appendChild(actions);
         return wrap;
     },
 };

JoshHeaps.Net/wwwroot/js/casino/holdem/state.js

@@ -58,6 +58,14 @@ const HoldemState = {
             bigBlind: 2,
             defaultStack: null,
             street: "preflop",
+            // The community cards, in the order they hit the felt: three, then one,
+            // then one. Empty strings for the ones that haven't been turned yet, the
+            // same way a seat's two hole cards are stored.
+            board: ["", "", "", "", ""],
+            // Whether the dealer wants to be asked for them at all. Off is the tool as
+            // it has always been: no board on the felt, and the pot wizard asks who
+            // won. On, it reads the showdown and answers that question itself.
+            trackBoard: false,
             currentBet: 0,
             lastRaiseSize: 0,
             actingSeatIndex: null,
@@ -216,6 +224,11 @@ const HoldemState = {
             // A table saved before the dead button existed has neither flag. False is
             // the right guess for both: it means the rotation carries on the ordinary
             // way, which is what that table was already doing.
+            // A table saved before the board existed has no board and was not tracking
+            // one. Off is also the right guess for a dealer who never asked for it.
+            parsed.trackBoard = parsed.trackBoard === true;
+            parsed.board = this.normaliseBoard(parsed.board);
+
             parsed.deadButton = parsed.deadButton === true;
             parsed.deadSmallBlind = parsed.deadSmallBlind === true;
             // An old save has no record of who was dealt last hand. Empty is the
@@ -705,6 +718,7 @@ const HoldemState = {
         // that stayed on would take chips off somebody who never said the word.
         state.straddleCount = 0;
         state.postedStraddles = [];
+        this.clearBoard(state);
         state.seats.forEach(p => {
             p.folded = false;
             p.cards = ["", ""];
@@ -717,6 +731,21 @@ const HoldemState = {
         this.ensureButtonOnPlayer(state);
     },
 
+    // Exactly five slots of either a two-character card or an empty string, whatever
+    // arrived. A board read off a file is the one that has been somewhere this tool
+    // cannot see.
+    normaliseBoard(board) {
+        const source = Array.isArray(board) ? board : [];
+        return Array.from({ length: 5 }, (_, i) => {
+            const card = typeof source[i] === "string" ? source[i].trim().toUpperCase() : "";
+            return /^[23456789TJQKA][SHDC]$/.test(card) ? card : "";
+        });
+    },
+
+    clearBoard(state) {
+        state.board = ["", "", "", "", ""];
+    },
+
     // Appends a resolved-pot record to the running session log a dealer can point to
     // for chip-count disputes, then advances the hand counter. `payouts` is
     // [{ id, label, amount }, ...] — who got paid what out of this hand's pot(s).

tests/holdem/README.md

@@ -20,6 +20,8 @@ other in the page, in the load order from `TexasHoldem.cshtml`.
 | `betting.test.js` | Blinds, street completion, raises reopening the action, all-in run-outs |
 | `state.test.js` | Button rotation, which seats post blinds, session standings |
 | `splitCalculator.test.js` | Chopped pots, odd chips, unmakeable denominations |
+| `handRank.test.js` | What beats what: categories, the wheel, kickers, best five of seven |
+| `readingTheShowdown.test.js` | The board, the picker that fills it, and who wins a side pot |
 | `invariants.test.js` | 1000 randomised hands per property (see below) |
 
 ## The layout audit
@@ -33,10 +35,38 @@ node tests/holdem/runUiAudit.js          # needs Playwright and the site running
 
 It checks WCAG AA contrast, touch target sizes, touch-action and tap highlights, that fields stay selectable and are large
 enough not to trigger iOS zoom, that nothing important is covered or off-screen with
-no way to scroll to it, that no grid column has collapsed to nothing, and that no
-text is painted outside its box. It runs over the table, all five overlays and Quick
-Split, at thirteen viewports from a 360px phone to a 1600px desktop and the same
-phones on their side — each viewport twice, once with a mouse and once with a thumb.
+no way to scroll to it, that no grid column has collapsed to nothing, that no
+text is painted outside its box, that no seat covers another seat's writing, and that
+no bet chip is hidden behind a seat or another chip. It
+runs over the table, all five overlays and Quick Split, at thirteen viewports from a
+360px phone to a 1600px desktop and the same phones on their side — each viewport
+twice, once with a mouse and once with a thumb, and at three table shapes, mid-hand:
+the sweep posts blinds, deals and calls three times before it measures anything, so
+the bets layer has chips on it. It used to audit a table nobody had ever bet at,
+which is a third of what is on the felt during a hand left unlooked at.
+
+"Covered" and "out of view" are told apart before anything is hit-tested. An element
+clipped away by a panel that scrolls is not covered by whatever happens to be painted
+at that height, and reporting it that way is a diagnosis of something that isn't
+happening: the pot wizard's third winner, below the fold in a list, came back for four
+viewports running as "fully covered by button Confirm Winner(s)".
+
+More than one table shape is also newer than it sounds. The sweep ran on one — five
+players at a nine-handed table — for its whole life, and how much room the felt has
+depends on how many people are sitting down far more than on anything a viewport can
+report. A full table is where the seats crowd and it had never been looked at: the
+first sweep that did found one player's name or stack painted under another on eighteen
+of the configurations, on every phone in the list.
+
+The third shape is the same five players with the board turned on and five cards on it.
+It is deliberately the same roster in the same chairs as the second, so the only thing
+that differs between the two rows is the board — a seventh player would have told us
+that a seven-handed table crowds, which is true, already known, and not what that row
+is there to find out. It earned its place immediately: the board takes the middle of
+the felt, or a strip above it where the middle is too small, and the strip costs the
+felt height. The seats follow a felt that changes size on their own, being placed in
+percentages of it; the chips anchored inside them do not, being placed in pixels. They
+were laid out for a felt 23px taller than the one they were painted on.
 
 That last part, and two of those checks, are newer than this paragraph claiming them.
 `runAll()` did not include the contrast check at all, and touch targets sat behind an
@@ -53,8 +83,8 @@ offenders, which reads identically to a pass. The same goes for the two exemptio
 that exist: a disabled control, which WCAG itself excuses, and a link inside a
 sentence, whose size is set by the line it sits in.
 
-Deliberately not in CI: the unit suite is dependency-free and runs in a second, while
-this one needs a browser and a running site. Run it after any change that adds or
+Deliberately not in CI: the unit suite is dependency-free and runs in about five
+seconds, while this one needs a browser and a running site. Run it after any change that adds or
 resizes a control. With no Playwright to hand, paste `uiAudit.js` into devtools and
 call `LayoutAudit.runAll()`.
 
@@ -77,6 +107,23 @@ something changed, and each time they only covered whatever had just been touche
 That is how the standings table shipped with a player-name column zero pixels wide
 and kept it for two rounds.
 
+## Why the hand evaluator is tested the way it is
+
+`handRank.js` is the only module here that can pay the wrong person without anybody
+noticing. A pot wizard that asks "who won?" is never more wrong than the dealer ticking
+the box; one that answers the question itself is wrong on its own account, silently,
+and a table that has come to trust it will not check.
+
+So besides the hand-written cases — every category against every other, the wheel, three
+kickers deep, the counterfeited two pair, the board that plays — it is checked against a
+second evaluator written a different way, on twenty thousand random seven-card hands and
+four thousand random pairs of them. The second one is in the test file and shares no
+helper with the module: it scores a hand by counting ranks into a sortable list rather
+than by asking about shapes. That pass is most of the suite's running time and it is
+worth it; three deliberately introduced bugs — an ace that keeps its value in the wheel,
+a dropped two-pair kicker, a flush read only by its top card — are each caught by it as
+well as by the cases written for them.
+
 ## Why the randomised suite exists
 
 Every correctness bug found in this tool so far has been shape-dependent: blinds posted

tests/holdem/announcing.test.js

@@ -0,0 +1,135 @@
+// What the tool says out loud between taps.
+//
+// The action bar carried `aria-live="polite"` on the element whose entire contents
+// render() rebuilds every action, so a screen reader read the whole bar after every
+// single tap. Measured, after one:
+//
+//   "PRE-FLOP Pot $15 9 players Bob · $250 behind 0:00 ↶ dealing the hand U
+//    Fold Call $10 Raise"
+//
+// Nothing in that is wrong. It is just unusable at the speed a hand is dealt, and it
+// is the documented anti-pattern: a live region has to be a stable element whose text
+// changes, not a container that gets replaced.
+//
+// What a dealer running this by ear needs is who is on the clock and what it costs
+// them. These are the sentences.
+const test = require("node:test");
+const { describe } = require("node:test");
+const assert = require("node:assert");
+const { ActionBar, Betting, HoldemState, makeTable } = require("./load");
+
+const barOn = (state) => {
+    ActionBar.state = state;
+    ActionBar.lastAnnouncement = undefined;
+    return ActionBar;
+};
+
+describe("before the cards are out", () => {
+    test("a table that could be dealt says so", () => {
+        const state = makeTable({ seatCount: 6, seated: [0, 1, 2, 3] });
+        assert.equal(barOn(state).announcement(), "Ready to deal.");
+    });
+
+    test("and an empty table says what it is, rather than nothing", () => {
+        // This used to announce "" — silence being the honest answer when the bar was
+        // hidden on a table nobody was sitting at. The bar isn't hidden any more: an
+        // empty table is the one state a dealer meets before they know how any of this
+        // works, so it says so and offers the next step.
+        const state = HoldemState.createDefault();
+        assert.equal(barOn(state).announcement(), "Nobody's at the table yet.");
+    });
+
+    test("and a table with one named player has moved on from that", () => {
+        const state = HoldemState.createDefault();
+        state.seats[0].name = "Ann";
+        assert.notEqual(barOn(state).announcement(), "Nobody's at the table yet.");
+    });
+});
+
+describe("while a hand is running", () => {
+    const dealt = () => {
+        const state = makeTable({ seatCount: 6, seated: [0, 1, 2, 3], smallBlind: 1, bigBlind: 2 });
+        state.seats.forEach((p, i) => { p.name = ["Ann", "Ben", "Cal", "Dee"][i] || ""; });
+        Betting.beginHand(state);
+        return state;
+    };
+
+    test("names who is on the clock, what it costs, and the pot", () => {
+        const state = dealt();
+        const said = barOn(state).announcement();
+        assert.match(said, /to act/);
+        assert.match(said, /\$2 to call/);
+        assert.match(said, /Pot \$3\./);
+    });
+
+    test("it is the seat that is actually acting, not the one after it", () => {
+        const state = dealt();
+        const name = HoldemState.displayName(state.seats[state.actingSeatIndex], state.actingSeatIndex);
+        assert.ok(barOn(state).announcement().startsWith(name + " to act"));
+    });
+
+    test("says checked to them rather than $0 to call", () => {
+        // A number nobody owes is a number a dealer has to stop and parse.
+        const state = dealt();
+        state.currentBet = 0;
+        assert.match(barOn(state).announcement(), /checked to them/);
+        assert.doesNotMatch(barOn(state).announcement(), /\$0 to call/);
+    });
+
+    test("a street with nobody left to act says the street is done", () => {
+        const state = dealt();
+        state.actingSeatIndex = null;
+        state.street = "flop";
+        assert.equal(barOn(state).announcement(), "Flop betting is done.");
+    });
+
+    test("and a finished hand points at the pot", () => {
+        const state = dealt();
+        state.seats.forEach((p, i) => { if (i !== 0) p.folded = true; });
+        assert.equal(Betting.isHandOver(state), true);
+        assert.equal(barOn(state).announcement(), "Hand over. Resolve the pot.");
+    });
+});
+
+describe("how often it speaks", () => {
+    // A live region that is handed the same sentence twice reads it out twice, and
+    // render() runs for a great many reasons that have nothing to do with whose turn
+    // it is — a resize, a panel opening, a note being typed.
+    const stubTarget = () => {
+        const el = { textContent: "" };
+        global.document = { getElementById: id => (id === "actionAnnouncer" ? el : null) };
+        return el;
+    };
+
+    test("says nothing new when nothing changed", () => {
+        const el = stubTarget();
+        const state = makeTable({ seatCount: 6, seated: [0, 1, 2, 3], smallBlind: 1, bigBlind: 2 });
+        Betting.beginHand(state);
+        const bar = barOn(state);
+
+        bar.announce();
+        const first = el.textContent;
+        assert.ok(first.length > 0);
+
+        el.textContent = "CLEARED";       // if announce() writes again, this comes back
+        bar.announce();
+        assert.equal(el.textContent, "CLEARED", "the same sentence was not written twice");
+        delete global.document;
+    });
+
+    test("but does when the action moves", () => {
+        const el = stubTarget();
+        const state = makeTable({ seatCount: 6, seated: [0, 1, 2, 3], smallBlind: 1, bigBlind: 2 });
+        Betting.beginHand(state);
+        const bar = barOn(state);
+
+        bar.announce();
+        const first = el.textContent;
+        Betting.call(state, state.actingSeatIndex);
+        bar.announce();
+
+        assert.notEqual(el.textContent, first);
+        assert.match(el.textContent, /to act/);
+        delete global.document;
+    });
+});

tests/holdem/betChips.test.js

@@ -0,0 +1,473 @@
+// Where a player's bet sits on the felt.
+//
+// The chips were the last thing on the table nobody had measured. They were laid out on
+// a ring of their own at a fixed 27/44 of the seat's reach — a fraction picked when the
+// seat ring was itself fixed — and the choice between that ring and clinging to the
+// puck was made by asking how wide the *window* was. Neither of those is a question
+// about the felt. A phone held sideways is 844px across and gives the felt 117px of
+// height: it passed the width test, took the ring, and every chip on it was painted
+// behind a seat. The seats carry `z-index: 1` and the bets layer carries none, so a
+// chip under a puck is not dimmed or half-hidden — it is gone, and with it the only
+// number saying what that player has in.
+//
+// Everything below is geometry, so it runs without a browser. The felts are the ones
+// measured on real screens: 692x117 is a phone on its side, 300x216 a phone upright.
+const test = require("node:test");
+const { describe } = require("node:test");
+const assert = require("node:assert");
+const { Render } = require("./load");
+
+const RADII = { rx: 44, ry: 42 };
+const PUCK = { width: 96, height: 64 };
+const CHIP = { width: 44, height: 22 };
+
+const felt = (width, height) => ({ width, height });
+const spacingOn = (box, puck = PUCK, chip = CHIP) =>
+    ({ box, puck, chip, clearance: Render.BET_CLEARANCE_PX });
+
+// A chip's centre in pixels on a given felt.
+const chipPixels = (index, count, box, spacing, radii = RADII) => {
+    const pos = Render.betPosition(index, count, box.width / box.height, radii, spacing);
+    return { x: (pos.left / 100) * box.width, y: (pos.top / 100) * box.height };
+};
+
+const seatPixels = (index, count, box, radii = RADII) => {
+    const pos = Render.seatPosition(index, count, box.width / box.height, radii);
+    return { x: (pos.left / 100) * box.width, y: (pos.top / 100) * box.height };
+};
+
+// Two axis-aligned boxes, by their centres. The same test the seats settle among
+// themselves — see seatFootprint.test.js — because it is the same question.
+const boxesOverlap = (a, b, sizeA, sizeB) =>
+    Math.abs(a.x - b.x) < (sizeA.width + sizeB.width) / 2 - 0.5 &&
+    Math.abs(a.y - b.y) < (sizeA.height + sizeB.height) / 2 - 0.5;
+
+describe("reaching the edge of a box", () => {
+    test("straight out the side is half the width, straight up half the height", () => {
+        assert.equal(Render.edgeReach(96, 64, 1, 0), 48);
+        assert.equal(Render.edgeReach(96, 64, 0, -1), 32);
+        assert.equal(Render.edgeReach(96, 64, -1, 0), 48);
+    });
+
+    test("a corner is whichever edge the ray meets first", () => {
+        // 45° out of a 96x64 box: the top and bottom are nearer than the sides, so the
+        // ray leaves through one of those. A single radius — half the diagonal, 57.7 —
+        // would put a chip 25px further out than it needs to be, which on a 117px felt
+        // is the difference between clear of the puck and off the table.
+        const diagonal = Render.edgeReach(96, 64, Math.SQRT1_2, Math.SQRT1_2);
+        assert.ok(Math.abs(diagonal - 32 / Math.SQRT1_2) < 1e-9, `got ${diagonal}`);
+        assert.ok(diagonal < 48, "and never further than the wider half-edge");
+    });
+
+    test("a square is the same in every direction it is asked about", () => {
+        const straight = Render.edgeReach(50, 50, 0, 1);
+        assert.equal(Render.edgeReach(50, 50, 1, 0), straight);
+    });
+});
+
+describe("getting one box clear of another", () => {
+    const PUCK96 = { width: 96, height: 88 };
+    const CHIP50 = { width: 50, height: 21 };
+
+    // Travel `t` along `u` from the puck's centre, then ask whether the two boxes are
+    // apart. This is the question clearanceReach is supposed to answer, asked the long
+    // way round so the answer can be checked rather than restated.
+    const apart = (t, ux, uy) =>
+        Math.abs(t * ux) >= (PUCK96.width + CHIP50.width) / 2 - 1e-9 ||
+        Math.abs(t * uy) >= (PUCK96.height + CHIP50.height) / 2 - 1e-9;
+
+    test("straight up and down it is what adding the two half-heights gives", () => {
+        assert.equal(Render.clearanceReach(PUCK96, CHIP50, 0, -1), (88 + 21) / 2);
+        assert.equal(Render.clearanceReach(PUCK96, CHIP50, 1, 0), (96 + 50) / 2);
+    });
+
+    test("on a diagonal it is more than the two reaches added together", () => {
+        // The bug this replaced. Two axis-aligned boxes are clear when they are apart on
+        // one axis, and the travel that achieves that is the reach of the box you get by
+        // adding their widths and their heights — which is never less, and on a diagonal
+        // is more, than adding the two boxes' own reaches. At 768x1024 the difference was
+        // 11px, and 11px is how far inside its own player's puck the chip came to rest.
+        const dirs = [[0.83, 0.56], [0.6, 0.8], [0.9, 0.44], [Math.SQRT1_2, Math.SQRT1_2]];
+        let shortfalls = 0;
+        dirs.forEach(([ux, uy]) => {
+            const summed = Render.edgeReach(PUCK96.width, PUCK96.height, ux, uy) +
+                Render.edgeReach(CHIP50.width, CHIP50.height, ux, uy);
+            const proper = Render.clearanceReach(PUCK96, CHIP50, ux, uy);
+            assert.ok(proper >= summed - 1e-9, `${ux},${uy}: ${proper} < ${summed}`);
+            if (proper <= summed + 1e-9) return;
+            // Where they differ, the old answer was not merely smaller — it was wrong:
+            // travelling that far leaves the two boxes still on top of each other.
+            shortfalls++;
+            assert.ok(!apart(summed, ux, uy),
+                `${ux},${uy}: adding the reaches gave ${summed.toFixed(1)}, which is enough after all`);
+        });
+        assert.ok(shortfalls >= 2, "expected the two to actually differ on some of these");
+    });
+
+    test("and it is exactly enough — a hair less is not", () => {
+        const dirs = [[0, -1], [1, 0], [0.83, 0.56], [0.6, -0.8], [-0.9, 0.44]];
+        dirs.forEach(([ux, uy]) => {
+            const t = Render.clearanceReach(PUCK96, CHIP50, ux, uy);
+            assert.ok(apart(t, ux, uy), `${ux},${uy}: ${t}px still overlaps`);
+            assert.ok(!apart(t - 0.5, ux, uy), `${ux},${uy}: ${t}px is further than needed`);
+        });
+    });
+});
+
+describe("a chip on the ring", () => {
+    test("with nothing measured it keeps the proportions the ring always used", () => {
+        // The unit suite and the very first render both call it this way. 27/44 and
+        // 25/42 of the seat's own reach, so the chips move with the ring rather than
+        // with the felt.
+        const seat = Render.seatPosition(3, 9, 1.4, RADII);
+        const chip = Render.betPosition(3, 9, 1.4, RADII, null);
+        assert.ok(Math.abs((chip.left - 50) - (seat.left - 50) * (27 / 44)) < 1e-9);
+        assert.ok(Math.abs((chip.top - 50) - (seat.top - 50) * (25 / 42)) < 1e-9);
+    });
+
+    test("measured, it sits just clear of its own puck rather than at a fixed fraction", () => {
+        const box = felt(560, 400);
+        const spacing = spacingOn(box);
+        for (let i = 0; i < 9; i++) {
+            const seat = seatPixels(i, 9, box);
+            const chip = chipPixels(i, 9, box, spacing);
+            assert.ok(!boxesOverlap(seat, chip, PUCK, CHIP),
+                `seat ${i + 1}'s own chip is under its own puck`);
+        }
+    });
+
+    test("and never wanders into the pot badge in the middle", () => {
+        // The floor: 38% of the way out. Without it a chip walked in past a puck that
+        // is 96px wide on a felt whose reach is 90px would land on the pot total, which
+        // is the same fault at the other end of the spoke.
+        const box = felt(300, 216);
+        const spacing = spacingOn(box, { width: 200, height: 150 });
+        for (let i = 0; i < 6; i++) {
+            const seat = seatPixels(i, 6, box);
+            const chip = chipPixels(i, 6, box, spacing);
+            const seatReach = Math.hypot(seat.x - box.width / 2, seat.y - box.height / 2);
+            const chipReach = Math.hypot(chip.x - box.width / 2, chip.y - box.height / 2);
+            assert.ok(chipReach >= seatReach * 0.38 - 0.5,
+                `chip ${i + 1} is ${chipReach.toFixed(1)}px out of a possible ${seatReach.toFixed(1)}`);
+        }
+    });
+
+    test("a seat sitting dead centre gets its chip put somewhere, not NaN", () => {
+        const box = felt(400, 300);
+        const pos = Render.betPosition(0, 1, box.width / box.height, { rx: 0, ry: 0 }, spacingOn(box));
+        assert.ok(Number.isFinite(pos.left) && Number.isFinite(pos.top), JSON.stringify(pos));
+    });
+});
+
+describe("deciding whether a ring of chips fits at all", () => {
+    // The shape the layout works in: one entry per chip on the felt, carrying its own
+    // measured size, because a chip is as wide as the number printed on it.
+    const tableOf = (count, chip = CHIP) => {
+        Render.state = { seatCount: count, seats: Array.from({ length: count }, () => ({})) };
+        return Array.from({ length: count }, (_, i) =>
+            ({ index: i, width: chip.width, height: chip.height }));
+    };
+
+    test("a desktop felt has room for one", () => {
+        const box = felt(700, 500);
+        const all = tableOf(9);
+        assert.equal(Render.chipsClearTheSeats(all, box.width / box.height, RADII, spacingOn(box)), true);
+    });
+
+    test("a phone on its side has not, whatever the window's width says about it", () => {
+        // 692x117, measured on an 844px-wide phone in landscape — a viewport that sails
+        // past a `max-width: 700px` test. The ring is a slot: the 38% floor puts a chip
+        // on the seat across the table from it.
+        const box = felt(692, 117);
+        const all = tableOf(9);
+        assert.equal(Render.chipsClearTheSeats(all, box.width / box.height, RADII, spacingOn(box)), false,
+            "this is the felt where every chip was painted behind a puck");
+    });
+
+    test("its answer is the one the geometry gives", () => {
+        // Not a restatement of the two cases above: whatever it returns, it has to agree
+        // with laying the chips out and measuring them against every puck on the felt.
+        const shapes = [felt(700, 500), felt(692, 117), felt(300, 216), felt(1000, 620), felt(480, 300)];
+        shapes.forEach(box => {
+            const all = tableOf(9);
+            const spacing = spacingOn(box);
+            const aspect = box.width / box.height;
+            const clear = all.every(({ index: i }) => {
+                const chip = chipPixels(i, 9, box, spacing);
+                return all.every(({ index: j }) => !boxesOverlap(chip, seatPixels(j, 9, box), CHIP, PUCK));
+            });
+            assert.equal(Render.chipsClearTheSeats(all, aspect, RADII, spacing), clear,
+                `${box.width}x${box.height}`);
+        });
+    });
+
+    test("each chip is asked about at its own size", () => {
+        // A chip is as wide as the number on it: at 768x1024, four players' chips were
+        // 40px and one holding a four-figure bet was 47. Answering the question once, at
+        // the widest, told the whole felt it had no room for a ring — and dropped all
+        // five chips onto their own players' stacks to solve a problem four of them
+        // did not have.
+        // The felt and the pucks are the iPad's own, measured: 672x672, pucks 96x88.
+        const box = felt(672, 672);
+        const narrow = { width: 40, height: 20 };
+        const huge = { width: 420, height: 380 };
+        const spacing = { box, puck: { width: 96, height: 88 }, clearance: Render.BET_CLEARANCE_PX };
+        const aspect = box.width / box.height;
+
+        assert.equal(Render.chipsClearTheSeats(tableOf(9, narrow), aspect, RADII, spacing), true,
+            "a felt this size has room for a ring of ordinary chips");
+        assert.equal(Render.chipsClearTheSeats(tableOf(9, huge), aspect, RADII, spacing), false,
+            "and none at all for a ring of chips that size");
+
+        // Now one of each. The answer has to come from the sizes in the list, not from
+        // one size standing in for all of them: with one huge chip the ring is off, and
+        // taking that chip out of the question puts it back.
+        const mixed = tableOf(9, narrow);
+        mixed[0] = { index: 0, ...huge };
+        assert.equal(Render.chipsClearTheSeats(mixed, aspect, RADII, spacing), false);
+        assert.equal(Render.chipsClearTheSeats(mixed.slice(1), aspect, RADII, spacing), true,
+            "the eight ordinary ones were never the problem");
+    });
+
+    test("and at the chips against each other", () => {
+        // The ring is a smaller circle than the seats' one and holds the same number of
+        // things, so it is the tighter fit of the two. Measured on a Pixel 7 with twelve
+        // players, two $10s on the ring sat on top of each other while every puck on the
+        // felt was clear — so a check that only looks at the pucks says yes to a layout
+        // that hides one player's bet behind another's.
+        // The Pixel 7's own numbers, measured: a 300x336 felt on a ring of 44 by 40.84,
+        // pucks 44x55, a pot badge 64x53, and the seven players who had put chips in.
+        const box = felt(300, 336);
+        const radii = { rx: 44, ry: 40.84 };
+        const puck = { width: 44, height: 55 };
+        const pot = { x: 150, y: 168, width: 64, height: 53 };
+        const spacing = { box, puck, pot, clearance: Render.BET_CLEARANCE_PX };
+        const aspect = box.width / box.height;
+
+        Render.state = { seatCount: 12, seats: Array.from({ length: 12 }, () => ({})) };
+        const bettors = [2, 3, 4, 5, 6, 7, 8];
+        const all = bettors.map(index => ({ index, width: index === 2 ? 44.1 : 49.7, height: 21 }));
+
+        const chips = all.map(entry => ({
+            ...entry,
+            ...chipPixels(entry.index, 12, box,
+                { ...spacing, chip: { width: entry.width, height: entry.height } }, radii),
+        }));
+        const seats = [...Array(12).keys()].map(i => seatPixels(i, 12, box, radii));
+
+        assert.equal(chips.some(chip => seats.some(seat => boxesOverlap(chip, seat, chip, puck))), false,
+            "this table is chosen so the pucks are not the problem");
+        assert.equal(chips.some(chip => boxesOverlap(chip, pot, chip, pot)), false,
+            "nor the pot");
+        assert.ok(chips.some((chip, i) => chips.some((other, j) => i !== j &&
+            boxesOverlap(chip, other, chip, other))), "and so the chips are");
+
+        assert.equal(Render.chipsClearTheSeats(all, aspect, radii, spacing), false);
+    });
+
+    test("it looks at every puck, not just the chip's own", () => {
+        // The rule this replaced walked each chip in until it was clear of its *own*
+        // seat and stopped there, which says nothing about the eight other pucks on the
+        // felt. 560x190 is a felt where that rule is satisfied and the ring is still no
+        // good: every chip clears the puck it belongs to, and one of them is sitting on
+        // a stranger's.
+        const box = felt(560, 190);
+        const all = tableOf(9);
+        const spacing = spacingOn(box);
+        const chips = all.map(({ index: i }) => chipPixels(i, 9, box, spacing));
+
+        all.forEach(({ index: i }) => assert.ok(!boxesOverlap(chips[i], seatPixels(i, 9, box), CHIP, PUCK),
+            `chip ${i + 1} is under its own puck, so this felt does not make the point`));
+        assert.ok(all.some(({ index: i }) => all.some(({ index: j }) => i !== j &&
+                boxesOverlap(chips[i], seatPixels(j, 9, box), CHIP, PUCK))),
+            "expected a chip on somebody else's puck — what the old rule could not see");
+        assert.equal(Render.chipsClearTheSeats(all, box.width / box.height, RADII, spacing), false);
+    });
+});
+
+describe("a chip clinging to its puck", () => {
+    // The fallback when there is no room for a ring: the chip hangs off the puck on the
+    // side facing the empty middle of the felt.
+    //
+    // The felts the app actually draws, with the puck size each one settles on — read
+    // off the running site at each viewport rather than guessed here, because the tight
+    // felts shrink the puck too, and a test that holds the desktop's 96x64 puck against
+    // a 528x83 felt is describing a table the app never draws.
+    const REAL = [
+        ["Galaxy S8 on its side", felt(528, 83), { width: 44, height: 55 }],
+        ["iPhone on its side", felt(692, 113), { width: 72, height: 55 }],
+        ["Galaxy S8 upright, six-handed", felt(248, 280), { width: 78, height: 88 }],
+        ["Galaxy S8 upright, packed", felt(248, 280), { width: 44, height: 55 }],
+    ];
+    // Chips are 21x17 everywhere they are anchored — the amount is the only thing on
+    // one at that size.
+    const TIGHT = { puck: { width: 44, height: 55 }, chip: { width: 21, height: 17 } };
+
+    // What each chip is told to do, worked out for the whole table at once.
+    const placeAll = (count, box, puck, chip = TIGHT.chip, radii = RADII) => {
+        Render.state = { seatCount: count, seats: Array.from({ length: count }, () => ({})) };
+        const indices = [...Array(count).keys()];
+        const entries = indices.map(i => ({ index: i, width: chip.width, height: chip.height }));
+        const spacing = { box, puck, clearance: Render.BET_CLEARANCE_PX };
+        const offs = Render.anchoredChipTravel(entries, count, box.width / box.height, RADII, spacing);
+        return indices.map(i => {
+            const seat = Render.seatPosition(i, count, box.width / box.height, radii);
+            const at = { x: (seat.left / 100) * box.width, y: (seat.top / 100) * box.height };
+            const off = offs[i];
+            return { seat: at, chip: { x: at.x + off.x, y: at.y + off.y }, off, puck };
+        });
+    };
+
+    test("on its own, a chip asks to sit just clear of its puck", () => {
+        const box = felt(692, 113);
+        const puck = { width: 72, height: 55 };
+        const seat = Render.seatPosition(0, 9, box.width / box.height, RADII);
+        const off = Render.anchoredChipOffset(seat, puck, TIGHT.chip, box, RADII);
+        const travel = Math.hypot(off.x, off.y);
+        const ux = off.x / travel, uy = off.y / travel;
+        assert.ok(Math.abs(travel -
+            (Render.clearanceReach(puck, TIGHT.chip, ux, uy) + Render.BET_CLEARANCE_PX)) < 1e-6, `travelled ${travel}`);
+    });
+
+    test("and gets it, where nothing else is in the way", () => {
+        // 692x113 is the roomier of the two landscape felts. Every chip clears its own
+        // puck and none of them has to give ground.
+        placeAll(9, felt(692, 113), { width: 72, height: 55 }).forEach((row, i) => {
+            assert.ok(!boxesOverlap(row.seat, row.chip, row.puck, TIGHT.chip),
+                `seat ${i + 1}'s chip is inside its own puck, and nothing made it go there`);
+        });
+    });
+
+    test("it goes off the rim, not down the spoke", () => {
+        // On a felt 692x113 the two are nowhere near the same direction. A seat on the
+        // top rail has a spoke pointing diagonally inward — which along a slot this flat
+        // runs sideways into the next seat — while off the rim is very nearly straight
+        // down, into the empty middle. This is the seam that put two chips on top of
+        // each other on a Galaxy S8.
+        const box = felt(692, 113);
+        const rows = placeAll(9, box, { width: 72, height: 55 });
+        const top = rows.find(r => r.seat.y < box.height * 0.3 && r.seat.x > box.width * 0.7);
+        assert.ok(top, "expected a seat up on the right of the top rail");
+        assert.ok(Math.abs(top.off.y) > Math.abs(top.off.x) * 2,
+            `chip went ${top.off.x.toFixed(1)},${top.off.y.toFixed(1)} — that is along the rail, not off it`);
+        assert.ok(top.off.y > 0, "and downward, toward the middle of the felt");
+    });
+
+    test("no two chips on a full table land on each other", () => {
+        // Four sides gave three seams. There are no sides now, and this is what that is
+        // worth — on every felt the app actually draws, at every table size.
+        REAL.forEach(([name, box, puck]) => {
+            [6, 9, 12].forEach(count => {
+                const rows = placeAll(count, box, puck);
+                for (let a = 0; a < rows.length; a++) {
+                    for (let b = a + 1; b < rows.length; b++) {
+                        assert.ok(!boxesOverlap(rows[a].chip, rows[b].chip, TIGHT.chip, TIGHT.chip),
+                            `${name}, ${count} seats: chips ${a + 1} and ${b + 1} overlap`);
+                    }
+                }
+            });
+        });
+    });
+
+    test("a chip only gives ground when it is standing on something", () => {
+        // The point of pulling chips back one at a time rather than by a rule that
+        // applies to all of them. Stopping every chip at the felt's midline, or scaling
+        // them all by one factor, takes travel from the seats at the ends of a long
+        // rail — measured at 692x113, that parked two chips on their own player's stack
+        // to solve a collision that did not exist.
+        //
+        // So: every chip either got everything it asked for, or putting the last step
+        // back reintroduces a collision it is part of. The step goes back for all of
+        // them at once, because a facing pair gives ground together and the pass that
+        // finally separates such a pair moves both ends of it.
+        const step = Render.CHIP_PULLBACK_STEP_PX;
+        REAL.forEach(([name, box, puck]) => {
+            [6, 9, 12].forEach(count => {
+                const rows = placeAll(count, box, puck).map(row => {
+                    const travel = Math.hypot(row.off.x, row.off.y);
+                    const dir = travel < 1e-6 ? { x: 0, y: 0 } : { x: row.off.x / travel, y: row.off.y / travel };
+                    const asked = travel < 1e-6 ? 0 :
+                        Render.clearanceReach(puck, TIGHT.chip, dir.x, dir.y) +
+                        Render.BET_CLEARANCE_PX;
+                    const back = Math.min(asked, travel + step);
+                    return { ...row, travel, asked,
+                        undone: { x: row.seat.x + dir.x * back, y: row.seat.y + dir.y * back } };
+                });
+
+                rows.forEach((row, i) => {
+                    if (row.travel < 1e-6 || row.travel >= row.asked - 1e-6) return;
+                    const onSomething = rows.some((other, j) => j !== i && (
+                        boxesOverlap(row.undone, other.undone, TIGHT.chip, TIGHT.chip) ||
+                        boxesOverlap(row.undone, other.seat, TIGHT.chip, puck)));
+                    assert.ok(onSomething,
+                        `${name}, ${count} seats: chip ${i + 1} stopped ${(row.asked - row.travel).toFixed(1)}px short of nothing`);
+                });
+            });
+        });
+    });
+
+    test("nobody is pushed further out than they asked to go", () => {
+        // The pull-back only ever takes travel away. A chip pushed past its clearance
+        // would be floating out on the felt with nothing to say whose it is.
+        REAL.forEach(([name, box, puck]) => {
+            placeAll(12, box, puck).forEach((row, i) => {
+                const travel = Math.hypot(row.off.x, row.off.y);
+                if (travel < 1e-6) return;
+                const ux = row.off.x / travel, uy = row.off.y / travel;
+                const asked = Render.clearanceReach(puck, TIGHT.chip, ux, uy) +
+                    Render.BET_CLEARANCE_PX;
+                assert.ok(travel <= asked + 0.5,
+                    `${name}: chip ${i + 1} went ${travel.toFixed(1)}px when ${asked.toFixed(1)} was asked for`);
+            });
+        });
+    });
+
+    test("the direction turns smoothly all the way round the ring", () => {
+        // The property a set of discrete sides cannot have, and the reason there are
+        // none: neighbouring seats get neighbouring directions, with no step anywhere.
+        const box = felt(560, 400);
+        const angles = Array.from({ length: 24 }, (_, i) => {
+            const seat = Render.seatPosition(i, 24, box.width / box.height, RADII);
+            const off = Render.anchoredChipOffset(seat, PUCK, CHIP, box, RADII);
+            return Math.atan2(off.y, off.x);
+        });
+        angles.forEach((angle, i) => {
+            const next = angles[(i + 1) % angles.length];
+            let turn = Math.abs(next - angle);
+            if (turn > Math.PI) turn = 2 * Math.PI - turn;
+            assert.ok(turn < Math.PI / 4,
+                `between seats ${i + 1} and ${i + 2} the chip jumps ${(turn * 180 / Math.PI).toFixed(0)}°`);
+        });
+    });
+
+    test("it stops short of the pot total as well", () => {
+        // The other thing in the middle of the felt, and the one number on the table
+        // that belongs to everybody. A seat on the middle of a rail sends its chip
+        // straight at it: on a Galaxy S8 on its side, seat 1's $10 was sitting on
+        // "Pot $65". The badge is measured rather than assumed to be at the centre —
+        // it lives on the table, not on either overlay, and the two are not concentric.
+        const box = felt(528, 83);
+        const pot = { x: 264, y: 40, width: 96, height: 34 };
+        Render.state = { seatCount: 12, seats: Array.from({ length: 12 }, () => ({})) };
+        const indices = [...Array(12).keys()];
+        const entries = indices.map(i =>
+            ({ index: i, width: TIGHT.chip.width, height: TIGHT.chip.height }));
+        const spacing = { box, puck: TIGHT.puck, clearance: Render.BET_CLEARANCE_PX, pot };
+        const offs = Render.anchoredChipTravel(entries, 12, box.width / box.height, RADII, spacing);
+
+        indices.forEach(i => {
+            const seat = Render.seatPosition(i, 12, box.width / box.height, RADII);
+            const chip = { x: (seat.left / 100) * box.width + offs[i].x,
+                           y: (seat.top / 100) * box.height + offs[i].y };
+            assert.ok(!boxesOverlap(chip, pot, TIGHT.chip, pot),
+                `chip ${i + 1} is on the pot total`);
+        });
+    });
+
+    test("a seat with no rim to be off still gets an answer", () => {
+        const box = felt(400, 300);
+        const off = Render.anchoredChipOffset({ left: 50, top: 50 }, PUCK, CHIP, box, RADII);
+        assert.ok(Number.isFinite(off.x) && Number.isFinite(off.y), JSON.stringify(off));
+        assert.ok(off.y < 0, "upward, which is where a puck's own writing is not");
+    });
+});

tests/holdem/board.test.js

@@ -0,0 +1,118 @@
+// The community cards, as a thing the table remembers.
+//
+// The board is the one part of a hand this tool never had a place for: hole cards go on
+// a seat, chips go in the middle, and the five cards everybody plays lived only on the
+// felt in front of the dealer. It is opt-in, because most nights nobody wants to enter
+// cards at all — and everything that reads it has to behave exactly as it always did
+// when it is off.
+const test = require("node:test");
+const { describe } = require("node:test");
+const assert = require("node:assert");
+const { Betting, HoldemState, makeTable } = require("./load");
+
+const dealt = (over = {}) => {
+    const state = { ...makeTable({ seatCount: 6, stack: 200, smallBlind: 1, bigBlind: 2 }), ...over };
+    Betting.beginHand(state);
+    return state;
+};
+
+describe("a new table", () => {
+    test("has five empty slots and is not tracking them", () => {
+        const state = HoldemState.createDefault();
+        assert.deepEqual(state.board, ["", "", "", "", ""]);
+        assert.equal(state.trackBoard, false,
+            "off by default: a dealer who never asked for this should not be asked for cards");
+    });
+});
+
+describe("how many cards are out", () => {
+    test("none, three, four, five", () => {
+        assert.equal(Betting.boardCardsFor("preflop"), 0);
+        assert.equal(Betting.boardCardsFor("flop"), 3);
+        assert.equal(Betting.boardCardsFor("turn"), 4);
+        assert.equal(Betting.boardCardsFor("river"), 5);
+    });
+
+    test("and nothing at all for a street that isn't one", () => {
+        assert.equal(Betting.boardCardsFor("showdown"), 0);
+        assert.equal(Betting.boardCardsFor(undefined), 0);
+    });
+
+    test("it matches the streets the betting engine actually deals", () => {
+        // If a street is ever added, this is the table that has to hear about it —
+        // a street with no entry silently draws no cards.
+        Betting.STREETS.forEach(street => {
+            assert.ok(Object.prototype.hasOwnProperty.call(Betting.BOARD_CARDS, street),
+                `${street} has no card count`);
+        });
+    });
+});
+
+describe("clearing it", () => {
+    test("the next hand takes the board with it", () => {
+        const state = dealt();
+        state.board = ["AH", "KD", "9C", "4S", "2H"];
+        HoldemState.startNextHand(state);
+        assert.deepEqual(state.board, ["", "", "", "", ""]);
+    });
+
+    test("so does a misdeal", () => {
+        const state = dealt();
+        state.board = ["AH", "KD", "9C", "", ""];
+        Betting.takeHandBack(state);
+        assert.deepEqual(state.board, ["", "", "", "", ""]);
+    });
+
+    test("and so does dealing a hand, however the table got here", () => {
+        // Dealing twice off the same button — after a misdeal, or a table set up and
+        // then dealt again — would otherwise rank the last hand's board against this
+        // hand's cards, and it would look exactly like a real answer.
+        const state = dealt();
+        state.board = ["AH", "KD", "9C", "4S", "2H"];
+        Betting.beginHand(state);
+        assert.deepEqual(state.board, ["", "", "", "", ""]);
+    });
+
+    test("but turning the tracking off does not", () => {
+        // A dealer who taps the toggle mid-hand can tap it back; wiping the flop on
+        // the way past would cost them cards they had already entered.
+        const state = dealt({ trackBoard: true });
+        state.board = ["AH", "KD", "9C", "", ""];
+        state.trackBoard = false;
+        assert.deepEqual(state.board, ["AH", "KD", "9C", "", ""]);
+    });
+});
+
+describe("a board off a file", () => {
+    const migrated = board => HoldemState.migrate({
+        seats: [HoldemState.makePlayer()], board,
+    }).board;
+
+    test("a table saved before boards existed gets an empty one, not tracked", () => {
+        const parsed = HoldemState.migrate({ seats: [HoldemState.makePlayer()] });
+        assert.deepEqual(parsed.board, ["", "", "", "", ""]);
+        assert.equal(parsed.trackBoard, false);
+    });
+
+    test("it is always five slots, however many arrived", () => {
+        assert.equal(migrated(["AH"]).length, 5);
+        assert.equal(migrated(["AH", "KD", "9C", "4S", "2H", "3D", "5S"]).length, 5);
+        assert.equal(migrated("not an array").length, 5);
+        assert.equal(migrated(null).length, 5);
+    });
+
+    test("anything that isn't a card becomes an empty slot", () => {
+        assert.deepEqual(migrated(["AH", "ZZ", 7, null, "10H"]), ["AH", "", "", "", ""]);
+    });
+
+    test("and a real one keeps its place, upper-cased", () => {
+        assert.deepEqual(migrated(["ah", " kd ", "9c", "", ""]), ["AH", "KD", "9C", "", ""]);
+    });
+
+    test("tracking is only on if the file said so, in so many words", () => {
+        const on = HoldemState.migrate({ seats: [HoldemState.makePlayer()], trackBoard: true });
+        const truthy = HoldemState.migrate({ seats: [HoldemState.makePlayer()], trackBoard: "yes" });
+        assert.equal(on.trackBoard, true);
+        assert.equal(truthy.trackBoard, false, "a string is a file that has been edited, not a yes");
+    });
+});

tests/holdem/designTokens.test.js

@@ -0,0 +1,158 @@
+// The stylesheet has to keep picking from the scales.
+//
+// Before the token block went in, holdem.css used 42 different spacing values, 31 font
+// sizes, 14 border-radii and 139 colours, and defined three custom properties. None of
+// that was anybody's decision — each number was chosen at the moment it was needed, and
+// the result read as crowded even where nothing overlapped, because a 0.42rem gap next
+// to a 0.45rem gap next to a 0.5rem gap gives the eye no rhythm to lock onto.
+//
+// A one-off cleanup would last until the next `padding: 0.35rem`. So the scales are
+// checked here instead: a new value has to be a step on one of them, or an exception
+// with a reason written next to it.
+//
+// This reads the stylesheet as text rather than through a CSS parser, on purpose — the
+// rest of this suite has no dependencies and runs in a second, and a regex over
+// declarations is enough to catch the thing that actually happens, which is somebody
+// (me) typing a number instead of reaching for a token.
+const test = require("node:test");
+const { describe } = require("node:test");
+const assert = require("node:assert");
+const fs = require("fs");
+const path = require("path");
+
+const CSS_PATH = path.join(__dirname, "..", "..", "JoshHeaps.Net", "wwwroot", "css", "casino", "holdem.css");
+const css = fs.readFileSync(CSS_PATH, "utf8");
+
+// Comments carry measurements and rationale — "measured at 342px tall on a phone" —
+// and none of that is a declaration. Strip them before looking for values.
+const declarations = css.replace(/\/\*[\s\S]*?\*\//g, "");
+
+// Everything after the token block. The :root block is where the raw values are
+// supposed to be.
+const body = declarations.slice(declarations.indexOf("#potBadgeAmount,"));
+
+const valuesIn = (properties, source = body) => {
+    const found = [];
+    // The lookbehind rejects a hyphen as well as a word character, so `scroll-padding`
+    // isn't read as `padding`. It caught the pot wizard's scroll-padding the moment
+    // that went in — correctly by its own rules, wrongly in substance: those two values
+    // are the measured heights of the dialog's sticky header and footer, a clearance
+    // like the action bar's, not a step in the spacing rhythm.
+    const re = new RegExp(`(?<![\\w-])(${properties.join("|")})\\s*:([^;}]*)`, "g");
+    let m;
+    while ((m = re.exec(source)) !== null) {
+        const [, prop, value] = m;
+        // calc() and env() do their own arithmetic on tokens; the tokens inside them
+        // are still checked, because the regex below looks at bare numbers only.
+        (value.match(/(?<![\w-])[0-9.]+(rem|px|em)\b/g) || []).forEach(v => found.push({ prop, value: v }));
+    }
+    return found;
+};
+
+describe("the spacing scale", () => {
+    // Hairlines and structural clearances, each one deliberate:
+    //   0px/1px/3px/4px/22px — borders, hairline offsets, and the 22px of room a seat
+    //                          puck's name gives the dealer button beside it.
+    //   5rem/9rem            — how far the page has to scroll past the fixed action
+    //                          bar. Not rhythm; clearance for a thing of a known size.
+    const ALLOWED = new Set(["0px", "1px", "2px", "3px", "4px", "22px", "5rem", "9rem"]);
+
+    test("every gap, margin and padding comes off the scale", () => {
+        const offenders = valuesIn(
+            ["padding", "padding-top", "padding-right", "padding-bottom", "padding-left",
+             "margin", "margin-top", "margin-right", "margin-bottom", "margin-left",
+             "gap", "row-gap", "column-gap"])
+            .filter(v => !ALLOWED.has(v.value));
+
+        assert.deepEqual(offenders, [],
+            "use var(--sp-0 … --sp-6), or add the value to ALLOWED with a reason");
+    });
+
+    test("and the scale is the one the tokens define", () => {
+        ["--sp-0", "--sp-1", "--sp-2", "--sp-3", "--sp-4", "--sp-5", "--sp-6"]
+            .forEach(token => assert.match(css, new RegExp(`\\${token}:`), `${token} is not defined`));
+    });
+});
+
+describe("the type scale", () => {
+    //   16px  — the size below which iOS zooms the page when a field is focused, so
+    //           every text input states it. Not a typographic choice.
+    //   1.05em, 0.44rem — relative bumps inside a component, and the shrink-to-fit for
+    //           a six-figure stack on a 58px seat puck.
+    const ALLOWED = new Set(["16px", "1.05em", "0.44rem"]);
+
+    test("every font-size comes off the scale", () => {
+        const offenders = valuesIn(["font-size"]).filter(v => !ALLOWED.has(v.value));
+        assert.deepEqual(offenders, [],
+            "use var(--fs-2xs … --fs-2xl), or add the value to ALLOWED with a reason");
+    });
+
+    test("and the scale is the one the tokens define", () => {
+        ["--fs-2xs", "--fs-xs", "--fs-sm", "--fs-base", "--fs-md", "--fs-lg", "--fs-xl", "--fs-2xl"]
+            .forEach(token => assert.match(css, new RegExp(`\\${token}:`), `${token} is not defined`));
+    });
+});
+
+describe("the corner scale", () => {
+    test("every border-radius comes off it", () => {
+        const offenders = valuesIn(["border-radius"]).filter(v => v.value !== "2px");
+        assert.deepEqual(offenders, [], "use var(--r-sm | --r-md | --r-lg | --r-pill)");
+    });
+});
+
+describe("the tokens themselves", () => {
+    test("every var() the stylesheet reaches for exists", () => {
+        const used = new Set((css.match(/var\((--[a-zA-Z0-9-]+)/g) || []).map(s => s.slice(4)));
+        // Published from JS at runtime rather than declared here — see actionBar.js for
+        // the first two, render.js for the seat width and the chip's denomination colour,
+        // which is picked per chip from the amount on it.
+        const fromScript = new Set([
+            "--actionBarHeight", "--feltRoomPx", "--seatPuckW", "--betChipColour",
+        ]);
+        // Defined by the site's own stylesheet, not this one.
+        const fromSite = new Set(["--font-mono"]);
+        const missing = [...used].filter(t =>
+            !fromScript.has(t) && !fromSite.has(t) && !new RegExp(`\\${t}\\s*:`).test(css));
+        assert.deepEqual(missing, []);
+    });
+
+    test("and the dark surfaces stay off pure black", () => {
+        // Shadows are invisible on #000, so depth has to come from a lighter surface —
+        // and white text on true black is the pairing that smears on an OLED phone,
+        // which is what this tool is used on.
+        const surfaces = [...css.matchAll(/--surface-\d+:\s*(#[0-9a-fA-F]{3,8})/g)].map(m => m[1].toLowerCase());
+        assert.ok(surfaces.length >= 4, "the surface ramp should have four steps");
+        surfaces.forEach(hex => {
+            assert.notEqual(hex, "#000");
+            assert.notEqual(hex, "#000000");
+        });
+    });
+});
+
+describe("numbers that change while the dealer watches", () => {
+    test("are set in tabular figures", () => {
+        // A "1" is narrower than an "8" in a proportional face, so a pot ticking from
+        // $18 to $21 shuffles its digits sideways and the eye reads movement where
+        // there is only arithmetic.
+        assert.match(css, /font-variant-numeric:\s*tabular-nums/);
+        ["#potBadgeAmount", ".seatPuckStack", ".actionPot", ".standingsRow"]
+            .forEach(sel => assert.ok(css.includes(sel),
+                `${sel} should be in the tabular-figures list`));
+    });
+});
+
+describe("controls that must not resize under a thumb", () => {
+    test("the undo pill has a fixed width, not a content width", () => {
+        // It names the last action, so its label changes on every tap. Measured across
+        // one hand before this: 148, 110, 109, 108, 109, 110, 120.
+        const rule = css.match(/\.actionUndo\s*\{[^}]*\}/);
+        assert.ok(rule, ".actionUndo rule not found");
+        assert.match(rule[0], /width:\s*[0-9.]+rem/);
+    });
+
+    test("and its label is the part that ellipsizes", () => {
+        const rule = css.match(/\.actionUndoLabel\s*\{[^}]*\}/);
+        assert.ok(rule, ".actionUndoLabel rule not found");
+        assert.match(rule[0], /text-overflow:\s*ellipsis/);
+    });
+});

tests/holdem/feltRoom.test.js

@@ -71,3 +71,74 @@ describe("a window with room to spare", () => {
         assert.equal(room(844.4, 127.6, 223.2) % 1, 0);
     });
 });
+
+describe("when the felt changes size, so does everything on it", () => {
+    // The felt is published as a custom property and the seats follow it on their own,
+    // because they are placed in percentages of it. A bet chip anchored inside a seat
+    // is not: it is placed in pixels off that seat's centre, worked out against the felt
+    // as it was, and nothing re-ran that when the felt moved.
+    //
+    // Measured at 640x360 with five players, on a felt that lost 23px of height: the
+    // chips were positioned for the old one and painted on the new, and seat 3's $5 came
+    // to rest on seat 4's puck. The seat pucks had the same problem — they are searched
+    // for a width the felt no longer has — and two pairs of them were overlapping.
+    //
+    // A side pot appearing does this too: it is 55px off the felt, mid-hand.
+    const fakeDom = (windowHeight = 900) => {
+        const root = { style: { values: {},
+            setProperty(k, v) { this.values[k] = v; },
+            getPropertyValue(k) { return this.values[k] || ""; } } };
+        global.document = {
+            documentElement: root,
+            getElementById: id => (id === "pokerTableWrap"
+                ? { getBoundingClientRect: () => ({ top: 146 }) }
+                : null),
+        };
+        global.window = { innerHeight: windowHeight, scrollY: 0 };
+        return root;
+    };
+
+    const spyRender = () => {
+        const calls = { density: 0, seats: 0 };
+        global.Render = {
+            state: {},
+            updateSeatDensity() { calls.density++; },
+            renderSeats() { calls.seats++; },
+        };
+        return calls;
+    };
+
+    test.afterEach(() => {
+        delete global.document;
+        delete global.window;
+        delete global.Render;
+    });
+
+    test("a felt that changed height lays the seats out again", () => {
+        fakeDom();
+        const calls = spyRender();
+        ActionBar.publishFeltRoom(117);
+        assert.equal(calls.seats, 1, "first publish sets a height it did not have");
+
+        ActionBar.publishFeltRoom(172);
+        assert.equal(calls.seats, 2, "and a side pot grew the bar by 55px");
+    });
+
+    test("a felt that did not change is left alone", () => {
+        fakeDom();
+        const calls = spyRender();
+        ActionBar.publishFeltRoom(117);
+        ActionBar.publishFeltRoom(117);
+        ActionBar.publishFeltRoom(117);
+        assert.equal(calls.seats, 1,
+            "re-laying the table on every resize tick would be work for nothing");
+        assert.equal(calls.density, 3, "the cheap re-measure still runs each time");
+    });
+
+    test("it publishes the height whether or not anything is listening", () => {
+        const root = fakeDom();
+        delete global.Render;
+        assert.doesNotThrow(() => ActionBar.publishFeltRoom(117));
+        assert.match(root.style.getPropertyValue("--feltRoomPx"), /^\d+px$/);
+    });
+});

tests/holdem/handRank.test.js

@@ -0,0 +1,415 @@
+// What beats what.
+//
+// This is the one module in the tool that can pay the wrong person without anybody
+// noticing. A pot wizard that asks "who won?" is only ever as wrong as the dealer
+// ticking the box; one that answers the question itself is wrong on its own account,
+// silently, and a table that has come to trust it will not check. So the ordering is
+// tested from both ends: every category against every other, the traps that a
+// hand-rolled evaluator gets wrong (the wheel, the counterfeited kicker, the board
+// that plays), and then a lot of random seven-card hands against a second
+// implementation written a different way.
+const test = require("node:test");
+const { describe } = require("node:test");
+const assert = require("node:assert");
+const { HandRank } = require("./load");
+
+const best = (...cards) => HandRank.best(cards.flat());
+const beats = (a, b) => HandRank.compare(a, b) > 0;
+const chops = (a, b) => HandRank.compare(a, b) === 0;
+
+// A hand written as one string, for the tests that list a lot of them.
+const hand = str => str.trim().split(/\s+/);
+
+describe("the five-card hand", () => {
+    const of = str => HandRank.evaluate5(hand(str));
+
+    test("each category is recognised", () => {
+        const cases = [
+            ["AS KS QS JS TS", HandRank.STRAIGHT_FLUSH, "Royal flush"],
+            ["9H 8H 7H 6H 5H", HandRank.STRAIGHT_FLUSH, "Straight flush, nine high"],
+            ["7C 7D 7H 7S 2C", HandRank.FOUR_OF_A_KIND, "Four of a kind, sevens"],
+            ["KC KD KH 3S 3C", HandRank.FULL_HOUSE, "Full house, kings full of threes"],
+            ["AD JD 8D 5D 2D", HandRank.FLUSH, "Flush, ace high"],
+            ["9C 8D 7H 6S 5C", HandRank.STRAIGHT, "Straight, nine high"],
+            ["QC QD QH 7S 2C", HandRank.THREE_OF_A_KIND, "Three of a kind, queens"],
+            ["AC AD 8H 8S 4C", HandRank.TWO_PAIR, "Two pair, aces and eights"],
+            ["QC QD 9H 5S 2C", HandRank.ONE_PAIR, "Pair of queens"],
+            ["AC JD 8H 5S 2C", HandRank.HIGH_CARD, "Ace high"],
+        ];
+        cases.forEach(([cards, category, described]) => {
+            const ranked = of(cards);
+            assert.equal(ranked.category, category, cards);
+            assert.equal(HandRank.describe(ranked), described, cards);
+        });
+    });
+
+    test("the categories are in the right order", () => {
+        // Listed weakest first, then every pair checked both ways round. The order
+        // itself is the thing being asserted, so it is written out rather than derived
+        // from the constants it is meant to be checking.
+        const ladder = [
+            "AC JD 8H 5S 2C",   // ace high
+            "2C 2D 8H 5S 3C",   // a pair of deuces
+            "AC AD 8H 8S 4C",   // two pair
+            "3C 3D 3H 7S 2C",   // trips
+            "9C 8D 7H 6S 5C",   // straight
+            "AD JD 8D 5D 2D",   // flush
+            "2C 2D 2H 3S 3C",   // full house
+            "7C 7D 7H 7S 2C",   // quads
+            "9H 8H 7H 6H 5H",   // straight flush
+        ];
+        for (let low = 0; low < ladder.length; low++) {
+            for (let high = low + 1; high < ladder.length; high++) {
+                assert.ok(beats(of(ladder[high]), of(ladder[low])),
+                    `${ladder[high]} should beat ${ladder[low]}`);
+                assert.ok(!beats(of(ladder[low]), of(ladder[high])),
+                    `${ladder[low]} should not beat ${ladder[high]}`);
+            }
+        }
+    });
+
+    test("the same hand in different suits is a chop", () => {
+        assert.ok(chops(of("AC KD 8H 5S 2C"), of("AS KH 8D 5C 2H")));
+        assert.ok(chops(of("QC QD 9H 5S 2C"), of("QH QS 9D 5C 2H")));
+    });
+
+    test("nothing but five real cards is a hand", () => {
+        assert.equal(HandRank.evaluate5(hand("AC KD 8H 5S")), null, "four");
+        assert.equal(HandRank.evaluate5(hand("AC KD 8H 5S 2C 3D")), null, "six");
+        assert.equal(HandRank.evaluate5(["AC", "KD", "8H", "5S", "ZZ"]), null, "nonsense");
+        assert.equal(HandRank.evaluate5(["AC", "KD", "8H", "5S", ""]), null, "an empty slot");
+        assert.equal(HandRank.evaluate5(null), null);
+    });
+
+    test("case does not matter, because the picker is not the only way cards get in", () => {
+        assert.ok(chops(of("ac kd 8h 5s 2c"), of("AC KD 8H 5S 2C")));
+    });
+});
+
+describe("the wheel", () => {
+    const of = str => HandRank.evaluate5(hand(str));
+
+    // The straight that runs through the ace at the bottom. An evaluator that scores a
+    // straight by its top card and lets the ace keep its usual value makes A-2-3-4-5
+    // the best straight there is instead of the worst — which pays the wrong player
+    // every time it comes up against a broadway.
+    test("A-2-3-4-5 is a straight", () => {
+        assert.equal(of("AC 2D 3H 4S 5C").category, HandRank.STRAIGHT);
+        assert.equal(HandRank.describe(of("AC 2D 3H 4S 5C")), "Straight, five high");
+    });
+
+    test("and it is the lowest one", () => {
+        assert.ok(beats(of("6C 5D 4H 3S 2C"), of("AC 2D 3H 4S 5C")),
+            "six high beats the wheel");
+        assert.ok(beats(of("AC KD QH JS TC"), of("AC 2D 3H 4S 5C")),
+            "broadway beats the wheel");
+    });
+
+    test("the same goes for a steel wheel", () => {
+        const steel = of("AH 2H 3H 4H 5H");
+        assert.equal(steel.category, HandRank.STRAIGHT_FLUSH);
+        assert.equal(HandRank.describe(steel), "Straight flush, five high");
+        assert.ok(beats(of("6H 5H 4H 3H 2H"), steel));
+        assert.ok(beats(of("AS KS QS JS TS"), steel));
+    });
+
+    test("an ace with a four-card run either side of it is not a straight", () => {
+        // K-A-2-3-4 does not wrap.
+        assert.equal(of("KC AD 2H 3S 4C").category, HandRank.HIGH_CARD);
+        assert.equal(of("QC KD AH 2S 3C").category, HandRank.HIGH_CARD);
+    });
+});
+
+describe("kickers", () => {
+    const of = str => HandRank.evaluate5(hand(str));
+
+    test("the fifth card settles two pair", () => {
+        assert.ok(beats(of("AC AD 8H 8S KC"), of("AH AS 8D 8C QC")));
+        assert.ok(chops(of("AC AD 8H 8S KC"), of("AH AS 8D 8C KD")));
+    });
+
+    test("and a pair, three kickers deep", () => {
+        assert.ok(beats(of("9C 9D AH 7S 2C"), of("9H 9S KD 7C 2D")), "first kicker");
+        assert.ok(beats(of("9C 9D AH 7S 2C"), of("9H 9S AD 6C 2D")), "second");
+        assert.ok(beats(of("9C 9D AH 7S 3C"), of("9H 9S AD 7C 2D")), "third");
+    });
+
+    test("the higher pair beats the better kicker", () => {
+        assert.ok(beats(of("TC TD 3H 2S 4C"), of("9C 9D AH KS QC")));
+    });
+
+    test("two pair is read by its higher pair first", () => {
+        assert.ok(beats(of("KC KD 2H 2S 3C"), of("QC QD JH JS AC")));
+    });
+
+    test("a full house is read by its trips first", () => {
+        assert.ok(beats(of("3C 3D 3H 2S 2C"), of("2D 2H 2C AS AD")));
+    });
+
+    test("a flush is read all the way down", () => {
+        assert.ok(beats(of("AD KD 8D 5D 3D"), of("AH KH 8H 5H 2H")));
+        assert.ok(chops(of("AD KD 8D 5D 3D"), of("AH KH 8H 5H 3H")));
+    });
+});
+
+describe("the best five out of seven", () => {
+    test("it finds the flush the two hole cards do not both belong to", () => {
+        const ranked = best(["AH", "2C"], ["KH", "QH", "7H", "3H", "9S"]);
+        assert.equal(ranked.category, HandRank.FLUSH);
+        assert.equal(HandRank.describe(ranked), "Flush, ace high");
+    });
+
+    test("it plays the board when the board is better than anything you hold", () => {
+        // Nobody can beat a royal on the board; both hands are the board itself.
+        const board = ["AS", "KS", "QS", "JS", "TS"];
+        const one = best(["2C", "3D"], board);
+        const two = best(["7H", "8H"], board);
+        assert.equal(HandRank.describe(one), "Royal flush");
+        assert.ok(chops(one, two), "the board plays, so it is a chop");
+    });
+
+    test("it does not use three hole cards, because there are only two", () => {
+        const ranked = best(["AC", "AD"], ["AH", "KS", "KD", "2C", "3H"]);
+        assert.equal(HandRank.describe(ranked), "Full house, aces full of kings");
+    });
+
+    test("a counterfeited two pair falls back on the board", () => {
+        // Holding 6-5 on a board of 8-8-K-K-A: the player's two pair is counterfeited —
+        // the best five are the board's two pair with the ace, and the sixes never play.
+        const ranked = best(["6C", "5D"], ["8H", "8S", "KC", "KD", "AC"]);
+        assert.equal(HandRank.describe(ranked), "Two pair, kings and eights");
+        assert.deepEqual([...ranked.cards].sort(), ["8H", "8S", "AC", "KC", "KD"].sort());
+    });
+
+    test("five, six and seven cards are all rankable; four is not", () => {
+        assert.ok(best(["AC", "KD"], ["QH", "JS", "TC"]), "five");
+        assert.ok(best(["AC", "KD"], ["QH", "JS", "TC", "2D"]), "six");
+        assert.ok(best(["AC", "KD"], ["QH", "JS", "TC", "2D", "3H"]), "seven");
+        assert.equal(best(["AC", "KD"], ["QH", "JS"]), null, "four");
+    });
+
+    test("the same card twice is not a hand", () => {
+        // A mis-entry, not a monster. Ranking it would mean ranking a deck that does
+        // not exist, and the answer would look exactly as confident as a real one.
+        assert.equal(best(["AC", "AC"], ["KH", "QS", "JD", "2C", "3H"]), null);
+        assert.equal(best(["AC", "KD"], ["KD", "QS", "JD", "2C", "3H"]), null);
+    });
+});
+
+describe("the showdown", () => {
+    const board = ["AH", "KD", "9C", "4S", "2H"];
+
+    test("it names one winner and says how it read every hand", () => {
+        const result = HandRank.showdown(board, [
+            { id: "a", cards: ["AS", "KC"] },   // two pair, aces and kings
+            { id: "b", cards: ["9H", "9S"] },   // trips
+            { id: "c", cards: ["AC", "3D"] },   // a pair of aces
+        ]);
+        assert.equal(result.complete, true);
+        assert.deepEqual(result.winners, ["b"]);
+        assert.deepEqual(result.players.map(p => p.place), [2, 1, 3]);
+        assert.deepEqual(result.players.map(p => HandRank.describe(p.hand)), [
+            "Two pair, aces and kings",
+            "Three of a kind, nines",
+            "Pair of aces",
+        ]);
+    });
+
+    test("a chop comes back as two winners on the same place", () => {
+        const result = HandRank.showdown(board, [
+            { id: "a", cards: ["AS", "QC"] },
+            { id: "b", cards: ["AD", "QH"] },
+            { id: "c", cards: ["7H", "5S"] },
+        ]);
+        assert.deepEqual(result.winners.sort(), ["a", "b"]);
+        assert.deepEqual(result.players.map(p => p.place), [1, 1, 3],
+            "and the player behind them is third, not second");
+    });
+
+    test("an incomplete board is not a showdown", () => {
+        const result = HandRank.showdown(["AH", "KD", "9C"], [
+            { id: "a", cards: ["AS", "KC"] },
+            { id: "b", cards: ["9H", "9S"] },
+        ]);
+        assert.equal(result.boardComplete, false);
+        assert.equal(result.complete, false);
+        assert.deepEqual(result.winners, [], "it must not name one");
+    });
+
+    test("one unknown hand and it will not say who won", () => {
+        // Who is *ahead* and who *won* are different claims, and the second one is the
+        // only one that can pay somebody. A player who never turned their cards over
+        // could have anything.
+        const result = HandRank.showdown(board, [
+            { id: "a", cards: ["AS", "KC"] },
+            { id: "b", cards: ["", ""] },
+        ]);
+        assert.equal(result.complete, false);
+        assert.deepEqual(result.winners, []);
+        assert.deepEqual(result.unknown, ["b"]);
+        assert.ok(result.players[0].hand, "but it still reads the hands it can");
+        assert.equal(result.players[1].hand, null);
+    });
+
+    test("a card in two places stops it dead", () => {
+        const result = HandRank.showdown(board, [
+            { id: "a", cards: ["AH", "KC"] },   // the ace of hearts is on the board
+            { id: "b", cards: ["9H", "9S"] },
+        ]);
+        assert.equal(result.duplicated, true);
+        assert.equal(result.complete, false);
+        assert.deepEqual(result.winners, []);
+    });
+
+    test("no players at all is not a showdown either", () => {
+        const result = HandRank.showdown(board, []);
+        assert.equal(result.complete, false);
+        assert.deepEqual(result.winners, []);
+    });
+});
+
+describe("against a second opinion", () => {
+    // The strongest check available without a poker library: a completely separate
+    // ranking, written the other way round — every hand turned into a sortable string
+    // by counting ranks, with no shared helper between it and the module. Where the two
+    // disagree, one of them is wrong and the test says which hand it was.
+    const VALUE = { T: 10, J: 11, Q: 12, K: 13, A: 14 };
+    const val = card => VALUE[card[0]] || Number(card[0]);
+
+    function reference5(cards) {
+        const ranks = cards.map(val).sort((x, y) => y - x);
+        const suited = new Set(cards.map(c => c[1])).size === 1;
+
+        const tally = {};
+        ranks.forEach(r => { tally[r] = (tally[r] || 0) + 1; });
+        const bySize = Object.keys(tally)
+            .map(Number)
+            .sort((x, y) => tally[y] - tally[x] || y - x);
+        const sizes = bySize.map(r => tally[r]).join("");
+
+        const unique = [...new Set(ranks)];
+        let run = null;
+        if (unique.length === 5) {
+            if (unique[0] - unique[4] === 4) run = unique[0];
+            else if (unique.join(",") === "14,5,4,3,2") run = 5;
+        }
+
+        // [category, ...tiebreak], the same shape, arrived at from the group sizes
+        // rather than from a chain of ifs about shape[0] and shape[1].
+        if (suited && run !== null) return [8, run];
+        if (sizes === "41") return [7, ...bySize];
+        if (sizes === "32") return [6, ...bySize];
+        if (suited) return [5, ...ranks];
+        if (run !== null) return [4, run];
+        if (sizes === "311") return [3, ...bySize];
+        if (sizes === "221") return [2, ...bySize];
+        if (sizes === "2111") return [1, ...bySize];
+        return [0, ...ranks];
+    }
+
+    const worse = (a, b) => {
+        for (let i = 0; i < Math.max(a.length, b.length); i++) {
+            if ((a[i] || 0) !== (b[i] || 0)) return (a[i] || 0) < (b[i] || 0);
+        }
+        return false;
+    };
+
+    function referenceBest(cards) {
+        let top = null;
+        for (let a = 0; a < 3; a++) for (let b = a + 1; b < 4; b++)
+            for (let c = b + 1; c < 5; c++) for (let d = c + 1; d < 6; d++)
+                for (let e = d + 1; e < 7; e++) {
+                    const score = reference5([cards[a], cards[b], cards[c], cards[d], cards[e]]);
+                    if (!top || worse(top, score)) top = score;
+                }
+        return top;
+    }
+
+    // A deck, and a deterministic shuffle — a failure has to be reproducible or it is
+    // just an anecdote about one CI run.
+    const DECK = [];
+    "23456789TJQKA".split("").forEach(r => "SHDC".split("").forEach(s => DECK.push(r + s)));
+
+    let seed = 20260904;
+    const random = () => {
+        seed = (seed * 1103515245 + 12345) & 0x7fffffff;
+        return seed / 0x7fffffff;
+    };
+
+    test("twenty thousand random seven-card hands rank the same way", () => {
+        const deck = [...DECK];
+        for (let round = 0; round < 20000; round++) {
+            // Seven off the top of a shuffle of only what it needs.
+            const drawn = [];
+            for (let i = 0; i < 7; i++) {
+                const pick = i + Math.floor(random() * (deck.length - i));
+                [deck[i], deck[pick]] = [deck[pick], deck[i]];
+                drawn.push(deck[i]);
+            }
+            const mine = HandRank.best(drawn);
+            const theirs = referenceBest(drawn);
+            assert.deepEqual([mine.category, ...mine.tiebreak], theirs,
+                `disagreed on ${drawn.join(" ")}`);
+        }
+    });
+
+    test("and four thousand random pairs of hands are ordered the same way", () => {
+        // The categories agreeing is not the same as the comparison agreeing: this is
+        // the half that pays somebody.
+        const deck = [...DECK];
+        for (let round = 0; round < 4000; round++) {
+            const drawn = [];
+            for (let i = 0; i < 9; i++) {
+                const pick = i + Math.floor(random() * (deck.length - i));
+                [deck[i], deck[pick]] = [deck[pick], deck[i]];
+                drawn.push(deck[i]);
+            }
+            const board = drawn.slice(0, 5);
+            const one = [...board, drawn[5], drawn[6]];
+            const two = [...board, drawn[7], drawn[8]];
+
+            const mine = Math.sign(HandRank.compare(HandRank.best(one), HandRank.best(two)));
+            const a = referenceBest(one), b = referenceBest(two);
+            const theirs = worse(a, b) ? -1 : worse(b, a) ? 1 : 0;
+            assert.equal(mine, theirs,
+                `board ${board.join(" ")}: ${drawn[5]}${drawn[6]} vs ${drawn[7]}${drawn[8]}`);
+        }
+    });
+});
+
+describe("saying it in a sentence", () => {
+    // The verdict line reads "Alexandra wins $25 with ___", so the phrase has to carry
+    // its own article. Which article is not something the first letter can answer: it
+    // is "a full house" and "a flush", but "four of a kind" and "two pair" take none,
+    // and nobody has ever said "an ace high".
+    test("every category takes the article a table would give it", () => {
+        const said = str => HandRank.phrase(HandRank.evaluate5(hand(str)));
+        assert.deepEqual([
+            said("AS KS QS JS TS"),
+            said("9H 8H 7H 6H 5H"),
+            said("7C 7D 7H 7S 2C"),
+            said("KC KD KH 3S 3C"),
+            said("AD JD 8D 5D 2D"),
+            said("9C 8D 7H 6S 5C"),
+            said("QC QD QH 7S 2C"),
+            said("AC AD 8H 8S 4C"),
+            said("QC QD 9H 5S 2C"),
+            said("AC JD 8H 5S 2C"),
+        ], [
+            "a royal flush",
+            "a straight flush, nine high",
+            "four of a kind, sevens",
+            "a full house, kings full of threes",
+            "a flush, ace high",
+            "a straight, nine high",
+            "three of a kind, queens",
+            "two pair, aces and eights",
+            "a pair of queens",
+            "ace high",
+        ]);
+    });
+
+    test("with nothing to say it says something", () => {
+        assert.equal(HandRank.phrase(null), "the best hand");
+    });
+});

tests/holdem/load.js

@@ -15,6 +15,7 @@ function load(name) {
 }
 
 const Money = load("money");
+const HandRank = load("handRank");
 const PotCalculator = load("potCalculator");
 const SplitCalculator = load("splitCalculator");
 const Betting = load("betting");
@@ -33,6 +34,10 @@ globalThis.HoldemState = HoldemState;
 // its methods reach for both. Loaded last so nothing it touches is still undefined.
 const ActionBar = load("actionBar");
 
+// The picker's slot bookkeeping — which slots it is filling, what is already dealt —
+// is plain state work and testable here. Only render() and open() touch the DOM.
+const CardPicker = load("cardPicker");
+
 // ---- Table builders ----------------------------------------------------------
 
 // A table with `seatCount` chairs where `seated` (a list of seat indexes) are dealt
@@ -78,6 +83,7 @@ function contributions(state) {
 
 module.exports = {
     Money,
+    HandRank,
     Render,
     HandHistoryExport,
     PotCalculator,
@@ -86,6 +92,7 @@ module.exports = {
     Betting,
     HoldemState,
     ActionBar,
+    CardPicker,
     makeTable,
     chipsOnTable,
     contributions,

tests/holdem/readingTheShowdown.test.js

@@ -0,0 +1,274 @@
+// Reading the showdown: the picker that takes the cards in, and the pot maths that
+// pays somebody on the strength of them.
+//
+// The evaluator is tested to death next door in handRank.test.js. What is tested here
+// is the join — the two places where a correct ranking can still pay the wrong player:
+// a side pot, where the question is asked again of a smaller field, and a card entered
+// in two places, where the ranking is of a deck that does not exist.
+const test = require("node:test");
+const { describe } = require("node:test");
+const assert = require("node:assert");
+const { CardPicker, HandRank, HoldemState, PotCalculator, Money, makeTable } = require("./load");
+
+describe("the picker, filling the board", () => {
+    const table = (board = ["", "", "", "", ""]) => {
+        const state = makeTable({ seatCount: 4, stack: 100 });
+        state.board = board;
+        return state;
+    };
+
+    // The sheet's own drawing is the DOM's business; the slot bookkeeping under it is
+    // not, and that is what these ask about.
+    const headless = () => { CardPicker.render = () => {}; CardPicker.close = () => {}; };
+
+    const onBoard = (state, cardIndex = 0, limit = 5) => {
+        headless();
+        CardPicker.state = state;
+        CardPicker.hooks = { onPick: () => {} };
+        CardPicker.target = "board";
+        CardPicker.boardLimit = limit;
+        CardPicker.cardIndex = cardIndex;
+        return CardPicker;
+    };
+
+    const onSeat = (state, seatIndex, cardIndex = 0) => {
+        headless();
+        CardPicker.state = state;
+        CardPicker.hooks = { onPick: () => {} };
+        CardPicker.target = "seat";
+        CardPicker.seatIndex = seatIndex;
+        CardPicker.cardIndex = cardIndex;
+        return CardPicker;
+    };
+
+    test("it only offers the slots that have actually been turned", () => {
+        const state = table();
+        assert.equal(onBoard(state, 0, 3).slotValues().length, 3, "the flop is three");
+        assert.equal(onBoard(state, 0, 4).slotValues().length, 4, "the turn makes four");
+        assert.equal(onBoard(state, 0, 5).slotValues().length, 5);
+    });
+
+    test("a card goes into the board, not into a seat", () => {
+        const state = table();
+        onBoard(state, 1, 3).assign("KD");
+        assert.deepEqual(state.board, ["", "KD", "", "", ""]);
+        assert.deepEqual(state.seats[0].cards, ["", ""]);
+    });
+
+    test("it walks on to the next empty slot, so a flop is three taps", () => {
+        const state = table();
+        const picker = onBoard(state, 0, 3);
+        picker.assign("KD");
+        assert.equal(picker.cardIndex, 1);
+        picker.assign("9C");
+        assert.equal(picker.cardIndex, 2);
+        picker.assign("4S");
+        assert.deepEqual(state.board.slice(0, 3), ["KD", "9C", "4S"]);
+    });
+
+    test("and it does not walk past the slots it was given", () => {
+        // Filling the turn must not run on to the river: that card is still in the deck.
+        const state = table(["KD", "9C", "4S", "", ""]);
+        const picker = onBoard(state, 3, 4);
+        picker.assign("AH");
+        assert.deepEqual(state.board, ["KD", "9C", "4S", "AH", ""]);
+    });
+
+    test("clearing a slot empties that one and nothing else", () => {
+        const state = table(["KD", "9C", "4S", "", ""]);
+        onBoard(state, 1, 3).clearSlot();
+        assert.deepEqual(state.board, ["KD", "", "4S", "", ""]);
+    });
+});
+
+describe("a card cannot be in two places", () => {
+    const table = () => {
+        const state = makeTable({ seatCount: 4, stack: 100 });
+        state.board = ["KD", "9C", "4S", "", ""];
+        state.seats[0].cards = ["AH", "AS"];
+        state.seats[1].cards = ["", ""];
+        return state;
+    };
+
+    const picker = (state, setup) => {
+        CardPicker.state = state;
+        CardPicker.hooks = { onPick: () => {} };
+        setup();
+        return CardPicker;
+    };
+
+    test("the board's cards are unavailable to a hand", () => {
+        // The half of this that did not exist before: the picker knew about every
+        // player's cards and nothing about the middle of the table, so the king of
+        // diamonds on the flop could be dealt to somebody as well.
+        const state = table();
+        const used = picker(state, () => {
+            CardPicker.target = "seat";
+            CardPicker.seatIndex = 1;
+            CardPicker.cardIndex = 0;
+        }).cardsInUse();
+        assert.equal(used.get("KD"), "the board");
+        assert.equal(used.get("9C"), "the board");
+        assert.ok(used.has("AH"), "and the other players' are still in there");
+    });
+
+    test("and a hand's cards are unavailable to the board", () => {
+        const state = table();
+        const used = picker(state, () => {
+            CardPicker.target = "board";
+            CardPicker.boardLimit = 5;
+            CardPicker.cardIndex = 3;
+        }).cardsInUse();
+        assert.ok(used.has("AH"), "the ace in seat 1's hand");
+        assert.ok(used.has("KD"), "and the rest of the board");
+    });
+
+    test("but the slot being edited is not competing with itself", () => {
+        const state = table();
+        const used = picker(state, () => {
+            CardPicker.target = "board";
+            CardPicker.boardLimit = 3;
+            CardPicker.cardIndex = 0;
+        }).cardsInUse();
+        assert.equal(used.has("KD"), false, "re-picking what is already there is not a duplicate");
+        assert.equal(used.has("9C"), true);
+    });
+
+    test("the same goes for a seat's own slot", () => {
+        const state = table();
+        const used = picker(state, () => {
+            CardPicker.target = "seat";
+            CardPicker.seatIndex = 0;
+            CardPicker.cardIndex = 0;
+        }).cardsInUse();
+        assert.equal(used.has("AH"), false);
+        assert.equal(used.has("AS"), true, "the player's other card still is");
+    });
+});
+
+describe("who wins a side pot", () => {
+    // The case a single ranking of the whole table gets wrong. The best hand takes the
+    // main pot and is then out of the question: the side pot goes to the best of who is
+    // left, which is a different answer, and it is the answer the wizard has to ask for
+    // again each round rather than reading off one list.
+    //
+    // Three players, and the short stack has the best hand of the three.
+    const board = ["KD", "9C", "4S", "AH", "2H"];
+    const hands = {
+        short: ["9H", "9S"],   // trips — the best hand at the table
+        big: ["AS", "KC"],     // two pair, aces and kings
+        other: ["AC", "3D"],   // a pair of aces
+    };
+
+    const entries = () => PotCalculator.buildEntries([
+        { id: "short", contribution: 20, folded: false },
+        { id: "big", contribution: 100, folded: false },
+        { id: "other", contribution: 100, folded: false },
+    ]);
+
+    // What the wizard does: read the hands still in for *this* round.
+    const readRound = state => HandRank.showdown(board,
+        PotCalculator.getContenders(state).map(c => ({ id: c.id, cards: hands[c.id] })));
+
+    test("the short stack takes the main pot with the best hand", () => {
+        const round = readRound(entries());
+        assert.equal(round.complete, true);
+        assert.deepEqual(round.winners, ["short"]);
+    });
+
+    test("and the side pot goes to the best of the two who are left", () => {
+        const first = PotCalculator.resolveRound(entries(), ["short"]);
+        // 20 from each of three: the short stack is paid 60 and is out of it.
+        assert.equal(Money.round(first.perWinner[0].amount), 60);
+
+        const round = readRound(first.updatedEntries);
+        assert.deepEqual(round.winners, ["big"],
+            "not the trips, which are no longer eligible for this money");
+        assert.equal(round.players.length, 2, "and the short stack is not even offered");
+
+        const second = PotCalculator.resolveRound(first.updatedEntries, ["big"]);
+        assert.equal(Money.round(second.perWinner[0].amount), 160);
+    });
+
+    test("every chip is accounted for across the two rounds", () => {
+        const first = PotCalculator.resolveRound(entries(), ["short"]);
+        const second = PotCalculator.resolveRound(first.updatedEntries, ["big"]);
+        assert.equal(Money.round(first.payoutPool + second.payoutPool), 220);
+        assert.equal(PotCalculator.getContenders(second.updatedEntries).length, 0);
+    });
+});
+
+describe("what it refuses to answer", () => {
+    const board = ["KD", "9C", "4S", "AH", "2H"];
+
+    test("a folded player's cards are not in the running", () => {
+        // They are not a contender, so they are never handed in — which means a hand
+        // that would have won cannot take a pot its owner threw away.
+        const table = PotCalculator.buildEntries([
+            { id: "live", contribution: 50, folded: false },
+            { id: "mucked", contribution: 50, folded: true },
+        ]);
+        const contenders = PotCalculator.getContenders(table);
+        assert.deepEqual(contenders.map(c => c.id), ["live"]);
+
+        const round = HandRank.showdown(board, contenders.map(c => ({ id: c.id, cards: ["7C", "5D"] })));
+        assert.deepEqual(round.winners, ["live"], "even holding nothing");
+    });
+
+    test("one hand not turned over and it hands the question back", () => {
+        const round = HandRank.showdown(board, [
+            { id: "a", cards: ["9H", "9S"] },
+            { id: "b", cards: ["", ""] },
+        ]);
+        assert.equal(round.complete, false);
+        assert.deepEqual(round.winners, []);
+        assert.deepEqual(round.unknown, ["b"]);
+    });
+
+    test("a board still being dealt is not a showdown", () => {
+        const round = HandRank.showdown(["KD", "9C", "4S", "", ""], [
+            { id: "a", cards: ["9H", "9S"] },
+            { id: "b", cards: ["AS", "KC"] },
+        ]);
+        assert.equal(round.complete, false);
+        assert.deepEqual(round.winners, []);
+    });
+
+    test("a card entered twice stops it rather than guessing", () => {
+        // The failure mode that matters: a mis-typed card produces a hand that ranks
+        // perfectly well and is not the one on the table. Refusing is the only honest
+        // answer, and the dealer picks the winner the way they always could.
+        const round = HandRank.showdown(board, [
+            { id: "a", cards: ["9H", "9S"] },
+            { id: "b", cards: ["KD", "QC"] },   // the king of diamonds is on the board
+        ]);
+        assert.equal(round.duplicated, true);
+        assert.equal(round.complete, false);
+        assert.deepEqual(round.winners, []);
+    });
+
+    test("and two players holding the same card is caught as well", () => {
+        const round = HandRank.showdown(board, [
+            { id: "a", cards: ["QH", "QS"] },
+            { id: "b", cards: ["QH", "JC"] },
+        ]);
+        assert.equal(round.duplicated, true);
+        assert.equal(round.complete, false);
+    });
+});
+
+describe("a table that never asked for any of this", () => {
+    test("has no board, tracks nothing, and reads no showdown", () => {
+        const state = HoldemState.createDefault();
+        assert.equal(state.trackBoard, false);
+        // Which is what the wizard checks before it reads anything at all — see
+        // readShowdown() in potWizard.js. With it off, an empty board would decline
+        // anyway, so the tool cannot start answering a question nobody asked.
+        const round = HandRank.showdown(state.board, [
+            { id: "a", cards: ["9H", "9S"] },
+            { id: "b", cards: ["AS", "KC"] },
+        ]);
+        assert.equal(round.complete, false);
+        assert.deepEqual(round.winners, []);
+    });
+});

tests/holdem/runUiAudit.js

@@ -34,26 +34,55 @@ const OVERLAYS = [
 ];
 
 // A table part-way through a real night: long names, uneven stacks, a hand log.
-function seedTable() {
+//
+// Two of them. The sweep ran on one table shape for its whole life — five players at a
+// nine-handed table — and how much room the felt has depends on how many people are
+// sitting down more than on anything a viewport can tell you. A full table is where the
+// seats crowd, and it was never once looked at: measured when it finally was, seventeen
+// of fifty-four tables had one player's name or stack painted under another.
+const TABLES = [
+    ["half full", 9, [["Christopher", 1200], ["Mike from work", 800], ["Alexandra", 1500],
+        ["Bob", 250], ["Dee", 600]], false],
+    ["packed", 12, [["Christopher", 1200], ["Mike from work", 800], ["Alexandra", 1500],
+        ["Bob", 250], ["Dee", 600], ["Sam", 900], ["Priya", 1100], ["Jonathan", 480],
+        ["Al", 2000], ["Katarzyna", 340], ["Ted", 760], ["Wes", 1300]]],
+    // The half-full table again, with the board turned on and five cards on it. The
+    // board is furniture in the middle of the felt that nothing else was ever measured
+    // against — and where the middle cannot hold it, it moves to a strip above the
+    // table and takes the felt's height with it, which is the seats' problem too.
+    //
+    // Deliberately the same five players in the same nine chairs as the shape above, so
+    // the only thing that differs between them is the board. A seventh player would have
+    // told us that a seven-handed table crowds, which is true, already known, and not
+    // what this row is here to find out.
+    ["board", 9, [["Christopher", 1200], ["Mike from work", 800], ["Alexandra", 1500],
+        ["Bob", 250], ["Dee", 600]], true],
+];
+
+function seedTable([seats, players, board]) {
     const state = HoldemState.createDefault();
-    HoldemState.resizeSeats(state, 9);
-    [["Christopher", 1200], ["Mike from work", 800], ["Alexandra", 1500], ["Bob", 250], ["Dee", 600]]
-        .forEach(([name, cash], i) => {
-            state.seats[i].name = name;
-            state.seats[i].inHand = true;
-            state.seats[i].buyIns = cash;
-            state.seats[i].stack = cash;
-        });
+    HoldemState.resizeSeats(state, seats);
+    players.forEach(([name, cash], i) => {
+        state.seats[i].name = name;
+        state.seats[i].inHand = true;
+        state.seats[i].buyIns = cash;
+        state.seats[i].stack = cash;
+    });
     state.smallBlind = 5;
     state.bigBlind = 10;
     for (let hand = 1; hand <= 4; hand++) {
         state.handHistory.push({
             handNumber: hand, timestamp: Date.now(), pot: 120,
-            payouts: [{ id: state.seats[hand % 5].id, label: "P", amount: 120 }],
-            ledger: state.seats.slice(0, 5).map(p => ({ id: p.id, name: p.name, in: 30, won: 0 })),
+            payouts: [{ id: state.seats[hand % players.length].id, label: "P", amount: 120 }],
+            ledger: state.seats.slice(0, players.length)
+                .map(p => ({ id: p.id, name: p.name, in: 30, won: 0 })),
         });
     }
     state.handNumber = 5;
+    if (board) {
+        state.trackBoard = true;
+        state.board = ["KD", "9C", "4S", "AH", "2H"];
+    }
     HoldemState.save(state);
 }
 
@@ -85,33 +114,75 @@ async function main() {
         const hand = touch ? "touch" : "mouse";
 
         for (const [width, height, rawLabel] of VIEWPORTS) {
-            const label = `${rawLabel} · ${hand}`;
             await page.setViewportSize({ width, height });
 
-            await page.goto(`${BASE}/Casino/TexasHoldem`);
-            await page.evaluate(seedTable);
-            await page.goto(`${BASE}/Casino/TexasHoldem`);
-            await page.click("#nextHandBtn");
-            await page.waitForTimeout(300);
-            await page.evaluate(() => document.querySelectorAll("details.sidebarPanel").forEach(d => { d.open = true; }));
-            await page.addScriptTag({ path: AUDIT });
-
-            const table = await page.evaluate(() => LayoutAudit.runAll());
-            checks++;
-            if (!table.clean) failures.push({ where: `table · ${label} ${width}x${height}`, offenders: table.failed.flatMap(f => f.offenders) });
-            table.warnings.forEach(w => warnings.push(`${label}: ${w}`));
-
-            for (const [name, open] of OVERLAYS) {
-                await page.evaluate(open);
-                await page.waitForTimeout(180);
-                checks++;
-                const result = await page.evaluate(() => {
-                    const overlay = [...document.querySelectorAll("[id$='Overlay']")].find(o => !o.hidden);
-                    if (!overlay) return { offenders: [] };
-                    return LayoutAudit.occlusion(`#${overlay.id} button, #${overlay.id} a, #${overlay.id} input`);
+            for (const [shape, seats, players, board] of TABLES) {
+                const label = `${rawLabel} · ${shape} · ${hand}`;
+                await page.goto(`${BASE}/Casino/TexasHoldem`);
+                await page.evaluate(seedTable, [seats, players, board]);
+                await page.goto(`${BASE}/Casino/TexasHoldem`);
+                await page.click("#nextHandBtn");
+                await page.waitForTimeout(300);
+                // Deal it, and get chips in front of a few players. The sweep used to
+                // audit a table that had never had a bet on it, so the bets layer was
+                // empty at every viewport it visited and nothing it drew was ever looked
+                // at — which is a third of what is on the felt during a hand.
+                await page.evaluate(() => {
+                    const deal = [...document.querySelectorAll("#actionBar button")]
+                        .find(b => /post blinds/i.test(b.textContent));
+                    if (deal) deal.click();
                 });
-                await page.evaluate(() => document.querySelectorAll("[id$='Overlay']").forEach(o => { o.hidden = true; }));
-                if (result.offenders.length) failures.push({ where: `${name} · ${label} ${width}x${height}`, offenders: result.offenders });
+                await page.waitForTimeout(220);
+                for (let i = 0; i < 3; i++) {
+                    const acted = await page.evaluate(() => {
+                        const btn = [...document.querySelectorAll("#actionBar button")]
+                            .find(b => /^(Call|Check)/.test(b.textContent.trim()) && !b.disabled);
+                        if (!btn) return false;
+                        btn.click();
+                        return true;
+                    });
+                    if (!acted) break;
+                    await page.waitForTimeout(120);
+                }
+                if (board) {
+                    await page.evaluate(() => {
+                        const s = HoldemState.load();
+                        s.street = "river";
+                        s.board = ["KD", "9C", "4S", "AH", "2H"];
+                        s.seats.filter(p => p.inHand).forEach((p, i) => {
+                            p.cards = [["AS", "KC"], ["AC", "3D"], ["9H", "9S"],
+                                       ["7C", "5D"], ["JD", "TS"]][i] || ["", ""];
+                        });
+                        HoldemState.save(s);
+                    });
+                    await page.reload();
+                    await page.waitForTimeout(300);
+                }
+                await page.evaluate(() => document.querySelectorAll("details.sidebarPanel").forEach(d => { d.open = true; }));
+                await page.addScriptTag({ path: AUDIT });
+
+                const table = await page.evaluate(() => LayoutAudit.runAll());
+                checks++;
+                if (!table.clean) failures.push({ where: `table · ${label} ${width}x${height}`, offenders: table.failed.flatMap(f => f.offenders) });
+                table.warnings.forEach(w => warnings.push(`${label}: ${w}`));
+
+                // The overlays don't change with the table's shape, so they are only
+                // opened over the half-full one — twice the viewports is enough time
+                // spent already.
+                if (shape !== TABLES[0][0]) continue;
+
+                for (const [name, open] of OVERLAYS) {
+                    await page.evaluate(open);
+                    await page.waitForTimeout(180);
+                    checks++;
+                    const result = await page.evaluate(() => {
+                        const overlay = [...document.querySelectorAll("[id$='Overlay']")].find(o => !o.hidden);
+                        if (!overlay) return { offenders: [] };
+                        return LayoutAudit.occlusion(`#${overlay.id} button, #${overlay.id} a, #${overlay.id} input`);
+                    });
+                    await page.evaluate(() => document.querySelectorAll("[id$='Overlay']").forEach(o => { o.hidden = true; }));
+                    if (result.offenders.length) failures.push({ where: `${name} · ${label} ${width}x${height}`, offenders: result.offenders });
+                }
             }
         }
 

tests/holdem/seatFootprint.test.js

@@ -0,0 +1,174 @@
+// Whether two seats can end up wanting the same pixels.
+//
+// The felt used to decide this from one number: the straight-line distance between the
+// closest two *neighbouring* seats, judged against a fraction of a puck's width. Both
+// halves of that were wrong, and the running page showed it. Measured across nine
+// viewports and six table sizes:
+//
+//   seventeen of fifty-four tables had one seat's name or stack painted under another
+//   ten of them had the collision between two seats that are not neighbours
+//
+// The second number is the interesting one. On a phone held sideways the felt is 528px
+// wide and 87 tall, the ring is very nearly a straight line, and the seat that lands on
+// top of you is the one across the table — seats 1 and 7, overlapping by 68px, while
+// every neighbouring pair was fine and the check said the table was clean.
+//
+// The first is what a distance test can't fix even looking at every pair: distance has
+// one number in it, and a puck is half again as wide as it is tall. 84px between centres
+// is generous side by side and nowhere near enough on the diagonal.
+const test = require("node:test");
+const { describe } = require("node:test");
+const assert = require("node:assert");
+const fs = require("fs");
+const path = require("path");
+const { Render } = require("./load");
+
+const CSS = fs.readFileSync(
+    path.join(__dirname, "..", "..", "JoshHeaps.Net", "wwwroot", "css", "casino", "holdem.css"), "utf8");
+
+describe("two boxes on a ring", () => {
+    // A phone held sideways, once the action bar has taken its share: a felt far wider
+    // than it is tall, on which the ring is very nearly a straight line.
+    const felt = { width: 400, height: 70 };
+    const flat = { rx: 44, ry: 42 };
+
+    test("a pair that is not neighbours still counts", () => {
+        // Eight seats, a 48x60 puck. Every neighbouring pair is clear. Seats 1 and 5 are
+        // on top of each other, and so are 2 and 4, and 6 and 8 — the seat that lands on
+        // you here is the one across the table, because the rim doubles back at the ends
+        // of a shape this flat and a long walk apart is no distance at all.
+        assert.equal(Render.seatsCollide(8, felt, flat, 48, 60), true);
+
+        const neighbours = (count, w, h) => {
+            const points = Render.ringPositions(count, flat.rx, flat.ry, felt.width / felt.height)
+                .map(s => ({ x: (s.left / 100) * felt.width, y: (s.top / 100) * felt.height }));
+            return points.some((point, i) => {
+                const next = points[(i + 1) % points.length];
+                return Math.abs(point.x - next.x) < w && Math.abs(point.y - next.y) < h;
+            });
+        };
+        assert.equal(neighbours(8, 48 + Render.SEAT_CLEARANCE_PX, 60 + Render.SEAT_CLEARANCE_PX), false,
+            "no neighbouring pair overlaps, which is exactly why this had to look at all of them");
+    });
+
+    test("and the answer is the box, not a radius", () => {
+        // Two seats 60px apart across and 35px down, with a 64x40 puck. The distance
+        // between them is 69px, so the test this replaces — "are these two further apart
+        // than a puck is wide?" — called it clear. The boxes overlap by 4px by 5px.
+        const boxes = { width: 200, height: 100 };
+        const twoSeats = (dx, dy) => {
+            const original = Render.ringPositions;
+            Render.ringPositions = () => [
+                { left: 50 - (dx / 2 / boxes.width) * 100, top: 50 - (dy / 2 / boxes.height) * 100 },
+                { left: 50 + (dx / 2 / boxes.width) * 100, top: 50 + (dy / 2 / boxes.height) * 100 },
+            ];
+            try {
+                return Render.seatsCollide(2, boxes, flat, 64, 40, 0);
+            } finally {
+                Render.ringPositions = original;
+            }
+        };
+        assert.ok(Math.hypot(60, 35) > 64, "and a distance test would have passed it");
+        assert.equal(twoSeats(60, 35), true, "60 across and 35 down, with a 64x40 puck");
+        assert.equal(twoSeats(60, 45), false, "far enough apart vertically");
+        assert.equal(twoSeats(66, 10), false, "far enough apart horizontally");
+    });
+
+    test("with daylight, so rounded corners don't read as touching", () => {
+        assert.ok(Render.SEAT_CLEARANCE_PX > 0);
+    });
+
+    test("a table nobody is sitting at can't collide with itself", () => {
+        assert.equal(Render.seatsCollide(1, felt, flat, 96, 60), false);
+        assert.equal(Render.seatsCollide(9, null, flat, 96, 60), false);
+    });
+});
+
+describe("the widths the felt may choose from", () => {
+    test("run widest first, because the widest that fits is the one wanted", () => {
+        const widths = Render.seatWidthCandidates();
+        widths.slice(1).forEach((w, i) => assert.ok(w < widths[i], `${widths[i]} then ${w}`));
+    });
+
+    test("and span the whole declared range", () => {
+        const widths = Render.seatWidthCandidates();
+        assert.equal(widths[0], Render.SEAT_WIDTH_MAX);
+        assert.equal(widths[widths.length - 1], Render.SEAT_WIDTH_MIN);
+    });
+
+    test("the floor is still a name, not a smudge", () => {
+        // At 44px the tight layout shows about five characters and an ellipsis. Below
+        // that a puck stops telling the dealer who is sitting there, which is the one
+        // thing on the felt it exists to say.
+        assert.ok(Render.SEAT_WIDTH_MIN >= 44);
+        assert.ok(Render.SEAT_TIGHT_BELOW_PX > Render.SEAT_WIDTH_MIN);
+        assert.ok(Render.SEAT_TIGHT_BELOW_PX < Render.SEAT_WIDTH_MAX);
+    });
+
+    test("and there are few enough of them to measure after every tap", () => {
+        // Each one costs a forced layout. Ten is the budget; the cache carries the rest.
+        assert.ok(Render.seatWidthCandidates().length <= 12);
+    });
+});
+
+describe("what the stylesheet has to keep holding up", () => {
+    test("the puck's width comes from the property, not from a breakpoint", () => {
+        // Three media queries used to declare it — 78px, 64px and 62px — three guesses
+        // at a number that depends on how many people are sitting down, which no media
+        // query can see. The same 390px phone has room for 90px at four seats and 61 at
+        // nine.
+        const declarations = CSS.replace(/\/\*[\s\S]*?\*\//g, "");
+        const seatRules = [...declarations.matchAll(/(^|[,}])\s*(\.seat)\s*\{([^}]*)\}/g)]
+            .map(m => m[3]);
+        const hardWidths = seatRules
+            .flatMap(body => body.match(/(?<![\w-])width:\s*[^;]+/g) || [])
+            .filter(decl => !decl.includes("var(--seatPuckW"));
+        assert.deepEqual(hardWidths, []);
+    });
+
+    test("the puck measures what it says it measures", () => {
+        // As content-box a 64px puck came out 74 across once its padding and border were
+        // counted, and the search compares that number against distances on the felt.
+        const rule = CSS.match(/\n\.seat \{[^}]*\}/);
+        assert.ok(rule, ".seat rule not found");
+        assert.match(rule[0], /box-sizing:\s*border-box/);
+    });
+
+    test("and a blind badge never reaches sideways off its own puck", () => {
+        // This is what makes the footprint the puck and nothing more. The badge's
+        // remaining overhang is vertical, and it points away from the table at both ends
+        // — `top` on the upper arc, `bottom` on the lower — so two seats facing each
+        // other across the felt wear their badges back to back.
+        //
+        // It was not always so: at `left: -6px` the small blind hung off the puck's left
+        // corner, and on an iPhone Pro Max at six seats — pucks not touching at all —
+        // "SB" was painted across the next player's $1,137.
+        const badges = [...CSS.matchAll(/\.(sb|bb|str)Badge\s*\{([^}]*)\}/g)];
+        assert.equal(badges.length, 3, "expected all three blind badges");
+        badges.forEach(([, which, body]) => {
+            const left = body.match(/(?<![\w-])left:\s*(-?[\d.]+)px/);
+            assert.ok(left, `.${which}Badge should place itself horizontally`);
+            assert.ok(Number(left[1]) >= 0,
+                `.${which}Badge sits at ${left[1]}px, off the left of its own puck`);
+        });
+    });
+
+    test("and all three sit in the same place, so there is one spot to look at", () => {
+        // Two badges in two positions means the dealer's eye has to find which of them
+        // this seat is using before it can read which blind it is, forty times a night.
+        // A seat only ever wears one: the blinds are different players by construction,
+        // and the straddle chain starts left of the big blind and stops short of the
+        // small one.
+        const lefts = [...CSS.matchAll(/\.(sb|bb|str)Badge\s*\{([^}]*)\}/g)]
+            .map(([, , body]) => (body.match(/(?<![\w-])left:\s*(-?[\d.]+px)/) || [])[1]);
+        assert.equal(new Set(lefts).size, 1, `badges sit at ${lefts.join(", ")}`);
+    });
+});
+
+describe("with no browser to measure", () => {
+    test("nothing here throws", () => {
+        assert.equal(Render.measuredSeatBox(), null);
+        assert.equal(Render.seatFootprint(null), null);
+        assert.deepEqual(Render.seatFootprint({ width: 64, height: 40 }), { width: 64, height: 40 });
+    });
+});

tests/holdem/seatsOnTheTable.test.js

@@ -0,0 +1,110 @@
+// How wide the ring of seats is allowed to be.
+//
+// It sat at a flat 44% by 42% of the seat layer at every size — two numbers picked
+// against a phone, then asked to hold for a 754x543 felt on a desktop and a 686x141
+// one on a phone held sideways. A fixed pair can only be right for one ratio of puck
+// to felt, so measured on the running page:
+//
+//   1280x900, nine-handed   five of nine pucks past the table's outer edge, worst 49px
+//   844x390,  six-handed    all six, worst 20px
+//   430x932,  six-handed    two, worst 12px
+//
+// Seats floating off the felt onto the page read as the table being too small for the
+// game rather than as players at a rail, and it was a different amount of wrong on
+// every screen — which is what made it look like a mistake instead of a choice.
+//
+// The radius is derived from what has to fit now. Whether it *actually* fits is a
+// question about a laid-out browser, and it is checked there: uiAudit's "seats and
+// chips stay on the table" runs at every viewport in the sweep. What is checked here
+// is the contract that function has to honour, including the two ways it is allowed
+// to give up.
+const test = require("node:test");
+const { describe } = require("node:test");
+const assert = require("node:assert");
+const { Render } = require("./load");
+
+describe("with no browser to measure", () => {
+    test("it falls back to the ring the tool has always used", () => {
+        // Every call from this suite has no DOM, and so does the very first render,
+        // before the felt has been laid out. Neither should throw or return nonsense.
+        assert.deepEqual(Render.seatRingRadii(), { rx: Render.RING_MAX_RX, ry: Render.RING_MAX_RY });
+    });
+
+    test("so seatPosition still answers", () => {
+        const pos = Render.seatPosition(0, 9, 1);
+        assert.ok(Number.isFinite(pos.left) && Number.isFinite(pos.top));
+    });
+
+    test("and nothing is measured for the puck height either", () => {
+        assert.equal(Render.tooTallForTheTable(), false);
+    });
+});
+
+describe("the ceiling and the floor", () => {
+    test("the old ring is the widest it can ever be", () => {
+        // Nothing gets looser than it was; this can only pull seats in.
+        assert.ok(Render.RING_MAX_RX <= 44);
+        assert.ok(Render.RING_MAX_RY <= 42);
+    });
+
+    test("and it never collapses the ring into the pot", () => {
+        // A puck wider than the table it sits on has no honest answer. Overhanging is
+        // the lesser evil — seats stacked on the pot would be unreadable, and the
+        // density rule shrinks the pucks before it ever gets this far.
+        assert.ok(Render.RING_MIN_R > 0);
+        assert.ok(Render.RING_MIN_R < Render.RING_MAX_RY);
+    });
+
+    test("with a margin that keeps the outermost seat off the rail", () => {
+        assert.ok(Render.SEAT_EDGE_MARGIN_PX > 0);
+    });
+});
+
+describe("the ring the radii describe", () => {
+    const spread = (radii, count = 9, aspect = 1.4) =>
+        Render.ringPositions(count, radii.rx, radii.ry, aspect);
+
+    test("a smaller radius puts every seat nearer the middle", () => {
+        const wide = spread({ rx: 44, ry: 42 });
+        const tight = spread({ rx: 30, ry: 28 });
+        wide.forEach((seat, i) => {
+            const wideOut = Math.hypot(seat.left - 50, seat.top - 50);
+            const tightOut = Math.hypot(tight[i].left - 50, tight[i].top - 50);
+            assert.ok(tightOut < wideOut, `seat ${i + 1} should have come in`);
+        });
+    });
+
+    test("and the seats stay evenly spread however far in they are", () => {
+        // Pulling the ring in must not bunch them: the spacing rule is arc length, and
+        // shrinking an ellipse uniformly keeps every gap in proportion.
+        const gaps = ring => ring.map((p, i) => {
+            const next = ring[(i + 1) % ring.length];
+            return Math.hypot(p.left - next.left, p.top - next.top);
+        });
+        const tight = gaps(spread({ rx: 30, ry: 28 }));
+        const smallest = Math.min(...tight), largest = Math.max(...tight);
+        assert.ok(largest / smallest < 1.35, `gaps ranged ${smallest.toFixed(1)}..${largest.toFixed(1)}`);
+    });
+});
+
+describe("the bets follow the seats", () => {
+    test("a chip sits on its own player's spoke, proportionally in", () => {
+        // The chip is a fraction of the seat's reach rather than of the felt, so when
+        // the ring pulls in the chips come with it instead of being left by the rail.
+        const radii = { rx: 30, ry: 28 };
+        const seat = Render.seatPosition(2, 8, 1.4, radii);
+        const bet = Render.betPosition(2, 8, 1.4, radii);
+        const seatOut = Math.hypot(seat.left - 50, seat.top - 50);
+        const betOut = Math.hypot(bet.left - 50, bet.top - 50);
+        assert.ok(betOut < seatOut, "the chip is nearer the pot than the player is");
+        assert.ok(betOut > seatOut * 0.4, "but still out on the player's side of the felt");
+    });
+
+    test("and it is the same spoke, not just the same distance", () => {
+        const radii = { rx: 30, ry: 28 };
+        const seat = Render.seatPosition(5, 9, 1.2, radii);
+        const bet = Render.betPosition(5, 9, 1.2, radii);
+        const angleOf = p => Math.atan2(p.top - 50, p.left - 50);
+        assert.ok(Math.abs(angleOf(seat) - angleOf(bet)) < 0.01);
+    });
+});

tests/holdem/staleToast.test.js

@@ -0,0 +1,106 @@
+// The toast that outlived the thing it was talking about.
+//
+// Toasts are news about the last action, and they carry an Undo. Take another action
+// and both of those go wrong at once: the sentence describes a step that is now two
+// steps back, and the Undo behind it restores a snapshot from before everything since.
+// "Next hand — button on seat 2. [Undo]" left up while a hand is dealt and three
+// players call is an offer to move the button, wired to a button that deletes the hand.
+//
+// It is also simply in the way. The toast is a fixed strip near the bottom of the
+// screen; on a 360x640 phone there is nowhere for it to sit that is clear of both the
+// felt and the action bar, so a stale one covers seats and controls for eight seconds.
+//
+// So the bar drops it as it takes the next action. These tests hold that funnel shut:
+// every action a dealer can take from the bar goes through `commit`, and `commit`
+// clears the toast before it does anything else.
+const test = require("node:test");
+const { describe } = require("node:test");
+const assert = require("node:assert");
+const fs = require("fs");
+const path = require("path");
+const { ActionBar, Betting, makeTable } = require("./load");
+
+const SRC = path.join(__dirname, "..", "..", "JoshHeaps.Net", "wwwroot", "js", "casino", "holdem");
+const barSource = fs.readFileSync(path.join(SRC, "actionBar.js"), "utf8");
+
+// A bar sat at a real table, with the toast replaced by something that keeps a log
+// and the re-render stubbed out — none of these rules need the DOM back.
+function onTable() {
+    const state = makeTable({ seatCount: 6, stack: 200, smallBlind: 1, bigBlind: 2, dealerSeatIndex: 0 });
+    const log = [];
+    globalThis.Toast = {
+        onScreen: null,
+        show(message) { this.onScreen = message; log.push(`show ${message}`); },
+        dismiss() { this.onScreen = null; log.push("dismiss"); },
+    };
+    ActionBar.state = state;
+    ActionBar.render = () => {};
+    ActionBar.hooks = {
+        beginAction: label => log.push(`begin ${label}`),
+        save: () => log.push("save"),
+        rerender: () => log.push("rerender"),
+    };
+    return { bar: ActionBar, state, log };
+}
+
+test.afterEach(() => { delete globalThis.Toast; });
+
+describe("a toast does not outlive its own news", () => {
+    test("the next action from the bar clears it", () => {
+        const { bar, state, log } = onTable();
+        Toast.show("Next hand — button on seat 2.");
+
+        bar.commit("dealing the hand", () => Betting.beginHand(state));
+
+        assert.equal(Toast.onScreen, null, "the hand is dealt; the sentence about seating the table is not news any more");
+        assert.ok(log.indexOf("dismiss") < log.indexOf("begin dealing the hand"),
+            `the toast goes before the action is recorded, so nothing shown during the action is swept up with it: ${log.join(", ")}`);
+    });
+
+    test("a toast raised by the action itself survives", () => {
+        const { bar, log } = onTable();
+        bar.commit("something that talks back", () => Toast.show("Bob is all in."));
+
+        assert.equal(Toast.onScreen, "Bob is all in.",
+            "commit clears what was there when it started, not what its own work puts up");
+        assert.deepEqual(log, ["dismiss", "begin something that talks back", "show Bob is all in.", "save", "rerender"]);
+    });
+
+    test("with no toast on screen it is still just one dismiss", () => {
+        const { bar, log } = onTable();
+        bar.commit("a fold", () => {});
+        assert.equal(log.filter(l => l === "dismiss").length, 1);
+    });
+
+    test("a page with no Toast at all does not throw", () => {
+        const { bar, state } = onTable();
+        delete globalThis.Toast;
+        assert.doesNotThrow(() => bar.commit("dealing the hand", () => Betting.beginHand(state)));
+    });
+});
+
+describe("every bar action really does go through that funnel", () => {
+    // The dismiss is worth nothing if some button reaches past `commit` to mutate the
+    // table itself. Actions are written as `this.commit(...)`; a `Betting.` call made
+    // straight from a click handler would be one that skips the clean-up — and the
+    // undo history with it, which is the older reason this funnel exists.
+    test("no click handler calls the betting engine directly", () => {
+        const strays = barSource
+            .split("\n")
+            .map((line, i) => [i + 1, line])
+            .filter(([, line]) => /=>\s*Betting\.(fold|check|call|allIn|raiseTo|beginHand|closeStreet)\(/.test(line))
+            .filter(([, line]) => !/this\.commit\(/.test(line))
+            .map(([n, line]) => `actionBar.js:${n}: ${line.trim()}`);
+
+        assert.deepEqual(strays, [], "these change the table without going through commit");
+    });
+
+    test("commit is what clears the toast, not each button", () => {
+        const dismissals = barSource.split("\n").filter(l => /Toast\.dismiss\(\)/.test(l)).length;
+        // Two: this one, and the raise pad — which opens without committing anything,
+        // and needs the strip clear because the pad makes the bar tall enough to push
+        // a toast up over the seats.
+        assert.equal(dismissals, 2,
+            "a third dismissal means a button is doing by hand what commit already does");
+    });
+});

tests/holdem/uiAudit.js

@@ -93,6 +93,20 @@ const LayoutAudit = {
         return false;
     },
 
+    // The scrolling ancestor whose box this element has fallen outside of, if any.
+    // Wholly outside — an element straddling the edge is still half on show, and the
+    // half that is showing is worth hit-testing.
+    clippedOutBy(el, rect) {
+        for (let node = el.parentElement; node; node = node.parentElement) {
+            const style = getComputedStyle(node);
+            if (!/auto|scroll|hidden|overlay|clip/.test(style.overflowY + style.overflowX)) continue;
+            const box = node.getBoundingClientRect();
+            if (rect.top >= box.bottom - 0.5 || rect.bottom <= box.top + 0.5 ||
+                rect.left >= box.right - 0.5 || rect.right <= box.left + 0.5) return node;
+        }
+        return null;
+    },
+
     // The modal that is currently on top, if any.
     //
     // Every check below asks "is this thing reachable", and while a sheet or dialog is
@@ -143,6 +157,22 @@ const LayoutAudit = {
                 return;
             }
 
+            // Still on screen, but clipped away by a panel that scrolls. Hit-testing it
+            // is meaningless: the point lands on whatever the panel's own edge has next
+            // to it, and the answer comes back as a diagnosis of something that isn't
+            // happening. The pot wizard's third winner, below the fold in a list that
+            // scrolls, was reported for four viewports running as "fully covered by
+            // button Confirm Winner(s)" — the button is simply what is painted at that
+            // height. Whether it is a fault is the same question as for anything else
+            // out of view: can it be scrolled to.
+            const clip = this.clippedOutBy(el, rect);
+            if (clip) {
+                if (!this.reachableByScrolling(el)) {
+                    offenders.push(`${this.describe(el)} :: clipped out of ${this.describe(clip)}, which cannot be scrolled to it`);
+                }
+                return;
+            }
+
             const points = [
                 [rect.left + rect.width / 2, rect.top + rect.height / 2],
                 [rect.left + rect.width * 0.25, rect.top + rect.height * 0.3],
@@ -777,6 +807,140 @@ const LayoutAudit = {
         return { name: "seat names aren't painted under their own badges", offenders };
     },
 
+    // A seat's writing, covered by something belonging to a different seat.
+    //
+    // seatLabelsClear() above asks whether a puck's own decoration sits on its own name.
+    // This asks the other half, and the other half was where the tool was actually
+    // failing: measured across nine viewports and six table sizes before this went in,
+    // seventeen of fifty-four tables had one seat's name or stack painted under a
+    // neighbour. On a Galaxy S8 at the most ordinary table there is — six players at a
+    // nine-handed table — seat 2's stack was under seat 3's puck, and five pairs of
+    // seats overlapped, the worst by 61px.
+    //
+    // Two ways in, and it takes both:
+    //   - the pucks themselves overlapping, which the layout used to permit on the
+    //     reasoning that rounded corners forgive a little; what they were forgiving was
+    //     a four-figure stack.
+    //   - a blind badge hanging off its puck onto the seat next door, which no amount
+    //     of space between the pucks fixes on its own.
+    //
+    // Hit-tested against the glyph run rather than the element box. A name is centred,
+    // so its box is wider than its ink and testing the box would fail on empty air.
+    //
+    // The glyph run is then clipped back to what is actually painted, which is not the
+    // same thing and cost this check its first honest-looking failure. A puck's name is
+    // a `-webkit-box` with `line-clamp`, and getClientRects() hands back a rect for every
+    // line the text *would* have taken — including the two that the clamp throws away.
+    // Measured at 640x360 with twelve players: "Mike from work" reported a line 30px
+    // below its own puck, which duly hit-tested onto the seat underneath, and the seats
+    // were not touching at all. Nothing is painted there, so nothing can cover it.
+    seatsKeepTheirText() {
+        const name = "no seat covers another seat's writing";
+        if (this.modalScope()) {
+            return { name, offenders: [], skipped: "a sheet is open over the table" };
+        }
+        const seats = [...document.querySelectorAll(".seat")];
+        if (!seats.length) return { name, offenders: [], skipped: "no seats on this page" };
+
+        const offenders = [];
+        seats.forEach((seat, index) => {
+            seat.querySelectorAll(".seatPuckName, .seatPuckStack, .seatPuckTotal").forEach(label => {
+                const text = label.textContent.trim();
+                if (!text) return;
+                const painted = label.getBoundingClientRect();
+                if (!painted.width || !painted.height) return;
+                const range = document.createRange();
+                range.selectNodeContents(label);
+                [...range.getClientRects()].forEach(raw => {
+                    const line = {
+                        left: Math.max(raw.left, painted.left),
+                        right: Math.min(raw.right, painted.right),
+                        top: Math.max(raw.top, painted.top),
+                        bottom: Math.min(raw.bottom, painted.bottom),
+                    };
+                    if (line.right - line.left <= 1 || line.bottom - line.top <= 1) return;
+                    // Along the line rather than at its middle: a badge takes the start
+                    // of it and a neighbouring puck the end, and a single centre probe
+                    // walks past both.
+                    for (let across = 0.08; across <= 0.92; across += 0.14) {
+                        const x = line.left + (line.right - line.left) * across;
+                        const y = (line.top + line.bottom) / 2;
+                        const over = document.elementFromPoint(x, y);
+                        if (!over || seat.contains(over) || over.contains(seat)) continue;
+                        offenders.push(
+                            `seat ${index + 1} "${text}" is under ${this.describe(over)}`);
+                        return;
+                    }
+                });
+            });
+        });
+
+        return { name, offenders };
+    },
+
+    // The other thing on the felt that carries a number.
+    //
+    // A chip says what a player has put in this street, which is what a call costs. Two
+    // of them on top of each other is two figures a dealer can't read, and one behind a
+    // puck is a figure that isn't there at all.
+    //
+    // Geometry rather than hit-testing, which is the opposite of how every other check
+    // here works and is forced: #betsLayer is `pointer-events: none`, so elementFromPoint
+    // never returns a chip — it returns whatever is behind it, for every chip, always.
+    // A first version of this probe hit-tested and duly reported all forty-five
+    // configurations as broken, including the ones that were fine. Paint order is known
+    // and fixed instead: #betsLayer comes before #seatsLayer in the document and a seat
+    // carries `z-index: 1`, so a puck a chip overlaps is a puck that chip is behind.
+    chipsKeepTheirAmounts() {
+        const name = "no bet chip is hidden behind a seat or another chip";
+        if (this.modalScope()) {
+            return { name, offenders: [], skipped: "a sheet is open over the table" };
+        }
+        const chips = [...document.querySelectorAll(".betChip")]
+            .filter(c => { const r = c.getBoundingClientRect(); return r.width > 0 && r.height > 0; });
+        if (!chips.length) return { name, offenders: [], skipped: "no bets on the felt" };
+
+        const overlap = (A, B) =>
+            Math.min(A.right, B.right) - Math.max(A.left, B.left) > 0.5 &&
+            Math.min(A.bottom, B.bottom) - Math.max(A.top, B.top) > 0.5;
+        const offenders = [];
+        const seats = [...document.querySelectorAll(".seat")];
+
+        chips.forEach(chip => {
+            const label = chip.textContent.trim() || "a chip";
+            const box = chip.getBoundingClientRect();
+            seats.forEach((seat, index) => {
+                // Its own puck is the one thing it is allowed to touch: on a narrow felt
+                // the chip clings to the seat rather than taking a ring of its own.
+                if (seat.contains(chip)) return;
+                if (overlap(box, seat.getBoundingClientRect())) {
+                    offenders.push(`chip ${label} is behind seat ${index + 1}`);
+                }
+            });
+            // A chip wider than the puck it is clinging to is on its neighbours by
+            // construction, whatever the ring does.
+            const host = chip.closest(".seat");
+            if (host && chip.classList.contains("betChipAnchored")) {
+                const puck = host.getBoundingClientRect();
+                if (box.width > puck.width + 0.5) {
+                    offenders.push(`chip ${label} is ${Math.round(box.width - puck.width)}px ` +
+                        `wider than its own puck`);
+                }
+            }
+        });
+
+        for (let a = 0; a < chips.length; a++) {
+            for (let b = a + 1; b < chips.length; b++) {
+                if (overlap(chips[a].getBoundingClientRect(), chips[b].getBoundingClientRect())) {
+                    offenders.push(`chip ${chips[a].textContent.trim()} and chip ` +
+                        `${chips[b].textContent.trim()} are on top of each other`);
+                }
+            }
+        }
+
+        return { name, offenders: [...new Set(offenders)] };
+    },
+
     // ---- Runner ----
 
     // Every check this file knows how to make, every time.
@@ -793,8 +957,98 @@ const LayoutAudit = {
     // on a mouse-driven browser — that is right, a 24px control is fine with a cursor
     // on it — but "not measured" and "measured and fine" are different answers and the
     // runner now prints which one it got.
+    // The table has to be big enough for the game around it.
+    //
+    // The seat ring sat at a flat 44% by 42% of the seat layer at every size — two
+    // numbers picked against a phone. A fixed pair can only be right for one ratio of
+    // puck to felt, and measured on the running page it was wrong nearly everywhere:
+    // at 1280x900 nine-handed, five of nine pucks finished past the outer edge of the
+    // table, the worst by 49px; on a phone held sideways all six did. Seats floating
+    // off the felt onto the page read as the table being too small for the game rather
+    // than as players at a rail — and it was a different amount of wrong on every
+    // screen, which is what made it look like a mistake instead of a choice.
+    //
+    // Checked against the table's outer edge, border included: sitting *on* the rail is
+    // fine, hanging off the far side of it is not. The bet chips are held to the same
+    // rule — a chip is meant to be on the felt between its player and the pot.
+    seatsOnTheTable(slackPx = 1) {
+        const name = "seats and chips stay on the table";
+        if (this.modalScope()) {
+            return { name, offenders: [], skipped: "a sheet is open over the table" };
+        }
+        const table = document.getElementById("pokerTable");
+        if (!table) return { name, offenders: [], skipped: "no table on this page" };
+
+        const felt = table.getBoundingClientRect();
+        if (!felt.width || !felt.height) return { name, offenders: [], skipped: "table not laid out" };
+
+        const offenders = [];
+        const check = (el, what) => {
+            const r = el.getBoundingClientRect();
+            if (!r.width || !r.height) return;
+            const over = Math.max(
+                felt.left - r.left, r.right - felt.right,
+                felt.top - r.top, r.bottom - felt.bottom);
+            if (over > slackPx) offenders.push(`${what} hangs ${Math.round(over)}px past the rail`);
+        };
+
+        [...document.querySelectorAll(".seat")].forEach((seat, i) => check(seat, `seat ${i + 1}`));
+        [...document.querySelectorAll("#betsLayer > *")].forEach((chip, i) =>
+            check(chip, `bet chip ${i + 1}`));
+
+        return { name, offenders };
+    },
+
+    // An open dialog's buttons have to be on screen without hunting for them.
+    //
+    // The pot wizard — the one dialog in this tool that divides money — had its Confirm
+    // below a scroll at every size: measured with six players and a side pot, 244px off
+    // the bottom on an iPhone 14, 417 on a Galaxy S8, 583 on a phone held sideways, 172
+    // on a desktop. Nothing on screen said there was anything below the list of names,
+    // so the only control that finishes the hand was invisible with the table waiting.
+    //
+    // Checked by hit-testing rather than by geometry alone: a button can be inside the
+    // viewport and still be under something. Only dialogs that are actually open are
+    // considered, and only their buttons — a dialog with no buttons has nothing to find.
+    dialogActionsReachable() {
+        const name = "an open dialog's buttons are on screen";
+        const dialogs = [...document.querySelectorAll("[role='dialog'], [role='alertdialog']")]
+            .filter(d => {
+                const rect = d.getBoundingClientRect();
+                return rect.width > 0 && rect.height > 0 && getComputedStyle(d).visibility !== "hidden";
+            });
+        if (dialogs.length === 0) return { name, offenders: [], skipped: "no dialog is open" };
+
+        const offenders = [];
+        dialogs.forEach(dialog => {
+            const buttons = [...dialog.querySelectorAll("button")].filter(b => {
+                const r = b.getBoundingClientRect();
+                return r.width > 0 && r.height > 0;
+            });
+            buttons.forEach(button => {
+                const r = button.getBoundingClientRect();
+                const label = this.describe(button);
+                if (r.bottom > innerHeight || r.top < 0) {
+                    const off = r.bottom > innerHeight
+                        ? Math.round(r.bottom - innerHeight) + "px below the fold"
+                        : Math.round(-r.top) + "px above it";
+                    offenders.push(`${label} is ${off}`);
+                    return;
+                }
+                const hit = document.elementFromPoint(r.left + r.width / 2, r.top + r.height / 2);
+                if (hit && hit !== button && !button.contains(hit)) {
+                    offenders.push(`${label} is covered by ${this.describe(hit)}`);
+                }
+            });
+        });
+
+        return { name, offenders };
+    },
+
     runAll(options = {}) {
         const checks = [
+            this.seatsOnTheTable(options.seatSlack),
+            this.dialogActionsReachable(),
             this.touchAction(),
             this.tapHighlight(),
             this.textSelectable(),
@@ -807,6 +1061,8 @@ const LayoutAudit = {
             this.oneDealerButton(),
             this.hiddenButPainted(),
             this.seatLabelsClear(),
+            this.seatsKeepTheirText(),
+            this.chipsKeepTheirAmounts(),
             this.contrast(options.contrast),
             this.touchTargets(options.minimumTarget, options.touchTargets),
         ];