clay-server 2.34.1-beta.2 → 2.34.1-beta.3

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

@@ -103,30 +103,12 @@ function onConnected() {
 
   // Session restore is now server-driven (user-presence.json).
   // Mate DM restore is also server-driven via "restore_mate_dm" message.
-  // Fallback: if server doesn't restore DM within 2s, try localStorage
-  var savedDm = null;
-  try { savedDm = localStorage.getItem("clay-active-dm"); } catch (e) {}
-  if (savedDm && !store.get('dmMode') && !store.get('mateProjectSlug')) {
-    var dmFallbackTimer = setTimeout(function () {
-      if (!store.get('dmMode') && savedDm) {
-        console.log("[dm-restore] Server did not restore DM, using localStorage fallback:", savedDm);
-        openDm(savedDm);
-      }
-    }, 2000);
-    // Cancel fallback if server restores DM first
-    var patchedOnce = false;
-    var checkRestore = function (evt) {
-      try {
-        var d = JSON.parse(evt.data);
-        if (d.type === "restore_mate_dm" && !patchedOnce) {
-          patchedOnce = true;
-          clearTimeout(dmFallbackTimer);
-        }
-      } catch (e) {}
-    };
-    ws.addEventListener("message", checkRestore);
-    setTimeout(function () { ws.removeEventListener("message", checkRestore); }, 3000);
-  }
+  // Previously there was a 2s localStorage fallback that auto-called
+  // openDm(savedDm) on every reconnect. That fallback re-opened stale
+  // mate DMs on every refresh / project switch and was the root cause
+  // of the skill-install modal popping unprompted. Server-driven restore
+  // is authoritative — drop the client-side fallback entirely.
+  try { localStorage.removeItem("clay-active-dm"); } catch (e) {}
   // Safety: clear returningFromMateDm after initial messages settle
   if (store.get('returningFromMateDm')) {
     setTimeout(function () {

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

@@ -21,7 +21,6 @@ import { closeTerminal } from './terminal.js';
 import { openMobileSheet, setMobileSheetMateData } from './sidebar-mobile.js';
 import { getProfileLang } from './profile.js';
 import { isSchedulerOpen, closeScheduler } from './scheduler.js';
-import { requireClayMateInterview } from './app-skills-install.js';
 import { syncResizeHandles } from './sidebar.js';
 
 var MATE_ONBOARDING_KEY = "clay-mate-onboarding-shown";
@@ -77,13 +76,15 @@ export function openDm(targetUserId) {
   if (!ws || ws.readyState !== 1) return;
   // Persist DM state for refresh recovery
   try { localStorage.setItem("clay-active-dm", targetUserId); } catch (e) {}
-  // Check mate skill updates before opening mate DM
+  // Opening an existing mate DM does not require the clay-mate-interview
+  // skill — that skill is only used during new mate creation / reshaping.
+  // Showing onboarding + gating a skill version check here caused the
+  // "Skill Installation Required" modal to pop on every refresh / project
+  // switch via the localStorage DM-restore fallback in app-connection.js.
   if (typeof targetUserId === "string" && targetUserId.indexOf("mate_") === 0) {
     showMateOnboarding(function () {
-      requireClayMateInterview(function () {
-        var ws2 = getWs();
-        if (ws2) ws2.send(JSON.stringify({ type: "dm_open", targetUserId: targetUserId }));
-      });
+      var ws2 = getWs();
+      if (ws2) ws2.send(JSON.stringify({ type: "dm_open", targetUserId: targetUserId }));
     });
     return;
   }

package/lib/public/modules/app-skills-install.js

@@ -197,10 +197,14 @@ export function requireSkills(opts, cb) {
     .then(function (res) { return res.json(); })
     .then(function (data) {
       var results = data.results || [];
+      // Only "missing" skills block the feature. "outdated" skills already
+      // function — an available update should not hard-gate with a modal
+      // every time a user opens a DM or refreshes the page. Callers can
+      // surface outdated versions elsewhere (e.g. settings / notifications).
       var actionable = [];
       for (var i = 0; i < results.length; i++) {
         var r = results[i];
-        if (r.status === "missing" || r.status === "outdated") {
+        if (r.status === "missing") {
           var orig = null;
           for (var j = 0; j < opts.skills.length; j++) {
             if (opts.skills[j].name === r.name) { orig = opts.skills[j]; break; }

package/package.json

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