mandrel 2.59.0 → 2.60.0

package/.agents/scripts/check-workflow-citations.js

@@ -1,5 +1,5 @@
 /**
- * CLI: ratchet-down gate on provenance citations in workflow prose.
+ * CLI: provenance-citation **report** for workflow prose.
  *
  * Workflow documents ride resident in an executing agent's context. A
  * `(Story #1234)` aside costs the same tokens as instruction and teaches
@@ -7,28 +7,24 @@
  * citations that survive are the ones a reader must follow to act; the
  * rest belong in the commit trail and `docs/decisions.md`.
  *
- * Nothing stops the tax re-accumulating one well-meaning aside at a time,
- * so this gate holds the line: count every issue-shaped reference across
- * `.agents/workflows/**\/*.md` and fail when the total rises above the
- * committed baseline at `baselines/workflow-citations.json`.
+ * This command counts every issue-shaped reference across
+ * `.agents/workflows/**\/*.md` and prints the per-file tally. It **always
+ * exits 0**: Story #5340 demoted it from a ratchet to a report and deleted
+ * `baselines/workflow-citations.json` with it. The ratchet failed a rise
+ * above a committed total, so a prose fix that added one pointer had to be
+ * paid for with an unrelated trim in the same commit — and that is how three
+ * reference sections came to describe mechanisms the code had already
+ * retired. The count is still worth seeing on every change, so it stays a
+ * report. See `docs/decisions.md`, ADR 20260917-5340.
  *
  * The counted token is the bare `#NNNN` form rather than the
  * `(Story|Epic|issue|refs) #NNNN` phrasing, because the same citation is
- * written both ways — `(Story #4593)` and a bare `(#4593)` — and a gate
- * that only saw the prefixed form would wave the other one through.
- *
- * Ratchet semantics mirror `check-arch-cycles.js`:
- *   - Total above the baseline → exit 1, with the files that grew.
- *   - Total below the baseline → printed as shrinkage, exit 0. Refresh the
- *     baseline with `--update` to lock the reduction in.
- *   - Equal → exit 0.
+ * written both ways — `(Story #4593)` and a bare `(#4593)` — and a report
+ * that only saw the prefixed form would undercount the other one.
  *
  * Flags:
- *   --baseline <path>  override the baseline path (default
- *                      `baselines/workflow-citations.json`, resolved from cwd)
  *   --root <path>      scan a different workflow root (default
  *                      `.agents/workflows`)
- *   --update           rewrite the baseline from the live count
  *   --json             write the structured envelope to stdout
  */
 
@@ -40,9 +36,6 @@ import { runAsCli } from './lib/cli-utils.js';
 /** Default workflow root, relative to the repository root. */
 const DEFAULT_ROOT = path.join('.agents', 'workflows');
 
-/** Default baseline path, relative to the repository root. */
-const DEFAULT_BASELINE = path.join('baselines', 'workflow-citations.json');
-
 /**
  * Issue-shaped reference: a `#` followed by 3–5 digits. Narrow enough to
  * skip markdown headings and anchor links, wide enough to catch both the
@@ -54,27 +47,16 @@ const CITATION_RE = /#\d{3,5}\b/g;
  * Parse argv. Exported so unit tests can pin the parser.
  *
  * @param {string[]} argv
- * @returns {{ baselinePath: string | null, rootPath: string | null, update: boolean, json: boolean }}
+ * @returns {{ rootPath: string | null, json: boolean }}
  */
 export function parseArgv(argv = []) {
-  const out = {
-    baselinePath: null,
-    rootPath: null,
-    update: false,
-    json: false,
-  };
+  const out = { rootPath: null, json: false };
   for (let i = 0; i < argv.length; i += 1) {
     const flag = argv[i];
     const next = argv[i + 1];
-    const takesValue = next && !next.startsWith('--');
-    if (flag === '--baseline' && takesValue) {
-      out.baselinePath = next;
-      i += 1;
-    } else if (flag === '--root' && takesValue) {
+    if (flag === '--root' && next && !next.startsWith('--')) {
       out.rootPath = next;
       i += 1;
-    } else if (flag === '--update') {
-      out.update = true;
     } else if (flag === '--json') {
       out.json = true;
     }
@@ -121,7 +103,7 @@ export function countCitations(source) {
 /**
  * Count citations across a file set, relativizing paths against `cwd` so
  * the report serializes identically on every platform. Files with zero
- * citations are omitted — the baseline records where the tax lives.
+ * citations are omitted — the report names where the tax lives.
  *
  * @param {string[]} files absolute paths
  * @param {string} cwd
@@ -150,72 +132,17 @@ export function tallyCitations(files, cwd, { readFile } = {}) {
 }
 
 /**
- * Pure helper: read the baseline envelope. Returns `null` when the file is
- * missing or unparseable — the baseline is not distributed to consumers, so
- * its absence degrades to "no ceiling" rather than a false failure.
- *
- * @param {string} baselinePath
- * @returns {{ total?: number, files?: Array<{ path: string, count: number }> } | null}
- */
-export function loadBaseline(baselinePath) {
-  try {
-    const parsed = JSON.parse(fs.readFileSync(baselinePath, 'utf-8'));
-    return parsed && typeof parsed === 'object' ? parsed : null;
-  } catch {
-    return null;
-  }
-}
-
-/**
- * Pure helper: diff the live tally against the baseline. `grew` names the
- * files whose per-file count rose, so a failure points at the edit rather
- * than only at the total.
+ * Pure helper: render the human-readable report — one line per file that
+ * carries a citation, then the total. No `+` / `-` ratchet vocabulary: there
+ * is nothing to regress against.
  *
- * @param {{ total?: number, files?: Array<{ path: string, count: number }> } | null} baseline
  * @param {{ total: number, files: Array<{ path: string, count: number }> }} tally
- * @returns {{ baselineTotal: number | null, delta: number | null, grew: Array<{ path: string, from: number, to: number }> }}
- */
-export function diffTally(baseline, tally) {
-  const baselineTotal =
-    typeof baseline?.total === 'number' ? baseline.total : null;
-  const prior = new Map(
-    (baseline?.files ?? []).map((row) => [row.path, row.count]),
-  );
-  const grew = tally.files
-    .map((row) => ({
-      path: row.path,
-      from: prior.get(row.path) ?? 0,
-      to: row.count,
-    }))
-    .filter((row) => row.to > row.from);
-  return {
-    baselineTotal,
-    delta: baselineTotal === null ? null : tally.total - baselineTotal,
-    grew,
-  };
-}
-
-/**
- * Pure helper: render the human-readable diff, mirroring the `+` / `-`
- * vocabulary of the sibling ratchets.
- *
- * @param {{ baselineTotal: number | null, delta: number | null, grew: Array<{ path: string, from: number, to: number }> }} diff
- * @param {{ total: number }} tally
  * @returns {string}
  */
-export function renderDiff(diff, tally) {
-  const lines = [];
-  for (const row of diff.grew) {
-    lines.push(`+ ${row.path}: ${row.from} -> ${row.to}`);
-  }
-  if (diff.delta !== null && diff.delta < 0) {
-    lines.push(
-      `[workflow-citations] ⚠ ${-diff.delta} citation(s) below baseline — refresh with --update to lock the reduction in`,
-    );
-  }
-  const tag = diff.delta !== null && diff.delta > 0 ? '(gate fail)' : '(ok)';
+export function renderReport(tally) {
+  const lines = tally.files.map((row) => `  ${row.path}: ${row.count}`);
   lines.push(
-    `[workflow-citations] total=${tally.total} baseline=${diff.baselineTotal ?? 'none'} ${tag}`,
+    `[workflow-citations] total=${tally.total} across ${tally.files.length} file(s) — report only, never gated`,
   );
   return lines.join('\n');
 }
@@ -230,77 +157,41 @@ export function renderDiff(diff, tally) {
  *   stdout?: { write: (s: string) => void },
  *   stderr?: { write: (s: string) => void },
  * }} [opts]
- * @returns {Promise<number>} 0 = at or below baseline; 1 = regression
+ * @returns {Promise<number>} always 0 — this is a report, not a gate
  */
 export async function runCli({
   argv = process.argv.slice(2),
   cwd = process.cwd(),
   stdout = process.stdout,
-  stderr = process.stderr,
 } = {}) {
-  const { baselinePath, rootPath, update, json } = parseArgv(argv);
+  const { rootPath, json } = parseArgv(argv);
   const root = path.resolve(cwd, rootPath ?? DEFAULT_ROOT);
-  const resolvedBaselinePath = path.resolve(
-    cwd,
-    baselinePath ?? DEFAULT_BASELINE,
-  );
   if (!fs.existsSync(root)) {
     throw new Error(`[workflow-citations] workflow root not found: ${root}`);
   }
 
   const tally = tallyCitations(collectMarkdownFiles(root), cwd);
 
-  if (update) {
-    const envelope = {
-      $schema: 'https://mandrel.dev/baselines/workflow-citations.schema.json',
-      generatedAt: new Date().toISOString(),
-      total: tally.total,
-      files: tally.files,
-    };
-    fs.mkdirSync(path.dirname(resolvedBaselinePath), { recursive: true });
-    fs.writeFileSync(
-      resolvedBaselinePath,
-      `${JSON.stringify(envelope, null, 2)}\n`,
-    );
-    stdout.write(
-      `[workflow-citations] baseline written: ${resolvedBaselinePath} (total=${tally.total})\n`,
-    );
-    return 0;
-  }
-
-  const baseline = loadBaseline(resolvedBaselinePath);
-  const diff = diffTally(baseline, tally);
-  const exitCode = diff.delta !== null && diff.delta > 0 ? 1 : 0;
-
   if (json) {
     stdout.write(
       `${JSON.stringify(
         {
           kind: 'workflow-citations-report',
           root,
-          baselinePath: resolvedBaselinePath,
           total: tally.total,
-          baselineTotal: diff.baselineTotal,
-          delta: diff.delta,
           files: tally.files,
-          grew: diff.grew,
-          exitCode,
+          exitCode: 0,
         },
         null,
         2,
       )}\n`,
     );
-    return exitCode;
+    return 0;
   }
 
-  if (!baseline) {
-    stderr.write(
-      `[workflow-citations] ⚠ baseline not found at ${resolvedBaselinePath} — no ceiling enforced\n`,
-    );
-  }
-  stdout.write(`\n--- workflow-citations preview ---\n`);
-  stdout.write(`${renderDiff(diff, tally)}\n`);
-  return exitCode;
+  stdout.write(`\n--- workflow-citations report ---\n`);
+  stdout.write(`${renderReport(tally)}\n`);
+  return 0;
 }
 
 async function main() {
@@ -313,20 +204,15 @@ runAsCli(import.meta.url, main, {
   errorPrefix: '[workflow-citations] ❌ Fatal error',
   usage: {
     invocation:
-      'node .agents/scripts/check-workflow-citations.js [--baseline <path>] [--root <dir>] [--update] [--json]',
+      'node .agents/scripts/check-workflow-citations.js [--root <dir>] [--json]',
     summary:
-      'Ratchet on provenance citations in workflow prose: count issue-shaped references across .agents/workflows/** and fail when the total rises above the recorded baseline.',
+      'Report provenance citations in workflow prose: count issue-shaped references across .agents/workflows/** and print the per-file tally. Never fails.',
     flags: [
-      [
-        '--baseline <path>',
-        'Baseline file (default: baselines/workflow-citations.json).',
-      ],
       ['--root <dir>', 'Workflow root to scan (default: .agents/workflows).'],
-      ['--update', 'Rewrite the baseline from the live count.'],
-      ['--json', 'Emit the comparison envelope as JSON.'],
+      ['--json', 'Emit the report envelope as JSON.'],
     ],
     notes: [
-      'Exit codes:\n  0  at or below the baseline\n  1  the citation total regressed above the baseline',
+      'Exit codes:\n  0  always — this is a report, not a gate (Story #5340)',
     ],
   },
 });

package/.agents/scripts/coverage-capture.js

@@ -31,8 +31,8 @@
  *
  * Step 3 is preceded by the changed-file skip when
  * `delivery.quality.gates.crap.incrementalCoverage.skipWhenUnchanged` is on
- * (the default): no changed file under `crap.targetDirs` versus `baseRef`
- * means no capture at all.
+ * (the default): no changed file under `crap.targetDirs` versus the ref
+ * `resolveChangedFilesRef` resolves means no capture at all (Story #5365).
  *
  * Exit codes:
  *   0 — coverage is fresh (or capture skipped/succeeded).
@@ -64,15 +64,18 @@ import { hasNpmScript, readPackageScripts } from './lib/npm-scripts.js';
 /**
  * Parse the full `process.argv` (index 2 onward) into the capture options.
  *
+ * A `ref` of `null` means the caller named none; `resolveChangedFilesRef` owns
+ * the fallback (Story #5365).
+ *
  * @param {string[]} argv
- * @returns {{ skipWhenNoCrapFiles: boolean, requireCredited: boolean, ref: string, cwd: string }}
+ * @returns {{ skipWhenNoCrapFiles: boolean, requireCredited: boolean, ref: string | null, cwd: string }}
  */
 export function parseArgs(argv) {
   const out = {
     skipWhenNoCrapFiles: false,
     // Story #5278 — an ARGUMENT, never a config read. See `runCoverageCapture`.
     requireCredited: false,
-    ref: 'main',
+    ref: null,
     cwd: process.cwd(),
   };
   for (let i = 2; i < argv.length; i += 1) {

package/.agents/scripts/deliver-light.js

@@ -20,35 +20,35 @@
  *
  * Two modes:
  *
- *   - **gate** (default) — judge a prompt's predicted footprint. On
+ *   - **gate** (default) — judge a prompt's predicted footprint for RISK. On
  *     `proceed-light` it authors the receipt Story (via the plan-persist
- *     `createStoryIssues` surface) and prints the init/close hand-off,
- *     carrying any predicted-shape `warnings[]` (Story #5313: an over-ceiling
- *     prediction warns, it no longer stops). An un-ledgered verdict or an
- *     un-waivable risk rule emits an `escalated` terminal envelope, never
- *     landing silently. The former attended stop-and-ask outcome and the
- *     flag that answered it went with the gate they answered.
+ *     `createStoryIssues` surface) and prints the init/close hand-off. An
+ *     un-ledgered verdict or an un-waivable risk rule emits an `escalated`
+ *     terminal envelope, never landing silently. Story #5313 demoted the
+ *     predicted-shape ceilings to warnings and Story #5344 deleted them: a
+ *     size bucket the caller declares about its own request is not evidence,
+ *     and the backstop below measures the real thing.
  *   - **backstop** (`--backstop --story <id>`) — re-check the ACTUAL diff of
  *     the Story branch after implementation; exit non-zero when it exceeds the
  *     light ceilings, so an over-scope diff is blocked rather than landed.
  *
- * ## Escalation is terminal, not advisory (Story #4746)
+ * ## Escalation is terminal for THIS path, not for the session (Story #5344)
  *
- * Over-scope under `--yes` emits a schema-validated `story-deliver-terminal`
- * envelope with status `escalated` and **ends the session**. Before that it was
- * an ordinary gate envelope plus exit 2 — a warning a caller could walk past,
- * and one mandrel-bench 2.13.0 light-arm run did exactly that: it read the
- * escalation, invoked `/mandrel-plan` in the same session, and delivered. In-session
- * planning under-decomposed (ONE Story against the scenario's 3-5 contract,
- * where a fresh `/mandrel-plan` session on the identical seed authored four), so
- * escalation silently produced the outcome the guard exists to prevent.
+ * A refused gate emits a schema-validated `story-deliver-terminal` envelope
+ * with status `escalated`: nothing was created, and there is no smaller version
+ * of the light path to attempt. Story #4746 additionally required the SESSION
+ * to end, on one mandrel-bench 2.13.0 observation where an in-session
+ * `/mandrel-plan` under-decomposed. Story #5344 relaxes that half — the
+ * escalation still ends the light path, and `/mandrel-plan` may now be seeded
+ * with `escalation.reasons` in the same session (see
+ * `helpers/deliver-light.md`).
  * {@link module:lib/orchestration/story-deliver-terminal.buildEscalationTerminal}
- * carries the guarantees the schema then enforces.
+ * carries the guarantees the schema enforces either way.
  *
  * Usage:
  *   node .agents/scripts/deliver-light.js --prompt "<text>" \
- *     --creates path,path --acceptance 1 --route lite --reason "<why>"
- *   node .agents/scripts/deliver-light.js --prompt "<text>" --amends '#123' --route lite --reason "<why>"
+ *     --creates path,path --reason "<why>"
+ *   node .agents/scripts/deliver-light.js --prompt "<text>" --amends '#123' --reason "<why>"
  *   node .agents/scripts/deliver-light.js --backstop --story 4741
  *
  * Exit codes: 0 ok (proceed / clean backstop), 1 usage error, 2 the gate did
@@ -81,37 +81,26 @@ import { createProvider } from './lib/provider-factory.js';
 const HELP = `\
 Usage:
   deliver-light.js --prompt <text> [--creates csv] [--refactors csv]
-                   [--acceptance n] [--kinds csv] [--magnitude m]
-                   [--uncertainty u] [--route lite|full] [--reason <text>]
-                   [--amends '#id'] [--yes]
+                   --reason <text> [--amends '#id']
   deliver-light.js --backstop --story <id>
 
 The thin /deliver-light entry point: suitability gate → inline receipt Story →
 the same single-story-init.js / single-story-close.js engine /mandrel-deliver uses.
 
-The gate judges EFFORT and RISK, not artifact counts: N instances of one
-mechanical edit is one kind at N sites. A predicted shape past a light ceiling
-is a WARNING on the envelope, not a refusal (Story #5313); only an un-ledgered
-verdict or an un-waivable risk rule (sensitive path, migration span) escalates.
-The --backstop pass enforces size against the actual diff.
+The gate judges RISK, not size. Only two things refuse: a verdict with no
+recorded reason, and an un-waivable risk rule the predicted PATHS trip (a
+sensitive-path class, a migration paired with its consumers). The predicted-
+shape ceilings were self-declared buckets and are gone (Story #5344, after
+Story #5313 had already demoted them to warnings). The --backstop pass measures
+the ACTUAL diff and is the only size block.
 
 Gate options:
   --prompt <text>    Operator prompt describing the change. Required for the gate.
   --creates <csv>    Predicted NEW file paths (comma-separated).
   --refactors <csv>  Predicted edited/existing file paths (comma-separated).
-  --acceptance <n>   Predicted acceptance-criteria count (default 1). Not capped.
-  --kinds <csv>      Distinct KINDS of change (default: one per assumption, so
-                     N same-shaped edits count once).
-  --magnitude <m>    Coarse effort bucket: trivial | moderate | substantial
-                     (default moderate; substantial routes to /mandrel-plan).
-  --uncertainty <u>  determined (the request fixes the shape) | needs-design
-                     (default determined; needs-design routes to /mandrel-plan).
-  --route <r>        Ledgered model verdict route: lite | full.
-  --reason <text>    Recorded reason for a lite verdict (required for lite).
+  --reason <text>    Recorded reason for taking the light path. Required: an
+                     un-ledgered verdict escalates.
   --amends <#id>     Mark this as an amendment of an existing issue.
-  --yes              Unattended marker. Escalation is terminal either way;
-                     the flag is accepted so unattended callers keep their
-                     invocation shape.
 
 Backstop options:
   --backstop         Re-check the ACTUAL diff after implementation. Bounds the
@@ -153,23 +142,6 @@ export function buildPredictedChanges({ creates = [], refactors = [] } = {}) {
   ];
 }
 
-/**
- * Synthesize a predicted-acceptance array of the requested length — the shape
- * gate reads the count, not the text, so placeholder strings suffice. A count
- * below 1 yields a single-item array (a Story with no contract cannot be judged
- * trivial, and the shape derivation rejects a zero-length acceptance anyway).
- *
- * @param {unknown} count
- * @returns {string[]}
- */
-export function synthesizeAcceptance(count) {
-  const n =
-    typeof count === 'number' && Number.isFinite(count) && count >= 1
-      ? Math.floor(count)
-      : 1;
-  return Array.from({ length: n }, (_v, i) => `AC-${i + 1}`);
-}
-
 /**
  * Run the suitability gate purely — no I/O. Returns the outcome envelope the
  * CLI serializes. The prompt text and `--amends` target are deliberately **not**
@@ -180,37 +152,25 @@ export function synthesizeAcceptance(count) {
  * @param {{
  *   creates?: string[],
  *   refactors?: string[],
- *   acceptance?: number,
- *   kinds?: string[],
- *   magnitude?: string,
- *   uncertainty?: string,
- *   route?: string,
  *   reason?: string,
  *   injectedRules?: object,
- * }} args `kinds` / `magnitude` / `uncertainty` are the declared effort-and-risk
- *   axes the gate judges (Story #4764); omitting them declares no signal, not a
- *   small one — an unrecognized bucket is reported as a warning (Story #5313).
+ * }} args `reason` is the ledgered verdict (Story #5344: the `--route` half is
+ *   gone, and so are the declared effort axes it sat beside — the recorded
+ *   reason and the predicted paths are what the gate reads. Story #5366
+ *   removed the last of them, `--acceptance`, whose value the gate clamped to
+ *   a floor of one before reading it).
  * @returns {{ action: string, suitability: object, outcome: object }}
  */
 export function runLightGate({
   creates = [],
   refactors = [],
-  acceptance,
-  kinds,
-  magnitude,
-  uncertainty,
-  route,
   reason,
   injectedRules,
 } = {}) {
   const predictedChanges = buildPredictedChanges({ creates, refactors });
   const suitability = deriveLightSuitability({
     predictedChanges,
-    predictedAcceptance: synthesizeAcceptance(acceptance),
-    predictedKinds: kinds,
-    predictedMagnitude: magnitude,
-    predictedUncertainty: uncertainty,
-    verdict: { route, reason },
+    verdict: { reason },
     injectedRules,
   });
   const outcome = resolveLightGateOutcome({ suitability });
@@ -312,18 +272,17 @@ async function runBackstopMode(values, deps = {}) {
 /**
  * Gate mode — judge the prompt and, on proceed, author the receipt Story.
  *
- * The three outcomes are deliberately asymmetric in what they emit:
+ * The two outcomes are deliberately asymmetric in what they emit:
  *
  *   - **`escalate-plan`** returns a schema-validated `escalated` **terminal
  *     envelope** and stops (Story #4746). It is placed **first**, above every
  *     creation call site, so "nothing was started" is a property of the
  *     control flow rather than a claim the envelope makes about itself.
- *   - **`proceed-light`** authors the receipt Story and prints the hand-off,
- *     carrying the predicted-shape `warnings[]` (Story #5313) on the envelope
- *     and on stderr, so an over-ceiling prediction is stated rather than
- *     silently waved through. The former attended stop-and-ask outcome is
- *     gone: the gate never had a question a human could answer that the diff
- *     backstop does not answer better.
+ *   - **`proceed-light`** authors the receipt Story and prints the hand-off.
+ *     The former attended stop-and-ask outcome is gone, and so are the
+ *     predicted-shape warnings Story #5313 left behind (Story #5344): the gate
+ *     never had a question a human could answer that the diff backstop does
+ *     not answer better.
  *
  * The injectable seams exist so the no-side-effect guarantee is testable
  * without a network: a test asserts the escalate path never reaches them.
@@ -356,13 +315,6 @@ export async function runGateMode(values, deps = {}) {
   const gate = runLightGate({
     creates: parseCsvPaths(values.creates),
     refactors: parseCsvPaths(values.refactors),
-    acceptance: values.acceptance
-      ? Number.parseInt(String(values.acceptance), 10)
-      : 1,
-    kinds: parseCsvPaths(values.kinds),
-    magnitude: values.magnitude,
-    uncertainty: values.uncertainty,
-    route: values.route,
     reason: values.reason,
   });
 
@@ -376,12 +328,11 @@ export async function runGateMode(values, deps = {}) {
     // recalibratable from evidence even now that only risk rules refuse.
     await recordRefusalFn({ gate, amends: values.amends });
     Logger.warn(
-      `[deliver-light] ESCALATED to /mandrel-plan — this session ENDS here; run ${envelope.nextCommand} in a FRESH session: ${gate.outcome.reasons.join('; ')}`,
+      `[deliver-light] ESCALATED to /mandrel-plan — the light path ENDS here; run ${envelope.nextCommand}, seeded with these reasons: ${gate.outcome.reasons.join('; ')}`,
     );
     return exitCodeForTerminal(envelope);
   }
 
-  const warnings = gate.outcome.warnings ?? [];
   const provider = createProviderFn(resolveConfigFn());
   const receipt = await createReceiptFn({
     provider,
@@ -398,15 +349,11 @@ export async function runGateMode(values, deps = {}) {
       action: 'proceed-light',
       storyId: receipt.storyId,
       url: receipt.url,
-      warnings,
       nextCommands: buildNextCommands(receipt.storyId),
       outcome: gate.outcome,
     },
     values.pretty,
   );
-  for (const warning of warnings) {
-    Logger.warn(`[deliver-light] ⚠ ${warning}`);
-  }
   Logger.info(
     `[deliver-light] receipt Story #${receipt.storyId} created — hand off to single-story-init.js.`,
   );
@@ -419,14 +366,8 @@ async function main() {
       prompt: { type: 'string' },
       creates: { type: 'string' },
       refactors: { type: 'string' },
-      acceptance: { type: 'string' },
-      kinds: { type: 'string' },
-      magnitude: { type: 'string' },
-      uncertainty: { type: 'string' },
-      route: { type: 'string' },
       reason: { type: 'string' },
       amends: { type: 'string' },
-      yes: { type: 'boolean', default: false },
       backstop: { type: 'boolean', default: false },
       story: { type: 'string' },
       pretty: { type: 'boolean', default: false },