@pagelines/n8n-mcp 0.3.0 → 0.3.2

package/CHANGELOG.md

@@ -2,18 +2,34 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.3.2] - 2025-01-13
+
+### Removed
+- `node_types_list` tool - n8n API doesn't reliably expose node types across versions
+- Node type pre-validation from `workflow_create` and `workflow_update` - without reliable API, validation was causing false 404 errors
+
+### Fixed
+- `workflow_update` failing with "request/body/tags is read-only" error
+- Removed `tags` from `N8N_WORKFLOW_WRITABLE_FIELDS` since it's read-only in some n8n versions
+
+## [0.3.1] - 2025-01-13
+
+### Fixed
+- **Critical bug**: `workflow_update`, `workflow_format`, and `workflow_autofix` failing with "request/body must NOT have additional properties" error
+- Root cause: n8n API returns additional read-only properties that were being sent back on PUT requests
+- Solution: Schema-driven field filtering using `N8N_WORKFLOW_WRITABLE_FIELDS` allowlist (source of truth: n8n OpenAPI spec at `/api/v1/openapi.yml`)
+
+### Changed
+- Refactored `updateWorkflow` to use schema-driven approach instead of property denylist
+- Added `pickFields` generic utility for type-safe field filtering
+- Added comprehensive tests for schema-driven filtering
+
 ## [0.3.0] - 2025-01-13
 
 ### Added
 
-#### Node Type Discovery & Validation
-- `node_types_list` - Search available node types by name/category
-- Pre-validation blocks invalid node types before `workflow_create` and `workflow_update`
-- Fuzzy matching suggests correct types when invalid types detected
-
 #### Auto-Cleanup Pipeline
 - Every `workflow_create` and `workflow_update` now automatically:
-  - Validates node types (blocks if invalid)
   - Runs validation rules
   - Auto-fixes fixable issues (snake_case, $json refs, AI settings)
   - Formats workflow (sorts nodes, removes nulls)

package/dist/index.js

@@ -8,7 +8,7 @@ import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js'
 import { CallToolRequestSchema, ListToolsRequestSchema, } from '@modelcontextprotocol/sdk/types.js';
 import { N8nClient } from './n8n-client.js';
 import { tools } from './tools.js';
-import { validateWorkflow, validateNodeTypes } from './validators.js';
+import { validateWorkflow } from './validators.js';
 import { validateExpressions, checkCircularReferences } from './expressions.js';
 import { autofixWorkflow, formatWorkflow } from './autofix.js';
 import { initVersionControl, saveVersion, listVersions, getVersion, diffWorkflows, getVersionStats, } from './versions.js';
@@ -37,7 +37,7 @@ initVersionControl({
 // ─────────────────────────────────────────────────────────────
 const server = new Server({
     name: '@pagelines/n8n-mcp',
-    version: '0.3.0',
+    version: '0.3.2',
 }, {
     capabilities: {
         tools: {},
@@ -103,21 +103,6 @@ async function handleTool(name, args) {
         }
         case 'workflow_create': {
             const inputNodes = args.nodes;
-            // Validate node types BEFORE creating workflow
-            const availableTypes = await client.listNodeTypes();
-            const validTypeSet = new Set(availableTypes.map((nt) => nt.name));
-            const typeErrors = validateNodeTypes(inputNodes, validTypeSet);
-            if (typeErrors.length > 0) {
-                const errorMessages = typeErrors.map((e) => {
-                    let msg = e.message;
-                    if (e.suggestions && e.suggestions.length > 0) {
-                        msg += `. Did you mean: ${e.suggestions.join(', ')}?`;
-                    }
-                    return msg;
-                });
-                throw new Error(`Invalid node types detected:\n${errorMessages.join('\n')}\n\n` +
-                    `Use node_types_list to discover available node types.`);
-            }
             const nodes = inputNodes.map((n, i) => ({
                 id: crypto.randomUUID(),
                 name: n.name,
@@ -154,29 +139,6 @@ async function handleTool(name, args) {
         }
         case 'workflow_update': {
             const operations = args.operations;
-            // Extract addNode operations that need validation
-            const addNodeOps = operations.filter((op) => op.type === 'addNode');
-            if (addNodeOps.length > 0) {
-                // Fetch available types and validate
-                const availableTypes = await client.listNodeTypes();
-                const validTypeSet = new Set(availableTypes.map((nt) => nt.name));
-                const nodesToValidate = addNodeOps.map((op) => ({
-                    name: op.node.name,
-                    type: op.node.type,
-                }));
-                const typeErrors = validateNodeTypes(nodesToValidate, validTypeSet);
-                if (typeErrors.length > 0) {
-                    const errorMessages = typeErrors.map((e) => {
-                        let msg = e.message;
-                        if (e.suggestions && e.suggestions.length > 0) {
-                            msg += `. Did you mean: ${e.suggestions.join(', ')}?`;
-                        }
-                        return msg;
-                    });
-                    throw new Error(`Invalid node types in addNode operations:\n${errorMessages.join('\n')}\n\n` +
-                        `Use node_types_list to discover available node types.`);
-                }
-            }
             // Save version before updating
             const currentWorkflow = await client.getWorkflow(args.id);
             const versionSaved = await saveVersion(currentWorkflow, 'before_update');
@@ -297,37 +259,6 @@ async function handleTool(name, args) {
                 previewWorkflow: formatted,
             };
         }
-        // Node Discovery
-        case 'node_types_list': {
-            const nodeTypes = await client.listNodeTypes();
-            const search = args.search?.toLowerCase();
-            const category = args.category;
-            const limit = args.limit || 50;
-            let results = nodeTypes.map((nt) => ({
-                type: nt.name,
-                name: nt.displayName,
-                description: nt.description,
-                category: nt.codex?.categories?.[0] || nt.group?.[0] || 'Other',
-                version: nt.version,
-            }));
-            // Apply search filter
-            if (search) {
-                results = results.filter((nt) => nt.type.toLowerCase().includes(search) ||
-                    nt.name.toLowerCase().includes(search) ||
-                    nt.description.toLowerCase().includes(search));
-            }
-            // Apply category filter
-            if (category) {
-                results = results.filter((nt) => nt.category.toLowerCase().includes(category.toLowerCase()));
-            }
-            // Apply limit
-            results = results.slice(0, limit);
-            return {
-                nodeTypes: results,
-                total: results.length,
-                hint: 'Use the "type" field value when creating nodes (e.g., "n8n-nodes-base.webhook")',
-            };
-        }
         // Version Control
         case 'version_list': {
             const versions = await listVersions(args.workflowId);

package/dist/n8n-client.d.ts

@@ -2,7 +2,7 @@
  * n8n REST API Client
  * Clean, minimal implementation with built-in safety checks
  */
-import type { N8nWorkflow, N8nWorkflowListItem, N8nExecution, N8nExecutionListItem, N8nListResponse, N8nNode, N8nNodeType, PatchOperation } from './types.js';
+import { type N8nWorkflow, type N8nWorkflowListItem, type N8nExecution, type N8nExecutionListItem, type N8nListResponse, type N8nNode, type PatchOperation } from './types.js';
 export interface N8nClientConfig {
     apiUrl: string;
     apiKey: string;
@@ -51,5 +51,4 @@ export declare class N8nClient {
         version?: string;
         error?: string;
     }>;
-    listNodeTypes(): Promise<N8nNodeType[]>;
 }

package/dist/n8n-client.js

@@ -2,6 +2,7 @@
  * n8n REST API Client
  * Clean, minimal implementation with built-in safety checks
  */
+import { N8N_WORKFLOW_WRITABLE_FIELDS, pickFields, } from './types.js';
 export class N8nClient {
     baseUrl;
     headers;
@@ -56,8 +57,8 @@ export class N8nClient {
         return this.request('POST', '/api/v1/workflows', workflow);
     }
     async updateWorkflow(id, workflow) {
-        // Strip properties that n8n API doesn't accept on PUT
-        const { id: _id, createdAt, updatedAt, active, versionId, ...allowed } = workflow;
+        // Schema-driven: only send fields n8n accepts (defined in types.ts)
+        const allowed = pickFields(workflow, N8N_WORKFLOW_WRITABLE_FIELDS);
         return this.request('PUT', `/api/v1/workflows/${id}`, allowed);
     }
     async deleteWorkflow(id) {
@@ -274,10 +275,4 @@ export class N8nClient {
             };
         }
     }
-    // ─────────────────────────────────────────────────────────────
-    // Node Types
-    // ─────────────────────────────────────────────────────────────
-    async listNodeTypes() {
-        return this.request('GET', '/api/v1/nodes');
-    }
 }

package/dist/tools.js

@@ -325,35 +325,6 @@ Returns the fixed workflow and list of changes made.`,
         },
     },
     // ─────────────────────────────────────────────────────────────
-    // Node Discovery
-    // ─────────────────────────────────────────────────────────────
-    {
-        name: 'node_types_list',
-        description: `List available n8n node types. Use this to discover valid node types BEFORE creating workflows.
-
-Returns: type name, display name, description, category, and version for each node.
-Use the search parameter to filter by keyword (searches type name, display name, and description).
-
-IMPORTANT: Always check node types exist before using them in workflow_create or workflow_update.`,
-        inputSchema: {
-            type: 'object',
-            properties: {
-                search: {
-                    type: 'string',
-                    description: 'Filter nodes by keyword (searches name, type, description)',
-                },
-                category: {
-                    type: 'string',
-                    description: 'Filter by category (e.g., "Core Nodes", "Flow", "AI")',
-                },
-                limit: {
-                    type: 'number',
-                    description: 'Max results (default 50)',
-                },
-            },
-        },
-    },
-    // ─────────────────────────────────────────────────────────────
     // Version Control
     // ─────────────────────────────────────────────────────────────
     {

package/dist/types.d.ts

@@ -130,30 +130,20 @@ export interface N8nListResponse<T> {
     data: T[];
     nextCursor?: string;
 }
-export interface N8nNodeType {
-    name: string;
-    displayName: string;
-    description: string;
-    group: string[];
-    version: number;
-    defaults?: {
-        name: string;
-    };
-    codex?: {
-        categories?: string[];
-        alias?: string[];
-    };
-}
-export interface N8nNodeTypeSummary {
-    type: string;
-    name: string;
-    description: string;
-    category: string;
-    version: number;
-}
-export interface NodeTypeValidationError {
-    nodeType: string;
-    nodeName: string;
-    message: string;
-    suggestions?: string[];
-}
+/**
+ * Fields that n8n API accepts on PUT /workflows/:id
+ * All other fields (id, createdAt, homeProject, etc.) are read-only
+ *
+ * Derived from: n8n OpenAPI spec (GET {n8n-url}/api/v1/openapi.yml)
+ * Path: /workflows/{id} → put → requestBody → content → application/json → schema
+ *
+ * If n8n adds new writable fields, check the OpenAPI spec and update this array.
+ */
+export declare const N8N_WORKFLOW_WRITABLE_FIELDS: readonly ["name", "nodes", "connections", "settings", "staticData"];
+export type N8nWorkflowWritableField = (typeof N8N_WORKFLOW_WRITABLE_FIELDS)[number];
+export type N8nWorkflowUpdate = Pick<N8nWorkflow, N8nWorkflowWritableField>;
+/**
+ * Pick only specified fields from an object (strips everything else)
+ * Generic utility for schema-driven field filtering
+ */
+export declare function pickFields<T, K extends keyof T>(obj: T, fields: readonly K[]): Pick<T, K>;

package/dist/types.js

@@ -2,4 +2,37 @@
  * n8n API Types
  * Minimal types for workflow management
  */
-export {};
+// ─────────────────────────────────────────────────────────────
+// Schema-driven field definitions
+// Source of truth: n8n OpenAPI spec at /api/v1/openapi.yml
+// ─────────────────────────────────────────────────────────────
+/**
+ * Fields that n8n API accepts on PUT /workflows/:id
+ * All other fields (id, createdAt, homeProject, etc.) are read-only
+ *
+ * Derived from: n8n OpenAPI spec (GET {n8n-url}/api/v1/openapi.yml)
+ * Path: /workflows/{id} → put → requestBody → content → application/json → schema
+ *
+ * If n8n adds new writable fields, check the OpenAPI spec and update this array.
+ */
+export const N8N_WORKFLOW_WRITABLE_FIELDS = [
+    'name',
+    'nodes',
+    'connections',
+    'settings',
+    'staticData',
+    // Note: 'tags' is read-only in some n8n versions
+];
+/**
+ * Pick only specified fields from an object (strips everything else)
+ * Generic utility for schema-driven field filtering
+ */
+export function pickFields(obj, fields) {
+    const result = {};
+    for (const field of fields) {
+        if (field in obj && obj[field] !== undefined) {
+            result[field] = obj[field];
+        }
+    }
+    return result;
+}

package/dist/validators.d.ts

@@ -2,17 +2,9 @@
  * Opinion-based workflow validation
  * Enforces best practices from n8n-best-practices.md
  */
-import type { N8nWorkflow, ValidationResult, ValidationWarning, NodeTypeValidationError } from './types.js';
+import type { N8nWorkflow, ValidationResult, ValidationWarning } from './types.js';
 export declare function validateWorkflow(workflow: N8nWorkflow): ValidationResult;
 /**
  * Validate a partial update to ensure it won't cause issues
  */
 export declare function validatePartialUpdate(currentWorkflow: N8nWorkflow, nodeName: string, newParameters: Record<string, unknown>): ValidationWarning[];
-/**
- * Validate that all node types in an array exist in the available types
- * Returns errors for any invalid node types with suggestions
- */
-export declare function validateNodeTypes(nodes: Array<{
-    name: string;
-    type: string;
-}>, availableTypes: Set<string>): NodeTypeValidationError[];

package/dist/validators.js

@@ -236,88 +236,3 @@ export function validatePartialUpdate(currentWorkflow, nodeName, newParameters)
     }
     return warnings;
 }
-// ─────────────────────────────────────────────────────────────
-// Node Type Validation
-// ─────────────────────────────────────────────────────────────
-/**
- * Validate that all node types in an array exist in the available types
- * Returns errors for any invalid node types with suggestions
- */
-export function validateNodeTypes(nodes, availableTypes) {
-    const errors = [];
-    for (const node of nodes) {
-        if (!availableTypes.has(node.type)) {
-            errors.push({
-                nodeType: node.type,
-                nodeName: node.name,
-                message: `Invalid node type "${node.type}" for node "${node.name}"`,
-                suggestions: findSimilarTypes(node.type, availableTypes),
-            });
-        }
-    }
-    return errors;
-}
-/**
- * Find similar node types for suggestions (fuzzy matching)
- * Returns up to 3 suggestions
- */
-function findSimilarTypes(invalidType, availableTypes) {
-    const suggestions = [];
-    const searchTerm = invalidType.toLowerCase();
-    // Extract the last part after the dot (e.g., "webhook" from "n8n-nodes-base.webhook")
-    const typeParts = searchTerm.split('.');
-    const shortName = typeParts[typeParts.length - 1];
-    for (const validType of availableTypes) {
-        const validLower = validType.toLowerCase();
-        const validShortName = validLower.split('.').pop() || '';
-        // Check for partial matches (substring)
-        if (validLower.includes(shortName) ||
-            shortName.includes(validShortName) ||
-            validShortName.includes(shortName)) {
-            suggestions.push(validType);
-            if (suggestions.length >= 3)
-                break;
-            continue;
-        }
-        // Check for typos using Levenshtein distance
-        const distance = levenshteinDistance(shortName, validShortName);
-        const maxLen = Math.max(shortName.length, validShortName.length);
-        // Allow up to 2 character differences for short names, or 20% of length for longer ones
-        const threshold = Math.max(2, Math.floor(maxLen * 0.2));
-        if (distance <= threshold) {
-            suggestions.push(validType);
-            if (suggestions.length >= 3)
-                break;
-        }
-    }
-    return suggestions;
-}
-/**
- * Calculate Levenshtein distance between two strings
- */
-function levenshteinDistance(a, b) {
-    if (a.length === 0)
-        return b.length;
-    if (b.length === 0)
-        return a.length;
-    const matrix = [];
-    // Initialize first column
-    for (let i = 0; i <= a.length; i++) {
-        matrix[i] = [i];
-    }
-    // Initialize first row
-    for (let j = 0; j <= b.length; j++) {
-        matrix[0][j] = j;
-    }
-    // Fill in the rest of the matrix
-    for (let i = 1; i <= a.length; i++) {
-        for (let j = 1; j <= b.length; j++) {
-            const cost = a[i - 1] === b[j - 1] ? 0 : 1;
-            matrix[i][j] = Math.min(matrix[i - 1][j] + 1, // deletion
-            matrix[i][j - 1] + 1, // insertion
-            matrix[i - 1][j - 1] + cost // substitution
-            );
-        }
-    }
-    return matrix[a.length][b.length];
-}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pagelines/n8n-mcp",
-  "version": "0.3.0",
+  "version": "0.3.2",
   "description": "Opinionated MCP server for n8n workflow automation",
   "type": "module",
   "main": "dist/index.js",
@@ -26,7 +26,7 @@
   "license": "MIT",
   "repository": {
     "type": "git",
-    "url": "https://github.com/PageLines/n8n-mcp"
+    "url": "git+https://github.com/PageLines/n8n-mcp.git"
   },
   "engines": {
     "node": ">=18"

package/.github/workflows/ci.yml

@@ -1,38 +0,0 @@
-name: CI
-
-on:
-  push:
-    branches: [main]
-  pull_request:
-    branches: [main]
-
-jobs:
-  test:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-node@v4
-        with:
-          node-version: '20'
-          cache: 'npm'
-      - run: npm ci
-      - run: npm run build
-      - run: npm test -- --run
-
-  publish:
-    needs: test
-    runs-on: ubuntu-latest
-    if: github.ref == 'refs/heads/main' && github.event_name == 'push'
-    steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-node@v4
-        with:
-          node-version: '20'
-          cache: 'npm'
-          registry-url: 'https://registry.npmjs.org'
-      - run: npm ci
-      - run: npm run build
-      - run: npm publish --access public
-        env:
-          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
-        continue-on-error: true # Won't fail if version already exists

package/dist/n8n-client.test.d.ts

@@ -1 +0,0 @@
-export {};