@yeaft/webchat-agent 0.1.608 → 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.608",
+  "version": "0.1.609",
   "description": "Remote agent for Yeaft WebChat — connects worker machines to the central server",
   "main": "index.js",
   "type": "module",

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/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()}`;