@flowdrop/flowdrop 1.4.0 → 1.6.0

package/dist/services/globalSave.js

@@ -7,14 +7,14 @@
  * reimplementing the logic, ensuring "blur active element" flushing and metadata
  * construction happen in exactly one place.
  */
-import { tick } from "svelte";
-import { getWorkflowStore, workflowActions, markAsSaved as storeMarkAsSaved, } from "../stores/workflowStore.svelte.js";
-import { workflowApi, setEndpointConfig } from "./api.js";
-import { createEndpointConfig } from "../config/endpoints.js";
-import { v4 as uuidv4 } from "uuid";
-import { DEFAULT_WORKFLOW_FORMAT } from "../types/index.js";
-import { apiToasts, workflowToasts, dismissToast } from "./toastService.js";
-import { DEFAULT_FEATURES } from "../types/events.js";
+import { tick } from 'svelte';
+import { getWorkflowStore, workflowActions, markAsSaved as storeMarkAsSaved } from '../stores/workflowStore.svelte.js';
+import { workflowApi, setEndpointConfig } from './api.js';
+import { createEndpointConfig } from '../config/endpoints.js';
+import { v4 as uuidv4 } from 'uuid';
+import { DEFAULT_WORKFLOW_FORMAT } from '../types/index.js';
+import { apiToasts, workflowToasts, dismissToast } from './toastService.js';
+import { DEFAULT_FEATURES } from '../types/events.js';
 // ---------------------------------------------------------------------------
 // Internal helpers
 // ---------------------------------------------------------------------------
@@ -25,7 +25,7 @@ import { DEFAULT_FEATURES } from "../types/events.js";
  */
 async function ensureApiConfiguration() {
     try {
-        const { getEndpointConfig } = await import("./api.js");
+        const { getEndpointConfig } = await import('./api.js');
         const currentConfig = getEndpointConfig();
         if (currentConfig && currentConfig.baseUrl) {
             return;
@@ -35,20 +35,20 @@ async function ensureApiConfiguration() {
         // Could not check existing API configuration, initializing
     }
     // API configuration is not initialized — derive URL from window.location when available
-    const apiBaseUrl = typeof window !== "undefined"
+    const apiBaseUrl = typeof window !== 'undefined'
         ? `${window.location.protocol}//${window.location.host}/api/flowdrop`
-        : "/api/flowdrop";
+        : '/api/flowdrop';
     const config = createEndpointConfig(apiBaseUrl, {
         auth: {
-            type: "none",
+            type: 'none'
         },
         timeout: 10000,
         retry: {
             enabled: true,
             maxAttempts: 2,
             delay: 1000,
-            backoff: "exponential",
-        },
+            backoff: 'exponential'
+        }
     });
     setEndpointConfig(config);
 }
@@ -60,8 +60,7 @@ async function ensureApiConfiguration() {
  * Must be called once, in this file only, so the logic lives in exactly one place.
  */
 async function flushPendingFormChanges() {
-    if (typeof document !== "undefined" &&
-        document.activeElement instanceof HTMLElement) {
+    if (typeof document !== 'undefined' && document.activeElement instanceof HTMLElement) {
         document.activeElement.blur();
     }
     // Wait for any pending DOM / Svelte reactive updates before reading the store
@@ -94,7 +93,7 @@ export async function globalSaveWorkflow(options = {}) {
     const currentWorkflow = getWorkflowStore();
     if (!currentWorkflow) {
         if (features.showToasts) {
-            apiToasts.error("Save workflow", "No workflow to save");
+            apiToasts.error('Save workflow', 'No workflow to save');
         }
         return;
     }
@@ -105,9 +104,7 @@ export async function globalSaveWorkflow(options = {}) {
             return;
         }
     }
-    const loadingToast = features.showToasts
-        ? apiToasts.loading("Saving workflow")
-        : null;
+    const loadingToast = features.showToasts ? apiToasts.loading('Saving workflow') : null;
     try {
         // Ensure API configuration is initialised (needed when called outside App.svelte)
         await ensureApiConfiguration();
@@ -121,17 +118,17 @@ export async function globalSaveWorkflow(options = {}) {
         const workflowId = currentWorkflow.id || uuidv4();
         const finalWorkflow = {
             id: workflowId,
-            name: currentWorkflow.name || "Untitled Workflow",
-            description: currentWorkflow.description || "",
+            name: currentWorkflow.name || 'Untitled Workflow',
+            description: currentWorkflow.description || '',
             nodes: currentWorkflow.nodes || [],
             edges: currentWorkflow.edges || [],
             metadata: {
                 ...currentWorkflow.metadata,
-                version: currentWorkflow.metadata?.version || "1.0.0",
+                version: currentWorkflow.metadata?.version || '1.0.0',
                 format: currentWorkflow.metadata?.format || DEFAULT_WORKFLOW_FORMAT,
                 createdAt: currentWorkflow.metadata?.createdAt || new Date().toISOString(),
-                updatedAt: new Date().toISOString(),
-            },
+                updatedAt: new Date().toISOString()
+            }
         };
         // Step 4 — Persist
         let savedWorkflow;
@@ -161,8 +158,8 @@ export async function globalSaveWorkflow(options = {}) {
                 name: finalWorkflow.name,
                 metadata: {
                     ...finalWorkflow.metadata,
-                    ...savedWorkflow.metadata,
-                },
+                    ...savedWorkflow.metadata
+                }
             });
         }
         // Step 6a — Mark dirty state as clean
@@ -187,7 +184,7 @@ export async function globalSaveWorkflow(options = {}) {
     catch (error) {
         if (loadingToast)
             dismissToast(loadingToast);
-        const errorObj = error instanceof Error ? error : new Error("Unknown error occurred");
+        const errorObj = error instanceof Error ? error : new Error('Unknown error occurred');
         // onSaveError hook
         const currentWorkflowForError = getWorkflowStore();
         if (eventHandlers?.onSaveError && currentWorkflowForError) {
@@ -196,10 +193,10 @@ export async function globalSaveWorkflow(options = {}) {
         // onApiError hook — return true suppresses the default toast
         let suppressToast = false;
         if (eventHandlers?.onApiError) {
-            suppressToast = eventHandlers.onApiError(errorObj, "save") === true;
+            suppressToast = eventHandlers.onApiError(errorObj, 'save') === true;
         }
         if (features.showToasts && !suppressToast) {
-            apiToasts.error("Save workflow", errorObj.message);
+            apiToasts.error('Save workflow', errorObj.message);
         }
         throw error;
     }
@@ -220,30 +217,30 @@ export async function globalExportWorkflow(options = {}) {
         const currentWorkflow = getWorkflowStore();
         if (!currentWorkflow) {
             if (features.showToasts) {
-                apiToasts.error("Export workflow", "No workflow to export");
+                apiToasts.error('Export workflow', 'No workflow to export');
             }
             return;
         }
         // Build the canonical export object — preserve all metadata fields
         const finalWorkflow = {
-            id: currentWorkflow.id || "untitled-workflow",
-            name: currentWorkflow.name || "Untitled Workflow",
-            description: currentWorkflow.description || "",
+            id: currentWorkflow.id || 'untitled-workflow',
+            name: currentWorkflow.name || 'Untitled Workflow',
+            description: currentWorkflow.description || '',
             nodes: currentWorkflow.nodes || [],
             edges: currentWorkflow.edges || [],
             metadata: {
                 ...currentWorkflow.metadata,
-                version: currentWorkflow.metadata?.version || "1.0.0",
+                version: currentWorkflow.metadata?.version || '1.0.0',
                 format: currentWorkflow.metadata?.format || DEFAULT_WORKFLOW_FORMAT,
                 createdAt: currentWorkflow.metadata?.createdAt || new Date().toISOString(),
-                updatedAt: new Date().toISOString(),
-            },
+                updatedAt: new Date().toISOString()
+            }
         };
         // Trigger browser download
         const dataStr = JSON.stringify(finalWorkflow, null, 2);
-        const dataBlob = new Blob([dataStr], { type: "application/json" });
+        const dataBlob = new Blob([dataStr], { type: 'application/json' });
         const url = URL.createObjectURL(dataBlob);
-        const link = document.createElement("a");
+        const link = document.createElement('a');
         link.href = url;
         link.download = `${finalWorkflow.name}.json`;
         link.click();
@@ -253,7 +250,7 @@ export async function globalExportWorkflow(options = {}) {
         }
     }
     catch (error) {
-        const errorObj = error instanceof Error ? error : new Error("Unknown error occurred");
-        apiToasts.error("Export workflow", errorObj.message);
+        const errorObj = error instanceof Error ? error : new Error('Unknown error occurred');
+        apiToasts.error('Export workflow', errorObj.message);
     }
 }

package/dist/services/historyService.d.ts

@@ -6,7 +6,7 @@
  *
  * @module services/historyService
  */
-import type { Workflow } from "../types/index.js";
+import type { Workflow } from '../types/index.js';
 /**
  * Represents a single entry in the history stack
  */
@@ -140,8 +140,10 @@ export declare class HistoryService {
      * Cancel the current transaction without committing
      *
      * Discards the transaction without adding to history.
+     * Returns the snapshot captured at transaction start so the caller
+     * can restore the store to its pre-transaction state.
      */
-    cancelTransaction(): void;
+    cancelTransaction(): Workflow | null;
     /**
      * Clear all history
      *
@@ -189,9 +191,9 @@ export declare class HistoryService {
     /**
      * Create a deep clone of a workflow
      *
-     * Uses JSON parse/stringify to properly strip non-serializable values
-     * like functions (e.g., onConfigOpen callbacks) that are added to nodes
-     * at runtime but shouldn't be persisted in history.
+     * Strips non-serializable values (e.g., onConfigOpen callbacks) before cloning
+     * so structuredClone doesn't throw on function references.
+     * Falls back to JSON round-trip if structuredClone still fails (e.g., Svelte proxies).
      */
     private cloneWorkflow;
     /**

package/dist/services/historyService.js

@@ -6,8 +6,8 @@
  *
  * @module services/historyService
  */
-import { DEFAULT_BEHAVIOR_SETTINGS } from "../types/settings.js";
-import { logger } from "../utils/logger.js";
+import { DEFAULT_BEHAVIOR_SETTINGS } from '../types/settings.js';
+import { logger } from '../utils/logger.js';
 // =========================================================================
 // History Service Class
 // =========================================================================
@@ -77,7 +77,7 @@ export class HistoryService {
         this.undoStack.push({
             snapshot: this.cloneWorkflow(workflow),
             timestamp: Date.now(),
-            description: "Initial state",
+            description: 'Initial state'
         });
         this.notifyChange();
     }
@@ -154,7 +154,7 @@ export class HistoryService {
      */
     startTransaction(workflow, description) {
         if (this.inTransaction) {
-            logger.warn("HistoryService: Transaction already in progress, ignoring startTransaction");
+            logger.warn('HistoryService: Transaction already in progress, ignoring startTransaction');
             return;
         }
         this.inTransaction = true;
@@ -168,7 +168,7 @@ export class HistoryService {
      */
     commitTransaction() {
         if (!this.inTransaction || !this.transactionSnapshot) {
-            logger.warn("HistoryService: No transaction in progress, ignoring commitTransaction");
+            logger.warn('HistoryService: No transaction in progress, ignoring commitTransaction');
             return;
         }
         // Push the snapshot captured at transaction start
@@ -182,11 +182,15 @@ export class HistoryService {
      * Cancel the current transaction without committing
      *
      * Discards the transaction without adding to history.
+     * Returns the snapshot captured at transaction start so the caller
+     * can restore the store to its pre-transaction state.
      */
     cancelTransaction() {
+        const snapshot = this.transactionSnapshot;
         this.inTransaction = false;
         this.transactionSnapshot = null;
         this.transactionDescription = null;
+        return snapshot;
     }
     /**
      * Clear all history
@@ -205,7 +209,7 @@ export class HistoryService {
             this.undoStack.push({
                 snapshot: this.cloneWorkflow(currentWorkflow),
                 timestamp: Date.now(),
-                description: "Initial state",
+                description: 'Initial state'
             });
         }
         this.notifyChange();
@@ -221,7 +225,7 @@ export class HistoryService {
             canRedo: this.redoStack.length > 0,
             currentIndex: this.undoStack.length - 1,
             historyLength: this.undoStack.length + this.redoStack.length,
-            isInTransaction: this.inTransaction,
+            isInTransaction: this.inTransaction
         };
     }
     /**
@@ -272,7 +276,7 @@ export class HistoryService {
         this.undoStack.push({
             snapshot: this.cloneWorkflow(workflow),
             timestamp: Date.now(),
-            description,
+            description
         });
         // Trim history if over limit
         this.trimHistory();
@@ -292,14 +296,25 @@ export class HistoryService {
     /**
      * Create a deep clone of a workflow
      *
-     * Uses JSON parse/stringify to properly strip non-serializable values
-     * like functions (e.g., onConfigOpen callbacks) that are added to nodes
-     * at runtime but shouldn't be persisted in history.
+     * Strips non-serializable values (e.g., onConfigOpen callbacks) before cloning
+     * so structuredClone doesn't throw on function references.
+     * Falls back to JSON round-trip if structuredClone still fails (e.g., Svelte proxies).
      */
     cloneWorkflow(workflow) {
-        // Always use JSON parse/stringify to strip functions and other non-serializable values
-        // structuredClone would fail on workflows containing callback functions
-        return JSON.parse(JSON.stringify(workflow));
+        const cleaned = {
+            ...workflow,
+            nodes: workflow.nodes.map((n) => ({
+                ...n,
+                data: { ...n.data, onConfigOpen: undefined }
+            }))
+        };
+        try {
+            return structuredClone(cleaned);
+        }
+        catch {
+            // Fallback for environments where structuredClone can't handle proxied objects
+            return JSON.parse(JSON.stringify(cleaned));
+        }
     }
     /**
      * Notify all subscribers of state changes

package/dist/services/interruptService.d.ts

@@ -7,7 +7,7 @@
  *
  * @module services/interruptService
  */
-import type { Interrupt, InterruptPollingConfig } from "../types/interrupt.js";
+import type { Interrupt, InterruptPollingConfig } from '../types/interrupt.js';
 /**
  * Interrupt Service class
  *

package/dist/services/interruptService.js

@@ -7,10 +7,10 @@
  *
  * @module services/interruptService
  */
-import { defaultInterruptPollingConfig } from "../types/interrupt.js";
-import { buildEndpointUrl, getEndpointHeaders } from "../config/endpoints.js";
-import { getEndpointConfig } from "./api.js";
-import { logger } from "../utils/logger.js";
+import { defaultInterruptPollingConfig } from '../types/interrupt.js';
+import { buildEndpointUrl, getEndpointHeaders } from '../config/endpoints.js';
+import { getEndpointConfig } from './api.js';
+import { logger } from '../utils/logger.js';
 /**
  * Interrupt Service class
  *
@@ -29,8 +29,7 @@ export class InterruptService {
      */
     constructor() {
         this.pollingConfig = { ...defaultInterruptPollingConfig };
-        this.currentBackoff =
-            this.pollingConfig.interval ?? defaultInterruptPollingConfig.interval;
+        this.currentBackoff = this.pollingConfig.interval ?? defaultInterruptPollingConfig.interval;
     }
     /**
      * Get the singleton instance of InterruptService
@@ -50,8 +49,7 @@ export class InterruptService {
      */
     setPollingConfig(config) {
         this.pollingConfig = { ...this.pollingConfig, ...config };
-        this.currentBackoff =
-            this.pollingConfig.interval ?? defaultInterruptPollingConfig.interval;
+        this.currentBackoff = this.pollingConfig.interval ?? defaultInterruptPollingConfig.interval;
     }
     /**
      * Get the current polling configuration
@@ -79,10 +77,10 @@ export class InterruptService {
     getConfig() {
         const config = getEndpointConfig();
         if (!config) {
-            throw new Error("Endpoint configuration not set. Call setEndpointConfig() first.");
+            throw new Error('Endpoint configuration not set. Call setEndpointConfig() first.');
         }
         if (!config.endpoints.interrupts) {
-            throw new Error("Interrupt endpoints not configured.");
+            throw new Error('Interrupt endpoints not configured.');
         }
         return config;
     }
@@ -95,13 +93,13 @@ export class InterruptService {
      */
     async request(url, options = {}) {
         const config = this.getConfig();
-        const headers = getEndpointHeaders(config, "interrupts");
+        const headers = getEndpointHeaders(config, 'interrupts');
         const response = await fetch(url, {
             ...options,
             headers: {
                 ...headers,
-                ...options.headers,
-            },
+                ...options.headers
+            }
         });
         if (!response.ok) {
             const errorData = await response.json().catch(() => ({}));
@@ -124,11 +122,11 @@ export class InterruptService {
     async getInterrupt(interruptId) {
         const config = this.getConfig();
         const url = buildEndpointUrl(config, config.endpoints.interrupts.get, {
-            interruptId,
+            interruptId
         });
         const response = await this.request(url);
         if (!response.data) {
-            throw new Error("Interrupt not found");
+            throw new Error('Interrupt not found');
         }
         return response.data;
     }
@@ -142,15 +140,15 @@ export class InterruptService {
     async resolveInterrupt(interruptId, value) {
         const config = this.getConfig();
         const url = buildEndpointUrl(config, config.endpoints.interrupts.resolve, {
-            interruptId,
+            interruptId
         });
         const resolution = { value };
         const response = await this.request(url, {
-            method: "POST",
-            body: JSON.stringify(resolution),
+            method: 'POST',
+            body: JSON.stringify(resolution)
         });
         if (!response.data) {
-            throw new Error("Failed to resolve interrupt: No data returned");
+            throw new Error('Failed to resolve interrupt: No data returned');
         }
         return response.data;
     }
@@ -163,13 +161,13 @@ export class InterruptService {
     async cancelInterrupt(interruptId) {
         const config = this.getConfig();
         const url = buildEndpointUrl(config, config.endpoints.interrupts.cancel, {
-            interruptId,
+            interruptId
         });
         const response = await this.request(url, {
-            method: "POST",
+            method: 'POST'
         });
         if (!response.data) {
-            throw new Error("Failed to cancel interrupt: No data returned");
+            throw new Error('Failed to cancel interrupt: No data returned');
         }
         return response.data;
     }
@@ -182,7 +180,7 @@ export class InterruptService {
     async listSessionInterrupts(sessionId) {
         const config = this.getConfig();
         const url = buildEndpointUrl(config, config.endpoints.interrupts.listBySession, {
-            sessionId,
+            sessionId
         });
         const response = await this.request(url);
         return response.data ?? [];
@@ -196,7 +194,7 @@ export class InterruptService {
     async listPipelineInterrupts(pipelineId) {
         const config = this.getConfig();
         const url = buildEndpointUrl(config, config.endpoints.interrupts.listByPipeline, {
-            pipelineId,
+            pipelineId
         });
         const response = await this.request(url);
         return response.data ?? [];
@@ -215,32 +213,29 @@ export class InterruptService {
      */
     startPolling(sessionId, callback) {
         if (!this.pollingConfig.enabled) {
-            logger.warn("[InterruptService] Polling is disabled. Enable via setPollingConfig().");
+            logger.warn('[InterruptService] Polling is disabled. Enable via setPollingConfig().');
             return;
         }
         // Stop any existing polling
         this.stopPolling();
         this.pollingSessionId = sessionId;
-        this.currentBackoff =
-            this.pollingConfig.interval ?? defaultInterruptPollingConfig.interval;
+        this.currentBackoff = this.pollingConfig.interval ?? defaultInterruptPollingConfig.interval;
         const poll = async () => {
             if (this.pollingSessionId !== sessionId) {
                 return;
             }
             try {
                 const interrupts = await this.listSessionInterrupts(sessionId);
-                const pendingInterrupts = interrupts.filter((i) => i.status === "pending");
+                const pendingInterrupts = interrupts.filter((i) => i.status === 'pending');
                 // Reset backoff on successful request
-                this.currentBackoff =
-                    this.pollingConfig.interval ?? defaultInterruptPollingConfig.interval;
+                this.currentBackoff = this.pollingConfig.interval ?? defaultInterruptPollingConfig.interval;
                 // Call the callback with pending interrupts
                 callback(pendingInterrupts);
             }
             catch (error) {
-                logger.error("[InterruptService] Polling error:", error);
+                logger.error('[InterruptService] Polling error:', error);
                 // Exponential backoff on error
-                const maxBackoff = this.pollingConfig.maxBackoff ??
-                    defaultInterruptPollingConfig.maxBackoff;
+                const maxBackoff = this.pollingConfig.maxBackoff ?? defaultInterruptPollingConfig.maxBackoff;
                 this.currentBackoff = Math.min(this.currentBackoff * 2, maxBackoff);
             }
             // Schedule next poll
@@ -260,8 +255,7 @@ export class InterruptService {
             this.pollingInterval = null;
         }
         this.pollingSessionId = null;
-        this.currentBackoff =
-            this.pollingConfig.interval ?? defaultInterruptPollingConfig.interval;
+        this.currentBackoff = this.pollingConfig.interval ?? defaultInterruptPollingConfig.interval;
     }
     /**
      * Check if polling is active

package/dist/services/nodeExecutionService.d.ts

@@ -2,7 +2,7 @@
  * Node Execution Service
  * Handles fetching and managing node execution information from the backend
  */
-import type { NodeExecutionInfo } from "../types/index.js";
+import type { NodeExecutionInfo } from '../types/index.js';
 /**
  * Service for managing node execution information
  */