git-watchtower 1.14.3 → 1.14.5

package/bin/git-watchtower.js

@@ -1958,6 +1958,8 @@ async function pollGitChanges() {
         lastPrStatusFetch = Date.now();
         prStatusFetchInFlight = false;
       }).catch(() => {
+        // gh/glab errored (unauthed, rate-limited, network). PR indicators
+        // keep their last-known state; the next poll tick will retry.
         prStatusFetchInFlight = false;
       });
     }
@@ -2207,7 +2209,12 @@ function createStaticServer() {
   let realStaticDir;
   try {
     realStaticDir = fs.realpathSync(resolvedStaticDir);
-  } catch {
+  } catch (e) {
+    // STATIC_DIR comes from our own package layout, so a realpath failure
+    // means the install is broken (missing dir, permissions, etc.) — worth
+    // diagnosing. Fall back to the unresolved path so the request still
+    // gets its 403 rather than crashing.
+    telemetry.captureError(e);
     realStaticDir = resolvedStaticDir;
   }
   if (!resolvedPath.startsWith(realStaticDir + path.sep) && resolvedPath !== realStaticDir) {
@@ -2857,6 +2864,8 @@ function setupKeyboardInput() {
               render();
             }
           }).catch(() => {
+            // Async enrichment failed (no remote, gh/glab errored, etc.).
+            // Drop the spinner so the modal shows what we have from phase 1.
             if (store.get('actionMode') && store.get('actionData') && store.get('actionData').branch.name === branch.name) {
               store.setState({ actionLoading: false });
               render();
@@ -3529,6 +3538,23 @@ process.on('uncaughtException', async (err) => {
   process.exit(1);
 });
 
+// Mirror of uncaughtException for unhandled promise rejections. Without this,
+// Node 15+ crashes the process on a missed .catch() tail with no telemetry
+// and no terminal restore — leaving the TUI user in a broken terminal. Also
+// high-signal: an unhandled rejection reaching here means we missed a .catch()
+// somewhere and telemetry will tell us where.
+process.on('unhandledRejection', async (reason) => {
+  isShuttingDown = true;
+
+  cleanupResources();
+
+  const err = reason instanceof Error ? reason : new Error(String(reason));
+  try { telemetry.captureError(err); } catch (_) { /* telemetry must never prevent crash cleanup */ }
+  console.error('Unhandled rejection:', reason);
+  try { await telemetry.shutdown(); } catch (_) { /* telemetry must never prevent crash cleanup */ }
+  process.exit(1);
+});
+
 // ============================================================================
 // Startup
 // ============================================================================

package/package.json

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

package/src/server/coordinator.js

@@ -20,6 +20,7 @@ const fs = require('fs');
 const path = require('path');
 const os = require('os');
 const crypto = require('crypto');
+const telemetry = require('../telemetry');
 
 /**
  * Directory for watchtower runtime files
@@ -359,7 +360,12 @@ class Coordinator {
           try {
             const msg = JSON.parse(line);
             this._handleWorkerMessage(socket, msg, (id) => { workerId = id; }, () => workerId);
-          } catch (e) { /* malformed IPC frame from worker — skip it and keep reading */ }
+          } catch (e) {
+            // Both sides of this socket are our own code, so a JSON-parse
+            // failure indicates a protocol/version bug worth diagnosing.
+            telemetry.captureError(e);
+            /* skip malformed frame and keep reading */
+          }
         }
       }
     });
@@ -517,7 +523,12 @@ class Worker {
             try {
               const msg = JSON.parse(line);
               this._handleMessage(msg);
-            } catch (e) { /* malformed IPC frame from coordinator — skip it and keep reading */ }
+            } catch (e) {
+              // Both sides of this socket are our own code, so a JSON-parse
+              // failure indicates a protocol/version bug worth diagnosing.
+              telemetry.captureError(e);
+              /* skip malformed frame and keep reading */
+            }
           }
         }
       });