git-watchtower 1.9.1 → 1.9.3

package/bin/git-watchtower.js

@@ -82,6 +82,7 @@ const { parseGitHubPr, parseGitLabMr, parseGitHubPrList, parseGitLabMrList, isBa
 // Security & Validation (imported from src/git/branch.js and src/git/commands.js)
 // ============================================================================
 const { isValidBranchName, sanitizeBranchName, getGoneBranches, deleteGoneBranches } = require('../src/git/branch');
+const { pruneStaleEntries } = require('../src/polling/engine');
 const { isGitAvailable: checkGitAvailable, execGit, execGitSilent, getDiffStats: getDiffStatsSafe, getAheadBehind, getDiffShortstat } = require('../src/git/commands');
 
 // Session stats (always-on, non-casino stats)
@@ -993,11 +994,11 @@ async function refreshAllSparklines() {
     return; // Don't refresh too often
   }
 
-  try {
-    const currentBranches = store.get('branches');
-    for (const branch of currentBranches.slice(0, 20)) { // Limit to top 20
-      if (branch.isDeleted) continue;
+  const currentBranches = store.get('branches');
+  for (const branch of currentBranches.slice(0, 20)) { // Limit to top 20
+    if (branch.isDeleted) continue;
 
+    try {
       // Get commit counts for last 7 days (try remote, fall back to local)
       const sparkResult = await execGitSilent(
         ['log', `origin/${branch.name}`, '--since=7 days ago', '--format=%ad', '--date=format:%Y-%m-%d'],
@@ -1027,11 +1028,11 @@ async function refreshAllSparklines() {
 
       const counts = Array.from(dayCounts.values());
       store.get('sparklineCache').set(branch.name, generateSparkline(counts));
+    } catch (e) {
+      // Skip this branch - don't let one failure abort all sparkline updates
     }
-    lastSparklineUpdate = now;
-  } catch (e) {
-    // Silently fail - sparklines are optional
   }
+  lastSparklineUpdate = now;
 }
 
 async function getPreviewData(branchName) {
@@ -1817,6 +1818,16 @@ async function pollGitChanges() {
       }
     }
 
+    // Prune stale entries: remove branches from tracking sets/caches
+    // that no longer exist in git (deleted >30s ago or already gone)
+    pruneStaleEntries({
+      knownBranchNames,
+      fetchedBranchNames,
+      allBranches,
+      caches: [previousBranchStates, prInfoCache, store.get('sparklineCache'), store.get('aheadBehindCache')],
+      now,
+    });
+
     // Note: isNew flag is only cleared when branch becomes current (see below)
 
     // Keep deleted branches in the list (don't remove them)

package/package.json

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

package/src/polling/engine.js

@@ -147,6 +147,38 @@ function restoreSelection(branches, previousName, previousIndex) {
   return { selectedIndex: previousIndex, selectedBranchName: previousName };
 }
 
+/**
+ * Prune stale entries from tracking sets and caches for branches
+ * that no longer exist in git.
+ * @param {Object} opts
+ * @param {Set<string>} opts.knownBranchNames - Known branch names (mutated)
+ * @param {Set<string>} opts.fetchedBranchNames - Currently fetched branch names
+ * @param {Array<{name: string, isDeleted?: boolean, deletedAt?: number}>} opts.allBranches - All branches including deleted
+ * @param {Map<string, *>[]} opts.caches - Maps to prune stale keys from
+ * @param {number} [opts.retentionMs=30000] - How long to keep deleted branches before pruning
+ * @param {number} [opts.now] - Current timestamp (defaults to Date.now())
+ * @returns {string[]} Names of pruned branches
+ */
+function pruneStaleEntries({ knownBranchNames, fetchedBranchNames, allBranches, caches, retentionMs = 30000, now }) {
+  const timestamp = now ?? Date.now();
+  const pruned = [];
+  for (const knownName of [...knownBranchNames]) {
+    if (fetchedBranchNames.has(knownName)) continue;
+    const deletedBranch = allBranches.find(b => b.name === knownName && b.isDeleted);
+    const shouldPrune = deletedBranch
+      ? (deletedBranch.deletedAt && (timestamp - deletedBranch.deletedAt) > retentionMs)
+      : true; // Not in allBranches at all — stale entry
+    if (shouldPrune) {
+      knownBranchNames.delete(knownName);
+      for (const cache of caches) {
+        cache.delete(knownName);
+      }
+      pruned.push(knownName);
+    }
+  }
+  return pruned;
+}
+
 module.exports = {
   detectNewBranches,
   detectDeletedBranches,
@@ -154,4 +186,5 @@ module.exports = {
   sortBranches,
   calculateAdaptiveInterval,
   restoreSelection,
+  pruneStaleEntries,
 };