@flowdrop/flowdrop 2.0.0-beta.3 → 2.0.0-beta.5

package/dist/svelte-app.js

@@ -55,6 +55,58 @@ function releaseInstance(fd, isDefault) {
         fd.destroy();
     }
 }
+/**
+ * Resolve endpoint config, port config and categories from mount options and
+ * apply them to the given instance (API context, port-compatibility checker,
+ * categories). Shared by `mountFlowDropApp` and `mountWorkflowEditor`.
+ *
+ * @returns the resolved {@link EndpointConfig} (merged with defaults), which the
+ *   callers forward to their mounted component.
+ */
+async function configureInstance(fd, options) {
+    const { endpointConfig, portConfig, categories, authProvider } = options;
+    // Create endpoint configuration, merging with defaults so all required
+    // endpoints are present.
+    const { defaultEndpointConfig } = await import('./config/endpoints.js');
+    const config = endpointConfig
+        ? {
+            ...defaultEndpointConfig,
+            ...endpointConfig,
+            endpoints: {
+                ...defaultEndpointConfig.endpoints,
+                ...endpointConfig.endpoints
+            }
+        }
+        : defaultEndpointConfig;
+    // Initialize port configuration (fetch from API when not supplied).
+    let finalPortConfig = portConfig;
+    if (!finalPortConfig) {
+        try {
+            finalPortConfig = await fetchPortConfig(config, authProvider);
+        }
+        catch (error) {
+            logger.warn('Failed to fetch port config from API, using default:', error);
+            finalPortConfig = DEFAULT_PORT_CONFIG;
+        }
+    }
+    // Configure this instance's API context and port compatibility checker.
+    fd.api.configure(config, authProvider);
+    fd.portCompatibility.reinitialize(finalPortConfig);
+    // Initialize this instance's categories (fetch from API when not supplied).
+    if (categories) {
+        fd.categories.initialize(categories);
+    }
+    else {
+        try {
+            const fetchedCategories = await fetchCategories(config, authProvider);
+            fd.categories.initialize(fetchedCategories);
+        }
+        catch (error) {
+            logger.warn('Failed to fetch categories from API, using defaults:', error);
+        }
+    }
+    return config;
+}
 /**
  * Mount the full FlowDrop App with navbar, sidebars, and workflow editor
  *
@@ -103,61 +155,14 @@ export async function mountFlowDropApp(container, options = {}) {
     await initializeSettings({
         defaults: initialSettings
     });
-    // Create endpoint configuration
-    let config;
-    if (endpointConfig) {
-        // Merge with default configuration to ensure all required endpoints are present
-        const { defaultEndpointConfig } = await import('./config/endpoints.js');
-        config = {
-            ...defaultEndpointConfig,
-            ...endpointConfig,
-            endpoints: {
-                ...defaultEndpointConfig.endpoints,
-                ...endpointConfig.endpoints
-            }
-        };
-    }
-    else {
-        // Use default configuration if none provided
-        const { defaultEndpointConfig } = await import('./config/endpoints.js');
-        config = defaultEndpointConfig;
-    }
-    // Initialize port configuration
-    let finalPortConfig = portConfig;
-    if (!finalPortConfig && config) {
-        // Try to fetch port configuration from API
-        try {
-            finalPortConfig = await fetchPortConfig(config, authProvider);
-        }
-        catch (error) {
-            logger.warn('Failed to fetch port config from API, using default:', error);
-            finalPortConfig = DEFAULT_PORT_CONFIG;
-        }
-    }
-    else if (!finalPortConfig) {
-        finalPortConfig = DEFAULT_PORT_CONFIG;
-    }
-    // Configure this instance's API context (endpoints + auth provider) so
-    // <App> and services resolve it via getInstance().api.
-    if (config) {
-        fd.api.configure(config, authProvider);
-    }
-    // Re-initialize this instance's port compatibility checker with the resolved
-    // config (it was seeded with DEFAULT_PORT_CONFIG at construction).
-    fd.portCompatibility.reinitialize(finalPortConfig);
-    // Initialize this instance's categories
-    if (categories) {
-        fd.categories.initialize(categories);
-    }
-    else if (config) {
-        try {
-            const fetchedCategories = await fetchCategories(config, authProvider);
-            fd.categories.initialize(fetchedCategories);
-        }
-        catch (error) {
-            logger.warn('Failed to fetch categories from API, using defaults:', error);
-        }
-    }
+    // Resolve and apply endpoint config, port config and categories to this
+    // instance (see configureInstance).
+    const config = await configureInstance(fd, {
+        endpointConfig,
+        portConfig,
+        categories,
+        authProvider
+    });
     // Set up event handler callbacks in this instance's store
     if (eventHandlers?.onDirtyStateChange) {
         fd.workflow.setOnDirtyStateChange(eventHandlers.onDirtyStateChange);
@@ -338,58 +343,14 @@ export async function mountWorkflowEditor(container, options = {}) {
     const { workflow, endpointConfig, portConfig, categories, authProvider, instanceId, builtinEditors } = options;
     // Per-instance state container (see mountFlowDropApp)
     const { fd, isDefault } = acquireInstance(instanceId);
-    // Create endpoint configuration
-    let config;
-    if (endpointConfig) {
-        // Merge with default configuration to ensure all required endpoints are present
-        const { defaultEndpointConfig } = await import('./config/endpoints.js');
-        config = {
-            ...defaultEndpointConfig,
-            ...endpointConfig,
-            endpoints: {
-                ...defaultEndpointConfig.endpoints,
-                ...endpointConfig.endpoints
-            }
-        };
-    }
-    else {
-        // Use default configuration if none provided
-        const { defaultEndpointConfig } = await import('./config/endpoints.js');
-        config = defaultEndpointConfig;
-    }
-    // Initialize port configuration
-    let finalPortConfig = portConfig;
-    if (!finalPortConfig && config) {
-        // Try to fetch port configuration from API
-        try {
-            finalPortConfig = await fetchPortConfig(config, authProvider);
-        }
-        catch (error) {
-            logger.warn('Failed to fetch port config from API, using default:', error);
-            finalPortConfig = DEFAULT_PORT_CONFIG;
-        }
-    }
-    else if (!finalPortConfig) {
-        finalPortConfig = DEFAULT_PORT_CONFIG;
-    }
-    // Configure this instance's API context and port compatibility checker.
-    if (config) {
-        fd.api.configure(config, authProvider);
-    }
-    fd.portCompatibility.reinitialize(finalPortConfig);
-    // Initialize this instance's categories
-    if (categories) {
-        fd.categories.initialize(categories);
-    }
-    else if (config) {
-        try {
-            const fetchedCategories = await fetchCategories(config, authProvider);
-            fd.categories.initialize(fetchedCategories);
-        }
-        catch (error) {
-            logger.warn('Failed to fetch categories from API, using defaults:', error);
-        }
-    }
+    // Resolve and apply endpoint config, port config and categories to this
+    // instance (see configureInstance).
+    const config = await configureInstance(fd, {
+        endpointConfig,
+        portConfig,
+        categories,
+        authProvider
+    });
     // Seed the instance's workflow before mounting so the editor renders it
     // immediately. (1.x accepted this option but silently ignored it.)
     if (workflow) {

package/dist/types/index.d.ts

@@ -327,7 +327,7 @@ export interface AutocompleteConfig {
  *   method: "GET",
  *   headers: { "X-Custom-Header": "value" },
  *   parameterMapping: {
- *     nodeTypeId: "metadata.id",
+ *     nodeTypeId: "metadata.node_type_id",
  *     instanceId: "id"
  *   }
  * };
@@ -352,7 +352,7 @@ export interface DynamicSchemaEndpoint {
     /**
      * Maps template variables to their source paths.
      * Keys are variable names used in the URL, values are dot-notation paths
-     * to resolve from the node context (e.g., "metadata.id", "config.apiKey", "id")
+     * to resolve from the node context (e.g., "metadata.node_type_id", "config.apiKey", "id")
      */
     parameterMapping?: Record<string, string>;
     /**
@@ -381,7 +381,7 @@ export interface DynamicSchemaEndpoint {
  *   url: "https://admin.example.com/nodes/{nodeTypeId}/edit/{instanceId}",
  *   label: "Configure in Admin Panel",
  *   parameterMapping: {
- *     nodeTypeId: "metadata.id",
+ *     nodeTypeId: "metadata.node_type_id",
  *     instanceId: "id"
  *   },
  *   openInNewTab: true
@@ -408,7 +408,7 @@ export interface ExternalEditLink {
     /**
      * Maps template variables to their source paths.
      * Keys are variable names used in the URL, values are dot-notation paths
-     * to resolve from the node context (e.g., "metadata.id", "config.apiKey", "id")
+     * to resolve from the node context (e.g., "metadata.node_type_id", "config.apiKey", "id")
      */
     parameterMapping?: Record<string, string>;
     /**
@@ -602,7 +602,8 @@ export interface NodeExtensions {
  * Node configuration metadata
  */
 export interface NodeMetadata {
-    id: string;
+    /** The node type entity id — the runtime's resolution anchor. */
+    node_type_id: string;
     name: string;
     type?: NodeType;
     /**
@@ -1132,8 +1133,6 @@ export interface WorkflowNode extends Node {
         isProcessing?: boolean;
         /** Error message if the node execution failed */
         error?: string;
-        /** Alternative node identifier */
-        nodeId?: string;
         /** Node execution tracking information */
         executionInfo?: NodeExecutionInfo;
         /**

package/dist/utils/connections.js

@@ -119,7 +119,7 @@ export class PortCompatibilityChecker {
  * Get all possible connections from a source node to target nodes
  */
 export function getPossibleConnections(checker, sourceNode, targetNodes, nodeTypes) {
-    const sourceMetadata = nodeTypes.find((nt) => nt.id === sourceNode.data.metadata.id);
+    const sourceMetadata = nodeTypes.find((nt) => nt.node_type_id === sourceNode.data.metadata.node_type_id);
     if (!sourceMetadata)
         return [];
     const possibleConnections = [];
@@ -129,7 +129,7 @@ export function getPossibleConnections(checker, sourceNode, targetNodes, nodeTyp
     for (const targetNode of targetNodes) {
         if (targetNode.id === sourceNode.id)
             continue; // Skip self-connection
-        const targetMetadata = nodeTypes.find((nt) => nt.id === targetNode.data.metadata.id);
+        const targetMetadata = nodeTypes.find((nt) => nt.node_type_id === targetNode.data.metadata.node_type_id);
         if (!targetMetadata)
             continue;
         // Get all input ports from target node
@@ -170,8 +170,8 @@ export function validateConnection(checker, sourceNodeId, sourcePortId, targetNo
         return { valid: false, error: 'Cannot connect node to itself' };
     }
     // Get node metadata
-    const sourceMetadata = nodeTypes.find((nt) => nt.id === sourceNode.data.metadata.id);
-    const targetMetadata = nodeTypes.find((nt) => nt.id === targetNode.data.metadata.id);
+    const sourceMetadata = nodeTypes.find((nt) => nt.node_type_id === sourceNode.data.metadata.node_type_id);
+    const targetMetadata = nodeTypes.find((nt) => nt.node_type_id === targetNode.data.metadata.node_type_id);
     if (!sourceMetadata || !targetMetadata) {
         return { valid: false, error: 'Node metadata not found' };
     }
@@ -200,14 +200,14 @@ export function getConnectionSuggestions(checker, nodeId, nodes, nodeTypes) {
     const node = nodes.find((n) => n.id === nodeId);
     if (!node)
         return [];
-    const metadata = nodeTypes.find((nt) => nt.id === node.data.metadata.id);
+    const metadata = nodeTypes.find((nt) => nt.node_type_id === node.data.metadata.node_type_id);
     if (!metadata)
         return [];
     const suggestions = [];
     // Get all other nodes
     const otherNodes = nodes.filter((n) => n.id !== nodeId);
     for (const otherNode of otherNodes) {
-        const otherMetadata = nodeTypes.find((nt) => nt.id === otherNode.data.metadata.id);
+        const otherMetadata = nodeTypes.find((nt) => nt.node_type_id === otherNode.data.metadata.node_type_id);
         if (!otherMetadata)
             continue;
         // Check outputs from other nodes to inputs of current node
@@ -253,8 +253,17 @@ export function getConnectionSuggestions(checker, nodeId, nodes, nodeTypes) {
  * @returns True if any cycle exists in the workflow
  */
 export function hasCycles(nodes, edges) {
-    // Build adjacency map once (O(E)) so the DFS inner loop is O(1) per lookup
-    // instead of scanning all edges on every recursive call (which was O(V*E)).
+    return detectCycles(nodes, edges);
+}
+/**
+ * Detect whether a directed graph contains a cycle, using DFS over a pre-built
+ * adjacency map (O(E) build, O(1) per neighbour lookup — avoids the O(V*E) of
+ * re-scanning every edge on each recursive call).
+ *
+ * Shared by `hasCycles` (all edges) and `hasInvalidCycles` (loopback edges
+ * pre-filtered out by the caller).
+ */
+function detectCycles(nodes, edges) {
     const adjacencyMap = new Map();
     for (const node of nodes) {
         adjacencyMap.set(node.id, []);
@@ -274,7 +283,6 @@ export function hasCycles(nodes, edges) {
             return false;
         visited.add(nodeId);
         recursionStack.add(nodeId);
-        // Use pre-built adjacency map instead of filtering all edges each call
         const neighbors = adjacencyMap.get(nodeId) || [];
         for (const target of neighbors) {
             if (hasCycleUtil(target))
@@ -283,7 +291,6 @@ export function hasCycles(nodes, edges) {
         recursionStack.delete(nodeId);
         return false;
     }
-    // Check each node
     for (const node of nodes) {
         if (!visited.has(node.id)) {
             if (hasCycleUtil(node.id))
@@ -309,53 +316,10 @@ export function hasCycles(nodes, edges) {
  * ```
  */
 export function hasInvalidCycles(nodes, edges) {
-    // Filter out loopback edges - these create valid cycles for loop iteration
+    // Filter out loopback edges - these create valid cycles for loop iteration,
+    // then reuse the shared detector over the remaining edges.
     const nonLoopbackEdges = edges.filter((edge) => !isLoopbackEdge(edge));
-    // Build adjacency map from non-loopback edges once (O(E)) so the DFS inner
-    // loop is O(1) per lookup instead of scanning all edges on every recursive
-    // call (which was O(V*E)).
-    const adjacencyMap = new Map();
-    for (const node of nodes) {
-        adjacencyMap.set(node.id, []);
-    }
-    for (const edge of nonLoopbackEdges) {
-        const neighbors = adjacencyMap.get(edge.source);
-        if (neighbors) {
-            neighbors.push(edge.target);
-        }
-    }
-    // Check for cycles using only non-loopback edges
-    const visited = new Set();
-    const recursionStack = new Set();
-    /**
-     * DFS utility to detect cycles in the graph
-     * @param nodeId - Current node being visited
-     * @returns True if a cycle is found from this node
-     */
-    function hasCycleUtil(nodeId) {
-        if (recursionStack.has(nodeId))
-            return true;
-        if (visited.has(nodeId))
-            return false;
-        visited.add(nodeId);
-        recursionStack.add(nodeId);
-        // Use pre-built adjacency map instead of filtering all edges each call
-        const neighbors = adjacencyMap.get(nodeId) || [];
-        for (const target of neighbors) {
-            if (hasCycleUtil(target))
-                return true;
-        }
-        recursionStack.delete(nodeId);
-        return false;
-    }
-    // Check each node for cycles
-    for (const node of nodes) {
-        if (!visited.has(node.id)) {
-            if (hasCycleUtil(node.id))
-                return true;
-        }
-    }
-    return false;
+    return detectCycles(nodes, nonLoopbackEdges);
 }
 /**
  * Get the execution order for a workflow (topological sort)

package/dist/utils/nodeIds.d.ts

@@ -12,7 +12,7 @@ interface NodeWithMetadata {
     id: string;
     data?: {
         metadata?: {
-            id?: string;
+            node_type_id?: string;
         };
     };
 }

package/dist/utils/nodeIds.js

@@ -11,7 +11,7 @@
 export function generateNodeId(nodeTypeId, existingNodes) {
     // Count how many nodes of this type already exist
     const existingNodeIds = existingNodes
-        .filter((node) => node.data?.metadata?.id === nodeTypeId)
+        .filter((node) => node.data?.metadata?.node_type_id === nodeTypeId)
         .map((node) => node.id);
     // Extract the numbers from existing IDs with the same prefix
     const existingNumbers = existingNodeIds

package/dist/utils/nodeSwap.js

@@ -176,9 +176,6 @@ export function executeSwap(oldNode, newMetadata, preview, allNodes, allEdges) {
             label: newMetadata.name,
             config: mappedConfig,
             metadata: newMetadata,
-            // Node components derive their handle IDs from data.nodeId — without it
-            // every edge anchored to this node silently disappears from the canvas.
-            nodeId: newNodeId,
             extensions
         }
     };
@@ -218,7 +215,7 @@ export function executeSwap(oldNode, newMetadata, preview, allNodes, allEdges) {
  * @returns The newer NodeMetadata if an upgrade is available, null otherwise
  */
 export function getVersionUpgrade(currentMetadata, allNodeTypes) {
-    const available = allNodeTypes.find((n) => n.id === currentMetadata.id);
+    const available = allNodeTypes.find((n) => n.node_type_id === currentMetadata.node_type_id);
     if (!available)
         return null;
     if (compareSemver(available.version, currentMetadata.version) > 0) {
@@ -253,7 +250,7 @@ function classifyMatch(oldPort, matchedPort) {
 export function computeSwapPreviewWithOptions(oldNode, newMetadata, edges, allNodes, options) {
     const checker = options.checker ?? null;
     const oldNodeId = oldNode.id;
-    const newNodeId = generateNodeId(newMetadata.id, allNodes);
+    const newNodeId = generateNodeId(newMetadata.node_type_id, allNodes);
     // Collect connected edges
     const connectedEdges = edges.filter((e) => e.source === oldNodeId || e.target === oldNodeId);
     // Try strategy-based port mapping
@@ -420,7 +417,7 @@ export function computeSwapPreviewWithOptions(oldNode, newMetadata, edges, allNo
  */
 export function computeInteractiveState(oldNode, newMetadata, edges, allNodes, options = {}) {
     const oldNodeId = oldNode.id;
-    const newNodeId = generateNodeId(newMetadata.id, allNodes);
+    const newNodeId = generateNodeId(newMetadata.node_type_id, allNodes);
     const connectedEdges = edges.filter((e) => e.source === oldNodeId || e.target === oldNodeId);
     // Compute the base preview to get auto-matched ports
     const preview = computeSwapPreviewWithOptions(oldNode, newMetadata, edges, allNodes, options);

package/package.json

@@ -3,7 +3,7 @@
   "description": "A drop-in visual workflow editor for any web application. You own the backend. You own the data. You own the orchestration.",
   "license": "MIT",
   "private": false,
-  "version": "2.0.0-beta.3",
+  "version": "2.0.0-beta.5",
   "author": "Shibin Das (D34dMan)",
   "bugs": {
     "url": "https://github.com/flowdrop-io/flowdrop/issues"