git-watchtower 1.11.3 → 1.11.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "git-watchtower",
-  "version": "1.11.3",
+  "version": "1.11.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/config/loader.js

@@ -71,8 +71,9 @@ function loadConfig(projectRoot) {
     return null;
   }
 
-  // Migrate old format if needed
-  return migrateConfig(raw);
+  // Migrate old format if needed, then validate
+  const migrated = migrateConfig(raw);
+  return validateConfig(migrated);
 }
 
 /**

package/src/git/commands.js

@@ -336,6 +336,8 @@ async function getChangedFiles(branchName, baseBranch = 'HEAD', cwd) {
  * @returns {{added: number, deleted: number}}
  */
 function parseDiffStats(diffStatOutput) {
+  if (!diffStatOutput) return { added: 0, deleted: 0 };
+
   // Parse the summary line: "X files changed, Y insertions(+), Z deletions(-)"
   const match = diffStatOutput.match(/(\d+) insertions?\(\+\).*?(\d+) deletions?\(-\)/);
   if (match) {

package/src/server/coordinator.js

@@ -318,7 +318,15 @@ class Coordinator {
    */
   _handleWorkerMessage(socket, msg, setWorkerId, getWorkerId) {
     switch (msg.type) {
-      case 'register':
+      case 'register': {
+        // Prevent re-registration: a socket that already registered cannot change its ID
+        const currentId = getWorkerId();
+        if (currentId) break;
+
+        // Reject if this ID is already claimed by a different socket
+        const existingSocket = this.workerSockets.get(msg.id);
+        if (existingSocket && existingSocket !== socket) break;
+
         setWorkerId(msg.id);
         this.workerSockets.set(msg.id, socket);
         this.projects.set(msg.id, {
@@ -331,6 +339,7 @@ class Coordinator {
         this._sendMessage(socket, { type: 'registered', id: msg.id });
         this._notifyProjectsChanged();
         break;
+      }
 
       case 'state': {
         // Validate sender — only accept state for the worker's own registered ID

package/src/server/static.js

@@ -63,33 +63,9 @@ function injectLiveReload(html) {
   return html;
 }
 
-/**
- * Parse git diff --stat output into { added, deleted } counts.
- * @param {string} diffOutput - Output from `git diff --stat`
- * @returns {{ added: number, deleted: number }}
- */
-function parseDiffStats(diffOutput) {
-  if (!diffOutput) return { added: 0, deleted: 0 };
-
-  // Parse the summary line: "X files changed, Y insertions(+), Z deletions(-)"
-  const match = diffOutput.match(/(\d+) insertions?\(\+\).*?(\d+) deletions?\(-\)/);
-  if (match) {
-    return { added: parseInt(match[1], 10), deleted: parseInt(match[2], 10) };
-  }
-
-  // Try to match just insertions or just deletions
-  const insertMatch = diffOutput.match(/(\d+) insertions?\(\+\)/);
-  const deleteMatch = diffOutput.match(/(\d+) deletions?\(-\)/);
-  return {
-    added: insertMatch ? parseInt(insertMatch[1], 10) : 0,
-    deleted: deleteMatch ? parseInt(deleteMatch[1], 10) : 0,
-  };
-}
-
 module.exports = {
   MIME_TYPES,
   getMimeType,
   LIVE_RELOAD_SCRIPT,
   injectLiveReload,
-  parseDiffStats,
 };