@yeaft/webchat-agent 0.1.852 → 0.1.854

package/package.json

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

package/unify/conversation/persist.js

@@ -450,20 +450,22 @@ export class ConversationStore {
   }
 
   /**
-   * Update the compact summary (cumulative).
+   * Rewrite the compact summary in place.
    *
-   * @param {string} summary — new summary to append
+   * Compact's semantics are "the running summary of everything older than
+   * the hot window". Each compact pass already received the previous
+   * summary text as input and produced a *new* cumulative summary — so
+   * we overwrite, we never append. Appending was the original behaviour
+   * (kept around as a diary), but the engine reads the whole file back
+   * into `<conversation_summary>` on every turn, so appending grows the
+   * per-turn prompt without bound until it defeats compaction itself.
+   *
+   * @param {string} summary — the new, complete summary to persist
    */
-  updateCompactSummary(summary) {
-    let existing = '';
-    if (existsSync(this.#compactPath)) {
-      existing = readFileSync(this.#compactPath, 'utf8');
-    }
-
-    const date = new Date().toISOString().split('T')[0];
-    const entry = `\n## ${date}\n\n${summary}\n`;
+  replaceCompactSummary(summary) {
+    if (typeof summary !== 'string' || !summary) return;
     try {
-      writeFileSync(this.#compactPath, existing + entry, { encoding: 'utf8', mode: 0o644 });
+      writeFileSync(this.#compactPath, summary, { encoding: 'utf8', mode: 0o644 });
     } catch (err) {
       if (isPermissionError(err)) {
         if (!_permissionWarned) {
@@ -539,26 +541,19 @@ export class ConversationStore {
   }
 
   /**
-   * Append a per-(groupId, vpId) compact summary entry. Same append-only
-   * "## YYYY-MM-DD ..." structure as the legacy `updateCompactSummary`,
-   * but isolated to one file per (group, vp).
+   * Rewrite a per-(groupId, vpId) compact summary in place. See
+   * `replaceCompactSummary` for the rationale — same reason, scoped file.
    *
    * @param {string} groupId
    * @param {string} vpId
    * @param {string} summary
    */
-  updateCompactSummaryFor(groupId, vpId, summary) {
+  replaceCompactSummaryFor(groupId, vpId, summary) {
+    if (typeof summary !== 'string' || !summary) return;
     const path = this.#scopedCompactPath(groupId, vpId);
     if (!path) return;
-    let existing = '';
-    if (existsSync(path)) {
-      try { existing = readFileSync(path, 'utf8'); }
-      catch { existing = ''; }
-    }
-    const date = new Date().toISOString().split('T')[0];
-    const entry = `\n## ${date}\n\n${summary}\n`;
     try {
-      writeFileSync(path, existing + entry, { encoding: 'utf8', mode: 0o644 });
+      writeFileSync(path, summary, { encoding: 'utf8', mode: 0o644 });
     } catch (err) {
       if (isPermissionError(err)) {
         if (!_permissionWarned) {

package/unify/engine.js

@@ -991,14 +991,12 @@ export class Engine {
   #getCompactSummary() {
     if (!this.#conversationStore) return '';
     // Per-(group, vp) scoping: when this engine is bound to a fan-out VP,
-    // read its own summary file. On a miss (empty scoped file) we fall
-    // through to the legacy global file so pre-PR sessions whose only
-    // summary lives in compact.md still surface their context — matches
-    // the OR-fallback in web-bridge's `hasCompactSummary` flag.
+    // read ONLY its own summary file. Falling back to legacy compact.md here
+    // leaks another group/VP's summary into every new group turn after one
+    // post-turn compact writes the session-global file.
     if (this.#groupId && this.#vpId
         && typeof this.#conversationStore.readCompactSummaryFor === 'function') {
-      const scoped = this.#conversationStore.readCompactSummaryFor(this.#groupId, this.#vpId);
-      if (scoped) return scoped;
+      return this.#conversationStore.readCompactSummaryFor(this.#groupId, this.#vpId);
     }
     return this.#conversationStore.readCompactSummary();
   }
@@ -1149,18 +1147,32 @@ export class Engine {
     // a jarring locale flip mid-context.
     const isZh = String(this.#config.language || '').toLowerCase().startsWith('zh');
     const summariserSystem = isZh
-      ? '你是对话摘要器。请用中文写出 2–3 段简明摘要，保留决策、事实与上下文。'
-      : 'You are a conversation summarizer. Summarize concisely in 2–3 paragraphs, preserving decisions, facts, and context.';
+      ? '你是对话摘要器。下面包含「先前累计摘要」（可能为空）与「新待压缩对话」。请融合两者，输出一份「重写后的累计摘要」——不要分段罗列日期、不要保留 "## 2026-..." 等历史分节，直接产出一份连贯、可被下一轮直接重新注入 prompt 的摘要。保留关键决策、事实、上下文与人物意图。'
+      : 'You are a conversation summarizer. The input contains a "previous cumulative summary" (may be empty) plus a "new conversation to absorb". Merge them into ONE rewritten cumulative summary — do NOT keep dated section headers or any historical log structure. Output a single coherent summary suitable to be re-injected into the next turn\'s prompt as-is. Preserve key decisions, facts, context, and intent.';
     const summariserPromptPrefix = isZh ? '请概括：\n\n' : 'Summarize:\n\n';
 
     const hooks = {
       summarise: async () => {
         try {
+          // Read prior summary at call time, not at orchestrator setup,
+          // so the merge always sees the freshest on-disk state even if
+          // future orchestrator changes invoke summarise more than once.
+          const priorSummary = this.#getCompactSummary() || '';
+          const priorBlock = priorSummary
+            ? (isZh
+                ? `【先前累计摘要】\n${priorSummary}\n\n【新待压缩对话】\n`
+                : `[Previous cumulative summary]\n${priorSummary}\n\n[New conversation to absorb]\n`)
+            : '';
           const result = await adapter.call({
             model: fastConfig.model,
             system: summariserSystem,
-            messages: [{ role: 'user', content: `${summariserPromptPrefix}${toArchive.map(m => `[${m.role}] ${(m.content || '').slice(0, 500)}`).join('\n\n')}` }],
-            maxTokens: 1024,
+            messages: [{ role: 'user', content: `${summariserPromptPrefix}${priorBlock}${toArchive.map(m => `[${m.role}] ${(m.content || '').slice(0, 500)}`).join('\n\n')}` }],
+            // 10k output budget: the running summary is the engine's
+            // long-term memory of cold turns, so it deserves room to
+            // actually preserve detail. We rewrite-in-place each round,
+            // so size stays bounded by maxTokens regardless of how many
+            // compact passes have run.
+            maxTokens: 10240,
           });
           return (result.text || '').trim();
         } catch {
@@ -1204,10 +1216,10 @@ export class Engine {
         conversationStore.moveToColdBatch(archiveIds);
       }
       if (out.compactSummary) {
-        if (scoped && typeof conversationStore.updateCompactSummaryFor === 'function') {
-          conversationStore.updateCompactSummaryFor(this.#groupId, this.#vpId, out.compactSummary);
+        if (scoped && typeof conversationStore.replaceCompactSummaryFor === 'function') {
+          conversationStore.replaceCompactSummaryFor(this.#groupId, this.#vpId, out.compactSummary);
         } else {
-          conversationStore.updateCompactSummary(out.compactSummary);
+          conversationStore.replaceCompactSummary(out.compactSummary);
         }
       }
       // Index update only makes sense for the legacy path that actually

package/unify/web-bridge.js

@@ -3595,7 +3595,10 @@ export async function handleUnifyLoadHistory(msg) {
   const visiblePage = groupId
     ? loadVisibleGroupHistoryPage(session.conversationStore, groupId, limit)
     : { messages: limit > 0 ? pickRecent(session.conversationStore, limit) : [], oldestSeq: null, hasMore: false };
-  const compactSummary = session.conversationStore.readCompactSummary();
+  // Legacy compact.md is a non-group fallback only. For group replay, reading
+  // it makes every group show "has compact" once any legacy/non-scoped compact
+  // exists, even when this group has no scoped summary.
+  const compactSummary = groupId ? '' : session.conversationStore.readCompactSummary();
   const replayEntries = groupId
     ? visiblePage.messages
     : visiblePage.messages
@@ -3645,15 +3648,13 @@ export async function handleUnifyLoadHistory(msg) {
     oldestSeq = visiblePage.oldestSeq;
   }
 
-  // hasCompactSummary used to read a single session-global file, so it
-  // was always true once ANY group/VP in the session had compacted.
-  // Now we check (a) the scoped dir for any per-VP summary file in this
-  // group, falling back to (b) the legacy global file for sessions that
-  // pre-date the per-(group, vp) split.
+  // hasCompactSummary used to read a single session-global file, so it was
+  // always true once ANY group/VP in the session had compacted. For group
+  // replay, only scoped per-(group, vp) summaries count; legacy compact.md is
+  // reserved for non-group / pre-scoped 1:1 callers.
   let hasCompactSummaryFlag = !!compactSummary;
   if (groupId && typeof session.conversationStore.hasAnyCompactSummaryForGroup === 'function') {
-    hasCompactSummaryFlag = session.conversationStore.hasAnyCompactSummaryForGroup(groupId)
-      || hasCompactSummaryFlag;
+    hasCompactSummaryFlag = session.conversationStore.hasAnyCompactSummaryForGroup(groupId);
   }
 
   sendUnifyEvent({