@pmelab/gtd 36.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}`;
@@ -17436,9 +17455,11 @@ ${strays.map((path) => `  - \`${displayPath(path)}\``).join("\n")}
 - Introduce no new flag read and add no second read of an existing one:
   every match of \`${pattern}\` may move, never increase in number`;
 const withRules = (rules) => rules === void 0 ? "" : `${rules}\n`;
-const refactorBuildingPrompt = ({ pkg, strays, pattern, reference, feedback = false }) => `${stateFileRules}
+const refactorBuildingPrompt = ({ pkg, strays, pattern, reference, feedback = false, note = false }) => `${stateFileRules}
 - The package to implement is \`${pkg}\`; never delete it
-${feedback ? `- The package carries a \`## Review feedback\` section: a human's review of the
+${note ? `- The package's \`## Refactor note\` section names the seams to open and the
+  shape each takes, written from the whole naive pass: build exactly those
+` : ""}${feedback ? `- The package carries a \`## Review feedback\` section: a human's review of the
   last refactor. Act on it, within the rules below
 ` : ""}${refactorRules(strays, pattern)}
 ${reference === void 0 ? "" : `- Reference only: \`git show ${reference} -- ${strays.map(quote).join(" ")}\` shows an
@@ -18037,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
@@ -18057,22 +18078,54 @@ ${criteriaBinding(false)}
 - Leave \`.gtd/\` untouched
 - Leave everything uncommitted and finish your turn — do not commit
 ${retry === void 0 ? "" : naiveRetry(retry)}`;
-const implementBuildingPrompt = (reference, tier, amend, afterRefactor, flagMode = true) => `${stateFileRules}
-- Read \`.gtd/REQUIREMENTS.md\`: the settled requirements for this change
-${criteriaBinding(true)}
-${afterRefactor === true ? `- A reviewed refactor has just landed and may already carry part of the change,
+const carries = (rebuild) => (rebuild.acceptance?.ids.length ?? 0) > 0;
+const rebuildLines = ({ note, acceptance }) => `${note === void 0 ? "" : `- The refactor note (\`.gtd/REFACTOR-NOTE.md\`) named the seams the refactor
+  opened; plug the change into them:
+
+${note.trimEnd().split("\n").map((line) => line === "" ? "" : `  > ${line}`).join("\n")}
+
+`}${acceptance === void 0 || !carries({ acceptance }) ? "" : `- Carry over the naive pass's acceptance tests for ${acceptance.ids.join(", ")}, the
+  one part of it you take as it is: \`git show ${acceptance.hash} -- ${acceptance.tests.map(quote).join(" ")}\`.
+  Copy each test with its \`[AC-<n>]\` name into the tree as the refactor left
+  it — check a file out of the tip whole only where the refactor did not touch
+  it — changing only what reaches the release flag. A turn whose changed tests
+  leave an ID out is refused, and the suite must be green with them
+`}`;
+const refactorNotePrompt = (reference, strays) => `${stateFileRules}
+- The seam judge ruled the naive pass refactor-first: it is never applied.
+  A behaviour-neutral refactor lands first, then the change is rebuilt on
+  the refactored base. Both turns read the note you write
+- Read the whole naive pass: \`git show ${reference}\`. It changed these
+  existing files outside the allowlist:
+${listed(strays)}
+- Write \`.gtd/REFACTOR-NOTE.md\`: which seams the refactor opens (the existing
+  code to restructure, by path and symbol) and the shape each takes — the
+  signatures and where the new behaviour will plug in. Name only the files
+  listed above: the refactor may change no other path. Never the feature's own
+  behaviour, and no flag read: the refactor adds neither
+- Change nothing else — a turn that does is refused
+- Leave everything uncommitted and finish your turn — do not commit
+`;
+const afterRefactorLine = (rebuild) => `- A reviewed refactor has just landed and may already carry part of the change,
   though never a release flag. Build only what is still missing, held to every
-  rule below; if nothing is, change nothing and finish your turn
-` : ""}${amend === true ? `- The change already exists in the tree from an earlier lap; amend it to meet
+  rule below; if nothing is, ${carries(rebuild) ? "carry over the acceptance tests below and change nothing else" : "change nothing and finish your turn"}
+`;
+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
+${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
 ` : `- Implement them the quickest way that works — the naive version. Do not
   restructure, abstract or refactor around the change
 `}${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
-`}${!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`}
+  the change fresh${carries(rebuild) ? ", its acceptance tests excepted (below)" : ""}
+`}${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
@@ -18153,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
@@ -18180,7 +18233,9 @@ const ESCALATION = ".gtd/ESCALATION.md";
 const REQUIREMENTS = ".gtd/REQUIREMENTS.md";
 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), {
@@ -37692,12 +37747,17 @@ const ShippedOptions = Struct({
 	reviewSeverityMinP: setting,
 	refactorFlagPattern: setting,
 	seamAllow: setting,
+	sliceMaxCriteria: setting,
 	qualityReviews: setting,
 	refactorQualityReviews: setting,
 	testCommand: setting,
 	plannerModel: setting,
 	coderModel: setting,
-	flagMode: optionalWith(Boolean$, { exact: true })
+	flagMode: optionalWith(Boolean$, { exact: true }),
+	contextLimit: optionalWith(Struct({
+		soft: optionalWith(Number$, { exact: true }),
+		hard: optionalWith(Number$, { exact: true })
+	}), { exact: true })
 });
 const over = (base, options) => ({
 	...base,
@@ -37709,12 +37769,13 @@ const flagModeOn = () => {
 	if (value !== "true" && value !== "false") refuse(`gtd: flagMode must be "true" or "false", got ${JSON.stringify(value)}`);
 	return value === "true";
 };
+const limited = (flow, context) => context === void 0 ? flow : () => scope({ context }, flow);
 /** A shipped flow as a factory: its options override the settings it declares, each in its own kind. */
 const shipped = (flow, doors, base) => defineWorkflow(standardSchemaV1(ShippedOptions, {
 	onExcessProperty: "error",
 	errors: "all"
-}), (options) => ({
-	flow,
+}), ({ contextLimit, ...options }) => ({
+	flow: limited(flow, contextLimit),
 	defaults: over(defaults, options),
 	envDefaults: over(envDefaults, options),
 	doors,
@@ -37727,14 +37788,14 @@ const FlagOptions = Struct({
 const flagShipped = (flow, name) => defineWorkflow(standardSchemaV1(FlagOptions, {
 	onExcessProperty: "error",
 	errors: "all"
-}), (options) => {
+}), ({ contextLimit, ...options }) => {
 	const flagDefaults = {
 		...defaults,
 		flag: "",
 		flagRegistry: defaultFlagRegistry
 	};
 	return {
-		flow,
+		flow: limited(flow, contextLimit),
 		defaults: over(flagDefaults, options),
 		envDefaults: over(envDefaults, options),
 		doors: { [name]: {
@@ -37867,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);
@@ -37901,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
@@ -37910,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;
@@ -38473,7 +38540,8 @@ const plan = (shape, options) => {
 	return {
 		pattern,
 		strays,
-		reference: options.reference?.trim() ? options.reference : void 0
+		reference: options.reference?.trim() ? options.reference : void 0,
+		note: options.note?.trim() ? options.note : void 0
 	};
 };
 const guardOf = ({ pattern, strays }, entry) => () => {
@@ -38482,13 +38550,22 @@ const guardOf = ({ pattern, strays }, entry) => () => {
 };
 /** Derive, build, check and close one refactor lap; `feedback` is the collected review text of the lap before. */
 const refactorLap = async (p, guard, feedback) => {
-	const { pattern, strays, reference } = p;
-	await run("deriving", deriveScript([...packageLines(strays, pattern, reference), ...feedback === void 0 ? [] : [
-		"## Review feedback",
-		"",
-		...demoteHeadings(feedback),
-		""
-	]]), {
+	const { pattern, strays, reference, note } = p;
+	await run("deriving", deriveScript([
+		...packageLines(strays, pattern, reference),
+		...note === void 0 ? [] : [
+			"## Refactor note",
+			"",
+			...demoteHeadings(note),
+			""
+		],
+		...feedback === void 0 ? [] : [
+			"## Review feedback",
+			"",
+			...demoteHeadings(feedback),
+			""
+		]
+	]), {
 		label: "Deriving the refactor package",
 		file: REFACTOR_PACKAGE
 	});
@@ -38497,7 +38574,8 @@ const refactorLap = async (p, guard, feedback) => {
 		strays,
 		pattern,
 		reference,
-		feedback: feedback !== void 0
+		feedback: feedback !== void 0,
+		note: note !== void 0
 	}), {
 		label: "Refactoring",
 		model: coder(),
@@ -38583,22 +38661,27 @@ const COVERED_MIN_P = .7;
 /** Covering turns before the process rests at `uncovered`; a human edit there refills them. */
 const COVERING_RETRIES = 2;
 const frozen = () => requireCriteriaFrozen(REQUIREMENTS);
-const green = () => healthy(() => agentWithSkills("fix.suite.fixing", implementFixPrompt(), {
+const green = (guard) => healthy(() => agentWithSkills("fix.suite.fixing", implementFixPrompt(), {
 	label: "Fixing the check",
 	file: FEEDBACK,
 	model: coder(),
 	system: builderSystem()
 }), {
-	guard: frozen,
+	guard: () => {
+		frozen();
+		guard?.();
+	},
 	guardRests: false
 });
 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: []
@@ -38625,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(),
@@ -38663,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(),
@@ -38703,16 +38786,20 @@ const addedLines = (change) => {
 	const before = new Set((change.before ?? "").split("\n"));
 	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, slice) => criterionIds(read(file) ?? "", slice);
+const unnamed = (ids, tests) => {
+	const lines = tests.map(addedLines);
+	return ids.filter((id) => !lines.some((text) => named(id, text)));
+};
 /**
 * Why `tip`'s tests fall short, or `undefined`: a criterion ID of `file` no
 * 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 ids = (criteriaOf(read(file) ?? "") ?? []).map((c) => c.id).filter((id) => id !== "");
-	const tests = tip.changes.filter((c) => c.after !== void 0 && isCode$1(c.path) && isTestFile(c.path));
-	const lines = tests.map(addedLines);
-	const untested = ids.filter((id) => !lines.some((text) => named(id, text)));
+const acceptanceFault = (file, tip, slice) => {
+	const tests = acceptanceTests(tip.changes);
+	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) => {
@@ -38721,6 +38808,41 @@ 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, slice) => {
+	const ids = idsOf(REQUIREMENTS, slice);
+	const perTest = acceptanceTests(tip.changes).map((c) => {
+		const lines = addedLines(c);
+		return {
+			path: c.path,
+			ids: ids.filter((id) => named(id, lines))
+		};
+	});
+	return {
+		hash: tip.hash,
+		tests: perTest.filter((c) => c.ids.length > 0).map((c) => c.path),
+		ids: ids.filter((id) => perTest.some((c) => c.ids.includes(id)))
+	};
+};
+/**
+* Refuse a rebuild whose test changes since `since` leave out an ID the naive
+* pass's tests named. A skipped test still counts: the gate reads text, never runs it.
+*/
+const requireAcceptance = (acceptance, since) => {
+	const missing = unnamed(acceptance.ids, acceptanceTests(changesSince(since)));
+	if (missing.length === 0) return;
+	refuse(`gtd land: rebuild-acceptance: the rebuild's tests leave out ${missing.join(", ")} — carry over the naive pass's acceptance tests (\`git show ${acceptance.hash} -- ${acceptance.tests.map(quote).join(" ")}\`), each with its ID in the test or scenario name`);
+};
+/**
+* Refuse a fix turn that, on its own, drops an ID the naive pass's tests named
+* from a test file. Turn-local, so a person's edit at an escalation rest never
+* wedges later fix turns.
+*/
+const requireAcceptanceKept = (acceptance) => {
+	const dropped = acceptance.ids.filter((id) => changes().filter((c) => isCode$1(c.path) && isTestFile(c.path)).some((c) => named(id, c.before ?? "") && !named(id, c.after ?? "")));
+	if (dropped.length === 0) return;
+	refuse(`gtd land: rebuild-acceptance: this turn drops ${dropped.join(", ")} from the tests — the naive pass's acceptance tests stay; fix the code, not them`);
+};
 const restore = (path) => `if git cat-file -e ${quote(`HEAD:${path}`)} 2>/dev/null; then git checkout -q HEAD -- ${quote(path)}; else rm -f -- ${quote(path)}; fi`;
 /**
 * Run the suite with `tip`'s diff applied, then take it out again; resolves
@@ -38764,18 +38886,30 @@ function implement(first, tier, amend) {
 		amend
 	};
 	return scope("implement", async () => {
-		if (options.naive === void 0 || !await applied(options.naive)) await agentWithSkills("building", implementBuildingPrompt(options.reference, options.tier, options.amend, options.afterRefactor, flagModeOn()), {
+		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, sliceIds(options.slice)), {
 			label: "Implementing",
 			model: coder(),
 			system: builderSystem(),
 			...options.afterRefactor === true ? { allowEmpty: true } : {}
 		});
 		requireCriteriaFrozen(REQUIREMENTS);
-		await green();
-		await requireCoverage();
-		await run("closing", removeScript([REQUIREMENTS]), { label: "Closing out the implementation" });
+		if (built && acceptance !== void 0) requireAcceptance(acceptance, since);
+		await green(built && acceptance !== void 0 ? () => requireAcceptanceKept(acceptance) : void 0);
+		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", {
@@ -38806,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,
@@ -38823,33 +38957,56 @@ const vetted = async (tip, vetting) => {
 /** The judge's ruling on a vetted pass: the decision, or the next attempt's verdict. */
 const ruled = async (tip, again) => {
 	const shape = await judgeSeam(tip);
-	if (shape.choice === "land") return { reference: tip.hash };
+	if (shape.choice === "land") return {
+		reference: tip.hash,
+		tip
+	};
 	if (shape.choice === "refactor-first") return {
 		reference: tip.hash,
+		tip,
 		shape
 	};
 	const next = await again({
 		strays: shape.strays,
 		p: shape.judgment?.answers.shape?.p ?? 0
 	});
-	return next.kind === "overruled" ? { reference: tip.hash } : next;
+	return next.kind === "overruled" ? {
+		reference: tip.hash,
+		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) : { reference: tip.hash });
+		const outcome = await vetted(tip, vetting, slice) ?? (flagModeOn() ? await ruled(tip, vetting.again) : {
+			reference: tip.hash,
+			tip
+		});
 		if (!("kind" in outcome)) return outcome;
 		retry = outcome.kind === "retry" ? outcome.retry : void 0;
 	}
 });
-const seamBuild = async (base) => {
-	const { reference, shape } = await decide();
+/** Write the refactor note from the whole naive pass; a turn that changes anything else is refused. */
+const refactorNote = async (reference, strays) => {
+	await agent("note", refactorNotePrompt(reference, strays), {
+		label: "Writing the refactor note",
+		file: REFACTOR_NOTE,
+		model: coder(),
+		system: builderSystem()
+	});
+	const others = changes().map((c) => c.path).filter((path) => path !== REFACTOR_NOTE);
+	const note = read(".gtd/REFACTOR-NOTE.md") ?? "";
+	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, slice$1 = {}) => {
+	const { reference, tip, shape } = await decide(slice$1.criteria);
 	if (shape === void 0) return {
 		reference,
 		base,
@@ -38863,42 +39020,164 @@ const seamBuild = async (base) => {
 		}));
 		return {
 			reference,
-			base
+			base,
+			rebuild: { acceptance: naiveAcceptance(tip, slice$1.criteria) }
 		};
 	}
-	if (refactorStrays(shape).length === 0) return {
+	const strays = refactorStrays(shape);
+	if (strays.length === 0) return {
 		reference,
-		base
+		base,
+		rebuild: { acceptance: naiveAcceptance(tip, slice$1.criteria) }
 	};
-	const refactored = await slice("refactor", () => reviewedRefactor(shape, { reference }));
+	const note = await scope("seam", () => refactorNote(reference, strays));
+	const refactored = await slice(slice$1.refactor ?? "refactor", () => reviewedRefactor(shape, {
+		reference,
+		note
+	}));
 	return {
 		reference,
 		base: head$1(),
-		refactored
+		refactored,
+		rebuild: {
+			note,
+			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
+			naive: seam.land === true ? seam.reference : void 0,
+			rebuild: seam.rebuild,
+			slice: build.criteria,
+			base: build.criteria === void 0 ? void 0 : sliceBase,
+			more: build.more
 		});
-		const base = seam.refactored === true && !changesSince(seam.base).some((c) => isCode$1(c.path)) ? firstBase : seam.base;
+		const carried = seam.rebuild?.acceptance?.tests ?? [];
+		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
 		});
 	});
 };
@@ -38944,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", {
@@ -39138,4 +39418,4 @@ var workflows_default = feature;
 */
 const doors = {};
 //#endregion
-export { COMMENT, ESCALATION, FEEDBACK, FIX_CAP, FlagOptions, QUALITY, REFACTOR, 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, naivePass, ordinaryStart, planAndBuild, planner, qualityLap, reUnwind, refactorCollecting, refactorFirst, refactorStrays, 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 };