@octaviaflow/flow-rules 0.20.0 → 0.21.0

package/dist/index.cjs

@@ -121,6 +121,7 @@ __export(exports_src, {
   contractDrift: () => contractDrift,
   contractFieldsFor: () => contractFieldsFor,
   contractFor: () => contractFor,
+  contractValidationErrors: () => contractValidationErrors,
   contractVocabularyDrift: () => contractVocabularyDrift,
   declaredKeys: () => declaredKeys,
   detectRecordFields: () => detectRecordFields,
@@ -5249,6 +5250,21 @@ function missingContractRequired(node) {
   }
   return missing;
 }
+function contractValidationErrors(node) {
+  if (HIDDEN_STATUS_ACTIONS.has(node.actionDefinitionId))
+    return [];
+  const contract = contractFor(node.actionDefinitionId);
+  if (!contract?.validate)
+    return [];
+  try {
+    const verdict = contract.validate(node.config ?? {});
+    if (verdict.valid)
+      return [];
+    return [...new Set(verdict.errors.map((e) => String(e).trim()).filter(Boolean))];
+  } catch {
+    return [];
+  }
+}
 
 // src/rules/workflowValidator.ts
 var nodeName = (n) => n.label || n.id;
@@ -5284,8 +5300,19 @@ function validateNode(node, allNodes, allEdges) {
       nodeId: node.id
     });
   }
+  const contractIssues = cfg.missingRequired.length === 0 ? contractValidationErrors(node) : [];
   const contractGaps = missingContractRequired(node);
-  if (contractGaps.length > 0) {
+  if (contractIssues.length > 0) {
+    const shown = contractIssues.slice(0, 3).map((e) => e.replace(/[.\s]+$/, ""));
+    const more = contractIssues.length > 3 ? ` (+${contractIssues.length - 3} more)` : "";
+    errors.push({
+      id: `node-${node.id}-contract`,
+      type: "node",
+      severity: "warning",
+      message: `This step will not run as set up: ${shown.join("; ")}${more}. Open the step to fix it.`,
+      nodeId: node.id
+    });
+  } else if (contractGaps.length > 0) {
     const fields = contractGaps.slice(0, 4).join(", ");
     const more = contractGaps.length > 4 ? ` +${contractGaps.length - 4} more` : "";
     errors.push({

package/dist/index.d.ts

@@ -21,7 +21,7 @@
 export * from "./catalog/types";
 export { actionsRegistry, getAllActions, getActionById, getActionsByCategory, } from "./catalog/index";
 export { validateNode, validateEdge, validateWorkflow, getNodeValidationStatus, getEdgeValidationStatus, type ValidationError, type ValidationResult, } from "./rules/workflowValidator";
-export { getNodeConfigStatus, missingContractRequired, type NodeConfigStatus, type NodeConfigStatusResult, } from "./rules/nodeConfigStatus";
+export { getNodeConfigStatus, missingContractRequired, contractValidationErrors, type NodeConfigStatus, type NodeConfigStatusResult, } from "./rules/nodeConfigStatus";
 export { generateWorkflowId, generateStepId, generateEdgeId, generateUUIDv4, } from "./ids";
 export { CONDITION_OPERATORS, ConditionConfig, type ConditionFilter, type ConditionOperator, } from "./catalog/logic/condition/config";
 export { ForEachConfig, type ForEachConfigType, type ForEachMode, } from "./catalog/logic/for-each/config";

package/dist/index.js

@@ -5073,6 +5073,21 @@ function missingContractRequired(node) {
   }
   return missing;
 }
+function contractValidationErrors(node) {
+  if (HIDDEN_STATUS_ACTIONS.has(node.actionDefinitionId))
+    return [];
+  const contract = contractFor(node.actionDefinitionId);
+  if (!contract?.validate)
+    return [];
+  try {
+    const verdict = contract.validate(node.config ?? {});
+    if (verdict.valid)
+      return [];
+    return [...new Set(verdict.errors.map((e) => String(e).trim()).filter(Boolean))];
+  } catch {
+    return [];
+  }
+}
 
 // src/rules/workflowValidator.ts
 var nodeName = (n) => n.label || n.id;
@@ -5108,8 +5123,19 @@ function validateNode(node, allNodes, allEdges) {
       nodeId: node.id
     });
   }
+  const contractIssues = cfg.missingRequired.length === 0 ? contractValidationErrors(node) : [];
   const contractGaps = missingContractRequired(node);
-  if (contractGaps.length > 0) {
+  if (contractIssues.length > 0) {
+    const shown = contractIssues.slice(0, 3).map((e) => e.replace(/[.\s]+$/, ""));
+    const more = contractIssues.length > 3 ? ` (+${contractIssues.length - 3} more)` : "";
+    errors.push({
+      id: `node-${node.id}-contract`,
+      type: "node",
+      severity: "warning",
+      message: `This step will not run as set up: ${shown.join("; ")}${more}. Open the step to fix it.`,
+      nodeId: node.id
+    });
+  } else if (contractGaps.length > 0) {
     const fields = contractGaps.slice(0, 4).join(", ");
     const more = contractGaps.length > 4 ? ` +${contractGaps.length - 4} more` : "";
     errors.push({
@@ -6231,6 +6257,7 @@ export {
   contractDrift,
   contractFieldsFor,
   contractFor,
+  contractValidationErrors,
   contractVocabularyDrift,
   declaredKeys,
   detectRecordFields,

package/dist/rules/nodeConfigStatus.d.ts

@@ -43,3 +43,14 @@ export declare function getNodeConfigStatus(node: WorkflowNode): NodeConfigStatu
  * The drift test pins the list of such keys, so a new one is a decision.
  */
 export declare function missingContractRequired(node: WorkflowNode): string[];
+/**
+ * What the action's own contract check (`validate()`, the check its config
+ * panel runs) says is wrong with this node, deduplicated (DEV-643).
+ *
+ * `getNodeConfigStatus` only asks whether each required setting is filled in;
+ * a Variables step whose `fields` holds a row with no key passed it, so
+ * `check_flow` and the canvas called the flow fine while the panel and the
+ * MCP's `add_step` said "Each field needs a key". A validator that throws is
+ * treated as silent: a broken check must not invent a problem.
+ */
+export declare function contractValidationErrors(node: WorkflowNode): string[];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@octaviaflow/flow-rules",
-  "version": "0.20.0",
+  "version": "0.21.0",
   "description": "The flow action catalog and the rules engine behind Flow Doctor \u2014 one definition of what a step accepts and what makes a flow valid, shared by the editor and the server",
   "type": "module",
   "main": "./dist/index.cjs",