snow-flow 10.0.1-dev.489 → 10.0.1-dev.490

package/package.json

@@ -1,6 +1,6 @@
 {
   "$schema": "https://json.schemastore.org/package.json",
-  "version": "10.0.1-dev.489",
+  "version": "10.0.1-dev.490",
   "name": "snow-flow",
   "description": "Snow-Flow - ServiceNow Multi-Agent Development Framework powered by AI",
   "license": "Elastic-2.0",

package/src/servicenow/servicenow-mcp-unified/tools/flow-designer/snow_manage_flow.ts

@@ -241,7 +241,12 @@ async function buildActionInputsForInsert(
     };
   });
 
-  return { inputs, resolvedInputs, actionParams };
+  // Check for mandatory fields that are missing a value
+  var missingMandatory = inputs
+    .filter(function (inp: any) { return inp.parameter?.mandatory && !inp.value?.value; })
+    .map(function (inp: any) { return inp.name + ' (' + (inp.parameter?.label || inp.name) + ')'; });
+
+  return { inputs, resolvedInputs, actionParams, missingMandatory };
 }
 
 /**
@@ -373,7 +378,12 @@ async function buildFlowLogicInputsForInsert(
     variables: '[]'
   };
 
-  return { inputs, flowLogicDefinition, resolvedInputs, inputQueryError: inputQueryError || undefined, defParamsCount: defParams.length };
+  // Check for mandatory fields that are missing a value
+  var missingMandatory = inputs
+    .filter(function (inp: any) { return inp.parameter?.mandatory && !inp.value?.value; })
+    .map(function (inp: any) { return inp.name + ' (' + (inp.parameter?.label || inp.name) + ')'; });
+
+  return { inputs, flowLogicDefinition, resolvedInputs, inputQueryError: inputQueryError || undefined, defParamsCount: defParams.length, missingMandatory };
 }
 
 // Note: reordering of existing elements is NOT possible via Table API because
@@ -1127,6 +1137,12 @@ async function addActionViaGraphQL(
   steps.available_inputs = inputResult.actionParams.map((p: any) => ({ element: p.element, label: p.label }));
   steps.resolved_inputs = inputResult.resolvedInputs;
 
+  // Validate mandatory fields
+  if (inputResult.missingMandatory && inputResult.missingMandatory.length > 0) {
+    steps.missing_mandatory = inputResult.missingMandatory;
+    return { success: false, error: 'Missing required inputs for ' + actionType + ': ' + inputResult.missingMandatory.join(', ') + '. These fields are mandatory in Flow Designer.', steps };
+  }
+
   // Calculate insertion order
   const resolvedOrder = await calculateInsertOrder(client, flowId, parentUiId, order);
   steps.insert_order = resolvedOrder;
@@ -2017,6 +2033,12 @@ async function addFlowLogicViaGraphQL(
   steps.resolved_inputs = inputResult.resolvedInputs;
   steps.input_query_stats = { defParamsFound: inputResult.defParamsCount, inputsBuilt: inputResult.inputs.length, error: inputResult.inputQueryError };
 
+  // Validate mandatory fields (e.g. condition for IF/ELSEIF)
+  if (inputResult.missingMandatory && inputResult.missingMandatory.length > 0) {
+    steps.missing_mandatory = inputResult.missingMandatory;
+    return { success: false, error: 'Missing required inputs for ' + logicType + ': ' + inputResult.missingMandatory.join(', ') + '. These fields are mandatory in Flow Designer.', steps };
+  }
+
   // ── Detect condition that needs data pill transformation ────────────
   // Flow Designer sets conditions via a SEPARATE UPDATE after the element is created.
   // Three paths: