Structured control-flow constructs (ADR-0031) — the native + AI-authored
flow model: a loopcontainer, a parallelblock, and structured
try/catch/retry. Unlike BPMN's gateway/boundary/token graph (kept in the
protocol for interop only), these constructs are well-formed by
construction, locally composable, and statically analyzable — the right
substrate for LLM authoring (ADR-0010/0011).
ADR-0031 flagged two ways to carry structured containers in the flat
nodes[]+edges[] model:
(A) marker-delimited scoped regions (a container node + a scope-end
marker; the body is the edges between them in the main graph), or
(B) the container node carries a nested mini-flow in its config.
We adopt (B). Each container holds its body as a self-contained
FlowRegionSchema (config.body for loop, config.branches[] for
parallel, config.try/config.catch for try_catch). The reasons:
Well-formed by construction — a nested region is its own graph, so
single-entry is intrinsic; there are no scope markers to balance and no
way to "leak" an edge across a boundary. Validation is local.
The shared engine traversal stays untouched — the container executor
runs its own body via a scoped helper; the main DAG traverseNext never
learns about scope markers (important under the multi-agent discipline
around engine.ts). The container's ordinary out-edges remain the
"after-loop / after-block" continuation.
Cleaner AST for AI — ADR-0031 calls (B) "the cleaner long-term AST,"
and AI authoring is the design center.
Existing flat-graph loops (a loop node with no config.body) keep their
legacy behavior — the constructs are additive, activated only when the
nested structure is present.
The canonical construct type ids are LOOP_NODE_TYPE (loop,
pre-existing), PARALLEL_NODE_TYPE (parallel), and
TRY_CATCH_NODE_TYPE (try_catch). These are distinct from the BPMN
interop node types (parallel_gateway / join_gateway / boundary_event),
which remain author-invisible interchange representations.
Every shape below is strictObject. Before that they were plain z.object,
so zod's default .strip applied and a key this file does not declare was
discarded in silence — the container still parsed, still registered, and
still ran, with the author's configuration simply absent. On these five
shapes that silence is unusually expensive, because each one carries
control rather than data: a swallowed maxIterations is an uncapped loop,
a swallowed branch key is a branch that runs without what it was given.
validateControlFlow is a sibling guard, not a key gate — it answers
"is this region single-entry / single-exit / acyclic", which no amount of
key strictness can answer. The two do not overlap and cannot fight: the
schema rejects undeclared KEYS, the analysis rejects malformed STRUCTURE.
They do now meet at one seam, deliberately — validateControlFlowsafeParses each region slot before analyzing it, so from #4001 that parse
is also where a region's undeclared key surfaces, reported as
<where>: invalid region — <the strictObject message>. Nothing was
duplicated and nothing was removed; the structural prose this guard exists
for is untouched, and it simply stopped silently repairing its own input.
Action type — a built-in FlowNodeAction id or a plugin-registered node type. Validated against the live action registry at registerFlow() (ADR-0018), not by a closed enum.
[REMOVED] flow.nodes[].outputSchema was removed in @objectstack/spec 17.0.0 (audit close-out) — it was never validated: the engine does not check node outputs against it, so it documented a contract nothing enforced. Delete the key. Downstream nodes read prior outputs via expressions ({{nodeId.field}}) regardless of any declaration. Run os migrate meta --from 16 to list the mechanical edits for existing sources; apply them by hand.
Connection type: default (normal flow), fault (error path), conditional (expression-guarded), or back (ADR-0044 declared back-edge — traversed normally at run time, but excluded from DAG cycle validation so a revise/rework loop can re-enter an earlier node)
label
string
optional
Label on the connector
isDefault
boolean
optional (default: false)
BPMN default flow: traverse this edge only when no sibling conditional edge of the same source node matched. Mutually exclusive with condition; at most one per source node.
Action type — a built-in FlowNodeAction id or a plugin-registered node type. Validated against the live action registry at registerFlow() (ADR-0018), not by a closed enum.
[REMOVED] flow.nodes[].outputSchema was removed in @objectstack/spec 17.0.0 (audit close-out) — it was never validated: the engine does not check node outputs against it, so it documented a contract nothing enforced. Delete the key. Downstream nodes read prior outputs via expressions ({{nodeId.field}}) regardless of any declaration. Run os migrate meta --from 16 to list the mechanical edits for existing sources; apply them by hand.
Connection type: default (normal flow), fault (error path), conditional (expression-guarded), or back (ADR-0044 declared back-edge — traversed normally at run time, but excluded from DAG cycle validation so a revise/rework loop can re-enter an earlier node)
label
string
optional
Label on the connector
isDefault
boolean
optional (default: false)
BPMN default flow: traverse this edge only when no sibling conditional edge of the same source node matched. Mutually exclusive with condition; at most one per source node.
Retry attempts after the initial one. 0 (the default) means no retry — state a count to opt in.
backoffMs
integer
optional (default: 1000)
Base delay before the first retry (ms); subsequent delays multiply by backoffMultiplier
backoffMultiplier
number
optional (default: 1)
Exponential backoff multiplier; 1 (the default) keeps the delay flat
maxRetryDelayMs
integer
optional (default: 30000)
Ceiling for a single backoff delay (ms)
jitter
boolean
optional (default: false)
Randomize each delay within [50%, 100%] of its computed value — spreads a thundering herd of simultaneous retries
retryDelayMs
never
optional
[REMOVED] retryDelayMs was removed in @objectstack/spec 17.0.0 — the retry policy now has ONE spelling for its base delay across every surface that carries it: job.retryPolicy, a try_catch node's retry and flow.errorHandling. Rename the key to backoffMs; the value (milliseconds before the first retry) is unchanged. Run os migrate meta --from 16 to list the mechanical edits for existing sources; apply them by hand.
Retry attempts after the initial one. 0 (the default) means no retry — state a count to opt in.
backoffMs
integer
optional (default: 1000)
Base delay before the first retry (ms); subsequent delays multiply by backoffMultiplier
backoffMultiplier
number
optional (default: 1)
Exponential backoff multiplier; 1 (the default) keeps the delay flat
maxRetryDelayMs
integer
optional (default: 30000)
Ceiling for a single backoff delay (ms)
jitter
boolean
optional (default: false)
Randomize each delay within [50%, 100%] of its computed value — spreads a thundering herd of simultaneous retries
retryDelayMs
never
optional
[REMOVED] retryDelayMs was removed in @objectstack/spec 17.0.0 — the retry policy now has ONE spelling for its base delay across every surface that carries it: job.retryPolicy, a try_catch node's retry and flow.errorHandling. Rename the key to backoffMs; the value (milliseconds before the first retry) is unchanged. Run os migrate meta --from 16 to list the mechanical edits for existing sources; apply them by hand.