git-watchtower 2.3.6 → 2.3.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "git-watchtower",
-  "version": "2.3.6",
+  "version": "2.3.8",
   "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

@@ -283,6 +283,16 @@ function startSlotReels(renderCallback) {
  * @param {Object|null} winLevel - The win level object from getWinLevel()
  */
 function stopSlotReels(hadUpdates = false, renderCallback = null, winLevel = null) {
+  // No-op when casino mode is off. The bin's polling path captures
+  // `casinoOn` once at the top of pollGitChanges and continues to use that
+  // snapshot for the rest of the cycle, so a poll completing AFTER the
+  // user toggled casino off would otherwise install a fresh
+  // slotResultInterval that fires render() ~20× over the next 3s. The
+  // display getters are already guarded by `casinoEnabled`, so the user
+  // sees nothing — but render() still burns full-screen redraws and the
+  // interval keeps a closure alive. Drop the call cleanly here instead.
+  if (!casinoEnabled) return;
+
   if (slotReelInterval) {
     clearInterval(slotReelInterval);
     slotReelInterval = null;
@@ -689,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
  */
@@ -778,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