@yeaft/webchat-agent 0.1.863 → 0.1.864

package/package.json

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

package/yeaft/web-bridge.js

@@ -1717,6 +1717,32 @@ function chatErrorPayload(err) {
   };
 }
 
+/**
+ * Build the vpPersona payload threaded into engine.query so the worker
+ * system prompt carries the VP's identity/role/persona/planInstruction.
+ * Returns null on miss — callers treat that as "use generic prompt".
+ * Shared by handleYeaftChatSend and the group fan-out path so the field
+ * set stays in lockstep.
+ */
+function buildVpPersona(vpId) {
+  if (!vpId) return null;
+  try {
+    const vp = readVp(vpId);
+    if (!vp) return null;
+    return {
+      vpId,
+      displayName: vp.displayName || vpId,
+      displayNameZh: vp.displayNameZh || '',
+      role: vp.role || '',
+      roleZh: vp.roleZh || '',
+      persona: vp.persona || '',
+      planInstruction: typeof vp.planInstruction === 'string' ? vp.planInstruction : '',
+    };
+  } catch {
+    return null;
+  }
+}
+
 export function handleYeaftListChats(msg) {
   const requestId = msg && msg.requestId;
   try {
@@ -1729,19 +1755,34 @@ export function handleYeaftListChats(msg) {
 
 export function handleYeaftCreateChat(msg) {
   const requestId = msg && msg.requestId;
+  // Accept fields at top-level (current wire shape) AND inside `payload`
+  // (legacy / future) — keeps backwards compatibility cheap.
+  const top = msg || {};
   const payload = (msg && msg.payload) || {};
+  const displayName = payload.displayName || top.displayName;
+  const workDir = payload.workDir || top.workDir;
+  const explicitId = payload.id || top.id;
+  // Chat mode is 1:1 with the built-in Omni assistant by default. The
+  // VP picker is gone from the UI; callers may still override vpId to
+  // bind a chat to a specialist, but the default is omni.
+  const vpId = payload.vpId || top.vpId || 'omni';
   try {
     const yeaftDir = ctx.CONFIG?.yeaftDir;
     if (!yeaftDir) throw new Error('no yeaft directory configured');
-    if (!payload.vpId) throw new Error('vpId required');
+    // Fail loud at create time if the requested VP (default: omni) is
+    // not installed — otherwise the chat would silently degrade to a
+    // generic prompt at first send.
+    if (!buildVpPersona(vpId)) {
+      throw new Error(`VP '${vpId}' is not installed`);
+    }
     const root = chatsRootFor(yeaftDir);
-    const chatId = (payload.id && String(payload.id).trim())
+    const chatId = (explicitId && String(explicitId).trim())
       || `chat_${Date.now().toString(36)}_${Math.random().toString(36).slice(2, 8)}`;
     const h = createChatStore(root, {
       id: chatId,
-      vpId: payload.vpId,
-      displayName: payload.displayName,
-      workDir: payload.workDir,
+      vpId,
+      displayName,
+      workDir,
     });
     const meta = h.getMeta();
     h.close();
@@ -1954,6 +1995,11 @@ export async function handleYeaftChatSend(msg) {
       appendedUserPrompts,
     };
 
+    // Load the VP persona (defaults to omni) so the engine's worker
+    // prompt has the right identity/role/persona blocks. Without this,
+    // chat mode runs with a generic system prompt instead of Omni.
+    const vpPersona = buildVpPersona(vpId);
+
     for await (const event of eng.query({
       prompt: text,
       promptParts: attachmentBundle.promptParts && attachmentBundle.promptParts.length > 0 ? attachmentBundle.promptParts : null,
@@ -1961,6 +2007,7 @@ export async function handleYeaftChatSend(msg) {
       userAlreadyPersisted: false,
       threadId: 'main',
       senderVpId: vpId,
+      vpPersona,
     })) {
       resetQueryTimer();
       handleEngineEvent(event, handlerCtx);
@@ -2760,23 +2807,8 @@ export function buildVpQueryOpts({ vpId, groupCoordinator, groupId, envelope, th
   if (groupMeta && typeof groupMeta.workDir === 'string' && groupMeta.workDir.trim()) {
     out.workDir = groupMeta.workDir.trim();
   }
-  try {
-    const vp = readVp(resolvedVpId);
-    if (vp) {
-      out.vpPersona = {
-        vpId: resolvedVpId,
-        displayName: vp.displayName || resolvedVpId,
-        displayNameZh: vp.displayNameZh || '',
-        role: vp.role || '',
-        roleZh: vp.roleZh || '',
-        persona: vp.persona || '',
-        // Optional per-VP planning style for the `StartPlan` tool. Empty
-        // string means "fall back to the default template" — the tool
-        // handles the lookup so callers stay ignorant of the default.
-        planInstruction: typeof vp.planInstruction === 'string' ? vp.planInstruction : '',
-      };
-    }
-  } catch { /* persona load is best-effort */ }
+  const persona = buildVpPersona(resolvedVpId);
+  if (persona) out.vpPersona = persona;
   if (groupCoordinator && typeof groupCoordinator.ingest === 'function') {
     try {
       out.router = createRouter({ coordinator: groupCoordinator });