@juspay/neurolink 7.14.0 → 7.14.2

package/CHANGELOG.md

@@ -1,3 +1,15 @@
+## [7.14.2](https://github.com/juspay/neurolink/compare/v7.14.1...v7.14.2) (2025-08-16)
+
+### Bug Fixes
+
+- **(sdk):** add generateText backward compatibility and fix formatting consistency ([93ff23c](https://github.com/juspay/neurolink/commit/93ff23c766ca71cbe7f77821b35cf5156bbe9d1f))
+
+## [7.14.1](https://github.com/juspay/neurolink/compare/v7.14.0...v7.14.1) (2025-08-15)
+
+### Bug Fixes
+
+- **(mcp):** implement external MCP server integration with real tool execution ([9427a95](https://github.com/juspay/neurolink/commit/9427a95599a829f82e697eaf30388a8f3c899d4f))
+
 ## [7.14.0](https://github.com/juspay/neurolink/compare/v7.13.0...v7.14.0) (2025-08-14)
 
 ### Features

package/dist/agent/directTools.d.ts

@@ -219,12 +219,12 @@ export declare const directAgentTools: {
         content: z.ZodString;
         mode: z.ZodDefault<z.ZodEnum<["create", "overwrite", "append"]>>;
     }, "strip", z.ZodTypeAny, {
-        path: string;
         content: string;
+        path: string;
         mode: "create" | "overwrite" | "append";
     }, {
-        path: string;
         content: string;
+        path: string;
         mode?: "create" | "overwrite" | "append" | undefined;
     }>, {
         success: boolean;
@@ -249,8 +249,8 @@ export declare const directAgentTools: {
         written?: undefined;
     }> & {
         execute: (args: {
-            path: string;
             content: string;
+            path: string;
             mode: "create" | "overwrite" | "append";
         }, options: import("ai").ToolExecutionOptions) => PromiseLike<{
             success: boolean;

package/dist/agent/directTools.js

@@ -163,7 +163,7 @@ export const directAgentTools = {
                     for (const func of allowedMathFunctions) {
                         safeExpression = safeExpression.replace(new RegExp(func, "g"), func);
                     }
-                    // Remove any remaining non-safe characters except Math functions
+                    // Remove remaining non-safe characters except Math functions
                     const mathSafe = /^[0-9+\-*/().\s]|Math\.(abs|ceil|floor|round|sqrt|pow|sin|cos|tan|log|exp|PI|E)/g;
                     if (!safeExpression
                         .split("")

package/dist/cli/commands/mcp.js

@@ -3,6 +3,7 @@
  * Implements comprehensive MCP server management commands
  * Part of Phase 4.2 - MCP CLI Commands
  */
+import { createExternalServerInfo } from "../../lib/utils/mcpDefaults.js";
 import { NeuroLink } from "../../lib/neurolink.js";
 import { logger } from "../../lib/utils/logger.js";
 import chalk from "chalk";
@@ -80,91 +81,6 @@ const POPULAR_MCP_SERVERS = {
         description: "Bitbucket repository management and development workflows",
     },
 };
-/**
- * Type guard to check if an object conforms to MCPStatusResponse interface
- */
-function isMCPStatusResponse(obj) {
-    if (!obj || typeof obj !== "object") {
-        return false;
-    }
-    const record = obj;
-    // Check if autoDiscoveredServers is either undefined or an array
-    if (record.autoDiscoveredServers !== undefined) {
-        if (!Array.isArray(record.autoDiscoveredServers)) {
-            return false;
-        }
-        // Check each server object in the array
-        for (const server of record.autoDiscoveredServers) {
-            if (!server || typeof server !== "object") {
-                return false;
-            }
-            const serverRecord = server;
-            // Optional properties should be correct type if present
-            if (serverRecord.name !== undefined &&
-                typeof serverRecord.name !== "string") {
-                return false;
-            }
-            if (serverRecord.id !== undefined &&
-                typeof serverRecord.id !== "string") {
-                return false;
-            }
-            if (serverRecord.status !== undefined &&
-                typeof serverRecord.status !== "string") {
-                return false;
-            }
-            if (serverRecord.source !== undefined &&
-                typeof serverRecord.source !== "string") {
-                return false;
-            }
-        }
-    }
-    return true;
-}
-/**
- * Convert SDK MCPStatus to CLI format with server list
- */
-function convertMCPStatusForCLI(status, // Use flexible object type
-sdk) {
-    // Create server list from in-memory servers and discovered servers
-    const servers = [];
-    // Add in-memory servers
-    const inMemoryServers = sdk.getInMemoryServers();
-    inMemoryServers.forEach((config, name) => {
-        servers.push({
-            name,
-            connected: true, // In-memory servers are always "connected"
-            description: config.server?.title || "In-memory MCP server",
-            tools: [], // Could extract from config.server.tools if needed
-        });
-    });
-    // Add auto-discovered servers
-    // Cache the type guard result to avoid repeated validation
-    const isValidStatusResponse = isMCPStatusResponse(status);
-    if (isValidStatusResponse && status.autoDiscoveredServers) {
-        status.autoDiscoveredServers.forEach((server) => {
-            servers.push({
-                name: server.name || server.id || "Unknown",
-                connected: server.status === "connected",
-                description: server.source
-                    ? `Auto-discovered from ${server.source}`
-                    : `Auto-discovered server`,
-                tools: [],
-                error: server.status === "failed" ? "Connection failed" : undefined,
-            });
-        });
-    }
-    return {
-        mcpInitialized: status.mcpInitialized,
-        totalServers: status.totalServers,
-        availableServers: status.availableServers,
-        autoDiscoveredCount: status.autoDiscoveredCount,
-        totalTools: status.totalTools,
-        customToolsCount: status.customToolsCount,
-        inMemoryServersCount: status.inMemoryServersCount,
-        error: status.error,
-        servers,
-    };
-}
 /**
  * MCP CLI command factory
  */
@@ -280,7 +196,7 @@ export class MCPCommandFactory {
             demandOption: true,
         })
             .option("transport", {
-            choices: ["stdio", "sse", "ws"],
+            choices: ["stdio", "sse", "websocket"],
             default: "stdio",
             description: "Transport type for MCP communication",
         })
@@ -312,7 +228,7 @@ export class MCPCommandFactory {
             demandOption: true,
         })
             .option("transport", {
-            choices: ["stdio", "sse", "ws"],
+            choices: ["stdio", "sse", "websocket"],
             default: "stdio",
             description: "Transport type for MCP communication",
         })
@@ -393,12 +309,12 @@ export class MCPCommandFactory {
             const spinner = argv.quiet ? null : ora("Loading MCP servers...").start();
             // Get configured servers from NeuroLink
             const sdk = new NeuroLink();
-            const rawMcpStatus = await sdk.getMCPStatus();
-            const mcpStatus = convertMCPStatusForCLI(rawMcpStatus, sdk);
+            const mcpStatus = await sdk.getMCPStatus();
+            const allServers = await sdk.listMCPServers();
             if (spinner) {
-                spinner.succeed(`Found ${mcpStatus.servers.length} MCP servers`);
+                spinner.succeed(`Found ${allServers.length} MCP servers`);
             }
-            if (mcpStatus.servers.length === 0) {
+            if (allServers.length === 0) {
                 logger.always(chalk.yellow("No MCP servers configured."));
                 logger.always(chalk.blue("💡 Use 'neurolink mcp install <server>' to install popular servers"));
                 logger.always(chalk.blue("💡 Use 'neurolink discover' to find existing servers"));
@@ -408,17 +324,19 @@ export class MCPCommandFactory {
             if (argv.format === "json") {
                 logger.always(JSON.stringify(mcpStatus, null, 2));
             }
-            else if (argv.format && argv.format === "compact") {
-                mcpStatus.servers.forEach((server) => {
-                    const status = server.connected ? chalk.green("✓") : chalk.red("✗");
+            else if (argv.format === "compact") {
+                const allServers = await sdk.listMCPServers();
+                allServers.forEach((server) => {
+                    const status = server.status === "connected" ? chalk.green("✓") : chalk.red("✗");
                     logger.always(`${status} ${server.name} - ${server.description || "No description"}`);
                 });
             }
             else {
                 // Table format
                 logger.always(chalk.bold("\n🔧 MCP Servers:\n"));
-                for (const server of mcpStatus.servers) {
-                    const status = server.connected
+                const allServers = await sdk.listMCPServers();
+                for (const server of allServers) {
+                    const status = server.status === "connected"
                         ? chalk.green("CONNECTED")
                         : chalk.red("DISCONNECTED");
                     logger.always(`${chalk.cyan(server.name)} ${status}`);
@@ -474,58 +392,36 @@ export class MCPCommandFactory {
                     process.exit(1);
                 }
             }
-            // Create server configuration
-            const config = {
+            const serverInfo = createExternalServerInfo({
+                ...serverConfig,
+                id: serverName,
                 name: serverName,
-                command: serverConfig.command,
-                args: argv.args || serverConfig.args,
-                env,
-                transport: (argv.transport &&
-                    (argv.transport === "websocket"
-                        ? "ws"
-                        : argv.transport)) ||
-                    serverConfig.transport,
-                description: serverConfig.description,
-                installed: true,
-                status: "unknown",
-            };
-            // Add server to NeuroLink (using in-memory MCP server for now)
-            const sdk = new NeuroLink();
-            await sdk.addInMemoryMCPServer(serverName, {
-                server: {
-                    title: serverConfig.description,
-                    tools: {}, // Empty tools for external servers
-                    description: serverConfig.description,
-                },
-                metadata: {
-                    command: config.command,
-                    args: config.args,
-                    env: config.env,
-                    transport: config.transport,
-                },
             });
+            // Add server to NeuroLink - direct usage, zero transformations!
+            const sdk = new NeuroLink();
+            await sdk.addInMemoryMCPServer(serverName, serverInfo);
             if (spinner) {
                 spinner.succeed(chalk.green(`✅ Successfully installed ${serverName} MCP server`));
             }
             // Display configuration info
             logger.always(chalk.bold("\n📋 Server Configuration:"));
-            logger.always(`Name: ${config.name}`);
-            logger.always(`Command: ${config.command}`);
-            if (config.args?.length) {
-                logger.always(`Args: ${config.args.join(" ")}`);
+            logger.always(`Name: ${serverInfo.name}`);
+            logger.always(`Command: ${serverInfo.command}`);
+            if (serverInfo.args?.length) {
+                logger.always(`Args: ${serverInfo.args.join(" ")}`);
             }
-            if (config.env) {
-                logger.always(`Environment: ${Object.keys(config.env).length} variables`);
+            if (serverInfo.env) {
+                logger.always(`Environment: ${Object.keys(serverInfo.env).length} variables`);
             }
-            logger.always(`Transport: ${config.transport}`);
-            logger.always(`Description: ${config.description}`);
+            logger.always(`Transport: ${serverInfo.transport}`);
+            logger.always(`Description: ${serverInfo.description}`);
             // Test connection
             logger.always(chalk.blue("\n🔍 Testing connection..."));
             try {
                 const rawStatus = await sdk.getMCPStatus();
-                const status = convertMCPStatusForCLI(rawStatus, sdk);
-                const installedServer = status.servers.find((s) => s.name === serverName);
-                if (installedServer?.connected) {
+                const status = rawStatus;
+                const installedServer = status.externalMCPServers?.find((s) => s.name === serverName);
+                if (installedServer?.status === "connected") {
                     logger.always(chalk.green("✅ Server connected successfully"));
                     if (installedServer.tools?.length) {
                         logger.always(`🛠️  Available tools: ${installedServer.tools.length}`);
@@ -571,47 +467,32 @@ export class MCPCommandFactory {
                     process.exit(1);
                 }
             }
-            // Create server configuration
-            const config = {
+            const serverInfo = createExternalServerInfo({
+                id: name,
                 name,
                 command,
                 args: argv.args,
                 env,
-                transport: (argv.transport === "websocket"
-                    ? "ws"
-                    : argv.transport) || "stdio",
-                installed: true,
-                status: "unknown",
-            };
-            // Add server to NeuroLink
-            const sdk = new NeuroLink();
-            await sdk.addInMemoryMCPServer(name, {
-                server: {
-                    title: name,
-                    tools: {}, // Empty tools for external servers
-                    description: `Custom MCP server: ${command}`,
-                },
-                metadata: {
-                    command: config.command,
-                    args: config.args,
-                    env: config.env,
-                    transport: config.transport,
-                },
+                transport: argv.transport || "stdio",
+                description: command,
             });
+            // Add server to NeuroLink using MCPServerInfo directly
+            const sdk = new NeuroLink();
+            await sdk.addInMemoryMCPServer(name, serverInfo);
             if (spinner) {
                 spinner.succeed(chalk.green(`✅ Successfully added ${name} MCP server`));
             }
             // Display configuration
             logger.always(chalk.bold("\n📋 Server Configuration:"));
-            logger.always(`Name: ${config.name}`);
-            logger.always(`Command: ${config.command}`);
-            if (config.args?.length) {
-                logger.always(`Args: ${config.args.join(" ")}`);
+            logger.always(`Name: ${serverInfo.name}`);
+            logger.always(`Command: ${serverInfo.command}`);
+            if (serverInfo.args?.length) {
+                logger.always(`Args: ${serverInfo.args.join(" ")}`);
             }
-            if (config.env) {
-                logger.always(`Environment: ${Object.keys(config.env).length} variables`);
+            if (serverInfo.env) {
+                logger.always(`Environment: ${Object.keys(serverInfo.env).length} variables`);
             }
-            logger.always(`Transport: ${config.transport}`);
+            logger.always(`Transport: ${serverInfo.transport}`);
         }
         catch (_error) {
             logger.error(chalk.red(`❌ Add command failed: ${_error.message}`));
@@ -629,10 +510,9 @@ export class MCPCommandFactory {
                 : ora("Testing MCP server connections...").start();
             const sdk = new NeuroLink();
             const rawMcpStatus = await sdk.getMCPStatus();
-            const mcpStatus = convertMCPStatusForCLI(rawMcpStatus, sdk);
-            let serversToTest = mcpStatus.servers;
+            let serversToTest = await sdk.listMCPServers();
             if (targetServer) {
-                serversToTest = mcpStatus.servers.filter((s) => s.name === targetServer);
+                serversToTest = serversToTest.filter((s) => s.name === targetServer);
                 if (serversToTest.length === 0) {
                     if (spinner) {
                         spinner.fail();
@@ -647,11 +527,11 @@ export class MCPCommandFactory {
             // Display test results
             logger.always(chalk.bold("\n🧪 MCP Server Test Results:\n"));
             for (const server of serversToTest) {
-                const status = server.connected
+                const status = server.status === "connected"
                     ? chalk.green("✅ CONNECTED")
                     : chalk.red("❌ DISCONNECTED");
                 logger.always(`${server.name}: ${status}`);
-                if (server.connected) {
+                if (server.status === "connected") {
                     logger.always(`  Tools: ${server.tools?.length || 0} available`);
                     if (server.tools?.length) {
                         server.tools.slice(0, 3).forEach((tool) => {
@@ -671,7 +551,7 @@ export class MCPCommandFactory {
                 logger.always();
             }
             // Summary
-            const connected = serversToTest.filter((s) => s.connected).length;
+            const connected = serversToTest.filter((s) => s.status === "connected").length;
             const total = serversToTest.length;
             if (connected === total) {
                 logger.always(chalk.green(`🎉 All ${total} servers connected successfully`));
@@ -711,9 +591,8 @@ export class MCPCommandFactory {
             }
             const sdk = new NeuroLink();
             // Check if server exists and is connected
-            const rawMcpStatus = await sdk.getMCPStatus();
-            const mcpStatus = convertMCPStatusForCLI(rawMcpStatus, sdk);
-            const server = mcpStatus.servers.find((s) => s.name === serverName);
+            const allServers = await sdk.listMCPServers();
+            const server = allServers.find((s) => s.name === serverName);
             if (!server) {
                 if (spinner) {
                     spinner.fail();
@@ -721,7 +600,7 @@ export class MCPCommandFactory {
                 logger.error(chalk.red(`❌ Server not found: ${serverName}`));
                 process.exit(1);
             }
-            if (!server.connected) {
+            if (server.status !== "connected") {
                 if (spinner) {
                     spinner.fail();
                 }
@@ -815,9 +694,8 @@ export class MCPCommandFactory {
         try {
             const serverName = argv.server;
             const sdk = new NeuroLink();
-            const rawMcpStatus = await sdk.getMCPStatus();
-            const mcpStatus = convertMCPStatusForCLI(rawMcpStatus, sdk);
-            const server = mcpStatus.servers.find((s) => s.name === serverName);
+            const allServers = await sdk.listMCPServers();
+            const server = allServers.find((s) => s.name === serverName);
             if (!server) {
                 logger.error(chalk.red(`❌ Server not found: ${serverName}`));
                 process.exit(1);
@@ -907,19 +785,8 @@ export class MCPCommandFactory {
                 const sdk = new NeuroLink();
                 for (const server of discovered) {
                     try {
-                        await sdk.addInMemoryMCPServer(server.name, {
-                            server: {
-                                title: server.name,
-                                tools: {},
-                                description: server.description,
-                            },
-                            metadata: {
-                                command: server.command,
-                                args: server.args,
-                                env: server.env,
-                                transport: server.transport,
-                            },
-                        });
+                        // Use discovered MCPServerInfo directly - zero conversions!
+                        await sdk.addInMemoryMCPServer(server.name, server);
                         logger.always(chalk.green(`✅ Installed ${server.name}`));
                     }
                     catch (_error) {
@@ -950,17 +817,14 @@ export class MCPCommandFactory {
                     const config = JSON.parse(fs.readFileSync(configPath, "utf8"));
                     if (config.mcpServers) {
                         Object.entries(config.mcpServers).forEach(([name, serverConfig]) => {
-                            const config = serverConfig;
-                            servers.push({
+                            const typedConfig = serverConfig;
+                            // SMART DEFAULTS: Use utility to eliminate manual MCPServerInfo creation
+                            servers.push(createExternalServerInfo({
+                                ...typedConfig,
+                                id: name,
                                 name,
-                                command: config.command,
-                                args: config.args,
-                                env: config.env,
-                                transport: "stdio",
                                 description: "Discovered from Claude Desktop",
-                                installed: false,
-                                status: "unknown",
-                            });
+                            }));
                         });
                     }
                     break; // Found config file, stop searching
@@ -991,16 +855,12 @@ export class MCPCommandFactory {
                     if (settings["mcp.servers"]) {
                         Object.entries(settings["mcp.servers"]).forEach(([name, serverConfig]) => {
                             const config = serverConfig;
-                            servers.push({
+                            servers.push(createExternalServerInfo({
+                                ...config,
+                                id: name,
                                 name,
-                                command: config.command || "unknown",
-                                args: config.args,
-                                env: config.env,
-                                transport: "stdio",
                                 description: "Discovered from VS Code",
-                                installed: false,
-                                status: "unknown",
-                            });
+                            }));
                         });
                     }
                     break;

package/dist/cli/factories/commandFactory.js

@@ -288,7 +288,7 @@ export class CLICommandFactory {
         let analyticsText = "\n\n📊 Analytics:\n";
         // Provider and model info
         analyticsText += `   Provider: ${analytics.provider}`;
-        // Check for model in multiple locations: result.model, analytics.model, or any model property
+        // Check for model in multiple locations: result.model, analytics.model, or available model property
         const modelName = result.model ||
             analytics.model ||
             analytics.modelName;
@@ -703,6 +703,12 @@ export class CLICommandFactory {
                 return;
             }
             const sdk = new NeuroLink();
+            if (options.debug) {
+                logger.debug("CLI Tools configuration:", {
+                    disableTools: options.disableTools,
+                    toolsEnabled: !options.disableTools,
+                });
+            }
             const result = await sdk.generate({
                 input: { text: inputText },
                 provider: options.provider,

package/dist/cli/utils/interactiveSetup.js

@@ -146,7 +146,7 @@ export const PROVIDER_CONFIGS = [
             },
             {
                 key: "LITELLM_API_KEY",
-                prompt: "LiteLLM API Key (or any value)",
+                prompt: "LiteLLM API Key (or random value)",
                 default: "sk-anything",
                 secure: false,
                 optional: true,

package/dist/core/baseProvider.d.ts

@@ -3,43 +3,8 @@ import type { Schema } from "ai";
 import type { Tool, LanguageModelV1 } from "ai";
 import type { AIProvider, TextGenerationOptions, EnhancedGenerateResult, AnalyticsData, AIProviderName, EvaluationData } from "../core/types.js";
 import type { StreamOptions, StreamResult } from "../types/streamTypes.js";
-import type { JsonValue, UnknownRecord } from "../types/common.js";
-import type { ToolResult } from "../types/tools.js";
-/**
- * Interface for SDK with in-memory MCP servers and external MCP support
- */
-export interface NeuroLinkSDK {
-    getInMemoryServers?: () => Map<string, {
-        server: {
-            title?: string;
-            description?: string;
-            tools?: Map<string, ToolInfo> | Record<string, ToolInfo>;
-        };
-        category?: string;
-        metadata?: UnknownRecord;
-    }>;
-    externalServerManager?: {
-        getAllTools: () => Array<{
-            name: string;
-            description: string;
-            serverId: string;
-            isAvailable: boolean;
-            inputSchema?: Record<string, unknown>;
-        }>;
-        executeTool: (serverId: string, toolName: string, params: any) => Promise<any>;
-    };
-}
-/**
- * Interface for tool information in MCP servers
- */
-interface ToolInfo {
-    description?: string;
-    inputSchema?: ZodType<JsonValue>;
-    parameters?: ZodType<JsonValue>;
-    execute: (args: JsonValue) => Promise<JsonValue | ToolResult> | JsonValue | ToolResult;
-    isImplemented?: boolean;
-    metadata?: UnknownRecord;
-}
+import type { UnknownRecord } from "../types/common.js";
+import type { NeuroLink } from "../neurolink.js";
 /**
  * Abstract base class for all AI providers
  * Tools are integrated as first-class citizens - always available by default
@@ -50,10 +15,12 @@ export declare abstract class BaseProvider implements AIProvider {
     protected readonly defaultTimeout: number;
     protected readonly directTools: {};
     protected mcpTools?: Record<string, Tool>;
+    protected customTools?: Map<string, unknown>;
+    protected toolExecutor?: (toolName: string, params: unknown) => Promise<unknown>;
     protected sessionId?: string;
     protected userId?: string;
-    protected sdk?: NeuroLinkSDK;
-    constructor(modelName?: string, providerName?: AIProviderName, sdk?: NeuroLinkSDK);
+    protected neurolink?: NeuroLink;
+    constructor(modelName?: string, providerName?: AIProviderName, neurolink?: NeuroLink);
     /**
      * Check if this provider supports tool/function calling
      * Override in subclasses to disable tools for specific providers or models
@@ -176,4 +143,3 @@ export declare abstract class BaseProvider implements AIProvider {
      */
     static chunkPrompt(prompt: string, maxChunkSize?: number, overlap?: number): string[];
 }
-export {};