@bian-womp/spark-graph 0.2.59 → 0.2.60

package/lib/cjs/index.cjs

@@ -582,7 +582,7 @@ class GraphRuntime {
         const node = this.nodes.get(nodeId);
         return node?.outputs[output];
     }
-    static buildEdgeConverters(srcDeclared, dstDeclared, registry) {
+    static buildEdgeConverters(srcDeclared, dstDeclared, registry, edgeLabel) {
         if (!dstDeclared || !srcDeclared) {
             return {};
         }
@@ -601,15 +601,15 @@ class GraphRuntime {
             const payload = getTypedOutputValue(v);
             if (isUnion) {
                 if (!typeId) {
-                    throw new Error(`Typed output required for union source; allowed: ${srcTypes.join("|")}`);
+                    throw new Error(`Typed output required for union source (${edgeLabel}); allowed: ${srcTypes.join("|")}`);
                 }
                 if (!srcTypes.includes(typeId)) {
-                    throw new Error(`Invalid typed output ${typeId}; allowed: ${srcTypes.join("|")}`);
+                    throw new Error(`Invalid typed output ${typeId} (${edgeLabel}); allowed: ${srcTypes.join("|")}`);
                 }
             }
             else if (typeId) {
                 // Warn if typed output is used for non-union source
-                console.warn(`Typed output ${typeId} is fed even though source is not union: ${srcDeclared} -> ${dstDeclared}`);
+                console.warn(`Typed output ${typeId} is fed even though source is not union (${edgeLabel}): ${srcDeclared} -> ${dstDeclared}`);
             }
             return { typeId: typeId || srcTypes[0], payload };
         };
@@ -639,7 +639,7 @@ class GraphRuntime {
                 if (!res)
                     return payload;
                 if (res.kind === "async") {
-                    throw new Error("Async coercion required but convert used");
+                    throw new Error(`Async coercion required but convert used (${edgeLabel})`);
                 }
                 return res.convert(payload);
             },
@@ -1138,7 +1138,7 @@ class GraphRuntime {
                 if (resolved)
                     dstDeclared = getInputTypeId(resolved.inputs, e.target.handle);
             }
-            const { convert, convertAsync } = GraphRuntime.buildEdgeConverters(srcDeclared, dstDeclared, registry);
+            const { convert, convertAsync } = GraphRuntime.buildEdgeConverters(srcDeclared, dstDeclared, registry, `buildEdges: ${srcNode?.typeId || ""}.${e.source.nodeId}.${e.source.handle} -> ${dstNode?.typeId || ""}.${e.target.nodeId}.${e.target.handle}`);
             return {
                 id: e.id,
                 source: { ...e.source },
@@ -1168,6 +1168,8 @@ class GraphRuntime {
         this.resolvedByNode.set(nodeId, handles);
         // Recompute edge converter/type for edges where this node is source or target
         for (const e of this.edges) {
+            const srcNode = this.nodes.get(e.source.nodeId);
+            const dstNode = this.nodes.get(e.target.nodeId);
             let srcDeclared = e.effectiveTypeId; // Use effectiveTypeId as fallback
             let dstDeclared = e.dstDeclared;
             const oldDstDeclared = dstDeclared; // Track old value to detect resolution
@@ -1190,7 +1192,7 @@ class GraphRuntime {
                     e.dstDeclared = dstDeclared;
                 }
             }
-            const conv = GraphRuntime.buildEdgeConverters(srcDeclared, dstDeclared, registry);
+            const conv = GraphRuntime.buildEdgeConverters(srcDeclared, dstDeclared, registry, `updateNodeHandles: ${srcNode?.typeId || ""}.${e.source.nodeId}.${e.source.handle} -> ${dstNode?.typeId || ""}.${e.target.nodeId}.${e.target.handle}`);
             e.convert = conv.convert;
             e.convertAsync = conv.convertAsync;
             // If target handle was just resolved (was undefined, now has a type), re-propagate values