@yeaft/webchat-agent 0.1.811 → 0.1.813

package/package.json

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

package/unify/vp/vp-bridge.js

@@ -28,6 +28,7 @@ import { STOCK_VP_IDS } from './stock-ids.js';
 /** Process-singleton VpLoader; lazily started on first subscribe. */
 let _loaderStarted = false;
 let _loader = null;
+let _loaderDir = null;
 
 /**
  * Broadcast fan-out. VpLoader.onChange fires once per debounce batch for the
@@ -145,21 +146,33 @@ function _fanout(evt) {
   }
 }
 
-function ensureLoader(registry = defaultRegistry) {
-  if (_loaderStarted) return { loader: _loader, fresh: false };
+function ensureLoader(registry = defaultRegistry, options = {}) {
+  const desiredDir = registry === defaultRegistry && typeof options.dir === 'string' && options.dir.trim()
+    ? options.dir.trim()
+    : null;
+  if (_loaderStarted && _loaderDir === desiredDir) return { loader: _loader, fresh: false };
+
+  if (_loader) {
+    try { _loader.stop(); } catch { /* ignore */ }
+  }
   _loaderStarted = true;
+  _loader = null;
+  _loaderDir = desiredDir;
+  if (registry === defaultRegistry && registry.vpMap && typeof registry.vpMap.clear === 'function') {
+    registry.vpMap.clear();
+  }
   // For NON-default registries (unit tests seeding VPs manually) we MUST NOT
-  // start VpLoader — its .start() scans DEFAULT_VP_LIB_DIR and push-imports
-  // every on-disk VP into the test registry, overwriting/augmenting the
-  // fixture. Tests don't need hot-reload anyway; `_broadcastChangeForTest`
-  // drives the diff path directly.
+  // start VpLoader — its .start() scans the configured/default VP library and
+  // push-imports every on-disk VP into the test registry, overwriting or
+  // augmenting the fixture. Tests don't need hot-reload anyway;
+  // `_broadcastChangeForTest` drives the diff path directly.
   if (registry !== defaultRegistry) {
-    _loader = null;
     captureState(registry);
     return { loader: null, fresh: true };
   }
   try {
     _loader = new VpLoader({
+      ...(desiredDir ? { dir: desiredDir } : {}),
       registry,
       onChange: (summary) => broadcastChange(summary, registry),
     });
@@ -228,10 +241,11 @@ export function buildVpSnapshot(registry = defaultRegistry) {
  *
  * @param {(event: object) => void} sendUnifyEvent
  * @param {import('./registry.js').Registry} [registry]
+ * @param {{dir?: string}} [options]
  * @returns {() => void} unsubscribe fn
  */
-export function handleVpSubscribe(sendUnifyEvent, registry = defaultRegistry) {
-  const { loader, fresh } = ensureLoader(registry);
+export function handleVpSubscribe(sendUnifyEvent, registry = defaultRegistry, options = {}) {
+  const { loader, fresh } = ensureLoader(registry, options);
   // task-338-F2 + task-339-followup: replay semantics.
   //
   // The loader's own start() already scans on first creation, so on a
@@ -245,9 +259,9 @@ export function handleVpSubscribe(sendUnifyEvent, registry = defaultRegistry) {
   // production path so the first snapshot always reflects current disk.
   //
   // For NON-default registries (unit tests that seed VPs manually), we
-  // MUST skip the rescan: rescan against DEFAULT_VP_LIB_DIR would call
-  // registry.removeVp() for seeded ids that don't exist on disk, wiping
-  // the test fixture. See vp-bridge-live-diff.test.js and
+  // MUST skip the rescan: rescan against the configured/default VP library
+  // would call registry.removeVp() for seeded ids that don't exist on disk,
+  // wiping the test fixture. See vp-bridge-live-diff.test.js and
   // vp-bridge-first-subscribe-replay.test.js (test-seed preservation).
   //
   // On subsequent subscribes (page reload, reconnect, second web client)
@@ -277,6 +291,7 @@ export function _resetVpBridgeForTest() {
   }
   _loader = null;
   _loaderStarted = false;
+  _loaderDir = null;
   _subscribers.clear();
   _prevState.clear();
 }

package/unify/web-bridge.js

@@ -1188,12 +1188,27 @@ function sendUnifyEvent(event, { groupId, vpId, turnId, threadId } = {}) {
   });
 }
 
+function configuredVpPaths() {
+  const yeaftDir = ctx.CONFIG?.yeaftDir;
+  if (typeof yeaftDir !== 'string' || !yeaftDir.trim()) return {};
+  const root = yeaftDir.trim();
+  return {
+    libDir: join(root, 'virtual-persons'),
+    memoryRoot: join(root, 'memory'),
+  };
+}
+
 export function handleUnifyVpSubscribe(_msg) {
   if (_vpUnsubscribe) {
     try { _vpUnsubscribe(); } catch { /* ignore */ }
     _vpUnsubscribe = null;
   }
-  _vpUnsubscribe = handleVpSubscribe(sendUnifyEvent);
+  const { libDir } = configuredVpPaths();
+  _vpUnsubscribe = handleVpSubscribe(
+    sendUnifyEvent,
+    undefined,
+    libDir ? { dir: libDir } : {},
+  );
 }
 
 /**
@@ -1207,9 +1222,12 @@ export function handleUnifyVpCreate(msg) {
   const requestId = msg && msg.requestId;
   const payload = msg && msg.payload;
   try {
-    const yeaftDir = ctx.CONFIG?.yeaftDir;
-    const memoryRoot = yeaftDir ? join(yeaftDir, 'memory') : undefined;
-    const { vpId } = createVp(payload || {}, memoryRoot ? { memoryRoot } : {});
+    const { libDir, memoryRoot } = configuredVpPaths();
+    const options = {
+      ...(libDir ? { libDir } : {}),
+      ...(memoryRoot ? { memoryRoot } : {}),
+    };
+    const { vpId } = createVp(payload || {}, options);
     sendVpCrudResult({ op: 'create', requestId, ok: true, vpId });
   } catch (err) {
     sendVpCrudResult({
@@ -1229,7 +1247,8 @@ export function handleUnifyVpUpdate(msg) {
   const requestId = msg && msg.requestId;
   const payload = msg && msg.payload;
   try {
-    const { vpId } = updateVp(payload || {});
+    const { libDir } = configuredVpPaths();
+    const { vpId } = updateVp(payload || {}, libDir ? { libDir } : {});
     sendVpCrudResult({ op: 'update', requestId, ok: true, vpId });
   } catch (err) {
     sendVpCrudResult({
@@ -1249,9 +1268,12 @@ export function handleUnifyVpDelete(msg) {
   const requestId = msg && msg.requestId;
   const vpId = msg && msg.vpId;
   try {
-    const yeaftDir = ctx.CONFIG?.yeaftDir;
-    const memoryRoot = yeaftDir ? join(yeaftDir, 'memory') : undefined;
-    deleteVp(vpId, memoryRoot ? { memoryRoot } : {});
+    const { libDir, memoryRoot } = configuredVpPaths();
+    const options = {
+      ...(libDir ? { libDir } : {}),
+      ...(memoryRoot ? { memoryRoot } : {}),
+    };
+    deleteVp(vpId, options);
     // vp-status: a deleted VP must not haunt the snapshot. We don't
     // know up front which groups the VP appeared in (the registry's
     // delete already detached it from every group), so sweep every
@@ -1282,7 +1304,8 @@ export function handleUnifyVpDelete(msg) {
 export function handleUnifyVpRead(msg) {
   const requestId = msg && msg.requestId;
   const vpId = msg && msg.vpId;
-  const vp = readVp(vpId);
+  const { libDir } = configuredVpPaths();
+  const vp = readVp(vpId, libDir ? { libDir } : {});
   if (!vp) {
     sendVpCrudResult({
       op: 'read',