n8n-nodes-smart-browser-automation 1.6.22 → 1.6.24

package/dist/nodes/SmartBrowserAutomation/BrowserSessionManager.d.ts

@@ -12,6 +12,7 @@ declare class BrowserSessionManager {
     private tools;
     private constructor();
     static getInstance(): BrowserSessionManager;
+    private parseUrl;
     initialize(mcpEndpoint: string, useCDP: boolean, cdpEndpoint?: string): Promise<MCPTool[]>;
     private getAllTools;
     callTool(toolName: string, toolArgs?: any): Promise<any>;

package/dist/nodes/SmartBrowserAutomation/BrowserSessionManager.js

@@ -48,6 +48,17 @@ class BrowserSessionManager {
         }
         return BrowserSessionManager.instance;
     }
+    /*
+     * Safe URL parser helper
+     */
+    parseUrl(url, description) {
+        try {
+            return new URL(url);
+        }
+        catch (error) {
+            throw new Error(`Invalid ${description}: "${url}". Please ensure it is a valid URL (e.g. starting with http:// or https://).`);
+        }
+    }
     async initialize(mcpEndpoint, useCDP, cdpEndpoint) {
         // Validate endpoint inputs early to avoid confusing runtime errors
         let trimmedMcpEndpoint = String(mcpEndpoint ?? '').trim();
@@ -55,10 +66,11 @@ class BrowserSessionManager {
             throw new Error('MCP Endpoint is required');
         }
         // Be forgiving: if user passes host:port without scheme, assume http://
-        // Examples: localhost:3000, 127.0.0.1:3000, example.com:8080
         if (!/^[a-zA-Z][a-zA-Z0-9+.-]*:/.test(trimmedMcpEndpoint)) {
             trimmedMcpEndpoint = `http://${trimmedMcpEndpoint}`;
         }
+        // Validate URL format immediately
+        const mcpUrlObj = this.parseUrl(trimmedMcpEndpoint, 'MCP Endpoint');
         if (/^wss?:\/\//i.test(trimmedMcpEndpoint)) {
             throw new Error(`Invalid MCP Endpoint: "${trimmedMcpEndpoint}". ` +
                 'MCP Endpoint must be an http(s) URL (for SSE or Streamable HTTP). ' +
@@ -91,22 +103,23 @@ class BrowserSessionManager {
         let urlIsSse = false;
         if (isUrl) {
             try {
-                urlIsSse = /(^|\/)sse\/?(\?|#|$)/i.test(new URL(trimmedMcpEndpoint).pathname);
+                urlIsSse = /(^|\/)sse\/?(\?|#|$)/i.test(mcpUrlObj.pathname);
             }
             catch (e) {
-                throw new Error(`Invalid MCP Endpoint: "${trimmedMcpEndpoint}". MCP Endpoint must be a valid http(s) URL (or a local file path for stdio).`);
+                // Should be caught by parseUrl above, but just in case
+                throw new Error(`Invalid MCP Endpoint path: "${trimmedMcpEndpoint}".`);
             }
         }
         if (isUrl) {
             if (urlIsSse) {
                 // Connect via SSE
                 const { SSEClientTransport } = await Promise.resolve().then(() => __importStar(require('@modelcontextprotocol/sdk/client/sse.js')));
-                this.transport = new SSEClientTransport(new URL(trimmedMcpEndpoint));
+                this.transport = new SSEClientTransport(new URL(trimmedMcpEndpoint)); // Safe because we parsed it above
             }
             else {
                 // Connect via Streamable HTTP
                 const { StreamableHTTPClientTransport } = await Promise.resolve().then(() => __importStar(require('@modelcontextprotocol/sdk/client/streamableHttp.js')));
-                this.transport = new StreamableHTTPClientTransport(new URL(trimmedMcpEndpoint));
+                this.transport = new StreamableHTTPClientTransport(new URL(trimmedMcpEndpoint)); // Safe because we parsed it above
             }
         }
         else {

package/dist/nodes/SmartBrowserAutomation/SmartBrowserAutomationTools.node.js

@@ -86,8 +86,8 @@ class SmartBrowserAutomationTools {
             name: 'Browser Automation Tools',
         },
         inputs: [],
-        outputs: [n8n_workflow_1.NodeConnectionTypes.AiTool],
-        outputNames: ['Tools'],
+        outputs: [n8n_workflow_1.NodeConnectionTypes.AiTool, n8n_workflow_1.NodeConnectionTypes.Main],
+        outputNames: ['Tools', 'Debug'],
         icon: 'file:smartBrowserAutomation.svg',
         credentials: [
             {
@@ -140,7 +140,7 @@ class SmartBrowserAutomationTools {
             const e = error;
             const mode = useCDP ? 'cdp' : 'launch';
             const details = `mcp=${mcpEndpoint} | mode=${mode}${useCDP ? ` | cdp=${cdpUrl || 'unset'}` : ''}`;
-            throw new n8n_workflow_1.NodeOperationError(node, `Failed to connect to MCP: ${e.message}`, {
+            throw new n8n_workflow_1.NodeOperationError(node, `Failed to connect to MCP. ${details}. Original error: ${e.message}`, {
                 itemIndex,
                 description: details,
             });
@@ -149,6 +149,18 @@ class SmartBrowserAutomationTools {
         if (!mcpTools.length) {
             throw new n8n_workflow_1.NodeOperationError(node, 'MCP Server returned no tools', { itemIndex });
         }
+        // Emit a debug item on the Main output so users can see endpoints and tool list in the UI
+        const debugJson = {
+            mcpEndpoint,
+            browserMode: credentials.browserMode,
+            cdpEndpoint: useCDP ? cdpUrl : undefined,
+            toolCount: mcpTools.length,
+            tools: mcpTools.map((t) => ({
+                name: t.name,
+                description: t.description ?? '',
+            })),
+        };
+        this.addOutputData(n8n_workflow_1.NodeConnectionTypes.Main, 0, [[{ json: debugJson }]]);
         const { DynamicStructuredTool } = await importDynamicStructuredTool();
         const { Toolkit } = await importToolkitBase();
         const ctx = this;
@@ -173,13 +185,33 @@ class SmartBrowserAutomationTools {
                     catch { }
                     const { index } = ctx.addInputData(n8n_workflow_1.NodeConnectionTypes.AiTool, [[{ json: input }]], runIndex);
                     try {
+                        console.log(`[AI Agent] Executing tool ${toolName} with args:`, JSON.stringify(args));
                         const result = await sessionManager.callTool(toolName, args);
                         const output = { tool: toolName, result };
-                        ctx.addOutputData(n8n_workflow_1.NodeConnectionTypes.AiTool, index, [[{ json: output }]]);
+                        // Ensure the output is an array of INodeExecutionData
+                        const executionData = [[{ json: output }]];
+                        ctx.addOutputData(n8n_workflow_1.NodeConnectionTypes.AiTool, index, executionData);
                         return formatMcpToolResult(result);
                     }
                     catch (e) {
-                        ctx.addOutputData(n8n_workflow_1.NodeConnectionTypes.AiTool, index, e);
+                        const errorMsg = e.message || String(e);
+                        console.error(`[AI Agent] Tool ${toolName} failed:`, errorMsg);
+                        // Fix: properly format error for addOutputData
+                        const errorOutput = [[{
+                                    json: {
+                                        tool: toolName,
+                                        error: errorMsg,
+                                        stack: e.stack
+                                    }
+                                }]];
+                        // Safely attempt to report error to UI
+                        try {
+                            ctx.addOutputData(n8n_workflow_1.NodeConnectionTypes.AiTool, index, errorOutput);
+                        }
+                        catch (addError) {
+                            console.error('[AI Agent] Failed to add error output:', addError);
+                        }
+                        // Re-throw so the AI Agent knows it failed
                         throw e;
                     }
                 },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "n8n-nodes-smart-browser-automation",
-  "version": "1.6.22",
+  "version": "1.6.24",
   "description": "n8n node for AI-driven browser automation using MCP",
   "keywords": [
     "n8n-community-node-package",
@@ -63,4 +63,4 @@
     "@modelcontextprotocol/sdk": "1.17.0",
     "zod": "3.25.76"
   }
-}
+}