@sweny-ai/core 0.1.33 → 0.1.35

package/dist/__tests__/studio.test.js

@@ -65,6 +65,79 @@ describe("workflowToFlow", () => {
             expect(node.data.exec.status).toBe("pending");
         }
     });
+    // Defensive behavior: streaming LLM YAML (and user-authored
+    // workflows) routinely arrive with fields omitted. The viewer
+    // must not crash on these — it should render an empty state for
+    // the missing pieces.
+    describe("defensive handling of partial workflows", () => {
+        it("treats a node with undefined skills as having no skills", () => {
+            const partial = {
+                id: "partial",
+                name: "Partial",
+                description: "",
+                entry: "a",
+                nodes: {
+                    // Cast to unknown→Node: we're deliberately simulating
+                    // what a partial YAML parse produces at runtime.
+                    a: { name: "A", instruction: "Do A" },
+                },
+                edges: [],
+            };
+            expect(() => workflowToFlow(partial)).not.toThrow();
+            const { nodes } = workflowToFlow(partial);
+            expect(nodes[0].data.skills).toEqual([]);
+        });
+        it("treats a workflow with undefined edges as having no edges", () => {
+            const partial = {
+                id: "partial",
+                name: "Partial",
+                description: "",
+                entry: "a",
+                nodes: {
+                    a: { name: "A", instruction: "Do A", skills: [] },
+                },
+            };
+            expect(() => workflowToFlow(partial)).not.toThrow();
+            const { nodes, edges } = workflowToFlow(partial);
+            expect(edges).toEqual([]);
+            // The sole node is still terminal because there are no outgoing edges.
+            expect(nodes[0].data.isTerminal).toBe(true);
+        });
+        it("treats a workflow with undefined nodes as having no nodes", () => {
+            const partial = {
+                id: "partial",
+                name: "Partial",
+                description: "",
+                entry: "a",
+                edges: [],
+            };
+            expect(() => workflowToFlow(partial)).not.toThrow();
+            const { nodes, edges } = workflowToFlow(partial);
+            expect(nodes).toEqual([]);
+            expect(edges).toEqual([]);
+        });
+        it("treats a skill catalog entry with missing tools as toolCount=0", () => {
+            const brokenSkill = {
+                id: "broken",
+                name: "Broken",
+                description: "No tools array",
+                category: "general",
+                config: {},
+                // tools omitted on purpose
+            };
+            const wf = {
+                id: "t",
+                name: "T",
+                description: "",
+                entry: "a",
+                nodes: { a: { name: "A", instruction: "Do A", skills: ["broken"] } },
+                edges: [],
+            };
+            expect(() => workflowToFlow(wf, [brokenSkill])).not.toThrow();
+            const { nodes } = workflowToFlow(wf, [brokenSkill]);
+            expect(nodes[0].data.skills[0].toolCount).toBe(0);
+        });
+    });
 });
 describe("flowToWorkflow", () => {
     it("round-trips correctly", () => {

package/dist/studio.js

@@ -49,7 +49,15 @@ import { builtinSkills } from "./skills/index.js";
 export function workflowToFlow(workflow, skillCatalog = builtinSkills) {
     const skillMap = new Map(skillCatalog.map((s) => [s.id, s]));
     const terminalIds = findTerminals(workflow);
-    const nodes = Object.entries(workflow.nodes).map(([id, node]) => ({
+    // Defensive: streaming LLM YAML (or user-authored partial workflows)
+    // can produce nodes with missing `skills` / workflows with missing
+    // `edges`. The types say these are required, but runtime data from
+    // untrusted sources frequently violates that. Rather than crash the
+    // viewer with "Cannot read properties of undefined (reading 'map')",
+    // treat them as empty and let the caller decide what to render.
+    const workflowNodes = workflow.nodes ?? {};
+    const workflowEdges = workflow.edges ?? [];
+    const nodes = Object.entries(workflowNodes).map(([id, node]) => ({
         id,
         type: "skillNode",
         position: { x: 0, y: 0 }, // ELK will compute real positions
@@ -58,18 +66,18 @@ export function workflowToFlow(workflow, skillCatalog = builtinSkills) {
             node,
             isEntry: id === workflow.entry,
             isTerminal: terminalIds.has(id),
-            skills: node.skills.map((sid) => {
+            skills: (node.skills ?? []).map((sid) => {
                 const skill = skillMap.get(sid);
                 return {
                     id: sid,
                     name: skill?.name ?? sid,
-                    toolCount: skill?.tools.length ?? 0,
+                    toolCount: skill?.tools?.length ?? 0,
                 };
             }),
             exec: { status: "pending" },
         },
     }));
-    const edges = workflow.edges.map((edge) => ({
+    const edges = workflowEdges.map((edge) => ({
         id: `${edge.from}--${edge.to}`,
         source: edge.from,
         target: edge.to,
@@ -143,8 +151,8 @@ export function flowToWorkflow(meta, nodes, edges) {
 }
 // ─── Helpers ─────────────────────────────────────────────────────
 function findTerminals(workflow) {
-    const hasOutgoing = new Set(workflow.edges.map((e) => e.from));
-    return new Set(Object.keys(workflow.nodes).filter((id) => !hasOutgoing.has(id)));
+    const hasOutgoing = new Set((workflow.edges ?? []).map((e) => e.from));
+    return new Set(Object.keys(workflow.nodes ?? {}).filter((id) => !hasOutgoing.has(id)));
 }
 /**
  * Generate a TypeScript file from a Workflow definition.

package/package.json

@@ -1,11 +1,11 @@
 {
   "name": "@sweny-ai/core",
-  "version": "0.1.33",
+  "version": "0.1.35",
   "type": "module",
   "bin": {
     "sweny": "./dist/cli/main.js"
   },
-  "description": "Skill library + DAG workflow orchestration powered by Claude",
+  "description": "Skill library, DAG workflow orchestration, and E2E browser testing powered by Claude",
   "license": "MIT",
   "exports": {
     ".": "./dist/index.js",