@pmelab/gtd 24.3.0 → 25.0.0

package/dist/workflow-kit.mjs

@@ -1,5 +1,5 @@
 import { changes, changesSince, hasThreadFor, head, read, refuse, requireReplies, requireThreadsClosed, scope } from "./flows.mjs";
-import { c as stripCodeThreads, d as reviewNotes, f as reviewRisks } from "./chunks/steering-D-PvRC5W.mjs";
+import { c as stripCodeThreads, d as reviewNotes, f as reviewRisks } from "./chunks/steering-DVILanVf.mjs";
 //#region src/workflow-kit/health.ts
 /** Hand a red suite that repeated attempts could not fix to a person, until `limit` rounds are spent. */
 const escalation = async (count, steps) => {
@@ -67,102 +67,59 @@ const fixFindings = async (f) => {
 };
 //#endregion
 //#region src/workflow-kit/scenarios.ts
-const DOC_STRING = /^(\s*)("""|```)/;
-/**
-* Gherkin wording only: lines trimmed; blank, `#` comment and `@` tag lines
-* dropped — except inside a doc string, whose content is test input: kept
-* whole, indented relative to its opening delimiter (as Gherkin reads it).
-*/
-const scenarioText = (source) => {
+const DOC_STRING = /^("""|```)/;
+const THEN = /^Then\s/;
+const CONTINUATION = /^(And|But|\*)\s/;
+const HEADING = /^(Feature|Rule|Background|Scenario|Scenario Outline|Scenario Template|Example):/;
+/** Trimmed lines outside doc strings, whose content is test input, not steps. */
+const stepLines = (source) => {
 	const out = [];
-	let doc;
-	for (const line of source.split("\n")) {
-		const delimiter = DOC_STRING.exec(line);
-		if (doc === void 0 && delimiter !== null) {
-			doc = {
-				indent: delimiter[1].length,
-				fence: delimiter[2]
-			};
-			out.push(line.trim());
-		} else if (doc !== void 0 && line.trim().startsWith(doc.fence)) {
-			doc = void 0;
-			out.push(line.trim());
-		} else if (doc !== void 0) {
-			const lead = line.length - line.trimStart().length;
-			out.push(line.slice(Math.min(lead, doc.indent)).trimEnd());
-		} else {
-			const trimmed = line.trim();
-			if (trimmed !== "" && !trimmed.startsWith("#") && !trimmed.startsWith("@")) out.push(trimmed);
-		}
-	}
-	return out.join("\n");
-};
-const textOf = (path) => {
-	const source = read(path);
-	return source === void 0 ? void 0 : scenarioText(source);
-};
-/** Snapshot `.feature` paths at `head()`, merged over `previous`; `undefined` when nothing is frozen. */
-const freezeScenarios = (paths, previous) => {
-	if (paths.length === 0) return previous;
-	const texts = { ...previous?.texts };
-	for (const path of paths) {
-		const text = textOf(path);
-		if (text !== void 0) texts[path] = text;
+	let fence;
+	for (const trimmed of source.split("\n").map((l) => l.trim())) {
+		const opening = fence === void 0 ? DOC_STRING.exec(trimmed)?.[1] : void 0;
+		if (opening !== void 0) fence = opening;
+		else if (fence === void 0) out.push(trimmed);
+		else if (trimmed.startsWith(fence)) fence = void 0;
 	}
-	return Object.keys(texts).length === 0 ? previous : {
-		at: head(),
-		texts
-	};
-};
-/** Paths whose wording differs from the snapshot (a deleted file counts). */
-const driftedScenarios = (frozen) => Object.entries(frozen.texts).filter(([path, text]) => textOf(path) !== text).map(([path]) => path);
-const lines = (text) => text === void 0 || text === "" ? [] : text.split("\n");
-/** `table[i][j]`: the longest common subsequence of `a[i..]` and `b[j..]`. */
-const lcsTable = (a, b) => {
-	const table = Array.from({ length: a.length + 1 }, () => Array(b.length + 1).fill(0));
-	for (let i = a.length - 1; i >= 0; i--) for (let j = b.length - 1; j >= 0; j--) table[i][j] = a[i] === b[j] ? table[i + 1][j + 1] + 1 : Math.max(table[i + 1][j], table[i][j + 1]);
-	return table;
+	return out;
 };
-/** Changed lines in file order, `- ` / `+ `-prefixed, so a moved or repeated step still shows. */
-const lineDiff = (a, b) => {
-	const table = lcsTable(a, b);
+const neutral = (line) => line === "" || line.startsWith("#") || line.startsWith("|");
+/** Trimmed `Then` step lines, an `And`/`But`/`*` continuing one included; doc strings skipped. */
+const assertionLines = (source) => {
 	const out = [];
-	let i = 0;
-	let j = 0;
-	while (i < a.length && j < b.length) if (a[i] === b[j]) {
-		i++;
-		j++;
-	} else if (table[i + 1][j] >= table[i][j + 1]) out.push(`- ${a[i++]}`);
-	else out.push(`+ ${b[j++]}`);
-	return [
-		...out,
-		...a.slice(i).map((l) => `- ${l}`),
-		...b.slice(j).map((l) => `+ ${l}`)
-	];
+	let inThen = false;
+	let scenario = "";
+	for (const line of stepLines(source)) {
+		if (HEADING.test(line)) scenario = line;
+		if (THEN.test(line)) inThen = true;
+		else if (!CONTINUATION.test(line) && !neutral(line)) inThen = false;
+		if (inThen && !neutral(line)) out.push({
+			scenario,
+			line
+		});
+	}
+	return out;
 };
-const wordingDrift = (frozen, path) => ({
-	path,
-	lines: lineDiff(lines(frozen.texts[path]), lines(textOf(path)))
-});
+const key = (a) => `${a.scenario}\n${a.line}`;
 /**
-* After a turn: rest at `stop` while wording drifted from `frozen`. Whatever
-* still differs after it is accepted and moves the snapshot; what was
-* restored stays frozen as it was.
+* Assertions a `.feature` file that existed before `changes` lost — the one
+* way rewording a scenario can weaken it. Matched within each scenario and
+* counted, so a line moving to another scenario, or one of two identical
+* `Then`s dropped, still shows. A renamed scenario reports all of its own.
 */
-const holdWording = async (frozen, stop) => {
-	if (frozen === void 0) return;
-	const drifted = driftedScenarios(frozen);
-	if (drifted.length === 0) return;
-	await stop(frozen.at, drifted.map((path) => wordingDrift(frozen, path)));
-	const accepted = driftedScenarios(frozen);
-	if (accepted.length === 0) return;
-	for (const path of accepted) {
-		const text = textOf(path);
-		if (text === void 0) delete frozen.texts[path];
-		else frozen.texts[path] = text;
-	}
-	frozen.at = head();
-};
+const removedAssertions = (changes) => changes.filter((c) => c.path.endsWith(".feature") && c.before !== void 0).map((c) => {
+	const left = /* @__PURE__ */ new Map();
+	for (const a of assertionLines(c.after ?? "")) left.set(key(a), (left.get(key(a)) ?? 0) + 1);
+	const lines = assertionLines(c.before ?? "").filter((a) => {
+		const n = left.get(key(a)) ?? 0;
+		left.set(key(a), n - 1);
+		return n <= 0;
+	});
+	return {
+		path: c.path,
+		lines
+	};
+}).filter((r) => r.lines.length > 0);
 //#endregion
 //#region src/workflow-kit/review.ts
 const collect = async (p, capture) => {
@@ -196,16 +153,17 @@ const converse = async (p, base, collectedBefore) => {
 		if (!hasThreadFor("human", p.file)) return collectedAt;
 	}
 };
-/** Judge every note, then run each verdict's route: answers, then nit fixes, then the edits' lap. */
+/** Judge every note, then run each verdict's route: answers, then nit fixes, then deferrals, then the edits' lap. */
 const routeNotes = async (p, notes, r) => {
 	const verdicts = await p.triage(notes);
 	const of = (v) => notes.filter((n) => (verdicts.get(n.id) ?? "edit") === v);
-	const [edits, questions, nits] = [
+	const [edits, questions, nits, deferred] = [
 		of("edit"),
 		of("question"),
-		of("nit")
+		of("nit"),
+		of("defer")
 	];
-	if (edits.length + questions.length + nits.length === 0) return r.unfolded();
+	if (edits.length + questions.length + nits.length + deferred.length === 0) return r.unfolded();
 	let answeredAt;
 	if (questions.length > 0) {
 		await p.answerQuestions(questions);
@@ -215,6 +173,7 @@ const routeNotes = async (p, notes, r) => {
 		await p.fixNits(nits);
 		await p.keepGreen();
 	}
+	if (deferred.length > 0) await p.defer(deferred);
 	if (edits.length > 0) {
 		await r.close();
 		return r.outcome(await collect(p, p.captures.editNotes(r.round, edits, answeredAt)));
@@ -228,7 +187,7 @@ const routeNotes = async (p, notes, r) => {
 	}
 	return {
 		next: "await",
-		reviewed: answeredAt
+		reviewed: answeredAt ?? r.round
 	};
 };
 /** A code edit, not a thread-only change: lines that only add, answer or remove code threads don't count. */
@@ -349,4 +308,4 @@ const reviewToHandOff = async (base, p) => {
 	}
 };
 //#endregion
-export { driftedScenarios, escalation, fixFindings, freezeScenarios, gate, healthy, holdWording, lensScopes, lensesOf, qualityLap, review, reviewToHandOff, scenarioText, untilSettled, wordingDrift };
+export { assertionLines, escalation, fixFindings, gate, healthy, lensScopes, lensesOf, qualityLap, removedAssertions, review, reviewToHandOff, routeNotes, untilSettled };

package/dist/workflows/feature.mjs

@@ -1,6 +1,6 @@
 import { answered, changes, changesSince, glob, hasThreadFor, head, human, judge, numeric, read, refuse, removeScript, requireAnswers, requireProgress, requireReplies, requireRevert, requireThreadsClosed, restoreScript, revertScript, run, scope, start, vars } from "../flows.mjs";
-import { freezeScenarios, review as review$1 } from "../workflow-kit.mjs";
-import { $ as unwindFailure, A as architectureAuthorPrompt, B as designTriagePrompt, C as fixCheck, D as reviewing, E as reviewMissing, F as buildReviewFixNitsPrompt, G as packagesItemFixSuitePrompt, I as buildReviewTriageMessage, K as reviewEditNotesCapture, L as builderSystem, M as architectureGateAnswerMessage, N as buildReviewAnswerQuestionsPrompt, O as agentWithSkills, P as buildReviewCollectingPrompt, Q as unwindFailedMessage, R as designGateAnswerMessage, S as coder, T as planner, U as idleMessage, W as packagesItemBuildingPrompt, X as reviewerSystem, Y as reviewNotesCapture, Z as startGateBlockedMessage, _ as ARCHITECTURE, b as REVIEW, c as refusePendingHandOff, d as shipped, f as defaults, g as healthy, h as gate, i as skills, j as architectureDecomposePrompt, k as architectSystem, l as tail, n as summary, o as closeReview, p as envDefaults, q as reviewEditsCapture, r as access, t as steering, u as guarded, v as FEEDBACK, w as fixRisks, x as awaitReview, y as REQUIREMENTS, z as designSystem } from "../chunks/reserved-DJwU5C4z.mjs";
+import { review as review$1 } from "../workflow-kit.mjs";
+import { $ as unwindFailedMessage, A as architectureDecomposePrompt, B as designSystem, C as fixRisks, D as agentWithSkills, E as reviewing, F as buildReviewFixNitsPrompt, G as packagesItemBuildingPrompt, I as buildReviewTriageMessage, J as reviewEditsCapture, K as packagesItemFixSuitePrompt, L as builderSystem, M as buildReviewAnswerQuestionsPrompt, N as buildReviewAwaitAfterDeferMessage, O as architectSystem, P as buildReviewCollectingPrompt, Q as startGateBlockedMessage, R as deferNotesScript, S as fixCheck, T as reviewMissing, V as designTriagePrompt, W as idleMessage, X as reviewNotesCapture, Z as reviewerSystem, _ as FEEDBACK, b as awaitReview, c as refusePendingHandOff, d as defaults, et as unwindFailure, f as envDefaults, g as ARCHITECTURE, h as healthy, i as skills, j as architectureGateAnswerMessage, k as architectureAuthorPrompt, l as tail, m as gate, n as summary, o as closeReview, q as reviewEditNotesCapture, r as access, t as steering, u as shipped, v as REQUIREMENTS, w as planner, x as coder, y as REVIEW, z as designGateAnswerMessage } from "../chunks/reserved-6pAgFGgB.mjs";
 //#region src/workflows/feature/steps.ts
 /** Turn the sketch since `base` into `.gtd/REQUIREMENTS.md`. */
 const triage = (base) => agentWithSkills("triage", designTriagePrompt(base), {
@@ -71,10 +71,13 @@ const collecting = (capture) => agentWithSkills("review.collecting", buildReview
 	system: reviewerSystem(),
 	allowEmpty: true
 });
+/** `build.review.defer`: append `notes` to `.gtd/TODO.md` as one section; the step's landing is the commit. */
+const deferNotes = (notes, round) => run("review.defer", deferNotesScript(notes, round), {
+	label: "Deferring could-notes to TODO.md",
+	file: ".gtd/TODO.md"
+});
 //#endregion
 //#region src/workflows/feature/packages.ts
-/** Package 0: the e2e scenarios, written red. */
-const SCENARIO_PACKAGE = ".gtd/packages/00-e2e-scenarios.md";
 const TEST_LINE = /^\s*-\s+(unit|e2e):\s+`([^`]+)`/;
 /** The `## Tests` entries of a package file; malformed lines are ignored. */
 const declaredTests = (packageText) => {
@@ -104,47 +107,29 @@ const requireDeclaredTests = (pkg, since) => {
 	if (missing.length === 0) return;
 	refuse(`gtd land: declared-tests: ${pkg} declares tests that are ${satisfied ? "missing from the tree" : "not in the package's diff"} — write them first:\n${missing.map((path) => `  - ${path}`).join("\n")}`);
 };
-/** Build `pkg`, check its declared tests, keep the fast suite green, and close it out, which removes it. */
-const packageItem = async (pkg, frozen) => {
+/**
+* Build `pkg`, check its declared tests, keep the full suite green, and close
+* it out, which removes it — and `.gtd/ARCHITECTURE.md` with the last one.
+*/
+const packageItem = async (pkg) => {
 	const from = head();
 	const declared = (turn) => async () => {
 		await turn();
 		requireDeclaredTests(pkg, from);
 	};
-	await guarded(frozen, declared(() => build(pkg)))();
-	await healthy(guarded(frozen, declared(fixSuite)), { suite: "fast" });
-	await run("closing", removeScript([pkg, ".gtd/SATISFIED.md"]), { label: "Closing out the package" });
-	return {
+	await declared(() => build(pkg))();
+	await healthy(declared(fixSuite));
+	await run("closing", removeScript([
 		pkg,
-		from,
-		to: head()
-	};
+		".gtd/SATISFIED.md",
+		...glob(".gtd/packages/*.md").length === 1 ? [ARCHITECTURE] : []
+	]), { label: "Closing out the package" });
 };
 /** The first queued package — the one a package step works on. */
 const nextPackage = () => [...glob(".gtd/packages/*.md")].sort()[0];
-const outsideGtd = (path) => path !== ".gtd" && !path.startsWith(".gtd/");
 /** Build every package file under `.gtd/packages/`, in name order. */
-const packages = (carried) => scope("packages", async () => {
-	let frozen = carried;
-	const ranges = [];
-	let touched = [];
-	for (let pkg = nextPackage(); pkg !== void 0; pkg = nextPackage()) {
-		const range = await scope("item", () => packageItem(pkg, pkg === ".gtd/packages/00-e2e-scenarios.md" ? void 0 : frozen));
-		ranges.push(range);
-		if (pkg === ".gtd/packages/00-e2e-scenarios.md") {
-			touched = changesSince(range.from).filter((c) => outsideGtd(c.path) && c.status !== "deleted");
-			frozen = freezeScenarios(touched.filter((c) => c.path.endsWith(".feature")).map((c) => c.path), frozen);
-		}
-	}
-	const paths = (status) => touched.filter((c) => c.status === status).map((c) => c.path);
-	return {
-		ranges,
-		scenarios: {
-			added: paths("added"),
-			changed: paths("modified")
-		},
-		frozen
-	};
+const packages = () => scope("packages", async () => {
+	for (let pkg = nextPackage(); pkg !== void 0; pkg = nextPackage()) await scope("item", () => packageItem(pkg));
 });
 //#endregion
 //#region src/workflows/feature/planning.ts
@@ -177,7 +162,7 @@ const architecture = () => scope("architecture", async () => {
 	} while (await noteGate(ARCHITECTURE, answerTechnicalQuestions));
 	await decompose();
 	if (changes(".gtd/packages/**").length === 0) refuse("gtd land: decompose: write at least one package under .gtd/packages/");
-	if (changes(".gtd/ARCHITECTURE.md").some((c) => c.status === "deleted")) refuse("gtd land: decompose: keep .gtd/ARCHITECTURE.md — the build tail's full run needs it; it is removed only once that run is green");
+	if (changes(".gtd/ARCHITECTURE.md").some((c) => c.status === "deleted")) refuse("gtd land: decompose: keep .gtd/ARCHITECTURE.md — the packages are built from it; it is removed with the last package");
 });
 //#endregion
 //#region src/workflows/feature/review.ts
@@ -194,23 +179,106 @@ const verdictQuestion = (note) => ({
 	instructions: `Classify the human's note on review item "${note.anchor}" (in .gtd/REVIEW.md). Its evidence holds the anchor, the reviewer's text and the human's text.`,
 	criteria: "edit: a request to change behaviour, design or scope — anything needing a plan, or any note you are unsure about. question: asks something and wants an answer, with no change requested. nit: a small, local, unambiguous fix (naming, typo, formatting, comment wording) needing no re-plan. praise: an approving remark with nothing to do."
 });
+const SEVERITY_ANSWERS = {
+	security: ["yes", "no"],
+	reversibility: ["one-way", "two-way"],
+	blastRadius: [
+		"local",
+		"module",
+		"contract"
+	],
+	newConcept: ["yes", "no"]
+};
 /**
-* One judge rest, one verdict per note. Dismissing a note is the risky
+* Deferring is the risky direction: anything unanswered, unrecognised, cut or
+* below `minP` is `critical`, never `could`.
+*/
+const severityOf = (note, judgment, minP) => {
+	const ans = (q) => judgment.answers[`${note.id}.${q}`];
+	const a = (q) => ans(q)?.answer;
+	const rank = () => a("reversibility") === "one-way" ? 0 : a("newConcept") === "yes" ? 1 : 2;
+	if (note.forced) return {
+		tier: "critical",
+		reason: "critical: forced by `!`",
+		rank: 2
+	};
+	if (judgment.truncated.includes(note.id) || Object.entries(SEVERITY_ANSWERS).some(([q, allowed]) => !allowed.includes(a(q) ?? "") || !((ans(q)?.p ?? NaN) >= minP))) return {
+		tier: "critical",
+		reason: "critical: judge unsure",
+		rank: 2
+	};
+	const critical = [...a("security") === "yes" ? ["security"] : [], ...a("reversibility") === "one-way" ? ["one-way door"] : []];
+	if (critical.length > 0) return {
+		tier: "critical",
+		reason: `critical: ${critical.join(", ")}`,
+		rank: rank()
+	};
+	const should = [...a("blastRadius") === "contract" ? ["contract"] : [], ...a("newConcept") === "yes" ? ["new concept"] : []];
+	if (should.length > 0) return {
+		tier: "should",
+		reason: `should: ${should.join(", ")}`,
+		rank: rank()
+	};
+	return {
+		tier: "could",
+		reason: "could",
+		rank: 2
+	};
+};
+const severityQuestions = (note) => [
+	{
+		id: `${note.id}.security`,
+		primitive: "noul",
+		instructions: `Does the change the human asks for in note "${note.id}" (on "${note.anchor}") touch security — authentication, authorisation, secrets, untrusted input or data exposure?`,
+		criteria: "yes: it changes or could weaken a security property. no: it does not."
+	},
+	{
+		id: `${note.id}.reversibility`,
+		primitive: "choice",
+		instructions: `Once the change in note "${note.id}" ships, can it be undone without breaking anyone who already depends on it?`,
+		criteria: "one-way: a published API, file format, persisted data, protocol or other contract that cannot be taken back. two-way: can be reverted freely."
+	},
+	{
+		id: `${note.id}.blastRadius`,
+		primitive: "choice",
+		instructions: `How far does the change in note "${note.id}" reach?`,
+		criteria: "local: one function or file. module: a whole module or several files. contract: a public interface other code or users depend on."
+	},
+	{
+		id: `${note.id}.newConcept`,
+		primitive: "noul",
+		instructions: `Does the change in note "${note.id}" introduce a concept new to this codebase — a new abstraction, entity or term? Judge from the evidence, never a glossary; reusing or extending an existing concept is not new.`,
+		criteria: "yes: a genuinely new concept. no: reuses or extends an existing one."
+	}
+];
+/**
+* One judge rest: a verdict per unforced note (a forced note is asked
+* nothing) plus four severity answers. Dismissing a note is the risky
 * direction, so only a confident non-`edit` verdict on uncut evidence counts;
-* a cut, unanswered or below-floor note is an `edit`, and a blank floor makes
-* every note one.
+* a cut, unanswered or below-floor note is an `edit`. Severity answers on a
+* non-`edit` note are ignored. A confident `could` edit becomes `defer`.
 */
-const triage$1 = async (notes) => {
-	const { answers, truncated } = await judge("review.triage", {
-		questions: notes.map(verdictQuestion),
-		evidence: Object.fromEntries(notes.map((n) => [n.id, `Anchor: ${n.anchor}\nReviewer's text: ${n.before}\nHuman's text: ${n.text}`])),
+const triage$1 = (severities) => async (notes) => {
+	const asked = notes.filter((n) => !n.forced);
+	const judgment = asked.length === 0 ? {
+		answers: {},
+		truncated: []
+	} : await judge("review.triage", {
+		questions: asked.flatMap((n) => [verdictQuestion(n), ...severityQuestions(n)]),
+		evidence: Object.fromEntries(asked.map((n) => [n.id, `Anchor: ${n.anchor}\nReviewer's text: ${n.before}\nHuman's text: ${n.text}`])),
 		message: buildReviewTriageMessage(),
 		label: "Judging each note"
 	});
+	const { answers, truncated } = judgment;
 	const minP = numeric(vars.reviewNoteActionable, Infinity);
+	const severityMinP = numeric(vars.reviewSeverityMinP, Infinity);
+	severities.clear();
 	return new Map(notes.map((n) => {
-		const verdict = VERDICTS.find((v) => v !== "edit" && answered(answers[n.id], v, minP));
-		return [n.id, truncated.includes(n.id) || verdict === void 0 ? "edit" : verdict];
+		const verdict = n.forced ? void 0 : VERDICTS.find((v) => v !== "edit" && answered(answers[n.id], v, minP));
+		if (verdict !== void 0 && !truncated.includes(n.id)) return [n.id, verdict];
+		const severity = severityOf(n, judgment, severityMinP);
+		severities.set(n.id, severity);
+		return [n.id, severity.tier === "could" ? "defer" : "edit"];
 	}));
 };
 const isCode = (path) => path !== ".gtd" && !path.startsWith(".gtd/");
@@ -220,32 +288,61 @@ const isCode = (path) => path !== ".gtd" && !path.startsWith(".gtd/");
 * another planning lap, questions are answered in place, nits fixed in one
 * batch, praise dropped.
 */
-const review = (base, escalations = { rounds: 0 }, frozen) => review$1(base, {
-	file: REVIEW,
-	feedback: REQUIREMENTS,
-	isCode,
-	reviewing,
-	fixRisks: (risks) => guarded(frozen, () => fixRisks(risks), "review")(),
-	keepGreen: () => healthy(guarded(frozen, fixCheck, "review"), { escalations }),
-	awaitReview,
-	reviewMissing,
-	collecting,
-	triage: triage$1,
-	answerQuestions: answerReviewQuestions,
-	fixNits: (notes) => guarded(frozen, () => fixNits(notes), "review")(),
-	close: closeReview,
-	captures: {
-		edits: reviewEditsCapture,
-		notes: reviewNotesCapture,
-		editNotes: reviewEditNotesCapture
-	},
-	refusals: {
-		deleted: "gtd land: review-doc: .gtd/REVIEW.md was deleted — restore it, or leave a note (or edit code) to request changes.",
-		unmatched: "gtd land: no declared pattern matches the pending changes — write .gtd/REQUIREMENTS.md from the feedback, or change nothing when it asks for nothing"
-	}
-});
-/** The build tail: fix (when entered red), keep green, the quality lap, then human review since `base`. */
-const buildTail = (fixFirst, base, built) => tail(fixFirst, (escalations, frozen) => review(base, escalations, frozen), built);
+const review = (base, escalations = { rounds: 0 }) => {
+	const severities = /* @__PURE__ */ new Map();
+	let deferred = [];
+	return review$1(base, {
+		file: REVIEW,
+		feedback: REQUIREMENTS,
+		isCode,
+		reviewing: (b, carry) => {
+			deferred = [];
+			return reviewing(b, carry);
+		},
+		fixRisks,
+		keepGreen: () => healthy(fixCheck, { escalations }),
+		awaitReview: (b) => {
+			const notes = deferred;
+			deferred = [];
+			return notes.length > 0 ? awaitReview(b, buildReviewAwaitAfterDeferMessage(b, notes), true) : awaitReview(b);
+		},
+		reviewMissing,
+		collecting,
+		triage: triage$1(severities),
+		defer: (notes) => {
+			deferred = notes.map((n) => ({
+				id: n.id,
+				anchor: n.anchor,
+				text: n.text,
+				label: severities.get(n.id).reason
+			}));
+			return deferNotes(deferred, head());
+		},
+		answerQuestions: answerReviewQuestions,
+		fixNits,
+		close: closeReview,
+		captures: {
+			edits: reviewEditsCapture,
+			notes: reviewNotesCapture,
+			editNotes: (round, edits, answeredAt) => reviewEditNotesCapture(round, edits.map((n, i) => ({
+				n,
+				i,
+				s: severities.get(n.id)
+			})).sort((x, y) => Number(x.s?.tier === "should") - Number(y.s?.tier === "should") || (x.s?.rank ?? 2) - (y.s?.rank ?? 2) || x.i - y.i).map(({ n, s }) => ({
+				id: n.id,
+				anchor: n.anchor,
+				text: n.forced ? n.text.replace(/^!\s*/, "") : n.text,
+				...s === void 0 ? {} : { label: s.reason }
+			})), answeredAt)
+		},
+		refusals: {
+			deleted: "gtd land: review-doc: .gtd/REVIEW.md was deleted — restore it, or leave a note (or edit code) to request changes.",
+			unmatched: "gtd land: no declared pattern matches the pending changes — write .gtd/REQUIREMENTS.md from the feedback, or change nothing when it asks for nothing"
+		}
+	});
+};
+/** The build tail: fix (when entered red), keep green unless `green`, the quality lap, then human review since `base`. */
+const buildTail = (fixFirst, base, green = false) => tail(fixFirst, (escalations) => review(base, escalations), green);
 //#endregion
 //#region src/workflows/feature/feature.ts
 /** Revert the sketch that started the process out of the working tree; its intent survives in history. */
@@ -272,13 +369,11 @@ const reUnwind = async (feedback) => {
 /** Plan, build and review until a review round signs off; feedback re-plans from scratch. */
 const planAndBuild = async (firstBase) => {
 	let base = firstBase;
-	let carried;
 	for (;;) {
 		await design(base);
 		await architecture();
-		const built = await packages(carried);
-		carried = built.frozen;
-		const outcome = await buildTail(false, base, built);
+		await packages();
+		const outcome = await buildTail(false, base, true);
 		if (outcome.verdict === "signoff") return;
 		await reUnwind(outcome);
 		base = outcome.base;
@@ -312,4 +407,4 @@ const feature = shipped(ordinaryStart);
 //#region src/workflows/feature/index.ts
 var feature_default = feature;
 //#endregion
-export { SCENARIO_PACKAGE, access, afterTail, answerProductQuestions, answerReviewQuestions, answerTechnicalQuestions, architecture, author, build, buildTail, collecting, declaredTests, decompose, feature_default as default, defaults, design, envDefaults, feature, fixNits, fixSuite, nextPackage, ordinaryStart, packageItem, packages, planAndBuild, reUnwind, requireDeclaredTests, skills, steering, summary, triage, unwind };
+export { access, afterTail, answerProductQuestions, answerReviewQuestions, answerTechnicalQuestions, architecture, author, build, buildTail, collecting, declaredTests, decompose, feature_default as default, defaults, deferNotes, design, envDefaults, feature, fixNits, fixSuite, nextPackage, ordinaryStart, packageItem, packages, planAndBuild, reUnwind, requireDeclaredTests, skills, steering, summary, triage, unwind };

package/dist/workflows/fix.mjs

@@ -1,6 +1,6 @@
 import { read, refuse, removeScript, run, scope, start } from "../flows.mjs";
-import { d as reviewNotes, f as reviewRisks } from "../chunks/steering-D-PvRC5W.mjs";
-import { H as fixReviewNotesPrompt, O as agentWithSkills, S as coder, V as finisherSystem, a as REVIEW_ORIG, b as REVIEW, d as shipped, f as defaults, i as skills, m as baseline, n as summary, p as envDefaults, r as access, s as handOffTail, t as steering } from "../chunks/reserved-DJwU5C4z.mjs";
+import { d as reviewNotes, f as reviewRisks } from "../chunks/steering-DVILanVf.mjs";
+import { D as agentWithSkills, H as finisherSystem, U as fixReviewNotesPrompt, a as REVIEW_ORIG, d as defaults, f as envDefaults, i as skills, n as summary, p as baseline, r as access, s as handOffTail, t as steering, u as shipped, x as coder, y as REVIEW } from "../chunks/reserved-6pAgFGgB.mjs";
 //#region src/workflows/fix/fix.ts
 /**
 * Fix every note a human left in a handed-off `.gtd/REVIEW.md` and every

package/dist/workflows/review.mjs

@@ -1,5 +1,5 @@
 import { start } from "../flows.mjs";
-import { J as reviewGateBlockedMessage, c as refusePendingHandOff, d as shipped, f as defaults, h as gate, i as skills, n as summary, p as envDefaults, r as access, s as handOffTail, t as steering } from "../chunks/reserved-DJwU5C4z.mjs";
+import { Y as reviewGateBlockedMessage, c as refusePendingHandOff, d as defaults, f as envDefaults, i as skills, m as gate, n as summary, r as access, s as handOffTail, t as steering, u as shipped } from "../chunks/reserved-6pAgFGgB.mjs";
 //#region src/workflows/review/review.ts
 /**
 * Pure review of everything since `reviewBase`: a green gate, the quality