@pmelab/gtd 51.0.1 → 52.1.0

package/dist/web/main.js

@@ -31565,6 +31565,8 @@ const qaDescriptor = {
 	unansweredQuestions,
 	openThreads: openThreadsOf
 };
+const ANSWER_FIX = "fix as suggested";
+const ANSWER_WONT_FIX = "won't fix";
 /** A chunk's leading `_critical: security_` paragraph: the tier gtd ranked it with, and why. */
 const SEVERITY_LINE_RE = /^_((?:critical|should|could)(?::[^_\n]*)?)_$/;
 /**
@@ -31871,8 +31873,8 @@ const splitSeverity = (content, run) => {
 	};
 };
 /** Splits one chunk's body nodes into its file pointers (hunk pointers are task items, collected recursively at ANY nesting depth via `taskItems` — a nested hunk is the same kind of hunk as a top-level one) and description prose. The description is the chunk's own leading run and never a node that contains a hunk pointer: it stops at the first node whose `taskItems` include a real pointer (at any depth — a blockquote wrapping a pointer list stops it just as a top-level list would). Every node kind in what's left before that point contributes — a `footnoteDefinition` is the one exception, excluded by `blockNodesOfRun` itself (it's a note, never document content). */
-const parseChunkBody = (content, lines, title, body) => {
-	const answer = answerListOf(content, body);
+const parseChunkBody = (content, lines, title, body, answerable) => {
+	const answer = answerable ? answerListOf(content, body) : void 0;
 	const isAnswer = (n) => answer !== void 0 && (n === answer.list || n === body[answer.lead]);
 	const firstPointerIndex = body.findIndex((n) => !isAnswer(n) && taskItems(n).some((item) => hasPointerToken(content, item)));
 	const run = (firstPointerIndex === -1 ? body : body.slice(0, firstPointerIndex)).filter((n) => !isAnswer(n));
@@ -31894,45 +31896,78 @@ const parseChunkBody = (content, lines, title, body) => {
 		descriptionInline,
 		files,
 		errors,
-		...answer === void 0 ? {} : { answers: answer.options },
+		...answer === void 0 ? {} : {
+			answers: answer.options,
+			answerLine: toLspPosition(body[answer.lead].position.start).line
+		},
 		...severity === void 0 ? {} : { severity },
 		boxed: body.some((n) => !isAnswer(n) && taskItems(n).length > 0)
 	};
 };
-/** Splits the tree's top-level nodes into consecutive `##` chunk blocks — a `## ` line found only inside a fence never becomes a heading node at all, so it's never mistaken for a chunk. */
-const splitChunks = (tree) => {
-	const blocks = [];
-	let i = 0;
-	while (i < tree.children.length) {
-		const node = tree.children[i];
-		if (node.type !== "heading" || node.depth !== 2) {
-			i += 1;
-			continue;
-		}
-		const heading = node;
-		const headingLine = toLspPosition(heading.position.start).line;
-		i += 1;
-		const body = [];
-		while (i < tree.children.length) {
-			const next = tree.children[i];
-			if (next.type === "heading" && next.depth <= 2) break;
-			body.push(next);
-			i += 1;
+/** The `## ` headings gtd groups the review's chunks under; any other `## ` is a chunk of its own. */
+const REVIEW_SECTIONS = {
+	findings: "Findings",
+	walkthrough: "Walkthrough"
+};
+const sectionOf = (content, node) => {
+	if (node.type !== "heading" || node.depth !== 2) return void 0;
+	const text = headingText$1(content, node);
+	return Object.keys(REVIEW_SECTIONS).find((s) => REVIEW_SECTIONS[s] === text);
+};
+/** The nodes after `from` up to the next heading at `depth` or shallower. */
+const bodyFrom = (children, from, depth) => {
+	const end = children.findIndex((n, i) => i >= from && n.type === "heading" && n.depth <= depth);
+	return children.slice(from, end === -1 ? void 0 : end);
+};
+/**
+* Splits the tree's top-level nodes into chunk blocks — a `## ` line found only
+* inside a fence never becomes a heading node at all. A document with a
+* `## Findings` or `## Walkthrough` heading nests `### ` chunks under them;
+* one without is the flat layout, every `## ` a chunk.
+*/
+const splitChunks = (tree, content) => {
+	const sectioned = tree.children.some((n) => sectionOf(content, n) !== void 0);
+	const chunks = [];
+	const sections = [];
+	let section;
+	tree.children.forEach((node, i) => {
+		if (node.type !== "heading") return;
+		const headingLine = toLspPosition(node.position.start).line;
+		if (sectioned && node.depth <= 2) {
+			const s = sectionOf(content, node);
+			section = s === void 0 ? void 0 : {
+				heading: node,
+				headingLine,
+				section: s,
+				body: []
+			};
+			if (section !== void 0) {
+				sections.push(section);
+				return;
+			}
 		}
-		blocks.push({
-			heading,
+		const depth = section === void 0 ? 2 : 3;
+		if (node.depth !== depth) return;
+		const body = bodyFrom(tree.children, i + 1, depth);
+		chunks.push({
+			heading: node,
 			headingLine,
-			body
+			body,
+			...section && { section: section.section }
 		});
-	}
-	return blocks;
+		section?.body.push(node, ...body);
+	});
+	return {
+		chunks,
+		sections
+	};
 };
 const parseChangesets = (tree, content, lines) => {
 	const changesets = [];
 	const errors = [];
-	for (const { heading, headingLine, body } of splitChunks(tree)) {
+	for (const { heading, headingLine, body, section } of splitChunks(tree, content).chunks) {
 		const title = headingText$1(content, heading);
-		const { description, descriptionNodes, descriptionInline, files, answers, severity, boxed, errors: bodyErrors } = parseChunkBody(content, lines, title, body);
+		const { description, descriptionNodes, descriptionInline, files, answers, answerLine, severity, boxed, errors: bodyErrors } = parseChunkBody(content, lines, title, body, section !== "walkthrough");
 		errors.push(...bodyErrors);
 		if (files.length === 0 && boxed) errors.push({
 			message: `Chunk "${title}" has no file pointers`,
@@ -31946,8 +31981,12 @@ const parseChangesets = (tree, content, lines) => {
 			descriptionInline,
 			files,
 			headingLine,
-			...answers === void 0 ? {} : { answers },
-			...severity === void 0 ? {} : { severity }
+			...answers === void 0 ? {} : {
+				answers,
+				answerLine
+			},
+			...severity === void 0 ? {} : { severity },
+			...section === void 0 ? {} : { section }
 		});
 	}
 	if (changesets.length === 0) errors.push({ message: "REVIEW.md has no '##' chunks" });
@@ -32029,7 +32068,7 @@ const isHunkPointerItem = (content, item) => {
 */
 const reviewClearTicks = (content) => {
 	const tree = parseMarkdown(content);
-	const answers = new Set(splitChunks(tree).flatMap(({ body }) => answerListOf(content, body)?.list.children ?? []));
+	const answers = new Set(splitChunks(tree, content).chunks.flatMap(({ body }) => answerListOf(content, body)?.list.children ?? []));
 	const offsets = taskItems(tree).filter((item) => item.checked === true && !answers.has(item) && isHunkPointerItem(content, item)).map((item) => checkboxOffset(content, item)).filter((o) => o !== void 0).sort((a, b) => a - b);
 	if (offsets.length === 0) return content;
 	let result = "";
@@ -32066,42 +32105,80 @@ const toggleFilePointer = (content, line) => {
 const chunkEndLines = (content) => {
 	const tree = parseMarkdown(content);
 	const map = /* @__PURE__ */ new Map();
-	for (const block of splitChunks(tree)) map.set(block.headingLine, blockEndLine(block));
+	for (const block of splitChunks(tree, content).chunks) map.set(block.headingLine, blockEndLine(block));
 	return map;
 };
+const CHOICE_NAMES = {
+	fix: ANSWER_FIX,
+	wontfix: ANSWER_WONT_FIX,
+	answer: ""
+};
+/** A finding's decision as an outline node of its own; none for a chunk with no answer boxes. */
+const decisionNodes = (lines, chunk, end) => {
+	if (chunk.answerLine === void 0) return [];
+	const d = decisionOf(chunk);
+	return [{
+		name: `Decision: ${d === void 0 ? "open" : CHOICE_NAMES[d.choice] || d.text}`,
+		range: spanRange(lines, chunk.answerLine, Math.max(chunk.answerLine, end)),
+		selectionRange: lineRange(lines, chunk.answerLine)
+	}];
+};
+const isOpenChunk = (chunk, markers) => chunk.files.some((file) => !file.checked) || markers > 0 || chunk.answers !== void 0 && decisionOf(chunk) === void 0;
 /**
-* Document outline for `.gtd/REVIEW.md`: the headlines of chunks (the
-* user-facing "work packages") that still carry at least one unchecked hunk
-* — the outline is the list of packages left to review — PLUS any chunk
-* that carries at least one footnote, even when every hunk in it is ticked:
-* `reviewOutline` otherwise drops a fully-checked chunk, which would make a
-* human's own comment vanish the moment they tick the last box. A chunk's
-* footnotes are `leaf: true` children; a chunk with none has no children.
+* Document outline for `.gtd/REVIEW.md`: the chunks still left to review — an
+* unchecked hunk, a footnote (so a human's comment never vanishes with the
+* last tick) or an open decision. A finding's decision is a node of its own.
+* In the sectioned layout `Findings` and `Walkthrough` are containers, and
+* every other `## ` (`Fixed automatically`, …) stays listed.
 */
 const reviewOutline = (content) => {
 	const { changesets } = parseReviewDoc(content);
+	const { sections } = splitChunks(parseMarkdown(content), content);
 	const { markers, definitions } = parseFootnotes(content);
 	const definitionByName = new Map(definitions.map((d) => [d.name, d.body]));
 	const lines = content.split(/\r?\n/);
 	const endLines = chunkEndLines(content);
 	const threads = threadOutlineNodes(content);
 	const nested = /* @__PURE__ */ new Set();
-	return [...changesets.map((chunk) => {
+	const chunkNodes = changesets.flatMap((chunk) => {
 		const start = chunk.headingLine;
 		const end = Math.max(start, endLines.get(start) ?? start);
 		const checkedCount = chunk.files.filter((file) => file.checked).length;
 		const chunkMarkers = markers.filter((m) => m.line >= start && m.line <= end);
 		const chunkThreads = threads.filter((t) => t.markerLine !== void 0 && t.markerLine >= start && t.markerLine <= end).map((t) => t.node);
 		chunkThreads.forEach((n) => nested.add(n));
-		const children = [...chunkMarkers.map((m) => footnoteLeaf(lines, definitionByName, m)), ...chunkThreads];
-		return {
-			name: `${chunk.title} (${checkedCount}/${chunk.files.length})`,
+		const children = [
+			...decisionNodes(lines, chunk, end),
+			...chunkMarkers.map((m) => footnoteLeaf(lines, definitionByName, m)),
+			...chunkThreads
+		];
+		if (!(sections.length > 0 && chunk.section === void 0) && !isOpenChunk(chunk, chunkMarkers.length)) return [];
+		const node = {
+			name: chunk.files.length === 0 ? chunk.title : `${chunk.title} (${checkedCount}/${chunk.files.length})`,
 			range: spanRange(lines, start, end),
 			selectionRange: lineRange(lines, start),
-			...children.length > 0 ? { children } : {},
-			unchecked: checkedCount < chunk.files.length || chunkMarkers.length > 0
+			...children.length > 0 ? { children } : {}
 		};
-	}).filter((node) => node.unchecked).map(({ unchecked: _unchecked, ...node }) => node), ...threads.map((t) => t.node).filter((n) => !nested.has(n))];
+		return [{
+			line: start,
+			section: chunk.section,
+			node
+		}];
+	});
+	const containers = sections.map((section) => ({
+		line: section.headingLine,
+		section: void 0,
+		node: {
+			name: REVIEW_SECTIONS[section.section],
+			range: spanRange(lines, section.headingLine, blockEndLine(section)),
+			selectionRange: lineRange(lines, section.headingLine),
+			children: chunkNodes.filter((c) => c.section === section.section && c.line > section.headingLine).filter((c) => {
+				const next = sections.find((o) => o.headingLine > section.headingLine);
+				return next === void 0 || c.line < next.headingLine;
+			}).map((c) => c.node)
+		}
+	}));
+	return [...[...chunkNodes.filter((c) => c.section === void 0), ...containers].sort((a, b) => a.line - b.line).map((c) => c.node), ...threads.map((t) => t.node).filter((n) => !nested.has(n))];
 };
 /**
 * The target state a whole-chunk toggle drives every hunk to: `true` (check
@@ -32153,6 +32230,7 @@ const reviewActions = (content, range) => {
 	const tree = parseMarkdown(content);
 	const lines = content.split(/\r?\n/);
 	const cursorLine = range.start.line;
+	const endLines = chunkEndLines(content);
 	const actions = [...threadReplyActions(content, range.start)];
 	changesets.forEach((chunk, i) => {
 		const hunk = chunk.files.find((file) => cursorLine >= file.sourceLine && cursorLine <= file.endLine);
@@ -32163,7 +32241,7 @@ const reviewActions = (content, range) => {
 				edits: [edit]
 			});
 		}
-		const chunkEnd = Math.max(chunk.headingLine, (changesets[i + 1]?.headingLine ?? lines.length) - 1);
+		const chunkEnd = chunk.section === void 0 ? Math.max(chunk.headingLine, (changesets[i + 1]?.headingLine ?? lines.length) - 1) : endLines.get(chunk.headingLine) ?? chunk.headingLine;
 		if (chunk.files.length > 0 && cursorLine >= chunk.headingLine && cursorLine <= chunkEnd) {
 			const edits = toggleChunkEdits(content, chunk.headingLine);
 			if (edits.length > 0) {
@@ -32231,7 +32309,7 @@ const hunkLinkFor = (content, item) => {
 const reviewDocumentLinks = (content) => {
 	const tree = parseMarkdown(content);
 	const links = [];
-	for (const block of splitChunks(tree)) for (const item of block.body.flatMap((n) => taskItems(n))) {
+	for (const block of splitChunks(tree, content).chunks) for (const item of block.body.flatMap((n) => taskItems(n))) {
 		const link = hunkLinkFor(content, item);
 		if (link) links.push(link);
 	}
@@ -32253,6 +32331,19 @@ const reviewView = (content) => {
 				kind: "chunk",
 				index: chunkIndex
 			},
+			...chunk.section === void 0 ? {} : { group: REVIEW_SECTIONS[chunk.section] },
+			...chunk.answers === void 0 ? {} : {
+				answered: decisionOf(chunk) !== void 0,
+				decision: chunk.answers.map((option, index) => ({
+					title: option.text,
+					checked: option.checked,
+					anchor: {
+						kind: "option",
+						questionIndex: chunkIndex,
+						index
+					}
+				}))
+			},
 			...noteAt(chunk.headingLine),
 			children: chunk.files.map((file, index) => ({
 				title: pointerDisplay(file),
@@ -32350,8 +32441,66 @@ const setFileTickEdits = (content, files, checked) => {
 	}
 	return edits;
 };
-/** A `hunk` anchor sets one tick; a `chunk` anchor sets every hunk beneath it to the SAME target state — never `toggleChunkEdits`'s majority-flip heuristic, since the caller already knows the state it wants. */
+/**
+* A finding's answer boxes are radio: ticking one unticks the others, and
+* `text` replaces the ticked box's label in the same edit set — the phone's
+* free-text answer, normalized as `qa`'s is.
+*/
+const decisionEdits = (content, chunkIndex, index, opts) => {
+	const items = answerItemsOf(content, chunkIndex);
+	const target = items[index];
+	if (target === void 0) return {
+		ok: false,
+		reason: "anchor-not-found"
+	};
+	const checked = opts.checked ?? true;
+	const ticks = items.flatMap((item) => {
+		const want = item === target ? checked : !checked && item.checked === true;
+		return item.checked === true === want ? [] : tickEdit(content, item, want);
+	});
+	const text = opts.text === void 0 ? [] : labelEdit(content, target, opts.text);
+	return {
+		ok: true,
+		edits: [...ticks, ...text]
+	};
+};
+/** A finding's answer boxes; none for a walkthrough chunk or a stale index. */
+const answerItemsOf = (content, chunkIndex) => {
+	const chunk = splitChunks(parseMarkdown(content), content).chunks[chunkIndex];
+	if (chunk === void 0 || chunk.section === "walkthrough") return [];
+	return answerListOf(content, chunk.body)?.list.children ?? [];
+};
+const tickEdit = (content, item, checked) => {
+	const offset = checkboxOffset(content, item);
+	if (offset === void 0) return [];
+	const start = toLspPositionFromOffset(content, offset);
+	return [{
+		range: {
+			start,
+			end: {
+				...start,
+				character: start.character + 1
+			}
+		},
+		newText: checked ? "x" : " "
+	}];
+};
+/** Replaces a box's label, from after its marker to the end of its first paragraph. */
+const labelEdit = (content, item, text) => {
+	const from = firstParagraphContentOffset(item);
+	const to = item.children.find((c) => c.type === "paragraph")?.position?.end.offset;
+	if (from === void 0 || to === void 0) return [];
+	return [{
+		range: {
+			start: toLspPositionFromOffset(content, from),
+			end: toLspPositionFromOffset(content, to)
+		},
+		newText: normalizeFreeTextAnswer(text)
+	}];
+};
+/** A `hunk` anchor sets one tick; a `chunk` anchor sets every hunk beneath it to the SAME target state — never `toggleChunkEdits`'s majority-flip heuristic, since the caller already knows the state it wants. An `option` anchor is a finding's decision. */
 const reviewApply = (content, anchor, opts) => {
+	if (anchor.kind === "option") return decisionEdits(content, anchor.questionIndex, anchor.index, opts);
 	const { changesets } = parseReviewDoc(content);
 	const checked = opts.checked ?? true;
 	if (anchor.kind === "hunk") {
@@ -32381,10 +32530,8 @@ const reviewApply = (content, anchor, opts) => {
 		reason: "anchor-not-found"
 	};
 };
-//#endregion
-//#region src/steering/index.ts
-/** The built-in registry: mode name → its format, in registry order. The single place `qa` and `review` are wired into names — everything else in this module dispatches over a resolved `SteeringFormat`, never a mode-name string. */
-const REGISTRY = /* @__PURE__ */ new Map([["qa", qaDescriptor], ["review", {
+/** The `review` format: `.gtd/REVIEW.md`'s in-process format — validation, outline, code actions, footnote + hunk `pointerAt`/`documentLinks`, and a hunk-pointer-scoped `clearTicks` (never "every checked task item" — see `reviewClearTicks`). */
+const reviewDescriptor = {
 	sample: REVIEW_SAMPLE,
 	validate: (content) => [...parseReviewDoc(content).findings, ...parseFootnotes(content).findings],
 	outline: reviewOutline,
@@ -32396,7 +32543,30 @@ const REGISTRY = /* @__PURE__ */ new Map([["qa", qaDescriptor], ["review", {
 	apply: reviewApply,
 	clearTicks: reviewClearTicks,
 	openThreads: openThreadsOf
-}]]);
+};
+const CHOICES = [
+	"fix",
+	"wontfix",
+	"answer"
+];
+const decisionOf = (chunk) => {
+	if (chunk.answers?.length !== 3) return void 0;
+	const ticked = chunk.answers.flatMap((o, i) => o.checked ? [i] : []);
+	if (ticked.length !== 1) return void 0;
+	const i = ticked[0];
+	const text = i === 2 ? chunk.answers[2].text : "";
+	if (i === 2 && text === "") return void 0;
+	return {
+		anchor: chunk.title,
+		body: chunk.description,
+		choice: CHOICES[i],
+		text
+	};
+};
+//#endregion
+//#region src/steering/index.ts
+/** The built-in registry: mode name → its format, in registry order. The single place `qa` and `review` are wired into names — everything else in this module dispatches over a resolved `SteeringFormat`, never a mode-name string. */
+const REGISTRY = /* @__PURE__ */ new Map([["qa", qaDescriptor], ["review", reviewDescriptor]]);
 [...REGISTRY.keys()];
 /** The built-in `SteeringFormat` registered under `mode`'s name, or `undefined` when it isn't a built-in mode at all. The one place a bare mode-name string is ever looked up — every consumer takes the RESOLVED value this returns, never a name, so "mode is required, never inferred from a file's basename" is a type, not a rule to remember. */
 const steeringFormatFor = (mode) => REGISTRY.get(mode);
@@ -33836,7 +34006,7 @@ const useScrollRestoration = () => {
 //#endregion
 //#region src/web/screens/Question.tsx
 /** `""` for an untouched/placeholder-only answer (case-insensitive) — the SAME sentinel and the SAME normalization the completeness gate and the open-questions check both apply server-side (`OpenQuestions.ts#FREE_TEXT_PLACEHOLDER`), redone here so the client never has to round-trip through a write to know if it's answered. */
-const normalizeAnswerText = (text) => {
+const normalizeAnswerText$1 = (text) => {
 	const trimmed = text.trim();
 	return trimmed.toLowerCase() === "_your answer_".toLowerCase() ? "" : trimmed;
 };
@@ -33946,7 +34116,7 @@ const Question = ({ node, filePath, onCommitAnswer, noteOverrides, onOpenNote })
 	const openAnswerSheet = () => setAnswerSheetOpen(true);
 	/** Fired only by a deliberate Save tap — both fields go in one write, one cell. */
 	const commitFreeText = (text) => {
-		const current = normalizeAnswerText(text);
+		const current = normalizeAnswerText$1(text);
 		const anchor = freeTextAnchor();
 		if (anchor === void 0) return;
 		const checked = current.length > 0;
@@ -33967,7 +34137,7 @@ const Question = ({ node, filePath, onCommitAnswer, noteOverrides, onOpenNote })
 	};
 	const answered = isAnswered(options.map((option, index) => ({
 		checked: selected === index,
-		text: index === lastIndex ? normalizeAnswerText(freeText) : option.title,
+		text: index === lastIndex ? normalizeAnswerText$1(freeText) : option.title,
 		freeText: index === lastIndex
 	})));
 	const freeTextOptionAnchor = freeTextAnchor();
@@ -34612,6 +34782,19 @@ const Hunk = ({ node, diff, index, total, checked, hasNote, note, thread, onTogg
 //#region src/web/screens/Review.tsx
 /** Every hunk-anchored descendant of a chunk node, at any depth. */
 const hunksOf = (node) => (node.children ?? []).flatMap((child) => [...child.anchor.kind === "hunk" ? [child] : [], ...hunksOf(child)]);
+/** The free-text slot's index among a finding's three boxes. */
+const ANSWER_SLOT = 2;
+const defaultDecisionFor = ({ decision = [] }) => {
+	const ticked = decision.flatMap((option, i) => option.checked === true ? [i] : []);
+	return {
+		selected: ticked.length === 1 ? ticked[0] : void 0,
+		freeText: decision[ANSWER_SLOT]?.title ?? ""
+	};
+};
+const normalizeAnswerText = (text) => {
+	const trimmed = text.trim();
+	return trimmed.toLowerCase() === "_your answer_".toLowerCase() ? "" : trimmed;
+};
 /**
 * All of `ReviewView`'s local/optimistic UI state plus the derived helpers
 * every branch below needs. `ticked`/notes read straight off the write
@@ -34619,17 +34802,56 @@ const hunksOf = (node) => (node.children ?? []).flatMap((child) => [...child.anc
 * falls back to the node's own last-server-confirmed value automatically —
 * no local `useState`, no per-hunk seq guard of its own.
 */
-const useReviewState = (filePath, scrollRef, onSaveNote, onDoneNote, onSetValue) => {
+const useReviewState = (filePath, scrollRef, onSaveNote, onDoneNote, onSetValue, onCommitDecision) => {
 	const overlay = useWriteStore((s) => s.overlay);
 	const save = useWriteStore((s) => s.save);
 	const shadow = useWriteStore((s) => s.shadow);
 	const [openChunkIndex, setOpenChunkIndex] = (0, import_react.useState)(void 0);
 	const [deckIndex, setDeckIndex] = (0, import_react.useState)(0);
 	const [noteSheet, setNoteSheet] = (0, import_react.useState)(void 0);
+	const [answerSheet, setAnswerSheet] = (0, import_react.useState)(void 0);
 	const scroll = useScrollRestoration();
 	const checkedCell = (anchor) => cellKey(filePath, anchor, "checked");
 	const noteCell = (anchor) => cellKey(filePath, anchor, "note");
 	const doneCell = (anchor) => cellKey(filePath, anchor, "done");
+	const decisionCell = (chunk) => cellKey(filePath, chunk.anchor, "decision");
+	const decisionOf = (chunk) => overlay[decisionCell(chunk)]?.value ?? defaultDecisionFor(chunk);
+	/** The same rule the landing's `answer-completeness` check applies: three boxes, one ticked, a ticked answer filled in. */
+	const isDecided = (chunk) => {
+		const options = chunk.decision ?? [];
+		const { selected, freeText } = decisionOf(chunk);
+		return options.length === 3 && isAnswered(options.map((option, i) => ({
+			checked: selected === i,
+			text: i === ANSWER_SLOT ? normalizeAnswerText(freeText) : option.title,
+			freeText: i === ANSWER_SLOT
+		})));
+	};
+	const writeDecision = (chunk, index, optimistic, opts) => {
+		const anchor = chunk.decision?.[index]?.anchor;
+		if (anchor === void 0) return;
+		save({
+			cell: decisionCell(chunk),
+			optimistic,
+			write: onCommitDecision === void 0 ? void 0 : onCommitDecision(anchor, opts)
+		}).catch(() => {});
+	};
+	const chooseDecision = (chunk, index) => writeDecision(chunk, index, {
+		...decisionOf(chunk),
+		selected: index
+	}, { checked: true });
+	/** A Save in the answer sheet: an empty answer unticks the slot rather than ticking a blank one. */
+	const commitAnswer = (chunk, text) => {
+		setAnswerSheet(void 0);
+		const answer = normalizeAnswerText(text);
+		const { selected } = decisionOf(chunk);
+		writeDecision(chunk, ANSWER_SLOT, {
+			selected: answer === "" ? selected === ANSWER_SLOT ? void 0 : selected : ANSWER_SLOT,
+			freeText: answer
+		}, {
+			checked: answer !== "",
+			text: answer
+		});
+	};
 	const isChecked = (hunk) => overlay[checkedCell(hunk.anchor)]?.value ?? hunk.checked === true;
 	/** Reads BOTH a plain save's own `"note"` cell and that same anchor's `"done"` cell (a Save & Done) — only one of the two is ever written per interaction, so whichever is newer is what should show. */
 	const savedNoteOf = (node) => latestOverlayValue(overlay, [noteCell(node.anchor), doneCell(node.anchor)]);
@@ -34706,7 +34928,13 @@ const useReviewState = (filePath, scrollRef, onSaveNote, onDoneNote, onSetValue)
 		setDeckIndex,
 		noteSheet,
 		setNoteSheet,
+		answerSheet,
+		setAnswerSheet,
 		exitToChunkList,
+		decisionOf,
+		isDecided,
+		chooseDecision,
+		commitAnswer,
 		isChecked,
 		hasNoteText,
 		noteTextOf,
@@ -34744,7 +34972,7 @@ const hunkPropsFor = (hunk, index, hunks, state) => ({
 	onApprove: () => state.approveAndAdvance(hunks),
 	onOpenNote: () => state.openNoteSheet(hunk)
 });
-const HunkDeck = ({ chunk, live, state, onDone, busy }) => {
+const HunkDeck = ({ chunk, live, state, onDone, busy, undecided }) => {
 	const hunks = hunksOf(chunk);
 	return /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", {
 		"data-testid": "hunk-deck",
@@ -34757,7 +34985,7 @@ const HunkDeck = ({ chunk, live, state, onDone, busy }) => {
 			...onDone !== void 0 ? {
 				onDone,
 				doneLabel: "Done",
-				doneDisabled: busy
+				doneDisabled: busy === true || undecided > 0
 			} : {},
 			renderItem: (hunk, i) => {
 				const props = hunkPropsFor(hunk, i, hunks, state);
@@ -34770,85 +34998,184 @@ const HunkDeck = ({ chunk, live, state, onDone, busy }) => {
 		})
 	});
 };
+const DECISION_LABELS = [
+	"Fix as suggested",
+	"Won't fix",
+	"Your answer"
+];
+/** A finding's decision: three radios, the last opening the answer sheet. */
+const DecisionControl = ({ chunk, chunkIndex, state }) => {
+	const { selected, freeText } = state.decisionOf(chunk);
+	const choose = (i) => i === ANSWER_SLOT ? state.setAnswerSheet(chunk) : state.chooseDecision(chunk, i);
+	return /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("fieldset", {
+		"data-testid": `chunk-decision-${chunkIndex}`,
+		className: "m-0 mt-1 flex flex-col border-0 p-0 pl-13",
+		children: [
+			/* @__PURE__ */ (0, import_jsx_runtime.jsxs)("legend", {
+				className: "sr-only",
+				children: ["Decision for ", chunk.title]
+			}),
+			(chunk.decision ?? []).map((option, i) => /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("label", {
+				className: "flex min-h-11 items-center gap-3",
+				children: [/* @__PURE__ */ (0, import_jsx_runtime.jsx)("input", {
+					type: "radio",
+					name: `decision-${chunkIndex}`,
+					"data-testid": `chunk-decision-${chunkIndex}-${i}`,
+					checked: selected === i,
+					onChange: () => choose(i)
+				}), /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", {
+					className: selected === i ? "font-medium" : void 0,
+					children: DECISION_LABELS[i] ?? option.title
+				})]
+			}, i)),
+			/* @__PURE__ */ (0, import_jsx_runtime.jsx)(Button, {
+				variant: "ghost",
+				"data-testid": `chunk-decision-answer-${chunkIndex}`,
+				onClick: () => state.setAnswerSheet(chunk),
+				className: "w-full px-0 pl-7 text-left text-small font-normal text-muted",
+				children: freeText.trim().length > 0 ? freeText : "Tap to write your answer"
+			})
+		]
+	});
+};
+/** A chunk with neither hunks nor a decision (`Fixed automatically`, …) is a note to read, not an item to tick. */
+const isPlainNote = (chunk) => chunk.decision === void 0 && hunksOf(chunk).length === 0;
+const ChunkSummary = ({ chunk, chunkIndex }) => /* @__PURE__ */ (0, import_jsx_runtime.jsxs)(import_jsx_runtime.Fragment, { children: [
+	chunk.status !== void 0 && /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", {
+		"data-testid": `chunk-severity-${chunkIndex}`,
+		className: cn("text-small font-semibold", chunk.status.startsWith("critical") ? "text-warning" : "text-muted"),
+		children: chunk.status
+	}),
+	/* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", {
+		className: "font-semibold",
+		children: chunk.title
+	}),
+	chunk.detail !== void 0 && chunk.detail.length > 0 && /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", {
+		className: "text-small text-muted",
+		children: /* @__PURE__ */ (0, import_jsx_runtime.jsx)(Inline, {
+			inline: chunk.detailInline,
+			fallback: chunk.detail
+		})
+	})
+] });
+/** A finding's done mark, a walkthrough chunk's tick-all box, nothing for a plain note. */
+const ChunkMark = ({ chunk, chunkIndex, state }) => {
+	const hunks = hunksOf(chunk);
+	if (chunk.decision !== void 0) {
+		const decided = state.isDecided(chunk);
+		return /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", {
+			"data-testid": `chunk-decision-status-${chunkIndex}`,
+			"aria-label": decided ? "Decided" : "Needs a decision",
+			className: cn("flex min-h-11 min-w-11 shrink-0 items-center justify-center font-semibold", decided ? "text-heading-c" : "text-warning"),
+			children: decided ? "✓" : "●"
+		});
+	}
+	if (hunks.length === 0) return null;
+	return /* @__PURE__ */ (0, import_jsx_runtime.jsx)("label", {
+		"data-testid": `chunk-check-all-label-${chunkIndex}`,
+		className: "flex min-h-11 min-w-11 shrink-0 items-center justify-center",
+		children: /* @__PURE__ */ (0, import_jsx_runtime.jsx)("input", {
+			type: "checkbox",
+			"data-testid": `chunk-check-all-${chunkIndex}`,
+			checked: hunks.every(state.isChecked),
+			onChange: () => state.toggleChunk(chunk)
+		})
+	});
+};
 const ChunkRow = ({ chunk, chunkIndex, state }) => {
 	const hunks = hunksOf(chunk);
-	const allChecked = hunks.length > 0 && hunks.every(state.isChecked);
 	const footnoteKeepsRoundOpen = state.hasNoteText(chunk);
 	return /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", {
 		"data-testid": `chunk-card-${chunkIndex}`,
 		className: "border-b border-divider px-3 py-2.5",
-		children: [/* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", {
-			className: "flex items-start gap-2",
-			children: [
-				/* @__PURE__ */ (0, import_jsx_runtime.jsx)("label", {
-					"data-testid": `chunk-check-all-label-${chunkIndex}`,
-					className: "flex min-h-11 min-w-11 shrink-0 items-center justify-center",
-					children: /* @__PURE__ */ (0, import_jsx_runtime.jsx)("input", {
-						type: "checkbox",
-						"data-testid": `chunk-check-all-${chunkIndex}`,
-						checked: allChecked,
+		children: [
+			/* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", {
+				className: "flex items-start gap-2",
+				children: [
+					/* @__PURE__ */ (0, import_jsx_runtime.jsx)(ChunkMark, {
+						chunk,
+						chunkIndex,
+						state
+					}),
+					isPlainNote(chunk) ? /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", {
+						"data-testid": `chunk-plain-note-${chunkIndex}`,
+						className: "flex-1 py-1",
+						children: /* @__PURE__ */ (0, import_jsx_runtime.jsx)(ChunkSummary, {
+							chunk,
+							chunkIndex
+						})
+					}) : /* @__PURE__ */ (0, import_jsx_runtime.jsxs)(Button, {
+						variant: "ghost",
+						"data-testid": `chunk-open-${chunkIndex}`,
+						onClick: () => state.openChunk(chunkIndex),
 						disabled: hunks.length === 0,
-						onChange: () => state.toggleChunk(chunk)
-					})
-				}),
-				/* @__PURE__ */ (0, import_jsx_runtime.jsxs)(Button, {
-					variant: "ghost",
-					"data-testid": `chunk-open-${chunkIndex}`,
-					onClick: () => state.openChunk(chunkIndex),
-					disabled: hunks.length === 0,
-					className: "flex-1 text-left",
-					children: [
-						chunk.status !== void 0 && /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", {
-							"data-testid": `chunk-severity-${chunkIndex}`,
-							className: cn("text-small font-semibold", chunk.status.startsWith("critical") ? "text-warning" : "text-muted"),
-							children: chunk.status
-						}),
-						/* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", {
-							className: "font-semibold",
-							children: chunk.title
-						}),
-						chunk.detail !== void 0 && chunk.detail.length > 0 && /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", {
-							className: "text-small text-muted",
-							children: /* @__PURE__ */ (0, import_jsx_runtime.jsx)(Inline, {
-								inline: chunk.detailInline,
-								fallback: chunk.detail
+						className: "flex-1 text-left",
+						children: [
+							/* @__PURE__ */ (0, import_jsx_runtime.jsx)(ChunkSummary, {
+								chunk,
+								chunkIndex
+							}),
+							hunks.length === 0 && /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", {
+								className: "mt-0.5 text-small text-muted",
+								children: "No file pointers"
+							}),
+							footnoteKeepsRoundOpen && /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", {
+								"data-testid": `chunk-footnote-badge-${chunkIndex}`,
+								className: "mt-1 text-small text-warning",
+								children: "Note keeps this round open"
 							})
-						}),
-						hunks.length === 0 && /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", {
-							className: "mt-0.5 text-small text-muted",
-							children: "No file pointers"
-						}),
-						footnoteKeepsRoundOpen && /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", {
-							"data-testid": `chunk-footnote-badge-${chunkIndex}`,
-							className: "mt-1 text-small text-warning",
-							children: "Note keeps this round open"
-						})
-					]
-				}),
-				!footnoteKeepsRoundOpen && /* @__PURE__ */ (0, import_jsx_runtime.jsx)(Button, {
-					variant: "secondary",
-					"data-testid": `chunk-note-${chunkIndex}`,
-					onClick: () => state.openNoteSheet(chunk),
-					children: "Note"
-				})
-			]
-		}), footnoteKeepsRoundOpen && /* @__PURE__ */ (0, import_jsx_runtime.jsx)(Button, {
-			variant: "ghost",
-			"data-testid": `chunk-note-${chunkIndex}`,
-			onClick: () => state.openNoteSheet(chunk),
-			className: "w-full rounded border border-divider px-2 py-1 text-left text-small font-normal text-muted",
-			children: state.threadOf(chunk) !== void 0 ? /* @__PURE__ */ (0, import_jsx_runtime.jsx)(Thread, {
-				thread: state.threadOf(chunk),
-				testId: `chunk-thread-${chunkIndex}`
-			}) : state.noteTextOf(chunk)
-		})]
+						]
+					}),
+					!footnoteKeepsRoundOpen && /* @__PURE__ */ (0, import_jsx_runtime.jsx)(Button, {
+						variant: "secondary",
+						"data-testid": `chunk-note-${chunkIndex}`,
+						onClick: () => state.openNoteSheet(chunk),
+						children: "Note"
+					})
+				]
+			}),
+			chunk.decision !== void 0 && /* @__PURE__ */ (0, import_jsx_runtime.jsx)(DecisionControl, {
+				chunk,
+				chunkIndex,
+				state
+			}),
+			footnoteKeepsRoundOpen && /* @__PURE__ */ (0, import_jsx_runtime.jsx)(Button, {
+				variant: "ghost",
+				"data-testid": `chunk-note-${chunkIndex}`,
+				onClick: () => state.openNoteSheet(chunk),
+				className: "w-full rounded border border-divider px-2 py-1 text-left text-small font-normal text-muted",
+				children: state.threadOf(chunk) !== void 0 ? /* @__PURE__ */ (0, import_jsx_runtime.jsx)(Thread, {
+					thread: state.threadOf(chunk),
+					testId: `chunk-thread-${chunkIndex}`
+				}) : state.noteTextOf(chunk)
+			})
+		]
 	});
 };
-const approvedCount = (nodes, state) => nodes.filter((chunk) => {
+/** A finding is done once decided, any other chunk once every hunk is ticked; a plain note never counts. */
+const isChunkDone = (chunk, state) => {
+	if (chunk.decision !== void 0) return state.isDecided(chunk);
 	const hunks = hunksOf(chunk);
 	return hunks.length > 0 && hunks.every(state.isChecked);
-}).length;
-const ChunkList = ({ nodes, state, scrollRef, onDone, busy }) => /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", {
+};
+const undecidedCount = (nodes, state) => nodes.filter((chunk) => chunk.decision !== void 0 && !state.isDecided(chunk)).length;
+/** Consecutive chunks sharing a `group` (the review's Findings / Walkthrough), chunk indexes kept for their anchors. */
+const groupsOf = (nodes) => nodes.reduce((groups, chunk, chunkIndex) => {
+	const last = groups.at(-1);
+	if (last !== void 0 && last.title === chunk.group) last.chunks.push({
+		chunk,
+		chunkIndex
+	});
+	else groups.push({
+		title: chunk.group,
+		chunks: [{
+			chunk,
+			chunkIndex
+		}]
+	});
+	return groups;
+}, []);
+const ChunkList = ({ nodes, state, scrollRef, onDone, busy, undecided }) => /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", {
 	"data-testid": "review-screen",
 	className: "flex h-full min-h-0 flex-1 flex-col",
 	children: [
@@ -34861,9 +35188,10 @@ const ChunkList = ({ nodes, state, scrollRef, onDone, busy }) => /* @__PURE__ */
 			}), /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("span", {
 				className: "text-small text-muted tabular-nums",
 				children: [
-					approvedCount(nodes, state),
-					" / ",
-					nodes.length,
+					nodes.filter((chunk) => isChunkDone(chunk, state)).length,
+					" /",
+					" ",
+					nodes.filter((chunk) => !isPlainNote(chunk)).length,
 					" chunks approved"
 				]
 			})]
@@ -34872,22 +35200,32 @@ const ChunkList = ({ nodes, state, scrollRef, onDone, busy }) => /* @__PURE__ */
 			ref: scrollRef,
 			"data-testid": "review-scroll-container",
 			className: "min-h-0 flex-1 overflow-auto",
-			children: /* @__PURE__ */ (0, import_jsx_runtime.jsx)(CardList, { children: nodes.map((chunk, chunkIndex) => /* @__PURE__ */ (0, import_jsx_runtime.jsx)(ChunkRow, {
-				chunk,
-				chunkIndex,
-				state
-			}, chunkIndex)) })
+			children: groupsOf(nodes).map((group) => /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("section", {
+				...group.title === void 0 ? {} : { "data-testid": `review-section-${group.title.toLowerCase()}` },
+				children: [group.title !== void 0 && /* @__PURE__ */ (0, import_jsx_runtime.jsx)("h2", {
+					className: "m-0 border-b border-divider bg-surface px-3 py-1.5 text-small font-semibold text-muted",
+					children: group.title
+				}), /* @__PURE__ */ (0, import_jsx_runtime.jsx)(CardList, { children: group.chunks.map(({ chunk, chunkIndex }) => /* @__PURE__ */ (0, import_jsx_runtime.jsx)(ChunkRow, {
+					chunk,
+					chunkIndex,
+					state
+				}, chunkIndex)) })]
+			}, group.chunks[0].chunkIndex))
 		}),
-		onDone !== void 0 && /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", {
+		onDone !== void 0 && /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", {
 			"data-testid": "review-done-row",
-			className: "flex shrink-0 items-center justify-end border-t border-border p-3",
-			children: /* @__PURE__ */ (0, import_jsx_runtime.jsx)(Button, {
+			className: "flex shrink-0 items-center justify-end gap-3 border-t border-border p-3",
+			children: [undecided > 0 && /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("span", {
+				"data-testid": "review-done-blocked",
+				className: "text-small text-warning",
+				children: [undecided === 1 ? "1 finding needs" : `${undecided} findings need`, " a decision"]
+			}), /* @__PURE__ */ (0, import_jsx_runtime.jsx)(Button, {
 				variant: "primary",
 				"data-testid": "review-done",
 				onClick: onDone,
-				disabled: busy,
+				disabled: busy === true || undecided > 0,
 				children: "Done"
-			})
+			})]
 		})
 	]
 });
@@ -34895,10 +35233,10 @@ const ChunkList = ({ nodes, state, scrollRef, onDone, busy }) => /* @__PURE__ */
 * Presentational review screen — chunk list drilling into a per-chunk deck
 * of hunks, mirroring `Plan.tsx`'s `PlanView`/`Plan` split.
 */
-const ReviewView = ({ view, filePath, isLoading, readError, live, onSaveNote, onDoneNote, onDone, onSetValue, busy }) => {
+const ReviewView = ({ view, filePath, isLoading, readError, live, onSaveNote, onDoneNote, onDone, onSetValue, onCommitDecision, busy }) => {
 	const scrollRef = (0, import_react.useRef)(null);
 	const save = useWriteStore((s) => s.save);
-	const state = useReviewState(filePath, scrollRef, onSaveNote, onDoneNote, onSetValue);
+	const state = useReviewState(filePath, scrollRef, onSaveNote, onDoneNote, onSetValue, onCommitDecision);
 	if (view === void 0) {
 		const refusal = readError !== void 0 ? readRefusalFrom(readError) : void 0;
 		return /* @__PURE__ */ (0, import_jsx_runtime.jsx)(Notice, {
@@ -34906,8 +35244,19 @@ const ReviewView = ({ view, filePath, isLoading, readError, live, onSaveNote, on
 			children: isLoading ? "Loading the review…" : refusal !== void 0 ? messageForReadRefusal(refusal) : "Could not load the review."
 		});
 	}
+	const answerChunk = state.answerSheet;
+	const answerAnchor = answerChunk?.decision?.[ANSWER_SLOT]?.anchor;
+	const answerSheet = answerChunk !== void 0 && answerAnchor !== void 0 ? /* @__PURE__ */ (0, import_jsx_runtime.jsx)(NoteSheet, {
+		anchor: answerAnchor,
+		title: "Your answer",
+		label: "Answer text",
+		note: state.decisionOf(answerChunk).freeText,
+		onSave: (_anchor, text) => state.commitAnswer(answerChunk, text),
+		onDismiss: () => state.setAnswerSheet(void 0)
+	}) : null;
+	const undecided = undecidedCount(view.nodes, state);
 	const noteSheet = (() => {
-		if (state.noteSheet === void 0) return null;
+		if (state.noteSheet === void 0) return answerSheet;
 		return /* @__PURE__ */ (0, import_jsx_runtime.jsx)(NoteSheet, {
 			anchor: state.noteSheet.anchor,
 			...state.noteSheet.initialNote !== void 0 ? { note: state.noteSheet.initialNote } : {},
@@ -34933,14 +35282,16 @@ const ReviewView = ({ view, filePath, isLoading, readError, live, onSaveNote, on
 		live: live === true,
 		state,
 		onDone: onDone === void 0 ? void 0 : runDone,
-		busy
+		busy,
+		undecided
 	}), noteSheet] });
 	return /* @__PURE__ */ (0, import_jsx_runtime.jsxs)(import_jsx_runtime.Fragment, { children: [/* @__PURE__ */ (0, import_jsx_runtime.jsx)(ChunkList, {
 		nodes: view.nodes,
 		state,
 		scrollRef,
 		onDone: onDone === void 0 ? void 0 : runDone,
-		busy
+		busy,
+		undecided
 	}), noteSheet] });
 };
 /** The terminal panel after `done` resolves. */
@@ -34972,6 +35323,16 @@ const useReviewMutations = (filePath, onFormatNotice) => {
 		onFormatNotice(result.formatNotice);
 		return result;
 	});
+	const onCommitDecision = (anchor, opts) => (cas) => setValue.mutateAsync({
+		filePath,
+		...cas,
+		mode: "review",
+		anchor,
+		...opts
+	}).then((result) => {
+		onFormatNotice(result.formatNotice);
+		return result;
+	});
 	const onSaveNote = (anchor, text) => (cas) => writeNote.mutateAsync({
 		filePath,
 		...cas,
@@ -34995,6 +35356,7 @@ const useReviewMutations = (filePath, onFormatNotice) => {
 		onDoneNote,
 		onDone,
 		onSetValue,
+		onCommitDecision,
 		isDone: done.isSuccess
 	};
 };
@@ -35003,7 +35365,7 @@ const ReviewInner = ({ filePath, data, isLoading, error }) => {
 	const indicatorProps = useSaveIndicatorProps();
 	const pending = useWriteStore((s) => s.pendingCount);
 	const [formatNotice, setFormatNotice] = (0, import_react.useState)(void 0);
-	const { onSaveNote, onDoneNote, onDone, onSetValue, isDone } = useReviewMutations(filePath, setFormatNotice);
+	const { onSaveNote, onDoneNote, onDone, onSetValue, onCommitDecision, isDone } = useReviewMutations(filePath, setFormatNotice);
 	return /* @__PURE__ */ (0, import_jsx_runtime.jsxs)(import_jsx_runtime.Fragment, { children: [
 		/* @__PURE__ */ (0, import_jsx_runtime.jsx)(SaveIndicator, { ...indicatorProps }),
 		/* @__PURE__ */ (0, import_jsx_runtime.jsx)(FormatNoticeBanner, {
@@ -35020,6 +35382,7 @@ const ReviewInner = ({ filePath, data, isLoading, error }) => {
 			onDoneNote,
 			onDone,
 			onSetValue,
+			onCommitDecision,
 			busy: pending > 0
 		})
 	] });