@yeaft/webchat-agent 0.1.518 → 0.1.519

package/connection/message-router.js

@@ -36,7 +36,7 @@ import { sendToServer, flushMessageBuffer } from './buffer.js';
 import { handleRestartAgent, handleUpgradeAgent } from './upgrade.js';
 import { loadMcpServers, updateMcpConfig } from '../mcp.js';
 import { getLlmConfig, updateLlmConfig, getUnifySettings, updateUnifySettings } from '../unify/config-api.js';
-import { handleUnifyChat, handleUnifyModeSwitch, handleUnifyModelSwitch, resetUnifySession, handleUnifyLoadHistory, handleUnifyMergeThread, handleUnifyForkThread, handleUnifyAbortThread, handleUnifyAbortAll } from '../unify/web-bridge.js';
+import { handleUnifyChat, handleUnifyModeSwitch, handleUnifyModelSwitch, resetUnifySession, handleUnifyLoadHistory, handleUnifyMergeThread, handleUnifyForkThread, handleUnifyAbortThread, handleUnifyAbortAll, handleUnifyVpSubscribe } from '../unify/web-bridge.js';
 
 export async function handleMessage(msg) {
   switch (msg.type) {
@@ -397,6 +397,12 @@ export async function handleMessage(msg) {
       handleUnifyAbortAll();
       break;
 
+    // task-334-ui-a: VP library subscribe — replies with one-shot
+    // vp_snapshot event. Live diff (vp_updated/vp_removed) deferred to 334h.
+    case 'unify_vp_subscribe':
+      handleUnifyVpSubscribe(msg);
+      break;
+
     // Expert roles definition (for ExpertPanel detail view)
     case 'get_expert_roles': {
       const { getExpertRolesDefinition } = await import('../expert-roles.js');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yeaft/webchat-agent",
-  "version": "0.1.518",
+  "version": "0.1.519",
   "description": "Remote agent for Yeaft WebChat — connects worker machines to the central server",
   "main": "index.js",
   "type": "module",

package/unify/vp/vp-bridge.js

@@ -0,0 +1,102 @@
+/**
+ * vp-bridge.js — task-334-ui-a snapshot-only WS adapter.
+ *
+ * Per ruling .crew/context/task-334-ui-a-ruling.md §3 (D3 = mixed):
+ *   • snapshot path lives here (this slice, in-scope)
+ *   • live diff (vp_updated / vp_removed) deferred to 334h — see TODO below
+ *
+ * This module is the SOLE serialiser for the wire-format VP shape.
+ * Per ruling §1 (D1 = (b) web-bridge boundary rename), the entity layer
+ * (`vp-store.js` / `registry.js`) keeps `id` / `name`; here we map to
+ * `vpId` / `displayName` per the spec (§2.1) / architecture §R6.11.
+ *
+ * Per ruling §2 (D2):
+ *   • subtitle  → agent emits `vp.role` directly
+ *   • personaHash → agent emits `vp.personaHash` (added by dev-1's
+ *     334a-followup patch). Until that lands, this serialiser falls back
+ *     to undefined and the web layer simply omits the field.
+ *   • color / avatar → web-derived, NOT emitted here
+ */
+
+import { defaultRegistry } from './registry.js';
+import { VpLoader } from './vp-loader.js';
+
+/** Process-singleton VpLoader; lazily started on first subscribe. */
+let _loaderStarted = false;
+let _loader = null;
+
+function ensureLoader() {
+  if (_loaderStarted) return _loader;
+  _loaderStarted = true;
+  try {
+    _loader = new VpLoader({ registry: defaultRegistry });
+    _loader.start();
+  } catch {
+    // Hot-reload optional; subscribe still returns whatever scan loaded.
+    _loader = null;
+  }
+  return _loader;
+}
+
+/**
+ * Serialise a VP (entity layer shape) to the wire-format the web layer
+ * expects (spec §2.1). Pure; no IO.
+ *
+ * @param {{id:string,name:string,role:string,traits?:string[],modelHint?:string,personaHash?:string}} vp
+ * @returns {{vpId:string,displayName:string,subtitle:string,role:string,traits:string[],modelHint:?string,personaHash:?string}}
+ */
+export function serializeVpForWire(vp) {
+  return {
+    vpId: vp.id,
+    displayName: vp.name,
+    role: vp.role || '',
+    subtitle: vp.role || '',
+    traits: Array.isArray(vp.traits) ? vp.traits.slice() : [],
+    modelHint: vp.modelHint ?? null,
+    personaHash: vp.personaHash ?? null,
+  };
+}
+
+/**
+ * Build a vp_snapshot event payload from the registry.
+ * @param {import('./registry.js').Registry} [registry]
+ * @returns {{type:'vp_snapshot', vps:Array, emptyLibrary:boolean}}
+ */
+export function buildVpSnapshot(registry = defaultRegistry) {
+  const vps = registry.listVps().map(serializeVpForWire);
+  return {
+    type: 'vp_snapshot',
+    vps,
+    emptyLibrary: vps.length === 0,
+  };
+}
+
+/**
+ * Handle an `unify_vp_subscribe` request from the web client.
+ * Lazily starts the VpLoader on first call (debounced rescan watchers).
+ *
+ * @param {(event: object) => void} sendUnifyEvent — emit fn (web-bridge wires this)
+ * @param {import('./registry.js').Registry} [registry]
+ */
+export function handleVpSubscribe(sendUnifyEvent, registry = defaultRegistry) {
+  ensureLoader();
+  try {
+    sendUnifyEvent(buildVpSnapshot(registry));
+  } catch {
+    // Never crash the WS pipeline from snapshot serialisation.
+  }
+  // TODO(334h): vp_updated / vp_removed live broadcast.
+  // Wire VpLoader.onChange → emit per-vp `vp_updated` and `vp_removed`
+  // events using serializeVpForWire(). Out of scope for 334-ui-a.
+}
+
+/**
+ * Test seam: reset the lazy loader (for vitest).
+ */
+export function _resetVpBridgeForTest() {
+  if (_loader) {
+    try { _loader.stop(); } catch { /* ignore */ }
+  }
+  _loader = null;
+  _loaderStarted = false;
+}

package/unify/web-bridge.js

@@ -26,6 +26,7 @@ import { loadSession } from './session.js';
 import { sendToServer } from '../connection/buffer.js';
 import ctx from '../context.js';
 import { getThreadStore, MAIN_THREAD_ID } from './threads/store.js';
+import { handleVpSubscribe } from './vp/vp-bridge.js';
 
 /** @type {import('./session.js').Session | null} */
 let session = null;
@@ -103,6 +104,15 @@ function sendUnifyEvent(event) {
   });
 }
 
+/**
+ * task-334-ui-a: respond to `unify_vp_subscribe` from the web client by
+ * pushing a one-shot `vp_snapshot` event. Live diff (vp_updated /
+ * vp_removed) is intentionally deferred to 334h per ruling §3.
+ */
+export function handleUnifyVpSubscribe(_msg) {
+  handleVpSubscribe(sendUnifyEvent);
+}
+
 /**
  * task-318 rev-1 fix: install live-setter bridge between the session's
  * runtime handles (engineRegistry + threadStore) and `ctx.unifyRuntimeSettings`,