@pmelab/gtd 37.0.0 → 38.0.0

package/dist/workflows.mjs

@@ -16934,6 +16934,7 @@ const defaults = {
 	refactorFlagPattern: "\\bGTD_FLAG_\\w+",
 	seamAllow: [...testFileGlobs, ".gtd/**"].join(", "),
 	flagMode: "true",
+	sliceMaxCriteria: "8",
 	flag_risk_tier: "0",
 	qualityReviews: "correctness, owasp-security, ponytail-review, test-audit, conventions, spec-challenge",
 	refactorQualityReviews: "correctness, ponytail-review, conventions"
@@ -17393,6 +17394,24 @@ printf '%s\\n' ${[
 	].map(quote).join(" ")} > ${quote(".gtd/REQUIREMENTS.md")}
 `;
 };
+const slicingPrompt = (cap) => `${stateFileRules}
+- Read \`.gtd/REQUIREMENTS.md\`: its \`## Acceptance Criteria\` are settled
+  and more than ${cap}, too many to build in one slice
+- Group them into slices in \`.gtd/SLICES.md\`, one \`## <title>\` heading per
+  slice over a \`- AC-<n>\` list item per criterion it builds. Put criteria of
+  the same flow or subsystem together; every criterion in exactly one slice,
+  at most ${cap} per slice
+- Keep it coarse: as few slices as the cap allows, never one per criterion
+- Order the slices so each builds on the ones before: they run in that order,
+  each from where the previous one landed
+- Change nothing but \`.gtd/SLICES.md\` — leave the criteria as they are
+- Leave everything uncommitted and finish your turn — do not commit
+`;
+/** The line telling a slice's turn which criteria are its own; empty for a single slice. */
+const sliceLine = (ids) => ids === void 0 ? "" : `- This slice builds ${ids.join(", ")} only: every rule here about the criteria
+  means these. The others are for other slices, before or after it — leave
+  them unbuilt and untested
+`;
 const designSystem = () => `${designPersona}
 
 ${agentConduct}`;
@@ -18039,9 +18058,9 @@ re-implement. It changed these files outside the expected set:
 ${listed(retry.strays)}
 Start over and choose a different approach; do not rebuild the same diff.
 `;
-const naivePassPrompt = (retry) => `${stateFileRules}
+const naivePassPrompt = (retry, slice) => `${stateFileRules}
 - Read \`.gtd/REQUIREMENTS.md\`: the settled requirements for this change
-${criteriaBinding(false)}
+${sliceLine(slice)}${criteriaBinding(false)}
 - Implement them the quickest way that works — the naive version. Do not
   restructure, abstract or refactor around the change; it may land exactly as
   you leave it, and a reviewer makes it fit
@@ -18091,9 +18110,13 @@ const afterRefactorLine = (rebuild) => `- A reviewed refactor has just landed an
   though never a release flag. Build only what is still missing, held to every
   rule below; if nothing is, ${carries(rebuild) ? "carry over the acceptance tests below and change nothing else" : "change nothing and finish your turn"}
 `;
-const implementBuildingPrompt = (reference, tier, amend, afterRefactor, flagMode = true, rebuild = {}) => `${stateFileRules}
+const flagLine = (flagMode, tier, amend, sliced) => !flagMode ? "- Add no release flag — this project ships without release flags" : tier === "low" ? "- Add no release flag — this change is low risk tier" : amend === true ? "- Keep the release flag the earlier lap added, if it added one; otherwise put\n  the behaviour behind a new release flag" : sliced ? `- Put the new behaviour behind the release flag an earlier slice of this
+  change added; only when none did, behind a new one, following the
+  repository's flag conventions` : `- Put the new behaviour behind a new release flag, following the
+  repository's flag conventions`;
+const implementBuildingPrompt = (reference, tier, amend, afterRefactor, flagMode = true, rebuild = {}, slice) => `${stateFileRules}
 - Read \`.gtd/REQUIREMENTS.md\`: the settled requirements for this change
-${criteriaBinding(true)}
+${sliceLine(slice)}${criteriaBinding(true)}
 ${afterRefactor === true ? afterRefactorLine(rebuild) : ""}${amend === true ? `- The change already exists in the tree from an earlier lap; amend it to meet
   the re-derived requirements instead of rewriting it
 ` : flagMode ? `- Implement them properly, fitting the codebase's seams and conventions
@@ -18102,8 +18125,7 @@ ${afterRefactor === true ? afterRefactorLine(rebuild) : ""}${amend === true ? `-
 `}${reference === void 0 || amend === true ? "" : `- Reference only: \`git show ${reference}\` shows an earlier naive attempt.
   Read it for ideas; never apply, cherry-pick or check it out — write
   the change fresh${carries(rebuild) ? ", its acceptance tests excepted (below)" : ""}
-`}${rebuildLines(rebuild)}${!flagMode ? "- Add no release flag — this project ships without release flags" : tier === "low" ? "- Add no release flag — this change is low risk tier" : amend === true ? "- Keep the release flag the earlier lap added, if it added one; otherwise put\n  the behaviour behind a new release flag" : `- Put the new behaviour behind a new release flag, following the
-  repository's flag conventions`}
+`}${rebuildLines(rebuild)}${flagLine(flagMode, tier, amend, slice !== void 0)}
 - Write the tests first, then the code that makes them pass. Name each
   criterion's ID in the test that covers it (\`[AC-3] …\`): once the suite is
   green a judge checks every ID has a passing test, and sends uncovered ones back
@@ -18184,9 +18206,9 @@ const coverageMessage = () => `Judging acceptance-criteria coverage. Run \`gtd j
 \`yes\`/\`no\` verdict per criterion ID — or land without a verdict, which
 counts every criterion as uncovered.
 `;
-const coveringPrompt = (uncovered) => `${stateFileRules}
+const coveringPrompt = (uncovered, slice) => `${stateFileRules}
 - Read \`.gtd/REQUIREMENTS.md\`: the settled requirements for this change
-${criteriaBinding(true)}
+${sliceLine(slice)}${criteriaBinding(true)}
 - A judge found no passing test in this change that covers ${uncovered.join(", ")}.
   Write a behaviour-level test for each, naming its ID in the test or scenario
   name (\`[${uncovered[0]}] …\`), and fix the code if the test shows it falls short
@@ -18213,6 +18235,7 @@ const REVIEW = ".gtd/REVIEW.md";
 const REFACTOR = ".gtd/REFACTOR.md";
 const REFACTOR_NOTE = ".gtd/REFACTOR-NOTE.md";
 const QUALITY = ".gtd/QUALITY.md";
+const SLICES = ".gtd/SLICES.md";
 const planner = () => env.plannerModel ?? "";
 const coder = () => env.coderModel ?? "";
 const fixCheck = (rules) => agentWithSkills("fix", buildFixPrompt(rules), {
@@ -37724,6 +37747,7 @@ const ShippedOptions = Struct({
 	reviewSeverityMinP: setting,
 	refactorFlagPattern: setting,
 	seamAllow: setting,
+	sliceMaxCriteria: setting,
 	qualityReviews: setting,
 	refactorQualityReviews: setting,
 	testCommand: setting,
@@ -37904,13 +37928,15 @@ const markFenced = (text) => {
 		};
 	});
 };
+/** A markdown heading: its `#` run, then its text. */
+const HEADING = /^ {0,3}(#{1,6})\s+(.*?)\s*#*\s*$/;
+/** A list item, its checkbox dropped: its text. */
+const ITEM = /^ ?(?:[-*+]|\d+[.)])\s+(?:\[[ xX]\]\s+)?(.*)$/;
 //#endregion
 //#region src/workflows/feature/criteria.ts
 /** Below this the judge's `yes` is not trusted: a wrong skip builds against criteria nobody can check. */
 const CLEAR_MIN_P = .8;
-const HEADING = /^ {0,3}(#{1,6})\s+(.*?)\s*#*\s*$/;
-const ITEM = /^ ?(?:[-*+]|\d+[.)])\s+(?:\[[ xX]\]\s+)?(.*)$/;
-const ID = /^(AC-\d+):\s*(.*)$/;
+const ID$1 = /^(AC-\d+):\s*(.*)$/;
 /** The list items under the first heading `title` matches (at `level` when given), up to the next heading; wrapped lines joined, fenced lines ignored. */
 const listUnder = (text, title, level) => {
 	const lines = markFenced(text);
@@ -37938,7 +37964,7 @@ const listUnder = (text, title, level) => {
 };
 /** The `- AC-<n>: …` list of `## Acceptance Criteria`, or `undefined` when the section is missing. */
 const criteriaOf = (text) => listUnder(text, /^acceptance criteria$/i, 2)?.map((item) => {
-	const id = ID.exec(item);
+	const id = ID$1.exec(item);
 	return id === null ? {
 		id: "",
 		text: item
@@ -37947,16 +37973,20 @@ const criteriaOf = (text) => listUnder(text, /^acceptance criteria$/i, 2)?.map((
 		text: id[2]
 	};
 });
+/** The ID'd criteria of `text`, only `slice`'s when given. */
+const criterionIds = (text, slice) => (criteriaOf(text) ?? []).map((c) => c.id).filter((id) => id !== "" && (slice?.only === void 0 || slice.only.includes(id)) && !(slice?.except ?? []).includes(id));
+/** `slice`'s IDs in `.gtd/REQUIREMENTS.md` as it is now; `undefined` for an unsliced change. */
+const sliceIds = (slice) => slice === void 0 ? void 0 : criterionIds(read(".gtd/REQUIREMENTS.md") ?? "", slice);
 /** Items numbered `AC-1`…, unless every one already carries its own distinct ID. */
 const numbered = (items) => {
-	const ids = items.map((item) => ID.exec(item));
+	const ids = items.map((item) => ID$1.exec(item));
 	const own = ids.every((id) => id !== null) && new Set(ids.map((id) => id[1])).size === ids.length;
 	return items.map((item, i) => own ? {
 		id: ids[i][1],
 		text: ids[i][2]
 	} : {
 		id: `AC-${i + 1}`,
-		text: item.replace(ID, "$2")
+		text: item.replace(ID$1, "$2")
 	});
 };
 const STATED = /^acceptance criteria:?$/i;
@@ -38646,10 +38676,12 @@ const green = (guard) => healthy(() => agentWithSkills("fix.suite.fixing", imple
 const testsSince = (base) => changesSince(base).filter((c) => c.after !== void 0 && isCode$1(c.path) && isTestFile(c.path)).sort((a, b) => Number(/\bAC-\d/.test(a.after)) - Number(/\bAC-\d/.test(b.after))).map((c) => `=== ${c.path} (${c.status})\n${c.after}`).join("\n");
 /**
 * The criterion IDs of `.gtd/REQUIREMENTS.md` — read afresh, never carried
-* over — no passing test since `base` covers, by the judge.
+* over, only `slice`'s when given — no passing test since `base` covers, by the judge.
 */
-const uncoveredSince = async (base) => {
-	const criteria = (criteriaOf(read(".gtd/REQUIREMENTS.md") ?? "") ?? []).filter((c) => c.id !== "");
+const uncoveredSince = async (base, slice) => {
+	const requirements = read(".gtd/REQUIREMENTS.md") ?? "";
+	const own = criterionIds(requirements, slice);
+	const criteria = (criteriaOf(requirements) ?? []).filter((c) => own.includes(c.id));
 	if (criteria.length === 0) return {
 		covered: [],
 		uncovered: []
@@ -38676,14 +38708,14 @@ const record = (covered, uncovered = []) => {
 * sending uncovered IDs back to a covering turn; marks the IDs covered on the
 * landing that ends it. A human may overrule once the retries are spent.
 */
-const requireCoverage = async (base = start()) => {
+const requireCoverage = async (base = start(), slice) => {
 	let retries = 0;
 	for (;;) {
-		const { covered, uncovered } = await uncoveredSince(base);
+		const { covered, uncovered } = await uncoveredSince(base, slice);
 		if (uncovered.length === 0) return record(covered);
 		if (retries < COVERING_RETRIES) {
 			retries++;
-			await agentWithSkills("covering", coveringPrompt(uncovered), {
+			await agentWithSkills("covering", coveringPrompt(uncovered, sliceIds(slice)), {
 				label: "Covering the acceptance criteria",
 				model: coder(),
 				system: builderSystem(),
@@ -38714,7 +38746,7 @@ const judgeSeam = (tip) => judgeDiffShape("judge", {
 	allow: (vars.seamAllow ?? defaults.seamAllow).split(",").map((glob) => glob.trim()).filter((glob) => glob !== ""),
 	exempt: tip.changes.filter((c) => c.status === "added").map((c) => c.path)
 });
-const naivePass = (retry) => agentWithSkills("naive", naivePassPrompt(retry), {
+const naivePass = (retry, slice) => agentWithSkills("naive", naivePassPrompt(retry, sliceIds(slice)), {
 	label: retry === void 0 ? "Writing the naive pass" : "Rewriting the naive pass",
 	keep: true,
 	model: coder(),
@@ -38755,7 +38787,7 @@ const addedLines = (change) => {
 	return change.after.split("\n").filter((line) => !before.has(line)).join("\n");
 };
 const acceptanceTests = (changes) => changes.filter((c) => c.after !== void 0 && isCode$1(c.path) && isTestFile(c.path));
-const idsOf = (file) => (criteriaOf(read(file) ?? "") ?? []).map((c) => c.id).filter((id) => id !== "");
+const idsOf = (file, slice) => criterionIds(read(file) ?? "", slice);
 const unnamed = (ids, tests) => {
 	const lines = tests.map(addedLines);
 	return ids.filter((id) => !lines.some((text) => named(id, text)));
@@ -38765,9 +38797,9 @@ const unnamed = (ids, tests) => {
 * test line of the pass names, or a test importing a file the pass added —
 * naive structure a reset and refactor replaces, which the tests must survive.
 */
-const acceptanceFault = (file, tip) => {
+const acceptanceFault = (file, tip, slice) => {
 	const tests = acceptanceTests(tip.changes);
-	const untested = unnamed(idsOf(file), tests);
+	const untested = unnamed(idsOf(file, slice), tests);
 	if (untested.length > 0) return `acceptance-tests: no test of the naive pass names ${untested.join(", ")} — write a behaviour-level test for each criterion with its ID in the test or scenario name, e.g. "[${untested[0]}] …"`;
 	const added = tip.changes.filter((c) => c.status === "added" && isCode$1(c.path) && !isTestFile(c.path)).map((c) => c.path);
 	const reaching = tests.filter((test) => holdsCallSites(test.path)).flatMap((test) => {
@@ -38777,8 +38809,8 @@ const acceptanceFault = (file, tip) => {
 	if (reaching.length > 0) return `acceptance-internals: these tests import files the naive pass added — test through code that existed before it (a public entry point, an existing module's exports):\n${reaching.join("\n")}`;
 };
 /** What a rebuild carries over from the naive pass `tip`: the IDs its tests name, and those test files. */
-const naiveAcceptance = (tip) => {
-	const ids = idsOf(REQUIREMENTS);
+const naiveAcceptance = (tip, slice) => {
+	const ids = idsOf(REQUIREMENTS, slice);
 	const perTest = acceptanceTests(tip.changes).map((c) => {
 		const lines = addedLines(c);
 		return {
@@ -38857,7 +38889,7 @@ function implement(first, tier, amend) {
 		const since = head$1();
 		const acceptance = options.rebuild?.acceptance;
 		const built = options.naive === void 0 || !await applied(options.naive);
-		if (built) await agentWithSkills("building", implementBuildingPrompt(options.reference, options.tier, options.amend, options.afterRefactor, flagModeOn(), options.rebuild), {
+		if (built) await agentWithSkills("building", implementBuildingPrompt(options.reference, options.tier, options.amend, options.afterRefactor, flagModeOn(), options.rebuild, sliceIds(options.slice)), {
 			label: "Implementing",
 			model: coder(),
 			system: builderSystem(),
@@ -38866,10 +38898,18 @@ function implement(first, tier, amend) {
 		requireCriteriaFrozen(REQUIREMENTS);
 		if (built && acceptance !== void 0) requireAcceptance(acceptance, since);
 		await green(built && acceptance !== void 0 ? () => requireAcceptanceKept(acceptance) : void 0);
-		await requireCoverage();
-		await run("closing", removeScript([REQUIREMENTS, REFACTOR_NOTE]), { label: "Closing out the implementation" });
+		await requireCoverage(options.base, options.slice);
+		await closing(options.more === true);
 	});
 }
+/** The last slice's closing, for a last slice a steer left with no criterion to build. */
+const closeOut = () => scope("implement", () => closing(false));
+/** A slice with `more` after it keeps the criteria and the slicing for the next one. */
+const closing = (more) => run("closing", removeScript(more ? [REFACTOR_NOTE] : [
+	REQUIREMENTS,
+	REFACTOR_NOTE,
+	SLICES
+]), { label: "Closing out the implementation" });
 /** Resolves `true` when the human steered (edited), `false` on a clean overrule. */
 const steer = async (retry) => {
 	await human("steer", {
@@ -38900,8 +38940,8 @@ const budget = () => {
 	};
 };
 /** Sends a pass whose tests fall short or run red back (`undefined`: it goes on, an overruled one as it is). */
-const vetted = async (tip, vetting) => {
-	const fault = vetting.waived ? void 0 : acceptanceFault(REQUIREMENTS, tip);
+const vetted = async (tip, vetting, slice) => {
+	const fault = vetting.waived ? void 0 : acceptanceFault(REQUIREMENTS, tip, slice);
 	if (fault !== void 0) {
 		const verdict = await vetting.again({
 			fault,
@@ -38935,16 +38975,16 @@ const ruled = async (tip, again) => {
 		tip
 	} : next;
 };
-const decide = () => scope("seam", async () => {
+const decide = (slice) => scope("seam", async () => {
 	const vetting = {
 		again: budget(),
 		waived: false
 	};
 	let retry;
 	for (;;) {
-		await naivePass(retry);
+		await naivePass(retry, slice);
 		const tip = kept("naive");
-		const outcome = await vetted(tip, vetting) ?? (flagModeOn() ? await ruled(tip, vetting.again) : {
+		const outcome = await vetted(tip, vetting, slice) ?? (flagModeOn() ? await ruled(tip, vetting.again) : {
 			reference: tip.hash,
 			tip
 		});
@@ -38965,8 +39005,8 @@ const refactorNote = async (reference, strays) => {
 	if (others.length > 0 || note.trim() === "") refuse(`gtd land: refactor-note: write ${REFACTOR_NOTE} and change nothing else${others.length > 0 ? ` — revert these:\n${others.map((path) => `  - ${path}`).join("\n")}` : ""}`);
 	return note;
 };
-const seamBuild = async (base) => {
-	const { reference, tip, shape } = await decide();
+const seamBuild = async (base, slice$1 = {}) => {
+	const { reference, tip, shape } = await decide(slice$1.criteria);
 	if (shape === void 0) return {
 		reference,
 		base,
@@ -38981,17 +39021,17 @@ const seamBuild = async (base) => {
 		return {
 			reference,
 			base,
-			rebuild: { acceptance: naiveAcceptance(tip) }
+			rebuild: { acceptance: naiveAcceptance(tip, slice$1.criteria) }
 		};
 	}
 	const strays = refactorStrays(shape);
 	if (strays.length === 0) return {
 		reference,
 		base,
-		rebuild: { acceptance: naiveAcceptance(tip) }
+		rebuild: { acceptance: naiveAcceptance(tip, slice$1.criteria) }
 	};
 	const note = await scope("seam", () => refactorNote(reference, strays));
-	const refactored = await slice("refactor", () => reviewedRefactor(shape, {
+	const refactored = await slice(slice$1.refactor ?? "refactor", () => reviewedRefactor(shape, {
 		reference,
 		note
 	}));
@@ -39001,34 +39041,143 @@ const seamBuild = async (base) => {
 		refactored,
 		rebuild: {
 			note,
-			acceptance: naiveAcceptance(tip)
+			acceptance: naiveAcceptance(tip, slice$1.criteria)
 		}
 	};
 };
 //#endregion
+//#region src/workflows/feature/slicing.ts
+const ID = /^(AC-\d+)(?![\w-])/;
+/** The `sliceMaxCriteria` setting. Anything but a positive integer fails rather than guess a cap. */
+const sliceCap = () => {
+	const value = vars.sliceMaxCriteria ?? defaults.sliceMaxCriteria;
+	if (!/^[1-9]\d*$/.test(value)) refuse(`gtd: sliceMaxCriteria must be a positive integer, got ${JSON.stringify(value)}`);
+	return Number(value);
+};
+/** Each `## ` heading of `text` with the list items under it; an item not led by an ID keeps an empty one. */
+const slicesOf = (text) => {
+	const slices = [];
+	let open = false;
+	for (const { line, fenced } of markFenced(text)) {
+		if (fenced) continue;
+		const heading = HEADING.exec(line);
+		if (heading !== null) {
+			open = heading[1] === "##";
+			if (open) slices.push({
+				title: heading[2],
+				ids: []
+			});
+			continue;
+		}
+		const item = ITEM.exec(line);
+		if (item !== null && open) slices.at(-1).ids.push(ID.exec(item[1])?.[1] ?? "");
+	}
+	return slices;
+};
+const twice = (list) => [...new Set(list.filter((id, i) => list.indexOf(id) !== i))];
+/** Why `slices` fail to put each of `ids` in exactly one slice of at most `cap`, or `undefined`. */
+const sliceFault = (slices, ids, cap) => {
+	const listed = slices.flatMap((s) => s.ids);
+	if (slices.length === 0) return `no "## " slice`;
+	const empty = slices.filter((s) => s.ids.length === 0).map((s) => s.title);
+	if (empty.length > 0) return `a slice with no criterion: ${empty.join(", ")}`;
+	if (listed.includes("")) return "a list item not led by a criterion ID";
+	const over = slices.filter((s) => s.ids.length > cap).map((s) => s.title);
+	if (over.length > 0) return `more than ${cap} criteria in: ${over.join(", ")}`;
+	const repeated = twice(listed);
+	if (repeated.length > 0) return `an ID in more than one slice: ${repeated.join(", ")}`;
+	const unknown = listed.filter((id) => !ids.includes(id));
+	if (unknown.length > 0) return `an ID ${REQUIREMENTS} does not list: ${unknown.join(", ")}`;
+	const missing = ids.filter((id) => !listed.includes(id));
+	if (missing.length > 0) return `no slice for: ${missing.join(", ")}`;
+};
+/** Refuse a turn whose `.gtd/SLICES.md` does not put each of `ids` in exactly one slice of at most `cap`. */
+const requireSlices = (ids, cap) => {
+	const slices = slicesOf(read(".gtd/SLICES.md") ?? "");
+	const why = sliceFault(slices, ids, cap);
+	if (why !== void 0) refuse(`gtd land: slices: ${SLICES} has ${why} — list each slice as a "## <title>" heading over "- AC-<n>" items, every criterion of ${REQUIREMENTS} in exactly one slice, at most ${cap} per slice`);
+	return slices;
+};
+/**
+* The slices to build, in order. Criteria within `sliceMaxCriteria` are one
+* slice and no turn; above it `design.slicing` groups them.
+*/
+const slicing = async () => {
+	const all = criterionIds(read(".gtd/REQUIREMENTS.md") ?? "");
+	const cap = sliceCap();
+	if (all.length <= cap) return [{
+		title: "",
+		ids: all
+	}];
+	return scope("design", async () => {
+		await agentWithSkills("slicing", slicingPrompt(cap), {
+			label: "Slicing the change",
+			file: SLICES,
+			model: planner(),
+			system: designSystem()
+		});
+		return requireSlices(all, cap);
+	});
+};
+/** Slice `i`'s criteria; `undefined` when `slices` is the one slice of an unsliced change. */
+const sliceCriteria = (slices, i) => slices.length === 1 ? void 0 : i < slices.length - 1 ? { only: slices[i].ids } : { except: slices.slice(0, -1).flatMap((s) => s.ids) };
+//#endregion
 //#region src/workflows/feature/feature.ts
 /** Revert the sketch that started the process out of the working tree; its intent survives in history. */
 const unwind = () => {
 	const commit = head$1();
 	return run("unwind", revertScript(commit, FEEDBACK, unwindFailure(commit)), { label: "Unwinding your input" });
 };
-/** A naive pass (judged only with `flagMode` on), then implement and the quality lap; design must have run since `firstBase`. */
+/**
+* The slices `design.slicing` grouped the criteria into, each built in turn
+* from where the previous one landed; design must have run since `firstBase`.
+*/
 const buildFeature = async (firstBase) => {
 	const tier = riskTierOn() ? recordRisk() : void 0;
-	const seam = await seamBuild(firstBase);
-	await slice("feature", async () => {
+	const slices = await slicing();
+	const one = slices.length === 1;
+	for (const i of slices.keys()) {
+		const more = i < slices.length - 1;
+		const criteria = sliceCriteria(slices, i);
+		if (sliceIds(criteria)?.length === 0) {
+			if (!more) await closeOut();
+			continue;
+		}
+		const n = one ? "" : `-${i + 1}`;
+		await buildSlice(i === 0 ? firstBase : head$1(), {
+			tier,
+			criteria,
+			refactor: `refactor${n}`,
+			feature: `feature${n}`,
+			more
+		});
+	}
+};
+/** A naive pass (judged only with `flagMode` on), then implement and the quality lap. */
+const buildSlice = async (sliceBase, build) => {
+	const seam = await seamBuild(sliceBase, {
+		criteria: build.criteria,
+		refactor: build.refactor
+	});
+	await slice(build.feature, async () => {
 		await implement({
 			reference: seam.reference,
-			tier,
+			tier: build.tier,
 			afterRefactor: seam.refactored,
 			naive: seam.land === true ? seam.reference : void 0,
-			rebuild: seam.rebuild
+			rebuild: seam.rebuild,
+			slice: build.criteria,
+			base: build.criteria === void 0 ? void 0 : sliceBase,
+			more: build.more
 		});
 		const carried = seam.rebuild?.acceptance?.tests ?? [];
-		const base = seam.refactored === true && !changesSince(seam.base).some((c) => isCode$1(c.path) && !carried.includes(c.path)) ? firstBase : seam.base;
+		const base = seam.refactored === true && !changesSince(seam.base).some((c) => isCode$1(c.path) && !carried.includes(c.path)) ? sliceBase : seam.base;
+		const guard = build.more ? () => requireCriteriaFrozen(REQUIREMENTS) : void 0;
 		await tail(settleQuality, {
 			green: true,
-			lensBase: base
+			lensBase: base,
+			guard,
+			guardRests: false
 		});
 	});
 };
@@ -39074,6 +39223,7 @@ const ordinaryStart = async () => {
 	});
 	refusePendingHandOff();
 	flagModeOn();
+	sliceCap();
 	const stated = statedCriteria(changes(), head$1());
 	await unwind();
 	if (changes(".gtd/FEEDBACK.md").some((c) => c.status !== "deleted")) await human("unwind-failed", {
@@ -39268,4 +39418,4 @@ var workflows_default = feature;
 */
 const doors = {};
 //#endregion
-export { COMMENT, ESCALATION, FEEDBACK, FIX_CAP, FlagOptions, QUALITY, REFACTOR, REFACTOR_NOTE, REFACTOR_PACKAGE, REQUIREMENTS, REVIEW, REVIEW_ORIG, ShippedOptions, acceptanceFault, access, afterTail, agentWithSkills, answerProductQuestions, answerReviewQuestions, awaitReview, base, baseline, buildFeature, buildTail, builtInLenses, cleanup, cleanupFlow, closeReview, coder, collecting, criteriaOf, workflows_default as default, defaults, deferNotes, describeEscalation, design, doors, envDefaults, escalate, escalation, escalationExhausted, feature, fix, fixCheck, fixFlow, fixNits, fixQuality, fixQualityFindings, fixReviewNotes, fixRisks, gate, guarded, handOffReview, handOffTail, healthy, holdsCallSites, implement, isTestFile, judgeSeam, lensEntriesOf, lensOf, lensesOf, naiveAcceptance, naivePass, ordinaryStart, planAndBuild, planner, qualityLap, reUnwind, refactorCollecting, refactorFirst, refactorNote, refactorStrays, requireAcceptance, requireAcceptanceKept, requireCriteria, requireCriteriaFrozen, requireNoNewFlags, requireNoTestFiles, requireTestCallSitesOnly, requireWithin, retryQuestion, review, reviewFlow, reviewMissing, reviewQuality, reviewedRefactor, reviewing, rollout, rolloutFlow, seamBuild, skills, skipDesign, statedCriteria, steering, summary, tail, threads, triage, unwind };
+export { COMMENT, ESCALATION, FEEDBACK, FIX_CAP, FlagOptions, QUALITY, REFACTOR, REFACTOR_NOTE, REFACTOR_PACKAGE, REQUIREMENTS, REVIEW, REVIEW_ORIG, SLICES, ShippedOptions, acceptanceFault, access, afterTail, agentWithSkills, answerProductQuestions, answerReviewQuestions, awaitReview, base, baseline, buildFeature, buildTail, builtInLenses, cleanup, cleanupFlow, closeOut, closeReview, coder, collecting, criteriaOf, criterionIds, workflows_default as default, defaults, deferNotes, describeEscalation, design, doors, envDefaults, escalate, escalation, escalationExhausted, feature, fix, fixCheck, fixFlow, fixNits, fixQuality, fixQualityFindings, fixReviewNotes, fixRisks, gate, guarded, handOffReview, handOffTail, healthy, holdsCallSites, implement, isTestFile, judgeSeam, lensEntriesOf, lensOf, lensesOf, naiveAcceptance, naivePass, ordinaryStart, planAndBuild, planner, qualityLap, reUnwind, refactorCollecting, refactorFirst, refactorNote, refactorStrays, requireAcceptance, requireAcceptanceKept, requireCriteria, requireCriteriaFrozen, requireNoNewFlags, requireNoTestFiles, requireSlices, requireTestCallSitesOnly, requireWithin, retryQuestion, review, reviewFlow, reviewMissing, reviewQuality, reviewedRefactor, reviewing, rollout, rolloutFlow, seamBuild, skills, skipDesign, sliceCap, sliceCriteria, sliceIds, slicesOf, slicing, statedCriteria, steering, summary, tail, threads, triage, unwind };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pmelab/gtd",
-  "version": "37.0.0",
+  "version": "38.0.0",
   "private": false,
   "description": "Git-aware CLI that emits the next prompt for an autonomous coding agent based on the current repository state",
   "bin": {

package/src/testkit/steps.ts

@@ -26,6 +26,8 @@ export class TestkitWorld extends QuickPickleWorld {
   /** Per step, what its scripted actor does, in the order the scenario declared it. */
   actions = new Map<string, Actor[]>()
   verdicts = new Map<string, string>()
+  /** Per judge step, a verdict for each text its questions may contain; the first match answers. */
+  conditionalVerdicts = new Map<string, [text: string, verdict: string][]>()
   private created: TestRepo | undefined = undefined
 
   // The repository is created on first use and keeps the config and
@@ -79,6 +81,19 @@ export class TestkitWorld extends QuickPickleWorld {
       }
       actors[step] = () => verdict
     }
+    for (const [step, choices] of this.conditionalVerdicts) {
+      if (this.actions.has(step) || this.verdicts.has(step)) {
+        throw new Error(`step "${step}" has a conditional verdict and another script`)
+      }
+      // Matched against the pending questions, which `gtd judge` prints and the beat does not.
+      actors[step] = async (_, repo) => {
+        const { stdout } = await repo.gtd("judge")
+        const hit = choices.find(([text]) => stdout.includes(text))
+        if (hit === undefined)
+          throw new Error(`judge step "${step}" matched no verdict:\n${stdout}`)
+        return hit[1]
+      }
+    }
     await this.repo().drive(actors)
   }
 }
@@ -137,6 +152,16 @@ Given(
   },
 )
 
+// Lets one step write differently per visit, told apart by what it is prompted with.
+Given(
+  "step {string}, when prompted with {string}, writes {string} with:",
+  (world: TestkitWorld, step: string, text: string, path: string, content: string) => {
+    world.act(step, (beat, repo) => {
+      if (beat.content.includes(text)) repo.write(path, withNewline(content))
+    })
+  },
+)
+
 // A standing assertion like any queued action: it applies on every later visit
 // to the step until `from now on` clears the list.
 Given(
@@ -169,6 +194,16 @@ Given("judge step {string} answers:", (world: TestkitWorld, step: string, verdic
   world.verdicts.set(step, String(verdict))
 })
 
+Given(
+  "judge step {string}, when its questions contain {string}, answers:",
+  (world: TestkitWorld, step: string, text: string, verdict: string) => {
+    world.conditionalVerdicts.set(step, [
+      ...(world.conditionalVerdicts.get(step) ?? []),
+      [text, String(verdict)],
+    ])
+  },
+)
+
 When("gtd starts workflow {string}", async (world: TestkitWorld, workflow: string) => {
   await world.repo().start(workflow)
 })

package/src/workflows/feature/coverage.ts

@@ -11,7 +11,13 @@ import {
 import { healthy } from "../shared/health.js"
 import { coder, FEEDBACK, REQUIREMENTS } from "../shared/steps.js"
 import * as t from "../shared/text.js"
-import { criteriaOf, requireCriteriaFrozen } from "./criteria.js"
+import {
+  criteriaOf,
+  criterionIds,
+  requireCriteriaFrozen,
+  sliceIds,
+  type CriteriaSlice,
+} from "./criteria.js"
 import { isTestFile } from "./refactor.js"
 import { isCode } from "./review.js"
 
@@ -51,13 +57,15 @@ const testsSince = (base: string): string =>
 
 /**
  * The criterion IDs of `.gtd/REQUIREMENTS.md` — read afresh, never carried
- * over — no passing test since `base` covers, by the judge.
+ * over, only `slice`'s when given — no passing test since `base` covers, by the judge.
  */
 const uncoveredSince = async (
   base: string,
+  slice: CriteriaSlice | undefined,
 ): Promise<{ readonly covered: string[]; readonly uncovered: string[] }> => {
   const requirements = read(REQUIREMENTS) ?? ""
-  const criteria = (criteriaOf(requirements) ?? []).filter((c) => c.id !== "")
+  const own = criterionIds(requirements, slice)
+  const criteria = (criteriaOf(requirements) ?? []).filter((c) => own.includes(c.id))
   if (criteria.length === 0) return { covered: [], uncovered: [] }
   const { answers } = await judge("coverage", {
     questions: criteria.map(t.coverageQuestion),
@@ -81,14 +89,14 @@ const record = (covered: readonly string[], uncovered: readonly string[] = []):
  * sending uncovered IDs back to a covering turn; marks the IDs covered on the
  * landing that ends it. A human may overrule once the retries are spent.
  */
-export const requireCoverage = async (base = start()): Promise<void> => {
+export const requireCoverage = async (base = start(), slice?: CriteriaSlice): Promise<void> => {
   let retries = 0
   for (;;) {
-    const { covered, uncovered } = await uncoveredSince(base)
+    const { covered, uncovered } = await uncoveredSince(base, slice)
     if (uncovered.length === 0) return record(covered)
     if (retries < COVERING_RETRIES) {
       retries++
-      await t.agentWithSkills("covering", t.coveringPrompt(uncovered), {
+      await t.agentWithSkills("covering", t.coveringPrompt(uncovered, sliceIds(slice)), {
         label: "Covering the acceptance criteria",
         model: coder(),
         system: t.builderSystem(),

package/src/workflows/feature/criteria.test.ts

@@ -1,6 +1,6 @@
 import { describe, expect, it } from "vitest"
 import type { Change } from "../../flows/index.js"
-import { criteriaOf, statedCriteria } from "./criteria.js"
+import { criteriaOf, criterionIds, statedCriteria } from "./criteria.js"
 
 const added = (path: string, after: string): Change => ({
   path,
@@ -104,3 +104,19 @@ describe("statedCriteria", () => {
     ).toBeUndefined()
   })
 })
+
+describe("criterionIds", () => {
+  const requirements = "## Acceptance Criteria\n\n- AC-1: a\n- AC-2: b\n- AC-4: added\n- unnamed\n"
+
+  it("lists every ID'd criterion of an unsliced change", () => {
+    expect(criterionIds(requirements)).toEqual(["AC-1", "AC-2", "AC-4"])
+  })
+
+  it("keeps `only` to the IDs the file still lists", () => {
+    expect(criterionIds(requirements, { only: ["AC-1", "AC-3"] })).toEqual(["AC-1"])
+  })
+
+  it("gives `except` every other ID, one added since included", () => {
+    expect(criterionIds(requirements, { except: ["AC-1", "AC-3"] })).toEqual(["AC-2", "AC-4"])
+  })
+})