@popovych.co/witness 0.16.0 → 0.17.0

package/README.md

@@ -182,7 +182,7 @@ the wrong file refuses. There are no `WITNESS_*` env vars for configuration.
 | `read <id> [--design] [--outline] [--lines <a>-<b>]` | print a spec/plan/design from the primary root — the read route, since a worktree carries no canon copy; `--outline` names each anchor's line range for fat artifacts |
 | `check` | schema, graph, invariants, needs, trailer audit, probes |
 | `index` | id · summary · status · depends across the canon, then each spec's plans |
-| `satisfy <id> --need <text\|n>` | flip a manual need via the write path |
+| `satisfy <id> --need <id\|n\|text>` | journal a manual need as met — the spec is never rewritten |
 | `log <id>` | render a journal stream |
 | `gate <decompose\|plan\|implement\|ship\|design> <id>` | run the reviewer gate; journals the round, stamps on pass |
 | `decide <gate> <id> --approve\|--revise [--pin <policy>]` | record the human decision on a stopped gate; `--pin` adds a standing content policy |

package/dist/allowlist.js

@@ -1,5 +1,6 @@
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
 import { join } from 'node:path';
+import { heldCriteria } from './needs.js';
 const allowPath = (root) => join(root, '.witness', 'allow.json');
 function load(root) {
     if (!existsSync(allowPath(root)))
@@ -35,11 +36,15 @@ export function grantCommands(root, cmds) {
 // artifact is the SPEC that owns the criteria: a plan gate judges a plan, but criteria live
 // on the spec it derives from, and a block that listed nothing there would hide the very
 // commands its approve is about to leave blocked.
+//
+// Row 179: a criterion an unmet need holds is not run, so it needs no trust until the need is
+// met. `witness trust <id>` still lists every command, so a human may trust one ahead of time.
 export function untrustedCmdsFor(root, doc) {
     if (!doc)
         return [];
+    const held = heldCriteria(doc);
     const list = Array.isArray(doc.meta.criteria) ? doc.meta.criteria : [];
-    const cmds = list.map((c) => c.cmd).filter((c) => typeof c === 'string');
+    const cmds = list.filter((c) => !held.has(String(c.id))).map((c) => c.cmd).filter((c) => typeof c === 'string');
     const trusted = new Set(load(root));
     return [...new Set(cmds.filter((c) => !trusted.has(c)))];
 }

package/dist/arbiter.js

@@ -1,3 +1,4 @@
+import { askOf, askText } from './ask.js';
 import { decidedBy } from './autonomy.js';
 import { loadLocalConfig } from './config.js';
 import { openDeferrals } from './deferral.js';
@@ -13,7 +14,7 @@ import { anchorRecurrence, gateRuns, roundBudget } from './rounds.js';
 import { findById } from './scan.js';
 import { canonicalJson } from './sha.js';
 // How a later mode (plan A2's recap check) adds its own prompt without a second invoker.
-export const PROMPT_OF = { decision: 'arbiter' };
+export const PROMPT_OF = { decision: 'arbiter', recap: 'arbiter-recap' };
 const TEXT_MAX = 600;
 function renderOptions(menu) {
     return menu.flatMap((o, i) => [
@@ -87,7 +88,7 @@ export async function arbitrate(ctx, root, cfg, req) {
         }
         const ruling = readRuling(text, req.menu.length);
         if (ruling.ok) {
-            return ok({ ...ruling.value, model: chain[rung], prompts_sha: promptsSha([{ name: 'arbiter', contents: lens.contents }]) });
+            return ok({ ...ruling.value, model: chain[rung], prompts_sha: promptsSha([{ name: PROMPT_OF[req.mode], contents: lens.contents }]) });
         }
         if (attempt === 0) {
             // One self-repair reroll on the same rung, in the shape runBattery gives a lens.
@@ -184,6 +185,11 @@ export function decisionEvidence(root, canon, stop) {
             }
         }
         sections.push({ title: 'Scope', body: scope.join('\n\n') });
+        // Row 177. A decompose stop is judged against the ask as well as the recap: both of its
+        // arbiter routes (the feature-scope stop and amendment-ack) re-scope what the human asked.
+        const ask = gate === 'decompose' ? askOf(root, target) : undefined;
+        if (ask !== undefined)
+            sections.push({ title: 'The ask', body: `${ask.id} — ${askText(ask.text)}` });
     }
     return sections;
 }

package/dist/ask.js

@@ -0,0 +1,182 @@
+import { FLOOR_STREAM } from './floor.js';
+import { stateCommit } from './gitio.js';
+import { appendEntry, effortStreams, entryLine, journalRel, readStream } from './journal.js';
+import { acquireLock } from './lock.js';
+import { ok, refuse, v } from './refusal.js';
+import { ROUND_BOUND } from './rounds.js';
+import { crashPoint, pendingTxn, withTxn } from './txn.js';
+// Row 176. The human's ask is journaled state, on a stream witness owns the way it owns
+// `floor`. This module is its one home: the entries, the queue derived from them, and the
+// one write path the verb, `/witness <text>` and `drive --ask` share — so a crashed session
+// loses nothing, because the brainstorm that answers an ask reads it from here, never from
+// its prompt.
+export const ASK_STREAM = 'asks';
+// The house bound on review rounds: a recap sent back three times waits for a human.
+export const RECAP_CHECK_BOUND = ROUND_BOUND;
+export function recapChecks(root, id) {
+    return readStream(root, ASK_STREAM)
+        .filter((e) => e.t === 'human-decision' && e.gate === 'recap' && e.artifact === id);
+}
+// The one rendering of why a check sent a recap back, shared by `recap`'s output, `next`'s
+// rung and the arbiter's own prior-checks section — so it lives here, and this module never
+// imports recap-check.ts.
+export function checkReason(e) {
+    return e.rule === 'recap-class'
+        ? `class — the recap says ${e.class} and the check decided ${e.decided ?? '?'} · ${e.reason}`
+        : e.reason;
+}
+// Display only: the stored text is the human's, trimmed and otherwise as given.
+export function askText(text) {
+    return oneLine(text);
+}
+// Text a human or a model wrote, made safe to print where a reader parses lines: the
+// skills read `next`'s unit line by line, and a prompt section must never open a heading of
+// its own. Whitespace runs collapse to one space; the words stay as written.
+export function oneLine(text) {
+    return text.trim().replace(/\s+/g, ' ');
+}
+export function askQueue(root) {
+    const out = [];
+    for (const e of readStream(root, ASK_STREAM)) {
+        if (e.t === 'ask')
+            out.push({ id: String(e.id), text: String(e.text), state: 'unanswered', checks: 0 });
+        else if (e.t === 'ask-withdrawn') {
+            const a = out.find((x) => x.id === e.ask);
+            if (a)
+                a.state = 'withdrawn';
+        }
+    }
+    for (const a of out) {
+        const checks = recapChecks(root, a.id);
+        const last = checks.at(-1);
+        a.checks = checks.length;
+        if (last?.decision === 'revise') {
+            a.reason = oneLine(checkReason({ rule: String(last.rule), class: last.class, decided: last.decided, reason: String(last.reason ?? '') }));
+            if (last.note !== undefined)
+                a.note = oneLine(last.note);
+        }
+        if (a.state !== 'unanswered' || last === undefined)
+            continue;
+        if (last.decision === 'stop')
+            a.parked = 'malformed';
+        else if (last.decision === 'revise' && checks.length >= RECAP_CHECK_BOUND)
+            a.parked = 'bound';
+        if (a.parked !== undefined)
+            a.state = 'parked';
+    }
+    // An ask is answered once any recap names it, even one an abandon later ended: re-asking is
+    // the human's act, so an abandoned effort never puts its ask back in the queue.
+    const answeredBy = new Map();
+    for (const effort of effortStreams(root)) {
+        for (const e of readStream(root, effort)) {
+            if (e.t === 'recap' && typeof e.ask === 'string' && !answeredBy.has(e.ask))
+                answeredBy.set(e.ask, effort);
+        }
+    }
+    for (const a of out) {
+        const effort = answeredBy.get(a.id);
+        if (effort !== undefined) {
+            a.state = 'answered';
+            a.effort = effort;
+            delete a.parked;
+        }
+    }
+    return out;
+}
+// The one list of stream ids witness owns. A stream id is an artifact's whole history, so an
+// effort, spec, plan or rename that took one of these would merge two histories into one.
+export function reservedId(id) {
+    if (id === ASK_STREAM)
+        return "witness's own ask queue";
+    if (id === FLOOR_STREAM)
+        return "witness's own floor stream";
+    return undefined;
+}
+const TEXT_REQUIRED = v('text', 'required', 'absent', 'what to build in words — witness ask <text>');
+export function queueAsk(ctx, root, text) {
+    if (askText(text) === '')
+        return refuse([TEXT_REQUIRED]);
+    const lock = acquireLock(root);
+    if (!lock.ok)
+        return lock;
+    try {
+        // Under the lock: a writer that crashed before this one took it left its marker here.
+        const pending = pendingTxn(root);
+        if (pending)
+            return refuse([v('.witness/txn.json', 'pending-txn', pending.op, 'witness recover --complete | --rollback')]);
+        // Final review F7: a stream an older witness wrote under this name is another artifact's
+        // history, and asks appended to it would be read as part of that artifact.
+        const first = readStream(root, ASK_STREAM)[0];
+        if (first !== undefined && first.t !== 'ask') {
+            return refuse([v(ASK_STREAM, 'stream-taken', `the stream starts with ${first.t}`, 'a stream for asks alone — witness check names the artifact that holds it')]);
+        }
+        // Final review F10: the same words while that ask is still open are the same ask — a
+        // crashed `/witness <words>` run again must not queue, and build, the work twice.
+        const queue = askQueue(root);
+        const unanswered = () => askQueue(root).filter((a) => a.state === 'unanswered').length;
+        const open = queue.find((a) => (a.state === 'unanswered' || a.state === 'parked') && askText(a.text) === askText(text));
+        if (open !== undefined)
+            return ok({ id: open.id, unanswered: unanswered(), already: true });
+        // numbered under the lock: two queuers racing must not both take the next id
+        const id = `a${queue.length + 1}`;
+        const entry = { t: 'ask', id, text: text.trim() };
+        const stream = journalRel(ASK_STREAM);
+        const res = withTxn(root, { op: `ask(${id})`, files: [stream], journal: { stream: ASK_STREAM, line: entryLine(entry) } }, () => {
+            appendEntry(root, ASK_STREAM, entry);
+            crashPoint(ctx.env, 'journal-append');
+            return stateCommit(root, [stream], `ask(${id}): ${askText(text).slice(0, 60)}`);
+        });
+        if (!res.ok)
+            return res;
+        return ok({ id, unanswered: unanswered(), already: false });
+    }
+    finally {
+        lock.value();
+    }
+}
+export function withdrawAsk(ctx, root, id, note) {
+    const lock = acquireLock(root);
+    if (!lock.ok)
+        return lock;
+    try {
+        const pending = pendingTxn(root);
+        if (pending)
+            return refuse([v('.witness/txn.json', 'pending-txn', pending.op, 'witness recover --complete | --rollback')]);
+        const ask = askQueue(root).find((a) => a.id === id);
+        if (!ask)
+            return refuse([v('--withdraw', 'unknown-ask', id, 'an ask id — witness status lists the queue')]);
+        if (ask.state === 'withdrawn')
+            return ok({ already: true });
+        if (ask.state === 'answered') {
+            return refuse([v('--withdraw', 'ask-answered', `${id} is answered by ${ask.effort}`, `an unanswered ask — ${ask.effort} answers it now · witness abandon ${ask.effort} ends that effort`)]);
+        }
+        const entry = { t: 'ask-withdrawn', ask: id, ...(note !== undefined ? { note } : {}) };
+        const stream = journalRel(ASK_STREAM);
+        const res = withTxn(root, { op: `ask(${id})`, files: [stream], journal: { stream: ASK_STREAM, line: entryLine(entry) } }, () => {
+            appendEntry(root, ASK_STREAM, entry);
+            crashPoint(ctx.env, 'journal-append');
+            return stateCommit(root, [stream], `ask(${id}): withdrawn`);
+        });
+        if (!res.ok)
+            return res;
+        return ok({ already: false });
+    }
+    finally {
+        lock.value();
+    }
+}
+// The one derivation of which ask is routed next: the oldest still open. A parked ask waits
+// for a human, so it is routable whenever a human interviews and skipped under autonomy,
+// where re-offering it would repeat a command `recap` refuses.
+export function nextAsk(root, autonomous) {
+    return askQueue(root).find((a) => a.state === 'unanswered' || (a.state === 'parked' && !autonomous));
+}
+// The ask an effort answers: what the decompose arbiter judges a slicing against beside the
+// recap itself (row 177). Named by the recap that birthed the effort — an amend carries no
+// ask, and it is exactly a re-scoped slicing that must still be judged against the ask.
+export function askOf(root, effort) {
+    const id = readStream(root, effort).find((e) => e.t === 'recap' && typeof e.ask === 'string')?.ask;
+    if (id === undefined)
+        return undefined;
+    return readStream(root, ASK_STREAM).find((e) => e.t === 'ask' && e.id === id);
+}

package/dist/cli.js

@@ -9,6 +9,8 @@ const VERBS = {
     recover: () => import('./verbs/recover.js'),
     init: () => import('./verbs/init.js'),
     recap: () => import('./verbs/recap.js'),
+    // Row 176. The human's ask, queued as state so `next` routes it and no session can lose it.
+    ask: () => import('./verbs/ask.js'),
     write: () => import('./verbs/write.js'),
     // Row 132's read route. Canon lives at the primary root and nowhere else, so reading it
     // is a verb rather than a path — which is also what lets a worktree carry no copy.
@@ -51,6 +53,7 @@ const VERBS = {
 const VERB_USAGE = {
     abandon: 'witness abandon <plan-id | effort-slug>',
     adopt: 'witness adopt <specs/... | plans/...>.md',
+    ask: 'witness ask <text> | --withdraw <id> [--note <why>]',
     calibrate: 'witness calibrate <exact-model-id> [--suite all|reviewers|skills] [--only <name>] [--samples <n>] [--publish]',
     check: 'witness check',
     clean: 'witness clean',
@@ -66,10 +69,10 @@ const VERB_USAGE = {
     log: 'witness log <stream> [--all] [--lineage]',
     next: 'witness next',
     read: 'witness read <id> [--design] [--outline] [--lines <a>-<b>]',
-    recap: 'witness recap --file <recap.json> [--amend]',
+    recap: 'witness recap --file <recap.json> [--ask <id>] [--amend] [--manual]',
     recover: 'witness recover [--complete | --rollback]',
     rename: 'witness rename <old-id> <new-id>',
-    satisfy: 'witness satisfy <doc-id> --need <text | index>',
+    satisfy: 'witness satisfy <doc-id> --need <id | index | text>',
     dismiss: 'witness dismiss <artifact> --deferral <id|index> --cause <superseded|lens-retired|judged-wrong> --note "<why>"',
     ship: 'witness ship <plan-id> [--manual]',
     start: 'witness start <plan-id>',

package/dist/criteria.js

@@ -1,9 +1,15 @@
 import { ensureTrusted } from './allowlist.js';
 import { loadConfig } from './config.js';
 import { matchesTag, sourceTags } from './matcher.js';
+import { heldCriteria, satisfyCommand } from './needs.js';
 import { ok } from './refusal.js';
 import { criteriaExcludes, execCommand, runFiltered, runFullSuite, runnerConfig, untrustedOf } from './runner.js';
 import { canonicalSha } from './sha.js';
+export function criterionMark(c) {
+    if (c.held !== undefined)
+        return `held by ${c.held.join(' · ')}`;
+    return c.ok ? 'ok' : 'fail';
+}
 export async function runCriteria(runRoot, ctx, doc, opts = {}) {
     const trustRoot = opts.trustRoot ?? runRoot;
     // D154 root unification. The trust list and the runner config are two halves of one
@@ -34,9 +40,18 @@ export async function runCriteria(runRoot, ctx, doc, opts = {}) {
     const src = rc.mode === 'filtered'
         ? sourceTags(runRoot, criteriaExcludes(cfg.value))
         : undefined;
+    const held = heldCriteria(doc);
     const criteria = [];
     for (const c of list) {
         const cid = String(c.id);
+        // A hold names a `cmd:` record, never just an id: a hand-edited doc can give a test
+        // criterion the same id, and holding it would skip the spec's tests.
+        const holders = typeof c.cmd === 'string' ? held.get(cid) : undefined;
+        if (holders !== undefined) {
+            criteria.push({ id: cid, kind: 'cmd', ok: false, held: holders,
+                detail: `held by ${holders.join(' · ')} — ${satisfyCommand(id, holders[0])}` });
+            continue;
+        }
         if (typeof c.test === 'string') {
             if (rc.mode === 'full-suite') {
                 if (!suite) {
@@ -94,7 +109,9 @@ export async function runCriteria(runRoot, ctx, doc, opts = {}) {
     return ok({
         spec: id,
         sha: canonicalSha(doc.meta, doc.body),
-        ok: criteria.length > 0 && criteria.every((c) => c.ok),
+        // A lane passes only when something actually passed and nothing failed: a held criterion
+        // is not a failure, and it is never a pass either.
+        ok: criteria.some((c) => c.ok) && criteria.every((c) => c.ok || c.held !== undefined),
         mode: rc.mode,
         tagCount,
         criteria,
@@ -107,12 +124,18 @@ export function laneCheck(lane) {
     const untrusted = lane.ok
         ? [...new Set(lane.value.criteria.flatMap((c) => (c.untrusted !== undefined ? [c.untrusted] : [])))]
         : [];
+    // Row 179: a lane that passed with held criteria says so in its own record, so no reader
+    // of the gate-run can take the green for a verification of what was not run.
+    const held = lane.ok
+        ? lane.value.criteria.flatMap((c) => (c.held ?? []).map((need) => ({ criterion: c.id, need, satisfy: satisfyCommand(lane.value.spec, need) })))
+        : [];
     return {
         name: 'drift-lane',
         ok: lane.ok && lane.value.ok,
         detail: lane.ok
-            ? lane.value.criteria.map((c) => `${c.id}:${c.ok ? 'ok' : 'fail'}`).join(' · ')
+            ? lane.value.criteria.map((c) => `${c.id}:${criterionMark(c)}`).join(' · ')
             : lane.violations.map((x) => `${x.field}:${x.rule}`).join(' · '),
         ...(untrusted.length > 0 ? { untrusted } : {}),
+        ...(held.length > 0 ? { held } : {}),
     };
 }

package/dist/departure.js

@@ -2,7 +2,7 @@ import { existsSync, mkdirSync, readdirSync, unlinkSync, writeFileSync } from 'n
 import { dirname, join } from 'node:path';
 import { deferralMoves } from './deferral.js';
 import { designRel } from './design.js';
-import { FLOOR_STREAM } from './floor.js';
+import { reservedId } from './ask.js';
 import { splitDoc } from './fm.js';
 import { adoptedCommits, lastWitnessed } from './adopt.js';
 import { gitBlob, tryGit } from './gitio.js';
@@ -130,13 +130,14 @@ function returnsThrough(root, paths, commit, chain, unwound, takenBack, seen) {
 // Whether a NEW artifact may take this id. An id names one artifact for the life of the repo,
 // because its stream IS that artifact's history. Three streams already belong to something
 // else: a superseded spec's (a successor owns its content, and its finished plans resolve their
-// parent there), an effort's, and witness's own floor pins. The one re-use is the same artifact
+// parent there), an effort's, and witness's own (`reservedId`: the floor pins and the ask queue). The one re-use is the same artifact
 // again: a spec its own abandon unwound may be written again as a spec. A stream that records
 // none of these — a stray, or a doc canon cannot parse — does not hold its id; the findings
 // that already name it (`orphan-journal`, the parse error) are its remedy.
 export function idRetired(root, id, type) {
-    if (id === FLOOR_STREAM)
-        return v('id', 'id-taken', id, `a new id — ${id} is witness's own floor stream`);
+    const reserved = reservedId(id);
+    if (reserved !== undefined)
+        return v('id', 'id-taken', id, `a new id — ${id} is ${reserved}`);
     const entries = readStream(root, id);
     if (entries[0]?.t === 'recap')
         return v('id', 'id-taken', id, `a new id — ${id} is an effort's slug`);

package/dist/drift.js

@@ -2,7 +2,7 @@ import { randomBytes } from 'node:crypto';
 import { join } from 'node:path';
 import { EXIT } from './cli.js';
 import { loadConfig, loadLocalConfig } from './config.js';
-import { runCriteria } from './criteria.js';
+import { criterionMark, runCriteria } from './criteria.js';
 import { serializeDoc, writeDoc } from './fm.js';
 import { acquireLock } from './lock.js';
 import { appendEntry, entryLine, journalRel, readStream } from './journal.js';
@@ -25,9 +25,17 @@ export function driftEntry(run, runId) {
         artifact_sha: run.sha,
         ok: run.ok,
         tags: run.tagCount,
-        criteria: run.criteria.map((c) => ({ id: c.id, kind: c.kind, ok: c.ok, detail: c.detail })),
+        criteria: run.criteria.map(criterionRecord),
     };
 }
+// Row 179: the journaled record of one criterion carries its hold, so a reader of the drift
+// history sees what was not run rather than an unexplained `ok: false` on a passing spec.
+const criterionRecord = (c) => ({ id: c.id, kind: c.kind, ok: c.ok, detail: c.detail, ...(c.held !== undefined ? { held: c.held } : {}) });
+// ` · <n> held` for a summary line, or nothing when the sweep held nothing.
+const heldSuffix = (runs) => {
+    const n = runs.reduce((k, r) => k + r.criteria.filter((c) => c.held !== undefined).length, 0);
+    return n > 0 ? ` · ${n} held` : '';
+};
 export function trailingFails(entries) {
     const checks = entries.filter((e) => e.t === 'drift-check');
     let n = 0;
@@ -133,7 +141,7 @@ export async function deepDrift(root, ctx, canon, specId) {
         renderRefusal(localR.violations).forEach(ctx.err);
         return EXIT.REFUSED;
     }
-    const prompt = `${lensR.value.contents}\n\n## Reviewed content\n\n### Spec: ${specId}\n${serializeDoc({ meta: doc.meta, body: doc.body })}\n\n(Deterministic lane: ${lane.ok ? lane.value.criteria.map((c) => `${c.id}:${c.ok ? 'ok' : 'fail'}`).join(' · ') : 'unrunnable'})\n`;
+    const prompt = `${lensR.value.contents}\n\n## Reviewed content\n\n### Spec: ${specId}\n${serializeDoc({ meta: doc.meta, body: doc.body })}\n\n(Deterministic lane: ${lane.ok ? lane.value.criteria.map((c) => `${c.id}:${criterionMark(c)}`).join(' · ') : 'unrunnable'})\n`;
     const invoked = invokeReviewer(ctx, hxR.value.harness, {
         cwd: root, prompt,
         timeoutMs: cfgR.ok ? cfgR.value.gates.reviewerTimeoutMs : undefined,
@@ -152,7 +160,7 @@ export async function deepDrift(root, ctx, canon, specId) {
         v: 1, t: 'drift-check', run_id: newRunId(), artifact: specId,
         artifact_sha: lane.ok ? lane.value.sha : canonicalSha(doc.meta, doc.body),
         ok: lane.ok && lane.value.ok, tags: lane.ok ? lane.value.tagCount : 0,
-        criteria: lane.ok ? lane.value.criteria.map((c) => ({ id: c.id, kind: c.kind, ok: c.ok, detail: c.detail })) : [],
+        criteria: lane.ok ? lane.value.criteria.map(criterionRecord) : [],
         deep: true,
         ...(violations.length === 0 && parsed.ok ? { verdict: parsed.value } : { malformed: violations }),
     };
@@ -233,7 +241,7 @@ export async function runDrift(ctx, argv) {
         })).forEach(ctx.out);
     }
     if (readonly) {
-        ctx.out(kv('drift-summary', `${failing.length}/${runs.length} failing · mode read-only`));
+        ctx.out(kv('drift-summary', `${failing.length}/${runs.length} failing${heldSuffix(runs)} · mode read-only`));
         ctx.out('help: read-only run — the next local check --drift journals and stamps');
         return failing.length ? EXIT.FINDINGS : EXIT.OK;
     }
@@ -282,7 +290,7 @@ export async function runDrift(ctx, argv) {
         const stamped = actions.filter((a) => a.stamp).length;
         const cleared = actions.filter((a) => a.clear).length;
         const suffix = `${stamped ? ` · stamped ${stamped}` : ''}${cleared ? ` · cleared ${cleared}` : ''}`;
-        ctx.out(kv('drift-summary', `${failing.length}/${runs.length} failing · mode local${suffix}`));
+        ctx.out(kv('drift-summary', `${failing.length}/${runs.length} failing${heldSuffix(runs)} · mode local${suffix}`));
     }
     finally {
         lock.ok && lock.value();

package/dist/dsl.js

@@ -44,6 +44,7 @@ export function validateNeeds(raw) {
         return [v('needs', 'shape', typeof raw, 'list of {env} | {cmd} | {manual, satisfied}')];
     }
     const out = [];
+    const ids = new Set();
     raw.forEach((n, i) => {
         const at = `needs[${i}]`;
         if (typeof n !== 'object' || n === null) {
@@ -63,6 +64,72 @@ export function validateNeeds(raw) {
         if (kind === 'manual' && typeof e.satisfied !== 'boolean') {
             out.push(v(`${at}.satisfied`, 'required', String(e.satisfied ?? 'absent'), 'boolean'));
         }
+        if (e.id !== undefined) {
+            if (typeof e.id !== 'string' || !ID_RE.test(e.id))
+                out.push(v(`${at}.id`, 'id-charset', String(e.id), '[a-z0-9-]+'));
+            else if (ids.has(e.id))
+                out.push(v(`${at}.id`, 'id-unique', e.id, 'unique per doc'));
+            else
+                ids.add(e.id);
+        }
+        if (e.holds === undefined)
+            return;
+        if (kind !== 'manual') {
+            out.push(v(`${at}.holds`, 'forbidden', JSON.stringify(e.holds), 'holds rides a manual need'));
+            return;
+        }
+        const holds = e.holds;
+        if (!Array.isArray(holds) || holds.length === 0 || new Set(holds).size !== holds.length ||
+            holds.some((h) => typeof h !== 'string' || !ID_RE.test(h))) {
+            out.push(v(`${at}.holds`, 'shape', JSON.stringify(holds), 'a non-empty list of criterion ids'));
+        }
+        if (e.id === undefined)
+            out.push(v(`${at}.id`, 'required', 'absent', 'an id — a held criterion reports held by <id>'));
     });
     return out;
 }
+// Row 179. What a need's `holds` may name, judged against the doc it sits in: a spec's own
+// `cmd:` criteria only — test criteria share one @spec run, so holding one holds none — and
+// never all of them, so the lane always runs something while every need is unmet. The
+// shapes themselves are validateNeeds'; a list it refused is not judged twice here.
+export function validateHolds(meta) {
+    const needs = Array.isArray(meta.needs) ? meta.needs : [];
+    const criteria = meta.type === 'spec' && Array.isArray(meta.criteria)
+        ? meta.criteria.filter((c) => typeof c === 'object' && c !== null)
+        : [];
+    const ids = (pick) => criteria.filter((c) => typeof c.id === 'string' && pick(c)).map((c) => c.id);
+    const cmdIds = ids((c) => typeof c.cmd === 'string');
+    const testIds = ids((c) => typeof c.test === 'string');
+    const out = [];
+    const held = new Set();
+    needs.forEach((n, i) => {
+        if (typeof n !== 'object' || n === null)
+            return;
+        const e = n;
+        if (typeof e.manual !== 'string' || !Array.isArray(e.holds))
+            return;
+        if (meta.type !== 'spec') {
+            out.push(v(`needs[${i}].holds`, 'forbidden', JSON.stringify(e.holds), 'holds is spec-only — a plan need blocks its start instead'));
+            return;
+        }
+        e.holds.forEach((h, j) => {
+            if (typeof h !== 'string')
+                return;
+            const at = `needs[${i}].holds[${j}]`;
+            if (testIds.includes(h)) {
+                out.push(v(at, 'held-test-criterion', h, 'a cmd criterion — test criteria share one @spec run so holding one holds none'));
+            }
+            else if (!cmdIds.includes(h)) {
+                out.push(v(at, 'unknown-criterion', h, `a cmd criterion id of this spec: ${cmdIds.join(' · ') || '(none)'}`));
+            }
+            else {
+                held.add(h);
+            }
+        });
+    });
+    const all = ids(() => true);
+    if (held.size > 0 && all.every((id) => held.has(id))) {
+        out.push(v('needs', 'holds-every-criterion', [...held].join(' · '), 'at least one criterion the lane runs while every need is unmet'));
+    }
+    return out;
+}

package/dist/gate.js

@@ -65,6 +65,9 @@ export function renderGateRun(ctx, entry, mode, opts = {}) {
     if (entry.rerolled?.length)
         ctx.out(kv('rerolled', entry.rerolled.join(' · ')));
     ctx.out(rows('checks', ['name', 'ok', 'detail'], entry.checks.map((c) => ({ name: c.name, ok: String(c.ok), detail: c.detail ?? '' }))).join('\n'));
+    // Row 179: what the lane did not run, beside the row that passed without it.
+    for (const h of entry.checks.flatMap((c) => c.held ?? []))
+        ctx.out(`held: ${h.criterion} — ${h.need} unmet · ${h.satisfy}`);
     const findings = (entry.verdicts ?? []).flatMap((rv) => rv.findings.map((f) => ({
         reviewer: rv.reviewer, blocking: String(f.blocking),
         pin: f.contradicts_pin !== undefined ? `#${f.contradicts_pin}` : '',
@@ -188,7 +191,7 @@ async function judge(ctx, gateName, target, flags) {
         return owed !== undefined && !owed.manual;
     };
     const canon = loadCanon(root);
-    const inputR = await spec.resolve(root, ctx, canon, cfgR.value, target);
+    const inputR = await spec.resolve(root, ctx, canon, cfgR.value, target, { autonomous });
     if (!inputR.ok) {
         renderRefusal(inputR.violations).forEach((l) => ctx.err(l));
         return EXIT.REFUSED;
@@ -300,7 +303,9 @@ async function judge(ctx, gateName, target, flags) {
     // A check trust kept from running and one that ran and failed are both `ok: false`, but they
     // are different judgments: granting the command flips the first into the second, and without
     // this term the gate resumed the blocked round forever while the policy re-ran it (F3).
-    const checkOk = (cs) => cs.map((c) => `${c.name}:${c.ok}${(c.untrusted?.length ?? 0) > 0 ? ':blocked' : ''}`).sort().join('|');
+    // A hold that lifts or lands is a new judgment too (row 179): the lane may stay green while
+    // the set of criteria it did not run changes, and a resumed round would re-render a stale `held:`.
+    const checkOk = (cs) => cs.map((c) => `${c.name}:${c.ok}${(c.untrusted?.length ?? 0) > 0 ? ':blocked' : ''}:${(c.held ?? []).map((h) => h.criterion).sort().join('+')}`).sort().join('|');
     if ((kind.kind === 'resume' || kind.kind === 'changed-nothing') &&
         checkOk(kind.entry.checks) !== checkOk(input.checks) &&
         kind.entry.outcome !== 'malformed' && (kind.entry.verdicts?.length ?? 0) > 0) {

package/dist/gates/design.js

@@ -15,7 +15,7 @@ const reviewedShaOf = (html, spec) => html !== undefined ? designPairSha(html, s
 registerGate({
     gate: 'design',
     targetKind: 'spec',
-    async resolve(root, _ctx, canon, cfg, specId) {
+    async resolve(root, _ctx, canon, cfg, specId, run) {
         const spec = findById(canon, specId);
         if (!spec || spec.meta.type !== 'spec') {
             return refuse([v('spec', 'unknown-spec', specId, 'a specs/ doc id')]);
@@ -31,7 +31,12 @@ registerGate({
         // changed-nothing, bound, malformed-streak), producing a window the journal never
         // learns about. An agent that drops the show step now hits a loud refusal its repair
         // loop already handles, instead of silently gating something nobody was shown.
-        const unseen = designUnseen(root, cfg.paths, specId);
+        //
+        // Row 178 moves sight from the gate to the approval for an autonomous run. Gating unseen
+        // mattered because a human decided each round from findings; under autonomy nobody does
+        // until the kept stop, and `decide design --approve` still refuses a look nobody opened,
+        // so the human's stop is where it is opened and nothing stamps a look unseen.
+        const unseen = run?.autonomous ? undefined : designUnseen(root, cfg.paths, specId);
         if (unseen !== undefined) {
             return refuse([v('design', 'design-unseen', `no sight witnessed for ${short(unseen)}`, `a human shown this artifact — run: witness design ${specId} --open`)]);
         }