@bluewombat/runtime 0.9.0 → 0.11.0

package/dist/config/home.d.ts

@@ -11,6 +11,8 @@ export declare const HOME_LAYOUT: {
     readonly workspaces: "workspaces";
     readonly ledger: "ledger";
     readonly workLine: "work-line";
+    readonly targetLine: "target-line";
+    readonly featureWorkspaces: "feature-workspaces";
 };
 export declare function homeOf(configDir: string, home: string | undefined): string;
 export declare function inHome(home: string, part: keyof typeof HOME_LAYOUT): string;

package/dist/config/home.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"home.d.ts","sourceRoot":"","sources":["../../src/config/home.ts"],"names":[],"mappings":"AAGA;;;;;;;GAOG;AACH,eAAO,MAAM,YAAY,aAAkB,CAAC;AAE5C,eAAO,MAAM,WAAW;;;;CAId,CAAC;AAEX,wBAAgB,MAAM,CAAC,SAAS,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,SAAS,GAAG,MAAM,CAE1E;AAED,wBAAgB,MAAM,CAAC,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,OAAO,WAAW,GAAG,MAAM,CAE3E"}
+{"version":3,"file":"home.d.ts","sourceRoot":"","sources":["../../src/config/home.ts"],"names":[],"mappings":"AAGA;;;;;;;GAOG;AACH,eAAO,MAAM,YAAY,aAAkB,CAAC;AAE5C,eAAO,MAAM,WAAW;;;;;;CAMd,CAAC;AAEX,wBAAgB,MAAM,CAAC,SAAS,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,SAAS,GAAG,MAAM,CAE1E;AAED,wBAAgB,MAAM,CAAC,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,OAAO,WAAW,GAAG,MAAM,CAE3E"}

package/dist/config/home.js

@@ -13,6 +13,8 @@ export const HOME_LAYOUT = {
     workspaces: "workspaces",
     ledger: "ledger",
     workLine: "work-line",
+    targetLine: "target-line",
+    featureWorkspaces: "feature-workspaces",
 };
 export function homeOf(configDir, home) {
     return resolve(configDir, home ?? DEFAULT_HOME);

package/dist/config/home.js.map

@@ -1 +1 @@
-{"version":3,"file":"home.js","sourceRoot":"","sources":["../../src/config/home.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAC1C,OAAO,EAAE,QAAQ,EAAE,MAAM,yBAAyB,CAAC;AAEnD;;;;;;;GAOG;AACH,MAAM,CAAC,MAAM,YAAY,GAAG,KAAK,QAAQ,EAAE,CAAC;AAE5C,MAAM,CAAC,MAAM,WAAW,GAAG;IACzB,UAAU,EAAE,YAAY;IACxB,MAAM,EAAE,QAAQ;IAChB,QAAQ,EAAE,WAAW;CACb,CAAC;AAEX,MAAM,UAAU,MAAM,CAAC,SAAiB,EAAE,IAAwB;IAChE,OAAO,OAAO,CAAC,SAAS,EAAE,IAAI,IAAI,YAAY,CAAC,CAAC;AAClD,CAAC;AAED,MAAM,UAAU,MAAM,CAAC,IAAY,EAAE,IAA8B;IACjE,OAAO,IAAI,CAAC,IAAI,EAAE,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC;AACvC,CAAC"}
+{"version":3,"file":"home.js","sourceRoot":"","sources":["../../src/config/home.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,IAAI,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAC1C,OAAO,EAAE,QAAQ,EAAE,MAAM,yBAAyB,CAAC;AAEnD;;;;;;;GAOG;AACH,MAAM,CAAC,MAAM,YAAY,GAAG,KAAK,QAAQ,EAAE,CAAC;AAE5C,MAAM,CAAC,MAAM,WAAW,GAAG;IACzB,UAAU,EAAE,YAAY;IACxB,MAAM,EAAE,QAAQ;IAChB,QAAQ,EAAE,WAAW;IACrB,UAAU,EAAE,aAAa;IACzB,iBAAiB,EAAE,oBAAoB;CAC/B,CAAC;AAEX,MAAM,UAAU,MAAM,CAAC,SAAiB,EAAE,IAAwB;IAChE,OAAO,OAAO,CAAC,SAAS,EAAE,IAAI,IAAI,YAAY,CAAC,CAAC;AAClD,CAAC;AAED,MAAM,UAAU,MAAM,CAAC,IAAY,EAAE,IAA8B;IACjE,OAAO,IAAI,CAAC,IAAI,EAAE,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC;AACvC,CAAC"}

package/dist/config/load-config.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"load-config.d.ts","sourceRoot":"","sources":["../../src/config/load-config.ts"],"names":[],"mappings":"AAWA,OAAO,KAAK,EAIV,SAAS,EAMV,MAAM,YAAY,CAAC;AA+BpB,MAAM,MAAM,YAAY,GAAG;IAAE,EAAE,EAAE,IAAI,CAAC;IAAC,UAAU,EAAE,SAAS,CAAA;CAAE,GAAG;IAAE,EAAE,EAAE,KAAK,CAAC;IAAC,MAAM,EAAE,MAAM,CAAA;CAAE,CAAC;AAE/F;;GAEG;AACH,wBAAgB,UAAU,CAAC,IAAI,EAAE,MAAM,GAAG,YAAY,CAiKrD"}
+{"version":3,"file":"load-config.d.ts","sourceRoot":"","sources":["../../src/config/load-config.ts"],"names":[],"mappings":"AAUA,OAAO,KAAK,EAIV,SAAS,EASV,MAAM,YAAY,CAAC;AAwDpB,MAAM,MAAM,YAAY,GAAG;IAAE,EAAE,EAAE,IAAI,CAAC;IAAC,UAAU,EAAE,SAAS,CAAA;CAAE,GAAG;IAAE,EAAE,EAAE,KAAK,CAAC;IAAC,MAAM,EAAE,MAAM,CAAA;CAAE,CAAC;AAE/F;;GAEG;AACH,wBAAgB,UAAU,CAAC,IAAI,EAAE,MAAM,GAAG,YAAY,CAwLrD"}

package/dist/config/load-config.js

@@ -2,28 +2,53 @@ import { existsSync, readFileSync, statSync } from "node:fs";
 import { dirname, resolve } from "node:path";
 import { parse as parseYaml } from "yaml";
 import { expandAgentPass, readAgents, SHORT_PASS_KEYS, } from "./agent-shorthand.js";
+import { SKIP_WHEN } from "./types.js";
 const KNOWN_KEYS = new Set([
-    "featureTracker",
-    "ticketTracker",
     "home",
     "workLine",
     "workspaceRoot",
     "ledger",
     "persist",
     "agents",
-    "builder",
-    "assembly",
+    "ticket",
+    "feature",
     "authority",
     "observability",
     "timeoutMs",
-    "maxRefusals",
     "pollIntervalMs",
 ]);
-const WORK_LINE_KEYS = new Set(["stable", "branch", "isolation", "isolationOptions", "warm"]);
+const WORK_LINE_KEYS = new Set([
+    "stable",
+    "targetStable",
+    "branch",
+    "isolation",
+    "isolationOptions",
+    "warm",
+]);
 const PASS_KEYS = new Set(["cmd", "timeoutMs"]);
 const GATED_PASS_KEYS = new Set(["cmd", "timeoutMs", "gates"]);
-const STAGE_KEYS = new Set(["producer", "repair", "maxAttempts"]);
-const ASSEMBLY_KEYS = new Set(["fix", "validate", "maxAttempts", "gates"]);
+const STAGE_KEYS = new Set(["producer", "repair", "gates", "maxAttempts"]);
+const TICKET_KEYS = new Set([
+    "tracker",
+    "build",
+    "maxParallel",
+    "maxAttempts",
+    "maxRefusals",
+    "fix",
+    "validate",
+    "gates",
+]);
+const FEATURE_KEYS = new Set([
+    "tracker",
+    "test",
+    "maxAttempts",
+    "maxRefusals",
+    "fix",
+    "validate",
+    "gates",
+]);
+/** Keys of a level that name what it builds or tracks, not how its result is judged. */
+const BUILDING_KEYS = new Set(["tracker", "build", "test"]);
 const GATES_KEYS = new Set(["defaultTimeoutMs", "gates"]);
 const AUTHORITY_KEYS = new Set(["enabled", "publish", "refresh", "describe"]);
 const OBSERVABILITY_KEYS = new Set(["streams"]);
@@ -61,8 +86,15 @@ export function loadConfig(path) {
     }
     const configDir = dirname(file);
     const invocation = { configDir };
-    for (const side of ["featureTracker", "ticketTracker"]) {
-        const spec = readManagerSpec(object[side], side);
+    for (const [level, side] of [
+        ["feature", "featureTracker"],
+        ["ticket", "ticketTracker"],
+    ]) {
+        const block = object[level];
+        const raw = block !== null && typeof block === "object" && !Array.isArray(block)
+            ? block.tracker
+            : undefined;
+        const spec = readManagerSpec(raw, `${level}.tracker`);
         if (spec.ok === false) {
             return spec;
         }
@@ -85,6 +117,9 @@ export function loadConfig(path) {
         if (workLine.stable !== undefined) {
             invocation.workLineStable = workLine.stable;
         }
+        if (workLine.targetStable !== undefined) {
+            invocation.workLineTargetStable = workLine.targetStable;
+        }
         if (workLine.branch !== undefined) {
             invocation.workLineBranch = workLine.branch;
         }
@@ -127,19 +162,35 @@ export function loadConfig(path) {
         }
         agents = read.value;
     }
-    if (object.builder !== undefined) {
-        const builder = readBuilder(object.builder, configDir, agents);
-        if (builder.ok === false) {
-            return builder;
+    if (object.ticket !== undefined) {
+        const ticket = readTicket(object.ticket, configDir, agents);
+        if (ticket.ok === false) {
+            return ticket;
+        }
+        if (ticket.builder !== undefined) {
+            invocation.builder = ticket.builder;
+        }
+        if (ticket.value !== undefined) {
+            invocation.ticket = ticket.value;
         }
-        invocation.builder = builder.value;
     }
-    if (object.assembly !== undefined) {
-        const assembly = readAssembly(object.assembly, configDir, agents);
-        if (assembly.ok === false) {
-            return assembly;
+    if (object.feature !== undefined) {
+        const feature = readFeature(object.feature, configDir, agents);
+        if (feature.ok === false) {
+            return feature;
+        }
+        if (feature.test !== undefined) {
+            if (invocation.builder === undefined) {
+                return {
+                    ok: false,
+                    reason: 'Config "feature.test" needs "ticket.build": the Ticket that holds the suite is made like any other.',
+                };
+            }
+            invocation.builder.featureTest = feature.test;
+        }
+        if (feature.value !== undefined) {
+            invocation.feature = feature.value;
         }
-        invocation.assembly = assembly.value;
     }
     if (object.authority !== undefined) {
         const authority = readAuthority(object.authority, configDir, agents);
@@ -162,15 +213,6 @@ export function loadConfig(path) {
     if (timeoutMs.value !== undefined) {
         invocation.timeoutMs = timeoutMs.value;
     }
-    for (const key of ["maxRefusals"]) {
-        const read = positiveIntField(object, key);
-        if (read.ok === false) {
-            return read;
-        }
-        if (read.value !== undefined) {
-            invocation[key] = read.value;
-        }
-    }
     const pollIntervalMs = positiveIntField(object, "pollIntervalMs");
     if (pollIntervalMs.ok === false) {
         return pollIntervalMs;
@@ -178,8 +220,47 @@ export function loadConfig(path) {
     if (pollIntervalMs.value !== undefined) {
         invocation.pollIntervalMs = pollIntervalMs.value;
     }
+    const featureTestRefusal = checkFeatureTestDeclaration(invocation);
+    if (featureTestRefusal !== undefined) {
+        return { ok: false, reason: featureTestRefusal };
+    }
     return { ok: true, invocation };
 }
+/**
+ * The feature-test check is declared by a pair: `feature.test`, which makes the Ticket that holds the
+ * suite, and a Feature Gate with `skipWhen: skip-feature-test`, which runs it. One without the other
+ * is a check that never runs or a Ticket nobody judges, so neither is accepted alone. A `skipWhen`
+ * outside a Feature's Gates has nothing to read.
+ */
+function checkFeatureTestDeclaration(invocation) {
+    const outside = [
+        ["ticket.build.gates", invocation.builder?.producer.gates],
+        ["feature.test.gates", invocation.builder?.featureTest?.gates],
+        ["ticket.gates", invocation.ticket?.gates],
+        ["ticket.fix.gates", invocation.ticket?.fix?.gates],
+        ["ticket.validate.gates", invocation.ticket?.validate?.gates],
+    ];
+    for (const [key, gates] of outside) {
+        const bad = gates?.find((gate) => gate.skipWhen !== undefined);
+        if (bad !== undefined) {
+            return `Config "${key}" gate "${bad.id}" has "skipWhen", which only a Feature's gates (feature.validate, feature.gates, feature.fix) can have.`;
+        }
+    }
+    const feature = invocation.feature;
+    const skippable = [
+        ...(feature?.validate.gates ?? []),
+        ...(feature?.gates ?? []),
+        ...(feature?.fix.gates ?? []),
+    ].filter((gate) => gate.skipWhen === "skip-feature-test");
+    const declared = invocation.builder?.featureTest !== undefined;
+    if (skippable.length > 0 && !declared) {
+        return `Config gate "${skippable[0]?.id}" has "skipWhen: skip-feature-test", so the feature-test check is declared: "feature.test" is required (the pass that makes the Ticket holding the suite).`;
+    }
+    if (declared && feature !== undefined && skippable.length === 0) {
+        return 'Config "feature.test" is declared, but no Feature gate runs the suite. Add a gate to "feature.validate.gates" with "skipWhen: skip-feature-test".';
+    }
+    return undefined;
+}
 const MANAGER_SPEC_KEYS = new Set(["package", "options"]);
 function readManagerSpec(value, side) {
     if (value === undefined) {
@@ -257,6 +338,10 @@ function readWorkLine(value, configDir) {
     if (stable.ok === false) {
         return { ok: false, reason: `Config "workLine.stable" must be a non-empty string.` };
     }
+    const targetStable = stringField(object, "targetStable");
+    if (targetStable.ok === false) {
+        return { ok: false, reason: `Config "workLine.targetStable" must be a non-empty string.` };
+    }
     const branch = stringField(object, "branch");
     if (branch.ok === false) {
         return { ok: false, reason: `Config "workLine.branch" must be a non-empty string.` };
@@ -286,6 +371,9 @@ function readWorkLine(value, configDir) {
     return {
         ok: true,
         ...(stable.value === undefined ? {} : { stable: resolve(configDir, stable.value) }),
+        ...(targetStable.value === undefined
+            ? {}
+            : { targetStable: resolve(configDir, targetStable.value) }),
         ...(branch.value === undefined ? {} : { branch: branch.value }),
         ...(isolation.value === undefined ? {} : { isolation: isolation.value }),
         ...(isolationOptions === undefined
@@ -457,8 +545,9 @@ function readGates(value, configDir, key) {
     if (fallback.ok === false) {
         return { ok: false, reason: `Config "${key}.defaultTimeoutMs" must be a positive integer.` };
     }
+    // A bare `gates:` in YAML is a null: an empty list, written the short way.
     const entries = object.gates;
-    if (entries === undefined) {
+    if (entries === undefined || entries === null) {
         return { ok: true, value: [] };
     }
     if (!Array.isArray(entries)) {
@@ -496,7 +585,19 @@ function readGates(value, configDir, key) {
                 reason: `Config gate "${id}" has no "timeoutMs", and "${key}" names no "defaultTimeoutMs".`,
             };
         }
-        gates.push({ id, argv: resolveArgv(argv, configDir), timeoutMs });
+        const skipWhen = entry.skipWhen;
+        if (skipWhen !== undefined && !SKIP_WHEN.includes(skipWhen)) {
+            return {
+                ok: false,
+                reason: `Config gate "${id}" has "skipWhen: ${String(skipWhen)}". Known: ${SKIP_WHEN.join(", ")}.`,
+            };
+        }
+        gates.push({
+            id,
+            argv: resolveArgv(argv, configDir),
+            timeoutMs,
+            ...(skipWhen === undefined ? {} : { skipWhen: skipWhen }),
+        });
     }
     return { ok: true, value: gates };
 }
@@ -505,7 +606,7 @@ function readGates(value, configDir, key) {
  *
  * `timeoutMs` is required: without it nothing bounds an agent, and an agent
  * that never returns is the failure this shape exists to prevent. `repair` and
- * `assembly` inherit both the command and the ceiling, so a Project with one
+ * `ticket` inherit both the command and the ceiling, so a Project with one
  * agent writes one command.
  */
 function readPass(value, configDir, key) {
@@ -538,12 +639,13 @@ function readPass(value, configDir, key) {
     };
 }
 /**
- * A command, and its own Gate sequence — `builder.producer`,
- * `builder.repair`, `assembly.fix`, `assembly.validate`. Deliberately its own
- * ~10 lines rather than a parameterized `readPass`: each reader's error
- * messages and key set stay independently legible.
+ * A command, and its own Gate sequence — `ticket.fix`, `ticket.validate`.
+ * `ticket.build.producer` and `ticket.build.repair` read the same shape with `gated`
+ * false: their Gates are the one `ticket.build.gates` list, read by `readBuilder`.
+ * Deliberately its own ~10 lines rather than a parameterized `readPass`: each
+ * reader's error messages and key set stay independently legible.
  */
-function readGatedPass(value, configDir, key, agents) {
+function readGatedPass(value, configDir, key, agents, gated = true) {
     if (value === null || typeof value !== "object" || Array.isArray(value)) {
         return {
             ok: false,
@@ -553,6 +655,13 @@ function readGatedPass(value, configDir, key, agents) {
     const object = value;
     const short = object.agent !== undefined;
     for (const name of Object.keys(object)) {
+        if (!gated && name === "gates") {
+            return {
+                ok: false,
+                reason: `Config "${key}" has no "gates". The builder's Gates are "ticket.build.gates": ` +
+                    "one list, judging the producer and the repair alike.",
+            };
+        }
         const known = GATED_PASS_KEYS.has(name) || (short && SHORT_PASS_KEYS.has(name));
         if (!known) {
             return { ok: false, reason: `Config "${key}" has unknown key "${name}".` };
@@ -593,40 +702,84 @@ function readGatedPass(value, configDir, key, agents) {
     }
     return { ok: true, value: { cmd, timeoutMs: timeoutMs.value, gates } };
 }
+/**
+ * A review: an agent and its Gates, or `agent: false` and the Gates alone.
+ * The agent is turned off by saying so; a review with neither an agent nor a
+ * Gate would judge nothing, and is refused.
+ */
+function readReview(value, configDir, key, agents) {
+    if (value === null || typeof value !== "object" || Array.isArray(value)) {
+        return {
+            ok: false,
+            reason: `Config "${key}" must be a YAML mapping with an agent (or "cmd") and "timeoutMs", or "agent: false" and "gates".`,
+        };
+    }
+    const object = value;
+    if (object.agent !== false) {
+        return readGatedPass(value, configDir, key, agents);
+    }
+    for (const name of Object.keys(object)) {
+        if (name !== "agent" && name !== "gates") {
+            return {
+                ok: false,
+                reason: `Config "${key}" has "agent: false", so it has no "${name}": only its "gates" run.`,
+            };
+        }
+    }
+    // No Gates either: the review is off, which is how a Project says it.
+    if (object.gates === undefined || object.gates === null) {
+        return { ok: true, value: { agent: false, gates: [] } };
+    }
+    const gates = readGates(object.gates, configDir, `${key}.gates`);
+    if (gates.ok === false) {
+        return gates;
+    }
+    return { ok: true, value: { agent: false, gates: gates.value } };
+}
 /**
  * Making the unit of work: both passes named, neither borrowing the other's command.
  */
 function readBuilder(value, configDir, agents) {
     if (value === null || typeof value !== "object" || Array.isArray(value)) {
-        return { ok: false, reason: 'Config "builder" must be a YAML mapping.' };
+        return { ok: false, reason: 'Config "ticket.build" must be a YAML mapping.' };
     }
     const object = value;
     for (const name of Object.keys(object)) {
         if (!STAGE_KEYS.has(name)) {
-            return { ok: false, reason: `Config "builder" has unknown key "${name}".` };
+            return { ok: false, reason: `Config "ticket.build" has unknown key "${name}".` };
         }
     }
     if (object.producer === undefined) {
-        return { ok: false, reason: 'Config "builder" needs a "producer".' };
+        return { ok: false, reason: 'Config "ticket.build" needs a "producer".' };
     }
     if (object.repair === undefined) {
-        return { ok: false, reason: 'Config "builder" needs a "repair".' };
+        return { ok: false, reason: 'Config "ticket.build" needs a "repair".' };
     }
-    const readProducer = readGatedPass(object.producer, configDir, "builder.producer", agents);
+    const readProducer = readGatedPass(object.producer, configDir, "ticket.build.producer", agents, false);
     if (readProducer.ok === false) {
         return readProducer;
     }
-    const readRepair = readGatedPass(object.repair, configDir, "builder.repair", agents);
+    const readRepair = readGatedPass(object.repair, configDir, "ticket.build.repair", agents, false);
     if (readRepair.ok === false) {
         return readRepair;
     }
+    let gates = [];
+    if (object.gates !== undefined) {
+        const read = readGates(object.gates, configDir, "ticket.build.gates");
+        if (read.ok === false) {
+            return read;
+        }
+        gates = read.value;
+    }
+    // One list, two passes: a repair is judged by exactly what judged the producer,
+    // so it cannot undo what a Gate refused. Implementer still takes two sequences.
     const spec = {
-        producer: readProducer.value,
-        repair: readRepair.value,
+        producer: { ...readProducer.value, gates },
+        repair: { ...readRepair.value, gates: [...gates] },
     };
     const maxAttempts = positiveIntField(object, "maxAttempts");
     if (maxAttempts.ok === false) {
-        return { ok: false, reason: 'Config "builder.maxAttempts" must be a positive integer.' };
+        return { ok: false, reason: 'Config "ticket.build.maxAttempts" must be a positive integer.' };
     }
     if (maxAttempts.value !== undefined) {
         spec.maxAttempts = maxAttempts.value;
@@ -637,53 +790,139 @@ function readBuilder(value, configDir, agents) {
  * Judging the assembled ticket. A command here is a fix of what a judgement
  * refused, not a second first-pass of the request.
  */
-function readAssembly(value, configDir, agents) {
+function readTicket(value, configDir, agents) {
     if (value === null || typeof value !== "object" || Array.isArray(value)) {
-        return { ok: false, reason: 'Config "assembly" must be a YAML mapping.' };
+        return { ok: false, reason: 'Config "ticket" must be a YAML mapping.' };
     }
     const object = value;
     for (const name of Object.keys(object)) {
         if (name === "producer" || name === "repair") {
             return {
                 ok: false,
-                reason: `Config "assembly" has no "${name}". The assembled ticket is judged ` +
-                    '("assembly.gates"); a refusal is "assembly.fix".',
+                reason: `Config "ticket" has no "${name}". The assembled ticket is judged ` +
+                    '("ticket.gates"); a refusal is "ticket.fix".',
             };
         }
-        if (!ASSEMBLY_KEYS.has(name)) {
-            return { ok: false, reason: `Config "assembly" has unknown key "${name}".` };
+        if (!TICKET_KEYS.has(name)) {
+            return { ok: false, reason: `Config "ticket" has unknown key "${name}".` };
+        }
+    }
+    let builder;
+    if (object.build !== undefined) {
+        const read = readBuilder(object.build, configDir, agents);
+        if (read.ok === false) {
+            return read;
         }
+        builder = read.value;
+    }
+    if (Object.keys(object).every((name) => BUILDING_KEYS.has(name))) {
+        return { ok: true, ...(builder === undefined ? {} : { builder }) };
+    }
+    if (object.validate === undefined) {
+        return {
+            ok: false,
+            reason: 'Config "ticket" needs a "validate": the review of the assembled ticket. To turn it off, write "validate: { agent: false }".',
+        };
     }
     const spec = { gates: [] };
     if (object.fix !== undefined) {
-        const read = readGatedPass(object.fix, configDir, "assembly.fix", agents);
+        const read = readGatedPass(object.fix, configDir, "ticket.fix", agents);
         if (read.ok === false) {
             return read;
         }
         spec.fix = read.value;
     }
     if (object.validate !== undefined) {
-        const read = readGatedPass(object.validate, configDir, "assembly.validate", agents);
+        const read = readReview(object.validate, configDir, "ticket.validate", agents);
         if (read.ok === false) {
             return read;
         }
         spec.validate = read.value;
     }
-    const maxAttempts = positiveIntField(object, "maxAttempts");
-    if (maxAttempts.ok === false) {
-        return { ok: false, reason: 'Config "assembly.maxAttempts" must be a positive integer.' };
-    }
-    if (maxAttempts.value !== undefined) {
-        spec.maxAttempts = maxAttempts.value;
+    for (const key of ["maxParallel", "maxAttempts", "maxRefusals"]) {
+        const read = positiveIntField(object, key);
+        if (read.ok === false) {
+            return { ok: false, reason: `Config "ticket.${key}" must be a positive integer.` };
+        }
+        if (read.value !== undefined) {
+            spec[key] = read.value;
+        }
     }
     if (object.gates !== undefined) {
-        const gates = readGates(object.gates, configDir, "assembly.gates");
+        const gates = readGates(object.gates, configDir, "ticket.gates");
         if (gates.ok === false) {
             return gates;
         }
         spec.gates = gates.value;
     }
-    return { ok: true, value: spec };
+    return { ok: true, value: spec, ...(builder === undefined ? {} : { builder }) };
+}
+/**
+ * Judging a Feature's whole line, and repairing what that judgement refused.
+ * Its own gates, fix and budget: nothing is shared with a Ticket's.
+ */
+function readFeature(value, configDir, agents) {
+    if (value === null || typeof value !== "object" || Array.isArray(value)) {
+        return { ok: false, reason: 'Config "feature" must be a YAML mapping.' };
+    }
+    const object = value;
+    for (const name of Object.keys(object)) {
+        if (!FEATURE_KEYS.has(name)) {
+            return { ok: false, reason: `Config "feature" has unknown key "${name}".` };
+        }
+    }
+    let test;
+    if (object.test !== undefined) {
+        const read = readGatedPass(object.test, configDir, "feature.test", agents);
+        if (read.ok === false) {
+            return read;
+        }
+        test = read.value;
+    }
+    if (Object.keys(object).every((name) => BUILDING_KEYS.has(name))) {
+        return { ok: true, ...(test === undefined ? {} : { test }) };
+    }
+    if (object.fix === undefined) {
+        return {
+            ok: false,
+            reason: 'Config "feature" needs a "fix": what repairs the line when its gates refuse it. There is no feature without one.',
+        };
+    }
+    if (object.validate === undefined) {
+        return {
+            ok: false,
+            reason: 'Config "feature" needs a "validate": the review of the whole line against the Feature\'s request, whose gates run the project\'s own checks on it.',
+        };
+    }
+    if (object.gates === undefined) {
+        return {
+            ok: false,
+            reason: 'Config "feature" needs "gates": what judges the published line before it is folded (the CI). They are not borrowed from "ticket".',
+        };
+    }
+    const validate = readReview(object.validate, configDir, "feature.validate", agents);
+    if (validate.ok === false) {
+        return validate;
+    }
+    const fix = readGatedPass(object.fix, configDir, "feature.fix", agents);
+    if (fix.ok === false) {
+        return fix;
+    }
+    const gates = readGates(object.gates, configDir, "feature.gates");
+    if (gates.ok === false) {
+        return gates;
+    }
+    const spec = { fix: fix.value, validate: validate.value, gates: gates.value };
+    for (const key of ["maxAttempts", "maxRefusals"]) {
+        const read = positiveIntField(object, key);
+        if (read.ok === false) {
+            return { ok: false, reason: `Config "feature.${key}" must be a positive integer.` };
+        }
+        if (read.value !== undefined) {
+            spec[key] = read.value;
+        }
+    }
+    return { ok: true, value: spec, ...(test === undefined ? {} : { test }) };
 }
 /**
  * Flags whose value is a path this system writes to.