@yeaft/webchat-agent 0.1.499 → 0.1.501

package/package.json

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

package/unify/engine.js

@@ -26,8 +26,17 @@ import { buildMemoryInjection } from './memory/layout.js';
 import { runStopHooks } from './stop-hooks.js';
 import { getThreadStore, MAIN_THREAD_ID } from './threads/store.js';
 
-/** Maximum number of turns before the engine stops to prevent infinite loops. */
-const MAX_TURNS = 25;
+/**
+ * task-324 — Turn cap removed.
+ *
+ * Previously MAX_TURNS=25 broke the query loop out of long tool-driven
+ * conversations (user report: Unify loop errored at the cap). The engine
+ * now runs until the LLM itself returns stopReason='end_turn' or a
+ * non-retryable error surfaces. Real runaway loops are still bounded by:
+ *   • provider rate limits / context window (LLMContextError → compact)
+ *   • user-initiated abort (AbortController / cancel)
+ *   • MAX_CONTINUE_TURNS for the max_tokens auto-continue path
+ */
 
 /** Maximum auto-continue turns when stopReason is 'max_tokens'. */
 const MAX_CONTINUE_TURNS = 3;
@@ -401,15 +410,9 @@ export class Engine {
     while (true) {
       turnNumber++;
 
-      // Safety: prevent infinite loops
-      if (turnNumber > MAX_TURNS) {
-        yield {
-          type: 'error',
-          error: new Error(`Max turns (${MAX_TURNS}) reached — stopping to prevent infinite loop`),
-          retryable: false,
-        };
-        break;
-      }
+      // task-324: no hard MAX_TURNS cap. Loop terminates on end_turn,
+      // non-retryable error, LLMContextError (after compact retry), or
+      // caller abort. Keeping this comment so the removal is traceable.
 
       const turnId = this.#trace.startTurn({
         traceId: this.#traceId,

package/unify/web-bridge.js

@@ -921,20 +921,31 @@ export async function handleUnifyLoadHistory(msg) {
       const bucket = getThreadMessages(tid);
       bucket.push({ role: m.role, content: m.content });
     }
-
-    sendUnifyEvent({
-      type: 'session_ready',
-      conversationId: unifyConversationId,
-      model: session.config.model,
-      availableModels: session.config.availableModels || [],
-      skills: session.status.skills,
-      mcpServers: session.status.mcpServers,
-      tools: session.status.tools,
-    });
-    // task-301 Part 2: initial thread snapshot for history-load session.
-    sendThreadListUpdate();
   }
 
+  // task-322: replay `session_ready` + `thread_list_updated` UNCONDITIONALLY
+  // on every load-history call. The module-level `session` is a process-wide
+  // singleton — on page refresh the agent reuses it, so the lazy-init block
+  // above is skipped. The frontend's `enterUnify()` resets
+  // `unifyModel=null`, `unifyThreads=[]`, `unifySessionReady=false` every
+  // time, so without this replay the UI is left with the model selector
+  // stuck on the placeholder, sidebar empty, and the local→agent
+  // conversationId migration never triggers (leaving the main pane blank).
+  //
+  // Frontend is idempotent: the `session_ready` handler either migrates
+  // local→agent convId (first time) or just updates model/status fields
+  // (repeat) — receiving it twice is a no-op on state invariants.
+  sendUnifyEvent({
+    type: 'session_ready',
+    conversationId: unifyConversationId,
+    model: session.config.model,
+    availableModels: session.config.availableModels || [],
+    skills: session.status.skills,
+    mcpServers: session.status.mcpServers,
+    tools: session.status.tools,
+  });
+  sendThreadListUpdate();
+
   const limit = msg.limit || 50;
   const messages = session.conversationStore.loadRecent(limit);
   const compactSummary = session.conversationStore.readCompactSummary();