@yeaft/webchat-agent 0.1.689 → 0.1.691

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, getSearchSettings, updateSearchSettings, fetchTavilyUsage } from '../unify/config-api.js';
-import { handleUnifyGroupChat, handleUnifyModeSwitch, handleUnifyModelSwitch, resetUnifySession, handleUnifyLoadHistory, handleUnifyAbortThread, handleUnifyAbortAll, handleUnifyAbortTurn, handleUnifyVpSubscribe, handleUnifyVpCreate, handleUnifyVpUpdate, handleUnifyVpDelete, handleUnifyVpRead, handleUnifyFeatureMessage, handleUnifyFetchSummaryHistory, handleUnifyFeatureCrud, handleUnifyListGroups, handleUnifyCreateGroup, handleUnifyRenameGroup, handleUnifyArchiveGroup, handleUnifyDeleteGroup, handleUnifyAddMember, handleUnifyRemoveMember, handleUnifySetDefaultVp, handleUnifyDreamTrigger } from '../unify/web-bridge.js';
+import { handleUnifyGroupChat, handleUnifyModeSwitch, handleUnifyModelSwitch, resetUnifySession, handleUnifyLoadHistory, handleUnifyAbortThread, handleUnifyAbortAll, handleUnifyAbortTurn, handleUnifyVpSubscribe, handleUnifyVpCreate, handleUnifyVpUpdate, handleUnifyVpDelete, handleUnifyVpRead, handleUnifyFeatureMessage, handleUnifyFetchSummaryHistory, handleUnifyFeatureCrud, handleUnifyListGroups, handleUnifyCreateGroup, handleUnifyRenameGroup, handleUnifyUpdateGroup, handleUnifyArchiveGroup, handleUnifyDeleteGroup, handleUnifyAddMember, handleUnifyRemoveMember, handleUnifySetDefaultVp, handleUnifyDreamTrigger } from '../unify/web-bridge.js';
 
 export async function handleMessage(msg) {
   switch (msg.type) {
@@ -468,6 +468,9 @@ export async function handleMessage(msg) {
     case 'unify_rename_group':
       handleUnifyRenameGroup(msg);
       break;
+    case 'unify_update_group':
+      handleUnifyUpdateGroup(msg);
+      break;
     case 'unify_archive_group':
       handleUnifyArchiveGroup(msg);
       break;

package/package.json

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

package/unify/engine.js

@@ -563,7 +563,7 @@ export class Engine {
    * @param {{user?:string, group?:string, vp?:string}} [summaries]
    * @returns {string}
    */
-  #buildSystemPrompt(memory, compactSummary, prompt, memoryInjection, userProfile, vpPersona, summaries) {
+  #buildSystemPrompt(memory, compactSummary, prompt, memoryInjection, userProfile, vpPersona, summaries, groupAnnouncement) {
     // Get relevant skill content if SkillManager is wired
     let skillContent = '';
     if (this.#skillManager && prompt) {
@@ -585,6 +585,7 @@ export class Engine {
       userProfile,
       vpPersona,
       summaries,
+      groupAnnouncement,
       // Worker-shape harness is descriptive metadata for human inspection;
       // production prompts skip it to save tokens. Re-enable via env when
       // diagnosing prompt structure issues.
@@ -859,7 +860,7 @@ export class Engine {
    *   SCENARIO_EFFORT. Unknown values fall through to 'high'.
    * @yields {EngineEvent}
    */
-  async *query({ prompt, messages = [], signal, userEffort = null, scenario = 'chat', vpPersona, router, senderVpId, inboundEnvelope, taskId, taskMembers, groupId, vpPlan } = {}) {
+  async *query({ prompt, messages = [], signal, userEffort = null, scenario = 'chat', vpPersona, router, senderVpId, inboundEnvelope, taskId, taskMembers, groupId, vpPlan, groupAnnouncement } = {}) {
     if (!prompt || typeof prompt !== 'string' || !prompt.trim()) {
       yield {
         type: 'error',
@@ -910,7 +911,7 @@ export class Engine {
     const runSignal = abortCtrl.signal;
 
     try {
-      yield* this.#runQuery({ prompt: effectivePrompt, messages, signal: runSignal, userEffort: effectiveUserEffort, scenario, vpPersona, router, senderVpId, inboundEnvelope, taskId, taskMembers, groupId, vpPlan });
+      yield* this.#runQuery({ prompt: effectivePrompt, messages, signal: runSignal, userEffort: effectiveUserEffort, scenario, vpPersona, router, senderVpId, inboundEnvelope, taskId, taskMembers, groupId, vpPlan, groupAnnouncement });
     } finally {
       if (signal) {
         try { signal.removeEventListener('abort', onExternalAbort); } catch { /* ignore */ }
@@ -928,7 +929,7 @@ export class Engine {
    * in a try/finally without indenting the whole loop.
    * @private
    */
-  async *#runQuery({ prompt, messages, signal, userEffort = null, scenario = 'chat', vpPersona, router, senderVpId, inboundEnvelope, taskId, taskMembers, groupId, vpPlan }) {
+  async *#runQuery({ prompt, messages, signal, userEffort = null, scenario = 'chat', vpPersona, router, senderVpId, inboundEnvelope, taskId, taskMembers, groupId, vpPlan, groupAnnouncement }) {
 
     // ─── Pre-query: FTS5 Memory Recall + Compact Summary ──
     // Memory feed comes from two places:
@@ -1004,7 +1005,7 @@ export class Engine {
         : amsContext.snapshotBlock;
     }
 
-    const systemPrompt = this.#buildSystemPrompt(undefined, compactSummary, prompt, memoryInjection, userProfile, vpPersona, summaries);
+    const systemPrompt = this.#buildSystemPrompt(undefined, compactSummary, prompt, memoryInjection, userProfile, vpPersona, summaries, groupAnnouncement);
 
     // Build conversation: existing messages + new user message
     const conversationMessages = [

package/unify/groups/group-crud.js

@@ -162,6 +162,26 @@ export function renameGroup(yeaftDir, groupId, newName) {
   return next;
 }
 
+/**
+ * (A.2.b) Update announcement — group-wide system-prompt prefix shared by
+ * every VP in the group (CLAUDE.md-style). Empty/whitespace clears it.
+ *
+ * `text` must be a string. Trimmed before persist so leading/trailing
+ * whitespace doesn't pollute the prompt.
+ */
+export function updateGroupAnnouncement(yeaftDir, groupId, text) {
+  if (typeof text !== 'string') {
+    throw new GroupCrudError('invalid_announcement', groupId);
+  }
+  const announcement = text.trim();
+  const handle = requireGroup(yeaftDir, groupId);
+  const meta = handle.getMeta();
+  handle.saveMeta({ ...meta, announcement });
+  const next = handle.getMeta();
+  handle.close();
+  return next;
+}
+
 /**
  * (A.3) Archive — renames the dir to `.archived-<ts>-<id>`. Directory
  * prefix `.` keeps `listGroups` from picking it up (readdirSync filter in

package/unify/groups/group-store.js

@@ -126,6 +126,7 @@ export function createGroup(groupsRoot, spec) {
     name: spec.name || spec.id,
     roster,
     defaultVpId: spec.defaultVpId || null,
+    announcement: typeof spec.announcement === 'string' ? spec.announcement : '',
     createdAt: spec.createdAt || new Date().toISOString(),
   };
   h.saveMeta(meta);
@@ -140,6 +141,9 @@ export function loadGroupMeta(dir) {
     const raw = readFileSync(path, 'utf8');
     const parsed = JSON.parse(raw);
     validateMeta(parsed);
+    // Legacy groups created before the announcement field was added are
+    // forward-compat: missing field reads back as empty string.
+    if (typeof parsed.announcement !== 'string') parsed.announcement = '';
     return parsed;
   } catch {
     return null;
@@ -173,6 +177,9 @@ function validateMeta(meta) {
   if (meta.defaultVpId != null && typeof meta.defaultVpId !== 'string') {
     throw new Error('group.defaultVpId must be string|null');
   }
+  if (meta.announcement != null && typeof meta.announcement !== 'string') {
+    throw new Error('group.announcement must be string');
+  }
 }
 
 /**

package/unify/prompts.js

@@ -283,6 +283,7 @@ export function buildSystemPrompt({
   coreMemory,
   memoryTraceAvailable = false,
   vpPersona,
+  groupAnnouncement = '',
 } = {}) {
   // Fallback to English for unknown languages
   const lang = PROMPTS[language] || PROMPTS.en;
@@ -310,6 +311,15 @@ export function buildSystemPrompt({
     }
   }
 
+  // ─── 1.5  Group Announcement (CLAUDE.md-style shared prefix) ───
+  // When a group has set an announcement, every VP in the group sees it
+  // near the top of the system prompt — before tools, memory, mode-specific
+  // instructions. Empty/whitespace = no block emitted.
+  const annText = (typeof groupAnnouncement === 'string') ? groupAnnouncement.trim() : '';
+  if (annText) {
+    parts.push(`[Group Announcement]\n${annText}`);
+  }
+
   // ─── 2. Date Metadata ──────────────────────────────────
   parts.push(lang.date(new Date().toISOString().split('T')[0]));
 

package/unify/web-bridge.js

@@ -33,6 +33,7 @@ import {
   GroupCrudError,
   createGroupFromSpec,
   renameGroup,
+  updateGroupAnnouncement,
   archiveGroup,
   deleteGroup,
   purgeArchivedGroups,
@@ -318,6 +319,45 @@ export function handleUnifyRenameGroup(msg) {
   }
 }
 
+/**
+ * `unify_update_group` — generalised group meta patch. Currently accepts
+ * `name` and `announcement` keys. Empty patch is rejected; an empty/
+ * whitespace-only `name` is also rejected up front rather than letting
+ * `renameGroup` raise a less-specific error deeper in the call stack.
+ *
+ * Partial-success contract: when a single patch contains BOTH `name` and
+ * `announcement`, the rename is committed first; if the announcement
+ * write throws, the rename has already persisted on disk and the client
+ * receives `ok:false` for the announcement error — i.e. the WS op is not
+ * atomic. Today's UI binds Save buttons per pane in `GroupSettingsModal`
+ * so this is theoretical; readers extending the patch shape should know
+ * the contract permits half-commits.
+ */
+export function handleUnifyUpdateGroup(msg) {
+  const requestId = msg && msg.requestId;
+  const groupId = msg && msg.groupId;
+  const patch = (msg && msg.patch && typeof msg.patch === 'object') ? msg.patch : null;
+  try {
+    const hasName = patch && typeof patch.name === 'string' && patch.name.trim().length > 0;
+    const hasAnnouncement = patch && typeof patch.announcement === 'string';
+    if (!patch || (!hasName && !hasAnnouncement)) {
+      throw new GroupCrudError('invalid_patch', groupId);
+    }
+    const yeaftDir = ctx.CONFIG?.yeaftDir;
+    let group = null;
+    if (hasName) {
+      group = renameGroup(yeaftDir, groupId, patch.name);
+    }
+    if (hasAnnouncement) {
+      group = updateGroupAnnouncement(yeaftDir, groupId, patch.announcement);
+    }
+    sendGroupCrudResult({ op: 'update', requestId, ok: true, group });
+    sendGroupSnapshotBroadcast();
+  } catch (err) {
+    sendGroupCrudResult({ op: 'update', requestId, ok: false, error: groupErrorPayload(err) });
+  }
+}
+
 export function handleUnifyArchiveGroup(msg) {
   const requestId = msg && msg.requestId;
   const groupId = msg && msg.groupId;
@@ -912,16 +952,24 @@ export async function handleUnifyGroupChat(msg) {
  * Build the per-query VP context for the Engine.
  */
 export function buildVpQueryOpts({ vpId, groupCoordinator, groupId }) {
+  // Read the group meta once and reuse for both defaultVpId fallback and
+  // announcement injection. Each .getMeta() reload reads + parses the
+  // group.json file, so calling it twice per turn is wasteful — and
+  // (more importantly) opens a window where a concurrent group edit
+  // could land between the two reads, giving the engine a defaultVpId
+  // from one snapshot and an announcement from a newer one.
+  let groupMeta = null;
+  try {
+    groupMeta = groupCoordinator && groupCoordinator.group
+      && typeof groupCoordinator.group.getMeta === 'function'
+      ? groupCoordinator.group.getMeta() : null;
+  } catch { /* coordinator inspection is best-effort */ }
+
   let resolvedVpId = vpId;
   if (!resolvedVpId) {
-    try {
-      const meta = groupCoordinator && groupCoordinator.group
-        && typeof groupCoordinator.group.getMeta === 'function'
-        ? groupCoordinator.group.getMeta() : null;
-      if (meta && typeof meta.defaultVpId === 'string' && meta.defaultVpId) {
-        resolvedVpId = meta.defaultVpId;
-      }
-    } catch { /* coordinator inspection is best-effort */ }
+    if (groupMeta && typeof groupMeta.defaultVpId === 'string' && groupMeta.defaultVpId) {
+      resolvedVpId = groupMeta.defaultVpId;
+    }
   }
   if (!resolvedVpId) {
     const cfgDefault = session?.config?.defaultVpId;
@@ -943,6 +991,12 @@ export function buildVpQueryOpts({ vpId, groupCoordinator, groupId }) {
   if (typeof groupId === 'string' && groupId.trim()) {
     out.groupId = groupId.trim();
   }
+  // task-334-group-editor: surface the group announcement to the engine so
+  // buildWorkerPrompt can inject it as a CLAUDE.md-style shared prefix.
+  // Empty/missing reads as '' and prompts.js skips the section.
+  if (groupMeta && typeof groupMeta.announcement === 'string') {
+    out.groupAnnouncement = groupMeta.announcement;
+  }
   try {
     const vp = readVp(resolvedVpId);
     if (vp) {