neoagent 2.5.2-beta.5 → 2.5.2-beta.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "neoagent",
-  "version": "2.5.2-beta.5",
+  "version": "2.5.2-beta.7",
   "description": "Proactive personal AI agent with no limits",
   "license": "AGPL-3.0-only",
   "main": "server/index.js",

package/server/public/.last_build_id

@@ -1 +1 @@
-686c0241f8a0ccd4581a0d28d54b568e
+c71ef84ad1471500295da6f6c6f13299

package/server/public/flutter_bootstrap.js

@@ -37,6 +37,6 @@ _flutter.buildConfig = {"engineRevision":"77e2e94772b6eb43759e34ed1ad7da4674e19c
 
 _flutter.loader.load({
   serviceWorkerSettings: {
-    serviceWorkerVersion: "3861825221" /* Flutter's service worker is deprecated and will be removed in a future Flutter release. */
+    serviceWorkerVersion: "2830694115" /* Flutter's service worker is deprecated and will be removed in a future Flutter release. */
   }
 });

package/server/public/main.dart.js

@@ -134794,7 +134794,7 @@ r===$&&A.b()
 p.push(A.jP(q,A.j9(!1,new A.a_(B.uG,A.d8(new A.cA(B.jt,new A.a7N(r,q),q),q,q),q),!1,B.H,!0),q,q,0,0,0,q))}r=!1
 if(!s.ay)if(!s.ch){r=s.e
 r===$&&A.b()
-r=B.b.u("mqfhwocj-bf9ea71").length!==0&&r.b}if(r){r=s.d
+r=B.b.u("mqfoxi8l-ab0f828").length!==0&&r.b}if(r){r=s.d
 r===$&&A.b()
 r=r.aP&&!r.ai?84:0
 s=s.e
@@ -140506,7 +140506,7 @@ $S:0}
 A.a_6.prototype={}
 A.SQ.prototype={
 nb(a){var s=this
-if(B.b.u("mqfhwocj-bf9ea71").length===0||s.a!=null)return
+if(B.b.u("mqfoxi8l-ab0f828").length===0||s.a!=null)return
 s.AU()
 s.a=A.on(B.RH,new A.bc8(s))},
 AU(){var s=0,r=A.l(t.H),q,p=2,o=[],n=this,m,l,k,j,i,h,g,f
@@ -140524,7 +140524,7 @@ if(!t.f.b(k)){s=1
 break}i=J.a3(k,"buildId")
 h=i==null?null:B.b.u(J.p(i))
 j=h==null?"":h
-if(J.bi(j)===0||J.d(j,"mqfhwocj-bf9ea71")){s=1
+if(J.bi(j)===0||J.d(j,"mqfoxi8l-ab0f828")){s=1
 break}n.b=!0
 n.F()
 p=2
@@ -140541,7 +140541,7 @@ case 2:return A.i(o.at(-1),r)}})
 return A.k($async$AU,r)},
 vE(){var s=0,r=A.l(t.H),q,p=2,o=[],n=this,m,l,k,j,i,h,g,f,e,d,c,b,a,a0,a1
 var $async$vE=A.h(function(a2,a3){if(a2===1){o.push(a3)
-s=p}for(;;)switch(s){case 0:if(B.b.u("mqfhwocj-bf9ea71").length===0||n.c){s=1
+s=p}for(;;)switch(s){case 0:if(B.b.u("mqfoxi8l-ab0f828").length===0||n.c){s=1
 break}n.c=!0
 n.F()
 p=4

package/server/services/ai/engine.js

@@ -144,6 +144,7 @@ function normalizeErrorKey(errorMsg) {
   if (/enoent|no such file/i.test(msg)) return 'enoent';
   if (/can.?t cd to|no such directory/i.test(msg)) return 'bad_cwd';
   if (/not found/i.test(msg)) return 'not_found';
+  if (/owner_repo.*format|must be.*owner.*repo|owner.*repo.*string|owner.*repo.*combined/i.test(msg)) return 'owner_repo_format';
   return msg.slice(0, 60);
 }
 
@@ -155,10 +156,20 @@ function trackErrorPattern(errorMsg, runMeta) {
 }
 
 function buildErrorPatternGuidance(key, count) {
+  // Immediate guidance on first occurrence for high-signal patterns that waste
+  // multiple iterations before self-correcting.
+  const immediateGuides = {
+    eisdir: 'That path is a directory (or a VM-only path like /tmp that read_file cannot reach). Use execute_command with `cat <path>` to read files inside VMs, or list_directory to inspect a directory.',
+    owner_repo_format: 'The parameter "owner_repo" expects a single combined string like "NeoLabs-Systems/NeoAgent" — not separate owner/repo fields. Pass the full "owner/repo" as one value.',
+  };
+  if (immediateGuides[key]) {
+    const prefix = count > 1 ? `REPEATED ERROR (${count}×): ` : 'ERROR GUIDANCE: ';
+    return `${prefix}${immediateGuides[key]}`;
+  }
+
   if (count < 3) return null;
   const guides = {
     outside_workspace: 'read_file cannot access /tmp paths. Use execute_command with `cat <path>` instead.',
-    eisdir: 'That path is a directory, not a file. Use list_directory or execute_command with `ls` to inspect it.',
     enoent: 'That path does not exist. Use execute_command with `find . -name "..."` to locate the correct path first.',
     bad_cwd: 'The VM home directory is not ~/. Use absolute paths starting from /tmp or discover the workspace root first.',
     not_found: 'This path or resource was not found. Try listing the parent directory or checking with a broader search first.',
@@ -168,6 +179,19 @@ function buildErrorPatternGuidance(key, count) {
   return `REPEATED ERROR (${count}×): ${guide}`;
 }
 
+const OUTPUT_FINGERPRINT_TOOLS = /^(list_|search_|read_|get_|find_|github_list|github_get|github_search)/;
+
+function fingerprintOutput(toolName, result) {
+  if (!toolName || !OUTPUT_FINGERPRINT_TOOLS.test(toolName)) return null;
+  const raw = typeof result === 'string' ? result : JSON.stringify(result ?? '');
+  if (raw.length < 200) return null;
+  // djb2 hash over first 3000 chars — fast, collision-unlikely for our sizes
+  let h = 5381;
+  const limit = Math.min(raw.length, 3000);
+  for (let i = 0; i < limit; i++) h = ((h << 5) + h) ^ raw.charCodeAt(i);
+  return h >>> 0;
+}
+
 function resolveModelCallTimeoutMs(options = {}) {
   const requested = Number(options?.modelCallTimeoutMs);
   if (Number.isFinite(requested) && requested > 0) {
@@ -2228,7 +2252,7 @@ class AgentEngine {
     if (!runMeta || runMeta.aborted || runMeta.triggerSource !== 'messaging') {
       return { sent: false, skipped: true };
     }
-    if (runMeta.finalDeliverySent === true || runMeta.terminalInterim) {
+    if (runMeta.terminalInterim) {
       return { sent: false, skipped: true };
     }
 
@@ -2596,6 +2620,7 @@ class AgentEngine {
     this.activeRuns.set(runId, {
       userId,
       agentId,
+      title: runTitle,
       status: 'running',
       aborted: false,
       messagingSent: false,
@@ -2622,6 +2647,7 @@ class AgentEngine {
       systemSteeringQueue: [],
       toolPids: new Set(),
       repetitionGuard: new ToolRepetitionGuard(),
+      seenOutputHashes: new Map(),
       messagingContext: triggerSource === 'messaging'
         ? {
           platform: options.source || null,
@@ -3010,6 +3036,13 @@ class AgentEngine {
       }
       messages = sanitizeConversationMessages(messages);
 
+      if (analysis.mode === 'execute' || analysis.mode === 'plan_execute') {
+        messages.push({
+          role: 'system',
+          content: 'Research budget: after 3 read/list/search tool calls, you must take a concrete action (write, create, send, update) or explain clearly why you cannot. Work on one item at a time — do not queue up more reads.',
+        });
+      }
+
       directAnswerEligible = isDirectAnswerEligibleAnalysis(analysis)
         && Boolean(normalizeOutgoingMessage(analysis.draft_reply));
 
@@ -3594,6 +3627,22 @@ class AgentEngine {
             }
           } else {
             consecutiveToolFailures = 0;
+            // Output fingerprint guard: steer away from re-fetching data already seen.
+            if (!toolErrorMessage) {
+              const currentRunMeta = this.getRunMeta(runId);
+              const fp = fingerprintOutput(toolName, toolResult);
+              if (fp !== null && currentRunMeta?.seenOutputHashes) {
+                const prior = currentRunMeta.seenOutputHashes.get(fp);
+                if (prior) {
+                  messages.push({
+                    role: 'system',
+                    content: `DUPLICATE DATA: This response is identical to what "${prior.toolName}" returned in iteration ${prior.iteration}. You already have this information. Stop fetching and use what you have — proceed to the next concrete action.`,
+                  });
+                } else {
+                  currentRunMeta.seenOutputHashes.set(fp, { toolName, iteration });
+                }
+              }
+            }
           }
 
           if (toolName === 'send_interim_update') {
@@ -3748,9 +3797,43 @@ class AgentEngine {
           );
         }
         if (iteration >= maxIterations) {
-          throw new Error(`Iteration limit reached before explicit completion after ${maxIterations} iterations.`);
+          // Grace call: budget exhausted but no content yet.
+          // Strip tools and ask the model to summarise what it accomplished.
+          // Mirrors the Hermes handle_max_iterations() pattern.
+          console.warn(`[Run ${shortenRunId(runId)}] iteration_limit runId=${shortenRunId(runId)} — making grace call`);
+          try {
+            const graceMessages = sanitizeConversationMessages([
+              ...messages,
+              {
+                role: 'user',
+                content: 'You have reached the maximum number of tool-calling iterations allowed. Please provide a final response summarising what you found and accomplished so far, without calling any more tools.',
+              },
+            ]);
+            const graceResponse = await withModelCallTimeout(
+              provider.chat(graceMessages, [], {
+                model,
+                reasoningEffort: this.getReasoningEffort(providerName, options),
+              }),
+              options,
+              `Grace call after ${maxIterations} iterations`,
+            );
+            totalTokens += graceResponse.usage?.totalTokens || 0;
+            lastContent = sanitizeModelOutput(graceResponse.content || '', { model });
+            if (lastContent) {
+              messages.push({ role: 'assistant', content: lastContent });
+              if (conversationId) {
+                db.prepare('INSERT INTO conversation_messages (conversation_id, role, content, tokens) VALUES (?, ?, ?, ?)')
+                  .run(conversationId, 'assistant', lastContent, graceResponse.usage?.totalTokens || 0);
+              }
+            }
+          } catch (graceErr) {
+            console.warn(`[Run ${shortenRunId(runId)}] grace call failed: ${graceErr?.message}`);
+          }
+          if (!normalizeOutgoingMessage(lastContent, options?.source || null)) {
+            throw new Error(`Iteration limit reached before explicit completion after ${maxIterations} iterations.`);
+          }
         }
-        if (stepIndex > 0 && !lastToolWasMessaging) {
+        if (stepIndex > 0 && !lastToolWasMessaging && iteration < maxIterations) {
           throw new Error('Run ended without an explicit completion or blocker reply.');
         }
       }