git-watchtower 2.3.7 → 2.3.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "git-watchtower",
-  "version": "2.3.7",
+  "version": "2.3.9",
   "description": "Terminal-based Git branch monitor with activity sparklines and optional dev server with live reload",
   "main": "bin/git-watchtower.js",
   "bin": {

package/src/casino/index.js

@@ -699,6 +699,24 @@ function getStats() {
   };
 }
 
+/**
+ * Stats payload safe for SSE serialization. Excludes the random/clock-driven
+ * decorative fields (luckMeter, houseEdge, vibesQuality, timeSinceLastHit)
+ * because they tick on every call — leaving them in the payload defeats
+ * web.js's `lastPushedJson` dedup, so the full ~100 KB state is shipped to
+ * every connected client every 500 ms even when nothing has actually
+ * changed. The dashboard recomputes these client-side at render time.
+ *
+ * Stable but slow-ticking fields (sessionDuration, which changes once per
+ * minute) are kept — the cost is at most one extra push per minute.
+ *
+ * @returns {Object}
+ */
+function getSerializableStats() {
+  const { luckMeter, houseEdge, vibesQuality, timeSinceLastHit, ...stable } = getStats();
+  return stable;
+}
+
 /**
  * Reset session stats
  */
@@ -788,6 +806,7 @@ module.exports = {
   // Stats
   recordPoll,
   getStats,
+  getSerializableStats,
   resetStats,
   getStatsDisplay,
 };

package/src/server/web-ui/js.js

@@ -1241,7 +1241,31 @@ ${pureFnBlock}
     return left + right;
   }
 
+  // Decorative casino fields the server intentionally does NOT include in
+  // the SSE payload because they're random/clock-driven and would defeat
+  // the lastPushedJson dedup. Recompute them locally at render time so the
+  // dashboard stays visually identical to the TUI.
+  const CASINO_VIBES = ['\u{1f60e}','\u{1f525}','✨','\u{1f4ab}','\u{1f31f}','⚡','\u{1f3af}','\u{1f4aa}','\u{1f680}','\u{1f4af}'];
+  function computeCasinoDecorations(cs) {
+    const baseLuck = 50 + Math.random() * 30;
+    const streakBonus = Math.min((cs.consecutivePolls || 0) * 5, 20);
+    return {
+      luckMeter: Math.min(Math.round(baseLuck + streakBonus), 99),
+      houseEdge: Math.round(55 + Math.random() * 45),
+      vibesQuality: CASINO_VIBES[Math.floor(Date.now() / 3000) % CASINO_VIBES.length],
+    };
+  }
+
   function renderCasinoStatsRow(cs) {
+    // Server-supplied (preferred) OR locally computed (default). The
+    // fallback covers older servers that still ship the decorative fields,
+    // plus tests that stub them in explicitly. != null treats undefined and
+    // null as "missing" so legitimate zero values from the server survive.
+    const local = computeCasinoDecorations(cs);
+    const houseEdge = (cs.houseEdge != null) ? cs.houseEdge : local.houseEdge;
+    const luckMeter = (cs.luckMeter != null) ? cs.luckMeter : local.luckMeter;
+    const vibesQuality = cs.vibesQuality || local.vibesQuality;
+
     const netClass = cs.netWinnings >= 0 ? 'pos' : 'neg';
     const netSign = cs.netWinnings >= 0 ? '+' : '';
     const stat = (k, v) => '<span class="stat"><span class="stat-k">' + k + '</span><span class="stat-v">' + v + '</span></span>';
@@ -1256,9 +1280,9 @@ ${pureFnBlock}
     let right = '<div class="stats-group">';
     right += stat('\u{1f4b8} Cost', '<span class="neg">' + dollar + (cs.totalPolls || 0) + '</span>');
     right += stat('\u{1f4b0} Net', '<span class="' + netClass + '">' + netSign + dollar + (cs.netWinnings || 0) + '</span>');
-    right += stat('\u{1f3b0} Edge', '<span class="neon">' + (cs.houseEdge || 0) + '%</span>');
-    right += stat('\u{1f3b2} Luck', '<span class="gold">' + (cs.luckMeter || 0) + '%</span>');
-    right += stat('\u{1f60e} Vibes', escHtml(cs.vibesQuality || ''));
+    right += stat('\u{1f3b0} Edge', '<span class="neon">' + houseEdge + '%</span>');
+    right += stat('\u{1f3b2} Luck', '<span class="gold">' + luckMeter + '%</span>');
+    right += stat('\u{1f60e} Vibes', escHtml(vibesQuality));
     right += stat('\u{1f9e0} Hits', '<span class="pos">' + (cs.dopamineHits || 0) + '</span>');
     if (cs.consecutivePolls > 1) right += stat('\u{1f525} Streak', '<span class="gold">' + cs.consecutivePolls + 'x</span>');
     right += '</div>';

package/src/server/web.js

@@ -147,8 +147,12 @@ class WebDashboardServer {
       // the mode on, so the web dashboard can render the same winnings box
       // the terminal does. Null when disabled — keeps payload small and
       // avoids ticking Math.random()/Date.now() into every SSE push when
-      // nobody's asked for the effect.
-      casinoStats: s.casinoModeEnabled ? casino.getStats() : null,
+      // nobody's asked for the effect. We use `getSerializableStats` (not
+      // the full getStats) so the random/clock-driven decorative fields
+      // (luckMeter, houseEdge, vibesQuality, timeSinceLastHit) don't
+      // defeat the lastPushedJson dedup — the dashboard recomputes them
+      // client-side from the stable counters that ARE in the payload.
+      casinoStats: s.casinoModeEnabled ? casino.getSerializableStats() : null,
       projectName: s.projectName,
 
       // Activity

package/src/ui/renderer.js

@@ -1505,13 +1505,19 @@ function renderUpdateModal(state, write) {
   lines.push(updateCmd);
   lines.push('');
 
+  // Tracks where the selectable option block starts so the highlight
+  // logic below can reverse-map line index → option index. -1 when
+  // there are no options (the updateInProgress path). Computed from
+  // `lines.length` so adding any line above never silently desyncs the
+  // highlight — previously this was a hardcoded `7` that happened to
+  // match the line count above by coincidence.
+  let optionStartIdx = -1;
   if (state.updateInProgress) {
     lines.push('Updating...');
     lines.push('');
     lines.push('Please wait...');
   } else {
-    // Option lines
-    const optionStartIdx = lines.length;
+    optionStartIdx = lines.length;
     for (const opt of options) {
       lines.push(opt);
     }
@@ -1519,7 +1525,6 @@ function renderUpdateModal(state, write) {
     lines.push('[Enter] Select  [Esc] Dismiss');
   }
 
-  const optionStartIdx = state.updateInProgress ? -1 : 7;
   const height = lines.length + 2;
 
   // Draw magenta double-border box