From a0a8469448cb3efa49dd508e417f400e3bd65b06 Mon Sep 17 00:00:00 2001
From: malkuthro <166633396+malkuthro@users.noreply.github.com>
Date: Wed, 6 May 2026 16:06:02 +0200
Subject: [PATCH] =?UTF-8?q?feat(sidebar):=20build-tag=20footer=20+=20theme?=
 =?UTF-8?q?=20auto-pull=20+=20=C3=97=20removability=20+=20pack=20badges=20?=
 =?UTF-8?q?(#98)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Five user-visible changes to the Kforge Labs sidebar that complete the
dev-sync verification loop and fix theme-mode discovery:

- Build-tag footer at the bottom of the sidebar: `dev <sha> · <time>`
  on line 1 (SHA at 13px monospace, the at-a-glance identifier),
  italic <scope> on line 2. Reads `web/_dev_build.json` written by
  `scripts/sync_to_dev.py --scope "<short scope>"`. Visible-in-app
  companion to PR #95's chat-report convention — closes the gap when
  the maintainer is juggling parallel sessions and can't tell whether
  the browser tab is showing the latest sync. Absent on registry
  installs; footer renders empty there.

- Theme/category mode now mirrors repo-mode population: REPOS auto-
  pull union user-picks, deduped, with the new auto-pull-hidden set
  filtered out. The earlier picks-only design dropped the entire
  Koolook pack from theme mode on a stock install, even though those
  same nodes appeared in repo mode — silently violating the "my
  favorites in two views" mental model. `gatherNodesByTheme` rewritten
  to seed from REPOS first, then user picks, with the same
  autoCategoryRoots + hidden filtering repo mode uses.

- `×` works on every favorite, including auto-pulled REPOS nodes.
  Adds `koolook.autoPullHidden.v1` localStorage Set; `×` on an auto-
  pulled row calls `hideAutoPullType(type)` so the next render skips
  it. `addToMyPicks` calls `unhideAutoPullType` as a side effect so
  re-adding via `+` automatically restores visibility. Same uniform
  "remove / bring back" UX whether the node was a user pick or REPOS
  auto-pull.

- Pack-name badges restored on theme-mode leaves only. Small dim
  `Koolook` / `KJNodes` text on the right of each row, same opacity
  as the search-flatten breadcrumb. Memory aid for "where did this
  node come from?" Repo mode and search-flatten still skip the badge
  (origin is conveyed by tree nesting / breadcrumb prefix).

- `scripts/sync_to_dev.py --scope` flag: extends PR #95's brief-
  output design with a JSON artifact (`<target>/web/_dev_build.json`)
  carrying commit + timestamp + worktree + scope, which the sidebar
  footer fetches on render. Doesn't change the existing one-line
  chat-report contract or the `_build_line()` shape.

Tested via dev-sync against the live ComfyUI install:
- Footer renders correctly (`dev afe539e · 2026-05-06 15:38` + scope)
- Image bucket in theme mode now contains Koolook image nodes
- Pack badges visible on theme leaves
- × hides auto-pulled Koolook nodes; toolbar + restores them

CHANGELOG entries added; the prior conflicting [Unreleased] notes
about picks-only theme + pack-badge removal are flagged as superseded
in the new entries (release prep can squash).

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 .gitignore                 |   4 +
 CHANGELOG.md               |  46 +++++++++++
 CLAUDE.md                  |  11 ++-
 scripts/sync_to_dev.py     |  43 ++++++++++
 web/sidebar/constants.js   |  11 +++
 web/sidebar/picks_store.js |  49 +++++++++++
 web/sidebar/tree.js        | 164 ++++++++++++++++++++++++++++++++++---
 7 files changed, 317 insertions(+), 11 deletions(-)

diff --git a/.gitignore b/.gitignore
index aaaa7ee..8b60c92 100644
--- a/.gitignore
+++ b/.gitignore
@@ -12,6 +12,10 @@ __pycache__/
 *.pyo
 .DS_Store
 
+# Generated by `scripts/sync_to_dev.py` — written into the *target* ComfyUI
+# install, never the source tree, but ignored here as a defensive guard.
+web/_dev_build.json
+
 # Local maintainer dev workspaces — not part of the published package.
 # These contain vendored 3rd-party clones and debugging scratchpads kept
 # only on the maintainer's disk. Anything in here would otherwise leak
diff --git a/CHANGELOG.md b/CHANGELOG.md
index def453d..0f65a66 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -6,7 +6,53 @@ The format is inspired by Keep a Changelog and SemVer.
 
 ## [Unreleased]
 
+### Added
+- **Dev-sync build-tag footer at the bottom of the Kforge Labs sidebar.**
+  `scripts/sync_to_dev.py --scope "<short scope>"` now writes a tiny
+  `web/_dev_build.json` next to the sidebar JS. The sidebar fetches it
+  on render and emits a discreet two-line footer: `dev <sha> · <time>`
+  (line 1, SHA at 13px monospace) + the scope (line 2, italic
+  proportional). Visible-in-app counterpart to the chat-report
+  convention from #95 — when the maintainer is juggling multiple
+  parallel worktree sessions, a glance at the footer confirms which
+  one's code is actually live in the browser tab. Cache-busted fetch
+  so a re-sync's new mtime reaches the browser without a hard refresh.
+  Absent on registry installs (the JSON is dev-only and gitignored);
+  the footer renders empty there. The `--scope` value is the same
+  ≤10-word change-summary the agent quotes on chat-report line 2 —
+  passing it once now feeds both surfaces.
+- **`×` removability across both group modes for any visible favorite,
+  including auto-pulled REPOS{select: "all"} nodes.** Adds a
+  `koolook.autoPullHidden.v1` localStorage set: clicking `×` on a row
+  that came from auto-pull adds the type to the hidden set so the
+  next render skips it (without the set, the next gather pass would
+  re-pull the type from the registry and the click would feel like a
+  no-op). User picks still go through `removeFromMyPicks`. Re-adding
+  via the toolbar `+` button or right-click → "Add to Kforge Labs"
+  clears the hide as a side effect of `addToMyPicks`, so users don't
+  have to think about which list a type lives in. Same uniform
+  "remove this from my favorites / bring it back" UX whether the node
+  is a Koolook auto-pull or a KJNodes user-pick.
+
 ### Changed
+- **Theme mode now mirrors repo-mode population** — auto-pulled
+  REPOS{select: "all"} nodes appear alongside user picks (deduped),
+  filtered by the new auto-pull hidden set. Supersedes the earlier
+  `[Unreleased]` "Picks-only (no REPOS-driven auto-pulled candidates)"
+  intent below: the picks-only model dropped the entire Koolook pack
+  from theme mode on a stock install, which silently violated the
+  "my favorites" mental model (the user expected their visible
+  favorites in repo mode to also show in theme mode, just regrouped).
+  With the new × removability, the user can still curate exactly what
+  they see — without the discoverability cost of dropping auto-pull
+  entirely.
+- **Pack-name badges restored on theme-mode leaves.** Small dim
+  `Koolook` / `KJNodes` etc. on the right of each row, same opacity
+  as the search-flatten breadcrumb. Memory aid for "where did this
+  node come from?" — useful when the user is learning their favorites
+  pack-by-pack. Reverts (for theme mode only) the earlier
+  `[Unreleased]` removal further down; search-flatten still skips the
+  badge since its breadcrumb prefix already conveys origin.
 - **Sidebar second mode — "Theme" instead of "Category".** The sitemap-icon
   toggle in the Nodes action row now groups picks by **semantic theme**
   rather than by raw CATEGORY first-segment. The new algorithm strips the
diff --git a/CLAUDE.md b/CLAUDE.md
index e896285..92ab958 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -40,9 +40,18 @@
 Fast iteration without cutting a release. Copies runtime files to
 `KOLOOK_COMFYUI_DEV_PATH` (set in `.env`; see `.env.example`).
 
-- `python scripts/sync_to_dev.py` — brief two-line summary; use `-v` for per-file output
+- `python scripts/sync_to_dev.py --scope "<short scope>"` — brief two-line summary; use `-v` for per-file output
 - `--dry-run` — preview without touching the target
 
+**Always pass `--scope`** with the same ≤10-word scope summary you'll
+put on chat-report line 2. The flag persists the scope to
+`<target>/web/_dev_build.json`, which the Kforge Labs sidebar reads on
+load and renders as a discreet two-line footer (`dev <sha> · <time>` +
+italic `<scope>` underneath). Same identifier the chat report quotes,
+but visible in the running ComfyUI itself — closes the gap when the
+maintainer can't tell at a glance whether the browser tab is showing
+the latest sync.
+
 **Trigger phrase: `dev-sync`** (or "copy those files", "sync dev", "push to dev").
 The agent runs the script and reports completion. The script errors
 cleanly when the env var is unset; never guess a path.
diff --git a/scripts/sync_to_dev.py b/scripts/sync_to_dev.py
index c89d75e..b538504 100644
--- a/scripts/sync_to_dev.py
+++ b/scripts/sync_to_dev.py
@@ -43,10 +43,12 @@ overwrite files in the target — that's the point.
 from __future__ import annotations
 
 import argparse
+import json
 import os
 import shutil
 import subprocess
 import sys
+from datetime import datetime
 from pathlib import Path
 
 REPO_ROOT = Path(__file__).resolve().parent.parent
@@ -98,6 +100,30 @@ def _build_line() -> str:
     sha = _get_short_sha() or "unknown"
     return f"{sha} - {_get_worktree_name()}"
 
+
+def write_build_info(target: Path, scope: str | None) -> None:
+    """Drop a tiny JSON next to the sidebar JS so the in-browser footer
+    can render `dev <sha> · <time>` (and an italic <scope> on a second
+    line) — same identifier the chat report quotes, but visible in the
+    running ComfyUI itself. Absent on registry installs (the file is
+    only written by this dev script), so the footer stays empty there.
+
+    Best-effort: missing git → omit `commit` field; no `--scope` → omit
+    `scope`; the timestamp alone is still useful for the maintainer to
+    eyeball "did my last sync land in this browser tab?\""""
+    info: dict[str, str] = {
+        "synced_at": datetime.now().strftime("%Y-%m-%d %H:%M"),
+        "worktree": _get_worktree_name(),
+    }
+    sha = _get_short_sha()
+    if sha:
+        info["commit"] = sha
+    if scope:
+        info["scope"] = scope
+    out = target / "web" / "_dev_build.json"
+    out.parent.mkdir(parents=True, exist_ok=True)
+    out.write_text(json.dumps(info, indent=2) + "\n", encoding="utf-8")
+
 # Files / dirs ComfyUI actually loads at runtime. Anything outside this
 # list (CI, docs, .claude/, .github/, .cursor/, CHANGELOG, LICENSE,
 # pyproject.toml, README, fork manifest YAML, etc.) does not affect what
@@ -245,6 +271,21 @@ def main() -> int:
             "the one-line build summary."
         ),
     )
+    parser.add_argument(
+        "--scope",
+        type=str,
+        default=None,
+        help=(
+            "Short (≤10 word) description of what this build is about — "
+            "the same scope summary that goes in the chat report's second "
+            "line. Persisted in `web/_dev_build.json` and rendered in the "
+            "Kforge Labs sidebar footer (italic, second line, below the "
+            "`dev <sha> · <time>` identifier) so the maintainer can "
+            "correlate live ComfyUI state with chat history when juggling "
+            "multiple parallel worktree sessions. Optional — when absent, "
+            "the footer renders just identifier + timestamp."
+        ),
+    )
     args = parser.parse_args()
 
     load_dotenv(REPO_ROOT / ".env")
@@ -271,6 +312,8 @@ def main() -> int:
     # mechanical sync details.
     print(_build_line())
     print(f"{verb} {n} entries -> {target}")
+    if not args.dry_run:
+        write_build_info(target, args.scope)
     return 0
 
 
diff --git a/web/sidebar/constants.js b/web/sidebar/constants.js
index 7457d6d..ba32a99 100644
--- a/web/sidebar/constants.js
+++ b/web/sidebar/constants.js
@@ -36,6 +36,13 @@ export const WORKFLOWS_GROUP_LABEL = "Workflows";
 export const STORAGE_KEY = "koolook.curated.userPicks.v1";
 export const PICKS_CHANGED_EVENT = "koolook-picks-changed";
 
+// Set of node types the user has explicitly removed via the `×` button on
+// rows that would otherwise be auto-pulled by `REPOS{select: "all"}`. Lets
+// `×` work uniformly for any visible favorite — without this, auto-pulled
+// nodes (the whole Koolook pack on a stock install) would silently re-
+// appear on every render. Cleared per-type when the user re-adds via `+`.
+export const AUTOPULL_HIDDEN_KEY = "koolook.autoPullHidden.v1";
+
 // Fired when the snapshot status changes for reasons other than picks /
 // workflows mutations (which already fire their own events). Specifically:
 //   • markStateSaved() — after a successful named save or applySnapshot
@@ -228,6 +235,7 @@ export function ensureStyle() {
 .koolook-mode-toggle-btn.koolook-mode-active { background: rgba(80,140,235,0.25); opacity: 1; }
 .koolook-leaf-unresolved { opacity: 0.55; font-style: italic; }
 .koolook-leaf-crumb { opacity: 0.5; font-size: 11px; margin-right: 1px; }
+.koolook-pack-badge { opacity: 0.5; font-size: 11px; margin-left: 6px; flex-shrink: 0; white-space: nowrap; }
 .koolook-preview-card { position: fixed; z-index: 10000; background: var(--comfy-menu-bg, #232323); border: 1px solid var(--border-color, rgba(255,255,255,0.22)); border-radius: 12px; box-shadow: 0 6px 24px rgba(0,0,0,0.55); color: var(--input-text, #ddd); font-size: 12px; min-width: 300px; max-width: 90vw; max-height: calc(100vh - 16px); pointer-events: none; overflow: hidden auto; padding-bottom: 10px; }
 .koolook-preview-header { display: flex; align-items: center; gap: 9px; padding: 8px 13px 7px; font-size: 14px; line-height: 1; white-space: nowrap; overflow: hidden; }
 .koolook-preview-headtitle { overflow: hidden; text-overflow: ellipsis; white-space: nowrap; flex: 1; min-width: 0; }
@@ -248,6 +256,9 @@ export function ensureStyle() {
 .koolook-preview-widget-name { text-align: left; text-overflow: ellipsis; white-space: nowrap; }
 .koolook-preview-widget-value { text-align: right; font-size: 11px; opacity: 0.85; font-family: monospace; text-overflow: ellipsis; white-space: nowrap; }
 .koolook-preview-desc { margin: 10px 9px 0; padding: 7px 9px; background: rgba(255,255,255,0.06); border-radius: 6px; font-style: italic; font-weight: 500; font-size: 11px; line-height: 1.45; word-break: break-word; opacity: 0.85; max-height: 120px; overflow: hidden; }
+.koolook-build-tag { flex-shrink: 0; padding: 6px 10px 14px; font-size: 10px; opacity: 0.5; text-align: center; letter-spacing: 0.04em; font-family: monospace; color: var(--input-text, inherit); line-height: 1.45; }
+.koolook-build-sha { font-size: 13px; letter-spacing: 0.06em; }
+.koolook-build-scope { display: block; font-size: 12px; margin-top: 3px; letter-spacing: 0.02em; font-family: var(--font-family, sans-serif); font-style: italic; }
 `;
     document.head.appendChild(s);
 }
diff --git a/web/sidebar/picks_store.js b/web/sidebar/picks_store.js
index f04a29c..0419f83 100644
--- a/web/sidebar/picks_store.js
+++ b/web/sidebar/picks_store.js
@@ -7,6 +7,7 @@
 import {
     STORAGE_KEY,
     PICKS_CHANGED_EVENT,
+    AUTOPULL_HIDDEN_KEY,
 } from "./constants.js";
 
 export function loadUserPicks() {
@@ -38,6 +39,13 @@ function saveUserPicks(picks) {
 // succeeded when localStorage rejected it (quota / private mode).
 export function addToMyPicks(typeName) {
     if (!typeName) return "failed";
+    // The `+` action always means "make this visible in favorites." If the
+    // user previously hid this type via `×` (when it was auto-pulled by a
+    // REPOS{select: "all"} entry), clear that hide as a side effect — so a
+    // subsequent gather pass surfaces it again either via picks or via auto-
+    // pull. Without this, `×` then `+` on the same Koolook node would do
+    // nothing visible — the hide would silently outlast the re-add.
+    unhideAutoPullType(typeName);
     const picks = loadUserPicks();
     if (picks.includes(typeName)) return "duplicate";
     picks.push(typeName);
@@ -48,6 +56,47 @@ export function removeFromMyPicks(typeName) {
     saveUserPicks(loadUserPicks().filter(p => p !== typeName));
 }
 
+// =============================================================================
+// Auto-pull hidden set — types the user has explicitly excluded from the
+// REPOS{select: "all"} auto-pull. See AUTOPULL_HIDDEN_KEY in constants.js.
+// =============================================================================
+export function loadAutoPullHidden() {
+    try {
+        const raw = localStorage.getItem(AUTOPULL_HIDDEN_KEY);
+        if (!raw) return new Set();
+        const parsed = JSON.parse(raw);
+        return new Set(Array.isArray(parsed) ? parsed.filter(x => typeof x === "string") : []);
+    } catch (e) {
+        console.warn("[Koolook] failed to parse autoPullHidden; starting empty:", e);
+        return new Set();
+    }
+}
+
+function saveAutoPullHidden(set) {
+    try {
+        localStorage.setItem(AUTOPULL_HIDDEN_KEY, JSON.stringify([...set]));
+        return true;
+    } catch (e) {
+        console.warn("[Koolook] failed to save autoPullHidden:", e);
+        return false;
+    }
+}
+
+export function hideAutoPullType(typeName) {
+    if (!typeName) return;
+    const set = loadAutoPullHidden();
+    if (set.has(typeName)) return;
+    set.add(typeName);
+    saveAutoPullHidden(set);
+}
+
+export function unhideAutoPullType(typeName) {
+    if (!typeName) return;
+    const set = loadAutoPullHidden();
+    if (!set.delete(typeName)) return;
+    saveAutoPullHidden(set);
+}
+
 // Bulk-replace the entire pick list. Used by the snapshot import flow —
 // the caller has just deserialized a preset and wants picks to mirror it
 // exactly. Filters non-strings defensively (snapshot files are user-
diff --git a/web/sidebar/tree.js b/web/sidebar/tree.js
index 89fcea1..f8d8464 100644
--- a/web/sidebar/tree.js
+++ b/web/sidebar/tree.js
@@ -32,6 +32,8 @@ import {
     removeFromMyPicks,
     notifyPicksChanged,
     addToMyPicks,
+    loadAutoPullHidden,
+    hideAutoPullType,
 } from "./picks_store.js";
 import {
     persistMutation,
@@ -519,6 +521,28 @@ function subcategoryFor(category, categoryRoot) {
 // LiteGraph registry (caller should skip silently — the node won't render
 // in the sidebar either way). Used by `spotlightAddedPicks` to know which
 // folder to expand after a save.
+// True if this type would be auto-pulled by a REPOS{select: "all"} entry
+// — i.e. its category prefix matches a repo's `categoryRoot`, OR it's
+// listed in a `select: [...]` array. Used by the `×` handler to decide
+// whether removing-from-picks is sufficient (no — registry-driven re-pull
+// would resurface it on the next render) or whether the type also needs to
+// be added to the auto-pull hidden list.
+function isAutoPulled(typeName) {
+    const registry = (typeof LiteGraph !== "undefined" && LiteGraph.registered_node_types) || {};
+    const nc = registry[typeName];
+    if (!nc) return false;
+    const cat = nc.category || "";
+    for (const repo of REPOS) {
+        if (repo.select === "all") {
+            const root = repo.categoryRoot || "";
+            if (root && (cat === root || cat.startsWith(root + "/"))) return true;
+        } else if (Array.isArray(repo.select) && repo.select.includes(typeName)) {
+            return true;
+        }
+    }
+    return false;
+}
+
 function findPackPathForType(typeName) {
     const registry = (typeof LiteGraph !== "undefined" && LiteGraph.registered_node_types) || {};
     const nc = registry[typeName];
@@ -550,6 +574,7 @@ function matchesQuery(display, type, q) {
 function gatherNodesByRepo(query) {
     const q = (query || "").trim().toLowerCase();
     const registry = (typeof LiteGraph !== "undefined" && LiteGraph.registered_node_types) || {};
+    const hidden = loadAutoPullHidden();
     const out = [];
 
     for (const repo of REPOS) {
@@ -557,13 +582,14 @@ function gatherNodesByRepo(query) {
         if (repo.select === "all") {
             const root = repo.categoryRoot || "";
             candidateIds = Object.entries(registry)
-                .filter(([, nc]) => {
+                .filter(([type, nc]) => {
+                    if (hidden.has(type)) return false;
                     const cat = (nc && nc.category) || "";
                     return root && (cat === root || cat.startsWith(root + "/"));
                 })
                 .map(([type]) => type);
         } else if (Array.isArray(repo.select)) {
-            candidateIds = repo.select.filter(t => registry[t] !== undefined);
+            candidateIds = repo.select.filter(t => registry[t] !== undefined && !hidden.has(t));
         } else {
             candidateIds = [];
         }
@@ -869,6 +895,57 @@ function gatherNodesByTheme(query) {
     const registry = (typeof LiteGraph !== "undefined" && LiteGraph.registered_node_types) || {};
     const userPicks = loadUserPicks();
 
+    // Population mirror of repo mode: auto-pulled REPOS nodes plus user
+    // picks for packs that aren't auto-pulled. Without this seeding, theme
+    // mode silently drops every node the maintainer has registered as
+    // always-on via REPOS{select: "all"} (on a stock install: the entire
+    // Koolook pack), even though those same nodes do appear in repo mode.
+    // Hidden auto-pulls (×'d by the user) are filtered out so the user's
+    // visible-favorites set is consistent across both modes.
+    const autoCategoryRoots = new Set(
+        REPOS.filter(r => r.select === "all" && r.categoryRoot).map(r => r.categoryRoot)
+    );
+    const hidden = loadAutoPullHidden();
+    const seen = new Set();
+    const seeds = [];
+
+    for (const repo of REPOS) {
+        if (repo.select === "all") {
+            const root = repo.categoryRoot || "";
+            if (!root) continue;
+            for (const [type, nc] of Object.entries(registry)) {
+                if (seen.has(type) || hidden.has(type)) continue;
+                const cat = (nc && nc.category) || "";
+                if (cat !== root && !cat.startsWith(root + "/")) continue;
+                const display = (nc && nc.title) || type;
+                if (repo.excludePatterns && repo.excludePatterns.some(re => re.test(display))) continue;
+                seen.add(type);
+                seeds.push({ type });
+            }
+        } else if (Array.isArray(repo.select)) {
+            for (const type of repo.select) {
+                if (seen.has(type) || hidden.has(type) || registry[type] === undefined) continue;
+                seen.add(type);
+                seeds.push({ type });
+            }
+        }
+    }
+
+    for (const type of userPicks) {
+        if (seen.has(type)) continue;
+        // Skip picks already covered by an auto-pulled REPO (would double-
+        // count with the auto-pull pass above). A type that's been hidden
+        // from auto-pull AND explicitly user-picked still gets surfaced
+        // here, since the auto-pull pass skipped it.
+        const nc = registry[type];
+        if (nc && !hidden.has(type)) {
+            const packLabel = (nc.category || "").split("/")[0] || "";
+            if (autoCategoryRoots.has(packLabel)) continue;
+        }
+        seen.add(type);
+        seeds.push({ type });
+    }
+
     // Map<canonicalThemeKey, { displayLabels: Map<rawLabel, count>, nodes: [...] }>
     const themes = new Map();
     const ensureBucket = (key, seedLabel) => {
@@ -881,7 +958,7 @@ function gatherNodesByTheme(query) {
         return themes.get(key);
     };
 
-    for (const type of userPicks) {
+    for (const { type } of seeds) {
         const result = extractTheme(type, registry);
 
         if (result.kind === "unresolved") {
@@ -890,7 +967,7 @@ function gatherNodesByTheme(query) {
             // Display falls back to the bare type since `nc.title` is unreachable.
             if (!matchesQuery(type, type, q)) continue;
             const bucket = ensureBucket("(unresolved)", "(unresolved)");
-            bucket.nodes.push({ type, display: type, isUserPick: true, unresolved: true });
+            bucket.nodes.push({ type, display: type, unresolved: true });
             continue;
         }
 
@@ -903,7 +980,7 @@ function gatherNodesByTheme(query) {
 
         if (result.kind === "uncategorized") {
             const bucket = ensureBucket("(uncategorized)", "(uncategorized)");
-            bucket.nodes.push({ type, display, isUserPick: true });
+            bucket.nodes.push({ type, display });
             continue;
         }
 
@@ -912,7 +989,7 @@ function gatherNodesByTheme(query) {
             result.themeRaw,
             (bucket.displayLabels.get(result.themeRaw) || 0) + 1,
         );
-        bucket.nodes.push({ type, display, isUserPick: true });
+        bucket.nodes.push({ type, display });
     }
 
     return themes;
@@ -1139,7 +1216,7 @@ function makeFolderRow({ name, count, iconKind, onToggle, onContextMenu, draggab
     return { row, chevron };
 }
 
-function makeNodeLeafRow({ display, type, removable, onClick, unresolved, breadcrumb }) {
+function makeNodeLeafRow({ display, type, removable, onClick, unresolved, breadcrumb, packBadge }) {
     const row = document.createElement("div");
     row.className = "koolook-row koolook-leaf";
     if (unresolved) row.classList.add("koolook-leaf-unresolved");
@@ -1175,6 +1252,16 @@ function makeNodeLeafRow({ display, type, removable, onClick, unresolved, breadc
     }
     row.appendChild(nameEl);
 
+    // Right-side pack badge — used in theme mode as a quiet "where did
+    // this come from" memory aid. Repo mode and search-flatten skip it
+    // (origin is already conveyed by tree nesting / breadcrumb prefix).
+    if (packBadge) {
+        const badge = document.createElement("span");
+        badge.className = "koolook-pack-badge";
+        badge.textContent = packBadge;
+        row.appendChild(badge);
+    }
+
     if (removable) {
         const rm = document.createElement("span");
         rm.className = "koolook-remove";
@@ -1182,7 +1269,13 @@ function makeNodeLeafRow({ display, type, removable, onClick, unresolved, breadc
         rm.title = "Remove from favorites";
         rm.addEventListener("click", (e) => {
             e.stopPropagation();
+            // Two paths a row can land in favorites: explicit user pick
+            // (drop it from the picks list) or REPOS{select: "all"} auto-
+            // pull (add it to the auto-pull hidden set so the next render
+            // doesn't re-pull it). A type can in principle be both at
+            // once — running both ops is idempotent and covers the union.
             removeFromMyPicks(type);
+            if (isAutoPulled(type)) hideAutoPullType(type);
             notifyPicksChanged();
         });
         row.appendChild(rm);
@@ -1686,7 +1779,7 @@ addSection({
                                         row: makeNodeLeafRow({
                                             display: n.display,
                                             type: n.type,
-                                            removable: !!pack.isUserPicks,
+                                            removable: true,
                                             onClick: () => insertNode(n.type),
                                         }),
                                     });
@@ -1722,7 +1815,7 @@ function emitNodesFlatSearchResults(ctx) {
                 type: item.type,
                 breadcrumb: item.breadcrumb,
                 unresolved: !!item.unresolved,
-                removable: !!item.removable,
+                removable: true,
                 onClick: () => insertNode(item.type),
             }),
         });
@@ -1796,12 +1889,23 @@ function emitThemeChildren(parentCtx, themes) {
                     (a, b) => compareNames(a.display, b.display),
                 );
                 for (const n of sortedNodes) {
+                    // Pack badge — only for real pack labels. Skip
+                    // unresolved (origin IS the unknown) and synthetic
+                    // (uncategorized)/(unresolved) parens-prefixed names.
+                    let badge = null;
+                    if (!n.unresolved) {
+                        const loc = findPackPathForType(n.type);
+                        if (loc && loc.packLabel && !loc.packLabel.startsWith("(")) {
+                            badge = loc.packLabel;
+                        }
+                    }
                     sub.leaf({
                         row: makeNodeLeafRow({
                             display: n.display,
                             type: n.type,
-                            removable: !!n.isUserPick,
+                            removable: true,
                             unresolved: !!n.unresolved,
+                            packBadge: badge,
                             onClick: () => insertNode(n.type),
                         }),
                     });
@@ -2485,6 +2589,46 @@ export function renderPanel(container) {
 
     renderTree({ treeEl: tree, query: "" });
 
+    // ---- Build tag (dev-sync verification) ----
+    // `scripts/sync_to_dev.py` writes `web/_dev_build.json` post-sync so the
+    // maintainer can eyeball whether the live ComfyUI is running the latest
+    // synced code. Absent on registry installs and plain git checkouts —
+    // silently renders nothing. Cache-bust query param so a re-sync's new
+    // mtime reaches the browser without a hard refresh.
+    const buildTag = document.createElement("div");
+    buildTag.className = "koolook-build-tag";
+    container.appendChild(buildTag);
+    fetch(new URL("../_dev_build.json", import.meta.url).href + `?t=${Date.now()}`)
+        .then(r => (r.ok ? r.json() : null))
+        .then(info => {
+            if (!info) return;
+            // Layout, top-to-bottom:
+            //   dev <SHA>  · <time>     ← line 1: identifier + timestamp
+            //   <scope>                 ← line 2: human-readable purpose
+            // SHA gets its own span (13px monospace) — the at-a-glance
+            // identifier when comparing chat history with the live footer.
+            // Scope renders as a block in proportional italic so it reads
+            // as prose, not as part of the build identifier.
+            buildTag.textContent = "";
+            buildTag.appendChild(document.createTextNode("dev "));
+            if (info.commit) {
+                const shaEl = document.createElement("span");
+                shaEl.className = "koolook-build-sha";
+                shaEl.textContent = info.commit;
+                buildTag.appendChild(shaEl);
+            }
+            if (info.synced_at) {
+                buildTag.appendChild(document.createTextNode(` · ${info.synced_at}`));
+            }
+            if (info.scope) {
+                const scopeEl = document.createElement("span");
+                scopeEl.className = "koolook-build-scope";
+                scopeEl.textContent = info.scope;
+                buildTag.appendChild(scopeEl);
+            }
+        })
+        .catch(() => { /* not in a dev install — leave empty */ });
+
     // ---- Search wiring ----
     let debounce = null;
     search.addEventListener("input", (e) => {