neoagent 2.5.2-beta.6 → 2.5.2-beta.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "neoagent",
-  "version": "2.5.2-beta.6",
+  "version": "2.5.2-beta.8",
   "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 @@
-1050e01e6b4a9c529922c7db724d0188
+bb621393d8fc51c33384ab5b836db272

package/server/public/flutter_bootstrap.js

@@ -37,6 +37,6 @@ _flutter.buildConfig = {"engineRevision":"77e2e94772b6eb43759e34ed1ad7da4674e19c
 
 _flutter.loader.load({
   serviceWorkerSettings: {
-    serviceWorkerVersion: "1247404091" /* Flutter's service worker is deprecated and will be removed in a future Flutter release. */
+    serviceWorkerVersion: "2920927188" /* 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("mqfmnj9i-24c1816").length!==0&&r.b}if(r){r=s.d
+r=B.b.u("mqfpjvdl-88fa84f").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("mqfmnj9i-24c1816").length===0||s.a!=null)return
+if(B.b.u("mqfpjvdl-88fa84f").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,"mqfmnj9i-24c1816")){s=1
+if(J.bi(j)===0||J.d(j,"mqfpjvdl-88fa84f")){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("mqfmnj9i-24c1816").length===0||n.c){s=1
+s=p}for(;;)switch(s){case 0:if(B.b.u("mqfpjvdl-88fa84f").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,31 @@ 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;
+}
+
+// Tools that represent concrete forward progress (write, create, send, update, run).
+// Anything NOT in this set is considered read-only for the analysis-paralysis gate.
+// execute_command counts as progress — it can do anything, including modify state.
+function isProgressTool(toolName) {
+  if (!toolName) return false;
+  // Neutral / bookkeeping — don't count either way
+  if (toolName === 'activate_tools' || toolName === 'save_widget_snapshot') return false;
+  // Explicitly read-only patterns
+  if (/^(list_|search_|read_file|get_file|find_files?|github_list|github_get|github_search|browser_get|browser_read)/.test(toolName)) return false;
+  return true;
+}
+
 function resolveModelCallTimeoutMs(options = {}) {
   const requested = Number(options?.modelCallTimeoutMs);
   if (Number.isFinite(requested) && requested > 0) {
@@ -2623,6 +2659,8 @@ class AgentEngine {
       systemSteeringQueue: [],
       toolPids: new Set(),
       repetitionGuard: new ToolRepetitionGuard(),
+      seenOutputHashes: new Map(),
+      consecutiveReadOnlyIterations: 0,
       messagingContext: triggerSource === 'messaging'
         ? {
           platform: options.source || null,
@@ -3042,6 +3080,21 @@ class AgentEngine {
         });
         messages = steeringAtLoopStart.messages;
         messages = sanitizeConversationMessages(messages);
+
+        // Analysis-paralysis gate: fire at the start of every iteration where
+        // the agent has spent N turns only reading/listing/searching without
+        // taking any concrete action. Escalates in urgency each turn.
+        if (analysis.mode === 'execute' || analysis.mode === 'plan_execute') {
+          const readOnlyCount = this.getRunMeta(runId)?.consecutiveReadOnlyIterations || 0;
+          if (readOnlyCount >= 3) {
+            const urgency = readOnlyCount >= 6 ? 'CRITICAL' : 'ACTION REQUIRED';
+            messages.push({
+              role: 'system',
+              content: `${urgency} — ${readOnlyCount} consecutive read-only turns: You have been gathering information for ${readOnlyCount} turns without writing, creating, sending, or running anything. You must take ONE concrete action this turn (create a file, open a PR, run a command that modifies state, send a message) or call task_complete to report what you found and why you cannot proceed. Do not read or list anything further.`,
+            });
+          }
+        }
+
         this.updateRunProgress(runId, {
           currentPhase: 'model',
           currentStep: `model:${iteration}`,
@@ -3595,6 +3648,40 @@ 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 });
+                  // External state: persist large read results to disk so the
+                  // model can reference them after context compaction without
+                  // re-fetching. Only for significant payloads.
+                  const persistRaw = typeof toolResult === 'string' ? toolResult : JSON.stringify(toolResult ?? '');
+                  if (persistRaw.length >= 1000 && runId) {
+                    const persistPath = `/tmp/run-${runId.slice(0, 8)}-${toolName}.json`;
+                    try {
+                      require('fs').writeFileSync(persistPath, persistRaw.slice(0, 40000));
+                      if (!currentRunMeta.persistedDataPaths) currentRunMeta.persistedDataPaths = [];
+                      if (!currentRunMeta.persistedDataPaths.includes(persistPath)) {
+                        currentRunMeta.persistedDataPaths.push(persistPath);
+                        messages.push({
+                          role: 'system',
+                          content: `Data from "${toolName}" (iteration ${iteration}) persisted to ${persistPath}. If context compacts and you need this data again, use execute_command with \`cat ${persistPath}\` instead of re-fetching.`,
+                        });
+                      }
+                    } catch { /* non-fatal — disk full or permissions */ }
+                  }
+                }
+              }
+            }
           }
 
           if (toolName === 'send_interim_update') {
@@ -3656,6 +3743,19 @@ class AgentEngine {
           }
         }
 
+        // Update analysis-paralysis counter after each iteration's tool calls.
+        // Resets to 0 when any progress tool was called; otherwise increments.
+        if (!directAnswerEligible && response?.toolCalls?.length > 0
+          && (analysis.mode === 'execute' || analysis.mode === 'plan_execute')) {
+          const iterMeta = this.getRunMeta(runId);
+          if (iterMeta) {
+            const calledProgress = response.toolCalls.some((tc) => isProgressTool(tc.function?.name || ''));
+            iterMeta.consecutiveReadOnlyIterations = calledProgress
+              ? 0
+              : (iterMeta.consecutiveReadOnlyIterations || 0) + 1;
+          }
+        }
+
         if (this.isRunStopped(runId)) break;
         if (this.getRunMeta(runId)?.terminalInterim) break;
         if (this.getRunMeta(runId)?.widgetSnapshotSaved) break;