@worca/ui 0.31.0 → 0.32.0

package/app/styles.css

@@ -2425,6 +2425,13 @@ sl-details.agent-prompt-section::part(content) {
   margin-bottom: 12px;
 }
 
+/* Divider + gap before the user message header */
+.agent-prompt-block + .agent-prompt-block {
+  margin-top: 18px;
+  padding-top: 16px;
+  border-top: 1px solid var(--border-subtle);
+}
+
 .agent-prompt-label-row {
   display: flex;
   align-items: center;
@@ -2455,12 +2462,14 @@ sl-details.agent-prompt-section::part(content) {
   flex-shrink: 0;
 }
 
+/* Accent the section headers (not the message bodies) so each block reads as
+   a labelled section; the bodies stay neutral. */
 .agent-prompt-label {
-  font-size: 11px;
-  font-weight: 600;
-  color: var(--muted);
+  font-size: 12px;
+  font-weight: 700;
+  color: var(--accent);
   text-transform: uppercase;
-  letter-spacing: 0.5px;
+  letter-spacing: 0.6px;
   margin-bottom: 4px;
 }
 
@@ -4833,7 +4842,7 @@ sl-tooltip.bead-tooltip::part(body) {
 
 .bead-tooltip-excerpt {
   font-size: 12px;
-  white-space: pre-wrap;
+  white-space: normal;
   margin-bottom: 2px;
 }
 
@@ -6248,6 +6257,47 @@ sl-dialog.markdown-dialog::part(body) {
   text-align: left;
 }
 
+/* --- Markdown context overrides --- */
+
+.markdown-inline.markdown-body {
+  display: inline;
+  font-size: inherit;
+  line-height: inherit;
+}
+.markdown-inline.markdown-body p {
+  display: inline;
+  margin: 0;
+}
+.markdown-inline.markdown-body code {
+  font-size: 0.85em;
+}
+
+.agent-prompt-content .markdown-body {
+  white-space: normal;
+  font-size: 12px;
+  line-height: 1.5;
+}
+.agent-prompt-content .markdown-body h1,
+.agent-prompt-content .markdown-body h2,
+.agent-prompt-content .markdown-body h3 {
+  margin: 0.8em 0 0.3em;
+}
+
+.bead-tooltip-excerpt.markdown-body {
+  white-space: normal;
+  font-size: 12px;
+  max-height: 200px;
+  overflow-y: auto;
+}
+.bead-tooltip-excerpt.markdown-body p {
+  margin: 0.3em 0;
+}
+.bead-tooltip-excerpt.markdown-body pre {
+  font-size: 11px;
+  max-height: 120px;
+  overflow: auto;
+}
+
 .workspace-run-card .workspace-card-root,
 .workspace-run-card .workspace-card-name {
   font-family: var(--font-mono);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@worca/ui",
-  "version": "0.31.0",
+  "version": "0.32.0",
   "description": "Pipeline monitoring UI for worca-cc",
   "license": "MIT",
   "author": "Sinisha Djukic",
@@ -65,6 +65,7 @@
     "@xterm/addon-fit": "^0.11.0",
     "@xterm/addon-search": "^0.16.0",
     "@xterm/xterm": "^6.0.0",
+    "dompurify": "^3.4.5",
     "express": "^5.2.1",
     "lit-html": "^3.3.1",
     "lucide": "^0.577.0",

package/server/beads-reader.js

@@ -40,7 +40,7 @@ function transformIssue(issue, deps) {
   };
 }
 
-async function enrichWithDeps(issues, dbPath) {
+export async function enrichIssuesWithDeps(issues, dbPath) {
   if (issues.length === 0) return [];
   const detailed = await runBd(['show', ...issues.map((i) => i.id)], dbPath);
   const detailMap = new Map(detailed.map((d) => [d.id, d]));
@@ -54,13 +54,21 @@ export function dbExists(beadsDb) {
   return existsSync(beadsDb);
 }
 
+export async function listIssuesShallow(beadsDb) {
+  try {
+    return await runBd(['list', '--limit', '0'], beadsDb);
+  } catch {
+    return [];
+  }
+}
+
 export async function listIssues(beadsDb) {
   try {
     const issues = await runBd(['list', '--limit', '0'], beadsDb);
-    // Must await here — without it, an enrichWithDeps rejection (e.g. bd show
+    // Must await here — without it, an enrichIssuesWithDeps rejection (e.g. bd show
     // SIGTERM under daemon contention) escapes the try/catch and propagates
     // to the WS handler as an unhandled rejection, crashing Node.
-    return await enrichWithDeps(issues, beadsDb);
+    return await enrichIssuesWithDeps(issues, beadsDb);
   } catch {
     return [];
   }
@@ -72,7 +80,7 @@ export async function listIssuesByLabel(beadsDb, label) {
       ['list', '--label-any', label, '--all', '--limit', '0'],
       beadsDb,
     );
-    return await enrichWithDeps(issues, beadsDb);
+    return await enrichIssuesWithDeps(issues, beadsDb);
   };
   try {
     return await attempt();

package/server/ws-beads-watcher.js

@@ -6,7 +6,11 @@
 
 import { existsSync, statSync, unwatchFile, watch, watchFile } from 'node:fs';
 import { join, resolve } from 'node:path';
-import { countIssuesByRunLabel, listIssues } from './beads-reader.js';
+import {
+  countIssuesByRunLabel,
+  enrichIssuesWithDeps,
+  listIssuesShallow,
+} from './beads-reader.js';
 
 const BEADS_DEBOUNCE_MS = 500;
 const BEADS_POLL_MS = 2000;
@@ -34,19 +38,73 @@ export function createBeadsWatcher({ worcaDir, broadcaster, projectId }) {
   let lastPayloadJson = null;
   let lastSelfReadWalStat = null;
   let latestCounts = {};
+  // In-flight guard + trailing coalesce. The refresh body spawns several `bd`
+  // subprocesses (listIssuesShallow, then enrichIssuesWithDeps +
+  // countIssuesByRunLabel when the fingerprint changes) that, on a large beads
+  // db, take seconds. The debounce only collapses scheduling — once the async body
+  // is awaiting, a fresh db/WAL event would otherwise start an overlapping
+  // refresh and pile up bd processes unbounded. Allow at most one refresh in
+  // flight; events arriving mid-refresh collapse into a single trailing pass.
+  let refreshing = false;
+  let refreshPending = false;
+  let lastListFingerprint = null;
+
+  function computeFingerprint(issues) {
+    const sorted = [...issues].sort((a, b) => (a.id < b.id ? -1 : 1));
+    return JSON.stringify(
+      sorted.map((i) => ({
+        id: i.id,
+        status: i.status,
+        priority: i.priority,
+        title: i.title,
+        updated_at: i.updated_at,
+        // dependency_count/dependent_count come free in `bd list` (no `bd show`).
+        // They catch dependency-edge changes (`bd dep add/remove`) that don't
+        // bump an issue's own updated_at — without them the fingerprint would
+        // bail and leave depends_on/blocked_by (and blocked badges) stale.
+        dependency_count: i.dependency_count,
+        dependent_count: i.dependent_count,
+      })),
+    );
+  }
+
+  function recordWalStat() {
+    try {
+      const s = statSync(beadsWalPath);
+      lastSelfReadWalStat = { mtimeMs: s.mtimeMs, size: s.size };
+    } catch {
+      lastSelfReadWalStat = null;
+    }
+  }
 
   function scheduleBeadsRefresh() {
     if (BEADS_REFRESH_TIMER) clearTimeout(BEADS_REFRESH_TIMER);
-    BEADS_REFRESH_TIMER = setTimeout(async () => {
-      BEADS_REFRESH_TIMER = null;
-      try {
-        const [issues, counts] = await Promise.all([
-          listIssues(beadsDbPath),
-          countIssuesByRunLabel(beadsDbPath).catch(() => ({})),
-        ]);
-        latestCounts = counts;
-        const payloadJson = JSON.stringify({ issues, counts });
-        if (payloadJson === lastPayloadJson) return;
+    BEADS_REFRESH_TIMER = setTimeout(runBeadsRefresh, BEADS_DEBOUNCE_MS);
+  }
+
+  async function runBeadsRefresh() {
+    BEADS_REFRESH_TIMER = null;
+    if (refreshing) {
+      refreshPending = true;
+      return;
+    }
+    refreshing = true;
+    try {
+      const shallowIssues = await listIssuesShallow(beadsDbPath);
+      const fingerprint = computeFingerprint(shallowIssues);
+      if (fingerprint === lastListFingerprint) {
+        recordWalStat();
+        return;
+      }
+      lastListFingerprint = fingerprint;
+
+      const [issues, counts] = await Promise.all([
+        enrichIssuesWithDeps(shallowIssues, beadsDbPath),
+        countIssuesByRunLabel(beadsDbPath).catch(() => ({})),
+      ]);
+      latestCounts = counts;
+      const payloadJson = JSON.stringify({ issues, counts });
+      if (payloadJson !== lastPayloadJson) {
         lastPayloadJson = payloadJson;
         broadcaster.broadcast(
           'beads-update',
@@ -58,16 +116,17 @@ export function createBeadsWatcher({ worcaDir, broadcaster, projectId }) {
           },
           projectId,
         );
-        try {
-          const s = statSync(beadsWalPath);
-          lastSelfReadWalStat = { mtimeMs: s.mtimeMs, size: s.size };
-        } catch {
-          lastSelfReadWalStat = null;
-        }
-      } catch {
-        /* ignore */
       }
-    }, BEADS_DEBOUNCE_MS);
+      recordWalStat();
+    } catch {
+      /* ignore */
+    } finally {
+      refreshing = false;
+      if (refreshPending) {
+        refreshPending = false;
+        scheduleBeadsRefresh();
+      }
+    }
   }
 
   if (existsSync(beadsDir)) {
@@ -116,7 +175,11 @@ export function createBeadsWatcher({ worcaDir, broadcaster, projectId }) {
   return { getBeadsDbPath, getLatestCounts, destroy };
 }
 
-const FALLBACK_TTL_MS = 5000;
+// Throttle window for the cold-path bead-count read (used by REST /runs and chat
+// when no live watcher is warm). The read costs seconds on a large db, so a short
+// TTL lets repeated /runs across many projects re-spawn `bd` too often. Counts are
+// advisory and the live watcher keeps a viewed project fresh, so 30s is ample.
+const FALLBACK_TTL_MS = 30000;
 /** @type {Map<string, { ts: number, counts: object }>} */
 const fallbackCache = new Map();
 /** @type {Map<string, Promise<object>>} */
@@ -163,3 +226,50 @@ export async function resolveBeadsCounts(wset) {
   fallbackInflight.set(key, promise);
   return promise;
 }
+
+/**
+ * Non-blocking variant of {@link resolveBeadsCounts}: returns whatever counts
+ * are already available (the live watcher cache, or a fresh TTL-cached fallback)
+ * WITHOUT ever awaiting the cold `bd` query. When nothing is cached it kicks off
+ * a background refresh (reusing the same TTL cache + in-flight dedup) so a later
+ * caller gets real data, and returns {} immediately.
+ *
+ * Used by the REST `/runs` endpoint: a cold `bd show` on a large beads db
+ * (worca-cc has hundreds of issues — the cold read takes ~10s) would otherwise
+ * block the run list and, fanned out across every project, stall the whole UI.
+ * Bead counts on run cards are advisory and arrive via the `beads-update`
+ * broadcast once warm; the web run-detail does not use them at all.
+ *
+ * @param {{ projectId?: string, worcaDir?: string, beadsWatcher?: { getLatestCounts: () => object } | null }} [wset]
+ * @returns {Record<string, { total: number, done: number }>}
+ */
+export function peekBeadsCounts(wset) {
+  if (!wset || !wset.worcaDir) return {};
+
+  const live = wset.beadsWatcher?.getLatestCounts();
+  if (live && Object.keys(live).length > 0) return live;
+
+  const key = wset.projectId ?? wset.worcaDir;
+  const cached = fallbackCache.get(key);
+  if (cached && Date.now() - cached.ts < FALLBACK_TTL_MS) return cached.counts;
+
+  // Cold cache: warm it in the background (dedup'd), but never block the caller.
+  if (!fallbackInflight.has(key)) {
+    const dbPath = beadsDbPathFor(wset.worcaDir);
+    if (existsSync(dbPath)) {
+      const promise = (async () => {
+        try {
+          const counts = await countIssuesByRunLabel(dbPath);
+          fallbackCache.set(key, { ts: Date.now(), counts });
+          return counts;
+        } catch {
+          return {};
+        } finally {
+          fallbackInflight.delete(key);
+        }
+      })();
+      fallbackInflight.set(key, promise);
+    }
+  }
+  return {};
+}

package/server/ws-modular.js

@@ -13,7 +13,7 @@ import { fleetRunsDir, workspaceRunsDir } from './paths.js';
 import { readProjects, synthesizeDefaultProject } from './project-registry.js';
 import { TIER_FULL, TIER_POLLING, WatcherSet } from './watcher-set.js';
 import { readProjectWorcaVersion } from './worca-setup.js';
-import { resolveBeadsCounts } from './ws-beads-watcher.js';
+import { peekBeadsCounts } from './ws-beads-watcher.js';
 import { createBroadcaster } from './ws-broadcaster.js';
 import { createClientManager } from './ws-client-manager.js';
 import { createFleetManifestWatcher } from './ws-fleet-manifest-watcher.js';
@@ -331,8 +331,10 @@ export function attachWsServer(httpServer, config) {
     return null;
   }
 
+  // Non-blocking: returns cached/live counts and warms cold caches in the
+  // background. The REST /runs endpoint must never block on a slow `bd` read.
   function getBeadsCounts(projectId) {
-    return resolveBeadsCounts(watcherSets.get(projectId));
+    return peekBeadsCounts(watcherSets.get(projectId));
   }
 
   return {