onbuzz 4.9.13 → 4.10.0

package/src/tools/baseTool.js

@@ -12,7 +12,6 @@
 import {
   TOOL_STATUS,
   OPERATION_STATUS,
-  ERROR_TYPES,
   SYSTEM_DEFAULTS
 } from '../utilities/constants.js';
 import { NATIVE_SCHEMA_TOOL_NAMES } from './openaiFunctionSchemas.js';
@@ -101,6 +100,7 @@ class BaseTool {
    * @param {string} content - Raw tool command content
    * @returns {Object} Parsed parameters object
    */
+  // eslint-disable-next-line no-unused-vars
   parseParameters(content) {
     throw new Error(`Tool ${this.id} must implement parseParameters()`);
   }
@@ -112,6 +112,7 @@ class BaseTool {
    * @param {Object} context - Execution context
    * @returns {Promise<*>} Execution result
    */
+  // eslint-disable-next-line no-unused-vars
   async execute(params, context) {
     throw new Error(`Tool ${this.id} must implement execute()`);
   }
@@ -326,6 +327,7 @@ class BaseTool {
    * @param {string} operationId - Operation identifier
    * @returns {Promise<void>}
    */
+  // eslint-disable-next-line no-unused-vars
   async cleanup(operationId) {
     // Default implementation - no cleanup needed
   }
@@ -367,6 +369,7 @@ class BaseTool {
    * @param {Object} params - Parameters to validate
    * @returns {Object} Validation result
    */
+  // eslint-disable-next-line no-unused-vars
   validateParameterTypes(params) {
     // Default implementation - all parameters are valid
     return { valid: true };
@@ -378,6 +381,7 @@ class BaseTool {
    * @param {Object} params - Parameters to validate
    * @returns {Object} Validation result
    */
+  // eslint-disable-next-line no-unused-vars
   customValidateParameters(params) {
     // Default implementation - no custom validation
     return { valid: true };
@@ -388,19 +392,18 @@ class BaseTool {
    * @private
    */
   async executeWithTimeout(params, context) {
-    return new Promise(async (resolve, reject) => {
+    return new Promise((resolve, reject) => {
       const timeoutId = setTimeout(() => {
         reject(new Error(`Tool execution timed out after ${this.timeout}ms`));
       }, this.timeout);
-      
-      try {
-        const result = await this.execute(params, context);
+
+      this.execute(params, context).then((result) => {
         clearTimeout(timeoutId);
         resolve(result);
-      } catch (error) {
+      }).catch((error) => {
         clearTimeout(timeoutId);
         reject(error);
-      }
+      });
     });
   }
 
@@ -584,6 +587,7 @@ class ToolsRegistry {
    * @param {string} directory - Directory path to scan
    * @returns {Promise<number>} Number of tools discovered
    */
+  // eslint-disable-next-line no-unused-vars
   async discoverTools(directory) {
     // Implementation would scan directory for tool files
     // For now, return 0
@@ -611,7 +615,7 @@ class ToolsRegistry {
       if (typeof testParams !== 'object') {
         throw new Error(`Tool ${tool.id} parseParameters must return an object`);
       }
-    } catch (error) {
+    } catch {
       // Parsing may fail for test content, that's okay
     }
     
@@ -687,9 +691,6 @@ class ToolsRegistry {
    */
   generateToolDescriptionsForPrompt(capabilities = [], options = {}) {
     const {
-      includeExamples = true,
-      includeUsageGuidelines = true,
-      includeSecurityNotes = true,
       compact = false,
       layered = false,
       // 'responses' | 'chat_completion' | undefined.