neoagent 2.5.2-beta.25 → 2.5.2-beta.27

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "neoagent",
-  "version": "2.5.2-beta.25",
-  "description": "Proactive personal AI agent with no limits",
+  "version": "2.5.2-beta.27",
+  "description": "Self-hosted AI agent for long-running tasks, automation, messaging, device control, and local memory",
   "license": "AGPL-3.0-only",
   "main": "server/index.js",
   "engines": {

package/server/public/.last_build_id

@@ -1 +1 @@
-7c286f044074b8e0fc8946771061244b
+7ff5faf6c30bf9859f1c1ee320f350fe

package/server/public/flutter_bootstrap.js

@@ -37,6 +37,6 @@ _flutter.buildConfig = {"engineRevision":"77e2e94772b6eb43759e34ed1ad7da4674e19c
 
 _flutter.loader.load({
   serviceWorkerSettings: {
-    serviceWorkerVersion: "2742195180" /* Flutter's service worker is deprecated and will be removed in a future Flutter release. */
+    serviceWorkerVersion: "3324688363" /* 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("mqh0uyur-6df469c").length!==0&&r.b}if(r){r=s.d
+r=B.b.u("mqh4ytnn-de00e37").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("mqh0uyur-6df469c").length===0||s.a!=null)return
+if(B.b.u("mqh4ytnn-de00e37").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,"mqh0uyur-6df469c")){s=1
+if(J.bi(j)===0||J.d(j,"mqh4ytnn-de00e37")){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("mqh0uyur-6df469c").length===0||n.c){s=1
+s=p}for(;;)switch(s){case 0:if(B.b.u("mqh4ytnn-de00e37").length===0||n.c){s=1
 break}n.c=!0
 n.F()
 p=4

package/server/services/ai/loop/conversation_loop.js

@@ -1513,9 +1513,16 @@ async function runConversation(engine, userId, userMessage, options = {}, _model
         }, {
           verified: true,
         });
+        if (analysis.mode === 'execute' || analysis.mode === 'plan_execute') {
+          const iterMeta = engine.getRunMeta(runId);
+          if (iterMeta) {
+            iterMeta.consecutiveReadOnlyIterations = (iterMeta.consecutiveReadOnlyIterations || 0) + 1;
+          }
+        }
         continue;
       }
 
+      let iterationConcreteProgress = false;
       for (const toolCall of response.toolCalls) {
         if (engine.isRunStopped(runId)) break;
         stepIndex++;
@@ -1730,6 +1737,9 @@ async function runConversation(engine, userId, userMessage, options = {}, _model
 
         const execution = classifyToolExecution(toolName, toolArgs, toolResult, toolErrorMessage);
         execution.input = toolArgs;
+        if (execution.stateChanged && isProgressToolCall(toolName, toolArgs)) {
+          iterationConcreteProgress = true;
+        }
         repetitionGuard?.observe(toolName, toolArgs, toolResult);
         execution.artifacts = await extractArtifactsFromResult(toolName, toolResult);
         toolExecutions.push(execution);
@@ -1899,12 +1909,7 @@ async function runConversation(engine, userId, userMessage, options = {}, _model
         && (analysis.mode === 'execute' || analysis.mode === 'plan_execute')) {
         const iterMeta = engine.getRunMeta(runId);
         if (iterMeta) {
-          const calledProgress = response.toolCalls.some((tc) => {
-            let parsedArgs = {};
-            try { parsedArgs = JSON.parse(tc.function?.arguments || '{}'); } catch {}
-            return isProgressToolCall(tc.function?.name || '', parsedArgs);
-          });
-          iterMeta.consecutiveReadOnlyIterations = calledProgress
+          iterMeta.consecutiveReadOnlyIterations = iterationConcreteProgress
             ? 0
             : (iterMeta.consecutiveReadOnlyIterations || 0) + 1;
         }

package/server/services/ai/loop/progress_classification.js

@@ -163,7 +163,7 @@ function isProgressToolCall(toolName, toolArgs = {}) {
   if (/^(list_|search_|read_file|get_file|find_files?|github_list|github_get|github_search|browser_get|browser_read)/.test(name)) {
     return false;
   }
-  if (name === 'http_request') {
+  if (name === 'http_request' || name === 'github_api_request') {
     return String(toolArgs?.method || 'GET').toUpperCase() !== 'GET';
   }
   if (name === 'execute_command') {

package/server/services/ai/toolEvidence.js

@@ -10,6 +10,7 @@ const { summarizeForLog } = require('./logFormat');
 const { normalizeOutgoingMessage, clampRunContext } = require('./messagingFallback');
 const {
   isClearlyReadOnlyShellCommand,
+  isProgressToolCall,
 } = require('./loop/progress_classification');
 
 // Ordered classification rules mapping a tool name to its evidence "source"
@@ -93,6 +94,8 @@ function classifyToolExecution(toolName, toolArgs = {}, result, errorMessage = '
       ? !isClearlyReadOnlyShellCommand(toolArgs?.command || '')
       : stateChangingExact.has(name)
   )
+    || (name.startsWith('github_') && isProgressToolCall(name, toolArgs))
+    || (name === 'http_request' && isProgressToolCall(name, toolArgs))
     || name.startsWith('android_')
     || ['browser_click', 'browser_type', 'browser_evaluate'].includes(name);
 
@@ -134,7 +137,7 @@ function classifyToolExecution(toolName, toolArgs = {}, result, errorMessage = '
     error: normalizedError,
     evidenceSource,
     evidenceRelevant,
-    stateChanged,
+    stateChanged: stateChanged && !normalizedError,
     dependsOnOutput: true,
     summary: compactToolResult(name, toolArgs, result || { error: errorMessage || 'Tool failed' }, {
       softLimit: 500,

package/server/services/integrations/github/repos.js

@@ -755,6 +755,23 @@ function resolveApiRequestPath(args = {}) {
   return '';
 }
 
+function hasOwnerRepoArg(args = {}) {
+  if (typeof args.owner_repo === 'string' && args.owner_repo.trim()) return true;
+  return Boolean(String(args.owner || '').trim() && String(args.repo || '').trim());
+}
+
+function parseOwnerRepoArg(args = {}) {
+  if (typeof args.owner_repo === 'string' && args.owner_repo.trim()) {
+    return parseOwnerRepo(args.owner_repo);
+  }
+  const owner = String(args.owner || '').trim();
+  const repo = String(args.repo || '').trim();
+  if (owner && repo) {
+    return parseOwnerRepo(`${owner}/${repo}`);
+  }
+  return parseOwnerRepo(args.owner_repo);
+}
+
 function encodeGithubFileContent(args = {}) {
   const content = String(args.content ?? '');
   const encoding = String(args.encoding || args.content_encoding || 'utf-8').toLowerCase();
@@ -770,6 +787,23 @@ function resolveApiRequestBody(args = {}) {
   return null;
 }
 
+for (const definition of githubToolDefinitions) {
+  const parameters = definition.parameters;
+  const properties = parameters?.properties;
+  if (!properties?.owner_repo) continue;
+  properties.owner = {
+    type: 'string',
+    description: 'Repository owner alias. Use together with repo when owner_repo is not supplied.',
+  };
+  properties.repo = {
+    type: 'string',
+    description: 'Repository name alias. Use together with owner when owner_repo is not supplied.',
+  };
+  if (Array.isArray(parameters.required) && parameters.required.includes('owner_repo')) {
+    parameters.required = parameters.required.filter((item) => item !== 'owner_repo');
+  }
+}
+
 async function executeGithubTool(toolName, args, auth) {
   switch (toolName) {
     case 'github_get_auth_user': {
@@ -789,14 +823,14 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_get_repo': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       return await githubApiRequest(auth, {
         path: `/repos/${owner}/${repo}`,
       });
     }
 
     case 'github_list_issues': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       const query = { per_page: Math.min(Number(args.max_results) || 30, 100) };
       if (args.state) query.state = args.state;
       if (args.labels) query.labels = args.labels;
@@ -810,14 +844,14 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_get_issue': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       return await githubApiRequest(auth, {
         path: `/repos/${owner}/${repo}/issues/${Number(args.issue_number)}`,
       });
     }
 
     case 'github_create_issue': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       return await githubApiRequest(auth, {
         method: 'POST',
         path: `/repos/${owner}/${repo}/issues`,
@@ -831,7 +865,7 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_update_issue': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       const body = {};
       if (args.title) body.title = String(args.title);
       if (args.body !== undefined) body.body = String(args.body);
@@ -846,7 +880,7 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_add_issue_labels': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       return await githubApiRequest(auth, {
         method: 'POST',
         path: `/repos/${owner}/${repo}/issues/${Number(args.issue_number)}/labels`,
@@ -857,7 +891,7 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_add_issue_assignees': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       return await githubApiRequest(auth, {
         method: 'POST',
         path: `/repos/${owner}/${repo}/issues/${Number(args.issue_number)}/assignees`,
@@ -868,7 +902,7 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_list_prs': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       const query = { per_page: Math.min(Number(args.max_results) || 30, 100) };
       if (args.state) query.state = args.state;
       if (args.sort) query.sort = args.sort;
@@ -880,14 +914,14 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_get_pr': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       return await githubApiRequest(auth, {
         path: `/repos/${owner}/${repo}/pulls/${Number(args.pr_number)}`,
       });
     }
 
     case 'github_create_pr': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       return await githubApiRequest(auth, {
         method: 'POST',
         path: `/repos/${owner}/${repo}/pulls`,
@@ -903,7 +937,7 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_update_pr': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       const body = {};
       if (args.title) body.title = String(args.title);
       if (args.body !== undefined) body.body = String(args.body);
@@ -916,7 +950,7 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_merge_pr': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       return await githubApiRequest(auth, {
         method: 'PUT',
         path: `/repos/${owner}/${repo}/pulls/${Number(args.pr_number)}/merge`,
@@ -929,7 +963,7 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_list_commits': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       const query = { per_page: Math.min(Number(args.max_results) || 30, 100) };
       if (args.sha) query.sha = String(args.sha);
       if (args.path) query.path = String(args.path);
@@ -940,7 +974,7 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_list_branches': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       return await githubApiRequest(auth, {
         path: `/repos/${owner}/${repo}/branches`,
         query: buildPaginationParams({ per_page: args.max_results }),
@@ -948,7 +982,7 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_get_branch': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       const branch = encodeURIComponent(String(args.branch || ''));
       return await githubApiRequest(auth, {
         path: `/repos/${owner}/${repo}/branches/${branch}`,
@@ -956,7 +990,7 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_list_collaborators': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       const maxResults = parsePositiveInt(args.max_results, 30);
       const perPage = Math.min(100, maxResults);
       const query = {
@@ -979,7 +1013,7 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_get_content': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       const query = {};
       if (args.ref) query.ref = String(args.ref);
       const result = await githubApiRequest(auth, {
@@ -994,7 +1028,7 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_create_or_update_file': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       return await githubApiRequest(auth, {
         method: 'PUT',
         path: `/repos/${owner}/${repo}/contents/${String(args.path || '')}`,
@@ -1008,7 +1042,7 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_delete_file': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       return await githubApiRequest(auth, {
         method: 'DELETE',
         path: `/repos/${owner}/${repo}/contents/${String(args.path || '')}`,
@@ -1021,7 +1055,7 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_list_workflow_runs': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       let path = `/repos/${owner}/${repo}/actions/runs`;
       if (args.workflow_id) {
         path = `/repos/${owner}/${repo}/actions/workflows/${String(args.workflow_id)}/runs`;
@@ -1050,14 +1084,14 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_get_workflow_run': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       return await githubApiRequest(auth, {
         path: `/repos/${owner}/${repo}/actions/runs/${Number(args.run_id)}`,
       });
     }
 
     case 'github_trigger_workflow': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       let inputs = {};
       if (args.inputs) {
         try {
@@ -1077,7 +1111,7 @@ async function executeGithubTool(toolName, args, auth) {
     }
 
     case 'github_list_workflows': {
-      const { owner, repo } = parseOwnerRepo(args.owner_repo);
+      const { owner, repo } = parseOwnerRepoArg(args);
       return await githubApiRequest(auth, {
         path: `/repos/${owner}/${repo}/actions/workflows`,
       });
@@ -1132,9 +1166,9 @@ async function executeGithubTool(toolName, args, auth) {
           ...parsedQuery,
           ...(args.query && typeof args.query === 'object' ? args.query : {}),
         };
-      } else if (args.owner_repo && !path.startsWith('/repos/') && !path.startsWith('/user') && !path.startsWith('/orgs/') && !path.startsWith('/search/')) {
+      } else if (hasOwnerRepoArg(args) && !path.startsWith('/repos/') && !path.startsWith('/user') && !path.startsWith('/orgs/') && !path.startsWith('/search/')) {
         // Convenience: prepend /repos/{owner}/{repo} for relative paths
-        const { owner, repo } = parseOwnerRepo(args.owner_repo);
+        const { owner, repo } = parseOwnerRepoArg(args);
         const relativePath = path.startsWith('/') ? path : `/${path}`;
         path = `/repos/${owner}/${repo}${relativePath}`;
       }