@juspay/neurolink 6.1.0 → 6.2.1

package/CHANGELOG.md

@@ -1,21 +1,31 @@
-# [6.1.0](https://github.com/juspay/neurolink/compare/v6.0.0...v6.1.0) (2025-07-24)
+## [6.2.1](https://github.com/juspay/neurolink/compare/v6.2.0...v6.2.1) (2025-07-31)
 
 
-### Features
+### Bug Fixes
 
-* **github:** enhance GitHub project configuration and community features ([deb1407](https://github.com/juspay/neurolink/commit/deb1407cb8c7be7eff4baf365f6600da33ac4255))
+* **logging:** consolidate MCP logging and add debug flag control ([ea0132d](https://github.com/juspay/neurolink/commit/ea0132dd954966cb42238dc3736f6cee9cc7b18d))
 
-# [6.0.0](https://github.com/juspay/neurolink/compare/v5.3.0...v6.0.0) (2025-07-24)
+# [6.2.0](https://github.com/juspay/neurolink/compare/v6.1.0...v6.2.0) (2025-07-30)
+
+### Features
 
+- systematic dead code elimination across entire codebase ([571060a](https://github.com/juspay/neurolink/commit/571060a6146dc13e486da22610122d599420fcb2))
+
+# [6.1.0](https://github.com/juspay/neurolink/compare/v6.0.0...v6.1.0) (2025-07-24)
 
 ### Features
 
-* **types:** eliminate all TypeScript any usage across entire codebase ([777c3cd](https://github.com/juspay/neurolink/commit/777c3cda582cbefcf01480a12d13a2adb7c140c8))
+- **github:** enhance GitHub project configuration and community features ([deb1407](https://github.com/juspay/neurolink/commit/deb1407cb8c7be7eff4baf365f6600da33ac4255))
+
+# [6.0.0](https://github.com/juspay/neurolink/compare/v5.3.0...v6.0.0) (2025-07-24)
+
+### Features
 
+- **types:** eliminate all TypeScript any usage across entire codebase ([777c3cd](https://github.com/juspay/neurolink/commit/777c3cda582cbefcf01480a12d13a2adb7c140c8))
 
 ### BREAKING CHANGES
 
-* **types:** Complete removal of TypeScript 'any' types for enhanced type safety
+- **types:** Complete removal of TypeScript 'any' types for enhanced type safety
 
 This comprehensive refactor eliminates all TypeScript 'any' usage across the entire
 NeuroLink codebase, affecting 140+ files with systematic type safety improvements:

package/dist/agent/direct-tools.js

@@ -6,6 +6,7 @@ import { tool } from "ai";
 import { z } from "zod";
 import * as fs from "fs";
 import * as path from "path";
+import { logger } from "../utils/logger.js";
 /**
  * Direct tool definitions that work immediately with Gemini/AI SDK
  * These bypass MCP complexity and provide reliable agent functionality
@@ -365,23 +366,23 @@ export function validateToolStructure() {
     try {
         for (const [name, tool] of Object.entries(directAgentTools)) {
             if (!tool.description || typeof tool.description !== "string") {
-                console.error(`❌ Tool ${name} missing description`);
+                logger.error(`❌ Tool ${name} missing description`);
                 return false;
             }
             if (!tool.parameters) {
-                console.error(`❌ Tool ${name} missing parameters`);
+                logger.error(`❌ Tool ${name} missing parameters`);
                 return false;
             }
             if (!tool.execute || typeof tool.execute !== "function") {
-                console.error(`❌ Tool ${name} missing execute function`);
+                logger.error(`❌ Tool ${name} missing execute function`);
                 return false;
             }
         }
-        console.log("✅ All tools have valid structure");
+        logger.info("✅ All tools have valid structure");
         return true;
     }
     catch (error) {
-        console.error("❌ Tool validation failed:", error);
+        logger.error("❌ Tool validation failed:", error);
         return false;
     }
 }

package/dist/cli/commands/config.d.ts

@@ -58,9 +58,9 @@ declare const ConfigSchema: z.ZodObject<{
             clientEmail?: string | undefined;
             privateKey?: string | undefined;
         }, {
-            projectId?: string | undefined;
             model?: string | undefined;
             location?: string | undefined;
+            projectId?: string | undefined;
             credentials?: string | undefined;
             serviceAccountKey?: string | undefined;
             clientEmail?: string | undefined;
@@ -117,12 +117,12 @@ declare const ConfigSchema: z.ZodObject<{
             model: z.ZodDefault<z.ZodString>;
             timeout: z.ZodDefault<z.ZodNumber>;
         }, "strip", z.ZodTypeAny, {
-            timeout: number;
             model: string;
+            timeout: number;
             baseUrl: string;
         }, {
-            timeout?: number | undefined;
             model?: string | undefined;
+            timeout?: number | undefined;
             baseUrl?: string | undefined;
         }>>;
         mistral: z.ZodOptional<z.ZodObject<{
@@ -176,8 +176,8 @@ declare const ConfigSchema: z.ZodObject<{
             apiKey?: string | undefined;
         } | undefined;
         ollama?: {
-            timeout: number;
             model: string;
+            timeout: number;
             baseUrl: string;
         } | undefined;
         mistral?: {
@@ -198,9 +198,9 @@ declare const ConfigSchema: z.ZodObject<{
             baseURL?: string | undefined;
         } | undefined;
         vertex?: {
-            projectId?: string | undefined;
             model?: string | undefined;
             location?: string | undefined;
+            projectId?: string | undefined;
             credentials?: string | undefined;
             serviceAccountKey?: string | undefined;
             clientEmail?: string | undefined;
@@ -225,8 +225,8 @@ declare const ConfigSchema: z.ZodObject<{
             apiKey?: string | undefined;
         } | undefined;
         ollama?: {
-            timeout?: number | undefined;
             model?: string | undefined;
+            timeout?: number | undefined;
             baseUrl?: string | undefined;
         } | undefined;
         mistral?: {
@@ -243,15 +243,15 @@ declare const ConfigSchema: z.ZodObject<{
         enableCaching: z.ZodDefault<z.ZodBoolean>;
         cacheStrategy: z.ZodDefault<z.ZodEnum<["memory", "file", "redis"]>>;
     }, "strip", z.ZodTypeAny, {
-        temperature: number;
         maxTokens: number;
+        temperature: number;
         outputFormat: "text" | "json" | "yaml";
         enableLogging: boolean;
         enableCaching: boolean;
         cacheStrategy: "file" | "memory" | "redis";
     }, {
-        temperature?: number | undefined;
         maxTokens?: number | undefined;
+        temperature?: number | undefined;
         outputFormat?: "text" | "json" | "yaml" | undefined;
         enableLogging?: boolean | undefined;
         enableCaching?: boolean | undefined;
@@ -300,8 +300,8 @@ declare const ConfigSchema: z.ZodObject<{
             apiKey?: string | undefined;
         } | undefined;
         ollama?: {
-            timeout: number;
             model: string;
+            timeout: number;
             baseUrl: string;
         } | undefined;
         mistral?: {
@@ -311,8 +311,8 @@ declare const ConfigSchema: z.ZodObject<{
     };
     profiles: Record<string, any>;
     preferences: {
-        temperature: number;
         maxTokens: number;
+        temperature: number;
         outputFormat: "text" | "json" | "yaml";
         enableLogging: boolean;
         enableCaching: boolean;
@@ -334,9 +334,9 @@ declare const ConfigSchema: z.ZodObject<{
             baseURL?: string | undefined;
         } | undefined;
         vertex?: {
-            projectId?: string | undefined;
             model?: string | undefined;
             location?: string | undefined;
+            projectId?: string | undefined;
             credentials?: string | undefined;
             serviceAccountKey?: string | undefined;
             clientEmail?: string | undefined;
@@ -361,8 +361,8 @@ declare const ConfigSchema: z.ZodObject<{
             apiKey?: string | undefined;
         } | undefined;
         ollama?: {
-            timeout?: number | undefined;
             model?: string | undefined;
+            timeout?: number | undefined;
             baseUrl?: string | undefined;
         } | undefined;
         mistral?: {
@@ -372,8 +372,8 @@ declare const ConfigSchema: z.ZodObject<{
     } | undefined;
     profiles?: Record<string, any> | undefined;
     preferences?: {
-        temperature?: number | undefined;
         maxTokens?: number | undefined;
+        temperature?: number | undefined;
         outputFormat?: "text" | "json" | "yaml" | undefined;
         enableLogging?: boolean | undefined;
         enableCaching?: boolean | undefined;

package/dist/cli/index.js

@@ -19,9 +19,7 @@ import { hideBin } from "yargs/helpers";
 import ora from "ora";
 import chalk from "chalk";
 import fs from "fs";
-import { addMCPCommands } from "./commands/mcp.js";
 import { addOllamaCommands } from "./commands/ollama.js";
-import { CLICommandFactory } from "./factories/command-factory.js";
 import { logger } from "../lib/utils/logger.js";
 function displayDebugInfo(title, data, debug) {
     if (debug) {
@@ -194,13 +192,10 @@ function handleError(error, context) {
 async function initializeCLI() {
     // Import and configure for CLI mode
     const { ProviderRegistry } = await import("../lib/factories/provider-registry.js");
-    const { initializeNeuroLinkMCP } = await import("../lib/mcp/initialize.js");
     // Enable manual MCP only for CLI
     ProviderRegistry.setOptions({
         enableManualMCP: true,
     });
-    // Initialize MCP system
-    await initializeNeuroLinkMCP();
     logger.debug("CLI initialized with manual MCP support enabled");
 }
 // Initialize SDK
@@ -1158,7 +1153,7 @@ const cli = yargs(args)
     .example("$0 get-best-provider", "Show best provider")
     .example("$0 get-best-provider --format json", "Show in JSON format"), async (argv) => {
     try {
-        const { getBestProvider } = await import("../lib/utils/providerUtils-fixed.js");
+        const { getBestProvider } = await import("../lib/utils/providerUtils.js");
         const bestProvider = await getBestProvider();
         if (argv.format === "json") {
             process.stdout.write(JSON.stringify({ provider: bestProvider }, null, 2) + "\n");
@@ -1184,9 +1179,8 @@ const cli = yargs(args)
     cli.showCompletionScript();
 });
 // Add NEW Generate Command (Primary)
-cli.command(CLICommandFactory.createGenerateCommand());
-// Add MCP Commands
-addMCPCommands(cli);
+// Removed CLICommandFactory call - commands are handled directly above.createGenerateCommand());
+// MCP Commands: Integrated within base provider functionality
 // Add Ollama Commands
 addOllamaCommands(cli);
 // Execute CLI

package/dist/config/configManager.js

@@ -5,6 +5,7 @@
 import { promises as fs } from "fs";
 import path from "path";
 import crypto from "crypto";
+import { logger } from "../utils/logger.js";
 const { readFile, writeFile, readdir, mkdir, unlink, access } = fs;
 /**
  * Enhanced Config Manager with automatic backup/restore capabilities
@@ -32,7 +33,7 @@ export class NeuroLinkConfigManager {
         if (createBackup) {
             await this.createBackup(reason);
             if (!silent) {
-                console.log("💾 Backup created before config update");
+                logger.info("💾 Backup created before config update");
             }
         }
         const existing = await this.loadConfig();
@@ -50,7 +51,7 @@ export class NeuroLinkConfigManager {
         try {
             await this.persistConfig(this.config);
             if (!silent) {
-                console.log("✅ Configuration updated successfully");
+                logger.info("✅ Configuration updated successfully");
             }
         }
         catch (error) {
@@ -58,7 +59,7 @@ export class NeuroLinkConfigManager {
             if (createBackup) {
                 await this.restoreLatestBackup();
                 if (!silent) {
-                    console.log("🔄 Auto-restored from backup due to error");
+                    logger.info("🔄 Auto-restored from backup due to error");
                 }
             }
             throw new Error(`Config update failed, restored from backup: ${error.message}`);
@@ -115,13 +116,13 @@ export default ${JSON.stringify(currentConfig, null, 2)};`;
                     });
                 }
                 catch (error) {
-                    console.warn(`Failed to read backup ${file}:`, error.message);
+                    logger.warn(`Failed to read backup ${file}:`, error.message);
                 }
             }
             return backups.sort((a, b) => b.metadata.timestamp - a.metadata.timestamp);
         }
         catch (error) {
-            console.warn("Failed to list backups:", error.message);
+            logger.warn("Failed to list backups:", error.message);
             return [];
         }
     }
@@ -142,7 +143,7 @@ export default ${JSON.stringify(currentConfig, null, 2)};`;
             }
             this.config = restoredConfig;
             await this.persistConfig(this.config);
-            console.log(`✅ Config restored from backup: ${backupFilename}`);
+            logger.info(`✅ Config restored from backup: ${backupFilename}`);
         }
         catch (error) {
             throw new Error(`Failed to restore from backup ${backupFilename}: ${error.message}`);
@@ -167,10 +168,10 @@ export default ${JSON.stringify(currentConfig, null, 2)};`;
         for (const backup of toDelete) {
             try {
                 await unlink(backup.path);
-                console.log(`🗑️ Deleted old backup: ${backup.filename}`);
+                logger.info(`🗑️ Deleted old backup: ${backup.filename}`);
             }
             catch (error) {
-                console.warn(`Failed to delete backup ${backup.filename}:`, error.message);
+                logger.warn(`Failed to delete backup ${backup.filename}:`, error.message);
             }
         }
     }
@@ -248,7 +249,7 @@ export default ${JSON.stringify(currentConfig, null, 2)};`;
             await mkdir(this.backupDir, { recursive: true });
         }
         catch (error) {
-            console.warn("Failed to create backup directory:", error.message);
+            logger.warn("Failed to create backup directory:", error.message);
         }
     }
     async readConfigFile() {
@@ -265,7 +266,7 @@ export default ${JSON.stringify(currentConfig, null, 2)};`;
             throw new Error("Invalid config file format");
         }
         catch (error) {
-            console.log("Config file not found, generating default...");
+            logger.info("Config file not found, generating default...");
             return await this.generateDefaultConfig();
         }
     }

package/dist/core/analytics.d.ts

@@ -17,8 +17,18 @@ export interface AnalyticsData {
     responseTime: number;
     context?: Record<string, JsonValue>;
     timestamp: string;
+    evaluation?: {
+        relevanceScore: number;
+        accuracyScore: number;
+        completenessScore: number;
+        overall: number;
+        evaluationProvider?: string;
+        evaluationTime?: number;
+        evaluationAttempt?: number;
+    };
+    costDetails?: UnknownRecord;
 }
 /**
  * Create analytics data structure from AI response
  */
-export declare function createAnalytics(provider: string, model: string, result: UnknownRecord, responseTime: number, context?: Record<string, JsonValue>): AnalyticsData;
+export declare function createAnalytics(provider: string, model: string, result: unknown, responseTime: number, context?: Record<string, unknown>): AnalyticsData;

package/dist/core/analytics.js

@@ -21,7 +21,7 @@ export function createAnalytics(provider, model, result, responseTime, context)
             tokens,
             cost,
             responseTime,
-            context,
+            context: context,
             timestamp: new Date().toISOString(),
         };
         logger.debug(`[${functionTag}] Analytics created`, {
@@ -41,7 +41,7 @@ export function createAnalytics(provider, model, result, responseTime, context)
             model,
             tokens: { input: 0, output: 0, total: 0 },
             responseTime,
-            context,
+            context: context,
             timestamp: new Date().toISOString(),
         };
     }

package/dist/core/base-provider.js

@@ -260,24 +260,10 @@ export class BaseProvider {
                 // Not an error - custom tools are optional
             }
         }
-        // Try to load MCP tools if not already loaded
+        // MCP tools loading simplified - removed function-calling dependency
         if (!this.mcpTools) {
-            try {
-                const { getAvailableFunctionTools } = await import("../mcp/function-calling.js");
-                const result = await getAvailableFunctionTools();
-                if (isValidToolsObject(result)) {
-                    this.mcpTools = result.toolsObject;
-                }
-                else {
-                    logger.debug(`Invalid or empty toolsObject for ${this.providerName}: Expected an object with at least one key, but got ${typeof result?.toolsObject} with ${result?.toolsObject
-                        ? Object.keys(result.toolsObject).length
-                        : 0} keys. Full result:`, result);
-                }
-            }
-            catch (error) {
-                logger.debug(`MCP tools not available for ${this.providerName}:`, error);
-                // Not an error - MCP tools are optional
-            }
+            // Set empty tools object - MCP tools are handled at a higher level
+            this.mcpTools = {};
         }
         // Add MCP tools if available
         if (this.mcpTools) {
@@ -353,7 +339,7 @@ export class BaseProvider {
         return enhancedResult;
     }
     async createAnalytics(result, responseTime, options) {
-        const { createAnalytics } = await import("../providers/analytics-helper.js");
+        const { createAnalytics } = await import("./analytics.js");
         return createAnalytics(this.providerName, this.modelName, result, responseTime, options.context);
     }
     async createEvaluation(result, options) {

package/dist/core/dynamic-models.d.ts

@@ -21,8 +21,8 @@ declare const ModelConfigSchema: z.ZodObject<{
     releaseDate: z.ZodString;
 }, "strip", z.ZodTypeAny, {
     id: string;
-    capabilities: string[];
     displayName: string;
+    capabilities: string[];
     deprecated: boolean;
     pricing: {
         input: number;
@@ -32,8 +32,8 @@ declare const ModelConfigSchema: z.ZodObject<{
     releaseDate: string;
 }, {
     id: string;
-    capabilities: string[];
     displayName: string;
+    capabilities: string[];
     deprecated: boolean;
     pricing: {
         input: number;
@@ -64,8 +64,8 @@ declare const ModelRegistrySchema: z.ZodObject<{
         releaseDate: z.ZodString;
     }, "strip", z.ZodTypeAny, {
         id: string;
-        capabilities: string[];
         displayName: string;
+        capabilities: string[];
         deprecated: boolean;
         pricing: {
             input: number;
@@ -75,8 +75,8 @@ declare const ModelRegistrySchema: z.ZodObject<{
         releaseDate: string;
     }, {
         id: string;
-        capabilities: string[];
         displayName: string;
+        capabilities: string[];
         deprecated: boolean;
         pricing: {
             input: number;
@@ -89,11 +89,10 @@ declare const ModelRegistrySchema: z.ZodObject<{
     defaults: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodString>>;
 }, "strip", z.ZodTypeAny, {
     version: string;
-    lastUpdated: string;
     models: Record<string, Record<string, {
         id: string;
-        capabilities: string[];
         displayName: string;
+        capabilities: string[];
         deprecated: boolean;
         pricing: {
             input: number;
@@ -102,15 +101,15 @@ declare const ModelRegistrySchema: z.ZodObject<{
         contextWindow: number;
         releaseDate: string;
     }>>;
+    lastUpdated: string;
     aliases?: Record<string, string> | undefined;
     defaults?: Record<string, string> | undefined;
 }, {
     version: string;
-    lastUpdated: string;
     models: Record<string, Record<string, {
         id: string;
-        capabilities: string[];
         displayName: string;
+        capabilities: string[];
         deprecated: boolean;
         pricing: {
             input: number;
@@ -119,6 +118,7 @@ declare const ModelRegistrySchema: z.ZodObject<{
         contextWindow: number;
         releaseDate: string;
     }>>;
+    lastUpdated: string;
     aliases?: Record<string, string> | undefined;
     defaults?: Record<string, string> | undefined;
 }>;

package/dist/core/factory.js

@@ -89,13 +89,12 @@ export class AIProviderFactory {
             //     );
             //   }
             // }
-            let provider;
             // PURE FACTORY PATTERN: No switch statements - use ProviderFactory exclusively
             const normalizedName = this.normalizeProviderName(providerName);
             const finalModelName = resolvedModelName === "default" || resolvedModelName === null
                 ? undefined
                 : resolvedModelName;
-            provider = await ProviderFactory.createProvider(normalizedName, finalModelName, sdk);
+            const provider = await ProviderFactory.createProvider(normalizedName, finalModelName, sdk);
             logger.debug(componentIdentifier, "Pure factory pattern provider created", {
                 providerName: normalizedName,
                 modelName: finalModelName,
@@ -105,15 +104,8 @@ export class AIProviderFactory {
             // Wrap with MCP if enabled
             if (enableMCP) {
                 try {
-                    logger.debug(`[${functionTag}] Enabling MCP wrapping for AI integration`);
-                    const { createMCPAwareProviderV3 } = await import("../providers/function-calling-provider.js");
-                    provider = createMCPAwareProviderV3(provider, {
-                        providerName,
-                        modelName: resolvedModelName || undefined,
-                        enableMCP: true,
-                        enableFunctionCalling: true,
-                    });
-                    logger.debug(`[${functionTag}] Provider wrapped with MCP support`);
+                    logger.debug(`[${functionTag}] MCP wrapping disabled - function-calling removed`);
+                    // MCP wrapping simplified - removed function-calling dependency
                 }
                 catch (mcpError) {
                     logger.warn(`[${functionTag}] Failed to wrap with MCP, using base provider`, {

package/dist/index.d.ts

@@ -10,8 +10,6 @@ import { AIProviderFactory } from "./core/factory.js";
 export { AIProviderFactory };
 export type { AIProvider, AIProviderName, ProviderConfig, StreamingOptions, ProviderAttempt, SupportedModelName, } from "./core/types.js";
 export type { GenerateOptions, GenerateResult, EnhancedProvider, } from "./types/generate-types.js";
-export { CompatibilityConversionFactory } from "./factories/compatibility-factory.js";
-export { ProviderGenerateFactory } from "./factories/provider-generate-factory.js";
 export type { SimpleTool, ToolContext } from "./sdk/tool-registration.js";
 export { createTool, createTypedTool, validateTool, } from "./sdk/tool-registration.js";
 export type { InMemoryMCPServerConfig, InMemoryToolInfo, InMemoryToolResult, } from "./types/mcp-types.js";
@@ -78,8 +76,8 @@ export declare function createBestAIProvider(requestedProvider?: string, modelNa
  * await writeFile('output.txt', 'Hello from MCP!');
  * ```
  */
-export { MCPEcosystem, mcpEcosystem, initializeMCPEcosystem, PluginManager, pluginManager, listMCPs, executeMCP, getMCPStats, readFile, writeFile, listFiles, createDirectory, MCP, SecurityManager, mcpLogger, FileSystemMCP, } from "./mcp/index.js";
-export type { MCPMetadata, ExecutionContext, MCPConstructor, MCPInstance, DiscoveredMCP, LogLevel, } from "./mcp/index.js";
+export { initializeMCPEcosystem, listMCPs, executeMCP, getMCPStats, mcpLogger, } from "./mcp/index.js";
+export type { McpMetadata, ExecutionContext, DiscoveredMcp, ToolInfo, ToolExecutionResult, LogLevel, } from "./mcp/index.js";
 export declare function initializeTelemetry(): Promise<boolean>;
 export declare function getTelemetryStatus(): {
     enabled: boolean;

package/dist/index.js

@@ -9,8 +9,6 @@
 // Core exports
 import { AIProviderFactory } from "./core/factory.js";
 export { AIProviderFactory };
-export { CompatibilityConversionFactory } from "./factories/compatibility-factory.js";
-export { ProviderGenerateFactory } from "./factories/provider-generate-factory.js";
 export { createTool, createTypedTool, validateTool, } from "./sdk/tool-registration.js";
 // Model enums
 export { BedrockModels, OpenAIModels, VertexModels, DEFAULT_PROVIDER_CONFIGS, } from "./core/types.js";
@@ -86,15 +84,8 @@ export async function createBestAIProvider(requestedProvider, modelName) {
  */
 export { 
 // Core MCP ecosystem
-MCPEcosystem, mcpEcosystem, initializeMCPEcosystem, 
-// Plugin management
-PluginManager, pluginManager, listMCPs, executeMCP, getMCPStats, 
-// Quick filesystem operations
-readFile, writeFile, listFiles, createDirectory, 
-// Core contracts and types
-MCP, SecurityManager, mcpLogger, 
-// Core plugins
-FileSystemMCP, } from "./mcp/index.js";
+// Simplified MCP exports
+initializeMCPEcosystem, listMCPs, executeMCP, getMCPStats, mcpLogger, } from "./mcp/index.js";
 // ============================================================================
 // REAL-TIME SERVICES & TELEMETRY - Enterprise Platform Features
 // ============================================================================

package/dist/lib/agent/direct-tools.js

@@ -6,6 +6,7 @@ import { tool } from "ai";
 import { z } from "zod";
 import * as fs from "fs";
 import * as path from "path";
+import { logger } from "../utils/logger.js";
 /**
  * Direct tool definitions that work immediately with Gemini/AI SDK
  * These bypass MCP complexity and provide reliable agent functionality
@@ -365,23 +366,23 @@ export function validateToolStructure() {
     try {
         for (const [name, tool] of Object.entries(directAgentTools)) {
             if (!tool.description || typeof tool.description !== "string") {
-                console.error(`❌ Tool ${name} missing description`);
+                logger.error(`❌ Tool ${name} missing description`);
                 return false;
             }
             if (!tool.parameters) {
-                console.error(`❌ Tool ${name} missing parameters`);
+                logger.error(`❌ Tool ${name} missing parameters`);
                 return false;
             }
             if (!tool.execute || typeof tool.execute !== "function") {
-                console.error(`❌ Tool ${name} missing execute function`);
+                logger.error(`❌ Tool ${name} missing execute function`);
                 return false;
             }
         }
-        console.log("✅ All tools have valid structure");
+        logger.info("✅ All tools have valid structure");
         return true;
     }
     catch (error) {
-        console.error("❌ Tool validation failed:", error);
+        logger.error("❌ Tool validation failed:", error);
         return false;
     }
 }