pokertools-arena 0.4.1 → 0.4.4

package/README.md

@@ -17,7 +17,7 @@
 | **Package** | `npx pokertools-arena` |
 | **Runtime** | Node.js ≥ 24 for tooling; any modern browser for the app |
 
-> **Current release: 0.4.1.** A maintenance release on top of the 0.4.0 methodology work: modal tooltips now float above overflow, the launcher starts exactly once when its default port is busy, and the real-API diagnostics skip cleanly when repository secrets are absent.
+> **Current release: 0.4.4.** A maintenance release on top of the 0.4.0 methodology work: modal tooltips float above overflow, the launcher starts exactly once when its default port is busy, real-API diagnostics skip cleanly when repository secrets are absent, the Decision sanity suite attributes results to the correct model, seat editors show the model-derived name, close buttons use centered SVG icons, and decision logs no longer repeat the typed-decision caption.
 
 ---
 

package/dist/app.js

@@ -115138,7 +115138,6 @@ function decide(agent, connection, args) {
 }
 var FAMILY_OBJECTIVE = "Choose exactly one legal action family that best maximizes tournament chip EV from the supplied state. Do not choose an amount; a size is chosen in a separate step.";
 var SIZE_OBJECTIVE = (family) => `The action family ${String(family).toUpperCase()} has already been selected. Choose exactly one legal ${family === ACTION_FAMILY.RAISE ? "raise size" : "bet size"} that best maximizes tournament chip EV. Do not change the action family.`;
-var SPECTATOR_NOTE = "This model returns typed decisions rather than a text rationale.";
 function buildJevFamilyQuestions(families, { instructions = FAMILY_OBJECTIVE, criteria = null } = {}) {
   return { type: "choice", instructions, criteria: criteria ?? familyCriteria(families) };
 }
@@ -115594,6 +115593,7 @@ var lastProcessedEventId = null;
 var MAX_LOBBY_SEATS = 10;
 var seatAssignments = Array(MAX_LOBBY_SEATS).fill(null);
 var editingSeatIndex = null;
+var seatNameAuto = false;
 var lobbyVisible = true;
 var pendingAutostart = false;
 var arenaMaxDecisions = 0;
@@ -116863,19 +116863,28 @@ function updateSeatSummary() {
   const count = seatAssignments.filter(Boolean).length;
   if (els.seatSummary) els.seatSummary.textContent = `${count} / ${MAX_LOBBY_SEATS} seated`;
 }
+function syncSeatNameFromModel() {
+  if (!seatNameAuto || editingSeatIndex == null) return;
+  const model = els.seatModel.value.trim();
+  els.seatName.value = model ? displayModelName(model) : `Player ${editingSeatIndex + 1}`;
+}
 function openSeatEditor(seatIndex) {
   if (!Number.isInteger(seatIndex) || seatIndex < 0 || seatIndex >= MAX_LOBBY_SEATS) return;
   editingSeatIndex = seatIndex;
   const locked = Boolean(director && ["RUNNING", "PAUSED"].includes(director.status));
   const draft = seatAssignments[seatIndex] ? { ...seatAssignments[seatIndex] } : defaultSeatDraft(seatIndex);
   els.seatDialogTitle.textContent = `Seat ${seatIndex + 1}`;
-  els.seatName.value = draft.name || `Player ${seatIndex + 1}`;
+  const placeholderName = `Player ${seatIndex + 1}`;
+  const storedName = String(draft.name || "").trim();
+  seatNameAuto = !storedName || /^player\s+\d+$/i.test(storedName) || Boolean(draft.model) && storedName === displayModelName(draft.model);
+  els.seatName.value = seatNameAuto && draft.model ? displayModelName(draft.model) : storedName || placeholderName;
   refreshSeatConnectionSelect(draft.connectionId || "");
   if (draft.connectionId && [...els.seatConnection.options].some((option) => option.value === draft.connectionId)) els.seatConnection.value = draft.connectionId;
   els.seatModel.value = draft.model || "";
   els.seatProtocol.value = draft.protocol || "tool";
   els.seatProvider.value = draft.provider || "";
   applySeatProtocolRules();
+  syncSeatNameFromModel();
   void refreshSeatModelCatalog();
   els.seatError.classList.add("hidden");
   els.seatLockNotice.classList.toggle("hidden", !locked);
@@ -117317,8 +117326,6 @@ function decisionTelemetryHtml(event, { limit = 4 } = {}) {
   const reasoningTokens = event?.usage?.completion_tokens_details?.reasoning_tokens;
   if (reasoningTokens) facts.push(`<span>Reasoning <b>${reasoningTokens} tokens</b></span>`);
   if (facts.length) parts.push(`<div class="decision-facts">${facts.join("")}</div>`);
-  const isTypedDecision = meta.family != null || meta.aggression != null || meta.bluffSpot != null || meta.method === "openrouter-decisions" || meta.method === "jev-choice" || String(meta.method || "").includes("hierarchical");
-  if (isTypedDecision) parts.push(`<div class="decision-facts-note">${escapeHtml(SPECTATOR_NOTE)} Bluff opportunity is a property of the spot, not the reason for the chosen action.</div>`);
   return parts.join("");
 }
 function setDecisionContext({ hand = "\u2014", street = "\u2014", position = "\u2014", options = "\u2014", label = "Legal actions", hint = "Choose one", labels = null } = {}) {
@@ -117576,7 +117583,7 @@ function cloneJson(value) {
 }
 function sanityAgents() {
   const connections = readConnections();
-  return readSeatPlayers().map((player) => ({ ...player, connection: connections.find((c) => c.id === player.connectionId) })).filter((row) => row.connection);
+  return readSeatPlayers().map((player) => ({ ...player, id: player.id || `player-${player.lobbySeat + 1}`, connection: connections.find((c) => c.id === player.connectionId) })).filter((row) => row.connection);
 }
 function sanityProtocolLabel(agent) {
   const protocol = effectiveProtocol(agent, agent.connection);
@@ -118288,9 +118295,16 @@ for (const dialog of [els.setupDialog, els.seatDialog, els.testsDialog, els.repl
 }
 els.seatConnection.addEventListener("change", () => {
   applySeatProtocolRules();
+  syncSeatNameFromModel();
   void refreshSeatModelCatalog();
 });
-els.seatModel.addEventListener("input", applySeatProtocolRules);
+els.seatModel.addEventListener("input", () => {
+  applySeatProtocolRules();
+  syncSeatNameFromModel();
+});
+els.seatName.addEventListener("input", () => {
+  seatNameAuto = false;
+});
 els.seatProtocol.addEventListener("change", applySeatProtocolRules);
 els.refreshModelsBtn?.addEventListener("click", () => void refreshSeatModelCatalog({ force: true }));
 els.removeSeatBtn.addEventListener("click", () => {

package/dist/index.html

@@ -165,7 +165,7 @@
           <h2>Settings</h2>
           <button class="info-button" type="button" aria-label="About settings" data-tip="API keys stay in this browser tab and are never written to saved configuration or tournament logs.">i</button>
         </div>
-        <button id="closeSetup" type="button" class="icon-button" aria-label="Close settings">×</button>
+        <button id="closeSetup" type="button" class="icon-button" aria-label="Close settings"><svg class="icon-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="M4.5 4.5l7 7M11.5 4.5l-7 7" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round"/></svg></button>
       </div>
 
       <div class="dialog-body setup-body">
@@ -246,7 +246,7 @@
           <h2 id="seatDialogTitle">Seat 1</h2>
           <button class="info-button" type="button" aria-label="About seat context" data-tip="Each model receives only its own hole cards plus the same canonical public table state, recent public hands, opponent statistics, and legal actions.">i</button>
         </div>
-        <button id="closeSeat" type="button" class="icon-button" aria-label="Close seat editor">×</button>
+        <button id="closeSeat" type="button" class="icon-button" aria-label="Close seat editor"><svg class="icon-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="M4.5 4.5l7 7M11.5 4.5l-7 7" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round"/></svg></button>
       </div>
       <div class="dialog-body seat-editor-body">
         <div id="seatLockNotice" class="seat-lock-notice hidden">Seat assignments are locked during a tournament.</div>
@@ -284,7 +284,7 @@
           <div class="dialog-title-row"><h2>Decision sanity suite</h2><button class="info-button" type="button" aria-label="About decision tests" data-tip="A small comprehension and reliability check. It is intentionally not a GTO solver, Elo score, or tournament-strength ranking.">i</button></div>
           <div class="dialog-subtitle">The same 10 fixed poker spots are sent independently to every seated model.</div>
         </div>
-        <button id="closeTests" type="button" class="icon-button" aria-label="Close tests">×</button>
+        <button id="closeTests" type="button" class="icon-button" aria-label="Close tests"><svg class="icon-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="M4.5 4.5l7 7M11.5 4.5l-7 7" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round"/></svg></button>
       </div>
       <div class="tests-body">
         <div class="tests-how" aria-label="How the suite works">
@@ -317,7 +317,7 @@
           <div class="dialog-title-row"><h2 id="replayTitle">Decision replay</h2><span id="replayBadge" class="replay-badge">HAND —</span></div>
           <div id="replaySubtitle" class="dialog-subtitle">Snapshot captured immediately before the model acted.</div>
         </div>
-        <button id="closeReplay" type="button" class="icon-button" aria-label="Close decision replay">×</button>
+        <button id="closeReplay" type="button" class="icon-button" aria-label="Close decision replay"><svg class="icon-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="M4.5 4.5l7 7M11.5 4.5l-7 7" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round"/></svg></button>
       </div>
       <div class="dialog-body replay-body">
         <section class="replay-table-wrap">
@@ -361,7 +361,7 @@
           <span class="field-label-row">Preset <button class="info-button inline-info" type="button" aria-label="About connection presets" data-tip="OpenAI-compatible keeps Base URL fully editable. OpenRouter only fills its standard URL and enables OpenRouter-specific routing. TypeSafe uses the native System One endpoint.">i</button></span>
           <select data-field="kind"><option value="openai">OpenAI-compatible</option><option value="openrouter">OpenRouter</option><option value="typesafe">TypeSafe System One</option></select>
         </label>
-        <button type="button" class="remove-connection icon-button" title="Remove connection" aria-label="Remove connection">×</button>
+        <button type="button" class="remove-connection icon-button" title="Remove connection" aria-label="Remove connection"><svg class="icon-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="M4.5 4.5l7 7M11.5 4.5l-7 7" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round"/></svg></button>
       </div>
       <div class="connection-main-grid">
         <label class="connection-url">Base URL<input data-field="baseUrl" spellcheck="false" placeholder="https://api.example.com/v1" /></label>

package/dist/pokertools-arena.html

@@ -895,7 +895,11 @@ input::placeholder{color:#4e5862}
 .button,.icon-button,.field-icon-button,input,select{box-sizing:border-box}
 .button{min-height:var(--control-h);padding:0 12px;border-radius:var(--radius-sm);font-size:10px;font-weight:800;line-height:1;white-space:nowrap}
 .small-button{min-height:30px;height:30px;padding-inline:10px;font-size:9px}
-.icon-button{width:30px;height:30px;min-width:30px;border-radius:var(--radius-sm);display:grid;place-items:center;padding:0}
+.icon-button{width:30px;height:30px;min-width:30px;border-radius:var(--radius-sm);display:grid;place-items:center;padding:0;line-height:1}
+/* SVG glyphs center geometrically; a text "×" sits on its baseline and looks
+   vertically off-centre inside these small buttons. */
+.icon-button svg{width:14px;height:14px;display:block;pointer-events:none}
+.dialog-head .icon-button svg{width:13px;height:13px}
 input,select{width:100%;min-width:0;height:var(--control-h);padding:0 9px;border-radius:var(--radius-sm);font-size:11px;background:#0a0d10;border:1px solid var(--modal-border);color:var(--text)}
 input:focus,select:focus{outline:none;border-color:rgba(146,201,164,.34);box-shadow:0 0 0 3px rgba(146,201,164,.05)}
 label{min-width:0}
@@ -2321,10 +2325,7 @@ a.button{text-decoration:none}
   .card .card-corner.bottom{display:none!important}
 }
 
-/* 0.2.23 — decision diagnostics + deterministic hand evaluation
-   Spectator-only annotations. The note explicitly separates typed model
-   answers from any fabricated rationale. */
-.decision-facts-note{margin-top:6px;font-size:10px;line-height:1.4;color:var(--muted,#8aa098);border-left:2px solid rgba(184,236,111,.35);padding-left:8px}
+/* 0.2.23 — decision diagnostics + deterministic hand evaluation. */
 .replay-reason-text{margin-bottom:10px;line-height:1.45}
 .replay-summary span b{white-space:nowrap}
 
@@ -2339,7 +2340,6 @@ a.button{text-decoration:none}
 .stage-chip{white-space:nowrap}
 .telemetry-group{margin-top:6px}
 .telemetry-caption{display:block;font-size:9px;letter-spacing:.08em;text-transform:uppercase;color:var(--muted,#8aa098);margin-bottom:3px}
-.decision-facts-note{white-space:normal}
 
 /* 0.4.0 — methodology, reproducibility and repository-organization release.
    Browser application source moved under src/; visual behavior is unchanged. */
@@ -2456,7 +2456,7 @@ a.button{text-decoration:none}
           <h2>Settings</h2>
           <button class="info-button" type="button" aria-label="About settings" data-tip="API keys stay in this browser tab and are never written to saved configuration or tournament logs.">i</button>
         </div>
-        <button id="closeSetup" type="button" class="icon-button" aria-label="Close settings">×</button>
+        <button id="closeSetup" type="button" class="icon-button" aria-label="Close settings"><svg class="icon-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="M4.5 4.5l7 7M11.5 4.5l-7 7" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round"/></svg></button>
       </div>
 
       <div class="dialog-body setup-body">
@@ -2537,7 +2537,7 @@ a.button{text-decoration:none}
           <h2 id="seatDialogTitle">Seat 1</h2>
           <button class="info-button" type="button" aria-label="About seat context" data-tip="Each model receives only its own hole cards plus the same canonical public table state, recent public hands, opponent statistics, and legal actions.">i</button>
         </div>
-        <button id="closeSeat" type="button" class="icon-button" aria-label="Close seat editor">×</button>
+        <button id="closeSeat" type="button" class="icon-button" aria-label="Close seat editor"><svg class="icon-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="M4.5 4.5l7 7M11.5 4.5l-7 7" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round"/></svg></button>
       </div>
       <div class="dialog-body seat-editor-body">
         <div id="seatLockNotice" class="seat-lock-notice hidden">Seat assignments are locked during a tournament.</div>
@@ -2575,7 +2575,7 @@ a.button{text-decoration:none}
           <div class="dialog-title-row"><h2>Decision sanity suite</h2><button class="info-button" type="button" aria-label="About decision tests" data-tip="A small comprehension and reliability check. It is intentionally not a GTO solver, Elo score, or tournament-strength ranking.">i</button></div>
           <div class="dialog-subtitle">The same 10 fixed poker spots are sent independently to every seated model.</div>
         </div>
-        <button id="closeTests" type="button" class="icon-button" aria-label="Close tests">×</button>
+        <button id="closeTests" type="button" class="icon-button" aria-label="Close tests"><svg class="icon-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="M4.5 4.5l7 7M11.5 4.5l-7 7" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round"/></svg></button>
       </div>
       <div class="tests-body">
         <div class="tests-how" aria-label="How the suite works">
@@ -2608,7 +2608,7 @@ a.button{text-decoration:none}
           <div class="dialog-title-row"><h2 id="replayTitle">Decision replay</h2><span id="replayBadge" class="replay-badge">HAND —</span></div>
           <div id="replaySubtitle" class="dialog-subtitle">Snapshot captured immediately before the model acted.</div>
         </div>
-        <button id="closeReplay" type="button" class="icon-button" aria-label="Close decision replay">×</button>
+        <button id="closeReplay" type="button" class="icon-button" aria-label="Close decision replay"><svg class="icon-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="M4.5 4.5l7 7M11.5 4.5l-7 7" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round"/></svg></button>
       </div>
       <div class="dialog-body replay-body">
         <section class="replay-table-wrap">
@@ -2652,7 +2652,7 @@ a.button{text-decoration:none}
           <span class="field-label-row">Preset <button class="info-button inline-info" type="button" aria-label="About connection presets" data-tip="OpenAI-compatible keeps Base URL fully editable. OpenRouter only fills its standard URL and enables OpenRouter-specific routing. TypeSafe uses the native System One endpoint.">i</button></span>
           <select data-field="kind"><option value="openai">OpenAI-compatible</option><option value="openrouter">OpenRouter</option><option value="typesafe">TypeSafe System One</option></select>
         </label>
-        <button type="button" class="remove-connection icon-button" title="Remove connection" aria-label="Remove connection">×</button>
+        <button type="button" class="remove-connection icon-button" title="Remove connection" aria-label="Remove connection"><svg class="icon-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="M4.5 4.5l7 7M11.5 4.5l-7 7" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round"/></svg></button>
       </div>
       <div class="connection-main-grid">
         <label class="connection-url">Base URL<input data-field="baseUrl" spellcheck="false" placeholder="https://api.example.com/v1" /></label>
@@ -117813,7 +117813,6 @@ function decide(agent, connection, args) {
 }
 var FAMILY_OBJECTIVE = "Choose exactly one legal action family that best maximizes tournament chip EV from the supplied state. Do not choose an amount; a size is chosen in a separate step.";
 var SIZE_OBJECTIVE = (family) => `The action family ${String(family).toUpperCase()} has already been selected. Choose exactly one legal ${family === ACTION_FAMILY.RAISE ? "raise size" : "bet size"} that best maximizes tournament chip EV. Do not change the action family.`;
-var SPECTATOR_NOTE = "This model returns typed decisions rather than a text rationale.";
 function buildJevFamilyQuestions(families, { instructions = FAMILY_OBJECTIVE, criteria = null } = {}) {
   return { type: "choice", instructions, criteria: criteria ?? familyCriteria(families) };
 }
@@ -118269,6 +118268,7 @@ var lastProcessedEventId = null;
 var MAX_LOBBY_SEATS = 10;
 var seatAssignments = Array(MAX_LOBBY_SEATS).fill(null);
 var editingSeatIndex = null;
+var seatNameAuto = false;
 var lobbyVisible = true;
 var pendingAutostart = false;
 var arenaMaxDecisions = 0;
@@ -119538,19 +119538,28 @@ function updateSeatSummary() {
   const count = seatAssignments.filter(Boolean).length;
   if (els.seatSummary) els.seatSummary.textContent = `${count} / ${MAX_LOBBY_SEATS} seated`;
 }
+function syncSeatNameFromModel() {
+  if (!seatNameAuto || editingSeatIndex == null) return;
+  const model = els.seatModel.value.trim();
+  els.seatName.value = model ? displayModelName(model) : `Player ${editingSeatIndex + 1}`;
+}
 function openSeatEditor(seatIndex) {
   if (!Number.isInteger(seatIndex) || seatIndex < 0 || seatIndex >= MAX_LOBBY_SEATS) return;
   editingSeatIndex = seatIndex;
   const locked = Boolean(director && ["RUNNING", "PAUSED"].includes(director.status));
   const draft = seatAssignments[seatIndex] ? { ...seatAssignments[seatIndex] } : defaultSeatDraft(seatIndex);
   els.seatDialogTitle.textContent = `Seat ${seatIndex + 1}`;
-  els.seatName.value = draft.name || `Player ${seatIndex + 1}`;
+  const placeholderName = `Player ${seatIndex + 1}`;
+  const storedName = String(draft.name || "").trim();
+  seatNameAuto = !storedName || /^player\s+\d+$/i.test(storedName) || Boolean(draft.model) && storedName === displayModelName(draft.model);
+  els.seatName.value = seatNameAuto && draft.model ? displayModelName(draft.model) : storedName || placeholderName;
   refreshSeatConnectionSelect(draft.connectionId || "");
   if (draft.connectionId && [...els.seatConnection.options].some((option) => option.value === draft.connectionId)) els.seatConnection.value = draft.connectionId;
   els.seatModel.value = draft.model || "";
   els.seatProtocol.value = draft.protocol || "tool";
   els.seatProvider.value = draft.provider || "";
   applySeatProtocolRules();
+  syncSeatNameFromModel();
   void refreshSeatModelCatalog();
   els.seatError.classList.add("hidden");
   els.seatLockNotice.classList.toggle("hidden", !locked);
@@ -119992,8 +120001,6 @@ function decisionTelemetryHtml(event, { limit = 4 } = {}) {
   const reasoningTokens = event?.usage?.completion_tokens_details?.reasoning_tokens;
   if (reasoningTokens) facts.push(`<span>Reasoning <b>${reasoningTokens} tokens</b></span>`);
   if (facts.length) parts.push(`<div class="decision-facts">${facts.join("")}</div>`);
-  const isTypedDecision = meta.family != null || meta.aggression != null || meta.bluffSpot != null || meta.method === "openrouter-decisions" || meta.method === "jev-choice" || String(meta.method || "").includes("hierarchical");
-  if (isTypedDecision) parts.push(`<div class="decision-facts-note">${escapeHtml(SPECTATOR_NOTE)} Bluff opportunity is a property of the spot, not the reason for the chosen action.</div>`);
   return parts.join("");
 }
 function setDecisionContext({ hand = "\u2014", street = "\u2014", position = "\u2014", options = "\u2014", label = "Legal actions", hint = "Choose one", labels = null } = {}) {
@@ -120251,7 +120258,7 @@ function cloneJson(value) {
 }
 function sanityAgents() {
   const connections = readConnections();
-  return readSeatPlayers().map((player) => ({ ...player, connection: connections.find((c) => c.id === player.connectionId) })).filter((row) => row.connection);
+  return readSeatPlayers().map((player) => ({ ...player, id: player.id || `player-${player.lobbySeat + 1}`, connection: connections.find((c) => c.id === player.connectionId) })).filter((row) => row.connection);
 }
 function sanityProtocolLabel(agent) {
   const protocol = effectiveProtocol(agent, agent.connection);
@@ -120963,9 +120970,16 @@ for (const dialog of [els.setupDialog, els.seatDialog, els.testsDialog, els.repl
 }
 els.seatConnection.addEventListener("change", () => {
   applySeatProtocolRules();
+  syncSeatNameFromModel();
   void refreshSeatModelCatalog();
 });
-els.seatModel.addEventListener("input", applySeatProtocolRules);
+els.seatModel.addEventListener("input", () => {
+  applySeatProtocolRules();
+  syncSeatNameFromModel();
+});
+els.seatName.addEventListener("input", () => {
+  seatNameAuto = false;
+});
 els.seatProtocol.addEventListener("change", applySeatProtocolRules);
 els.refreshModelsBtn?.addEventListener("click", () => void refreshSeatModelCatalog({ force: true }));
 els.removeSeatBtn.addEventListener("click", () => {

package/dist/styles.css

@@ -839,7 +839,11 @@ input::placeholder{color:#4e5862}
 .button,.icon-button,.field-icon-button,input,select{box-sizing:border-box}
 .button{min-height:var(--control-h);padding:0 12px;border-radius:var(--radius-sm);font-size:10px;font-weight:800;line-height:1;white-space:nowrap}
 .small-button{min-height:30px;height:30px;padding-inline:10px;font-size:9px}
-.icon-button{width:30px;height:30px;min-width:30px;border-radius:var(--radius-sm);display:grid;place-items:center;padding:0}
+.icon-button{width:30px;height:30px;min-width:30px;border-radius:var(--radius-sm);display:grid;place-items:center;padding:0;line-height:1}
+/* SVG glyphs center geometrically; a text "×" sits on its baseline and looks
+   vertically off-centre inside these small buttons. */
+.icon-button svg{width:14px;height:14px;display:block;pointer-events:none}
+.dialog-head .icon-button svg{width:13px;height:13px}
 input,select{width:100%;min-width:0;height:var(--control-h);padding:0 9px;border-radius:var(--radius-sm);font-size:11px;background:#0a0d10;border:1px solid var(--modal-border);color:var(--text)}
 input:focus,select:focus{outline:none;border-color:rgba(146,201,164,.34);box-shadow:0 0 0 3px rgba(146,201,164,.05)}
 label{min-width:0}
@@ -2265,10 +2269,7 @@ a.button{text-decoration:none}
   .card .card-corner.bottom{display:none!important}
 }
 
-/* 0.2.23 — decision diagnostics + deterministic hand evaluation
-   Spectator-only annotations. The note explicitly separates typed model
-   answers from any fabricated rationale. */
-.decision-facts-note{margin-top:6px;font-size:10px;line-height:1.4;color:var(--muted,#8aa098);border-left:2px solid rgba(184,236,111,.35);padding-left:8px}
+/* 0.2.23 — decision diagnostics + deterministic hand evaluation. */
 .replay-reason-text{margin-bottom:10px;line-height:1.45}
 .replay-summary span b{white-space:nowrap}
 
@@ -2283,7 +2284,6 @@ a.button{text-decoration:none}
 .stage-chip{white-space:nowrap}
 .telemetry-group{margin-top:6px}
 .telemetry-caption{display:block;font-size:9px;letter-spacing:.08em;text-transform:uppercase;color:var(--muted,#8aa098);margin-bottom:3px}
-.decision-facts-note{white-space:normal}
 
 /* 0.4.0 — methodology, reproducibility and repository-organization release.
    Browser application source moved under src/; visual behavior is unchanged. */

package/docs/diagnostics/SUMMARY.md

@@ -1,4 +1,4 @@
-# pokertools-arena 0.4.1 — benchmark methodology & results
+# pokertools-arena 0.4.4 — benchmark methodology & results
 
 Generated: 2026-09-19T18:19:58.352Z
 Methodology: paired fixed-state corpus; deterministic interleaving with recorded experiment seed.

package/docs/diagnostics/summary.json

@@ -1,6 +1,6 @@
 {
   "schemaVersion": 1,
-  "version": "0.4.1",
+  "version": "0.4.4",
   "generatedAt": "2026-09-19T18:19:58.352Z",
   "experiments": [
     "paired"

package/docs/releases/RELEASE_NOTES.md

@@ -1,5 +1,33 @@
 # Release notes
 
+## 0.4.4 — Maintenance
+
+- **Cleaner decision logs.** Removed the repeated spectator caption ("This
+  model returns typed decisions rather than a text rationale…") that appeared on
+  every typed decision in the live feed and the replay panel. Typed decisions
+  already show their selected family/size plus confidence telemetry, so logs are
+  concise and no longer restate that a rationale is absent. `SPECTATOR_NOTE` and
+  its CSS were removed; release checks guard against reintroduction.
+
+## 0.4.3 — Maintenance
+
+- **Seat editor name.** Opening a seat restored from `.env` or saved
+  configuration now shows the model-derived name (for example `Gemma`) instead
+  of the generic `Player 1`, matching the table, lobby and stats surfaces. The
+  name follows the chosen model until it is edited by hand.
+- **Close buttons.** Dialog close buttons and the remove-connection button use a
+  centered inline SVG cross instead of a baseline-aligned text `×`, so the glyph
+  is vertically centered.
+
+## 0.4.2 — Maintenance
+
+- **Decision sanity suite model attribution.** Seat assignments restored from
+  `.env` or saved configuration had no `id`, so every sanity agent shared an
+  undefined id. Each model summary then counted every model's decisions (for
+  example `24/10` when only 10 spots exist) and every scenario column rendered
+  the last model's result. Sanity agents now derive the same stable
+  `player-<seat>` id the tournament uses, so each model is reported separately.
+
 ## 0.4.1 — Maintenance
 
 A maintenance release on top of the 0.4.0 methodology work.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pokertools-arena",
-  "version": "0.4.1",
+  "version": "0.4.4",
   "description": "Browser-first AI poker benchmark powered by @pokertools/engine with generic OpenAI-compatible endpoints, optional OpenRouter/Jev Decisions routing, deterministic hierarchical decisions, paired fixed-state methodology, and reproducible reporting.",
   "type": "module",
   "bin": {

package/src/app.js

@@ -14,7 +14,7 @@ import {
   BENCHMARK_MODES, DECISION_ARCHITECTURES, DEFAULT_BENCHMARK_MODE, DEFAULT_DECISION_ARCHITECTURE,
   REPRESENTATION_MODES, DEFAULT_REPRESENTATION_MODE, DECISION_ARCHITECTURE_VERSION,
   legalActionFamilies, legalAggressiveSizes, buildHierarchicalDecision, familyCriteria, sizeCriteria,
-  applyBenchmarkMode, renderDecisionState, aggregateActionProbabilitiesByFamily, probabilityStats, SIZE_LABELS, SPECTATOR_NOTE,
+  applyBenchmarkMode, renderDecisionState, aggregateActionProbabilitiesByFamily, probabilityStats, SIZE_LABELS,
   isAggressiveType, familyForActionType, FAMILY_LABELS, decisionClockPhase,
 } from './lib/decision-core.js';
 
@@ -47,6 +47,7 @@ let lastProcessedEventId = null;
 const MAX_LOBBY_SEATS = 10;
 let seatAssignments = Array(MAX_LOBBY_SEATS).fill(null);
 let editingSeatIndex = null;
+let seatNameAuto = false;
 let lobbyVisible = true;
 let pendingAutostart = false;
 let arenaMaxDecisions = 0;
@@ -1065,19 +1066,31 @@ function updateSeatSummary() {
   const count = seatAssignments.filter(Boolean).length;
   if (els.seatSummary) els.seatSummary.textContent = `${count} / ${MAX_LOBBY_SEATS} seated`;
 }
+// The seat name follows the chosen model until the user edits it by hand, so
+// seats restored from .env or saved config show "Gemma" instead of the generic
+// "Player 1" the other surfaces already hide via visiblePlayerName().
+function syncSeatNameFromModel() {
+  if (!seatNameAuto || editingSeatIndex == null) return;
+  const model = els.seatModel.value.trim();
+  els.seatName.value = model ? displayModelName(model) : `Player ${editingSeatIndex + 1}`;
+}
 function openSeatEditor(seatIndex) {
   if (!Number.isInteger(seatIndex) || seatIndex < 0 || seatIndex >= MAX_LOBBY_SEATS) return;
   editingSeatIndex = seatIndex;
   const locked = Boolean(director && ['RUNNING', 'PAUSED'].includes(director.status));
   const draft = seatAssignments[seatIndex] ? { ...seatAssignments[seatIndex] } : defaultSeatDraft(seatIndex);
   els.seatDialogTitle.textContent = `Seat ${seatIndex + 1}`;
-  els.seatName.value = draft.name || `Player ${seatIndex + 1}`;
+  const placeholderName = `Player ${seatIndex + 1}`;
+  const storedName = String(draft.name || '').trim();
+  seatNameAuto = !storedName || /^player\s+\d+$/i.test(storedName) || (Boolean(draft.model) && storedName === displayModelName(draft.model));
+  els.seatName.value = seatNameAuto && draft.model ? displayModelName(draft.model) : (storedName || placeholderName);
   refreshSeatConnectionSelect(draft.connectionId || '');
   if (draft.connectionId && [...els.seatConnection.options].some(option => option.value === draft.connectionId)) els.seatConnection.value = draft.connectionId;
   els.seatModel.value = draft.model || '';
   els.seatProtocol.value = draft.protocol || 'tool';
   els.seatProvider.value = draft.provider || '';
   applySeatProtocolRules();
+  syncSeatNameFromModel();
   void refreshSeatModelCatalog();
   els.seatError.classList.add('hidden');
   els.seatLockNotice.classList.toggle('hidden', !locked);
@@ -1522,8 +1535,9 @@ function decisionTelemetryHtml(event, { limit = 4 } = {}) {
   const reasoningTokens = event?.usage?.completion_tokens_details?.reasoning_tokens;
   if (reasoningTokens) facts.push(`<span>Reasoning <b>${reasoningTokens} tokens</b></span>`);
   if (facts.length) parts.push(`<div class="decision-facts">${facts.join('')}</div>`);
-  const isTypedDecision = meta.family != null || meta.aggression != null || meta.bluffSpot != null || meta.method === 'openrouter-decisions' || meta.method === 'jev-choice' || String(meta.method || '').includes('hierarchical');
-  if (isTypedDecision) parts.push(`<div class="decision-facts-note">${escapeHtml(SPECTATOR_NOTE)} Bluff opportunity is a property of the spot, not the reason for the chosen action.</div>`);
+  // Typed decisions (hierarchical / Jev / OpenRouter-decisions) have no prose
+  // rationale; the telemetry above already shows family, size and confidence.
+  // A repeated "no text rationale" caption was removed as log noise.
   return parts.join('');
 }
 function setDecisionContext({ hand = '—', street = '—', position = '—', options = '—', label = 'Legal actions', hint = 'Choose one', labels = null } = {}) {
@@ -1746,7 +1760,13 @@ function openSetup({ preserveError = false } = {}) { if (!preserveError) els.set
 function cloneJson(value) { return JSON.parse(JSON.stringify(value)); }
 function sanityAgents() {
   const connections = readConnections();
-  return readSeatPlayers().map(player => ({ ...player, connection: connections.find(c => c.id === player.connectionId) })).filter(row => row.connection);
+  // Seat assignments restored from .env or saved config carry no id, so derive
+  // the same stable id the tournament uses. Without it every agent.id is
+  // undefined, so sanity results from all models collapse into one summary
+  // (e.g. "24/10") and every grid column shows the last model's result.
+  return readSeatPlayers()
+    .map(player => ({ ...player, id: player.id || `player-${player.lobbySeat + 1}`, connection: connections.find(c => c.id === player.connectionId) }))
+    .filter(row => row.connection);
 }
 function sanityProtocolLabel(agent) {
   const protocol = effectiveProtocol(agent, agent.connection);
@@ -2326,8 +2346,9 @@ for (const dialog of [els.setupDialog, els.seatDialog, els.testsDialog, els.repl
   dialog.addEventListener('click', event => { if (event.target === dialog) dialog.close(); });
   dialog.addEventListener('close', hideTooltip);
 }
-els.seatConnection.addEventListener('change', () => { applySeatProtocolRules(); void refreshSeatModelCatalog(); });
-els.seatModel.addEventListener('input', applySeatProtocolRules);
+els.seatConnection.addEventListener('change', () => { applySeatProtocolRules(); syncSeatNameFromModel(); void refreshSeatModelCatalog(); });
+els.seatModel.addEventListener('input', () => { applySeatProtocolRules(); syncSeatNameFromModel(); });
+els.seatName.addEventListener('input', () => { seatNameAuto = false; });
 els.seatProtocol.addEventListener('change', applySeatProtocolRules);
 els.refreshModelsBtn?.addEventListener('click', () => void refreshSeatModelCatalog({ force: true }));
 els.removeSeatBtn.addEventListener('click', () => {

package/src/index.html

@@ -165,7 +165,7 @@
           <h2>Settings</h2>
           <button class="info-button" type="button" aria-label="About settings" data-tip="API keys stay in this browser tab and are never written to saved configuration or tournament logs.">i</button>
         </div>
-        <button id="closeSetup" type="button" class="icon-button" aria-label="Close settings">×</button>
+        <button id="closeSetup" type="button" class="icon-button" aria-label="Close settings"><svg class="icon-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="M4.5 4.5l7 7M11.5 4.5l-7 7" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round"/></svg></button>
       </div>
 
       <div class="dialog-body setup-body">
@@ -246,7 +246,7 @@
           <h2 id="seatDialogTitle">Seat 1</h2>
           <button class="info-button" type="button" aria-label="About seat context" data-tip="Each model receives only its own hole cards plus the same canonical public table state, recent public hands, opponent statistics, and legal actions.">i</button>
         </div>
-        <button id="closeSeat" type="button" class="icon-button" aria-label="Close seat editor">×</button>
+        <button id="closeSeat" type="button" class="icon-button" aria-label="Close seat editor"><svg class="icon-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="M4.5 4.5l7 7M11.5 4.5l-7 7" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round"/></svg></button>
       </div>
       <div class="dialog-body seat-editor-body">
         <div id="seatLockNotice" class="seat-lock-notice hidden">Seat assignments are locked during a tournament.</div>
@@ -284,7 +284,7 @@
           <div class="dialog-title-row"><h2>Decision sanity suite</h2><button class="info-button" type="button" aria-label="About decision tests" data-tip="A small comprehension and reliability check. It is intentionally not a GTO solver, Elo score, or tournament-strength ranking.">i</button></div>
           <div class="dialog-subtitle">The same 10 fixed poker spots are sent independently to every seated model.</div>
         </div>
-        <button id="closeTests" type="button" class="icon-button" aria-label="Close tests">×</button>
+        <button id="closeTests" type="button" class="icon-button" aria-label="Close tests"><svg class="icon-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="M4.5 4.5l7 7M11.5 4.5l-7 7" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round"/></svg></button>
       </div>
       <div class="tests-body">
         <div class="tests-how" aria-label="How the suite works">
@@ -317,7 +317,7 @@
           <div class="dialog-title-row"><h2 id="replayTitle">Decision replay</h2><span id="replayBadge" class="replay-badge">HAND —</span></div>
           <div id="replaySubtitle" class="dialog-subtitle">Snapshot captured immediately before the model acted.</div>
         </div>
-        <button id="closeReplay" type="button" class="icon-button" aria-label="Close decision replay">×</button>
+        <button id="closeReplay" type="button" class="icon-button" aria-label="Close decision replay"><svg class="icon-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="M4.5 4.5l7 7M11.5 4.5l-7 7" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round"/></svg></button>
       </div>
       <div class="dialog-body replay-body">
         <section class="replay-table-wrap">
@@ -361,7 +361,7 @@
           <span class="field-label-row">Preset <button class="info-button inline-info" type="button" aria-label="About connection presets" data-tip="OpenAI-compatible keeps Base URL fully editable. OpenRouter only fills its standard URL and enables OpenRouter-specific routing. TypeSafe uses the native System One endpoint.">i</button></span>
           <select data-field="kind"><option value="openai">OpenAI-compatible</option><option value="openrouter">OpenRouter</option><option value="typesafe">TypeSafe System One</option></select>
         </label>
-        <button type="button" class="remove-connection icon-button" title="Remove connection" aria-label="Remove connection">×</button>
+        <button type="button" class="remove-connection icon-button" title="Remove connection" aria-label="Remove connection"><svg class="icon-glyph" viewBox="0 0 16 16" aria-hidden="true" focusable="false"><path d="M4.5 4.5l7 7M11.5 4.5l-7 7" fill="none" stroke="currentColor" stroke-width="1.6" stroke-linecap="round"/></svg></button>
       </div>
       <div class="connection-main-grid">
         <label class="connection-url">Base URL<input data-field="baseUrl" spellcheck="false" placeholder="https://api.example.com/v1" /></label>

package/src/lib/decision-core.js

@@ -1000,8 +1000,6 @@ export function decide(agent, connection, args) {
 // ===========================================================================
 export const FAMILY_OBJECTIVE = 'Choose exactly one legal action family that best maximizes tournament chip EV from the supplied state. Do not choose an amount; a size is chosen in a separate step.';
 export const SIZE_OBJECTIVE = family => `The action family ${String(family).toUpperCase()} has already been selected. Choose exactly one legal ${family === ACTION_FAMILY.RAISE ? 'raise size' : 'bet size'} that best maximizes tournament chip EV. Do not change the action family.`;
-export const SPECTATOR_NOTE = 'This model returns typed decisions rather than a text rationale.';
-
 export function buildJevFamilyQuestions(families, { instructions = FAMILY_OBJECTIVE, criteria = null } = {}) {
   return { type: 'choice', instructions, criteria: criteria ?? familyCriteria(families) };
 }

package/src/styles.css

@@ -839,7 +839,11 @@ input::placeholder{color:#4e5862}
 .button,.icon-button,.field-icon-button,input,select{box-sizing:border-box}
 .button{min-height:var(--control-h);padding:0 12px;border-radius:var(--radius-sm);font-size:10px;font-weight:800;line-height:1;white-space:nowrap}
 .small-button{min-height:30px;height:30px;padding-inline:10px;font-size:9px}
-.icon-button{width:30px;height:30px;min-width:30px;border-radius:var(--radius-sm);display:grid;place-items:center;padding:0}
+.icon-button{width:30px;height:30px;min-width:30px;border-radius:var(--radius-sm);display:grid;place-items:center;padding:0;line-height:1}
+/* SVG glyphs center geometrically; a text "×" sits on its baseline and looks
+   vertically off-centre inside these small buttons. */
+.icon-button svg{width:14px;height:14px;display:block;pointer-events:none}
+.dialog-head .icon-button svg{width:13px;height:13px}
 input,select{width:100%;min-width:0;height:var(--control-h);padding:0 9px;border-radius:var(--radius-sm);font-size:11px;background:#0a0d10;border:1px solid var(--modal-border);color:var(--text)}
 input:focus,select:focus{outline:none;border-color:rgba(146,201,164,.34);box-shadow:0 0 0 3px rgba(146,201,164,.05)}
 label{min-width:0}
@@ -2265,10 +2269,7 @@ a.button{text-decoration:none}
   .card .card-corner.bottom{display:none!important}
 }
 
-/* 0.2.23 — decision diagnostics + deterministic hand evaluation
-   Spectator-only annotations. The note explicitly separates typed model
-   answers from any fabricated rationale. */
-.decision-facts-note{margin-top:6px;font-size:10px;line-height:1.4;color:var(--muted,#8aa098);border-left:2px solid rgba(184,236,111,.35);padding-left:8px}
+/* 0.2.23 — decision diagnostics + deterministic hand evaluation. */
 .replay-reason-text{margin-bottom:10px;line-height:1.45}
 .replay-summary span b{white-space:nowrap}
 
@@ -2283,7 +2284,6 @@ a.button{text-decoration:none}
 .stage-chip{white-space:nowrap}
 .telemetry-group{margin-top:6px}
 .telemetry-caption{display:block;font-size:9px;letter-spacing:.08em;text-transform:uppercase;color:var(--muted,#8aa098);margin-bottom:3px}
-.decision-facts-note{white-space:normal}
 
 /* 0.4.0 — methodology, reproducibility and repository-organization release.
    Browser application source moved under src/; visual behavior is unchanged. */