@pmelab/gtd 27.7.0 → 27.8.0

package/claude/hooks/wording.ts

@@ -2,6 +2,9 @@
 
 import type { Stop } from "../types"
 import type { Beat } from "./drive"
+import type { Shipped } from "./ship"
+import { JOB_NAME } from "./tools"
+import type { JobName } from "./tools"
 
 const step = (s: { label?: string; state?: string }) => s.label ?? s.state ?? "the next step"
 
@@ -53,3 +56,11 @@ export function pushText(stop: Stop, repo: string, url?: string) {
   if (stop.kind !== "gate") return at
   return url ? `${at}. Open: ${url}` : at
 }
+
+const capitalised = (s: string) => s[0]!.toUpperCase() + s.slice(1)
+
+export const jobHeadline = (job: JobName, outcome: Shipped) =>
+  `${outcome.ok ? "✔" : "✘"} ${capitalised(JOB_NAME[job])} ${outcome.ok ? "finished" : "failed"}`
+
+export const jobPushText = (job: JobName, outcome: Shipped, repo: string) =>
+  `${repo}: ${JOB_NAME[job]} ${outcome.ok ? "finished" : "failed"}. ${outcome.text}`.trim()

package/dist/chunks/{reserved-Df8GeSHw.mjs → reserved-UdSuHDUJ.mjs}

@@ -1234,13 +1234,13 @@ const fixRisks = (notes) => agentWithSkills("review.fix.risks.fixing", buildRevi
 	system: reviewerSystem(),
 	allowEmpty: true
 });
-const awaitReview = (base, message = buildReviewAwaitReviewMessage(base), acceptClean = false) => human("review.await-review", {
+const awaitReview = (base, message = buildReviewAwaitReviewMessage(base)) => human("review.await-review", {
 	message,
 	label: "Awaiting your review",
 	file: REVIEW,
 	mode: "review",
 	base,
-	...acceptClean ? { acceptClean } : {}
+	acceptClean: true
 });
 /** The round at commit `round` left no `.gtd/REVIEW.md`. */
 const reviewMissing = (round) => human("review.review-missing", {

package/dist/chunks/{tail-DmIfEAI4.mjs → tail-fj60rQ3y.mjs}

@@ -1,6 +1,6 @@
 import { read, refuse, removeScript, run, scope, showScript, vars } from "../flows.mjs";
 import { fixFindings, lensesOf, qualityLap as qualityLap$1, reviewToHandOff } from "../workflow-kit.mjs";
-import { C as fixRisks, D as reviewing, E as reviewQuality, G as handOffAwaitReviewMessage, S as fixQuality, T as reviewMissing, f as guarded, g as QUALITY, p as healthy, u as escalation, v as REVIEW, x as fixCheck, y as awaitReview } from "./reserved-Df8GeSHw.mjs";
+import { C as fixRisks, D as reviewing, E as reviewQuality, G as handOffAwaitReviewMessage, S as fixQuality, T as reviewMissing, f as guarded, g as QUALITY, p as healthy, u as escalation, v as REVIEW, x as fixCheck, y as awaitReview } from "./reserved-UdSuHDUJ.mjs";
 //#region src/workflows/shared/tail.ts
 /** The reviewer's own `.gtd/REVIEW.md`, kept beside the human's notes so `fix` can tell them apart. */
 const REVIEW_ORIG = ".gtd/REVIEW.orig.md";

package/dist/flows.mjs

@@ -86,6 +86,20 @@ const scope = async (scope, fn) => {
 		context.popScope();
 	}
 };
+/**
+* Mark the span `fn` covers as one named slice of the process. Lands no
+* commit, prefixes no step name, opens no memory scope.
+*/
+const slice = async (name, fn) => {
+	const context = ctx();
+	if (typeof context.enterSlice !== "function") return fn();
+	context.enterSlice(name);
+	try {
+		return await fn();
+	} finally {
+		context.leaveSlice?.();
+	}
+};
 const read = (path) => ctx().read(path);
 /** Every path in the tree matching `pattern` (`*` stays within a segment, `**` crosses them). */
 const glob = (pattern) => ctx().glob(pattern);
@@ -652,10 +666,11 @@ var flows_exports = /* @__PURE__ */ __exportAll({
 	sections: () => sections,
 	showScript: () => showScript,
 	skillsFor: () => skillsFor,
+	slice: () => slice,
 	start: () => start,
 	threads: () => threads,
 	vars: () => vars,
 	wrote: () => wrote
 });
 //#endregion
-export { FLOWS_PROTOCOL, FlowsProtocolError, accessFor, agent, answered, changes, changesSince, check, checkScript, codeChanges, codeThreads, configProblems, configure, defineConfig, defineWorkflow, displayPath, env, glob, hasThreadFor, head, human, installContext, isFactoryWorkflow, judge, judgeDiffShape, kept, mark, matches, moveScript, numeric, openQuestions, protocolMismatch, quote, read, refuse, removeScript, requireAnswers, requireProgress, requireReplies, requireRevert, requireThreadsClosed, restart, restoreScript, revertScript, run, scope, seamGate, sectionBodies, sections, showScript, skillsFor, start, flows_exports as t, threads, vars, wrote };
+export { FLOWS_PROTOCOL, FlowsProtocolError, accessFor, agent, answered, changes, changesSince, check, checkScript, codeChanges, codeThreads, configProblems, configure, defineConfig, defineWorkflow, displayPath, env, glob, hasThreadFor, head, human, installContext, isFactoryWorkflow, judge, judgeDiffShape, kept, mark, matches, moveScript, numeric, openQuestions, protocolMismatch, quote, read, refuse, removeScript, requireAnswers, requireProgress, requireReplies, requireRevert, requireThreadsClosed, restart, restoreScript, revertScript, run, scope, seamGate, sectionBodies, sections, showScript, skillsFor, slice, start, flows_exports as t, threads, vars, wrote };

package/dist/gtd.bundle.mjs

@@ -21724,6 +21724,20 @@ const scope = async (scope, fn) => {
 		context.popScope();
 	}
 };
+/**
+* Mark the span `fn` covers as one named slice of the process. Lands no
+* commit, prefixes no step name, opens no memory scope.
+*/
+const slice = async (name, fn) => {
+	const context = ctx();
+	if (typeof context.enterSlice !== "function") return fn();
+	context.enterSlice(name);
+	try {
+		return await fn();
+	} finally {
+		context.leaveSlice?.();
+	}
+};
 const read = (path) => ctx().read(path);
 /** Every path in the tree matching `pattern` (`*` stays within a segment, `**` crosses them). */
 const glob = (pattern) => ctx().glob(pattern);
@@ -22290,6 +22304,7 @@ var flows_exports = /* @__PURE__ */ __exportAll({
 	sections: () => sections,
 	showScript: () => showScript,
 	skillsFor: () => skillsFor,
+	slice: () => slice,
 	start: () => start,
 	threads: () => threads,
 	vars: () => vars,
@@ -37670,6 +37685,32 @@ const refNameFault = (name) => {
 	}
 };
 //#endregion
+//#region src/replay/Slices.ts
+const IMPLICIT_SLICE = "main";
+/** Same shape as mark()'s name rule; kept local so @pmelab/gtd/flows publishes no regex. */
+const SLICE_NAME = /^[a-z][a-z0-9-]*$/;
+/**
+* Folds boundaries into ranges: each kept span's base is the previous kept
+* span's `leftAt`, so a gap before a span belongs to it, and the last span
+* runs to the episode tip. A closed span that landed nothing is dropped.
+*/
+const sliceRanges = (spans, episode) => {
+	if (spans.some((s) => s.enteredAt === "" || s.leftAt === "")) throw new Error("sliceRanges: a pending turn's \"\" hash is not a commit");
+	const kept = spans.filter((s) => s.leftAt === void 0 || s.enteredAt !== s.leftAt);
+	if (kept.length === 0) return [{
+		name: IMPLICIT_SLICE,
+		base: episode.base,
+		tip: episode.tip,
+		open: true
+	}];
+	return kept.map((s, i) => ({
+		name: s.name,
+		base: i === 0 ? episode.base : kept[i - 1].leftAt ?? episode.base,
+		tip: i === kept.length - 1 ? episode.tip : s.leftAt ?? episode.tip,
+		open: i === kept.length - 1
+	}));
+};
+//#endregion
 //#region src/replay/Tree.ts
 const treeFromRecord = (files) => ({
 	paths: () => Object.keys(files).sort(),
@@ -38017,6 +38058,7 @@ const replay = async (input) => {
 	const keptTips = /* @__PURE__ */ new Map();
 	let landed;
 	const marks = [];
+	const slices = [];
 	let outcome;
 	let signal;
 	const settled = new Promise((resolve) => {
@@ -38223,7 +38265,8 @@ const replay = async (input) => {
 			rest: step,
 			trace,
 			landed,
-			marks
+			marks,
+			slices: slices.map((s) => ({ ...s }))
 		});
 		return landPending(step, input.pending);
 	};
@@ -38287,6 +38330,31 @@ const replay = async (input) => {
 			scopes.pop();
 			scopeCallIds.pop();
 		},
+		enterSlice: (name) => {
+			if (outcome !== void 0) return;
+			const fail = (message) => {
+				finish({
+					kind: "failed",
+					message
+				});
+				throw new Stop();
+			};
+			if (typeof name !== "string" || !SLICE_NAME.test(name)) fail(`gtd: slice "${name}": name must match ${SLICE_NAME}`);
+			if (name === "main") fail(`gtd: slice "${IMPLICIT_SLICE}" is reserved for the implicit slice`);
+			const open = slices.find((s) => s.leftAt === void 0);
+			if (open !== void 0) fail(`gtd: slice "${name}" is entered inside slice "${open.name}" — slices do not nest`);
+			if (slices.some((s) => s.name === name)) fail(`gtd: slice "${name}" is entered twice in this episode — a loop must sit inside its slice`);
+			slices.push({
+				name,
+				enteredAt: position.hash,
+				leftAt: void 0
+			});
+		},
+		leaveSlice: () => {
+			if (outcome !== void 0) return;
+			const open = slices.find((s) => s.leftAt === void 0);
+			if (open !== void 0) open.leftAt = position.hash;
+		},
 		read: (path) => position.tree.read(path),
 		glob: (pattern) => position.tree.paths().filter((path) => globMatches(path, pattern)),
 		changes: () => changesBetween(previousPosition.tree, position.tree),
@@ -39976,6 +40044,10 @@ const pendingChanges = (git, withIndex) => gen(function* () {
 	const seen = new Set(worktree.map((e) => e.path));
 	return [...worktree, ...(yield* git.stagedPaths()).filter((e) => !seen.has(e.path))];
 });
+const sliceRangesOf = (slices, run) => sliceRanges(slices, {
+	base: run.startParentHash,
+	tip: run.trace.at(-1)?.hash ?? run.startParentHash
+});
 /** Resolve the rest at `ref`, or at HEAD when `ref` is `undefined`. */
 const restAt = (ref, startOverrides = {}) => gen(function* () {
 	const git = yield* GitService;
@@ -40018,7 +40090,8 @@ const restAt = (ref, startOverrides = {}) => gen(function* () {
 			total: run.costEntries.reduce((sum, entry) => sum + entry.cost, 0),
 			byModel: costByModel(run.costEntries)
 		},
-		setup
+		setup,
+		slices: sliceRangesOf(outcome.slices, run)
 	};
 });
 const currentRest = restAt(void 0);

package/dist/testkit.mjs

@@ -6817,6 +6817,32 @@ const refNameFault = (name) => {
 	}
 };
 //#endregion
+//#region src/replay/Slices.ts
+const IMPLICIT_SLICE = "main";
+/** Same shape as mark()'s name rule; kept local so @pmelab/gtd/flows publishes no regex. */
+const SLICE_NAME = /^[a-z][a-z0-9-]*$/;
+/**
+* Folds boundaries into ranges: each kept span's base is the previous kept
+* span's `leftAt`, so a gap before a span belongs to it, and the last span
+* runs to the episode tip. A closed span that landed nothing is dropped.
+*/
+const sliceRanges = (spans, episode) => {
+	if (spans.some((s) => s.enteredAt === "" || s.leftAt === "")) throw new Error("sliceRanges: a pending turn's \"\" hash is not a commit");
+	const kept = spans.filter((s) => s.leftAt === void 0 || s.enteredAt !== s.leftAt);
+	if (kept.length === 0) return [{
+		name: IMPLICIT_SLICE,
+		base: episode.base,
+		tip: episode.tip,
+		open: true
+	}];
+	return kept.map((s, i) => ({
+		name: s.name,
+		base: i === 0 ? episode.base : kept[i - 1].leftAt ?? episode.base,
+		tip: i === kept.length - 1 ? episode.tip : s.leftAt ?? episode.tip,
+		open: i === kept.length - 1
+	}));
+};
+//#endregion
 //#region src/replay/Tree.ts
 const treeFromRecord = (files) => ({
 	paths: () => Object.keys(files).sort(),
@@ -7164,6 +7190,7 @@ const replay = async (input) => {
 	const keptTips = /* @__PURE__ */ new Map();
 	let landed;
 	const marks = [];
+	const slices = [];
 	let outcome;
 	let signal;
 	const settled = new Promise((resolve) => {
@@ -7370,7 +7397,8 @@ const replay = async (input) => {
 			rest: step,
 			trace,
 			landed,
-			marks
+			marks,
+			slices: slices.map((s) => ({ ...s }))
 		});
 		return landPending(step, input.pending);
 	};
@@ -7434,6 +7462,31 @@ const replay = async (input) => {
 			scopes.pop();
 			scopeCallIds.pop();
 		},
+		enterSlice: (name) => {
+			if (outcome !== void 0) return;
+			const fail = (message) => {
+				finish({
+					kind: "failed",
+					message
+				});
+				throw new Stop();
+			};
+			if (typeof name !== "string" || !SLICE_NAME.test(name)) fail(`gtd: slice "${name}": name must match ${SLICE_NAME}`);
+			if (name === "main") fail(`gtd: slice "${IMPLICIT_SLICE}" is reserved for the implicit slice`);
+			const open = slices.find((s) => s.leftAt === void 0);
+			if (open !== void 0) fail(`gtd: slice "${name}" is entered inside slice "${open.name}" — slices do not nest`);
+			if (slices.some((s) => s.name === name)) fail(`gtd: slice "${name}" is entered twice in this episode — a loop must sit inside its slice`);
+			slices.push({
+				name,
+				enteredAt: position.hash,
+				leftAt: void 0
+			});
+		},
+		leaveSlice: () => {
+			if (outcome !== void 0) return;
+			const open = slices.find((s) => s.leftAt === void 0);
+			if (open !== void 0) open.leftAt = position.hash;
+		},
 		read: (path) => position.tree.read(path),
 		glob: (pattern) => position.tree.paths().filter((path) => globMatches(path, pattern)),
 		changes: () => changesBetween(previousPosition.tree, position.tree),
@@ -9112,6 +9165,10 @@ const pendingChanges = (git, withIndex) => gen(function* () {
 	const seen = new Set(worktree.map((e) => e.path));
 	return [...worktree, ...(yield* git.stagedPaths()).filter((e) => !seen.has(e.path))];
 });
+const sliceRangesOf = (slices, run) => sliceRanges(slices, {
+	base: run.startParentHash,
+	tip: run.trace.at(-1)?.hash ?? run.startParentHash
+});
 /** Resolve the rest at `ref`, or at HEAD when `ref` is `undefined`. */
 const restAt = (ref, startOverrides = {}) => gen(function* () {
 	const git = yield* GitService;
@@ -9154,7 +9211,8 @@ const restAt = (ref, startOverrides = {}) => gen(function* () {
 			total: run.costEntries.reduce((sum, entry) => sum + entry.cost, 0),
 			byModel: costByModel(run.costEntries)
 		},
-		setup
+		setup,
+		slices: sliceRangesOf(outcome.slices, run)
 	};
 });
 const currentRest = restAt(void 0);

package/dist/workflows/cleanup.mjs

@@ -1,6 +1,6 @@
 import { agent, changes, glob, read, refuse, scope, start, vars } from "../flows.mjs";
-import { a as flagShipped, b as coder, c as envDefaults, i as skills, n as summary, r as access, s as defaults, t as steering } from "../chunks/reserved-Df8GeSHw.mjs";
-import { i as refusePendingHandOff, r as handOffTail } from "../chunks/tail-DmIfEAI4.mjs";
+import { a as flagShipped, b as coder, c as envDefaults, i as skills, n as summary, r as access, s as defaults, t as steering } from "../chunks/reserved-UdSuHDUJ.mjs";
+import { i as refusePendingHandOff, r as handOffTail } from "../chunks/tail-fj60rQ3y.mjs";
 import { i as requireFlag, r as registryEntry, t as flagReferences } from "../chunks/registry-CP3YuV9j.mjs";
 //#region src/workflows/cleanup/cleanup.ts
 const removingPrompt = (name, registry) => `Clean up the release flag "${name}", which is default-on: delete its entry in ${registry}, and remove every \`GTD_FLAG_${name.toUpperCase()}\` check from the shipped sources. Keep the flag-on path; delete code only the off path reached, including tests that pin the off path.`;

package/dist/workflows/feature.mjs

@@ -1,7 +1,7 @@
 import { answered, changes, changesSince, glob, hasThreadFor, head, human, judge, judgeDiffShape, kept, mark, numeric, quote, read, refuse, removeScript, requireAnswers, requireProgress, requireReplies, requireRevert, requireThreadsClosed, restoreScript, revertScript, run, scope, seamGate, start, vars } from "../flows.mjs";
 import { review as review$1 } from "../workflow-kit.mjs";
-import { $ as refactorFixPrompt, A as architectureAuthorPrompt, B as designGateAnswerMessage, C as fixRisks, D as reviewing, F as buildReviewCollectingPrompt, H as designTriagePrompt, I as buildReviewFixNitsPrompt, J as implementFixPrompt, K as idleMessage, L as buildReviewTriageMessage, M as architectureGateAnswerMessage, N as buildReviewAnswerQuestionsPrompt, O as agentWithSkills, P as buildReviewAwaitAfterDeferMessage, Q as refactorBuildingPrompt, R as builderSystem, T as reviewMissing, V as designSystem, X as packagesItemBuildingPrompt, Y as naivePassPrompt, Z as packagesItemFixSuitePrompt, _ as REQUIREMENTS, at as seamByHandMessage, b as coder, c as envDefaults, ct as unwindFailedMessage, d as gate, et as reviewEditNotesCapture, h as FEEDBACK, i as skills, it as reviewerSystem, j as architectureDecomposePrompt, k as architectSystem, lt as unwindFailure, m as ARCHITECTURE, n as summary, o as shipped, ot as seamSteerMessage, p as healthy, q as implementBuildingPrompt, r as access, rt as reviewNotesCapture, s as defaults, st as startGateBlockedMessage, t as steering, tt as reviewEditsCapture, v as REVIEW, w as planner, x as fixCheck, y as awaitReview, z as deferNotesScript } from "../chunks/reserved-Df8GeSHw.mjs";
-import { a as tail, i as refusePendingHandOff, n as closeReview } from "../chunks/tail-DmIfEAI4.mjs";
+import { $ as refactorFixPrompt, A as architectureAuthorPrompt, B as designGateAnswerMessage, C as fixRisks, D as reviewing, F as buildReviewCollectingPrompt, H as designTriagePrompt, I as buildReviewFixNitsPrompt, J as implementFixPrompt, K as idleMessage, L as buildReviewTriageMessage, M as architectureGateAnswerMessage, N as buildReviewAnswerQuestionsPrompt, O as agentWithSkills, P as buildReviewAwaitAfterDeferMessage, Q as refactorBuildingPrompt, R as builderSystem, T as reviewMissing, V as designSystem, X as packagesItemBuildingPrompt, Y as naivePassPrompt, Z as packagesItemFixSuitePrompt, _ as REQUIREMENTS, at as seamByHandMessage, b as coder, c as envDefaults, ct as unwindFailedMessage, d as gate, et as reviewEditNotesCapture, h as FEEDBACK, i as skills, it as reviewerSystem, j as architectureDecomposePrompt, k as architectSystem, lt as unwindFailure, m as ARCHITECTURE, n as summary, o as shipped, ot as seamSteerMessage, p as healthy, q as implementBuildingPrompt, r as access, rt as reviewNotesCapture, s as defaults, st as startGateBlockedMessage, t as steering, tt as reviewEditsCapture, v as REVIEW, w as planner, x as fixCheck, y as awaitReview, z as deferNotesScript } from "../chunks/reserved-UdSuHDUJ.mjs";
+import { a as tail, i as refusePendingHandOff, n as closeReview } from "../chunks/tail-fj60rQ3y.mjs";
 //#region src/workflows/feature/tier.ts
 /** GTD_FLAG_NAIVE_PASS, GTD_FLAG_RISK_TIER: both pinned at process start, so a mid-process flip never moves a process. */
 const riskTierOn = () => vars.flag_naive_pass === "1" && vars.flag_risk_tier === "1";
@@ -355,7 +355,7 @@ const review = (base, escalations = { rounds: 0 }) => {
 		awaitReview: (b) => {
 			const notes = deferred;
 			deferred = [];
-			return notes.length > 0 ? awaitReview(b, buildReviewAwaitAfterDeferMessage(b, notes), true) : awaitReview(b);
+			return notes.length > 0 ? awaitReview(b, buildReviewAwaitAfterDeferMessage(b, notes)) : awaitReview(b);
 		},
 		reviewMissing,
 		collecting,

package/dist/workflows/fix.mjs

@@ -1,7 +1,7 @@
 import { read, refuse, removeScript, run, scope, start } from "../flows.mjs";
 import { d as reviewNotes, f as reviewRisks } from "../chunks/steering-DVILanVf.mjs";
-import { O as agentWithSkills, U as finisherSystem, W as fixReviewNotesPrompt, b as coder, c as envDefaults, i as skills, l as baseline, n as summary, o as shipped, r as access, s as defaults, t as steering, v as REVIEW } from "../chunks/reserved-Df8GeSHw.mjs";
-import { r as handOffTail, t as REVIEW_ORIG } from "../chunks/tail-DmIfEAI4.mjs";
+import { O as agentWithSkills, U as finisherSystem, W as fixReviewNotesPrompt, b as coder, c as envDefaults, i as skills, l as baseline, n as summary, o as shipped, r as access, s as defaults, t as steering, v as REVIEW } from "../chunks/reserved-UdSuHDUJ.mjs";
+import { r as handOffTail, t as REVIEW_ORIG } from "../chunks/tail-fj60rQ3y.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,6 +1,6 @@
 import { start } from "../flows.mjs";
-import { c as envDefaults, d as gate, i as skills, n as summary, nt as reviewGateBlockedMessage, o as shipped, r as access, s as defaults, t as steering } from "../chunks/reserved-Df8GeSHw.mjs";
-import { i as refusePendingHandOff, r as handOffTail } from "../chunks/tail-DmIfEAI4.mjs";
+import { c as envDefaults, d as gate, i as skills, n as summary, nt as reviewGateBlockedMessage, o as shipped, r as access, s as defaults, t as steering } from "../chunks/reserved-UdSuHDUJ.mjs";
+import { i as refusePendingHandOff, r as handOffTail } from "../chunks/tail-fj60rQ3y.mjs";
 //#region src/workflows/review/review.ts
 /**
 * Pure review of everything since `reviewBase`: a green gate, the quality

package/dist/workflows/rollout.mjs

@@ -1,5 +1,5 @@
 import { agent, changesSince, head, read, refuse, scope, vars } from "../flows.mjs";
-import { a as flagShipped, b as coder, c as envDefaults, d as gate, i as skills, n as summary, r as access, s as defaults, t as steering } from "../chunks/reserved-Df8GeSHw.mjs";
+import { a as flagShipped, b as coder, c as envDefaults, d as gate, i as skills, n as summary, r as access, s as defaults, t as steering } from "../chunks/reserved-UdSuHDUJ.mjs";
 import { i as requireFlag, n as flipProblems } from "../chunks/registry-CP3YuV9j.mjs";
 //#region src/workflows/rollout/rollout.ts
 const flippingPrompt = (name, registry) => `Roll out the release flag "${name}": in ${registry}, set \`defaultOn: true\` on its entry. Change nothing else — no other entry, no other field, no other file, no comments.`;

package/dist/workflows.mjs

@@ -17995,13 +17995,13 @@ const fixRisks = (notes) => agentWithSkills("review.fix.risks.fixing", buildRevi
 	system: reviewerSystem(),
 	allowEmpty: true
 });
-const awaitReview = (base, message = buildReviewAwaitReviewMessage(base), acceptClean = false) => human("review.await-review", {
+const awaitReview = (base, message = buildReviewAwaitReviewMessage(base)) => human("review.await-review", {
 	message,
 	label: "Awaiting your review",
 	file: REVIEW,
 	mode: "review",
 	base,
-	...acceptClean ? { acceptClean } : {}
+	acceptClean: true
 });
 /** The round at commit `round` left no `.gtd/REVIEW.md`. */
 const reviewMissing = (round) => human("review.review-missing", {
@@ -37911,7 +37911,7 @@ const review$1 = (base, escalations = { rounds: 0 }) => {
 		awaitReview: (b) => {
 			const notes = deferred;
 			deferred = [];
-			return notes.length > 0 ? awaitReview(b, buildReviewAwaitAfterDeferMessage(b, notes), true) : awaitReview(b);
+			return notes.length > 0 ? awaitReview(b, buildReviewAwaitAfterDeferMessage(b, notes)) : awaitReview(b);
 		},
 		reviewMissing,
 		collecting,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pmelab/gtd",
-  "version": "27.7.0",
+  "version": "27.8.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/edge/Edge.test.ts

@@ -596,6 +596,34 @@ export const summary = (it) => it.marks.map((m) => m.name + "=" + m.value).join(
   })
 })
 
+describe("Rest.slices", () => {
+  const SLICED = `import { agent, human, slice } from "@pmelab/gtd/flows"
+
+export default async () => {
+  await human("idle", { message: "idle-message" })
+  await slice("build", async () => {
+    await agent("building", "build-prompt")
+  })
+  await slice("verify", async () => {
+    await agent("checking", "check-prompt")
+  })
+}
+`
+
+  it("folds the replay's spans over the run's base and its last commit", async () => {
+    const repo = repoWith(SLICED)
+    const boundary = headOf(repo)
+    await land(repo, { "a.txt": "a\n" })
+    const checking = await land(repo, { "b.txt": "b\n" })
+    const rest = await provide(currentRest, repo)
+    expect(rest.state).toBe("checking")
+    expect(rest.slices).toEqual([
+      { name: "build", base: boundary, tip: checking, open: false },
+      { name: "verify", base: checking, tip: checking, open: true },
+    ])
+  })
+})
+
 describe("restAt", () => {
   it("restAt(ref) resolves where the process rested at that commit", async () => {
     const repo = repoWith(LINEAR)

package/src/edge/Edge.ts

@@ -37,11 +37,13 @@ import {
   isRecordableApproval,
   parseCommitMessage,
   replay,
+  sliceRanges,
   treeFromRecord,
   type EpisodeCommit,
   type JudgeVerdict,
   type ReachedStep,
   type ReplayOutcome,
+  type SliceRange,
   type TreeView,
 } from "../replay/index.js"
 import type { Mark, ScopeAccess } from "../flows/index.js"
@@ -895,6 +897,7 @@ interface Rest extends ResolvedRest {
   /** Total recorded cost of the process, and its per-model breakdown. */
   readonly cost: { readonly total: number; readonly byModel: readonly ModelCost[] }
   readonly setup: ReplaySetup
+  readonly slices: readonly SliceRange[]
 }
 
 /** Per-model token totals, highest-cost first (ties broken by model name). */
@@ -987,6 +990,15 @@ const pendingChanges = (git: GitOperations, withIndex: boolean) =>
     return [...worktree, ...(yield* git.stagedPaths()).filter((e) => !seen.has(e.path))]
   })
 
+const sliceRangesOf = (
+  slices: Parameters<typeof sliceRanges>[0],
+  run: { readonly startParentHash: string; readonly trace: readonly { readonly hash: string }[] },
+) =>
+  sliceRanges(slices, {
+    base: run.startParentHash,
+    tip: run.trace.at(-1)?.hash ?? run.startParentHash,
+  })
+
 /** Resolve the rest at `ref`, or at HEAD when `ref` is `undefined`. */
 export const restAt = (
   ref: string | undefined,
@@ -1060,6 +1072,7 @@ export const restAt = (
         byModel: costByModel(run.costEntries),
       },
       setup,
+      slices: sliceRangesOf(outcome.slices, run),
     }
   })
 

package/src/flows/runtime.test.ts

@@ -7,6 +7,7 @@ import {
   kept,
   mark,
   protocolMismatch,
+  slice,
   type FlowContext,
 } from "./runtime.js"
 
@@ -127,3 +128,29 @@ describe("kept()", () => {
     expect(FLOWS_PROTOCOL).toBe(1)
   })
 })
+
+describe("slice()", () => {
+  const withSlices = () => {
+    const calls: string[] = []
+    installContext({
+      enterSlice: (n: string) => calls.push(`enter ${n}`),
+      leaveSlice: () => calls.push("leave"),
+    } as unknown as FlowContext)
+    return calls
+  }
+
+  it("brackets fn with enter and leave and resolves to its value", async () => {
+    const calls = withSlices()
+    const value = await slice("refactor", async () => {
+      calls.push("fn")
+      return 42
+    })
+    expect(value).toBe(42)
+    expect(calls).toEqual(["enter refactor", "fn", "leave"])
+  })
+
+  it("runs fn unsliced on a context without enterSlice", async () => {
+    installContext(context)
+    expect(await slice("refactor", async () => "ran")).toBe("ran")
+  })
+})

package/src/flows/runtime.ts

@@ -150,6 +150,9 @@ export interface FlowContext {
   readonly refuse: (message: string) => never
   readonly pushScope: (scope: ScopeOptions) => void
   readonly popScope: () => void
+  /** Absent on an engine older than `slice`; FLOWS_PROTOCOL stays 1. */
+  readonly enterSlice?: (name: string) => void
+  readonly leaveSlice?: () => void
   readonly read: (path: string) => string | undefined
   readonly glob: (pattern: string) => readonly string[]
   /** The last step's changes, content read on demand. */
@@ -284,6 +287,21 @@ export const scope = async <T>(scope: string | ScopeOptions, fn: () => Promise<T
   }
 }
 
+/**
+ * Mark the span `fn` covers as one named slice of the process. Lands no
+ * commit, prefixes no step name, opens no memory scope.
+ */
+export const slice = async <T>(name: string, fn: () => Promise<T>): Promise<T> => {
+  const context = ctx()
+  if (typeof context.enterSlice !== "function") return fn()
+  context.enterSlice(name)
+  try {
+    return await fn()
+  } finally {
+    context.leaveSlice?.()
+  }
+}
+
 // ── Reads of the commit replay stands on ────────────────────────────────────
 
 export const read = (path: string): string | undefined => ctx().read(path)