@lhi/n8m 0.2.4 → 0.3.1

package/dist/utils/n8nClient.js

@@ -128,13 +128,56 @@ export class N8nClient {
      */
     async updateWorkflow(workflowId, workflowData) {
         try {
-            const sanitized = this.sanitizeSettings(workflowData);
-            const response = await fetch(`${this.apiUrl}/workflows/${workflowId}`, {
-                body: JSON.stringify(sanitized),
-                headers: this.headers,
-                method: 'PUT',
-            });
-            await this.assertOk(response, 'update workflow');
+            const w = workflowData;
+            const settings = { ...(w.settings || {}) };
+            delete settings.timezone;
+            const SAFE_NODE_PROPS = new Set([
+                'id', 'name', 'type', 'typeVersion', 'position', 'parameters',
+                'credentials', 'disabled', 'webhookId', 'notes', 'notesInFlow',
+                'continueOnFail', 'alwaysOutputData', 'executeOnce', 'retryOnFail',
+                'maxTries', 'waitBetweenTries',
+            ]);
+            const buildPayload = (sanitizeNodes, includeVersionId) => {
+                const nodes = sanitizeNodes
+                    ? (w.nodes || []).map((node) => {
+                        const clean = {};
+                        for (const key of Object.keys(node)) {
+                            if (SAFE_NODE_PROPS.has(key))
+                                clean[key] = node[key];
+                        }
+                        return clean;
+                    })
+                    : w.nodes;
+                const payload = {
+                    name: w.name,
+                    nodes,
+                    connections: w.connections,
+                    settings,
+                };
+                if (includeVersionId && w.versionId)
+                    payload.versionId = w.versionId;
+                return payload;
+            };
+            const attempts = [
+                buildPayload(false, true), // full nodes + versionId
+                buildPayload(true, true), // sanitized nodes + versionId
+                buildPayload(true, false), // sanitized nodes, no versionId
+            ];
+            let response;
+            let errorText = '';
+            for (const payload of attempts) {
+                response = await fetch(`${this.apiUrl}/workflows/${workflowId}`, {
+                    body: JSON.stringify(payload),
+                    headers: this.headers,
+                    method: 'PUT',
+                });
+                if (response.ok)
+                    return;
+                errorText = await response.text();
+                if (response.status !== 400 || !errorText.includes('additional properties'))
+                    break;
+            }
+            throw new Error(`update workflow failed: ${response.status} - ${errorText}`);
         }
         catch (error) {
             throw new Error(`Failed to update workflow: ${error.message}`);
@@ -174,10 +217,15 @@ export class N8nClient {
      */
     async createWorkflow(name, workflowData) {
         try {
-            const payload = this.sanitizeSettings({
+            const w = workflowData;
+            const settings = { ...(w.settings || {}) };
+            delete settings.timezone;
+            const payload = {
                 name,
-                ...workflowData,
-            });
+                nodes: w.nodes ?? [],
+                connections: w.connections ?? {},
+                settings,
+            };
             const response = await fetch(`${this.apiUrl}/workflows`, {
                 body: JSON.stringify(payload),
                 headers: this.headers,
@@ -257,7 +305,7 @@ export class N8nClient {
                 allWorkflows = [...allWorkflows, ...result.data];
                 cursor = result.nextCursor;
             } while (cursor);
-            return allWorkflows;
+            return allWorkflows.filter((w) => !w.isArchived);
         }
         catch (error) {
             throw new Error(`Failed to fetch workflows: ${error.message}`);
@@ -503,6 +551,7 @@ export class N8nClient {
                     mode: 'runOnceForAllItems',
                     jsCode: N8nClient.buildNetworkShimCode(node.type),
                 },
+                ...(node.onError ? { onError: node.onError } : {}),
             };
         });
     }