@onlineapps/cookbook-core 2.1.3 → 2.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/cookbook-core",
-  "version": "2.1.3",
+  "version": "2.1.4",
   "description": "Core cookbook parsing and validation - lightweight foundation for workflow definitions",
   "main": "src/index.js",
   "scripts": {

package/src/parser/cookbookValidatorV2.js

@@ -249,53 +249,48 @@ function validateSemantics(cookbook, version) {
       stepIds.add(stepId);
     }
 
-    // Check nested steps
-    if (step.steps) {
-      step.steps.forEach((nested, idx) =>
-        checkStep(nested, `${path}.steps[${idx}]`)
+    // Check nested steps - V2: all nested steps are objects keyed by step_id
+    if (step.steps && typeof step.steps === 'object') {
+      Object.entries(step.steps).forEach(([nestedKey, nested]) =>
+        checkStep(nested, `${path}.steps.${nestedKey}`)
       );
     }
-    if (step.body) {
-      step.body.forEach((nested, idx) =>
-        checkStep(nested, `${path}.body[${idx}]`)
+    if (step.body && typeof step.body === 'object') {
+      Object.entries(step.body).forEach(([nestedKey, nested]) =>
+        checkStep(nested, `${path}.body.${nestedKey}`)
       );
     }
-    if (step.branches) {
-      step.branches.forEach((branch, idx) => {
-        if (branch.steps) {
-          branch.steps.forEach((nested, jdx) =>
-            checkStep(nested, `${path}.branches[${idx}].steps[${jdx}]`)
-          );
-        }
+    if (step.branches && typeof step.branches === 'object') {
+      Object.entries(step.branches).forEach(([branchKey, branch]) => {
+        checkStep(branch, `${path}.branches.${branchKey}`);
       });
     }
     if (step.cases) {
       Object.keys(step.cases).forEach(caseKey => {
         const caseVal = step.cases[caseKey];
-        if (caseVal.steps) {
-          caseVal.steps.forEach((nested, idx) =>
-            checkStep(nested, `${path}.cases.${caseKey}.steps[${idx}]`)
-          );
-        }
+        checkStep(caseVal, `${path}.cases.${caseKey}`);
       });
     }
   }
 
-  // Check all steps
+  // Check all steps - V2: steps is an OBJECT keyed by step_id, NOT an array
   if (cookbook.steps) {
-    cookbook.steps.forEach((step, idx) =>
-      checkStep(step, `steps[${idx}]`)
+    if (Array.isArray(cookbook.steps)) {
+      throw new CookbookValidationError('V1 FORMAT BANNED! steps must be an object keyed by step_id, not an array');
+    }
+    Object.entries(cookbook.steps).forEach(([stepKey, step]) =>
+      checkStep(step, `steps.${stepKey}`)
     );
   }
 
   // Check dependencies exist
   if (version === '2.0' && cookbook.steps) {
-    cookbook.steps.forEach((step, idx) => {
+    Object.entries(cookbook.steps).forEach(([stepKey, step]) => {
       if (step.depends_on) {
         step.depends_on.forEach(dep => {
           if (!stepIds.has(dep)) {
             throw new CookbookValidationError(
-              `Step ${idx} depends on non-existent step_id '${dep}'`
+              `Step '${stepKey}' depends on non-existent step_id '${dep}'`
             );
           }
         });