@yeaft/webchat-agent 0.1.608 → 0.1.611

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.611",
   "description": "Remote agent for Yeaft WebChat — connects worker machines to the central server",
   "main": "index.js",
   "type": "module",
@@ -13,7 +13,7 @@
     "dev": "nodemon index.js"
   },
   "engines": {
-    "node": ">=18.0.0"
+    "node": ">=22.5.0"
   },
   "keywords": [
     "claude",
@@ -54,7 +54,6 @@
     "ext": "js"
   },
   "dependencies": {
-    "better-sqlite3": "^11.0.0",
     "dotenv": "^16.3.1",
     "tweetnacl": "^1.0.3",
     "tweetnacl-util": "^0.15.1",

package/unify/debug-trace.js

@@ -7,7 +7,7 @@
  * Reference: server/db/connection.js — Database(path), pragma WAL
  */
 
-import Database from 'better-sqlite3';
+import { DatabaseSync } from 'node:sqlite';
 import { randomUUID } from 'crypto';
 import { statSync } from 'fs';
 
@@ -80,7 +80,7 @@ function truncate(str, max) {
  * DebugTrace — SQLite-backed debug trace.
  */
 export class DebugTrace {
-  /** @type {Database.Database} */
+  /** @type {import('node:sqlite').DatabaseSync} */
   #db;
 
   /** @type {string} */
@@ -94,9 +94,9 @@ export class DebugTrace {
    */
   constructor(dbPath) {
     this.#dbPath = dbPath;
-    this.#db = new Database(dbPath);
-    this.#db.pragma('journal_mode = WAL');
-    this.#db.pragma('foreign_keys = ON');
+    this.#db = new DatabaseSync(dbPath);
+    this.#db.exec('PRAGMA journal_mode = WAL');
+    this.#db.exec('PRAGMA foreign_keys = ON');
     this.#db.exec(SCHEMA);
   }
 
@@ -282,9 +282,9 @@ export class DebugTrace {
    * @returns {{ turnCount: number, toolCount: number, eventCount: number, dbSizeBytes: number }}
    */
   stats() {
-    const turnCount = this.#db.prepare('SELECT COUNT(*) as c FROM trace_turns').get().c;
-    const toolCount = this.#db.prepare('SELECT COUNT(*) as c FROM trace_tools').get().c;
-    const eventCount = this.#db.prepare('SELECT COUNT(*) as c FROM trace_events').get().c;
+    const turnCount = Number(this.#db.prepare('SELECT COUNT(*) as c FROM trace_turns').get().c);
+    const toolCount = Number(this.#db.prepare('SELECT COUNT(*) as c FROM trace_tools').get().c);
+    const eventCount = Number(this.#db.prepare('SELECT COUNT(*) as c FROM trace_events').get().c);
     let dbSizeBytes = 0;
     try {
       dbSizeBytes = statSync(this.#dbPath).size;
@@ -301,17 +301,17 @@ export class DebugTrace {
    */
   cleanup(retentionDays = 30) {
     const cutoff = Date.now() - retentionDays * 24 * 60 * 60 * 1000;
-    const deletedTools = this.#db.prepare(`
+    const deletedTools = Number(this.#db.prepare(`
       DELETE FROM trace_tools WHERE turn_id IN (
         SELECT id FROM trace_turns WHERE started_at < ?
       )
-    `).run(cutoff).changes;
-    const deletedTurns = this.#db.prepare(`
+    `).run(cutoff).changes);
+    const deletedTurns = Number(this.#db.prepare(`
       DELETE FROM trace_turns WHERE started_at < ?
-    `).run(cutoff).changes;
-    const deletedEvents = this.#db.prepare(`
+    `).run(cutoff).changes);
+    const deletedEvents = Number(this.#db.prepare(`
       DELETE FROM trace_events WHERE created_at < ?
-    `).run(cutoff).changes;
+    `).run(cutoff).changes);
     return { deletedTurns, deletedTools, deletedEvents };
   }
 
@@ -333,7 +333,7 @@ export class DebugTrace {
    * Get or create a prepared statement.
    * @param {string} key
    * @param {string} sql
-   * @returns {Database.Statement}
+   * @returns {import('node:sqlite').StatementSync}
    */
   #prepare(key, sql) {
     if (!this.#stmts[key]) {

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