git-watchtower 1.14.3 → 1.14.4

package/bin/git-watchtower.js

@@ -2207,7 +2207,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) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "git-watchtower",
-  "version": "1.14.3",
+  "version": "1.14.4",
   "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 */
+            }
           }
         }
       });