@flowdrop/flowdrop 1.0.0 → 1.1.0

package/dist/services/globalSave.d.ts

@@ -7,8 +7,8 @@
  * reimplementing the logic, ensuring "blur active element" flushing and metadata
  * construction happen in exactly one place.
  */
-import type { Workflow } from '../types/index.js';
-import type { FlowDropEventHandlers, FlowDropFeatures } from '../types/events.js';
+import type { Workflow } from "../types/index.js";
+import type { FlowDropEventHandlers, FlowDropFeatures } from "../types/events.js";
 /**
  * Minimal interface for the enhanced API client used when an authProvider is present.
  * Matches the surface of EnhancedFlowDropApiClient that save needs.

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,7 +60,8 @@ 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
@@ -93,7 +94,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;
     }
@@ -104,33 +105,37 @@ 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();
         // Step 3 — Build the canonical workflow object.
         // Preserve all existing metadata fields (format, tags, etc.) so nothing is dropped.
+        //
+        // Determine new vs existing BEFORE the uuidv4() fallback: if the store already
+        // has an id (any format — integer, UUID, slug), the workflow came from a backend
+        // and must be updated. Only a missing id means "truly new".
+        const isExistingWorkflow = !!currentWorkflow.id;
         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;
         if (apiClient) {
-            // Enhanced client path — detects existing workflows by non-UUID ID
-            const isExistingWorkflow = finalWorkflow.id.length > 0 &&
-                !finalWorkflow.id.match(/^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i);
             if (isExistingWorkflow) {
                 savedWorkflow = await apiClient.updateWorkflow(finalWorkflow.id, finalWorkflow);
             }
@@ -140,7 +145,13 @@ export async function globalSaveWorkflow(options = {}) {
         }
         else {
             // Legacy path
-            savedWorkflow = await workflowApi.saveWorkflow(finalWorkflow);
+            if (isExistingWorkflow) {
+                savedWorkflow = await workflowApi.updateWorkflow(finalWorkflow.id, finalWorkflow);
+            }
+            else {
+                const { id: _id, ...workflowData } = finalWorkflow;
+                savedWorkflow = await workflowApi.createWorkflow(workflowData);
+            }
         }
         // Step 5 — If the server assigned a new ID, sync the store
         if (savedWorkflow.id && savedWorkflow.id !== finalWorkflow.id) {
@@ -150,8 +161,8 @@ export async function globalSaveWorkflow(options = {}) {
                 name: finalWorkflow.name,
                 metadata: {
                     ...finalWorkflow.metadata,
-                    ...savedWorkflow.metadata
-                }
+                    ...savedWorkflow.metadata,
+                },
             });
         }
         // Step 6a — Mark dirty state as clean
@@ -176,7 +187,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) {
@@ -185,10 +196,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;
     }
@@ -209,30 +220,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();
@@ -242,7 +253,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
  */

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
@@ -205,7 +205,7 @@ export class HistoryService {
             this.undoStack.push({
                 snapshot: this.cloneWorkflow(currentWorkflow),
                 timestamp: Date.now(),
-                description: 'Initial state'
+                description: "Initial state",
             });
         }
         this.notifyChange();
@@ -221,7 +221,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 +272,7 @@ export class HistoryService {
         this.undoStack.push({
             snapshot: this.cloneWorkflow(workflow),
             timestamp: Date.now(),
-            description
+            description,
         });
         // Trim history if over limit
         this.trimHistory();

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,7 +29,8 @@ 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
@@ -49,7 +50,8 @@ 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
@@ -77,10 +79,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;
     }
@@ -93,13 +95,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(() => ({}));
@@ -122,11 +124,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;
     }
@@ -140,15 +142,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;
     }
@@ -161,13 +163,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;
     }
@@ -180,7 +182,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 ?? [];
@@ -194,7 +196,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 ?? [];
@@ -213,29 +215,32 @@ 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
@@ -255,7 +260,8 @@ 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
  */