primitive-admin 1.1.0-alpha.9 → 1.1.0

package/dist/src/lib/workflow-config-apply.js

@@ -0,0 +1,137 @@
+/**
+ * Apply a workflow's named config sidecars, and honor `activeConfigName`
+ * (issue #2645, spec §Contracts/Named configs).
+ *
+ * ── What this is for ─────────────────────────────────────────────────────
+ * Named configs used to be reachable only through `workflows configs
+ * create/update/duplicate/activate`. Those verbs are removed, so `config push`
+ * has to apply the bodies now — otherwise the capability would have been
+ * deleted rather than moved to TOML, which the spec's criterion 2 forbids.
+ *
+ * ── Division of labour with `applyWorkflowBody` ──────────────────────────
+ * The ACTIVE config's body stays where it already is: the `steps` at the top of
+ * `workflows/<key>.toml`, written through `applyWorkflowBody`'s
+ * ordering-sensitive sequence (metadata PATCH → steps → deferred `syncCallable`
+ * PATCH, #807). That path is deliberately untouched — a second single-object
+ * apply is what broke that ordering before. This module handles the OTHER named
+ * configs, each from its own sidecar, and then the activation.
+ *
+ * So each config body is declared exactly once: the live one at the top of the
+ * workflow file, every other one in `workflows/<key>.configs/<name>.toml`.
+ * After a pull that follows an `activeConfigName` change, the two swap roles.
+ *
+ * ── Ordering ─────────────────────────────────────────────────────────────
+ * Bodies first, activation last. Activating a config before writing its steps
+ * would put an empty or stale body into service for the width of one call.
+ */
+/**
+ * Create or update each sidecar's config, then activate the named one.
+ *
+ * Reports whether it activated anything: activation writes the workflow
+ * DEFINITION (`activeConfigId`, and with it a fresh `modifiedAt`), so the
+ * caller's sync state is stale afterwards unless it re-reads — which is how a
+ * push that only switched the active config left the next one to conflict.
+ */
+export async function applyWorkflowConfigSidecars(client, appId, workflowId, sidecars, options) {
+    const { serverConfigs, activeConfigName, currentActiveConfigId, adoptActiveConfigName, } = options;
+    const log = options.log ?? (() => { });
+    const nameToId = new Map();
+    for (const config of serverConfigs ?? []) {
+        if (config?.configName && config?.configId) {
+            nameToId.set(config.configName, config.configId);
+        }
+    }
+    // The workflow file's own `[[steps]]` IS the active config's body — pull
+    // deliberately does not repeat it as a sidecar
+    // (`workflowConfigSidecarsToWrite`). So when `activeConfigName` names no
+    // config the server knows, the config that just received that body is the
+    // one the file is naming, and it takes the authored name.
+    //
+    // Without this, a pulled tree could not be pushed back: the server names a
+    // new workflow's first config "default" (`admin-api.ts`), so any workflow
+    // whose active config is called something else pulled to a tree whose
+    // `activeConfigName` matched no sidecar and no server config — push then
+    // refused a file pull had just written. This runs BEFORE the sidecar loop so
+    // a `default.toml` sidecar creates its own config rather than overwriting
+    // the active body on its way past.
+    if (adoptActiveConfigName &&
+        activeConfigName &&
+        !nameToId.has(activeConfigName) &&
+        currentActiveConfigId) {
+        await client.updateWorkflowConfig(appId, workflowId, currentActiveConfigId, { configName: activeConfigName });
+        // The name it had is now free, and the map must say so. Leaving the old
+        // entry pointing at this id makes the sidecar of that name look like an
+        // existing config: the loop below UPDATES the just-renamed one instead of
+        // creating its own, both sidecars collapse onto a single config, and
+        // whichever is applied last silently wins. A fresh cross-app push of a
+        // two-config workflow landed exactly one config because of this.
+        for (const [name, id] of nameToId) {
+            if (id === currentActiveConfigId)
+                nameToId.delete(name);
+        }
+        nameToId.set(activeConfigName, currentActiveConfigId);
+        log(`    Named the active config "${activeConfigName}"`);
+    }
+    for (const sidecar of sidecars) {
+        const existingId = nameToId.get(sidecar.name);
+        if (existingId) {
+            // In place: the config keeps its id, so anything pointing at it (a test
+            // case, a running trigger) keeps working across the edit.
+            //
+            // `description` and `status` travel with the steps: they are in the file,
+            // so the file owns them (#1567's convention — an omitted `description`
+            // sends `null` and clears, since that is what the round-trip means). An
+            // update that sent steps alone silently ignored half of what the author
+            // wrote, which is the same invisibility the flag removal was about.
+            await client.updateWorkflowConfig(appId, workflowId, existingId, {
+                steps: sidecar.steps,
+                expressions: sidecar.expressions,
+                description: sidecar.description ?? null,
+                // A defaulted enum, so there is no "clear": an absent key leaves the
+                // server's value alone rather than inventing one.
+                status: sidecar.status,
+            });
+            log(`    Updated config "${sidecar.name}"`);
+        }
+        else {
+            const created = await client.createWorkflowConfig(appId, workflowId, {
+                configName: sidecar.name,
+                description: sidecar.description,
+                steps: sidecar.steps,
+                expressions: sidecar.expressions,
+            });
+            if (created?.configId)
+                nameToId.set(sidecar.name, created.configId);
+            log(`    Created config "${sidecar.name}"`);
+            // Create always mints an ACTIVE config (admin-api.ts, createWorkflowConfig
+            // hard-codes it), so a sidecar that says otherwise needs the follow-up
+            // write or its `status` would be silently dropped on a fresh app.
+            if (created?.configId && sidecar.status && sidecar.status !== "active") {
+                await client.updateWorkflowConfig(appId, workflowId, created.configId, {
+                    status: sidecar.status,
+                });
+            }
+        }
+    }
+    let activated = false;
+    let activeConfigId = currentActiveConfigId;
+    if (activeConfigName) {
+        const targetId = nameToId.get(activeConfigName);
+        if (!targetId) {
+            // Leaving the old config live would mean the committed file says one
+            // config is running and another is — the exact drift this issue removes.
+            // A typo has to stop the push, not be absorbed by it.
+            throw new Error(`activeConfigName "${activeConfigName}" names no configuration. ` +
+                `Author it at workflows/<key>.configs/${activeConfigName}.toml, or ` +
+                `correct the name in workflows/<key>.toml.`);
+        }
+        if (targetId !== currentActiveConfigId) {
+            await client.activateWorkflowConfig(appId, workflowId, targetId);
+            log(`    Activated config "${activeConfigName}"`);
+            activated = true;
+        }
+        activeConfigId = targetId;
+    }
+    return { configIds: nameToId, activated, activeConfigId };
+}
+//# sourceMappingURL=workflow-config-apply.js.map

package/dist/src/lib/workflow-config-apply.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"workflow-config-apply.js","sourceRoot":"","sources":["../../../src/lib/workflow-config-apply.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;GAyBG;AAwCH;;;;;;;GAOG;AACH,MAAM,CAAC,KAAK,UAAU,2BAA2B,CAC/C,MAAiB,EACjB,KAAa,EACb,UAAkB,EAClB,QAA0C,EAC1C,OAAoC;IAEpC,MAAM,EACJ,aAAa,EACb,gBAAgB,EAChB,qBAAqB,EACrB,qBAAqB,GACtB,GAAG,OAAO,CAAC;IACZ,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,IAAI,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;IAEtC,MAAM,QAAQ,GAAG,IAAI,GAAG,EAAkB,CAAC;IAC3C,KAAK,MAAM,MAAM,IAAI,aAAa,IAAI,EAAE,EAAE,CAAC;QACzC,IAAI,MAAM,EAAE,UAAU,IAAI,MAAM,EAAE,QAAQ,EAAE,CAAC;YAC3C,QAAQ,CAAC,GAAG,CAAC,MAAM,CAAC,UAAU,EAAE,MAAM,CAAC,QAAQ,CAAC,CAAC;QACnD,CAAC;IACH,CAAC;IAED,yEAAyE;IACzE,+CAA+C;IAC/C,yEAAyE;IACzE,0EAA0E;IAC1E,0DAA0D;IAC1D,EAAE;IACF,2EAA2E;IAC3E,0EAA0E;IAC1E,sEAAsE;IACtE,yEAAyE;IACzE,6EAA6E;IAC7E,0EAA0E;IAC1E,mCAAmC;IACnC,IACE,qBAAqB;QACrB,gBAAgB;QAChB,CAAC,QAAQ,CAAC,GAAG,CAAC,gBAAgB,CAAC;QAC/B,qBAAqB,EACrB,CAAC;QACD,MAAM,MAAM,CAAC,oBAAoB,CAC/B,KAAK,EACL,UAAU,EACV,qBAAqB,EACrB,EAAE,UAAU,EAAE,gBAAgB,EAAE,CACjC,CAAC;QACF,wEAAwE;QACxE,wEAAwE;QACxE,0EAA0E;QAC1E,qEAAqE;QACrE,uEAAuE;QACvE,iEAAiE;QACjE,KAAK,MAAM,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,QAAQ,EAAE,CAAC;YAClC,IAAI,EAAE,KAAK,qBAAqB;gBAAE,QAAQ,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;QAC1D,CAAC;QACD,QAAQ,CAAC,GAAG,CAAC,gBAAgB,EAAE,qBAAqB,CAAC,CAAC;QACtD,GAAG,CAAC,gCAAgC,gBAAgB,GAAG,CAAC,CAAC;IAC3D,CAAC;IAED,KAAK,MAAM,OAAO,IAAI,QAAQ,EAAE,CAAC;QAC/B,MAAM,UAAU,GAAG,QAAQ,CAAC,GAAG,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;QAC9C,IAAI,UAAU,EAAE,CAAC;YACf,wEAAwE;YACxE,0DAA0D;YAC1D,EAAE;YACF,0EAA0E;YAC1E,uEAAuE;YACvE,wEAAwE;YACxE,wEAAwE;YACxE,oEAAoE;YACpE,MAAM,MAAM,CAAC,oBAAoB,CAAC,KAAK,EAAE,UAAU,EAAE,UAAU,EAAE;gBAC/D,KAAK,EAAE,OAAO,CAAC,KAAK;gBACpB,WAAW,EAAE,OAAO,CAAC,WAAW;gBAChC,WAAW,EAAE,OAAO,CAAC,WAAW,IAAI,IAAI;gBACxC,qEAAqE;gBACrE,kDAAkD;gBAClD,MAAM,EAAE,OAAO,CAAC,MAAM;aACvB,CAAC,CAAC;YACH,GAAG,CAAC,uBAAuB,OAAO,CAAC,IAAI,GAAG,CAAC,CAAC;QAC9C,CAAC;aAAM,CAAC;YACN,MAAM,OAAO,GAAG,MAAM,MAAM,CAAC,oBAAoB,CAAC,KAAK,EAAE,UAAU,EAAE;gBACnE,UAAU,EAAE,OAAO,CAAC,IAAI;gBACxB,WAAW,EAAE,OAAO,CAAC,WAAW;gBAChC,KAAK,EAAE,OAAO,CAAC,KAAK;gBACpB,WAAW,EAAE,OAAO,CAAC,WAAW;aACjC,CAAC,CAAC;YACH,IAAI,OAAO,EAAE,QAAQ;gBAAE,QAAQ,CAAC,GAAG,CAAC,OAAO,CAAC,IAAI,EAAE,OAAO,CAAC,QAAQ,CAAC,CAAC;YACpE,GAAG,CAAC,uBAAuB,OAAO,CAAC,IAAI,GAAG,CAAC,CAAC;YAC5C,2EAA2E;YAC3E,uEAAuE;YACvE,kEAAkE;YAClE,IAAI,OAAO,EAAE,QAAQ,IAAI,OAAO,CAAC,MAAM,IAAI,OAAO,CAAC,MAAM,KAAK,QAAQ,EAAE,CAAC;gBACvE,MAAM,MAAM,CAAC,oBAAoB,CAAC,KAAK,EAAE,UAAU,EAAE,OAAO,CAAC,QAAQ,EAAE;oBACrE,MAAM,EAAE,OAAO,CAAC,MAAM;iBACvB,CAAC,CAAC;YACL,CAAC;QACH,CAAC;IACH,CAAC;IAED,IAAI,SAAS,GAAG,KAAK,CAAC;IACtB,IAAI,cAAc,GAAG,qBAAqB,CAAC;IAC3C,IAAI,gBAAgB,EAAE,CAAC;QACrB,MAAM,QAAQ,GAAG,QAAQ,CAAC,GAAG,CAAC,gBAAgB,CAAC,CAAC;QAChD,IAAI,CAAC,QAAQ,EAAE,CAAC;YACd,qEAAqE;YACrE,yEAAyE;YACzE,sDAAsD;YACtD,MAAM,IAAI,KAAK,CACb,qBAAqB,gBAAgB,4BAA4B;gBAC/D,wCAAwC,gBAAgB,YAAY;gBACpE,2CAA2C,CAC9C,CAAC;QACJ,CAAC;QACD,IAAI,QAAQ,KAAK,qBAAqB,EAAE,CAAC;YACvC,MAAM,MAAM,CAAC,sBAAsB,CAAC,KAAK,EAAE,UAAU,EAAE,QAAQ,CAAC,CAAC;YACjE,GAAG,CAAC,yBAAyB,gBAAgB,GAAG,CAAC,CAAC;YAClD,SAAS,GAAG,IAAI,CAAC;QACnB,CAAC;QACD,cAAc,GAAG,QAAQ,CAAC;IAC5B,CAAC;IAED,OAAO,EAAE,SAAS,EAAE,QAAQ,EAAE,SAAS,EAAE,cAAc,EAAE,CAAC;AAC5D,CAAC"}

package/dist/src/lib/workflow-config-sidecar.d.ts

@@ -0,0 +1,63 @@
+/**
+ * Named workflow configs as files (issue #2645, spec §Contracts/Named configs).
+ *
+ * A workflow's named configurations used to reach the server only through
+ * `workflows configs create/update/duplicate/activate` — verbs, not files. That
+ * made the live body of a running workflow invisible to the repo and to review,
+ * which is precisely the failure #1976 records and the reason this issue exists.
+ * The design-gate decision was "TOML wins": each config body is a file at
+ * `workflows/<key>.configs/<name>.toml`, and `activeConfigName` in
+ * `workflows/<key>.toml` names the live one.
+ *
+ * The layout mirrors the `<key>.tests/` sidecar convention the sync tree
+ * already uses, so there is one directory idiom rather than two.
+ *
+ *   workflows/order-intake.toml            # the workflow, incl. activeConfigName
+ *   workflows/order-intake.configs/v2.toml # a named config body
+ *   workflows/order-intake.tests/*.toml    # test cases (pre-existing)
+ *
+ * File shape:
+ *
+ *   [config]
+ *   description = "second cut"
+ *   status = "active"
+ *
+ *   [[steps]]
+ *   id = "greet"
+ *   kind = "noop"
+ *
+ *   [expr.cel]
+ *   ready = "input.ok == true"
+ *
+ * The config's NAME is the file name and is deliberately not restated inside
+ * the file: a second declaration would be a second thing to change and a
+ * mismatch to adjudicate, which is the shape of problem this issue removes.
+ */
+/** Directory holding a workflow's named config bodies. */
+export declare function workflowConfigsDir(configDir: string, key: string): string;
+/** The config name a sidecar file declares — its base name. */
+export declare function workflowConfigNameFromFile(file: string): string;
+/** A named config body, as authored on disk. */
+export interface WorkflowConfigSidecar {
+    name: string;
+    description?: string;
+    status?: string;
+    /** Always an array once parsed: an empty body is a real, pushable state. */
+    steps: any[];
+    /** `[expr.cel]` named guards travelling with these steps (#1614). */
+    expressions?: {
+        cel: Record<string, string>;
+    };
+}
+/** Emit a server config record as its sidecar file body. */
+export declare function serializeWorkflowConfigSidecar(config: any): string;
+/** Read a parsed sidecar back into the body `config push` applies. */
+export declare function parseWorkflowConfigSidecar(name: string, tomlData: any): WorkflowConfigSidecar;
+/**
+ * Every named config authored for a workflow, in file-name order.
+ *
+ * An absent directory returns an empty list rather than throwing: a workflow
+ * with no sidecars is the ordinary single-config case, where the body is the
+ * `steps` in `workflows/<key>.toml`.
+ */
+export declare function readWorkflowConfigSidecars(configDir: string, key: string, readToml?: (path: string) => any): WorkflowConfigSidecar[];

package/dist/src/lib/workflow-config-sidecar.js

@@ -0,0 +1,96 @@
+/**
+ * Named workflow configs as files (issue #2645, spec §Contracts/Named configs).
+ *
+ * A workflow's named configurations used to reach the server only through
+ * `workflows configs create/update/duplicate/activate` — verbs, not files. That
+ * made the live body of a running workflow invisible to the repo and to review,
+ * which is precisely the failure #1976 records and the reason this issue exists.
+ * The design-gate decision was "TOML wins": each config body is a file at
+ * `workflows/<key>.configs/<name>.toml`, and `activeConfigName` in
+ * `workflows/<key>.toml` names the live one.
+ *
+ * The layout mirrors the `<key>.tests/` sidecar convention the sync tree
+ * already uses, so there is one directory idiom rather than two.
+ *
+ *   workflows/order-intake.toml            # the workflow, incl. activeConfigName
+ *   workflows/order-intake.configs/v2.toml # a named config body
+ *   workflows/order-intake.tests/*.toml    # test cases (pre-existing)
+ *
+ * File shape:
+ *
+ *   [config]
+ *   description = "second cut"
+ *   status = "active"
+ *
+ *   [[steps]]
+ *   id = "greet"
+ *   kind = "noop"
+ *
+ *   [expr.cel]
+ *   ready = "input.ok == true"
+ *
+ * The config's NAME is the file name and is deliberately not restated inside
+ * the file: a second declaration would be a second thing to change and a
+ * mismatch to adjudicate, which is the shape of problem this issue removes.
+ */
+import { existsSync, readdirSync, readFileSync } from "fs";
+import { basename, join } from "path";
+import { parseConfigToml, stringifyConfigToml } from "./config-toml.js";
+/** Directory holding a workflow's named config bodies. */
+export function workflowConfigsDir(configDir, key) {
+    return join(configDir, "workflows", `${key}.configs`);
+}
+/** The config name a sidecar file declares — its base name. */
+export function workflowConfigNameFromFile(file) {
+    return basename(file, ".toml");
+}
+/** Emit a server config record as its sidecar file body. */
+export function serializeWorkflowConfigSidecar(config) {
+    const cel = config?.expressions?.cel && typeof config.expressions.cel === "object"
+        ? config.expressions.cel
+        : undefined;
+    const data = {
+        config: {
+            description: config?.description || undefined,
+            status: config?.status || undefined,
+        },
+        steps: Array.isArray(config?.steps) ? config.steps : [],
+        ...(cel && Object.keys(cel).length > 0 ? { expr: { cel } } : {}),
+    };
+    for (const k of Object.keys(data.config)) {
+        if (data.config[k] === undefined)
+            delete data.config[k];
+    }
+    return stringifyConfigToml(data);
+}
+/** Read a parsed sidecar back into the body `config push` applies. */
+export function parseWorkflowConfigSidecar(name, tomlData) {
+    const config = tomlData?.config || {};
+    const cel = tomlData?.expr?.cel && typeof tomlData.expr.cel === "object"
+        ? tomlData.expr.cel
+        : undefined;
+    return {
+        name,
+        description: config.description ?? undefined,
+        status: config.status ?? undefined,
+        steps: Array.isArray(tomlData?.steps) ? tomlData.steps : [],
+        ...(cel ? { expressions: { cel } } : {}),
+    };
+}
+/**
+ * Every named config authored for a workflow, in file-name order.
+ *
+ * An absent directory returns an empty list rather than throwing: a workflow
+ * with no sidecars is the ordinary single-config case, where the body is the
+ * `steps` in `workflows/<key>.toml`.
+ */
+export function readWorkflowConfigSidecars(configDir, key, readToml = (p) => parseConfigToml(readFileSync(p, "utf-8"))) {
+    const dir = workflowConfigsDir(configDir, key);
+    if (!existsSync(dir))
+        return [];
+    return readdirSync(dir)
+        .filter((f) => f.endsWith(".toml"))
+        .sort()
+        .map((f) => parseWorkflowConfigSidecar(workflowConfigNameFromFile(f), readToml(join(dir, f))));
+}
+//# sourceMappingURL=workflow-config-sidecar.js.map

package/dist/src/lib/workflow-config-sidecar.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"workflow-config-sidecar.js","sourceRoot":"","sources":["../../../src/lib/workflow-config-sidecar.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAkCG;AAEH,OAAO,EAAE,UAAU,EAAE,WAAW,EAAE,YAAY,EAAE,MAAM,IAAI,CAAC;AAC3D,OAAO,EAAE,QAAQ,EAAE,IAAI,EAAE,MAAM,MAAM,CAAC;AACtC,OAAO,EAAE,eAAe,EAAE,mBAAmB,EAAE,MAAM,kBAAkB,CAAC;AAExE,0DAA0D;AAC1D,MAAM,UAAU,kBAAkB,CAAC,SAAiB,EAAE,GAAW;IAC/D,OAAO,IAAI,CAAC,SAAS,EAAE,WAAW,EAAE,GAAG,GAAG,UAAU,CAAC,CAAC;AACxD,CAAC;AAED,+DAA+D;AAC/D,MAAM,UAAU,0BAA0B,CAAC,IAAY;IACrD,OAAO,QAAQ,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC;AACjC,CAAC;AAaD,4DAA4D;AAC5D,MAAM,UAAU,8BAA8B,CAAC,MAAW;IACxD,MAAM,GAAG,GACP,MAAM,EAAE,WAAW,EAAE,GAAG,IAAI,OAAO,MAAM,CAAC,WAAW,CAAC,GAAG,KAAK,QAAQ;QACpE,CAAC,CAAC,MAAM,CAAC,WAAW,CAAC,GAAG;QACxB,CAAC,CAAC,SAAS,CAAC;IAChB,MAAM,IAAI,GAAQ;QAChB,MAAM,EAAE;YACN,WAAW,EAAE,MAAM,EAAE,WAAW,IAAI,SAAS;YAC7C,MAAM,EAAE,MAAM,EAAE,MAAM,IAAI,SAAS;SACpC;QACD,KAAK,EAAE,KAAK,CAAC,OAAO,CAAC,MAAM,EAAE,KAAK,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE;QACvD,GAAG,CAAC,GAAG,IAAI,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,EAAE,GAAG,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;KACjE,CAAC;IACF,KAAK,MAAM,CAAC,IAAI,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC;QACzC,IAAI,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,KAAK,SAAS;YAAE,OAAO,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC;IAC1D,CAAC;IACD,OAAO,mBAAmB,CAAC,IAAI,CAAC,CAAC;AACnC,CAAC;AAED,sEAAsE;AACtE,MAAM,UAAU,0BAA0B,CACxC,IAAY,EACZ,QAAa;IAEb,MAAM,MAAM,GAAG,QAAQ,EAAE,MAAM,IAAI,EAAE,CAAC;IACtC,MAAM,GAAG,GACP,QAAQ,EAAE,IAAI,EAAE,GAAG,IAAI,OAAO,QAAQ,CAAC,IAAI,CAAC,GAAG,KAAK,QAAQ;QAC1D,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,GAAG;QACnB,CAAC,CAAC,SAAS,CAAC;IAChB,OAAO;QACL,IAAI;QACJ,WAAW,EAAE,MAAM,CAAC,WAAW,IAAI,SAAS;QAC5C,MAAM,EAAE,MAAM,CAAC,MAAM,IAAI,SAAS;QAClC,KAAK,EAAE,KAAK,CAAC,OAAO,CAAC,QAAQ,EAAE,KAAK,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE;QAC3D,GAAG,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,WAAW,EAAE,EAAE,GAAG,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;KACzC,CAAC;AACJ,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,0BAA0B,CACxC,SAAiB,EACjB,GAAW,EACX,WAAkC,CAAC,CAAC,EAAE,EAAE,CACtC,eAAe,CAAC,YAAY,CAAC,CAAC,EAAE,OAAO,CAAC,CAAC;IAE3C,MAAM,GAAG,GAAG,kBAAkB,CAAC,SAAS,EAAE,GAAG,CAAC,CAAC;IAC/C,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC;QAAE,OAAO,EAAE,CAAC;IAChC,OAAO,WAAW,CAAC,GAAG,CAAC;SACpB,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC;SAClC,IAAI,EAAE;SACN,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CACT,0BAA0B,CAAC,0BAA0B,CAAC,CAAC,CAAC,EAAE,QAAQ,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,CAAC,CAAC,CAClF,CAAC;AACN,CAAC"}

package/dist/src/lib/workflow-defaults.d.ts

@@ -0,0 +1,29 @@
+/**
+ * The `[workflow]` fields whose model default the CLI has to know (#2743).
+ *
+ * Two call sites need the same map, and they must not drift:
+ *   - `buildWorkflowPayloadFromToml` (update mode) sends the CONCRETE default
+ *     for a field the TOML omits, so the push converges the server instead of
+ *     preserving the old value. It deliberately does NOT send `null` for these:
+ *     a server predating #2743 coerces a queue-limit `null` to `0`
+ *     (`Number(null)`), so an explicit default is the only value that converges
+ *     identically on both.
+ *   - `normalizeWorkflowTomlDefaults` (`config diff`) fills the same defaults into
+ *     the local side, so a hand-authored file that omits them compares equal to
+ *     a server holding the defaults — and unequal to one holding a real
+ *     override.
+ *
+ * These mirror `models.yaml` (`WorkflowDefinition`): `perUserMaxRunning=4`,
+ * `perUserMaxQueued=100`, `perAppMaxRunning=25`, `perAppMaxQueued=10000`,
+ * `queueTtlSeconds=43200`, `dequeueOrder="fifo"`, `requiresClientApply=true`,
+ * `syncCallable=false`.
+ *
+ * This is deliberately NOT a generic `models.yaml` default importer — the model
+ * carries defaults the serializer never writes. Only the serializer-emitted
+ * surface with a concrete default belongs here, because the remote side of a
+ * diff is hashed via `serializeWorkflow`. Emit-when-set fields whose GET value
+ * is null/empty when unset (`description`, `accessRule`, `runAs`,
+ * `capabilities`, the schemas, `lock`) have no default: push sends `null` for
+ * them and both sides of the diff simply omit the key.
+ */
+export declare const WORKFLOW_MODEL_DEFAULTS: Record<string, unknown>;

package/dist/src/lib/workflow-defaults.js

@@ -0,0 +1,41 @@
+/**
+ * The `[workflow]` fields whose model default the CLI has to know (#2743).
+ *
+ * Two call sites need the same map, and they must not drift:
+ *   - `buildWorkflowPayloadFromToml` (update mode) sends the CONCRETE default
+ *     for a field the TOML omits, so the push converges the server instead of
+ *     preserving the old value. It deliberately does NOT send `null` for these:
+ *     a server predating #2743 coerces a queue-limit `null` to `0`
+ *     (`Number(null)`), so an explicit default is the only value that converges
+ *     identically on both.
+ *   - `normalizeWorkflowTomlDefaults` (`config diff`) fills the same defaults into
+ *     the local side, so a hand-authored file that omits them compares equal to
+ *     a server holding the defaults — and unequal to one holding a real
+ *     override.
+ *
+ * These mirror `models.yaml` (`WorkflowDefinition`): `perUserMaxRunning=4`,
+ * `perUserMaxQueued=100`, `perAppMaxRunning=25`, `perAppMaxQueued=10000`,
+ * `queueTtlSeconds=43200`, `dequeueOrder="fifo"`, `requiresClientApply=true`,
+ * `syncCallable=false`.
+ *
+ * This is deliberately NOT a generic `models.yaml` default importer — the model
+ * carries defaults the serializer never writes. Only the serializer-emitted
+ * surface with a concrete default belongs here, because the remote side of a
+ * diff is hashed via `serializeWorkflow`. Emit-when-set fields whose GET value
+ * is null/empty when unset (`description`, `accessRule`, `runAs`,
+ * `capabilities`, the schemas, `lock`) have no default: push sends `null` for
+ * them and both sides of the diff simply omit the key.
+ */
+export const WORKFLOW_MODEL_DEFAULTS = {
+    // #2803 — no `status`. Availability is server-owned and off the config wire,
+    // so the file has no status line for a diff to normalize or a push to send.
+    perUserMaxRunning: 4,
+    perUserMaxQueued: 100,
+    perAppMaxRunning: 25,
+    perAppMaxQueued: 10000,
+    queueTtlSeconds: 43200,
+    dequeueOrder: "fifo",
+    requiresClientApply: true,
+    syncCallable: false,
+};
+//# sourceMappingURL=workflow-defaults.js.map

package/dist/src/lib/workflow-defaults.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"workflow-defaults.js","sourceRoot":"","sources":["../../../src/lib/workflow-defaults.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AACH,MAAM,CAAC,MAAM,uBAAuB,GAA4B;IAC9D,6EAA6E;IAC7E,4EAA4E;IAC5E,iBAAiB,EAAE,CAAC;IACpB,gBAAgB,EAAE,GAAG;IACrB,gBAAgB,EAAE,EAAE;IACpB,eAAe,EAAE,KAAK;IACtB,eAAe,EAAE,KAAK;IACtB,YAAY,EAAE,MAAM;IACpB,mBAAmB,EAAE,IAAI;IACzB,YAAY,EAAE,KAAK;CACpB,CAAC"}

package/dist/src/lib/workflow-fragments.d.ts

@@ -0,0 +1,64 @@
+/**
+ * CLI-only workflow fragment expansion.
+ *
+ * Workflows may declare `include = ["fragment-name", ...]` at the top of
+ * their TOML. The CLI expands those references into a fully-flattened
+ * `[[steps]]` list before push — the server never sees fragments and
+ * stores only the canonical, expanded JSON.
+ *
+ * Pinned decisions (#744):
+ *   - CLI-only. No server-side WorkflowFragment model.
+ *   - Fragments live at <workflowDir>/../workflow-fragments/<name>.toml.
+ *   - Fragment files are `[[steps]]` lists only — no `[workflow]` block.
+ *   - No recursive includes in v1 (fragments cannot themselves `include`).
+ *   - Unique step ids validated post-expansion; collisions name both
+ *     locations in the error message.
+ *
+ * Parameterized includes (#1448):
+ *   - Two include forms are supported. The bare form
+ *     `include = ["name", ...]` (an array of fragment-name strings) is
+ *     unchanged. The parameterized form uses `[[include]]` array-of-tables:
+ *
+ *       [[include]]
+ *       fragment = "lifecycle-email"          # required
+ *       runIf = "input.type == 'x'"           # optional; ANDed onto each step
+ *       with = { id = "...", subject = "..." } # params, exposed as {{ params.* }}
+ *
+ *   - A file uses one form or the other — mixing string and table entries in a
+ *     single `include` array is an error.
+ *   - `{{ params.* }}` references inside the fragment's steps are substituted
+ *     at EXPAND time from the include's `with` table. A whole-string
+ *     `{{ params.x }}` splices the raw typed value (number/bool/table);
+ *     an embedded `{{ params.x }}` inside a larger string stringifies it.
+ *     A missing param is a hard error. Only the `params.` namespace is
+ *     touched — every other `{{ ... }}` template passes through to the server
+ *     verbatim, and `params.` never survives into the expanded output.
+ *   - The include-level `runIf` (CEL) is ANDed onto every expanded step:
+ *     both present → `(<includeRunIf>) && (<stepRunIf>)`; one present → that
+ *     one; neither → no runIf.
+ *
+ * Wiring: `parseTomlFile()` in `cli/src/commands/sync.ts` calls
+ * `expandWorkflowTomlData()` after parsing. All 24 push parse sites in
+ * sync.ts route through that single seam, so the expansion is uniform.
+ * The standalone `expandWorkflow(filePath)` helper backs the
+ * `primitive workflows expand` subcommand for debugging.
+ */
+export interface ExpandedWorkflow {
+    workflow?: Record<string, any>;
+    steps: Array<Record<string, any>>;
+    include?: undefined;
+    [key: string]: any;
+}
+/**
+ * Read and expand a workflow TOML file from disk. Returns the parsed
+ * TOML with all `include` fragments spliced in. If the file has no
+ * `include` key, the parsed TOML is returned unchanged.
+ */
+export declare function expandWorkflow(workflowPath: string): ExpandedWorkflow;
+/**
+ * Expand the `include` key of an already-parsed TOML object. Resolves
+ * fragment files relative to `<workflowPath>/../../workflow-fragments/`.
+ * Use this when the TOML has already been parsed elsewhere (e.g. inside
+ * `parseTomlFile()`).
+ */
+export declare function expandWorkflowTomlData(parsed: Record<string, any>, workflowPath: string): ExpandedWorkflow;