@aragon-agent/core 0.2.28 → 0.2.30

package/dist/tools/executor.d.ts

@@ -15,7 +15,13 @@ import type { ToolExecutionResult, ToolProgressUpdate } from './types.js';
 export interface ToolExecutorOptions {
     /** Default per-tool execution timeout in milliseconds. Defaults to 120_000 (2 minutes). */
     defaultTimeout?: number;
-    /** Per-tool timeout overrides, keyed by tool name. */
+    /**
+     * Per-tool timeout overrides, keyed by tool name. An override of `0` means
+     * NO ceiling for that tool: no timer is armed, and only an external abort
+     * can end it. (`setTimeout(fn, 0)` fires immediately, so before this
+     * semantics existed a `0` override aborted the tool at once - a value no
+     * working caller could ever have wanted.)
+     */
     timeoutOverrides?: Record<string, number>;
     /** Maximum size (in bytes) of combined text output before truncation. Defaults to 100_000 (100 KB). */
     maxOutputSize?: number;

package/dist/tools/executor.js

@@ -83,16 +83,23 @@ export class ToolExecutor {
             return this.buildResult(toolCallId, toolName, errorResult(`Parameter validation error: ${err instanceof Error ? err.message : String(err)}`), startTime);
         }
         // 4. Set up timeout and merged abort signal.
+        //
+        // AN OVERRIDE OF `0` ARMS NO TIMER: the tool runs until it settles or an
+        // external abort arrives. The CLI's `task` fan-out is the caller this
+        // exists for - a dispatch must be able to outlast every generic
+        // `defaultTimeout` a session may carry, and a huge sentinel number would
+        // merely move the cliff.
         const timeoutMs = this.timeoutOverrides[toolName] ?? this.defaultTimeout;
         const controller = new AbortController();
-        const timeoutId = setTimeout(() => controller.abort(TIMEOUT_REASON), timeoutMs);
+        const timeoutId = timeoutMs > 0 ? setTimeout(() => controller.abort(TIMEOUT_REASON), timeoutMs) : null;
         // Propagate external signal to our controller.
         const externalAbortHandler = () => {
             controller.abort(signal?.reason ?? new Error('aborted'));
         };
         if (signal) {
             if (signal.aborted) {
-                clearTimeout(timeoutId);
+                if (timeoutId)
+                    clearTimeout(timeoutId);
                 return this.buildResult(toolCallId, toolName, errorResult('Execution aborted before start'), startTime);
             }
             signal.addEventListener('abort', externalAbortHandler, { once: true });
@@ -172,7 +179,8 @@ export class ToolExecutor {
             }
         }
         finally {
-            clearTimeout(timeoutId);
+            if (timeoutId)
+                clearTimeout(timeoutId);
             if (graceTimer)
                 clearTimeout(graceTimer);
             if (onAbandonAbort) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aragon-agent/core",
-  "version": "0.2.28",
+  "version": "0.2.30",
   "description": "AragonAgent core engine — LLM provider adapters, streaming, tool registry/execution, steering, idle watchdog, and an optional isolated-vm sandbox.",
   "license": "MIT",
   "author": "AragonAgent contributors",