@yeaft/webchat-agent 1.0.177 → 1.0.180

package/yeaft/vp/vp-crud.js

@@ -97,7 +97,7 @@ function vpRolePathFor(libDir, vpId) {
  * Serialise a VP payload into role.md text with YAML frontmatter matching
  * the parser in vp-store.js.
  *
- * @param {{vpId:string, displayName?:string, role?:string, traits?:string[], modelHint?:string, persona?:string}} p
+ * @param {{vpId:string, displayName?:string, displayNameZh?:string, aliases?:string[], description?:string, descriptionZh?:string, role?:string, roleZh?:string, area?:string, traits?:string[], modelHint?:string, persona?:string, planInstruction?:string}} p
  * @returns {string}
  */
 export function buildRoleMd(p) {
@@ -105,6 +105,8 @@ export function buildRoleMd(p) {
   const name = p.displayName != null ? String(p.displayName) : id;
   const nameZh = p.displayNameZh != null ? String(p.displayNameZh) : '';
   const aliases = Array.isArray(p.aliases) ? p.aliases.map(a => String(a)).filter(Boolean) : [];
+  const description = p.description != null ? String(p.description) : '';
+  const descriptionZh = p.descriptionZh != null ? String(p.descriptionZh) : '';
   const role = p.role != null ? String(p.role) : '';
   const roleZh = p.roleZh != null ? String(p.roleZh) : '';
   // Taxonomy bucket. Optional + additive — written only when present so
@@ -116,10 +118,15 @@ export function buildRoleMd(p) {
 
   const lines = ['---', `id: ${id}`, `name: ${yamlScalar(name)}`];
   if (nameZh) lines.push(`nameZh: ${yamlScalar(nameZh)}`);
+  if (description) lines.push(`description: ${yamlScalar(description)}`);
+  if (descriptionZh) lines.push(`descriptionZh: ${yamlScalar(descriptionZh)}`);
   lines.push(`role: ${yamlScalar(role)}`);
   if (roleZh) lines.push(`roleZh: ${yamlScalar(roleZh)}`);
   if (area) lines.push(`area: ${yamlScalar(area)}`);
   if (modelHint) lines.push(`modelHint: ${modelHint}`);
+  if (p.planInstruction != null && String(p.planInstruction)) {
+    lines.push(`planInstruction: ${yamlScalar(String(p.planInstruction))}`);
+  }
   if (traits.length > 0) {
     lines.push('traits:');
     for (const t of traits) lines.push(`  - ${yamlScalar(t)}`);
@@ -134,21 +141,58 @@ export function buildRoleMd(p) {
 
 function yamlScalar(v) {
   const s = String(v);
-  // Quote anything that the minimal parser might mis-read: colons, leading
-  // dashes, or surrounding whitespace. Plain text passes through unquoted.
-  if (/^[\s]|[\s]$|^[-:]|[:#]/.test(s)) {
-    return `"${s.replace(/"/g, '\\"')}"`;
+  // YAML single-quoted scalars keep backslashes and double quotes literal.
+  // Doubling an apostrophe is the only escape, and parseYamlScalar reverses
+  // exactly that operation. This also avoids interpreting legacy Windows
+  // paths such as `C:\temp` as JSON/YAML control escapes.
+  if (/^[\s]|[\s]$|^[-:]|[:#'"\\\n\r\t]/.test(s)) {
+    return `'${s.replace(/'/g, "''")}'`;
   }
   return s;
 }
 
+function hasOwn(obj, key) {
+  return Object.prototype.hasOwnProperty.call(obj, key);
+}
+
+/**
+ * Merge an update payload with the persisted editable VP shape. Missing keys
+ * mean "old client does not know this field" and must be preserved; an
+ * explicitly supplied empty value remains a deliberate clear operation.
+ */
+function mergeVpUpdate(current, payload, vpId) {
+  const fields = [
+    'displayName',
+    'displayNameZh',
+    'aliases',
+    'description',
+    'descriptionZh',
+    'role',
+    'roleZh',
+    'area',
+    'traits',
+    'modelHint',
+    'persona',
+    'planInstruction',
+  ];
+  const merged = { vpId };
+  for (const field of fields) {
+    merged[field] = hasOwn(payload, field) ? payload[field] : current[field];
+  }
+  return merged;
+}
+
 /**
  * Create a new VP. Writes `<lib>/<vpId>/role.md` and ensures `memory/` dir.
  *
  * @param {object} payload
  * @param {string} payload.vpId
  * @param {string} [payload.displayName]
+ * @param {string} [payload.displayNameZh]
+ * @param {string} [payload.description]
+ * @param {string} [payload.descriptionZh]
  * @param {string} [payload.role]
+ * @param {string} [payload.roleZh]
  * @param {string[]} [payload.traits]
  * @param {'primary'|'fast'} [payload.modelHint]
  * @param {string} [payload.persona]
@@ -208,7 +252,10 @@ export function updateVp(payload, options = {}) {
   if (!existsSync(dir) || !existsSync(vpRolePathFor(libDir, vpId))) {
     throw new VpCrudError('not_found', vpId);
   }
-  writeFileSync(vpRolePathFor(libDir, vpId), buildRoleMd({ ...payload, vpId }), 'utf-8');
+  const current = readVp(vpId, { libDir });
+  if (!current) throw new VpCrudError('not_found', vpId);
+  const merged = mergeVpUpdate(current, payload || {}, vpId);
+  writeFileSync(vpRolePathFor(libDir, vpId), buildRoleMd(merged), 'utf-8');
   return { vpId, dir };
 }
 
@@ -258,7 +305,7 @@ export function deleteVp(vpId, options = {}) {
  *
  * @param {string} vpId
  * @param {object} [options]
- * @returns {?{vpId:string, displayName:string, role:string, traits:string[], modelHint:?string, persona:string}}
+ * @returns {?{vpId:string, displayName:string, displayNameZh:string, aliases:string[], description:string, descriptionZh:string, role:string, roleZh:string, area:string, traits:string[], modelHint:?string, persona:string, planInstruction:string}}
  */
 export function readVp(vpId, options = {}) {
   const libDir = options.libDir || DEFAULT_VP_LIB_DIR;
@@ -279,8 +326,11 @@ export function readVp(vpId, options = {}) {
     displayName: String(meta.name || id),
     displayNameZh: typeof meta.nameZh === 'string' ? String(meta.nameZh) : '',
     aliases: Array.isArray(meta.aliases) ? meta.aliases.map(String) : [],
+    description: typeof meta.description === 'string' ? String(meta.description) : '',
+    descriptionZh: typeof meta.descriptionZh === 'string' ? String(meta.descriptionZh) : '',
     role: String(meta.role || ''),
     roleZh: typeof meta.roleZh === 'string' ? String(meta.roleZh) : '',
+    area: typeof meta.area === 'string' ? String(meta.area).trim() : '',
     traits: Array.isArray(meta.traits) ? meta.traits.map(String) : [],
     modelHint,
     persona: body,

package/yeaft/vp/vp-store.js

@@ -29,7 +29,11 @@ import { createHash } from 'crypto';
  * @typedef {Object} VP
  * @property {string} id              — VP id (default: dir name)
  * @property {string} name
+ * @property {string} nameZh          — optional Chinese display name; '' if absent.
+ * @property {string} description     — concise English capability summary for roster lists.
+ * @property {string} descriptionZh   — optional Chinese capability summary; '' if absent.
  * @property {string} role
+ * @property {string} roleZh          — optional Chinese role label; '' if absent.
  * @property {string} area            — taxonomy bucket (e.g. 'philosophy', 'investing'); '' if absent.
  *                                       Optional, additive: no consumer is required to dispatch on it.
  *                                       Sidebar grouping by area is intentionally a future PR.
@@ -70,6 +74,21 @@ export function personaHash(persona) {
  * @param {string} source
  * @returns {{ meta: Record<string, any>, body: string }}
  */
+function parseYamlScalar(raw) {
+  const value = String(raw || '').trim();
+  if (value.length >= 2 && value.startsWith('"') && value.endsWith('"')) {
+    // Compatibility for files written by the old CRUD serializer. It escaped
+    // only double quotes, not backslashes, so JSON.parse would corrupt valid
+    // Windows paths (`C:\temp` -> a tab). Undo only the escape the old writer
+    // actually introduced and leave every other backslash literal.
+    return value.slice(1, -1).replace(/\\"/g, '"');
+  }
+  if (value.length >= 2 && value.startsWith("'") && value.endsWith("'")) {
+    return value.slice(1, -1).replace(/''/g, "'");
+  }
+  return value;
+}
+
 export function parseRoleMd(source) {
   const match = source.match(/^---\r?\n([\s\S]*?)\r?\n---\r?\n?([\s\S]*)$/);
   if (!match) return { meta: {}, body: source };
@@ -84,7 +103,7 @@ export function parseRoleMd(source) {
     if (!line.trim()) continue;
     const listMatch = line.match(/^\s+-\s+(.+?)\s*$/);
     if (listMatch && currentList) {
-      currentList.push(listMatch[1].replace(/^['"]|['"]$/g, ''));
+      currentList.push(parseYamlScalar(listMatch[1]));
       continue;
     }
     const kvMatch = line.match(/^([\w-]+):\s*(.*)$/);
@@ -95,7 +114,7 @@ export function parseRoleMd(source) {
         currentList = [];
         meta[key] = currentList;
       } else {
-        meta[key] = value.replace(/^['"]|['"]$/g, '');
+        meta[key] = parseYamlScalar(value);
         currentList = null;
       }
     }
@@ -156,6 +175,8 @@ export function loadVpFromDir(dir) {
     aliases: Array.isArray(meta.aliases)
       ? meta.aliases.map(String).map(s => s.trim()).filter(Boolean)
       : [],
+    description: typeof meta.description === 'string' ? String(meta.description) : '',
+    descriptionZh: typeof meta.descriptionZh === 'string' ? String(meta.descriptionZh) : '',
     role: String(meta.role || ''),
     roleZh: typeof meta.roleZh === 'string' ? String(meta.roleZh) : '',
     // Taxonomy bucket for sidebar grouping / filtering. Absent for legacy

package/yeaft/web-bridge.js

@@ -63,6 +63,7 @@ import {
 import { persistYeaftAttachments, attachmentsForPersistence, persistedAttachmentPreviewPayload } from './attachments.js';
 import { ConversationStore, parseSeqFromId } from './conversation/persist.js';
 import { isHiddenConversationRow } from './conversation/internal-control.js';
+import { imageMetadataForPersistence } from './image-assets.js';
 import { sliceLastNTurns } from './turn-utils.js';
 import { pairSanitize } from './pair-sanitize.js';
 import { filterSnapshotForVp } from './snapshot-filter.js';
@@ -999,6 +1000,7 @@ function projectPersistedToHistoryEntry(m) {
   if (m.id) entry.id = m.id;
   entry.threadId = m.threadId || m.turnId || 'main';
   if (m.turnId) entry.turnId = m.turnId;
+  if (m.imageAssetAnchor) entry.imageAssetAnchor = true;
   if (m.sessionId) entry.sessionId = m.sessionId;
   if (m.clientMessageId) entry.clientMessageId = m.clientMessageId;
   if (m.speakerVpId) entry.speakerVpId = m.speakerVpId;
@@ -1016,8 +1018,9 @@ function projectPersistedToHistoryEntry(m) {
   if (m.isError) entry.isError = true;
   if (m.ts) entry.ts = m.ts;
   else if (m.time) entry.ts = m.time;
+  if (Array.isArray(m.images) && m.images.length > 0) entry.images = m.images;
   if (Array.isArray(m.attachments) && m.attachments.length > 0) entry.attachments = m.attachments;
-  if ((entry.role === 'user' || entry.role === 'assistant') && !entry.content && !entry.attachments && !entry.toolCalls && !entry.toolSummaryCount) return null;
+  if ((entry.role === 'user' || entry.role === 'assistant') && !entry.content && !entry.attachments && !entry.images && !entry.toolCalls && !entry.toolSummaryCount) return null;
   return entry;
 }
 
@@ -1106,7 +1109,9 @@ function projectVisibleHistoryChunkMessages(messages = []) {
       ...(m.clientMessageId ? { clientMessageId: m.clientMessageId } : {}),
       threadId: m.threadId || m.turnId || 'main',
       ...(m.turnId ? { turnId: m.turnId } : {}),
+      ...(m.imageAssetAnchor === true ? { imageAssetAnchor: true } : {}),
       ...(Array.isArray(m.attachments) && m.attachments.length > 0 ? { attachments: hydrateHistoryAttachmentPreviews(m.attachments) } : {}),
+      ...(Array.isArray(m.images) && m.images.length > 0 ? { images: m.images } : {}),
       ...(m.speakerVpId ? { speakerVpId: m.speakerVpId } : {}),
       ...(Number.isFinite(m.toolSummaryCount) && m.toolSummaryCount > 0
         ? { toolSummaryCount: m.toolSummaryCount }
@@ -1156,7 +1161,13 @@ function emitLegacyHistoryOutputFrames(replayEntries) {
       if (entry.speakerVpId) envelopeOpts.vpId = entry.speakerVpId;
       sendSessionOutputFrame({
         type: 'assistant',
-        message: { id: entry.id || null, content: [{ type: 'text', text: entry.content }] },
+        message: {
+          id: entry.id || null,
+          content: [
+            ...(entry.content ? [{ type: 'text', text: entry.content }] : []),
+            ...((entry.images || []).map(image => ({ type: 'image_asset', image }))),
+          ],
+        },
         ts: entry.ts || null,
       }, envelopeOpts);
       if (Array.isArray(entry.toolCalls) && entry.toolCalls.length > 0) {
@@ -2851,6 +2862,7 @@ export function handleYeaftDeleteSession(msg) {
   try {
     const yeaftDir = ctx.CONFIG?.yeaftDir;
     const result = deleteSession(yeaftDir, sessionId);
+    ctx.assetOutbox?.removeSession(sessionId);
     // Cascade: remove every persisted message stamped with this group id.
     // Hard delete (per user spec): no soft-archive, the bytes are gone.
     // Skipped silently if the session/store isn't initialized — the next
@@ -3299,12 +3311,34 @@ function handleEngineEvent(event, hctx) {
       }, envelope);
       break;
 
-    case 'tool_end':
+    case 'tool_end': {
+      const images = Array.isArray(event.displayImages) ? event.displayImages : [];
+      const displayOutput = typeof event.output === 'string' ? event.output : JSON.stringify(event.output ?? '');
+      for (const image of images) {
+        const persistedImage = imageMetadataForPersistence(image);
+        try {
+          if (!ctx.assetOutbox) throw new Error('asset outbox is unavailable');
+          const deliveryId = ctx.assetOutbox.enqueue({
+            conversationId: yeaftConversationId,
+            metadata: persistedImage,
+            sessionId: hctx.sessionId,
+            vpId: hctx.vpId,
+            turnId: hctx.turnId,
+            threadId: hctx.threadId || event.threadId,
+            image,
+          });
+          if (!deliveryId) throw new Error('asset outbox did not persist the image');
+          image.deliveryQueued = true;
+          ctx.assetOutbox.drain().catch(err => console.warn('[AssetOutbox] drain failed:', err?.message || err));
+        } catch (err) {
+          console.warn('[AssetOutbox] failed to queue image:', err?.message || err);
+        }
+      }
       if (hctx.toolResultsAccum) {
         hctx.toolResultsAccum.push({
           role: 'tool',
           toolCallId: event.id,
-          content: typeof event.output === 'string' ? event.output : JSON.stringify(event.output ?? ''),
+          content: displayOutput,
           isError: !!event.isError,
         });
       }
@@ -3330,7 +3364,7 @@ function handleEngineEvent(event, hctx) {
         tool_use_result: [{
           type: 'tool_result',
           tool_use_id: event.id,
-          content: event.output || '',
+          content: displayOutput,
           is_error: event.isError || false,
         }],
       }, envelope);
@@ -3341,6 +3375,7 @@ function handleEngineEvent(event, hctx) {
       // streaming' flicker on every tool call. Hold the 'tool' state
       // until the next real event arrives.
       break;
+    }
 
     case 'tool_result_update': {
       const content = typeof event.content === 'string'
@@ -6613,6 +6648,7 @@ export async function handleYeaftMcpReload(msg = {}) {
 
 export const __testHooks = {
   loadVisibleGroupHistoryPage,
+  projectVisibleHistoryChunkMessages,
   persistInboundMessageOnceByMsgId,
   buildPendingRescueEnvelope,
   runYeaftSessionSendForTest(msg) {

package/yeaft/work-center/bridge.js

@@ -66,6 +66,8 @@ async function getSettingsRuntime() {
       id: vp.id,
       name: vp.name || vp.id,
       nameZh: vp.nameZh || '',
+      description: vp.description || vp.role || '',
+      descriptionZh: vp.descriptionZh || vp.roleZh || vp.description || vp.role || '',
       role: vp.role || '',
       roleZh: vp.roleZh || '',
       area: vp.area || '',

package/yeaft/work-center/planner.js

@@ -6,6 +6,8 @@ function publicVp(vp) {
     id: vp.id,
     name: vp.name || vp.id,
     nameZh: vp.nameZh || '',
+    description: vp.description || vp.role || '',
+    descriptionZh: vp.descriptionZh || vp.roleZh || vp.description || vp.role || '',
     role: vp.role || '',
     roleZh: vp.roleZh || '',
     area: vp.area || '',