factoryhub 0.1.0 → 0.1.1

package/spec/schema/flow.schema.json

@@ -5,12 +5,22 @@
   "description": "A factory's executable workflow, format v1alpha1 (mindburn.flow/v1). Describes the document after YAML parsing. Rules across steps and files (references, cycles, agent files) are in spec/factory-format.md, section 6.",
   "type": "object",
   "required": ["steps"],
+  "allOf": [
+    {
+      "description": "A check that says on_fail: stop needs requires: [blocking-checks], so a runner without blocking checks refuses the flow instead of delivering work its checker refused.",
+      "if": {
+        "required": ["steps"],
+        "properties": { "steps": { "type": "object", "not": { "type": "object", "additionalProperties": { "not": { "$ref": "#/$defs/stopCheck" } } } } }
+      },
+      "then": { "required": ["requires"], "properties": { "requires": { "type": "array", "contains": { "const": "blocking-checks" } } } }
+    }
+  ],
   "properties": {
     "flow": { "description": "The flow format.", "const": "mindburn.flow/v1" },
     "requires": {
-      "description": "Execution features the flow depends on.",
+      "description": "Execution features the flow depends on. A flow with a check that says on_fail: stop MUST require blocking-checks.",
       "type": "array",
-      "items": { "enum": ["scoped-context", "multiple-outputs"] }
+      "items": { "enum": ["scoped-context", "multiple-outputs", "blocking-checks"] }
     },
     "title": { "type": "string" },
     "author": { "type": "string" },
@@ -69,6 +79,7 @@
             "properties": {
               "id": { "type": "string" },
               "name": { "type": "string" },
+              "title": { "description": "The member factory's title, at most 120 characters.", "type": "string", "maxLength": 120 },
               "version": { "type": "string" },
               "sha256": { "type": "string", "pattern": "^[a-f0-9]{64}$" }
             }
@@ -125,6 +136,7 @@
         "checks": { "$ref": "#/$defs/name" },
         "shows": { "$ref": "#/$defs/name" },
         "rounds": { "type": "integer", "minimum": 0, "maximum": 5 },
+        "on_fail": { "description": "For a check: what happens when the work still fails after its redos. flag (the default) delivers it and flags the check; stop holds it for the owner to release or discard. stop needs requires: [blocking-checks].", "enum": ["stop", "flag"] },
         "tools": { "$ref": "#/$defs/toolList" },
         "mcp": {}, "mcpServers": {}, "mcp_servers": {}, "connectors": {}, "plugins": {}, "skills": {},
         "context": {
@@ -157,9 +169,19 @@
         {
           "if": { "required": ["shows"] },
           "then": { "properties": { "kind": { "const": "approve" } }, "required": ["kind"] }
+        },
+        {
+          "if": { "required": ["on_fail"] },
+          "then": { "required": ["kind"], "properties": { "kind": { "const": "check" } } }
         }
       ]
     },
+    "stopCheck": {
+      "description": "A step that says on_fail: stop.",
+      "type": "object",
+      "required": ["on_fail"],
+      "properties": { "on_fail": { "const": "stop" } }
+    },
     "bindings": {
       "type": "object",
       "propertyNames": { "pattern": "^[a-z][a-z0-9_]{0,63}$" },

package/spec/skill-export.md

@@ -82,6 +82,9 @@ After the frontmatter, a blank line, then these parts in order:
      needs, its tools, then its task;
    - check: title, the reviewer agent, what it needs, the step it checks, the
      criteria as a quotation, and what happens when it fails, by `rounds`;
+     for a check with `on_fail: stop`, the stop text in
+     [the format, 5.3](factory-format.md#blocking-checks), which
+     `exportSkill()` does not write yet at this commit;
    - approve: title, "Who: the person", what it needs, then
      `Stop and ask the person to approve the result of "<subject title>".`,
      "Show it to them in full", its task, and what approve, change (by