shadok-ai 0.15.30 → 0.15.32

package/README.md

@@ -79,11 +79,12 @@ Enter to skip; you can add it later from the web UI).
 - **Groups column** — on a desktop the groups get a column of their own, left
   of the agents: pick a group and the agents column shows only its agents
   (*Ungrouped* holds the rest, the home agent included unless you file it). Each
-  group has an icon — drawn at random, click it to pick another or type any
-  emoji — and a **＋** that creates the new agent straight into that group.
+  group has an icon — drawn at random, double-click it to pick another or type
+  any emoji — and a **＋** that creates the new agent straight into that group.
+  A single click on a group (its icon or its row) selects it; this matters once
+  **«** folds the column to its icons, where the icon is the whole row.
   Dragging an agent onto a group still files it there, and dragging groups
-  reorders them. Double-click a name to rename it; **«** folds the column to
-  its icons. A dot on a group you are not looking at says one of its agents
+  reorders them. Double-click a name to rename it. A dot on a group you are not looking at says one of its agents
   is waiting for you, has an unread answer, or is working. On a phone nothing
   changes: the agents sheet still lists every group.
 - **Worktree isolation** — spawned agents get their own git worktree and branch

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "shadok-ai",
-  "version": "0.15.30",
+  "version": "0.15.32",
   "main": "dist/session.js",
   "scripts": {
     "test": "node --import tsx --test --test-timeout=120000 test/*.ts context/*/test/*.test.mjs",

package/public/index.html

@@ -4184,6 +4184,15 @@
   const moodOf = (t) =>
     ["needs-answer", "unread", "working"].find((m) => t.btnEl.classList.contains(m)) || null;
 
+  // A single click on a group's icon SELECTS the group (when the rail is
+  // collapsed to icons the icon is almost the whole row, so this is the primary
+  // action); a double-click opens the icon picker. The stamp has to live out
+  // here, not on the element: selecting a non-active group rebuilds the rail
+  // (renderRail keys on the active view) and replaces the very icon between the
+  // two clicks, so a native `dblclick` never fires and an element-local timer
+  // would be thrown away with it.
+  let lastIconClick = { view: null, t: 0 };
+
   function railItem({ view, icon, name, group }) {
     const members = tabsInView(view);
     const item = document.createElement("div");
@@ -4199,7 +4208,7 @@
     ic.type = "button";
     ic.className = "rail-icon";
     ic.textContent = icon;
-    if (group) ic.title = "Change the icon";
+    if (group) ic.title = "Double-click to change the icon";
     const nm = document.createElement("span");
     nm.className = "rail-name";
     nm.textContent = name;
@@ -4228,8 +4237,18 @@
         applyView();
       });
       ic.addEventListener("click", (e) => {
+        // stopPropagation so the item's own click doesn't double-fire setView,
+        // and so the document-level close-on-click doesn't shut the picker we
+        // may be about to open with this very click.
         e.stopPropagation();
-        openIconPicker(group, ic);
+        const now = Date.now();
+        if (lastIconClick.view === view && now - lastIconClick.t < 350) {
+          lastIconClick = { view: null, t: 0 };
+          openIconPicker(group, ic);
+        } else {
+          lastIconClick = { view, t: now };
+          setView(view);
+        }
       });
       nm.addEventListener("dblclick", (e) => {
         e.stopPropagation();