n8n-mcp 2.11.2 → 2.12.2

package/dist/services/workflow-diff-engine.js

@@ -336,7 +336,7 @@ class WorkflowDiffEngine {
         const node = this.findNode(workflow, operation.nodeId, operation.nodeName);
         if (!node)
             return;
-        Object.entries(operation.changes).forEach(([path, value]) => {
+        Object.entries(operation.updates).forEach(([path, value]) => {
             this.setNestedProperty(node, path, value);
         });
     }
@@ -406,17 +406,17 @@ class WorkflowDiffEngine {
             type: 'removeConnection',
             source: operation.source,
             target: operation.target,
-            sourceOutput: operation.changes.sourceOutput,
-            targetInput: operation.changes.targetInput
+            sourceOutput: operation.updates.sourceOutput,
+            targetInput: operation.updates.targetInput
         });
         this.applyAddConnection(workflow, {
             type: 'addConnection',
             source: operation.source,
             target: operation.target,
-            sourceOutput: operation.changes.sourceOutput,
-            targetInput: operation.changes.targetInput,
-            sourceIndex: operation.changes.sourceIndex,
-            targetIndex: operation.changes.targetIndex
+            sourceOutput: operation.updates.sourceOutput,
+            targetInput: operation.updates.targetInput,
+            sourceIndex: operation.updates.sourceIndex,
+            targetIndex: operation.updates.targetIndex
         });
     }
     applyUpdateSettings(workflow, operation) {

package/dist/services/workflow-ownership.d.ts

@@ -0,0 +1,35 @@
+import { InstanceContext } from '../types/instance-context';
+import { N8nApiClient } from './n8n-api-client';
+import { Workflow } from '../types/n8n-api';
+export interface WorkflowOwnershipInfo {
+    workflowId: string;
+    instanceId: string;
+    instanceUrl: string;
+    createdAt: Date;
+    lastVerified: Date;
+}
+export declare class WorkflowOwnershipService {
+    private apiClient?;
+    private ownershipCache;
+    private readonly CACHE_TTL;
+    private readonly INSTANCE_TAG_PREFIX;
+    private readonly MAX_CACHE_SIZE;
+    constructor(apiClient?: N8nApiClient | undefined);
+    setApiClient(client: N8nApiClient): void;
+    private getInstanceIdentifier;
+    private getInstanceTag;
+    private extractInstanceFromTags;
+    validateAccess(workflowId: string, context: InstanceContext): Promise<boolean>;
+    trackOwnership(workflowId: string, context: InstanceContext): Promise<void>;
+    private claimWorkflow;
+    filterWorkflowsByInstance(workflows: Workflow[], context: InstanceContext): Promise<Workflow[]>;
+    private updateOwnershipCache;
+    private isCacheValid;
+    clearCache(): void;
+    getCacheStats(): {
+        size: number;
+        maxSize: number;
+        ttl: number;
+    };
+}
+//# sourceMappingURL=workflow-ownership.d.ts.map

package/dist/services/workflow-ownership.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"workflow-ownership.d.ts","sourceRoot":"","sources":["../../src/services/workflow-ownership.ts"],"names":[],"mappings":"AAWA,OAAO,EAAE,eAAe,EAAE,MAAM,2BAA2B,CAAC;AAC5D,OAAO,EAAE,YAAY,EAAE,MAAM,kBAAkB,CAAC;AAChD,OAAO,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAI5C,MAAM,WAAW,qBAAqB;IACpC,UAAU,EAAE,MAAM,CAAC;IACnB,UAAU,EAAE,MAAM,CAAC;IACnB,WAAW,EAAE,MAAM,CAAC;IACpB,SAAS,EAAE,IAAI,CAAC;IAChB,YAAY,EAAE,IAAI,CAAC;CACpB;AAKD,qBAAa,wBAAwB;IAavB,OAAO,CAAC,SAAS,CAAC;IAX9B,OAAO,CAAC,cAAc,CAA4C;IAGlE,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAiB;IAG3C,OAAO,CAAC,QAAQ,CAAC,mBAAmB,CAAmB;IAGvD,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAS;gBAEpB,SAAS,CAAC,EAAE,YAAY,YAAA;IAK5C,YAAY,CAAC,MAAM,EAAE,YAAY,GAAG,IAAI;IAOxC,OAAO,CAAC,qBAAqB;IAW7B,OAAO,CAAC,cAAc;IAOtB,OAAO,CAAC,uBAAuB;IAYzB,cAAc,CAAC,UAAU,EAAE,MAAM,EAAE,OAAO,EAAE,eAAe,GAAG,OAAO,CAAC,OAAO,CAAC;IAiF9E,cAAc,CAAC,UAAU,EAAE,MAAM,EAAE,OAAO,EAAE,eAAe,GAAG,OAAO,CAAC,IAAI,CAAC;YA6CnE,aAAa;IAkCrB,yBAAyB,CAAC,SAAS,EAAE,QAAQ,EAAE,EAAE,OAAO,EAAE,eAAe,GAAG,OAAO,CAAC,QAAQ,EAAE,CAAC;IA2BrG,OAAO,CAAC,oBAAoB;IA0B5B,OAAO,CAAC,YAAY;IAQpB,UAAU,IAAI,IAAI;IAQlB,aAAa,IAAI;QACf,IAAI,EAAE,MAAM,CAAC;QACb,OAAO,EAAE,MAAM,CAAC;QAChB,GAAG,EAAE,MAAM,CAAC;KACb;CAOF"}

package/dist/services/workflow-ownership.js

@@ -0,0 +1,195 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.WorkflowOwnershipService = void 0;
+const logger_1 = require("../utils/logger");
+const cache_utils_1 = require("../utils/cache-utils");
+class WorkflowOwnershipService {
+    constructor(apiClient) {
+        this.apiClient = apiClient;
+        this.ownershipCache = new Map();
+        this.CACHE_TTL = 5 * 60 * 1000;
+        this.INSTANCE_TAG_PREFIX = 'mcp-instance:';
+        this.MAX_CACHE_SIZE = 10000;
+    }
+    setApiClient(client) {
+        this.apiClient = client;
+    }
+    getInstanceIdentifier(context) {
+        if (context.instanceId) {
+            return context.instanceId;
+        }
+        return (0, cache_utils_1.createCacheKey)(`${context.n8nApiUrl}:${context.n8nApiKey}`).substring(0, 16);
+    }
+    getInstanceTag(context) {
+        return `${this.INSTANCE_TAG_PREFIX}${this.getInstanceIdentifier(context)}`;
+    }
+    extractInstanceFromTags(tags) {
+        if (!tags || tags.length === 0)
+            return null;
+        const instanceTag = tags.find(tag => tag.startsWith(this.INSTANCE_TAG_PREFIX));
+        if (!instanceTag)
+            return null;
+        return instanceTag.replace(this.INSTANCE_TAG_PREFIX, '');
+    }
+    async validateAccess(workflowId, context) {
+        if (!context?.n8nApiUrl || !context?.n8nApiKey) {
+            logger_1.logger.debug('No instance context provided, allowing access for backward compatibility');
+            return true;
+        }
+        const instanceId = this.getInstanceIdentifier(context);
+        const cached = this.ownershipCache.get(workflowId);
+        if (cached && this.isCacheValid(cached)) {
+            const hasAccess = cached.instanceId === instanceId;
+            logger_1.logger.debug('Workflow ownership cache hit', {
+                workflowId,
+                instanceId,
+                cachedOwner: cached.instanceId,
+                hasAccess
+            });
+            return hasAccess;
+        }
+        logger_1.logger.debug('Workflow ownership cache miss, verifying ownership', {
+            workflowId,
+            instanceId
+        });
+        try {
+            if (!this.apiClient) {
+                logger_1.logger.warn('No API client available for ownership verification');
+                return true;
+            }
+            const workflow = await this.apiClient.getWorkflow(workflowId);
+            const workflowInstanceId = this.extractInstanceFromTags(workflow.tags);
+            if (workflowInstanceId === null) {
+                logger_1.logger.info('Workflow has no instance tag, treating as unclaimed', {
+                    workflowId,
+                    workflowName: workflow.name
+                });
+                await this.claimWorkflow(workflowId, workflow, context);
+                return true;
+            }
+            this.updateOwnershipCache(workflowId, workflowInstanceId, context.n8nApiUrl || '');
+            const hasAccess = workflowInstanceId === instanceId;
+            if (!hasAccess) {
+                logger_1.logger.warn('Access denied: workflow belongs to different instance', {
+                    workflowId,
+                    requestingInstance: instanceId,
+                    owningInstance: workflowInstanceId
+                });
+            }
+            return hasAccess;
+        }
+        catch (error) {
+            logger_1.logger.error('Error validating workflow ownership', {
+                workflowId,
+                instanceId,
+                error: error instanceof Error ? error.message : String(error)
+            });
+            return false;
+        }
+    }
+    async trackOwnership(workflowId, context) {
+        if (!context?.n8nApiUrl || !context?.n8nApiKey) {
+            logger_1.logger.debug('No instance context for ownership tracking');
+            return;
+        }
+        const instanceId = this.getInstanceIdentifier(context);
+        const instanceTag = this.getInstanceTag(context);
+        try {
+            if (this.apiClient) {
+                const workflow = await this.apiClient.getWorkflow(workflowId);
+                const tags = workflow.tags || [];
+                if (!tags.some(tag => tag.startsWith(this.INSTANCE_TAG_PREFIX))) {
+                    tags.push(instanceTag);
+                    await this.apiClient.updateWorkflow(workflowId, { tags });
+                    logger_1.logger.info('Added instance ownership tag to workflow', {
+                        workflowId,
+                        instanceId,
+                        instanceTag
+                    });
+                }
+            }
+            this.updateOwnershipCache(workflowId, instanceId, context.n8nApiUrl);
+        }
+        catch (error) {
+            logger_1.logger.error('Error tracking workflow ownership', {
+                workflowId,
+                instanceId,
+                error: error instanceof Error ? error.message : String(error)
+            });
+        }
+    }
+    async claimWorkflow(workflowId, workflow, context) {
+        if (!this.apiClient)
+            return;
+        const instanceTag = this.getInstanceTag(context);
+        const tags = workflow.tags || [];
+        tags.push(instanceTag);
+        try {
+            await this.apiClient.updateWorkflow(workflowId, { tags });
+            logger_1.logger.info('Claimed unclaimed workflow for instance', {
+                workflowId,
+                workflowName: workflow.name,
+                instanceId: this.getInstanceIdentifier(context)
+            });
+            this.updateOwnershipCache(workflowId, this.getInstanceIdentifier(context), context.n8nApiUrl || '');
+        }
+        catch (error) {
+            logger_1.logger.error('Error claiming workflow', {
+                workflowId,
+                error: error instanceof Error ? error.message : String(error)
+            });
+        }
+    }
+    async filterWorkflowsByInstance(workflows, context) {
+        if (!context?.n8nApiUrl || !context?.n8nApiKey) {
+            return workflows;
+        }
+        const instanceId = this.getInstanceIdentifier(context);
+        return workflows.filter(workflow => {
+            const workflowInstanceId = this.extractInstanceFromTags(workflow.tags);
+            if (workflowInstanceId === null) {
+                logger_1.logger.debug('Including unclaimed workflow in list', {
+                    workflowId: workflow.id,
+                    workflowName: workflow.name
+                });
+                return true;
+            }
+            return workflowInstanceId === instanceId;
+        });
+    }
+    updateOwnershipCache(workflowId, instanceId, instanceUrl) {
+        if (this.ownershipCache.size >= this.MAX_CACHE_SIZE) {
+            const entriesToRemove = Math.floor(this.MAX_CACHE_SIZE * 0.1);
+            const keys = Array.from(this.ownershipCache.keys()).slice(0, entriesToRemove);
+            keys.forEach(key => this.ownershipCache.delete(key));
+            logger_1.logger.debug('Evicted ownership cache entries', {
+                evicted: entriesToRemove,
+                remaining: this.ownershipCache.size
+            });
+        }
+        this.ownershipCache.set(workflowId, {
+            workflowId,
+            instanceId,
+            instanceUrl,
+            createdAt: new Date(),
+            lastVerified: new Date()
+        });
+    }
+    isCacheValid(entry) {
+        const age = Date.now() - entry.lastVerified.getTime();
+        return age < this.CACHE_TTL;
+    }
+    clearCache() {
+        this.ownershipCache.clear();
+        logger_1.logger.debug('Cleared workflow ownership cache');
+    }
+    getCacheStats() {
+        return {
+            size: this.ownershipCache.size,
+            maxSize: this.MAX_CACHE_SIZE,
+            ttl: this.CACHE_TTL
+        };
+    }
+}
+exports.WorkflowOwnershipService = WorkflowOwnershipService;
+//# sourceMappingURL=workflow-ownership.js.map

package/dist/services/workflow-ownership.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"workflow-ownership.js","sourceRoot":"","sources":["../../src/services/workflow-ownership.ts"],"names":[],"mappings":";;;AAcA,4CAAyC;AACzC,sDAAsD;AAatD,MAAa,wBAAwB;IAanC,YAAoB,SAAwB;QAAxB,cAAS,GAAT,SAAS,CAAe;QAXpC,mBAAc,GAAG,IAAI,GAAG,EAAiC,CAAC;QAGjD,cAAS,GAAG,CAAC,GAAG,EAAE,GAAG,IAAI,CAAC;QAG1B,wBAAmB,GAAG,eAAe,CAAC;QAGtC,mBAAc,GAAG,KAAK,CAAC;IAEO,CAAC;IAKhD,YAAY,CAAC,MAAoB;QAC/B,IAAI,CAAC,SAAS,GAAG,MAAM,CAAC;IAC1B,CAAC;IAKO,qBAAqB,CAAC,OAAwB;QACpD,IAAI,OAAO,CAAC,UAAU,EAAE,CAAC;YACvB,OAAO,OAAO,CAAC,UAAU,CAAC;QAC5B,CAAC;QAED,OAAO,IAAA,4BAAc,EAAC,GAAG,OAAO,CAAC,SAAS,IAAI,OAAO,CAAC,SAAS,EAAE,CAAC,CAAC,SAAS,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;IACtF,CAAC;IAKO,cAAc,CAAC,OAAwB;QAC7C,OAAO,GAAG,IAAI,CAAC,mBAAmB,GAAG,IAAI,CAAC,qBAAqB,CAAC,OAAO,CAAC,EAAE,CAAC;IAC7E,CAAC;IAKO,uBAAuB,CAAC,IAAe;QAC7C,IAAI,CAAC,IAAI,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC;YAAE,OAAO,IAAI,CAAC;QAE5C,MAAM,WAAW,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,CAAC,UAAU,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC,CAAC;QAC/E,IAAI,CAAC,WAAW;YAAE,OAAO,IAAI,CAAC;QAE9B,OAAO,WAAW,CAAC,OAAO,CAAC,IAAI,CAAC,mBAAmB,EAAE,EAAE,CAAC,CAAC;IAC3D,CAAC;IAKD,KAAK,CAAC,cAAc,CAAC,UAAkB,EAAE,OAAwB;QAE/D,IAAI,CAAC,OAAO,EAAE,SAAS,IAAI,CAAC,OAAO,EAAE,SAAS,EAAE,CAAC;YAC/C,eAAM,CAAC,KAAK,CAAC,0EAA0E,CAAC,CAAC;YACzF,OAAO,IAAI,CAAC;QACd,CAAC;QAED,MAAM,UAAU,GAAG,IAAI,CAAC,qBAAqB,CAAC,OAAO,CAAC,CAAC;QAGvD,MAAM,MAAM,GAAG,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,UAAU,CAAC,CAAC;QACnD,IAAI,MAAM,IAAI,IAAI,CAAC,YAAY,CAAC,MAAM,CAAC,EAAE,CAAC;YACxC,MAAM,SAAS,GAAG,MAAM,CAAC,UAAU,KAAK,UAAU,CAAC;YACnD,eAAM,CAAC,KAAK,CAAC,8BAA8B,EAAE;gBAC3C,UAAU;gBACV,UAAU;gBACV,WAAW,EAAE,MAAM,CAAC,UAAU;gBAC9B,SAAS;aACV,CAAC,CAAC;YACH,OAAO,SAAS,CAAC;QACnB,CAAC;QAGD,eAAM,CAAC,KAAK,CAAC,oDAAoD,EAAE;YACjE,UAAU;YACV,UAAU;SACX,CAAC,CAAC;QAEH,IAAI,CAAC;YAEH,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,CAAC;gBACpB,eAAM,CAAC,IAAI,CAAC,oDAAoD,CAAC,CAAC;gBAClE,OAAO,IAAI,CAAC;YACd,CAAC;YAED,MAAM,QAAQ,GAAG,MAAM,IAAI,CAAC,SAAS,CAAC,WAAW,CAAC,UAAU,CAAC,CAAC;YAG9D,MAAM,kBAAkB,GAAG,IAAI,CAAC,uBAAuB,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;YAEvE,IAAI,kBAAkB,KAAK,IAAI,EAAE,CAAC;gBAEhC,eAAM,CAAC,IAAI,CAAC,qDAAqD,EAAE;oBACjE,UAAU;oBACV,YAAY,EAAE,QAAQ,CAAC,IAAI;iBAC5B,CAAC,CAAC;gBAGH,MAAM,IAAI,CAAC,aAAa,CAAC,UAAU,EAAE,QAAQ,EAAE,OAAO,CAAC,CAAC;gBACxD,OAAO,IAAI,CAAC;YACd,CAAC;YAGD,IAAI,CAAC,oBAAoB,CAAC,UAAU,EAAE,kBAAkB,EAAE,OAAO,CAAC,SAAS,IAAI,EAAE,CAAC,CAAC;YAEnF,MAAM,SAAS,GAAG,kBAAkB,KAAK,UAAU,CAAC;YAEpD,IAAI,CAAC,SAAS,EAAE,CAAC;gBACf,eAAM,CAAC,IAAI,CAAC,uDAAuD,EAAE;oBACnE,UAAU;oBACV,kBAAkB,EAAE,UAAU;oBAC9B,cAAc,EAAE,kBAAkB;iBACnC,CAAC,CAAC;YACL,CAAC;YAED,OAAO,SAAS,CAAC;QAEnB,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YAEf,eAAM,CAAC,KAAK,CAAC,qCAAqC,EAAE;gBAClD,UAAU;gBACV,UAAU;gBACV,KAAK,EAAE,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC;aAC9D,CAAC,CAAC;YACH,OAAO,KAAK,CAAC;QACf,CAAC;IACH,CAAC;IAKD,KAAK,CAAC,cAAc,CAAC,UAAkB,EAAE,OAAwB;QAC/D,IAAI,CAAC,OAAO,EAAE,SAAS,IAAI,CAAC,OAAO,EAAE,SAAS,EAAE,CAAC;YAC/C,eAAM,CAAC,KAAK,CAAC,4CAA4C,CAAC,CAAC;YAC3D,OAAO;QACT,CAAC;QAED,MAAM,UAAU,GAAG,IAAI,CAAC,qBAAqB,CAAC,OAAO,CAAC,CAAC;QACvD,MAAM,WAAW,GAAG,IAAI,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC;QAEjD,IAAI,CAAC;YACH,IAAI,IAAI,CAAC,SAAS,EAAE,CAAC;gBAEnB,MAAM,QAAQ,GAAG,MAAM,IAAI,CAAC,SAAS,CAAC,WAAW,CAAC,UAAU,CAAC,CAAC;gBAG9D,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,IAAI,EAAE,CAAC;gBACjC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,CAAC,UAAU,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC,EAAE,CAAC;oBAChE,IAAI,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;oBAGvB,MAAM,IAAI,CAAC,SAAS,CAAC,cAAc,CAAC,UAAU,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC;oBAE1D,eAAM,CAAC,IAAI,CAAC,0CAA0C,EAAE;wBACtD,UAAU;wBACV,UAAU;wBACV,WAAW;qBACZ,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;YAGD,IAAI,CAAC,oBAAoB,CAAC,UAAU,EAAE,UAAU,EAAE,OAAO,CAAC,SAAS,CAAC,CAAC;QAEvE,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,eAAM,CAAC,KAAK,CAAC,mCAAmC,EAAE;gBAChD,UAAU;gBACV,UAAU;gBACV,KAAK,EAAE,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC;aAC9D,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAKO,KAAK,CAAC,aAAa,CAAC,UAAkB,EAAE,QAAkB,EAAE,OAAwB;QAC1F,IAAI,CAAC,IAAI,CAAC,SAAS;YAAE,OAAO;QAE5B,MAAM,WAAW,GAAG,IAAI,CAAC,cAAc,CAAC,OAAO,CAAC,CAAC;QACjD,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,IAAI,EAAE,CAAC;QAGjC,IAAI,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;QAEvB,IAAI,CAAC;YACH,MAAM,IAAI,CAAC,SAAS,CAAC,cAAc,CAAC,UAAU,EAAE,EAAE,IAAI,EAAE,CAAC,CAAC;YAC1D,eAAM,CAAC,IAAI,CAAC,yCAAyC,EAAE;gBACrD,UAAU;gBACV,YAAY,EAAE,QAAQ,CAAC,IAAI;gBAC3B,UAAU,EAAE,IAAI,CAAC,qBAAqB,CAAC,OAAO,CAAC;aAChD,CAAC,CAAC;YAGH,IAAI,CAAC,oBAAoB,CACvB,UAAU,EACV,IAAI,CAAC,qBAAqB,CAAC,OAAO,CAAC,EACnC,OAAO,CAAC,SAAS,IAAI,EAAE,CACxB,CAAC;QACJ,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,eAAM,CAAC,KAAK,CAAC,yBAAyB,EAAE;gBACtC,UAAU;gBACV,KAAK,EAAE,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC;aAC9D,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAKD,KAAK,CAAC,yBAAyB,CAAC,SAAqB,EAAE,OAAwB;QAC7E,IAAI,CAAC,OAAO,EAAE,SAAS,IAAI,CAAC,OAAO,EAAE,SAAS,EAAE,CAAC;YAE/C,OAAO,SAAS,CAAC;QACnB,CAAC;QAED,MAAM,UAAU,GAAG,IAAI,CAAC,qBAAqB,CAAC,OAAO,CAAC,CAAC;QAEvD,OAAO,SAAS,CAAC,MAAM,CAAC,QAAQ,CAAC,EAAE;YACjC,MAAM,kBAAkB,GAAG,IAAI,CAAC,uBAAuB,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;YAEvE,IAAI,kBAAkB,KAAK,IAAI,EAAE,CAAC;gBAEhC,eAAM,CAAC,KAAK,CAAC,sCAAsC,EAAE;oBACnD,UAAU,EAAE,QAAQ,CAAC,EAAE;oBACvB,YAAY,EAAE,QAAQ,CAAC,IAAI;iBAC5B,CAAC,CAAC;gBACH,OAAO,IAAI,CAAC;YACd,CAAC;YAED,OAAO,kBAAkB,KAAK,UAAU,CAAC;QAC3C,CAAC,CAAC,CAAC;IACL,CAAC;IAKO,oBAAoB,CAAC,UAAkB,EAAE,UAAkB,EAAE,WAAmB;QAEtF,IAAI,IAAI,CAAC,cAAc,CAAC,IAAI,IAAI,IAAI,CAAC,cAAc,EAAE,CAAC;YAEpD,MAAM,eAAe,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,cAAc,GAAG,GAAG,CAAC,CAAC;YAC9D,MAAM,IAAI,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,cAAc,CAAC,IAAI,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,eAAe,CAAC,CAAC;YAC9E,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,IAAI,CAAC,cAAc,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC;YAErD,eAAM,CAAC,KAAK,CAAC,iCAAiC,EAAE;gBAC9C,OAAO,EAAE,eAAe;gBACxB,SAAS,EAAE,IAAI,CAAC,cAAc,CAAC,IAAI;aACpC,CAAC,CAAC;QACL,CAAC;QAED,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,UAAU,EAAE;YAClC,UAAU;YACV,UAAU;YACV,WAAW;YACX,SAAS,EAAE,IAAI,IAAI,EAAE;YACrB,YAAY,EAAE,IAAI,IAAI,EAAE;SACzB,CAAC,CAAC;IACL,CAAC;IAKO,YAAY,CAAC,KAA4B;QAC/C,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,EAAE,GAAG,KAAK,CAAC,YAAY,CAAC,OAAO,EAAE,CAAC;QACtD,OAAO,GAAG,GAAG,IAAI,CAAC,SAAS,CAAC;IAC9B,CAAC;IAKD,UAAU;QACR,IAAI,CAAC,cAAc,CAAC,KAAK,EAAE,CAAC;QAC5B,eAAM,CAAC,KAAK,CAAC,kCAAkC,CAAC,CAAC;IACnD,CAAC;IAKD,aAAa;QAKX,OAAO;YACL,IAAI,EAAE,IAAI,CAAC,cAAc,CAAC,IAAI;YAC9B,OAAO,EAAE,IAAI,CAAC,cAAc;YAC5B,GAAG,EAAE,IAAI,CAAC,SAAS;SACpB,CAAC;IACJ,CAAC;CACF;AAvSD,4DAuSC"}

package/dist/services/workflow-validator.d.ts

@@ -84,6 +84,7 @@ export declare class WorkflowValidator {
     private validateAllNodes;
     private validateConnections;
     private validateConnectionOutputs;
+    private validateErrorOutputConfiguration;
     private validateAIToolConnection;
     private hasCycle;
     private validateExpressions;

package/dist/services/workflow-validator.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"workflow-validator.d.ts","sourceRoot":"","sources":["../../src/services/workflow-validator.ts"],"names":[],"mappings":"AAKA,OAAO,EAAE,cAAc,EAAE,MAAM,6BAA6B,CAAC;AAC7D,OAAO,EAAE,uBAAuB,EAAE,MAAM,6BAA6B,CAAC;AAMtE,UAAU,YAAY;IACpB,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC3B,UAAU,EAAE,GAAG,CAAC;IAChB,WAAW,CAAC,EAAE,GAAG,CAAC;IAClB,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,cAAc,CAAC,EAAE,OAAO,CAAC;IACzB,OAAO,CAAC,EAAE,uBAAuB,GAAG,qBAAqB,GAAG,cAAc,CAAC;IAC3E,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,gBAAgB,CAAC,EAAE,OAAO,CAAC;IAC3B,WAAW,CAAC,EAAE,OAAO,CAAC;CACvB;AAED,UAAU,kBAAkB;IAC1B,CAAC,UAAU,EAAE,MAAM,GAAG;QACpB,IAAI,CAAC,EAAE,KAAK,CAAC,KAAK,CAAC;YAAE,IAAI,EAAE,MAAM,CAAC;YAAC,IAAI,EAAE,MAAM,CAAC;YAAC,KAAK,EAAE,MAAM,CAAA;SAAE,CAAC,CAAC,CAAC;QACnE,KAAK,CAAC,EAAE,KAAK,CAAC,KAAK,CAAC;YAAE,IAAI,EAAE,MAAM,CAAC;YAAC,IAAI,EAAE,MAAM,CAAC;YAAC,KAAK,EAAE,MAAM,CAAA;SAAE,CAAC,CAAC,CAAC;QACpE,OAAO,CAAC,EAAE,KAAK,CAAC,KAAK,CAAC;YAAE,IAAI,EAAE,MAAM,CAAC;YAAC,IAAI,EAAE,MAAM,CAAC;YAAC,KAAK,EAAE,MAAM,CAAA;SAAE,CAAC,CAAC,CAAC;KACvE,CAAC;CACH;AAED,UAAU,YAAY;IACpB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,YAAY,EAAE,CAAC;IACtB,WAAW,EAAE,kBAAkB,CAAC;IAChC,QAAQ,CAAC,EAAE,GAAG,CAAC;IACf,UAAU,CAAC,EAAE,GAAG,CAAC;IACjB,OAAO,CAAC,EAAE,GAAG,CAAC;IACd,IAAI,CAAC,EAAE,GAAG,CAAC;CACZ;AAED,UAAU,eAAe;IACvB,IAAI,EAAE,OAAO,GAAG,SAAS,CAAC;IAC1B,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,CAAC,EAAE,GAAG,CAAC;CACf;AAED,MAAM,WAAW,wBAAwB;IACvC,KAAK,EAAE,OAAO,CAAC;IACf,MAAM,EAAE,eAAe,EAAE,CAAC;IAC1B,QAAQ,EAAE,eAAe,EAAE,CAAC;IAC5B,UAAU,EAAE;QACV,UAAU,EAAE,MAAM,CAAC;QACnB,YAAY,EAAE,MAAM,CAAC;QACrB,YAAY,EAAE,MAAM,CAAC;QACrB,gBAAgB,EAAE,MAAM,CAAC;QACzB,kBAAkB,EAAE,MAAM,CAAC;QAC3B,oBAAoB,EAAE,MAAM,CAAC;KAC9B,CAAC;IACF,WAAW,EAAE,MAAM,EAAE,CAAC;CACvB;AAED,qBAAa,iBAAiB;IAI1B,OAAO,CAAC,cAAc;IACtB,OAAO,CAAC,aAAa;IAJvB,OAAO,CAAC,eAAe,CAA6B;gBAG1C,cAAc,EAAE,cAAc,EAC9B,aAAa,EAAE,OAAO,uBAAuB;IAMvD,OAAO,CAAC,YAAY;IAYd,gBAAgB,CACpB,QAAQ,EAAE,YAAY,EACtB,OAAO,GAAE;QACP,aAAa,CAAC,EAAE,OAAO,CAAC;QACxB,mBAAmB,CAAC,EAAE,OAAO,CAAC;QAC9B,mBAAmB,CAAC,EAAE,OAAO,CAAC;QAC9B,OAAO,CAAC,EAAE,SAAS,GAAG,SAAS,GAAG,aAAa,GAAG,QAAQ,CAAC;KACvD,GACL,OAAO,CAAC,wBAAwB,CAAC;IA2FpC,OAAO,CAAC,yBAAyB;YA+HnB,gBAAgB;IAoM9B,OAAO,CAAC,mBAAmB;IA+H3B,OAAO,CAAC,yBAAyB;IAsFjC,OAAO,CAAC,wBAAwB;IA0ChC,OAAO,CAAC,QAAQ;IAsFhB,OAAO,CAAC,mBAAmB;IAoD3B,OAAO,CAAC,wBAAwB;IA2BhC,OAAO,CAAC,YAAY;IAgBpB,OAAO,CAAC,qBAAqB;IAyF7B,OAAO,CAAC,qBAAqB;IA6C7B,OAAO,CAAC,oBAAoB;IA4D5B,OAAO,CAAC,mBAAmB;IAmE3B,OAAO,CAAC,sBAAsB;IAoT9B,OAAO,CAAC,gCAAgC;IA8BxC,OAAO,CAAC,gCAAgC;IAsFxC,OAAO,CAAC,gBAAgB;IA4CxB,OAAO,CAAC,2BAA2B;CAmEpC"}
+{"version":3,"file":"workflow-validator.d.ts","sourceRoot":"","sources":["../../src/services/workflow-validator.ts"],"names":[],"mappings":"AAKA,OAAO,EAAE,cAAc,EAAE,MAAM,6BAA6B,CAAC;AAC7D,OAAO,EAAE,uBAAuB,EAAE,MAAM,6BAA6B,CAAC;AAMtE,UAAU,YAAY;IACpB,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC3B,UAAU,EAAE,GAAG,CAAC;IAChB,WAAW,CAAC,EAAE,GAAG,CAAC;IAClB,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,cAAc,CAAC,EAAE,OAAO,CAAC;IACzB,OAAO,CAAC,EAAE,uBAAuB,GAAG,qBAAqB,GAAG,cAAc,CAAC;IAC3E,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,gBAAgB,CAAC,EAAE,OAAO,CAAC;IAC3B,WAAW,CAAC,EAAE,OAAO,CAAC;CACvB;AAED,UAAU,kBAAkB;IAC1B,CAAC,UAAU,EAAE,MAAM,GAAG;QACpB,IAAI,CAAC,EAAE,KAAK,CAAC,KAAK,CAAC;YAAE,IAAI,EAAE,MAAM,CAAC;YAAC,IAAI,EAAE,MAAM,CAAC;YAAC,KAAK,EAAE,MAAM,CAAA;SAAE,CAAC,CAAC,CAAC;QACnE,KAAK,CAAC,EAAE,KAAK,CAAC,KAAK,CAAC;YAAE,IAAI,EAAE,MAAM,CAAC;YAAC,IAAI,EAAE,MAAM,CAAC;YAAC,KAAK,EAAE,MAAM,CAAA;SAAE,CAAC,CAAC,CAAC;QACpE,OAAO,CAAC,EAAE,KAAK,CAAC,KAAK,CAAC;YAAE,IAAI,EAAE,MAAM,CAAC;YAAC,IAAI,EAAE,MAAM,CAAC;YAAC,KAAK,EAAE,MAAM,CAAA;SAAE,CAAC,CAAC,CAAC;KACvE,CAAC;CACH;AAED,UAAU,YAAY;IACpB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,YAAY,EAAE,CAAC;IACtB,WAAW,EAAE,kBAAkB,CAAC;IAChC,QAAQ,CAAC,EAAE,GAAG,CAAC;IACf,UAAU,CAAC,EAAE,GAAG,CAAC;IACjB,OAAO,CAAC,EAAE,GAAG,CAAC;IACd,IAAI,CAAC,EAAE,GAAG,CAAC;CACZ;AAED,UAAU,eAAe;IACvB,IAAI,EAAE,OAAO,GAAG,SAAS,CAAC;IAC1B,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,CAAC,EAAE,GAAG,CAAC;CACf;AAED,MAAM,WAAW,wBAAwB;IACvC,KAAK,EAAE,OAAO,CAAC;IACf,MAAM,EAAE,eAAe,EAAE,CAAC;IAC1B,QAAQ,EAAE,eAAe,EAAE,CAAC;IAC5B,UAAU,EAAE;QACV,UAAU,EAAE,MAAM,CAAC;QACnB,YAAY,EAAE,MAAM,CAAC;QACrB,YAAY,EAAE,MAAM,CAAC;QACrB,gBAAgB,EAAE,MAAM,CAAC;QACzB,kBAAkB,EAAE,MAAM,CAAC;QAC3B,oBAAoB,EAAE,MAAM,CAAC;KAC9B,CAAC;IACF,WAAW,EAAE,MAAM,EAAE,CAAC;CACvB;AAED,qBAAa,iBAAiB;IAI1B,OAAO,CAAC,cAAc;IACtB,OAAO,CAAC,aAAa;IAJvB,OAAO,CAAC,eAAe,CAA6B;gBAG1C,cAAc,EAAE,cAAc,EAC9B,aAAa,EAAE,OAAO,uBAAuB;IAMvD,OAAO,CAAC,YAAY;IAYd,gBAAgB,CACpB,QAAQ,EAAE,YAAY,EACtB,OAAO,GAAE;QACP,aAAa,CAAC,EAAE,OAAO,CAAC;QACxB,mBAAmB,CAAC,EAAE,OAAO,CAAC;QAC9B,mBAAmB,CAAC,EAAE,OAAO,CAAC;QAC9B,OAAO,CAAC,EAAE,SAAS,GAAG,SAAS,GAAG,aAAa,GAAG,QAAQ,CAAC;KACvD,GACL,OAAO,CAAC,wBAAwB,CAAC;IA2FpC,OAAO,CAAC,yBAAyB;YA+HnB,gBAAgB;IAoM9B,OAAO,CAAC,mBAAmB;IA+H3B,OAAO,CAAC,yBAAyB;IA2FjC,OAAO,CAAC,gCAAgC;IAoFxC,OAAO,CAAC,wBAAwB;IA0ChC,OAAO,CAAC,QAAQ;IAsFhB,OAAO,CAAC,mBAAmB;IAqF3B,OAAO,CAAC,wBAAwB;IA2BhC,OAAO,CAAC,YAAY;IAgBpB,OAAO,CAAC,qBAAqB;IAyF7B,OAAO,CAAC,qBAAqB;IA6C7B,OAAO,CAAC,oBAAoB;IA4D5B,OAAO,CAAC,mBAAmB;IAmE3B,OAAO,CAAC,sBAAsB;IAoT9B,OAAO,CAAC,gCAAgC;IA8BxC,OAAO,CAAC,gCAAgC;IAsFxC,OAAO,CAAC,gBAAgB;IA4CxB,OAAO,CAAC,2BAA2B;CAmEpC"}

package/dist/services/workflow-validator.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.WorkflowValidator = void 0;
 const expression_validator_1 = require("./expression-validator");
+const expression_format_validator_1 = require("./expression-format-validator");
 const logger_1 = require("../utils/logger");
 const logger = new logger_1.Logger({ prefix: '[WorkflowValidator]' });
 class WorkflowValidator {
@@ -411,6 +412,9 @@ class WorkflowValidator {
     }
     validateConnectionOutputs(sourceName, outputs, nodeMap, nodeIdMap, result, outputType) {
         const sourceNode = nodeMap.get(sourceName);
+        if (outputType === 'main' && sourceNode) {
+            this.validateErrorOutputConfiguration(sourceName, sourceNode, outputs, nodeMap, result);
+        }
         outputs.forEach((outputConnections, outputIndex) => {
             if (!outputConnections)
                 return;
@@ -468,6 +472,70 @@ class WorkflowValidator {
             });
         });
     }
+    validateErrorOutputConfiguration(sourceName, sourceNode, outputs, nodeMap, result) {
+        const hasErrorOutputSetting = sourceNode.onError === 'continueErrorOutput';
+        const hasErrorConnections = outputs.length > 1 && outputs[1] && outputs[1].length > 0;
+        if (hasErrorOutputSetting && !hasErrorConnections) {
+            result.errors.push({
+                type: 'error',
+                nodeId: sourceNode.id,
+                nodeName: sourceNode.name,
+                message: `Node has onError: 'continueErrorOutput' but no error output connections in main[1]. Add error handler connections to main[1] or change onError to 'continueRegularOutput' or 'stopWorkflow'.`
+            });
+        }
+        if (!hasErrorOutputSetting && hasErrorConnections) {
+            result.warnings.push({
+                type: 'warning',
+                nodeId: sourceNode.id,
+                nodeName: sourceNode.name,
+                message: `Node has error output connections in main[1] but missing onError: 'continueErrorOutput'. Add this property to properly handle errors.`
+            });
+        }
+        if (outputs.length >= 1 && outputs[0] && outputs[0].length > 1) {
+            const potentialErrorHandlers = outputs[0].filter(conn => {
+                const targetNode = nodeMap.get(conn.node);
+                if (!targetNode)
+                    return false;
+                const nodeName = targetNode.name.toLowerCase();
+                const nodeType = targetNode.type.toLowerCase();
+                return nodeName.includes('error') ||
+                    nodeName.includes('fail') ||
+                    nodeName.includes('catch') ||
+                    nodeName.includes('exception') ||
+                    nodeType.includes('respondtowebhook') ||
+                    nodeType.includes('emailsend');
+            });
+            if (potentialErrorHandlers.length > 0) {
+                const errorHandlerNames = potentialErrorHandlers.map(conn => `"${conn.node}"`).join(', ');
+                result.errors.push({
+                    type: 'error',
+                    nodeId: sourceNode.id,
+                    nodeName: sourceNode.name,
+                    message: `Incorrect error output configuration. Nodes ${errorHandlerNames} appear to be error handlers but are in main[0] (success output) along with other nodes.\n\n` +
+                        `INCORRECT (current):\n` +
+                        `"${sourceName}": {\n` +
+                        `  "main": [\n` +
+                        `    [  // main[0] has multiple nodes mixed together\n` +
+                        outputs[0].map(conn => `      {"node": "${conn.node}", "type": "${conn.type}", "index": ${conn.index}}`).join(',\n') + '\n' +
+                        `    ]\n` +
+                        `  ]\n` +
+                        `}\n\n` +
+                        `CORRECT (should be):\n` +
+                        `"${sourceName}": {\n` +
+                        `  "main": [\n` +
+                        `    [  // main[0] = success output\n` +
+                        outputs[0].filter(conn => !potentialErrorHandlers.includes(conn)).map(conn => `      {"node": "${conn.node}", "type": "${conn.type}", "index": ${conn.index}}`).join(',\n') + '\n' +
+                        `    ],\n` +
+                        `    [  // main[1] = error output\n` +
+                        potentialErrorHandlers.map(conn => `      {"node": "${conn.node}", "type": "${conn.type}", "index": ${conn.index}}`).join(',\n') + '\n' +
+                        `    ]\n` +
+                        `  ]\n` +
+                        `}\n\n` +
+                        `Also add: "onError": "continueErrorOutput" to the "${sourceName}" node.`
+                });
+            }
+        }
+    }
     validateAIToolConnection(sourceName, targetNode, result) {
         let targetNodeInfo = this.nodeRepository.getNode(targetNode.type);
         if (!targetNodeInfo) {
@@ -589,6 +657,31 @@ class WorkflowValidator {
                     message: `Expression warning: ${warning}`
                 });
             });
+            const formatContext = {
+                nodeType: node.type,
+                nodeName: node.name,
+                nodeId: node.id
+            };
+            const formatIssues = expression_format_validator_1.ExpressionFormatValidator.validateNodeParameters(node.parameters, formatContext);
+            formatIssues.forEach(issue => {
+                const formattedMessage = expression_format_validator_1.ExpressionFormatValidator.formatErrorMessage(issue, formatContext);
+                if (issue.severity === 'error') {
+                    result.errors.push({
+                        type: 'error',
+                        nodeId: node.id,
+                        nodeName: node.name,
+                        message: formattedMessage
+                    });
+                }
+                else {
+                    result.warnings.push({
+                        type: 'warning',
+                        nodeId: node.id,
+                        nodeName: node.name,
+                        message: formattedMessage
+                    });
+                }
+            });
         }
     }
     countExpressionsInObject(obj) {
@@ -624,7 +717,7 @@ class WorkflowValidator {
         return false;
     }
     checkWorkflowPatterns(workflow, result, profile = 'runtime') {
-        const hasErrorHandling = Object.values(workflow.connections).some(outputs => outputs.error && outputs.error.length > 0);
+        const hasErrorHandling = Object.values(workflow.connections).some(outputs => outputs.main && outputs.main.length > 1 && outputs.main[1] && outputs.main[1].length > 0);
         if (!hasErrorHandling && workflow.nodes.length > 3 && profile !== 'minimal') {
             result.warnings.push({
                 type: 'warning',