impel-cli 0.20.33 → 0.20.35

package/RELEASE_NOTES.md

@@ -1,5 +1,24 @@
 # Release notes
 
+## 0.20.35 — Normalize the Claude Tasks destination
+
+- Waits for Claude's native New row before cloning the Tasks destination,
+  preventing the earlier Home/Code switcher from leaking its Code glyph into
+  the injected control.
+- Removes cloned vendor glyphs and their empty wrappers so Tasks renders with
+  exactly one board icon in both Claude modes.
+- Rebuilds existing managed bundles onto the corrected navigation lifecycle.
+
+## 0.20.34 — Stabilize desktop Tasks navigation
+
+- Places Tasks immediately after Claude's persistent Home/Code switcher in both
+  modes and uses the New row as its visual template, so the destination no
+  longer jumps above the switcher when the selected mode changes.
+- Prefers matching controls inside the host sidebar, preventing similarly named
+  content controls from moving the injected destination into the main surface.
+- Preserves ChatGPT/Codex placement immediately before Pull requests and
+  rebuilds existing managed bundles onto the corrected navigation contract.
+
 ## 0.20.33 — Faster Codex native-agent relay
 
 - Uses the ultra-fast Codex Spark model at low reasoning for fixed-binding

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "impel-cli",
-  "version": "0.20.33",
+  "version": "0.20.35",
   "description": "Prepare isolated Claude and Codex workspaces for every accessible Impel tenant",
   "type": "module",
   "bin": {

package/src/apps.js

@@ -299,7 +299,7 @@ export const CURRENT_CONFIG_VERSION = 32;
 // — which is what made every `impel update` re-trigger macOS permission
 // prompts. Bump this ONLY when a code change alters the bytes of a built
 // bundle; leave it alone for changes that don't touch bundle contents.
-export const BUNDLE_BUILD_FINGERPRINT = "bundle-2026-08-08.4";
+export const BUNDLE_BUILD_FINGERPRINT = "bundle-2026-08-09.4";
 
 /** Parse the tenant's install manifest, or null when absent/corrupt. */
 export function readTenantManifest(homeDir = os.homedir(), tenantId = null) {

package/src/desktopTasks.js

@@ -2,6 +2,19 @@ import path from "node:path";
 
 export const DESKTOP_TASKS_RESOURCE_URI = "ui://impel/tasks/v1/index.html";
 
+// Claude's mode switch is the stable landmark across Home and Code, but its
+// plain New row is the visual template. ChatGPT/Codex keeps the established
+// position immediately before Pull requests.
+export const DESKTOP_TASKS_NAV_ANCHOR_RULES = Object.freeze([
+  { pattern: "^pull requests?\\b", position: "before" },
+  { pattern: "^code\\b", position: "after", template: "\\bnew\\b" },
+  { pattern: "^(?:start\\s+)?new\\s+(?:chat|thread)\\b", position: "before" },
+  { pattern: "^new\\b", position: "before" },
+  { pattern: "^scheduled\\b", position: "before" },
+  { pattern: "^plugins?\\b", position: "before" },
+  { pattern: "^cowork\\b", position: "before" },
+]);
+
 export function desktopTasksAssetPaths(root) {
   const directory = path.join(root, "desktop-tasks");
   return {
@@ -12,10 +25,17 @@ export function desktopTasksAssetPaths(root) {
   };
 }
 
+const NAV_ANCHOR_RULES_SOURCE = JSON.stringify(DESKTOP_TASKS_NAV_ANCHOR_RULES);
+
 const NAVIGATION_SOURCE = String.raw`(() => {
   const NAV_ID = "impel-desktop-tasks-nav";
   const NAV_SLOT_ID = "impel-desktop-tasks-nav-slot";
   const ACTIVE_CLASS = "impel-desktop-tasks-active";
+  const NAV_ANCHOR_RULES = ${NAV_ANCHOR_RULES_SOURCE}.map((rule) => ({
+    ...rule,
+    pattern: new RegExp(rule.pattern, "i"),
+    template: rule.template ? new RegExp(rule.template, "i") : null,
+  }));
   const TASKS_ICON = '<svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" aria-hidden="true"><rect x="3" y="3" width="18" height="18" rx="2"></rect><path d="M9 3v18"></path><path d="M15 3v18"></path></svg>';
   if (window.__impelDesktopTasksCleanup) window.__impelDesktopTasksCleanup();
 
@@ -56,6 +76,12 @@ const NAVIGATION_SOURCE = String.raw`(() => {
 
   const findControl = (pattern) => {
     const matches = controls().filter((element) => pattern.test(label(element)));
+    const sidebar = findSidebar();
+    if (sidebar) {
+      const sidebarMatch = matches.find((element) => sidebar.contains(element) && visible(element))
+        || matches.find((element) => sidebar.contains(element));
+      if (sidebarMatch) return sidebarMatch;
+    }
     return matches.find(visible) || matches[0] || null;
   };
 
@@ -134,9 +160,9 @@ const NAVIGATION_SOURCE = String.raw`(() => {
     let changed = false;
     while (walker.nextNode()) {
       const text = walker.currentNode.nodeValue || "";
-      if (!/(?:new\s+(?:chat|thread)|pull requests?|scheduled|plugins?|skills?|cowork|code)\b/i.test(text)) continue;
+      if (!/(?:new(?:\s+(?:chat|thread))?|pull requests?|scheduled|plugins?|skills?|cowork|code)\b/i.test(text)) continue;
       walker.currentNode.nodeValue = text.replace(
-        /(?:new\s+(?:chat|thread)|pull requests?|scheduled|plugins?|skills?|cowork|code)\b/i,
+        /(?:new(?:\s+(?:chat|thread))?|pull requests?|scheduled|plugins?|skills?|cowork|code)\b/i,
         "Tasks",
       );
       changed = true;
@@ -158,26 +184,44 @@ const NAVIGATION_SOURCE = String.raw`(() => {
     element.querySelectorAll("kbd, [data-slot=\"shortcut\"]").forEach((candidate) => candidate.remove());
   };
 
+  const stripTemplateGlyphs = (element) => {
+    element.querySelectorAll("svg, [aria-hidden=\"true\"]").forEach((candidate) => candidate.remove());
+    const walker = document.createTreeWalker(element, NodeFilter.SHOW_TEXT);
+    const glyphs = [];
+    while (walker.nextNode()) {
+      if (/^(?:\+|＋|<\/>|<>)$/u.test((walker.currentNode.nodeValue || "").trim())) {
+        glyphs.push(walker.currentNode);
+      }
+    }
+    glyphs.forEach((candidate) => {
+      const parent = candidate.parentElement;
+      candidate.remove();
+      if (
+        parent
+        && parent !== element
+        && !parent.textContent?.trim()
+        && !parent.querySelector("svg, img")
+      ) parent.remove();
+    });
+    [...element.querySelectorAll("*")].reverse().forEach((candidate) => {
+      if (
+        !candidate.textContent?.trim()
+        && !candidate.querySelector("svg, img, input")
+      ) candidate.remove();
+    });
+  };
+
   const makeNav = (template) => {
     const button = template ? template.cloneNode(true) : document.createElement("button");
     stripVendorActions(button);
+    stripTemplateGlyphs(button);
     button.id = NAV_ID;
     button.type = "button";
     button.setAttribute("aria-label", "Tasks");
     button.setAttribute("title", "Tasks");
     button.classList.add("impel-desktop-tasks-nav");
     if (!replaceText(button)) button.textContent = "Tasks";
-    const icon = button.querySelector("svg");
-    if (icon) {
-      icon.setAttribute("viewBox", "0 0 24 24");
-      icon.setAttribute("fill", "none");
-      icon.setAttribute("stroke", "currentColor");
-      icon.setAttribute("stroke-width", "2");
-      icon.setAttribute("stroke-linecap", "round");
-      icon.setAttribute("stroke-linejoin", "round");
-      icon.innerHTML = '<rect x="3" y="3" width="18" height="18" rx="2"></rect><path d="M9 3v18"></path><path d="M15 3v18"></path>';
-    }
-    else button.insertAdjacentHTML("afterbegin", TASKS_ICON);
+    button.insertAdjacentHTML("afterbegin", TASKS_ICON);
     button.addEventListener("click", (event) => {
       event.preventDefault();
       event.stopPropagation();
@@ -195,32 +239,42 @@ const NAVIGATION_SOURCE = String.raw`(() => {
 
   const placement = (control) => {
     if (!control) return null;
+    const controlRect = control.getBoundingClientRect();
     let child = control;
     let parent = control.parentElement;
     while (parent && parent !== document.body) {
-      const style = getComputedStyle(parent);
-      if (style.display === "flex" && style.flexDirection === "column") {
-        return { stack: parent, child };
-      }
+      const rect = parent.getBoundingClientRect();
+      const sameVisualRow = Math.abs(rect.top - controlRect.top) <= 8
+        && Math.abs(rect.bottom - controlRect.bottom) <= 8;
+      if (!sameVisualRow) break;
       child = parent;
       parent = parent.parentElement;
     }
-    return null;
+    return parent && parent !== document.body ? { stack: parent, child } : null;
   };
 
   const ensureNav = () => {
-    const anchor = findControl(/^pull requests?\b/i)
-      || findControl(/^(?:start\s+)?new\s+(?:chat|thread)\b/i)
-      || findControl(/^scheduled\b/i)
-      || findControl(/^plugins?\b/i)
-      || findControl(/^cowork\b/i)
-      || findControl(/^code\b/i);
+    let anchorRule = null;
+    let anchor = null;
+    for (const rule of NAV_ANCHOR_RULES) {
+      anchor = findControl(rule.pattern);
+      if (anchor) {
+        anchorRule = rule;
+        break;
+      }
+    }
+    const template = anchorRule?.template ? findControl(anchorRule.template) : anchor;
     lastAnchor = anchor;
-    if (!nav?.isConnected) nav = makeNav(anchor);
+    // Claude mounts its Home/Code switcher before the New row. Do not clone the
+    // switcher as a temporary fallback: its Code glyph would remain in the
+    // injected control after the intended visual template appears.
+    if (!anchor || !template) return;
+    if (!nav?.isConnected) nav = makeNav(template);
     const target = placement(anchor);
+    const templateTarget = placement(template);
     if (target) {
       if (!slot?.isConnected || (slot !== nav && !slot.contains(nav))) {
-        slot = target.child === anchor ? nav : target.child.cloneNode(false);
+        slot = templateTarget?.child === template ? nav : templateTarget?.child.cloneNode(false) || nav;
         if (slot !== nav) {
           slot.id = NAV_SLOT_ID;
           slot.removeAttribute("aria-label");
@@ -228,11 +282,14 @@ const NAVIGATION_SOURCE = String.raw`(() => {
           slot.append(nav);
         }
       }
-      if (slot.parentElement !== target.stack || slot.nextElementSibling !== target.child) {
-        target.stack.insertBefore(slot, target.child);
+      const after = anchorRule?.position === "after";
+      const correctlyPlaced = slot.parentElement === target.stack
+        && (after ? slot.previousElementSibling === target.child : slot.nextElementSibling === target.child);
+      if (!correctlyPlaced) {
+        target.stack.insertBefore(slot, after ? target.child.nextElementSibling : target.child);
       }
     } else if (anchor?.parentElement) {
-      anchor.insertAdjacentElement("afterend", nav);
+      anchor.insertAdjacentElement(anchorRule?.position === "after" ? "afterend" : "beforebegin", nav);
       slot = nav;
     } else if (!nav.isConnected) {
       slot = null;