@flowdrop/flowdrop 1.0.0 → 1.1.0

package/dist/services/agentSpecExecutionService.js

@@ -6,9 +6,9 @@
  *
  * Follows the same singleton pattern as NodeExecutionService.
  */
-import { buildAgentSpecUrl, getAgentSpecAuthHeaders } from '../config/agentSpecEndpoints.js';
-import { AgentSpecAdapter } from '../adapters/agentspec/AgentSpecAdapter.js';
-import { logger } from '../utils/logger.js';
+import { buildAgentSpecUrl, getAgentSpecAuthHeaders, } from "../config/agentSpecEndpoints.js";
+import { AgentSpecAdapter } from "../adapters/agentspec/AgentSpecAdapter.js";
+import { logger } from "../utils/logger.js";
 /**
  * Service for executing FlowDrop workflows on Agent Spec runtimes.
  *
@@ -67,7 +67,7 @@ export class AgentSpecExecutionService {
             const url = buildAgentSpecUrl(this.config, this.config.endpoints.health);
             const response = await fetch(url, {
                 headers: getAgentSpecAuthHeaders(this.config),
-                signal: AbortSignal.timeout(5000)
+                signal: AbortSignal.timeout(5000),
             });
             return response.ok;
         }
@@ -90,29 +90,29 @@ export class AgentSpecExecutionService {
         // POST to runtime
         const url = buildAgentSpecUrl(this.getConfig(), this.getConfig().endpoints.execute);
         const response = await fetch(url, {
-            method: 'POST',
+            method: "POST",
             headers: getAgentSpecAuthHeaders(this.getConfig()),
             body: JSON.stringify({
                 flow: agentSpecFlow,
-                inputs: inputs || {}
+                inputs: inputs || {},
             }),
-            signal: AbortSignal.timeout(this.getConfig().timeout || 60000)
+            signal: AbortSignal.timeout(this.getConfig().timeout || 60000),
         });
         if (!response.ok) {
-            const errorText = await response.text().catch(() => 'Unknown error');
+            const errorText = await response.text().catch(() => "Unknown error");
             throw new Error(`Agent Spec runtime error (${response.status}): ${errorText}`);
         }
         const result = await response.json();
         const executionId = result.execution_id || result.id;
         if (!executionId) {
-            throw new Error('Runtime did not return an execution ID');
+            throw new Error("Runtime did not return an execution ID");
         }
         // Track execution
         const state = {
             id: executionId,
-            status: 'running',
+            status: "running",
             startedAt: new Date().toISOString(),
-            nodeStatuses: {}
+            nodeStatuses: {},
         };
         this.activeExecutions.set(executionId, state);
         // Build node name → FlowDrop node ID mapping for status updates
@@ -135,10 +135,10 @@ export class AgentSpecExecutionService {
         this.ensureConfigured();
         try {
             const url = buildAgentSpecUrl(this.getConfig(), this.getConfig().endpoints.status, {
-                id: executionId
+                id: executionId,
             });
             const response = await fetch(url, {
-                headers: getAgentSpecAuthHeaders(this.getConfig())
+                headers: getAgentSpecAuthHeaders(this.getConfig()),
             });
             if (!response.ok)
                 return null;
@@ -156,15 +156,15 @@ export class AgentSpecExecutionService {
         this.ensureConfigured();
         this.stopPolling(executionId);
         const url = buildAgentSpecUrl(this.getConfig(), this.getConfig().endpoints.cancel, {
-            id: executionId
+            id: executionId,
         });
         await fetch(url, {
-            method: 'POST',
-            headers: getAgentSpecAuthHeaders(this.getConfig())
+            method: "POST",
+            headers: getAgentSpecAuthHeaders(this.getConfig()),
         });
         const state = this.activeExecutions.get(executionId);
         if (state) {
-            state.status = 'cancelled';
+            state.status = "cancelled";
         }
     }
     /**
@@ -174,10 +174,10 @@ export class AgentSpecExecutionService {
         this.ensureConfigured();
         try {
             const url = buildAgentSpecUrl(this.getConfig(), this.getConfig().endpoints.results, {
-                id: executionId
+                id: executionId,
             });
             const response = await fetch(url, {
-                headers: getAgentSpecAuthHeaders(this.getConfig())
+                headers: getAgentSpecAuthHeaders(this.getConfig()),
             });
             if (!response.ok)
                 return null;
@@ -195,12 +195,15 @@ export class AgentSpecExecutionService {
         const agentSpecFlow = this.adapter.toAgentSpec(workflow);
         const url = buildAgentSpecUrl(this.getConfig(), this.getConfig().endpoints.validate);
         const response = await fetch(url, {
-            method: 'POST',
+            method: "POST",
             headers: getAgentSpecAuthHeaders(this.getConfig()),
-            body: JSON.stringify(agentSpecFlow)
+            body: JSON.stringify(agentSpecFlow),
         });
         if (!response.ok) {
-            return { valid: false, errors: [`Runtime validation failed: ${response.status}`] };
+            return {
+                valid: false,
+                errors: [`Runtime validation failed: ${response.status}`],
+            };
         }
         return await response.json();
     }
@@ -218,7 +221,7 @@ export class AgentSpecExecutionService {
     // ========================================================================
     ensureConfigured() {
         if (!this.config) {
-            throw new Error('AgentSpecExecutionService not configured. Call configure() with runtime endpoint config first.');
+            throw new Error("AgentSpecExecutionService not configured. Call configure() with runtime endpoint config first.");
         }
     }
     /** Get the config, throwing if not configured */
@@ -234,10 +237,10 @@ export class AgentSpecExecutionService {
         const poll = async () => {
             try {
                 const url = buildAgentSpecUrl(this.getConfig(), this.getConfig().endpoints.status, {
-                    id: executionId
+                    id: executionId,
                 });
                 const response = await fetch(url, {
-                    headers: getAgentSpecAuthHeaders(this.getConfig())
+                    headers: getAgentSpecAuthHeaders(this.getConfig()),
                 });
                 if (!response.ok) {
                     throw new Error(`Status check failed: ${response.status}`);
@@ -253,26 +256,27 @@ export class AgentSpecExecutionService {
                     onNodeUpdate?.(nodeId, info);
                 }
                 // Check if execution is done
-                if (executionStatus === 'completed' || executionStatus === 'success') {
-                    state.status = 'completed';
+                if (executionStatus === "completed" || executionStatus === "success") {
+                    state.status = "completed";
                     this.stopPolling(executionId);
                     const results = await this.getResults(executionId);
                     onComplete?.(results || {});
                 }
-                else if (executionStatus === 'failed' || executionStatus === 'error') {
-                    state.status = 'failed';
+                else if (executionStatus === "failed" ||
+                    executionStatus === "error") {
+                    state.status = "failed";
                     this.stopPolling(executionId);
-                    const errorMsg = data.error || data.message || 'Execution failed';
+                    const errorMsg = data.error || data.message || "Execution failed";
                     onError?.(new Error(errorMsg));
                 }
-                else if (executionStatus === 'cancelled') {
-                    state.status = 'cancelled';
+                else if (executionStatus === "cancelled") {
+                    state.status = "cancelled";
                     this.stopPolling(executionId);
                 }
             }
             catch (error) {
                 // Don't stop polling on transient errors — let it retry
-                logger.error('[AgentSpecExecution] Polling error:', error);
+                logger.error("[AgentSpecExecution] Polling error:", error);
             }
         };
         // Initial poll immediately, then at interval
@@ -295,38 +299,38 @@ export class AgentSpecExecutionService {
         return result;
     }
     mapSingleNodeStatus(status) {
-        const runtimeStatus = (status.status || status.state || 'idle');
+        const runtimeStatus = (status.status || status.state || "idle");
         return {
             status: this.mapToFlowDropStatus(runtimeStatus),
             executionCount: status.execution_count || 0,
-            isExecuting: runtimeStatus === 'running' || runtimeStatus === 'executing',
+            isExecuting: runtimeStatus === "running" || runtimeStatus === "executing",
             lastExecuted: status.started_at,
             lastExecutionDuration: status.duration_ms,
-            lastError: status.error
+            lastError: status.error,
         };
     }
     mapToFlowDropStatus(runtimeStatus) {
         switch (runtimeStatus) {
-            case 'running':
-            case 'executing':
-                return 'running';
-            case 'completed':
-            case 'success':
-            case 'done':
-                return 'completed';
-            case 'failed':
-            case 'error':
-                return 'failed';
-            case 'cancelled':
-            case 'canceled':
-                return 'cancelled';
-            case 'pending':
-            case 'queued':
-                return 'pending';
-            case 'skipped':
-                return 'skipped';
+            case "running":
+            case "executing":
+                return "running";
+            case "completed":
+            case "success":
+            case "done":
+                return "completed";
+            case "failed":
+            case "error":
+                return "failed";
+            case "cancelled":
+            case "canceled":
+                return "cancelled";
+            case "pending":
+            case "queued":
+                return "pending";
+            case "skipped":
+                return "skipped";
             default:
-                return 'idle';
+                return "idle";
         }
     }
 }

package/dist/services/api.d.ts

@@ -2,8 +2,8 @@
  * Client-side API service for FlowDrop
  * Provides methods to interact with the backend APIs using configurable endpoints
  */
-import type { NodeMetadata, Workflow } from '../types/index.js';
-import type { EndpointConfig } from '../config/endpoints.js';
+import type { NodeMetadata, Workflow } from "../types/index.js";
+import type { EndpointConfig } from "../config/endpoints.js";
 /**
  * Set the endpoint configuration at runtime
  */
@@ -59,7 +59,14 @@ export declare const workflowApi: {
      */
     deleteWorkflow(id: string): Promise<void>;
     /**
-     * Save workflow (create or update)
+     * Save workflow (create or update).
+     *
+     * A workflow is considered existing when it already has an id (any format —
+     * integer, UUID, slug). Only a missing/empty id means "truly new".
+     *
+     * Note: globalSave.ts bypasses this method and calls createWorkflow /
+     * updateWorkflow directly so it can capture the new/existing decision before
+     * the uuidv4() fallback. This method is kept for external callers.
      */
     saveWorkflow(workflow: Workflow): Promise<Workflow>;
 };
@@ -108,7 +115,14 @@ export declare const api: {
          */
         deleteWorkflow(id: string): Promise<void>;
         /**
-         * Save workflow (create or update)
+         * Save workflow (create or update).
+         *
+         * A workflow is considered existing when it already has an id (any format —
+         * integer, UUID, slug). Only a missing/empty id means "truly new".
+         *
+         * Note: globalSave.ts bypasses this method and calls createWorkflow /
+         * updateWorkflow directly so it can capture the new/existing decision before
+         * the uuidv4() fallback. This method is kept for external callers.
          */
         saveWorkflow(workflow: Workflow): Promise<Workflow>;
     };

package/dist/services/api.js

@@ -2,7 +2,7 @@
  * Client-side API service for FlowDrop
  * Provides methods to interact with the backend APIs using configurable endpoints
  */
-import { buildEndpointUrl, getEndpointMethod, getEndpointHeaders } from '../config/endpoints.js';
+import { buildEndpointUrl, getEndpointMethod, getEndpointHeaders, } from "../config/endpoints.js";
 let endpointConfig = null;
 /**
  * Set the endpoint configuration at runtime
@@ -21,7 +21,7 @@ export function getEndpointConfig() {
  */
 async function apiRequest(endpointKey, endpointPath, params, options = {}) {
     if (!endpointConfig) {
-        throw new Error('Endpoint configuration not set. Call setEndpointConfig() first.');
+        throw new Error("Endpoint configuration not set. Call setEndpointConfig() first.");
     }
     const url = buildEndpointUrl(endpointConfig, endpointPath, params);
     const method = getEndpointMethod(endpointConfig, endpointKey);
@@ -29,11 +29,11 @@ async function apiRequest(endpointKey, endpointPath, params, options = {}) {
     const response = await fetch(url, {
         method,
         headers,
-        ...options
+        ...options,
     });
     // Check if response is JSON
-    const contentType = response.headers.get('content-type');
-    const isJson = contentType?.includes('application/json');
+    const contentType = response.headers.get("content-type");
+    const isJson = contentType?.includes("application/json");
     if (!response.ok) {
         // Try to get error details
         let errorMessage = `HTTP ${response.status}: ${response.statusText}`;
@@ -73,18 +73,18 @@ export const nodeApi = {
      */
     async getNodes(options) {
         if (!endpointConfig) {
-            throw new Error('Endpoint configuration not set');
+            throw new Error("Endpoint configuration not set");
         }
         const params = new URLSearchParams();
         if (options?.category)
-            params.append('category', options.category);
+            params.append("category", options.category);
         if (options?.search)
-            params.append('search', options.search);
+            params.append("search", options.search);
         if (options?.limit)
-            params.append('limit', options.limit.toString());
+            params.append("limit", options.limit.toString());
         if (options?.offset)
-            params.append('offset', options.offset.toString());
-        const response = await apiRequest('nodes.list', endpointConfig.endpoints.nodes.list + '?' + params.toString());
+            params.append("offset", options.offset.toString());
+        const response = await apiRequest("nodes.list", endpointConfig.endpoints.nodes.list + "?" + params.toString());
         return response.data || [];
     },
     /**
@@ -92,14 +92,14 @@ export const nodeApi = {
      */
     async getNode(id) {
         if (!endpointConfig) {
-            throw new Error('Endpoint configuration not set');
+            throw new Error("Endpoint configuration not set");
         }
-        const response = await apiRequest('nodes.get', endpointConfig.endpoints.nodes.get, { id });
+        const response = await apiRequest("nodes.get", endpointConfig.endpoints.nodes.get, { id });
         if (!response.data) {
-            throw new Error('Node not found');
+            throw new Error("Node not found");
         }
         return response.data;
-    }
+    },
 };
 /**
  * Workflow API methods
@@ -110,16 +110,16 @@ export const workflowApi = {
      */
     async getWorkflows(options) {
         if (!endpointConfig) {
-            throw new Error('Endpoint configuration not set');
+            throw new Error("Endpoint configuration not set");
         }
         const params = new URLSearchParams();
         if (options?.search)
-            params.append('search', options.search);
+            params.append("search", options.search);
         if (options?.limit)
-            params.append('limit', options.limit.toString());
+            params.append("limit", options.limit.toString());
         if (options?.offset)
-            params.append('offset', options.offset.toString());
-        const response = await apiRequest('workflows.list', endpointConfig.endpoints.workflows.list + '?' + params.toString());
+            params.append("offset", options.offset.toString());
+        const response = await apiRequest("workflows.list", endpointConfig.endpoints.workflows.list + "?" + params.toString());
         return response.data || [];
     },
     /**
@@ -127,11 +127,11 @@ export const workflowApi = {
      */
     async getWorkflow(id) {
         if (!endpointConfig) {
-            throw new Error('Endpoint configuration not set');
+            throw new Error("Endpoint configuration not set");
         }
-        const response = await apiRequest('workflows.get', endpointConfig.endpoints.workflows.get, { id });
+        const response = await apiRequest("workflows.get", endpointConfig.endpoints.workflows.get, { id });
         if (!response.data) {
-            throw new Error('Workflow not found');
+            throw new Error("Workflow not found");
         }
         return response.data;
     },
@@ -140,18 +140,18 @@ export const workflowApi = {
      */
     async createWorkflow(workflow) {
         if (!endpointConfig) {
-            throw new Error('Endpoint configuration not set');
+            throw new Error("Endpoint configuration not set");
         }
         // Apply the consumer-provided payload transform (e.g. Drupal's label mapping).
         // The default is the identity function — no CMS-specific logic in the core library.
         const transform = endpointConfig.transformWorkflowPayload ?? ((w) => w);
         const body = transform(workflow);
-        const response = await apiRequest('workflows.create', endpointConfig.endpoints.workflows.create, undefined, {
-            method: 'POST',
-            body: JSON.stringify(body)
+        const response = await apiRequest("workflows.create", endpointConfig.endpoints.workflows.create, undefined, {
+            method: "POST",
+            body: JSON.stringify(body),
         });
         if (!response.data) {
-            throw new Error('Failed to create workflow');
+            throw new Error("Failed to create workflow");
         }
         return response.data;
     },
@@ -160,18 +160,18 @@ export const workflowApi = {
      */
     async updateWorkflow(id, workflow) {
         if (!endpointConfig) {
-            throw new Error('Endpoint configuration not set');
+            throw new Error("Endpoint configuration not set");
         }
         // Apply the consumer-provided payload transform (e.g. Drupal's label mapping).
         // The default is the identity function — no CMS-specific logic in the core library.
         const transform = endpointConfig.transformWorkflowPayload ?? ((w) => w);
         const body = transform(workflow);
-        const response = await apiRequest('workflows.update', endpointConfig.endpoints.workflows.update, { id }, {
-            method: 'PUT',
-            body: JSON.stringify(body)
+        const response = await apiRequest("workflows.update", endpointConfig.endpoints.workflows.update, { id }, {
+            method: "PUT",
+            body: JSON.stringify(body),
         });
         if (!response.data) {
-            throw new Error('Failed to update workflow');
+            throw new Error("Failed to update workflow");
         }
         return response.data;
     },
@@ -180,19 +180,22 @@ export const workflowApi = {
      */
     async deleteWorkflow(id) {
         if (!endpointConfig) {
-            throw new Error('Endpoint configuration not set');
+            throw new Error("Endpoint configuration not set");
         }
-        await apiRequest('workflows.delete', endpointConfig.endpoints.workflows.delete, { id }, { method: 'DELETE' });
+        await apiRequest("workflows.delete", endpointConfig.endpoints.workflows.delete, { id }, { method: "DELETE" });
     },
     /**
-     * Save workflow (create or update)
+     * Save workflow (create or update).
+     *
+     * A workflow is considered existing when it already has an id (any format —
+     * integer, UUID, slug). Only a missing/empty id means "truly new".
+     *
+     * Note: globalSave.ts bypasses this method and calls createWorkflow /
+     * updateWorkflow directly so it can capture the new/existing decision before
+     * the uuidv4() fallback. This method is kept for external callers.
      */
     async saveWorkflow(workflow) {
-        // Check if this is an existing workflow by looking for a valid ID
-        // Valid IDs should not be a UUID (which indicates a new workflow)
-        const isExistingWorkflow = workflow.id &&
-            workflow.id.length > 0 &&
-            !workflow.id.match(/^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i);
+        const isExistingWorkflow = !!(workflow.id && workflow.id.length > 0);
         if (isExistingWorkflow) {
             // Update existing workflow
             return this.updateWorkflow(workflow.id, workflow);
@@ -203,12 +206,12 @@ export const workflowApi = {
             const { id, ...workflowData } = workflow;
             return this.createWorkflow(workflowData);
         }
-    }
+    },
 };
 /**
  * Export the API service
  */
 export const api = {
     nodes: nodeApi,
-    workflows: workflowApi
+    workflows: workflowApi,
 };

package/dist/services/apiVariableService.d.ts

@@ -5,7 +5,7 @@
  *
  * @module services/apiVariableService
  */
-import type { VariableSchema, ApiVariablesConfig, AuthProvider } from '../types/index.js';
+import type { VariableSchema, ApiVariablesConfig, AuthProvider } from "../types/index.js";
 /**
  * Context for variable API requests
  */