@bian-womp/spark-graph 0.2.59 → 0.2.61

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);
             },
@@ -683,6 +683,7 @@ class GraphRuntime {
             }
         };
         return {
+            nodeId,
             state: node.state,
             setState: (next) => Object.assign(node.state, next),
             emit: emitHandler,
@@ -1077,6 +1078,7 @@ class GraphRuntime {
             try {
                 if (typeof desc.resolveHandles === "function") {
                     const maybe = desc.resolveHandles({
+                        nodeId: n.nodeId,
                         environment: environment || {},
                         params: n.params,
                         inputs: undefined,
@@ -1138,7 +1140,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 +1170,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 +1194,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
@@ -1668,6 +1672,7 @@ class GraphRuntime {
         let r;
         try {
             const res = resolveHandles({
+                nodeId,
                 environment: this.environment || {},
                 params: node.params,
                 inputs: node.inputs || {},