@yeaft/webchat-agent 0.1.787 → 0.1.789

package/package.json

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

package/unify/conversation/persist.js

@@ -107,6 +107,13 @@ function serializeMessage(msg) {
   // the speaker so the UI can render the message on the correct VP track.
   // For real user messages this is unset.
   if (msg.speakerVpId) fm.push(`speakerVpId: ${msg.speakerVpId}`);
+  // Internal/synthetic rows must round-trip so refresh/history replay can
+  // keep them out of the user-visible conversation. Reflection folding uses
+  // `_reflection`; other engine-only rows may use one of the explicit flags.
+  if (msg._reflection) fm.push('_reflection: true');
+  if (msg.internal) fm.push('internal: true');
+  if (msg.systemOnly) fm.push('systemOnly: true');
+  if (msg.systemOnlyMessage) fm.push('systemOnlyMessage: true');
 
   // Token estimate
   const content = msg.content || '';
@@ -179,6 +186,10 @@ export function parseMessage(raw) {
       case 'sourceThreadId': msg.sourceThreadId = value; break;
       case 'groupId': msg.groupId = value; break;
       case 'speakerVpId': msg.speakerVpId = value; break;
+      case '_reflection': msg._reflection = value === 'true'; break;
+      case 'internal': msg.internal = value === 'true'; break;
+      case 'systemOnly': msg.systemOnly = value === 'true'; break;
+      case 'systemOnlyMessage': msg.systemOnlyMessage = value === 'true'; break;
       // toolCalls are multi-line YAML — handled separately below
     }
   }

package/unify/engine.js

@@ -1782,7 +1782,7 @@ export class Engine {
 
         // PR-L: T2 end-of-turn (asynchronous) reflection. Fires when the
         // total tool count for this query() exceeds TURN_SUMMARY_THRESHOLD
-        // (5) AND no T1 has actually rewritten the arc yet. Kicks off the
+        // (8) AND no T1 has actually rewritten the arc yet. Kicks off the
         // primary-model call without await; the next query()'s
         // `#applyPendingT2Reflections` carries the result forward.
         //

package/unify/tool-folding/index.js

@@ -10,16 +10,30 @@
  *
  * The constants are NOT config-driven — V7 design freezes them in code.
  *
+ * Invariant: TURN_SUMMARY_THRESHOLD < TOOL_BATCH_SIZE. T1 runs inside the
+ * turn and collapses history in place; T2 fires at end_turn and is gated
+ * by `t1CollapsesDone === 0` (engine.js). If T2 were ever set ≥ T1, T1
+ * would collapse first and T2 could never fire — silently disabling the
+ * end-of-turn reflection path. Keep a usefully wide gap between the two
+ * so the (T2, T1) band where T2-alone applies stays meaningful.
+ *
  * TOOL_BATCH_SIZE history: was 13 originally; raised to 30 (2026-05-15)
  * after user feedback that 13 fired too often inside a single task and
  * fragmented otherwise-coherent tool arcs into multiple reflections. 30
  * keeps the periodic-reflection contract (it still fires every N tools,
  * not just once) but gives a single task arc room to breathe before the
  * arc gets collapsed.
+ *
+ * TURN_SUMMARY_THRESHOLD history: was 5 originally; raised to 8
+ * (2026-05-18). 5 was too aggressive — small "read a few files, edit one,
+ * run tests" turns crossed it and produced a T2 reflection card the user
+ * didn't want. 8 lets short-to-medium turns finish without a reflection
+ * while still folding the long ones that genuinely benefit from a summary
+ * before the next turn's history grows.
  */
 
 export const TOOL_BATCH_SIZE = 30;
-export const TURN_SUMMARY_THRESHOLD = 5;
+export const TURN_SUMMARY_THRESHOLD = 8;
 export const DUP_TOOL_THRESHOLD = 3;
 
 export { ExecLog, buildEntry, argsHashOf } from './exec-log.js';

package/unify/web-bridge.js

@@ -366,7 +366,11 @@ function makeGroupContextStub() {
 function projectPersistedToHistoryEntry(m) {
   if (!m) return null;
   if (m.role !== 'user' && m.role !== 'assistant' && m.role !== 'tool') return null;
+  if (m._reflection || m.internal || m.systemOnly || m.systemOnlyMessage) return null;
   const entry = { role: m.role, content: m.content };
+  if (m.id) entry.id = m.id;
+  if (m.groupId) entry.groupId = m.groupId;
+  if (m.speakerVpId) entry.speakerVpId = m.speakerVpId;
   if (m.toolCallId) entry.toolCallId = m.toolCallId;
   if (Array.isArray(m.toolCalls) && m.toolCalls.length > 0) {
     entry.toolCalls = m.toolCalls.map(tc => ({
@@ -2986,20 +2990,22 @@ export async function handleUnifyLoadHistory(msg) {
   const compactSummary = session.conversationStore.readCompactSummary();
 
   for (const m of messages) {
-    if (m.role === 'user') {
-      sendUnifyOutput({ type: 'user', message: { content: m.content } }, { groupId: m.groupId || null });
-    } else if (m.role === 'assistant') {
+    const entry = projectPersistedToHistoryEntry(m);
+    if (!entry) continue;
+    if (entry.role === 'user') {
+      sendUnifyOutput({ type: 'user', message: { content: entry.content, id: entry.id || null } }, { groupId: entry.groupId || null });
+    } else if (entry.role === 'assistant') {
       // speakerVpId rides on the envelope so the frontend can route this
       // replayed assistant text to the correct VP track. Without it, the
       // history replay would merge replies from different VPs onto one
       // anonymous assistant turn.
       const envelopeOpts = {
-        groupId: m.groupId || null,
+        groupId: entry.groupId || null,
       };
-      if (m.speakerVpId) envelopeOpts.vpId = m.speakerVpId;
+      if (entry.speakerVpId) envelopeOpts.vpId = entry.speakerVpId;
       sendUnifyOutput({
         type: 'assistant',
-        message: { content: [{ type: 'text', text: m.content }] },
+        message: { id: entry.id || null, content: [{ type: 'text', text: entry.content }] },
       }, envelopeOpts);
       sendUnifyOutput({ type: 'result', result_text: '' }, envelopeOpts);
     }
@@ -3081,22 +3087,19 @@ export async function handleUnifyLoadMoreHistory(msg) {
     result = { messages: [], oldestSeq: null, hasMore: false };
   }
 
-  // Wire shape mirrors handleUnifyLoadHistory's projection: only user /
-  // assistant text rows. Tool_use / tool_result replay is out of scope
-  // for this PR (today's bootstrap path drops them too).
-  //
-  // We intentionally do NOT ship `id` or `time` over the wire:
-  // `handleUnifyHistoryChunk` reads only role / content / groupId. The
-  // pagination cursor (`oldestSeq`) is sent once at the envelope level,
-  // so per-row ids are dead weight. `time` would be useful if we render
-  // "5 days ago" stamps on older history rows — when that ships, add it
-  // back here and consume it in conversationHandler.
+  // Wire shape mirrors handleUnifyLoadHistory's projection: only visible
+  // user / assistant text rows. Internal reflection/system-only rows stay
+  // server-side, and stable ids + speaker attribution ride with each row
+  // so older-history prepend renders exactly like refresh replay.
   const projected = (result.messages || [])
+    .map(projectPersistedToHistoryEntry)
     .filter(m => m && (m.role === 'user' || m.role === 'assistant'))
     .map(m => ({
+      ...(m.id ? { id: m.id } : {}),
       role: m.role,
       content: m.content,
       groupId: m.groupId || null,
+      ...(m.speakerVpId ? { speakerVpId: m.speakerVpId } : {}),
     }));
 
   emit({