fix(loom): loom-0.4.1 — stop no-op rebuild storm that swallowed panel clicks (#247)

montfort · claude · web-flow · commit 08b66bf344e9 · 2026-06-13T13:29:38.000-06:00
Re-saving a watched .md without changing its content (editor save, formatter,
touch, cloud-sync) made the parse cache re-parse, rebuild an identical graph,
and still broadcast a delta. Each delta re-rendered the side panels (innerHTML
rewrite), destroying their per-button click handlers — so dangling-reference
links, community buttons and stats sections felt dead under filesystem churn,
persisting across reload with no console errors.

- Server: the watcher diffs the rebuilt graph and only broadcasts on a real
  change. Added GraphDelta::is_noop + to_event; derived PartialEq on
  core::graph::Edge and the Loom ApiEdge to compare edge sets.
- Client: the stats and legend panels use event delegation on their stable
  containers instead of per-button listeners, so clicks survive the innerHTML
  rewrites under any update frequency.

Verified: rewriting identical content 5× → 10 no-op deltas before, 0 after; a
real change still broadcasts. core 42/42, loom 9/9. See AILOG-2026-06-13-003.

Co-authored-by: Claude Opus 4.8 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,20 @@ and this project uses [independent versioning](README.md#versioning) for Framewo
 
 ---
 
+## Loom 0.4.1 — fix: panel click responsiveness under filesystem churn
+
+### Fixed (Loom)
+
+- The watcher no longer broadcasts a WebSocket update when a rebuild yields an identical graph
+  (a file's mtime moved but its content did not — an editor save, formatter, `touch`, or
+  cloud-sync rewrite). These no-op broadcasts re-rendered open clients' side panels
+  continuously and swallowed clicks on dangling-reference links, community buttons and stats
+  sections.
+- The stats and legend panels use event delegation on their stable containers instead of
+  per-button listeners, so clicks survive the panels' innerHTML rewrites under frequent updates.
+
+---
+
 ## Loom 0.4.0 — connectivity: reference normalization + entity nodes
 
 Closes the Loom connectivity follow-ups (R1 + R2). On the Sentinel corpus, 330 of 395
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/README.md b/README.md
@@ -279,7 +279,7 @@ StrayMark uses independent version tags for each component:
 | --- | --- | --- | --- |
 | Framework | `fw-` | `fw-4.26.0` | Templates (12 types), governance, directives, Charter template + schema |
 | CLI | `cli-` | `cli-3.24.0` | The `straymark` binary |
-| Loom (EXPERIMENTAL) | `loom-` | `loom-0.4.0` | The `straymark-loom` visualization server, downloaded on demand by `straymark loom serve` |
+| Loom (EXPERIMENTAL) | `loom-` | `loom-0.4.1` | The `straymark-loom` visualization server, downloaded on demand by `straymark loom serve` |
 
 Check installed versions with `straymark status` or `straymark about`.
 
diff --git a/cli/Cargo.toml b/cli/Cargo.toml
@@ -17,7 +17,7 @@ name = "straymark"
 path = "src/main.rs"
 
 [dependencies]
-straymark-core = { version = "0.4.0", path = "../core" }
+straymark-core = { version = "0.4.1", path = "../core" }
 clap = { version = "4", features = ["derive"] }
 reqwest = { version = "0.12", features = ["blocking", "rustls-tls", "json"] }
 serde = { version = "1", features = ["derive"] }
diff --git a/core/Cargo.toml b/core/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "straymark-core"
-version = "0.4.0"
+version = "0.4.1"
 edition = "2021"
 description = "Shared document model and knowledge graph for StrayMark — parses governance documents and builds their typed traceability graph"
 license = "MIT"
diff --git a/core/src/graph.rs b/core/src/graph.rs
@@ -79,7 +79,7 @@ impl Node {
 /// A directed, typed edge. `source`/`target` are node ids; an edge whose
 /// target id is not present in the corpus is kept with `resolved: false`
 /// (a dangling reference — a first-class signal, never silently dropped).
-#[derive(Debug, Clone, Serialize)]
+#[derive(Debug, Clone, PartialEq, Serialize)]
 pub struct Edge {
     pub source: String,
     pub target: String,
diff --git a/docs/adopters/CLI-REFERENCE.md b/docs/adopters/CLI-REFERENCE.md
@@ -49,7 +49,7 @@ StrayMark uses **independent version tags** for each component:
 |-----------|-----------|---------|------------------|
 | Framework | `fw-` | `fw-4.26.0` | Templates (12 types), governance docs, directives, Charter template + schema |
 | CLI | `cli-` | `cli-3.24.0` | The `straymark` binary |
-| Loom (EXPERIMENTAL) | `loom-` | `loom-0.4.0` | The `straymark-loom` visualization server, downloaded on demand by `straymark loom serve` |
+| Loom (EXPERIMENTAL) | `loom-` | `loom-0.4.1` | The `straymark-loom` visualization server, downloaded on demand by `straymark loom serve` |
 
 Framework and CLI are released independently. A framework update does not require a CLI update, and vice versa.
 
@@ -1408,8 +1408,8 @@ $ straymark loom serve
      Unstable: API, CLI surface, and on-disk layout may change or be
      removed without a deprecation cycle. Loopback-only. Read-only.
 
-ℹ Downloading Loom 0.4.0 (x86_64-unknown-linux-gnu) — first use is opt-in by download
-✔ Loom 0.4.0 cached at ~/.straymark/bin/straymark-loom
+ℹ Downloading Loom 0.4.1 (x86_64-unknown-linux-gnu) — first use is opt-in by download
+✔ Loom 0.4.1 cached at ~/.straymark/bin/straymark-loom
 loom: watching /project/.straymark (142 docs, 318 links)
 loom: serving http://127.0.0.1:7700
 ```
diff --git a/docs/i18n/es/README.md b/docs/i18n/es/README.md
@@ -242,7 +242,7 @@ StrayMark usa tags de versión independientes para cada componente:
 |------------|---------------|---------|---------|
 | Framework | `fw-` | `fw-4.26.0` | Plantillas (12 tipos), gobernanza, directivas, plantilla + schema de Charter |
 | CLI | `cli-` | `cli-3.24.0` | El binario `straymark` |
-| Loom (EXPERIMENTAL) | `loom-` | `loom-0.4.0` | El servidor de visualización `straymark-loom`, descargado bajo demanda por `straymark loom serve` |
+| Loom (EXPERIMENTAL) | `loom-` | `loom-0.4.1` | El servidor de visualización `straymark-loom`, descargado bajo demanda por `straymark loom serve` |
 
 Verifica las versiones instaladas con `straymark status` o `straymark about`.
 
diff --git a/docs/i18n/es/adopters/CLI-REFERENCE.md b/docs/i18n/es/adopters/CLI-REFERENCE.md
@@ -49,7 +49,7 @@ StrayMark usa **tags de versión independientes** para cada componente:
 |------------|---------------|---------|-------------|
 | Framework | `fw-` | `fw-4.26.0` | Plantillas (12 tipos), docs de gobernanza, directivas |
 | CLI | `cli-` | `cli-3.24.0` | El binario `straymark` |
-| Loom (EXPERIMENTAL) | `loom-` | `loom-0.4.0` | El servidor de visualización `straymark-loom`, descargado bajo demanda por `straymark loom serve` |
+| Loom (EXPERIMENTAL) | `loom-` | `loom-0.4.1` | El servidor de visualización `straymark-loom`, descargado bajo demanda por `straymark loom serve` |
 
 Framework y CLI se publican de forma independiente. Una actualización del framework no requiere actualización del CLI, y viceversa.
 
@@ -1098,8 +1098,8 @@ $ straymark loom serve
      Unstable: API, CLI surface, and on-disk layout may change or be
      removed without a deprecation cycle. Loopback-only. Read-only.
 
-ℹ Downloading Loom 0.4.0 (x86_64-unknown-linux-gnu) — first use is opt-in by download
-✔ Loom 0.4.0 cached at ~/.straymark/bin/straymark-loom
+ℹ Downloading Loom 0.4.1 (x86_64-unknown-linux-gnu) — first use is opt-in by download
+✔ Loom 0.4.1 cached at ~/.straymark/bin/straymark-loom
 loom: watching /project/.straymark (142 docs, 318 links)
 loom: serving http://127.0.0.1:7700
 ```
diff --git a/docs/i18n/zh-CN/README.md b/docs/i18n/zh-CN/README.md
@@ -260,7 +260,7 @@ StrayMark 为每个组件使用独立的版本标签：
 |------|----------|------|----------|
 | Framework | `fw-` | `fw-4.26.0` | 模板（12 种类型）、治理文档、指令、Charter 模板 + schema |
 | CLI | `cli-` | `cli-3.24.0` | `straymark` 二进制文件 |
-| Loom（实验性） | `loom-` | `loom-0.4.0` | `straymark-loom` 可视化服务器，由 `straymark loom serve` 按需下载 |
+| Loom（实验性） | `loom-` | `loom-0.4.1` | `straymark-loom` 可视化服务器，由 `straymark loom serve` 按需下载 |
 
 使用 `straymark status` 或 `straymark about` 查看已安装的版本。
 
diff --git a/docs/i18n/zh-CN/adopters/CLI-REFERENCE.md b/docs/i18n/zh-CN/adopters/CLI-REFERENCE.md
@@ -49,7 +49,7 @@ StrayMark 为每个组件使用**独立的版本标签**：
 |------|----------|------|----------|
 | Framework | `fw-` | `fw-4.26.0` | 模板（12 种类型）、治理文档、指令 |
 | CLI | `cli-` | `cli-3.24.0` | `straymark` 二进制文件 |
-| Loom（实验性） | `loom-` | `loom-0.4.0` | `straymark-loom` 可视化服务器，由 `straymark loom serve` 按需下载 |
+| Loom（实验性） | `loom-` | `loom-0.4.1` | `straymark-loom` 可视化服务器，由 `straymark loom serve` 按需下载 |
 
 Framework 和 CLI 独立发布。Framework 更新不需要 CLI 更新，反之亦然。
 
@@ -1231,8 +1231,8 @@ $ straymark loom serve
      Unstable: API, CLI surface, and on-disk layout may change or be
      removed without a deprecation cycle. Loopback-only. Read-only.
 
-ℹ Downloading Loom 0.4.0 (x86_64-unknown-linux-gnu) — first use is opt-in by download
-✔ Loom 0.4.0 cached at ~/.straymark/bin/straymark-loom
+ℹ Downloading Loom 0.4.1 (x86_64-unknown-linux-gnu) — first use is opt-in by download
+✔ Loom 0.4.1 cached at ~/.straymark/bin/straymark-loom
 loom: watching /project/.straymark (142 docs, 318 links)
 loom: serving http://127.0.0.1:7700
 ```
diff --git a/experimento/AILOG-2026-06-13-003-loom-panel-update-storm-fix.md b/experimento/AILOG-2026-06-13-003-loom-panel-update-storm-fix.md
@@ -0,0 +1,81 @@
+---
+id: AILOG-2026-06-13-003
+title: Loom fix — panel clicks swallowed by no-op WebSocket rebuild storm
+status: accepted
+created: 2026-06-13
+agent: claude-fable-5
+confidence: high
+review_required: true
+risk_level: low
+eu_ai_act_risk: not_applicable
+nist_genai_risks: []
+iso_42001_clause: []
+lines_changed: 90
+files_modified: [CHANGELOG.md, Cargo.lock, README.md, cli/Cargo.toml, core/Cargo.toml, core/src/graph.rs, docs/adopters/CLI-REFERENCE.md, docs/i18n/es/README.md, docs/i18n/es/adopters/CLI-REFERENCE.md, docs/i18n/zh-CN/README.md, docs/i18n/zh-CN/adopters/CLI-REFERENCE.md, experimento/CHANGELOG.md, experimento/Cargo.toml, experimento/src/snapshot.rs, experimento/src/watcher.rs, experimento/web/package.json, experimento/web/src/main.ts]
+observability_scope: none
+tags: [loom, bugfix, websocket, watcher, ui, knowledge-graph]
+related: [AILOG-2026-06-13-002, AILOG-2026-06-12-003]
+---
+
+# AILOG: Loom fix — panel clicks swallowed by no-op rebuild storm
+
+## Summary
+
+An operator reported that Loom's left-hand panels (corpus stats, legend) stopped responding
+to clicks after some interaction — the item just clicked and its neighbours went dead while
+others worked, and it persisted across page reload and across browsers, with **no console
+errors**. The DevTools Elements view showed the `#stats` DOM **rewriting itself continuously
+at high speed**.
+
+Root cause: a **no-op WebSocket rebuild storm**. When something re-saves a watched `.md`
+without changing its content (an editor save, a formatter, a `touch`, a cloud-sync rewrite),
+the parse cache re-parses (its mtime moved), the graph is rebuilt **identical**, and the
+watcher still broadcast a `delta`. Each delta re-renders the panels (`innerHTML` rewrite),
+destroying the per-button click handlers bound on the previous render — so clicks landing
+between renders were lost. Fixed at two layers.
+
+## Actions Performed
+
+1. **Server — suppress no-op broadcasts** (`watcher.rs` + `snapshot.rs`): the watcher diffs
+   the rebuilt graph against the previous one and only broadcasts when something actually
+   changed. Added `GraphDelta::is_noop(&prev)` (no node added/removed/changed and identical
+   edges) and `GraphDelta::to_event`; derived `PartialEq` on `core::graph::Edge` and the
+   Loom `ApiEdge` to compare edge sets.
+2. **Client — event delegation** (`web/src/main.ts`): the stats and legend panels now handle
+   clicks via one delegated listener on their stable container element instead of binding a
+   listener per rendered button. Clicks survive the `innerHTML` rewrites the panels perform on
+   every rebuild, so the UI stays responsive even under legitimately frequent updates.
+   Native `<details>` toggling drives the open/closed state, mirrored into the persisted
+   open-section set.
+
+## Decisions Made
+
+- **Both layers, not one.** Server suppression removes the common cause (mtime-only churn);
+  client delegation makes the panels robust to *any* update frequency, including real frequent
+  edits. Defense in depth for a UI that re-renders on a live data feed.
+- **Equality over a dirty flag.** Comparing the new graph to the previous one (via the existing
+  diff plus edge-set equality) is exact and cheap for these corpus sizes; no heuristic.
+
+## Impact
+
+- **Functionality:** the panels stay clickable under filesystem churn; real changes still
+  propagate live. No API or visual change.
+- **Performance:** fewer needless rebuilds reach clients; the panels stop thrashing.
+- **Security:** unchanged.
+
+## Verification
+
+- [x] `cargo test` — core 42/42, loom 9/9 (added a no-op-delta assertion); clippy core+loom
+      clean; `npm run build` ok.
+- [x] Reproduction (temp project, WS client): rewriting identical content 5× produced **10
+      no-op deltas before the fix → 0 after**; a real change (new document) still produces a
+      delta after the fix.
+
+## Follow-ups
+
+- R3 (visual density at 100+ nodes) and the Architecture track (A1/A2) remain the next Loom
+  frontiers.
+
+---
+
+<!-- Template: StrayMark | https://strangedays.tech -->
diff --git a/experimento/CHANGELOG.md b/experimento/CHANGELOG.md
@@ -20,6 +20,21 @@ project adheres to [Semantic Versioning](https://semver.org/).
   overlay, layer toggle, component panel, and cross-linking with the Knowledge Graph.
 - A3 (north star): axonometric/BIM exploded-layers view.
 
+## [0.4.1] — 2026-06-13 (fix: panel click responsiveness under filesystem churn)
+
+### Fixed
+
+- The watcher no longer broadcasts a WebSocket update when a rebuild produces an
+  **identical** graph (a file's modification time moved but its content did not — an editor
+  save without changes, a formatter, a `touch`, a cloud-sync rewrite). Such no-op broadcasts
+  re-rendered every open client's side panels continuously, destroying their freshly-bound
+  click handlers and making actionable items (dangling-reference links, community buttons,
+  stats sections) feel unresponsive.
+- The stats and legend panels now use **event delegation** on their stable container elements
+  instead of binding a listener per rendered button. Clicks survive the innerHTML rewrite the
+  panels perform on every rebuild, so they keep working even under legitimately frequent
+  updates.
+
 ## [0.4.0] — 2026-06-13 (connectivity: reference normalization + entity nodes)
 
 Closes the M1 connectivity follow-ups (R1 + R2) surfaced by dogfooding against the Sentinel
@@ -133,7 +148,8 @@ corpus, where 330 of 395 references were dangling. All changes are in the shared
   (PR #239) together with the component's intention docs (README, SpecKit sets 001/002,
   `CHARTER-01-loom-server`, ADR-2026-06-02-001/-002).
 
-[Unreleased]: https://github.com/StrangeDaysTech/straymark/compare/loom-0.4.0...HEAD
+[Unreleased]: https://github.com/StrangeDaysTech/straymark/compare/loom-0.4.1...HEAD
+[0.4.1]: https://github.com/StrangeDaysTech/straymark/releases/tag/loom-0.4.1
 [0.4.0]: https://github.com/StrangeDaysTech/straymark/releases/tag/loom-0.4.0
 [0.3.0]: https://github.com/StrangeDaysTech/straymark/releases/tag/loom-0.3.0
 [0.2.0]: https://github.com/StrangeDaysTech/straymark/releases/tag/loom-0.2.0
diff --git a/experimento/Cargo.toml b/experimento/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "straymark-loom"
-version = "0.4.0"
+version = "0.4.1"
 edition = "2021"
 description = "Loom — StrayMark's experimental knowledge-graph visualization server (localhost development dashboard)"
 license = "MIT"
@@ -14,7 +14,7 @@ name = "straymark-loom"
 path = "src/main.rs"
 
 [dependencies]
-straymark-core = { version = "0.4.0", path = "../core" }
+straymark-core = { version = "0.4.1", path = "../core" }
 tokio = { version = "1", features = ["rt-multi-thread", "macros", "sync", "signal", "fs"] }
 axum = { version = "0.8", features = ["ws"] }
 notify-debouncer-full = "0.5"
diff --git a/experimento/src/snapshot.rs b/experimento/src/snapshot.rs
@@ -31,7 +31,7 @@ pub struct CachedDoc {
 
 /// An edge as served by the API: the core edge plus its stable index id
 /// (thread highlighting addresses edges by this id).
-#[derive(Debug, Clone, Serialize)]
+#[derive(Debug, Clone, PartialEq, Serialize)]
 pub struct ApiEdge {
     pub id: usize,
     #[serde(flatten)]
@@ -197,6 +197,29 @@ impl ApiGraph {
     }
 }
 
+impl GraphDelta {
+    /// True when this delta changes nothing versus `prev` — no node added,
+    /// removed or changed, and the same edge set. A rebuild can be a no-op when
+    /// a file's modification time moved but its content did not (an editor save
+    /// without changes, a formatter, a `touch`, a cloud-sync rewrite). The
+    /// watcher suppresses these so open clients are not re-rendered needlessly.
+    pub fn is_noop(&self, prev: &ApiGraph) -> bool {
+        self.added.is_empty()
+            && self.removed.is_empty()
+            && self.changed.is_empty()
+            && self.edges == prev.edges
+    }
+
+    /// Serialize as the `{event:"delta", …}` WS message.
+    pub fn to_event(&self) -> String {
+        let mut value = serde_json::to_value(self).unwrap_or_default();
+        if let Some(obj) = value.as_object_mut() {
+            obj.insert("event".into(), serde_json::json!("delta"));
+        }
+        value.to_string()
+    }
+}
+
 /// One rebuilt view of the corpus.
 pub struct Snapshot {
     /// The core graph (kept for thread queries).
@@ -280,16 +303,6 @@ impl Snapshot {
         serde_json::json!({ "event": "rebuild", "graph": &self.api }).to_string()
     }
 
-    /// The serialized WS `delta` event: this snapshot's graph diffed against
-    /// `prev` (T3.1). Sent on settled filesystem changes so open clients patch
-    /// rather than re-layout.
-    pub fn delta_event(&self, prev: &ApiGraph) -> String {
-        let mut value = serde_json::to_value(self.api.diff(prev)).unwrap_or_default();
-        if let Some(obj) = value.as_object_mut() {
-            obj.insert("event".into(), serde_json::json!("delta"));
-        }
-        value.to_string()
-    }
 }
 
 fn build_api_view(graph: &Graph) -> ApiGraph {
@@ -526,9 +539,12 @@ mod tests {
         assert_eq!(delta.changed.iter().map(|n| n.id.as_str()).collect::<Vec<_>>(), vec!["REQ-1"]);
 
         // The delta event is valid JSON tagged `delta`.
-        let event: serde_json::Value = serde_json::from_str(&next.delta_event(&prev.api)).unwrap();
+        let event: serde_json::Value =
+            serde_json::from_str(&next.api.diff(&prev.api).to_event()).unwrap();
         assert_eq!(event["event"], "delta");
         assert_eq!(event["removed"][0], "ADR-1");
+        // An identical rebuild produces a no-op delta.
+        assert!(next.api.diff(&next.api).is_noop(&next.api));
     }
 
     #[test]
diff --git a/experimento/src/watcher.rs b/experimento/src/watcher.rs
@@ -46,10 +46,15 @@ pub fn watch(
                     // Diff against the previous snapshot so clients patch
                     // (preserve layout) instead of re-laying out the graph.
                     let prev_api = snapshot.read().expect("snapshot lock poisoned").api.clone();
-                    let event = next.delta_event(&prev_api);
+                    let delta = next.api.diff(&prev_api);
                     *snapshot.write().expect("snapshot lock poisoned") = next;
-                    // No receivers (no open browser) is fine — ignore the error.
-                    let _ = events.send(event);
+                    // Suppress no-op rebuilds (a file's mtime moved but its
+                    // content did not): broadcasting them would churn every open
+                    // client's panels and swallow clicks for nothing.
+                    if !delta.is_noop(&prev_api) {
+                        // No receivers (no open browser) is fine — ignore the error.
+                        let _ = events.send(delta.to_event());
+                    }
                 }
                 Err(err) => {
                     eprintln!("loom: rebuild failed (will retry on next change): {err:#}");
diff --git a/experimento/web/package.json b/experimento/web/package.json
@@ -1,7 +1,7 @@
 {
   "name": "straymark-loom-web",
   "private": true,
-  "version": "0.4.0",
+  "version": "0.4.1",
   "type": "module",
   "scripts": {
     "dev": "vite",
diff --git a/experimento/web/src/main.ts b/experimento/web/src/main.ts
