@agentforge/core 0.15.3 → 0.15.5

package/dist/index.cjs

@@ -861,6 +861,9 @@ Examples:`);
   return parts.join("\n");
 }
 
+// src/tools/registry.ts
+var import_zod3 = require("zod");
+
 // src/langgraph/observability/logger.ts
 var LogLevel = /* @__PURE__ */ ((LogLevel2) => {
   LogLevel2["DEBUG"] = "debug";
@@ -965,6 +968,9 @@ var RegistryEvent = /* @__PURE__ */ ((RegistryEvent2) => {
   RegistryEvent2["REGISTRY_CLEARED"] = "registry:cleared";
   return RegistryEvent2;
 })(RegistryEvent || {});
+function eraseToolType(tool) {
+  return tool;
+}
 var ToolRegistry = class {
   tools = /* @__PURE__ */ new Map();
   eventHandlers = /* @__PURE__ */ new Map();
@@ -980,13 +986,14 @@ var ToolRegistry = class {
    * ```
    */
   register(tool) {
+    const erasedTool = eraseToolType(tool);
     const name = tool.metadata.name;
     if (this.tools.has(name)) {
       throw new Error(
         `Tool with name "${name}" is already registered. Use update() to modify it.`
       );
     }
-    this.tools.set(name, tool);
+    this.tools.set(name, erasedTool);
     this.emit("tool:registered" /* TOOL_REGISTERED */, tool);
   }
   /**
@@ -1057,6 +1064,7 @@ var ToolRegistry = class {
    * ```
    */
   update(name, tool) {
+    const erasedTool = eraseToolType(tool);
     if (!this.tools.has(name)) {
       return false;
     }
@@ -1065,7 +1073,7 @@ var ToolRegistry = class {
         `Cannot update tool: metadata.name "${tool.metadata.name}" does not match registry key "${name}". To rename a tool, remove it and register it again with the new name.`
       );
     }
-    this.tools.set(name, tool);
+    this.tools.set(name, erasedTool);
     this.emit("tool:updated" /* TOOL_UPDATED */, { name, tool });
     return true;
   }
@@ -1139,7 +1147,7 @@ var ToolRegistry = class {
   /**
    * Register multiple tools at once
    *
-   * @param tools - Array of tools to register
+   * @param tools - Iterable of tools to register
    * @throws Error if any tool name conflicts with existing tools
    *
    * @example
@@ -1148,9 +1156,10 @@ var ToolRegistry = class {
    * ```
    */
   registerMany(tools) {
+    const toolsToRegister = Array.from(tools);
     const inputNames = /* @__PURE__ */ new Set();
     const duplicatesInInput = [];
-    for (const tool of tools) {
+    for (const tool of toolsToRegister) {
       const name = tool.metadata.name;
       if (inputNames.has(name)) {
         duplicatesInInput.push(name);
@@ -1164,7 +1173,7 @@ var ToolRegistry = class {
       );
     }
     const conflicts = [];
-    for (const tool of tools) {
+    for (const tool of toolsToRegister) {
       if (this.tools.has(tool.metadata.name)) {
         conflicts.push(tool.metadata.name);
       }
@@ -1174,7 +1183,7 @@ var ToolRegistry = class {
         `Cannot register tools: the following names already exist: ${conflicts.join(", ")}`
       );
     }
-    for (const tool of tools) {
+    for (const tool of toolsToRegister) {
       this.register(tool);
     }
   }
@@ -1437,13 +1446,14 @@ var ToolRegistry = class {
       return lines;
     }
     lines.push(`- ${metadata.name}: ${metadata.description}`);
-    const schemaShape = tool.schema._def?.shape?.();
+    const schemaShape = this.getSchemaShape(tool.schema);
     if (schemaShape) {
       const params = Object.keys(schemaShape);
       if (params.length > 0) {
         const paramDescriptions = params.map((param) => {
           const field = schemaShape[param];
-          const type = field._def?.typeName?.replace("Zod", "").toLowerCase() || "any";
+          const typeName = field._def.typeName;
+          const type = typeName.replace("Zod", "").toLowerCase();
           return `${param} (${type})`;
         });
         lines.push(`  Parameters: ${paramDescriptions.join(", ")}`);
@@ -1503,6 +1513,12 @@ var ToolRegistry = class {
     }
     return lines;
   }
+  getSchemaShape(schema) {
+    if (schema instanceof import_zod3.z.ZodObject) {
+      return schema.shape;
+    }
+    return void 0;
+  }
 };
 
 // src/tools/executor.ts
@@ -1550,6 +1566,12 @@ function createToolExecutor(config = {}) {
     }
     return Math.min(delay, maxDelay);
   }
+  function toError(error) {
+    if (error instanceof Error) {
+      return error;
+    }
+    return new Error(String(error));
+  }
   async function executeWithRetry(tool, input, policy) {
     const executeFn = tool.invoke || tool.execute;
     if (!executeFn) {
@@ -1565,18 +1587,24 @@ function createToolExecutor(config = {}) {
     if (!policy) {
       return await executeFn.call(tool, input);
     }
+    if (!Number.isInteger(policy.maxAttempts) || policy.maxAttempts < 1) {
+      throw new Error(
+        `Invalid retry policy: maxAttempts must be an integer >= 1 (received ${String(policy.maxAttempts)})`
+      );
+    }
     let lastError;
     for (let attempt = 1; attempt <= policy.maxAttempts; attempt++) {
       try {
         return await executeFn.call(tool, input);
       } catch (error) {
-        lastError = error;
+        const currentError = toError(error);
+        lastError = currentError;
         if (policy.retryableErrors && policy.retryableErrors.length > 0) {
           const isRetryable = policy.retryableErrors.some(
-            (msg) => lastError.message.includes(msg)
+            (msg) => currentError.message.includes(msg)
           );
           if (!isRetryable) {
-            throw lastError;
+            throw currentError;
           }
         }
         if (attempt < policy.maxAttempts) {
@@ -1585,7 +1613,7 @@ function createToolExecutor(config = {}) {
         }
       }
     }
-    throw lastError;
+    throw lastError ?? new Error("Tool execution failed after retries");
   }
   async function executeSingle(tool, input, priority) {
     const startTime = Date.now();
@@ -1607,13 +1635,14 @@ function createToolExecutor(config = {}) {
       return result;
     } catch (error) {
       const duration = Date.now() - startTime;
+      const normalizedError = toError(error);
       metrics.totalExecutions++;
       metrics.failedExecutions++;
       metrics.totalDuration += duration;
       metrics.averageDuration = metrics.totalDuration / metrics.totalExecutions;
       metrics.byPriority[priority]++;
-      onExecutionError?.(tool, input, error, duration);
-      throw error;
+      onExecutionError?.(tool, input, normalizedError, duration);
+      throw normalizedError;
     }
   }
   async function processQueue() {
@@ -2075,8 +2104,8 @@ function createToolSimulator(config) {
     if (!latency) return 0;
     const u1 = Math.random();
     const u2 = Math.random();
-    const z3 = Math.sqrt(-2 * Math.log(u1)) * Math.cos(2 * Math.PI * u2);
-    return Math.max(0, latency.mean + z3 * latency.stddev);
+    const z4 = Math.sqrt(-2 * Math.log(u1)) * Math.cos(2 * Math.PI * u2);
+    return Math.max(0, latency.mean + z4 * latency.stddev);
   }
   return {
     execute: async (toolName, input) => {
@@ -4504,8 +4533,8 @@ function createDatabasePool(options) {
 
 // src/resources/http-pool.ts
 var MockHttpClient = class {
-  constructor(config) {
-    this.config = config;
+  constructor(_config) {
+    this._config = _config;
   }
   closed = false;
   async get(url, config) {
@@ -4520,7 +4549,7 @@ var MockHttpClient = class {
   async delete(url, config) {
     return this.request({ ...config, url, method: "DELETE" });
   }
-  async request(config) {
+  async request(_config) {
     if (this.closed) {
       throw new Error("Client is closed");
     }

package/dist/index.d.cts

@@ -1173,7 +1173,9 @@ declare enum RegistryEvent {
 /**
  * Event handler type
  */
-type EventHandler = (data: any) => void;
+type EventHandler = (data: unknown) => void;
+type RegistryTool = Tool<unknown, unknown>;
+type RegisterManyTool = Tool<never, unknown>;
 /**
  * Options for generating tool prompts
  */
@@ -1250,7 +1252,7 @@ declare class ToolRegistry {
      * registry.register(readFileTool);
      * ```
      */
-    register(tool: Tool<any, any>): void;
+    register<TInput, TOutput>(tool: Tool<TInput, TOutput>): void;
     /**
      * Get a tool by name
      *
@@ -1265,7 +1267,7 @@ declare class ToolRegistry {
      * }
      * ```
      */
-    get(name: string): Tool<any, any> | undefined;
+    get(name: string): RegistryTool | undefined;
     /**
      * Check if a tool exists in the registry
      *
@@ -1306,7 +1308,7 @@ declare class ToolRegistry {
      * const updated = registry.update('read-file', newReadFileTool);
      * ```
      */
-    update(name: string, tool: Tool<any, any>): boolean;
+    update<TInput, TOutput>(name: string, tool: Tool<TInput, TOutput>): boolean;
     /**
      * Get all registered tools
      *
@@ -1318,7 +1320,7 @@ declare class ToolRegistry {
      * console.log(`Total tools: ${allTools.length}`);
      * ```
      */
-    getAll(): Tool<any, any>[];
+    getAll(): RegistryTool[];
     /**
      * Get tools by category
      *
@@ -1330,7 +1332,7 @@ declare class ToolRegistry {
      * const fileTools = registry.getByCategory(ToolCategory.FILE_SYSTEM);
      * ```
      */
-    getByCategory(category: ToolCategory): Tool<any, any>[];
+    getByCategory(category: ToolCategory): RegistryTool[];
     /**
      * Get tools by tag
      *
@@ -1342,7 +1344,7 @@ declare class ToolRegistry {
      * const fileTools = registry.getByTag('file');
      * ```
      */
-    getByTag(tag: string): Tool<any, any>[];
+    getByTag(tag: string): RegistryTool[];
     /**
      * Search tools by name or description
      *
@@ -1357,11 +1359,11 @@ declare class ToolRegistry {
      * // Returns tools with 'file' in name or description
      * ```
      */
-    search(query: string): Tool<any, any>[];
+    search(query: string): RegistryTool[];
     /**
      * Register multiple tools at once
      *
-     * @param tools - Array of tools to register
+     * @param tools - Iterable of tools to register
      * @throws Error if any tool name conflicts with existing tools
      *
      * @example
@@ -1369,7 +1371,7 @@ declare class ToolRegistry {
      * registry.registerMany([tool1, tool2, tool3]);
      * ```
      */
-    registerMany(tools: Tool<any, any>[]): void;
+    registerMany(tools: Iterable<RegisterManyTool>): void;
     /**
      * Clear all tools from the registry
      *
@@ -1506,6 +1508,7 @@ declare class ToolRegistry {
      * @private
      */
     private formatRelations;
+    private getSchemaShape;
 }
 
 /**
@@ -1525,14 +1528,14 @@ interface ToolExecutorConfig {
     maxConcurrent?: number;
     timeout?: number;
     retryPolicy?: RetryPolicy;
-    priorityFn?: (tool: any) => Priority$1;
-    onExecutionStart?: (tool: any, input: any) => void;
-    onExecutionComplete?: (tool: any, input: any, result: any, duration: number) => void;
-    onExecutionError?: (tool: any, input: any, error: Error, duration: number) => void;
+    priorityFn?: (tool: ExecutableTool) => Priority$1;
+    onExecutionStart?: (tool: ExecutableTool, input: unknown) => void;
+    onExecutionComplete?: (tool: ExecutableTool, input: unknown, result: unknown, duration: number) => void;
+    onExecutionError?: (tool: ExecutableTool, input: unknown, error: Error, duration: number) => void;
 }
 interface ToolExecution {
-    tool: any;
-    input: any;
+    tool: ExecutableTool;
+    input: unknown;
     priority?: Priority$1;
 }
 interface ExecutionMetrics {
@@ -1543,14 +1546,21 @@ interface ExecutionMetrics {
     averageDuration: number;
     byPriority: Record<Priority$1, number>;
 }
+interface ExecutableTool<TInput = unknown, TOutput = unknown> {
+    metadata?: {
+        name?: string;
+    };
+    invoke?: (input: TInput) => Promise<TOutput>;
+    execute?: (input: TInput) => Promise<TOutput>;
+}
 /**
  * Create a tool executor with resource management
  */
 declare function createToolExecutor(config?: ToolExecutorConfig): {
-    execute: (tool: any, input: any, options?: {
+    execute: (tool: ExecutableTool, input: unknown, options?: {
         priority?: Priority$1;
-    }) => Promise<any>;
-    executeParallel: (executions: ToolExecution[]) => Promise<any[]>;
+    }) => Promise<unknown>;
+    executeParallel: (executions: ToolExecution[]) => Promise<unknown[]>;
     getMetrics: () => ExecutionMetrics;
     resetMetrics: () => void;
     getQueueStatus: () => {
@@ -4986,22 +4996,22 @@ interface HttpPoolConfig extends PoolConfig {
     keepAliveMsecs?: number;
 }
 interface HttpClient {
-    get<T = any>(url: string, config?: RequestConfig): Promise<HttpResponse<T>>;
-    post<T = any>(url: string, data?: any, config?: RequestConfig): Promise<HttpResponse<T>>;
-    put<T = any>(url: string, data?: any, config?: RequestConfig): Promise<HttpResponse<T>>;
-    delete<T = any>(url: string, config?: RequestConfig): Promise<HttpResponse<T>>;
-    request<T = any>(config: RequestConfig): Promise<HttpResponse<T>>;
+    get<T = unknown>(url: string, config?: RequestConfig): Promise<HttpResponse<T>>;
+    post<T = unknown, TData = unknown>(url: string, data?: TData, config?: RequestConfig<TData>): Promise<HttpResponse<T>>;
+    put<T = unknown, TData = unknown>(url: string, data?: TData, config?: RequestConfig<TData>): Promise<HttpResponse<T>>;
+    delete<T = unknown>(url: string, config?: RequestConfig): Promise<HttpResponse<T>>;
+    request<T = unknown, TData = unknown>(config: RequestConfig<TData>): Promise<HttpResponse<T>>;
     close(): Promise<void>;
 }
-interface RequestConfig {
+interface RequestConfig<TData = unknown> {
     url?: string;
     method?: string;
     headers?: Record<string, string>;
-    params?: Record<string, any>;
-    data?: any;
+    params?: Record<string, unknown>;
+    data?: TData;
     timeout?: number;
 }
-interface HttpResponse<T = any> {
+interface HttpResponse<T = unknown> {
     data: T;
     status: number;
     statusText: string;
@@ -5023,7 +5033,7 @@ declare class HttpPool {
     constructor(options: HttpPoolOptions);
     acquire(): Promise<HttpClient>;
     release(client: HttpClient): Promise<void>;
-    request<T = any>(config: RequestConfig): Promise<HttpResponse<T>>;
+    request<T = unknown, TData = unknown>(config: RequestConfig<TData>): Promise<HttpResponse<T>>;
     drain(): Promise<void>;
     clear(): Promise<void>;
     getStats(): PoolStats;

package/dist/index.d.ts

@@ -1173,7 +1173,9 @@ declare enum RegistryEvent {
 /**
  * Event handler type
  */
-type EventHandler = (data: any) => void;
+type EventHandler = (data: unknown) => void;
+type RegistryTool = Tool<unknown, unknown>;
+type RegisterManyTool = Tool<never, unknown>;
 /**
  * Options for generating tool prompts
  */
@@ -1250,7 +1252,7 @@ declare class ToolRegistry {
      * registry.register(readFileTool);
      * ```
      */
-    register(tool: Tool<any, any>): void;
+    register<TInput, TOutput>(tool: Tool<TInput, TOutput>): void;
     /**
      * Get a tool by name
      *
@@ -1265,7 +1267,7 @@ declare class ToolRegistry {
      * }
      * ```
      */
-    get(name: string): Tool<any, any> | undefined;
+    get(name: string): RegistryTool | undefined;
     /**
      * Check if a tool exists in the registry
      *
@@ -1306,7 +1308,7 @@ declare class ToolRegistry {
      * const updated = registry.update('read-file', newReadFileTool);
      * ```
      */
-    update(name: string, tool: Tool<any, any>): boolean;
+    update<TInput, TOutput>(name: string, tool: Tool<TInput, TOutput>): boolean;
     /**
      * Get all registered tools
      *
@@ -1318,7 +1320,7 @@ declare class ToolRegistry {
      * console.log(`Total tools: ${allTools.length}`);
      * ```
      */
-    getAll(): Tool<any, any>[];
+    getAll(): RegistryTool[];
     /**
      * Get tools by category
      *
@@ -1330,7 +1332,7 @@ declare class ToolRegistry {
      * const fileTools = registry.getByCategory(ToolCategory.FILE_SYSTEM);
      * ```
      */
-    getByCategory(category: ToolCategory): Tool<any, any>[];
+    getByCategory(category: ToolCategory): RegistryTool[];
     /**
      * Get tools by tag
      *
@@ -1342,7 +1344,7 @@ declare class ToolRegistry {
      * const fileTools = registry.getByTag('file');
      * ```
      */
-    getByTag(tag: string): Tool<any, any>[];
+    getByTag(tag: string): RegistryTool[];
     /**
      * Search tools by name or description
      *
@@ -1357,11 +1359,11 @@ declare class ToolRegistry {
      * // Returns tools with 'file' in name or description
      * ```
      */
-    search(query: string): Tool<any, any>[];
+    search(query: string): RegistryTool[];
     /**
      * Register multiple tools at once
      *
-     * @param tools - Array of tools to register
+     * @param tools - Iterable of tools to register
      * @throws Error if any tool name conflicts with existing tools
      *
      * @example
@@ -1369,7 +1371,7 @@ declare class ToolRegistry {
      * registry.registerMany([tool1, tool2, tool3]);
      * ```
      */
-    registerMany(tools: Tool<any, any>[]): void;
+    registerMany(tools: Iterable<RegisterManyTool>): void;
     /**
      * Clear all tools from the registry
      *
@@ -1506,6 +1508,7 @@ declare class ToolRegistry {
      * @private
      */
     private formatRelations;
+    private getSchemaShape;
 }
 
 /**
@@ -1525,14 +1528,14 @@ interface ToolExecutorConfig {
     maxConcurrent?: number;
     timeout?: number;
     retryPolicy?: RetryPolicy;
-    priorityFn?: (tool: any) => Priority$1;
-    onExecutionStart?: (tool: any, input: any) => void;
-    onExecutionComplete?: (tool: any, input: any, result: any, duration: number) => void;
-    onExecutionError?: (tool: any, input: any, error: Error, duration: number) => void;
+    priorityFn?: (tool: ExecutableTool) => Priority$1;
+    onExecutionStart?: (tool: ExecutableTool, input: unknown) => void;
+    onExecutionComplete?: (tool: ExecutableTool, input: unknown, result: unknown, duration: number) => void;
+    onExecutionError?: (tool: ExecutableTool, input: unknown, error: Error, duration: number) => void;
 }
 interface ToolExecution {
-    tool: any;
-    input: any;
+    tool: ExecutableTool;
+    input: unknown;
     priority?: Priority$1;
 }
 interface ExecutionMetrics {
@@ -1543,14 +1546,21 @@ interface ExecutionMetrics {
     averageDuration: number;
     byPriority: Record<Priority$1, number>;
 }
+interface ExecutableTool<TInput = unknown, TOutput = unknown> {
+    metadata?: {
+        name?: string;
+    };
+    invoke?: (input: TInput) => Promise<TOutput>;
+    execute?: (input: TInput) => Promise<TOutput>;
+}
 /**
  * Create a tool executor with resource management
  */
 declare function createToolExecutor(config?: ToolExecutorConfig): {
-    execute: (tool: any, input: any, options?: {
+    execute: (tool: ExecutableTool, input: unknown, options?: {
         priority?: Priority$1;
-    }) => Promise<any>;
-    executeParallel: (executions: ToolExecution[]) => Promise<any[]>;
+    }) => Promise<unknown>;
+    executeParallel: (executions: ToolExecution[]) => Promise<unknown[]>;
     getMetrics: () => ExecutionMetrics;
     resetMetrics: () => void;
     getQueueStatus: () => {
@@ -4986,22 +4996,22 @@ interface HttpPoolConfig extends PoolConfig {
     keepAliveMsecs?: number;
 }
 interface HttpClient {
-    get<T = any>(url: string, config?: RequestConfig): Promise<HttpResponse<T>>;
-    post<T = any>(url: string, data?: any, config?: RequestConfig): Promise<HttpResponse<T>>;
-    put<T = any>(url: string, data?: any, config?: RequestConfig): Promise<HttpResponse<T>>;
-    delete<T = any>(url: string, config?: RequestConfig): Promise<HttpResponse<T>>;
-    request<T = any>(config: RequestConfig): Promise<HttpResponse<T>>;
+    get<T = unknown>(url: string, config?: RequestConfig): Promise<HttpResponse<T>>;
+    post<T = unknown, TData = unknown>(url: string, data?: TData, config?: RequestConfig<TData>): Promise<HttpResponse<T>>;
+    put<T = unknown, TData = unknown>(url: string, data?: TData, config?: RequestConfig<TData>): Promise<HttpResponse<T>>;
+    delete<T = unknown>(url: string, config?: RequestConfig): Promise<HttpResponse<T>>;
+    request<T = unknown, TData = unknown>(config: RequestConfig<TData>): Promise<HttpResponse<T>>;
     close(): Promise<void>;
 }
-interface RequestConfig {
+interface RequestConfig<TData = unknown> {
     url?: string;
     method?: string;
     headers?: Record<string, string>;
-    params?: Record<string, any>;
-    data?: any;
+    params?: Record<string, unknown>;
+    data?: TData;
     timeout?: number;
 }
-interface HttpResponse<T = any> {
+interface HttpResponse<T = unknown> {
     data: T;
     status: number;
     statusText: string;
@@ -5023,7 +5033,7 @@ declare class HttpPool {
     constructor(options: HttpPoolOptions);
     acquire(): Promise<HttpClient>;
     release(client: HttpClient): Promise<void>;
-    request<T = any>(config: RequestConfig): Promise<HttpResponse<T>>;
+    request<T = unknown, TData = unknown>(config: RequestConfig<TData>): Promise<HttpResponse<T>>;
     drain(): Promise<void>;
     clear(): Promise<void>;
     getStats(): PoolStats;

package/dist/index.js

@@ -686,6 +686,9 @@ Examples:`);
   return parts.join("\n");
 }
 
+// src/tools/registry.ts
+import { z as z3 } from "zod";
+
 // src/langgraph/observability/logger.ts
 var LogLevel = /* @__PURE__ */ ((LogLevel2) => {
   LogLevel2["DEBUG"] = "debug";
@@ -790,6 +793,9 @@ var RegistryEvent = /* @__PURE__ */ ((RegistryEvent2) => {
   RegistryEvent2["REGISTRY_CLEARED"] = "registry:cleared";
   return RegistryEvent2;
 })(RegistryEvent || {});
+function eraseToolType(tool) {
+  return tool;
+}
 var ToolRegistry = class {
   tools = /* @__PURE__ */ new Map();
   eventHandlers = /* @__PURE__ */ new Map();
@@ -805,13 +811,14 @@ var ToolRegistry = class {
    * ```
    */
   register(tool) {
+    const erasedTool = eraseToolType(tool);
     const name = tool.metadata.name;
     if (this.tools.has(name)) {
       throw new Error(
         `Tool with name "${name}" is already registered. Use update() to modify it.`
       );
     }
-    this.tools.set(name, tool);
+    this.tools.set(name, erasedTool);
     this.emit("tool:registered" /* TOOL_REGISTERED */, tool);
   }
   /**
@@ -882,6 +889,7 @@ var ToolRegistry = class {
    * ```
    */
   update(name, tool) {
+    const erasedTool = eraseToolType(tool);
     if (!this.tools.has(name)) {
       return false;
     }
@@ -890,7 +898,7 @@ var ToolRegistry = class {
         `Cannot update tool: metadata.name "${tool.metadata.name}" does not match registry key "${name}". To rename a tool, remove it and register it again with the new name.`
       );
     }
-    this.tools.set(name, tool);
+    this.tools.set(name, erasedTool);
     this.emit("tool:updated" /* TOOL_UPDATED */, { name, tool });
     return true;
   }
@@ -964,7 +972,7 @@ var ToolRegistry = class {
   /**
    * Register multiple tools at once
    *
-   * @param tools - Array of tools to register
+   * @param tools - Iterable of tools to register
    * @throws Error if any tool name conflicts with existing tools
    *
    * @example
@@ -973,9 +981,10 @@ var ToolRegistry = class {
    * ```
    */
   registerMany(tools) {
+    const toolsToRegister = Array.from(tools);
     const inputNames = /* @__PURE__ */ new Set();
     const duplicatesInInput = [];
-    for (const tool of tools) {
+    for (const tool of toolsToRegister) {
       const name = tool.metadata.name;
       if (inputNames.has(name)) {
         duplicatesInInput.push(name);
@@ -989,7 +998,7 @@ var ToolRegistry = class {
       );
     }
     const conflicts = [];
-    for (const tool of tools) {
+    for (const tool of toolsToRegister) {
       if (this.tools.has(tool.metadata.name)) {
         conflicts.push(tool.metadata.name);
       }
@@ -999,7 +1008,7 @@ var ToolRegistry = class {
         `Cannot register tools: the following names already exist: ${conflicts.join(", ")}`
       );
     }
-    for (const tool of tools) {
+    for (const tool of toolsToRegister) {
       this.register(tool);
     }
   }
@@ -1262,13 +1271,14 @@ var ToolRegistry = class {
       return lines;
     }
     lines.push(`- ${metadata.name}: ${metadata.description}`);
-    const schemaShape = tool.schema._def?.shape?.();
+    const schemaShape = this.getSchemaShape(tool.schema);
     if (schemaShape) {
       const params = Object.keys(schemaShape);
       if (params.length > 0) {
         const paramDescriptions = params.map((param) => {
           const field = schemaShape[param];
-          const type = field._def?.typeName?.replace("Zod", "").toLowerCase() || "any";
+          const typeName = field._def.typeName;
+          const type = typeName.replace("Zod", "").toLowerCase();
           return `${param} (${type})`;
         });
         lines.push(`  Parameters: ${paramDescriptions.join(", ")}`);
@@ -1328,6 +1338,12 @@ var ToolRegistry = class {
     }
     return lines;
   }
+  getSchemaShape(schema) {
+    if (schema instanceof z3.ZodObject) {
+      return schema.shape;
+    }
+    return void 0;
+  }
 };
 
 // src/tools/executor.ts
@@ -1375,6 +1391,12 @@ function createToolExecutor(config = {}) {
     }
     return Math.min(delay, maxDelay);
   }
+  function toError(error) {
+    if (error instanceof Error) {
+      return error;
+    }
+    return new Error(String(error));
+  }
   async function executeWithRetry(tool, input, policy) {
     const executeFn = tool.invoke || tool.execute;
     if (!executeFn) {
@@ -1390,18 +1412,24 @@ function createToolExecutor(config = {}) {
     if (!policy) {
       return await executeFn.call(tool, input);
     }
+    if (!Number.isInteger(policy.maxAttempts) || policy.maxAttempts < 1) {
+      throw new Error(
+        `Invalid retry policy: maxAttempts must be an integer >= 1 (received ${String(policy.maxAttempts)})`
+      );
+    }
     let lastError;
     for (let attempt = 1; attempt <= policy.maxAttempts; attempt++) {
       try {
         return await executeFn.call(tool, input);
       } catch (error) {
-        lastError = error;
+        const currentError = toError(error);
+        lastError = currentError;
         if (policy.retryableErrors && policy.retryableErrors.length > 0) {
           const isRetryable = policy.retryableErrors.some(
-            (msg) => lastError.message.includes(msg)
+            (msg) => currentError.message.includes(msg)
           );
           if (!isRetryable) {
-            throw lastError;
+            throw currentError;
           }
         }
         if (attempt < policy.maxAttempts) {
@@ -1410,7 +1438,7 @@ function createToolExecutor(config = {}) {
         }
       }
     }
-    throw lastError;
+    throw lastError ?? new Error("Tool execution failed after retries");
   }
   async function executeSingle(tool, input, priority) {
     const startTime = Date.now();
@@ -1432,13 +1460,14 @@ function createToolExecutor(config = {}) {
       return result;
     } catch (error) {
       const duration = Date.now() - startTime;
+      const normalizedError = toError(error);
       metrics.totalExecutions++;
       metrics.failedExecutions++;
       metrics.totalDuration += duration;
       metrics.averageDuration = metrics.totalDuration / metrics.totalExecutions;
       metrics.byPriority[priority]++;
-      onExecutionError?.(tool, input, error, duration);
-      throw error;
+      onExecutionError?.(tool, input, normalizedError, duration);
+      throw normalizedError;
     }
   }
   async function processQueue() {
@@ -1900,8 +1929,8 @@ function createToolSimulator(config) {
     if (!latency) return 0;
     const u1 = Math.random();
     const u2 = Math.random();
-    const z3 = Math.sqrt(-2 * Math.log(u1)) * Math.cos(2 * Math.PI * u2);
-    return Math.max(0, latency.mean + z3 * latency.stddev);
+    const z4 = Math.sqrt(-2 * Math.log(u1)) * Math.cos(2 * Math.PI * u2);
+    return Math.max(0, latency.mean + z4 * latency.stddev);
   }
   return {
     execute: async (toolName, input) => {
@@ -4329,8 +4358,8 @@ function createDatabasePool(options) {
 
 // src/resources/http-pool.ts
 var MockHttpClient = class {
-  constructor(config) {
-    this.config = config;
+  constructor(_config) {
+    this._config = _config;
   }
   closed = false;
   async get(url, config) {
@@ -4345,7 +4374,7 @@ var MockHttpClient = class {
   async delete(url, config) {
     return this.request({ ...config, url, method: "DELETE" });
   }
-  async request(config) {
+  async request(_config) {
     if (this.closed) {
       throw new Error("Client is closed");
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentforge/core",
-  "version": "0.15.3",
+  "version": "0.15.5",
   "description": "Production-ready TypeScript agent framework built on LangGraph with orchestration, middleware, and typed abstractions.",
   "type": "module",
   "main": "./dist/index.cjs",