@lhi/n8m 0.1.2 → 0.2.0

package/dist/utils/n8nClient.js

@@ -12,13 +12,35 @@ export class N8nClient {
         // Priority: Explicit Config > Environment > Defaults
         this.apiUrl = config?.apiUrl ?? process.env.N8N_API_URL ?? 'http://localhost:5678/api/v1';
         this.apiKey = config?.apiKey ?? process.env.N8N_API_KEY ?? '';
+        // Normalize: ensure the URL ends with /api/v1 (config may store bare base URL)
+        if (!this.apiUrl.includes('/api/v1')) {
+            this.apiUrl = this.apiUrl.replace(/\/?$/, '') + '/api/v1';
+        }
         // Constructor validation moved to method call time to allow lazy loading
-        // from config file if not provided here. 
+        // from config file if not provided here.
         this.headers = {
             'Content-Type': 'application/json',
             'X-N8N-API-KEY': this.apiKey,
         };
     }
+    /**
+     * Assert that an API response is OK, throwing an actionable error for 401/403.
+     */
+    async assertOk(response, operation) {
+        if (response.ok)
+            return;
+        const errorText = await response.text();
+        if (response.status === 403) {
+            throw new Error(`${operation} failed (403 Forbidden). Your n8n API key does not have permission to access this resource. ` +
+                `In n8n, go to Settings → n8n API and ensure the key belongs to the same owner/project as the workflow. ` +
+                `n8n says: ${errorText}`);
+        }
+        if (response.status === 401) {
+            throw new Error(`${operation} failed (401 Unauthorized). Your n8n API key is missing or invalid. ` +
+                `Run: n8m config --n8n-key <your-key>`);
+        }
+        throw new Error(`${operation} failed: ${response.status} - ${errorText}`);
+    }
     /**
      * Activate a workflow
      */
@@ -27,10 +49,7 @@ export class N8nClient {
             headers: this.headers,
             method: 'POST',
         });
-        if (!response.ok) {
-            const errorText = await response.text();
-            throw new Error(`Failed to activate workflow: ${response.status} - ${errorText}`);
-        }
+        await this.assertOk(response, 'activate workflow');
     }
     /**
      * Deactivate a workflow
@@ -40,10 +59,7 @@ export class N8nClient {
             headers: this.headers,
             method: 'POST',
         });
-        if (!response.ok) {
-            const errorText = await response.text();
-            throw new Error(`Failed to deactivate workflow: ${response.status} - ${errorText}`);
-        }
+        await this.assertOk(response, 'deactivate workflow');
     }
     /**
      * Execute a workflow and return the result
@@ -56,10 +72,7 @@ export class N8nClient {
                 headers: this.headers,
                 method: 'POST',
             });
-            if (!response.ok) {
-                const errorText = await response.text();
-                throw new Error(`n8n Validation Error: ${response.status} - ${errorText}`);
-            }
+            await this.assertOk(response, 'execute workflow');
             // If activation succeeds, we assume basic validation passed.
             const result = await response.json();
             return {
@@ -83,9 +96,7 @@ export class N8nClient {
                 headers: this.headers,
                 method: 'GET',
             });
-            if (!response.ok) {
-                throw new Error(`Failed to get execution: ${response.status}`);
-            }
+            await this.assertOk(response, 'get execution');
             return response.json();
         }
         catch (error) {
@@ -104,9 +115,7 @@ export class N8nClient {
                 headers: this.headers,
                 method: 'GET',
             });
-            if (!response.ok) {
-                throw new Error(`Failed to get executions: ${response.status}`);
-            }
+            await this.assertOk(response, 'get workflow executions');
             const result = await response.json();
             return result.data;
         }
@@ -119,15 +128,13 @@ export class N8nClient {
      */
     async updateWorkflow(workflowId, workflowData) {
         try {
+            const sanitized = this.sanitizeSettings(workflowData);
             const response = await fetch(`${this.apiUrl}/workflows/${workflowId}`, {
-                body: JSON.stringify(workflowData),
+                body: JSON.stringify(sanitized),
                 headers: this.headers,
                 method: 'PUT',
             });
-            if (!response.ok) {
-                const errorText = await response.text();
-                throw new Error(`Failed to update workflow: ${response.status} - ${errorText}`);
-            }
+            await this.assertOk(response, 'update workflow');
         }
         catch (error) {
             throw new Error(`Failed to update workflow: ${error.message}`);
@@ -142,24 +149,35 @@ export class N8nClient {
                 headers: this.headers,
                 method: 'GET',
             });
-            if (!response.ok) {
-                throw new Error(`Failed to get workflow: ${response.status}`);
-            }
+            await this.assertOk(response, 'get workflow');
             return response.json();
         }
         catch (error) {
             throw new Error(`Failed to fetch workflow: ${error.message}`);
         }
     }
+    /**
+     * Strip invalid timezone from workflow settings so n8n activation never 400s.
+     */
+    sanitizeSettings(data) {
+        if (!data.settings || typeof data.settings !== 'object')
+            return data;
+        const settings = { ...data.settings };
+        // Always remove timezone — n8n validates against its own list and AI-generated
+        // values (including seemingly valid ones like "UTC") cause activation 400s.
+        // n8n will use the instance-level default timezone instead.
+        delete settings.timezone;
+        return { ...data, settings };
+    }
     /**
      * Create a new workflow
      */
     async createWorkflow(name, workflowData) {
         try {
-            const payload = {
+            const payload = this.sanitizeSettings({
                 name,
                 ...workflowData,
-            };
+            });
             // Debug logging for payload validation errors
             // console.log('DEBUG: createWorkflow payload keys:', Object.keys(payload));
             const response = await fetch(`${this.apiUrl}/workflows`, {
@@ -167,10 +185,7 @@ export class N8nClient {
                 headers: this.headers,
                 method: 'POST',
             });
-            if (!response.ok) {
-                const errorText = await response.text();
-                throw new Error(`Failed to create workflow: ${response.status} - ${errorText}`);
-            }
+            await this.assertOk(response, 'create workflow');
             const result = await response.json();
             return { id: result.id };
         }
@@ -179,121 +194,45 @@ export class N8nClient {
         }
     }
     /**
-     * Get all installed node types via Probe Workflow (Webhook)
-     *
-     * Strategy:
-     * 1. Create a workflow with Webhook -> HTTP Request (Internal API)
-     * 2. Activate it
-     * 3. Call the webhook -> Returns the node types
-     */
-    /**
-     * Get all installed node types via Probe Workflow (Webhook)
-     * Returns full node type objects including parameters, not just names.
+     * Get all installed node types directly from the n8n REST API.
+     * Handles paginated responses and returns the full node type objects.
      */
     async getNodeTypes() {
-        const probeId = `probe-${Math.random().toString(36).substring(7)}`;
-        const probePath = `n8m-probe-${Math.random().toString(36).substring(7)}`;
-        let workflowId = null;
         try {
-            const internalApiUrl = this.apiUrl + '/node-types';
-            const probeWorkflow = {
-                name: `[n8m:system] Node Probe ${probeId}`,
-                nodes: [
-                    {
-                        parameters: {
-                            httpMethod: "GET",
-                            path: probePath,
-                            responseMode: "lastNode",
-                            options: {}
-                        },
-                        id: "webhook",
-                        name: "ProbeWebhook",
-                        type: "n8n-nodes-base.webhook",
-                        typeVersion: 1,
-                        position: [400, 300],
-                        webhookId: probePath
-                    },
-                    {
-                        parameters: {
-                            url: internalApiUrl,
-                            method: "GET",
-                            authentication: "none",
-                            sendHeaders: true,
-                            headerParameters: {
-                                parameters: [
-                                    {
-                                        name: "X-N8N-API-KEY",
-                                        value: this.apiKey
-                                    }
-                                ]
-                            },
-                            options: {}
-                        },
-                        id: "http-request",
-                        name: "FetchNodes",
-                        type: "n8n-nodes-base.httpRequest",
-                        typeVersion: 4.1,
-                        position: [600, 300]
-                    }
-                ],
-                connections: {
-                    "ProbeWebhook": {
-                        main: [[{ node: "FetchNodes", type: "main", index: 0 }]]
-                    }
-                },
-                settings: {
-                    saveManualExecutions: false,
-                    callerPolicy: 'workflowsFromSameOwner'
+            let all = [];
+            let cursor = undefined;
+            do {
+                const url = new URL(`${this.apiUrl}/node-types`);
+                if (cursor)
+                    url.searchParams.set('cursor', cursor);
+                const response = await fetch(url.toString(), {
+                    headers: this.headers,
+                    method: 'GET',
+                });
+                if (!response.ok) {
+                    console.warn(`[N8nClient] node-types request failed (${response.status}) — validation/shimming disabled`);
+                    return [];
                 }
-            };
-            // Header Injection
-            probeWorkflow.nodes[1].parameters.authentication = 'none';
-            probeWorkflow.nodes[1].parameters.headerParameters = {
-                parameters: [
-                    { name: 'X-N8N-API-KEY', value: this.apiKey }
-                ]
-            };
-            // 1. Create
-            const { id } = await this.createWorkflow(probeWorkflow.name, probeWorkflow);
-            workflowId = id;
-            // 2. Activate
-            await this.activateWorkflow(id);
-            // 3. Trigger
-            const baseUrl = this.apiUrl.replace('/api/v1', '');
-            const webhookUrl = `${baseUrl}/webhook/${probePath}`;
-            console.log(`[N8nClient] Triggering probe at ${webhookUrl}...`);
-            const response = await fetch(webhookUrl);
-            if (!response.ok) {
-                const errorText = await response.text();
-                if (process.env.DEBUG) {
-                    console.warn(`[N8nClient] Probe webhook failed (Status ${response.status}): ${errorText}`);
+                const result = await response.json();
+                if (Array.isArray(result)) {
+                    all = [...all, ...result];
+                    cursor = undefined;
                 }
-                throw new Error(`Probe webhook failed: ${response.status}`);
-            }
-            const result = await response.json();
-            // Return full objects
-            if (Array.isArray(result)) {
-                return result;
-            }
-            if (result.data && Array.isArray(result.data)) {
-                return result.data;
-            }
-            return [];
+                else if (result.data && Array.isArray(result.data)) {
+                    all = [...all, ...result.data];
+                    cursor = result.nextCursor ?? undefined;
+                }
+                else {
+                    // Unknown format — stop paging
+                    break;
+                }
+            } while (cursor);
+            return all;
         }
         catch (error) {
-            if (process.env.DEBUG) {
-                console.warn(`[N8nClient] Probe failed: ${error.message}`);
-            }
+            console.warn(`[N8nClient] Failed to fetch node types: ${error.message}`);
             return [];
         }
-        finally {
-            if (workflowId) {
-                try {
-                    await this.deleteWorkflow(workflowId);
-                }
-                catch { /* intentionally empty */ }
-            }
-        }
     }
     /**
      * Get all workflows
@@ -312,9 +251,7 @@ export class N8nClient {
                     headers: this.headers,
                     method: 'GET',
                 });
-                if (!response.ok) {
-                    throw new Error(`Failed to get workflows: ${response.status}`);
-                }
+                await this.assertOk(response, 'get workflows');
                 const result = await response.json();
                 allWorkflows = [...allWorkflows, ...result.data];
                 cursor = result.nextCursor;
@@ -334,9 +271,7 @@ export class N8nClient {
                 headers: this.headers,
                 method: 'DELETE',
             });
-            if (!response.ok) {
-                throw new Error(`Failed to delete workflow: ${response.status}`);
-            }
+            await this.assertOk(response, 'delete workflow');
         }
         catch (error) {
             throw new Error(`Failed to delete workflow: ${error.message}`);

package/oclif.manifest.json

@@ -72,7 +72,7 @@
           "required": false
         }
       },
-      "description": "Generate n8n workflows from natural language using Gemini AI Agent",
+      "description": "Generate n8n workflows from natural language using an AI Agent",
       "examples": [
         "<%= config.bin %> <%= command.id %> \"Send a telegram alert when I receive an email\"",
         "echo \"Slack to Discord sync\" | <%= config.bin %> <%= command.id %>",
@@ -164,6 +164,64 @@
         "deploy.js"
       ]
     },
+    "doc": {
+      "aliases": [],
+      "args": {
+        "workflow": {
+          "description": "Path or Name of the workflow to document",
+          "name": "workflow",
+          "required": false
+        }
+      },
+      "description": "Generate visual and text documentation for n8n workflows",
+      "flags": {
+        "output": {
+          "char": "o",
+          "description": "Output directory for documentation (defaults to ./docs)",
+          "name": "output",
+          "hasDynamicHelp": false,
+          "multiple": false,
+          "type": "option"
+        }
+      },
+      "hasDynamicHelp": false,
+      "hiddenAliases": [],
+      "id": "doc",
+      "pluginAlias": "@lhi/n8m",
+      "pluginName": "@lhi/n8m",
+      "pluginType": "core",
+      "strict": true,
+      "enableJsonFlag": false,
+      "isESM": true,
+      "relativePath": [
+        "dist",
+        "commands",
+        "doc.js"
+      ]
+    },
+    "mcp": {
+      "aliases": [],
+      "args": {},
+      "description": "Launch the n8m MCP (Model Context Protocol) server",
+      "examples": [
+        "<%= config.bin %> <%= command.id %>"
+      ],
+      "flags": {},
+      "hasDynamicHelp": false,
+      "hiddenAliases": [],
+      "id": "mcp",
+      "pluginAlias": "@lhi/n8m",
+      "pluginName": "@lhi/n8m",
+      "pluginType": "core",
+      "strict": true,
+      "enableJsonFlag": false,
+      "isESM": true,
+      "relativePath": [
+        "dist",
+        "commands",
+        "mcp.js"
+      ]
+    },
     "modify": {
       "aliases": [],
       "args": {
@@ -178,7 +236,7 @@
           "required": false
         }
       },
-      "description": "Modify existing n8n workflows using Gemini AI Agent",
+      "description": "Modify existing n8n workflows using an AI Agent",
       "flags": {
         "multiline": {
           "char": "m",
@@ -309,6 +367,12 @@
           "name": "validate-only",
           "allowNo": false,
           "type": "boolean"
+        },
+        "ai-scenarios": {
+          "description": "Generate 3 diverse AI test scenarios (happy path, edge case, error)",
+          "name": "ai-scenarios",
+          "allowNo": false,
+          "type": "boolean"
         }
       },
       "hasDynamicHelp": false,
@@ -327,5 +391,5 @@
       ]
     }
   },
-  "version": "0.1.2"
+  "version": "0.2.0"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lhi/n8m",
-  "version": "0.1.2",
+  "version": "0.2.0",
   "description": "Agentic n8n CLI wrapper - A Skill Bridge for n8n workflow automation",
   "author": "Lem Canady",
   "license": "MIT",
@@ -18,10 +18,10 @@
     "oclif.manifest.json"
   ],
   "dependencies": {
-    "openai": "^4.0.0",
     "@langchain/core": "^1.1.18",
     "@langchain/langgraph": "^1.1.2",
     "@langchain/langgraph-checkpoint-sqlite": "^1.0.0",
+    "@modelcontextprotocol/sdk": "^1.27.1",
     "@oclif/core": "^4",
     "@oclif/plugin-help": "^6",
     "@oclif/plugin-plugins": "^5",
@@ -37,6 +37,8 @@
     "ink-spinner": "^5.0.0",
     "ink-text-input": "^6.0.0",
     "inquirer": "^13.2.2",
+    "jsonrepair": "^3.13.2",
+    "openai": "^4.0.0",
     "react": "^19.2.4"
   },
   "devDependencies": {
@@ -83,13 +85,14 @@
     }
   },
   "scripts": {
-    "build": "shx rm -rf dist && tsc -b",
+    "build": "shx rm -rf dist && tsc -b && shx mkdir -p dist/resources && shx cp src/resources/*.json dist/resources/",
     "lint": "eslint .",
     "postpack": "shx rm -f oclif.manifest.json",
     "posttest": "npm run lint",
     "prepack": "oclif manifest && oclif readme",
-    "test": "mocha --forbid-only \"test/**/*.test.ts\"",
+    "test": "NODE_ENV=test mocha --forbid-only \"test/**/*.test.ts\"",
     "n8m": "./bin/run.js",
+    "start": "npm run build && ./bin/run.js",
     "dev": "tsc -b -w",
     "version": "oclif readme && git add README.md"
   }