clay-server 2.29.4-beta.2 → 2.29.4

package/lib/public/modules/app-messages.js

@@ -1135,13 +1135,8 @@ export function processMessage(msg) {
           var _s1 = store.getState();
           if (fromId && fromId !== _s1.myUserId) {
             _s1.dmUnread[fromId] = (_s1.dmUnread[fromId] || 0) + 1;
-            // Only re-render the full strip if the sender isn't already visible
-            // (e.g. non-favorited user sending their first message). Otherwise
-            // just update the badge in-place to avoid a full DOM rebuild flicker.
-            var existingEl = document.querySelector('.icon-strip-user[data-user-id="' + fromId + '"]');
-            if (!existingEl) {
-              renderUserStrip(_s1.cachedAllUsers, _s1.cachedOnlineIds, _s1.myUserId, _s1.cachedDmFavorites, _s1.cachedDmConversations, _s1.dmUnread, _s1.dmRemovedUsers, _s1.cachedMatesList);
-            }
+            // Re-render strip so non-favorited sender appears
+            renderUserStrip(_s1.cachedAllUsers, _s1.cachedOnlineIds, _s1.myUserId, _s1.cachedDmFavorites, _s1.cachedDmConversations, _s1.dmUnread, _s1.dmRemovedUsers, _s1.cachedMatesList);
             updateDmBadge(fromId, _s1.dmUnread[fromId]);
           }
         }

package/lib/public/modules/app-projects.js

@@ -181,23 +181,40 @@ export function setCachedRemovedProjects(v) { cachedRemovedProjects = v; }
 
 export function updateProjectList(msg) {
   if (typeof msg.projectCount === "number") cachedProjectCount = msg.projectCount;
-  if (msg.projects) cachedProjects = msg.projects;
+
+  // Compare projects before caching to detect actual changes
+  var projectsChanged = false;
+  if (msg.projects) {
+    var projectsJson = JSON.stringify(msg.projects);
+    if (projectsJson !== _lastProjectsJson) {
+      projectsChanged = true;
+      _lastProjectsJson = projectsJson;
+    }
+    cachedProjects = msg.projects;
+  }
   if (msg.removedProjects) cachedRemovedProjects = msg.removedProjects;
   else if (msg.removedProjects === undefined) { /* keep cached */ }
   else cachedRemovedProjects = [];
-  var count = cachedProjectCount || 0;
-  renderProjectList();
-  var projectHint = _ctx.$("project-hint");
-  if (count === 1 && projectHint) {
-    try {
-      if (!localStorage.getItem("clay-project-hint-dismissed")) {
-        projectHint.classList.remove("hidden");
-      }
-    } catch (e) {}
-  } else if (projectHint) {
-    projectHint.classList.add("hidden");
+
+  // Only re-render project strip + title bar if data or active slug changed
+  var slugChanged = _ctx.currentSlug !== _lastRenderedSlug;
+  if (projectsChanged || slugChanged) {
+    _lastRenderedSlug = _ctx.currentSlug;
+    var count = cachedProjectCount || 0;
+    renderProjectList();
+    var projectHint = _ctx.$("project-hint");
+    if (count === 1 && projectHint) {
+      try {
+        if (!localStorage.getItem("clay-project-hint-dismissed")) {
+          projectHint.classList.remove("hidden");
+        }
+      } catch (e) {}
+    } else if (projectHint) {
+      projectHint.classList.add("hidden");
+    }
   }
-  // Update topbar with server-wide presence
+
+  // Update topbar with server-wide presence (renderTopbarPresence has its own guard)
   if (msg.serverUsers) {
     var newOnlineIds = msg.serverUsers.map(function (u) { return u.id; });
     var prevOnlineIds = _ctx.cachedOnlineIds || [];
@@ -211,12 +228,24 @@ export function updateProjectList(msg) {
       }
     }
   }
-  // Update user strip (DM targets) in icon strip
+
+  // Update user strip (DM targets) - only if user data actually changed
   if (msg.allUsers) {
+    var allUsersJson = JSON.stringify(msg.allUsers);
+    var dmFavJson = msg.dmFavorites ? JSON.stringify(msg.dmFavorites) : _lastDmFavJson;
+    var dmConvJson = msg.dmConversations ? JSON.stringify(msg.dmConversations) : _lastDmConvJson;
+    var userDataChanged = allUsersJson !== _lastAllUsersJson || dmFavJson !== _lastDmFavJson || dmConvJson !== _lastDmConvJson;
+
     _ctx.setCachedAllUsers(msg.allUsers);
     if (msg.dmFavorites) _ctx.setCachedDmFavorites(msg.dmFavorites);
     if (msg.dmConversations) _ctx.setCachedDmConversations(msg.dmConversations);
-    _ctx.renderUserStrip(msg.allUsers, _ctx.cachedOnlineIds, _ctx.myUserId, _ctx.cachedDmFavorites, _ctx.cachedDmConversations, _ctx.dmUnread, _ctx.dmRemovedUsers, _ctx.cachedMatesList);
+
+    if (userDataChanged) {
+      _lastAllUsersJson = allUsersJson;
+      _lastDmFavJson = dmFavJson;
+      _lastDmConvJson = dmConvJson;
+      _ctx.renderUserStrip(msg.allUsers, _ctx.cachedOnlineIds, _ctx.myUserId, _ctx.cachedDmFavorites, _ctx.cachedDmConversations, _ctx.dmUnread, _ctx.dmRemovedUsers, _ctx.cachedMatesList);
+    }
     if (document.body.classList.contains("mate-dm-active") || document.body.classList.contains("wide-view")) {
       var refreshedMyUser = _ctx.cachedAllUsers.find(function (u) { return u.id === _ctx.myUserId; });
       if (refreshedMyUser) {
@@ -240,6 +269,11 @@ export function updateProjectList(msg) {
 }
 
 var _lastTopbarUserIds = [];
+var _lastProjectsJson = "";
+var _lastRenderedSlug = null;
+var _lastAllUsersJson = "";
+var _lastDmFavJson = "";
+var _lastDmConvJson = "";
 export function renderTopbarPresence(serverUsers) {
   var countEl = document.getElementById("client-count");
   if (!countEl) return;
@@ -311,11 +345,7 @@ export function renderProjectList() {
       }
     }
   }
-  // Clear stale socket status from all project dots, then re-apply to active
-  var allDots = document.querySelectorAll("#icon-strip-projects .icon-strip-status");
-  for (var di = 0; di < allDots.length; di++) {
-    allDots[di].classList.remove("connected", "processing");
-  }
+  // Re-apply current socket status to the active icon's dot
   var dot = _ctx.getStatusDot();
   if (dot) {
     if (_ctx.connected && _ctx.processing) { dot.classList.add("connected"); dot.classList.add("processing"); }

package/lib/public/modules/sidebar-projects.js

@@ -1170,62 +1170,12 @@ function showWorktreeModal(parentSlug, parentName) {
 
 // --- Render icon strip ---
 
-// Build a fingerprint string for the project list so we can detect when only
-// the active slug changed (no structural DOM rebuild needed).
-function projectListFingerprint(projects) {
-  var parts = [];
-  for (var i = 0; i < projects.length; i++) {
-    var p = projects[i];
-    parts.push(p.slug + ":" + (p.icon || "") + ":" + (p.isProcessing ? 1 : 0) + ":" + (p.unread || 0) + ":" + (p.pendingPermissions || 0) + ":" + (p.isWorktree ? 1 : 0) + ":" + (p.parentSlug || "") + ":" + (p.worktreeAccessible !== false ? 1 : 0));
-  }
-  return parts.join("|");
-}
-var _lastStripFingerprint = "";
-
 export function renderIconStrip(projects, currentSlug) {
   cachedProjectList = projects;
   cachedCurrentSlug = currentSlug;
 
   var container = document.getElementById("icon-strip-projects");
   if (!container) return;
-
-  // Fast path: if only the active slug changed (project list unchanged),
-  // just toggle active classes instead of a full DOM rebuild.
-  var fp = projectListFingerprint(projects);
-  if (fp === _lastStripFingerprint && container.children.length > 0) {
-    var currentDmUserId2 = _ctx.getCurrentDmUserId ? _ctx.getCurrentDmUserId() : null;
-    var items = container.querySelectorAll(".icon-strip-item");
-    for (var fi = 0; fi < items.length; fi++) {
-      var isNowActive = items[fi].dataset.slug === currentSlug && !currentDmUserId2;
-      items[fi].classList.toggle("active", isNowActive);
-      // Clear stale socket status dot from inactive items
-      if (!isNowActive) {
-        var statusDot = items[fi].querySelector(".icon-strip-status");
-        if (statusDot) statusDot.classList.remove("connected", "processing");
-      }
-      // Update unread badge visibility (active items hide their badge)
-      var badge = items[fi].querySelector(".icon-strip-project-badge");
-      if (badge && badge.classList.contains("has-unread") && isNowActive) {
-        badge.textContent = "";
-        badge.classList.remove("has-unread");
-      }
-      // Update pending permission glow
-      if (isNowActive) items[fi].classList.remove("has-pending-perm");
-    }
-    // Update home icon
-    var homeIcon2 = document.querySelector(".icon-strip-home");
-    if (homeIcon2) {
-      if ((!currentSlug || projects.length === 0) && !currentDmUserId2) {
-        homeIcon2.classList.add("active");
-      } else {
-        homeIcon2.classList.remove("active");
-      }
-    }
-    renderProjectList(projects, currentSlug);
-    return;
-  }
-  _lastStripFingerprint = fp;
-
   container.innerHTML = "";
 
   var currentDmUserId = _ctx.getCurrentDmUserId ? _ctx.getCurrentDmUserId() : null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "clay-server",
-  "version": "2.29.4-beta.2",
+  "version": "2.29.4",
   "description": "Self-hosted Claude Code in your browser. Multi-session, multi-user, push notifications.",
   "bin": {
     "clay-server": "./bin/cli.js",