@yeaft/webchat-agent 0.1.607 → 0.1.609

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, handleUnifyGroupChat, handleUnifyModeSwitch, handleUnifyModelSwitch, resetUnifySession, handleUnifyLoadHistory, handleUnifyMergeThread, handleUnifyForkThread, handleUnifyAbortThread, handleUnifyAbortAll, handleUnifyVpSubscribe, handleUnifyVpCreate, handleUnifyVpUpdate, handleUnifyVpDelete, handleUnifyVpRead, handleUnifyTaskMessage, handleUnifyUserMemoryWrite, handleUnifyUserMemoryRemove, handleUnifyMemoryScopeList, handleUnifyMemoryQuery, handleUnifyMemoryTrace, handleUnifyFetchSummaryHistory, handleUnifyTaskCrud, handleUnifyListGroups, handleUnifyCreateGroup, handleUnifyRenameGroup, handleUnifyArchiveGroup, handleUnifyAddMember, handleUnifyRemoveMember, handleUnifySetDefaultVp, handleUnifyDreamTrigger } from '../unify/web-bridge.js';
+import { handleUnifyChat, handleUnifyGroupChat, handleUnifyModeSwitch, handleUnifyModelSwitch, resetUnifySession, handleUnifyLoadHistory, handleUnifyMergeThread, handleUnifyForkThread, handleUnifyAbortThread, handleUnifyAbortAll, handleUnifyVpSubscribe, handleUnifyVpCreate, handleUnifyVpUpdate, handleUnifyVpDelete, handleUnifyVpRead, handleUnifyTaskMessage, handleUnifyUserMemoryWrite, handleUnifyUserMemoryRemove, handleUnifyMemoryScopeList, handleUnifyMemoryQuery, handleUnifyMemoryTrace, handleUnifyFetchSummaryHistory, handleUnifyTaskCrud, handleUnifyListGroups, handleUnifyCreateGroup, handleUnifyRenameGroup, handleUnifyArchiveGroup, handleUnifyDeleteGroup, handleUnifyAddMember, handleUnifyRemoveMember, handleUnifySetDefaultVp, handleUnifyDreamTrigger } from '../unify/web-bridge.js';
 
 export async function handleMessage(msg) {
   switch (msg.type) {
@@ -479,6 +479,9 @@ export async function handleMessage(msg) {
     case 'unify_archive_group':
       handleUnifyArchiveGroup(msg);
       break;
+    case 'unify_delete_group':
+      handleUnifyDeleteGroup(msg);
+      break;
     case 'unify_add_member':
       handleUnifyAddMember(msg);
       break;

package/package.json

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

package/unify/conversation/persist.js

@@ -64,6 +64,11 @@ function serializeMessage(msg) {
   // their original thread id in `sourceThreadId` so the UI can still
   // render a small "#source" pill next to each bubble.
   if (msg.sourceThreadId) fm.push(`sourceThreadId: ${msg.sourceThreadId}`);
+  // Bug 6: persist groupId so history replay can stamp messages with the
+  // group they originated in. Without this, every replayed message lands
+  // in the default group and switching back to the originating group
+  // shows an empty pane.
+  if (msg.groupId) fm.push(`groupId: ${msg.groupId}`);
 
   // Token estimate
   const content = msg.content || '';
@@ -134,6 +139,7 @@ export function parseMessage(raw) {
       case 'tokens_est': msg.tokens_est = parseInt(value, 10); break;
       case 'threadId': msg.threadId = value; break;
       case 'sourceThreadId': msg.sourceThreadId = value; break;
+      case 'groupId': msg.groupId = value; break;
       // toolCalls are multi-line YAML — handled separately below
     }
   }

package/unify/engine.js

@@ -476,7 +476,7 @@ export class Engine {
    * @param {string} assistantContent
    * @param {object[]} [toolCalls]
    */
-  #persistMessages(userContent, assistantContent, toolCalls) {
+  #persistMessages(userContent, assistantContent, toolCalls, groupId) {
     if (!this.#conversationStore) return;
     if (this.#config._readOnly) return;
 
@@ -497,6 +497,8 @@ export class Engine {
       role: 'user',
       content: userContent,
       threadId,
+      // Bug 6: stamp groupId so history replay can route by group.
+      ...(groupId ? { groupId } : {}),
     });
 
     // Persist assistant message
@@ -505,6 +507,7 @@ export class Engine {
       content: assistantContent,
       model: this.#config.model,
       threadId,
+      ...(groupId ? { groupId } : {}),
     };
     if (toolCalls && toolCalls.length > 0) {
       assistantMsg.toolCalls = toolCalls;
@@ -1172,6 +1175,9 @@ export class Engine {
             primaryModel: this.#config.model,
             messages: conversationMessages,
             trace: this.#trace,
+            // Bug 6: tag persisted messages with the originating group so
+            // history replay can re-stamp them on reload.
+            groupId,
           });
 
           if (hookResult.consolidated) {
@@ -1182,7 +1188,7 @@ export class Engine {
           }
         } else {
           // Legacy path (no yeaftDir → use old behavior)
-          this.#persistMessages(prompt, fullResponseText, assistantMsg.toolCalls);
+          this.#persistMessages(prompt, fullResponseText, assistantMsg.toolCalls, groupId);
 
           const consolidated = await this.#maybeConsolidate();
           if (consolidated && consolidated.archivedCount > 0) {

package/unify/groups/group-crud.js

@@ -32,7 +32,7 @@
  *   'reserved'/'invalid_vp_id'/... — bubbled from ids.js validators
  */
 
-import { existsSync, renameSync } from 'fs';
+import { existsSync, renameSync, rmSync, readdirSync, statSync } from 'fs';
 import { randomBytes } from 'crypto';
 import { join } from 'path';
 import {
@@ -184,6 +184,74 @@ export function archiveGroup(yeaftDir, groupId) {
   return { groupId, archivedAs: dstDir };
 }
 
+/**
+ * (A.3.b) Delete — physically remove the group directory and all its
+ * contents (group.json, messages/, tasks/, vps/). Irreversible.
+ *
+ * Bug 8 fix: replaces the soft-archive flow that left `.archived-*` dirs
+ * lying around in `~/.yeaft/groups/`. Per user request, "delete" means
+ * physical deletion, not rename.
+ *
+ * Also sweeps any sibling `.archived-*-<groupId>` dirs that were left
+ * behind by the previous soft-archive implementation, so a single
+ * delete cleans up legacy state too.
+ */
+export function deleteGroup(yeaftDir, groupId) {
+  const root = groupsRoot(yeaftDir);
+  const srcDir = join(root, groupId);
+  const liveExists = existsSync(srcDir) && !!loadGroupMeta(srcDir);
+
+  // Collect any leftover soft-archive directories matching this groupId.
+  const legacyDirs = [];
+  if (existsSync(root)) {
+    for (const name of readdirSync(root)) {
+      if (!name.startsWith('.archived-')) continue;
+      // Soft-archive format: .archived-<ts>-<suffix>-<groupId>
+      if (!name.endsWith(`-${groupId}`)) continue;
+      const p = join(root, name);
+      try {
+        if (statSync(p).isDirectory()) legacyDirs.push(p);
+      } catch { /* skip */ }
+    }
+  }
+
+  if (!liveExists && legacyDirs.length === 0) {
+    throw new GroupCrudError('not_found', groupId);
+  }
+
+  if (liveExists) {
+    rmSync(srcDir, { recursive: true, force: true });
+  }
+  for (const dir of legacyDirs) {
+    rmSync(dir, { recursive: true, force: true });
+  }
+
+  return { groupId, deleted: true, legacyCleanedUp: legacyDirs.length };
+}
+
+/**
+ * Sweep any leftover `.archived-*` directories under groups/ that are
+ * orphans of the old soft-archive flow. Used at boot so users don't see
+ * ghost groups in subsequent loads. Returns the list of removed paths.
+ */
+export function purgeArchivedGroups(yeaftDir) {
+  const root = groupsRoot(yeaftDir);
+  if (!existsSync(root)) return [];
+  const removed = [];
+  for (const name of readdirSync(root)) {
+    if (!name.startsWith('.archived-')) continue;
+    const p = join(root, name);
+    try {
+      if (!statSync(p).isDirectory()) continue;
+    } catch { continue; }
+    try {
+      rmSync(p, { recursive: true, force: true });
+      removed.push(p);
+    } catch { /* skip */ }
+  }
+  return removed;
+}
+
 /**
  * (A.4) Add a VP to the group roster. Idempotent — no-op if already present.
  * Returns the new meta.

package/unify/groups/group-store.js

@@ -151,6 +151,8 @@ export function listGroups(groupsRoot) {
   if (!existsSync(groupsRoot)) return [];
   const out = [];
   for (const name of readdirSync(groupsRoot)) {
+    // Skip dotfiles and legacy soft-archive dirs (`.archived-*`).
+    if (name.startsWith('.')) continue;
     const p = join(groupsRoot, name);
     try {
       if (!statSync(p).isDirectory()) continue;

package/unify/groups/index.js

@@ -44,6 +44,8 @@ export {
   createGroupFromSpec,
   renameGroup,
   archiveGroup,
+  deleteGroup,
+  purgeArchivedGroups,
   addMember,
   removeMember,
   setGroupDefaultVp,

package/unify/stop-hooks.js

@@ -47,6 +47,10 @@ export async function runStopHooks(context) {
     messages = [],
     taskId,
     trace,
+    // Bug 6: groupId/threadId stamped on every persisted message so
+    // history replay can route messages back into the originating group.
+    groupId,
+    threadId,
   } = context;
 
   // Model name for persisted messages: use primaryModel if provided, else config.model
@@ -110,6 +114,9 @@ export async function runStopHooks(context) {
           record.toolCalls = msg.toolCalls;
         }
         if (msg.isError) record.isError = true;
+        // Bug 6: stamp groupId / threadId so replay can re-route by group.
+        if (groupId) record.groupId = groupId;
+        if (threadId) record.threadId = threadId;
         conversationStore.append(record);
         result.messagesPersisted++;
       }

package/unify/web-bridge.js

@@ -40,6 +40,8 @@ import {
   createGroupFromSpec,
   renameGroup,
   archiveGroup,
+  deleteGroup,
+  purgeArchivedGroups,
   addMember,
   removeMember,
   setGroupDefaultVp,
@@ -455,6 +457,23 @@ export function handleUnifyArchiveGroup(msg) {
   }
 }
 
+/**
+ * Bug 8: physical delete — removes the group dir and any legacy
+ * `.archived-*-<groupId>` siblings. Replies with op:'delete'.
+ */
+export function handleUnifyDeleteGroup(msg) {
+  const requestId = msg && msg.requestId;
+  const groupId = msg && msg.groupId;
+  try {
+    const yeaftDir = ctx.CONFIG?.yeaftDir;
+    const result = deleteGroup(yeaftDir, groupId);
+    sendGroupCrudResult({ op: 'delete', requestId, ok: true, groupId: result.groupId });
+    sendGroupSnapshotBroadcast();
+  } catch (err) {
+    sendGroupCrudResult({ op: 'delete', requestId, ok: false, error: groupErrorPayload(err) });
+  }
+}
+
 export function handleUnifyAddMember(msg) {
   const requestId = msg && msg.requestId;
   const groupId = msg && msg.groupId;
@@ -1370,6 +1389,21 @@ export async function handleUnifyChat(msg) {
       runAutoArchiveSweep(session);
       scheduleAutoArchive(session);
 
+      // Bug 8: clean up any legacy `.archived-*` group directories left
+      // behind by the previous soft-archive flow. This is a one-shot
+      // boot-time sweep — physical deletes after this point are immediate.
+      try {
+        const yeaftDir = ctx.CONFIG?.yeaftDir;
+        if (yeaftDir) {
+          const removed = purgeArchivedGroups(yeaftDir);
+          if (removed && removed.length > 0) {
+            console.log(`[Unify] purged ${removed.length} legacy .archived group dir(s)`);
+          }
+        }
+      } catch (err) {
+        console.warn('[Unify] purgeArchivedGroups failed:', err?.message || err);
+      }
+
       // Create a stable conversationId for the Unify session
       unifyConversationId = `unify-${Date.now()}`;
 
@@ -2238,13 +2272,17 @@ export async function handleUnifyLoadHistory(msg) {
   // Send each message through standard claude_output rendering pipeline
   for (const m of messages) {
     if (m.role === 'user') {
-      sendUnifyOutput({ type: 'user', message: { content: m.content } });
+      // Bug 6: forward groupId per message so the frontend re-stamps
+      // replayed messages into their originating group instead of the
+      // user's current filter (which would otherwise hide them when
+      // switching groups).
+      sendUnifyOutput({ type: 'user', message: { content: m.content } }, m.groupId || null);
     } else if (m.role === 'assistant') {
       sendUnifyOutput({
         type: 'assistant',
         message: { content: [{ type: 'text', text: m.content }] },
-      });
-      sendUnifyOutput({ type: 'result', result_text: '' });
+      }, m.groupId || null);
+      sendUnifyOutput({ type: 'result', result_text: '' }, m.groupId || null);
     }
   }