@danypops/pi-papyrus 0.43.1 → 0.43.3

package/extension/src/discuss.ts

@@ -12,11 +12,12 @@
  * playbooks.ts precedent -- Notes is the one kind with a human-facing creation command (/note),
  * because Notes exists specifically as a human-authored inbox.
  */
+
+import { type Artifact, type DiscussionAndRounds, readDiscussionExtra } from "@danypops/papyrus";
 import type { ExtensionCommandContext, Theme } from "@earendil-works/pi-coding-agent";
-import { readDiscussionExtra, type Artifact, type DiscussionAndRounds } from "@danypops/papyrus";
-import { askQuestion } from "./discuss-ask-view.ts";
 import { showArtifactBrowser } from "./artifact-browser.ts";
 import { DISCUSSION_STATE_PRESENTATION, DOC_STATUS_PRESENTATION } from "./artifact-status-presentation.ts";
+import { askQuestion } from "./discuss-ask-view.ts";
 import { discussionRoundCountOf, discussionStateOf, showDiscussionDetailView } from "./discussion-detail-view.ts";
 import { callService } from "./service-client.ts";
 
@@ -33,14 +34,22 @@ export async function openTaskChoices(cwd: string): Promise<Artifact[]> {
 export async function blockedTaskChoices(discussionId: string): Promise<Artifact[]> {
 	const tree = await callService<Record<string, unknown>, Artifact>("graph.tree", { id: discussionId, depth: 1 });
 	const blockedIds = (tree.edges ?? []).filter((edge) => edge.relation === "blocks" && edge.from === discussionId).map((edge) => edge.to);
-	const tasks = await Promise.all(blockedIds.map((id) => callService<Record<string, unknown>, Artifact | null>("tasks.show", { id }).catch(() => null)));
+	const tasks = await Promise.all(
+		blockedIds.map((id) => callService<Record<string, unknown>, Artifact | null>("tasks.show", { id }).catch(() => null)),
+	);
 	return tasks.filter((task): task is Artifact => task !== null);
 }
 
 /** Picking a task by title, the same ui.select pattern used for Discuss's own single-choice options -- no ecosystem extension (Pi's own docs/examples, pi-tasks) builds a bespoke fuzzy picker for a plain "choose one named thing" list. */
 export async function pickTaskByName(ctx: ExtensionCommandContext, title: string, tasks: Artifact[]): Promise<Artifact | undefined> {
-	if (tasks.length === 0) { ctx.ui.notify("No open tasks to choose from.", "info"); return undefined; }
-	const label = await ctx.ui.select(title, tasks.map((task) => `${task.title} [${task.status}]`));
+	if (tasks.length === 0) {
+		ctx.ui.notify("No open tasks to choose from.", "info");
+		return undefined;
+	}
+	const label = await ctx.ui.select(
+		title,
+		tasks.map((task) => `${task.title} [${task.status}]`),
+	);
 	if (!label) return undefined;
 	const index = tasks.map((task) => `${task.title} [${task.status}]`).indexOf(label);
 	return index === -1 ? undefined : tasks[index];
@@ -49,9 +58,17 @@ export async function pickTaskByName(ctx: ExtensionCommandContext, title: string
 export function discussionRowMeta(discussion: Artifact, theme: Theme): string {
 	const state = discussionStateOf(discussion);
 	const presentation = DISCUSSION_STATE_PRESENTATION[state];
-	const stateText = presentation ? theme.fg(presentation.color, `${presentation.glyph} ${presentation.label}`) : theme.fg("muted", "state unknown");
+	const stateText = presentation
+		? theme.fg(presentation.color, `${presentation.glyph} ${presentation.label}`)
+		: theme.fg("muted", "state unknown");
 	const rounds = discussionRoundCountOf(discussion);
-	const pending = (() => { try { return readDiscussionExtra(discussion.extra).pendingOptions; } catch { return undefined; } })();
+	const pending = (() => {
+		try {
+			return readDiscussionExtra(discussion.extra).pendingOptions;
+		} catch {
+			return undefined;
+		}
+	})();
 	const pendingText = pending && pending.length > 0 ? theme.fg("accent", ` · awaiting: ${pending.join("/")}`) : "";
 	return `${stateText} · ${rounds} round${rounds === 1 ? "" : "s"}${pendingText}`;
 }
@@ -79,18 +96,39 @@ export async function showDiscussions(ctx: ExtensionCommandContext): Promise<voi
 				return;
 			}
 			if (choice === "Reply") {
-				const pending = (() => { try { return readDiscussionExtra(discussion.extra); } catch { return undefined; } })();
+				const pending = (() => {
+					try {
+						return readDiscussionExtra(discussion.extra);
+					} catch {
+						return undefined;
+					}
+				})();
 				// Same fix as the live discuss tool: the most recent round's own content IS the real
 				// question -- the title becomes a plain orientation subtitle, not a labeled-backwards
 				// "Context:" section under a generic "Reply to <title>:" wrapper.
 				const transcript = await callService<Record<string, unknown>, DiscussionAndRounds>("discuss.show", { id: discussion.id });
 				const question = transcript.rounds.at(-1)?.content?.trim() || `Reply to "${discussion.title}":`;
 				const subtitle = discussion.title;
-				const answer = pending?.pendingOptions && pending.pendingOptions.length > 0 && pending.pendingOptionsMode
-					? await askQuestion(commandCtx, { question, subtitle, options: pending.pendingOptions.map((title, index) => ({ title, description: pending.pendingOptionDescriptions?.[index] || undefined })), allowMultiple: pending.pendingOptionsMode === "multi" })
-					: await askQuestion(commandCtx, { question, subtitle });
+				const answer =
+					pending?.pendingOptions && pending.pendingOptions.length > 0 && pending.pendingOptionsMode
+						? await askQuestion(commandCtx, {
+								question,
+								subtitle,
+								options: pending.pendingOptions.map((title, index) => ({
+									title,
+									description: pending.pendingOptionDescriptions?.[index] || undefined,
+								})),
+								allowMultiple: pending.pendingOptionsMode === "multi",
+							})
+						: await askQuestion(commandCtx, { question, subtitle });
 				if (!answer) return; // canceled
-				await callService("discuss.reply", { id: discussion.id, actor: ACTOR, content: answer.content, ...(answer.selected ? { selected: answer.selected } : {}), source: SOURCE });
+				await callService("discuss.reply", {
+					id: discussion.id,
+					actor: ACTOR,
+					content: answer.content,
+					...(answer.selected ? { selected: answer.selected } : {}),
+					source: SOURCE,
+				});
 				commandCtx.ui.notify(answer.selected ? `Selected: ${answer.selected.join(", ")}` : "Reply added.", "info");
 				return;
 			}
@@ -121,11 +159,22 @@ export async function showDiscussions(ctx: ExtensionCommandContext): Promise<voi
 			}
 			if (choice === "Unblock a task") {
 				const blocked = await blockedTaskChoices(discussion.id);
-				if (blocked.length === 0) { commandCtx.ui.notify("This discussion isn't blocking any task.", "info"); return; }
+				if (blocked.length === 0) {
+					commandCtx.ui.notify("This discussion isn't blocking any task.", "info");
+					return;
+				}
 				const target = await pickTaskByName(commandCtx, "Unblock which task?", blocked);
 				if (!target) return;
-				const result = await callService<Record<string, unknown>, { unblocked: boolean }>("discuss.unblock", { id: discussion.id, task_id: target.id, actor: ACTOR, source: SOURCE });
-				commandCtx.ui.notify(result.unblocked ? `"${discussion.title}" no longer blocks "${target.title}"` : "No such blocking relationship.", "info");
+				const result = await callService<Record<string, unknown>, { unblocked: boolean }>("discuss.unblock", {
+					id: discussion.id,
+					task_id: target.id,
+					actor: ACTOR,
+					source: SOURCE,
+				});
+				commandCtx.ui.notify(
+					result.unblocked ? `"${discussion.title}" no longer blocks "${target.title}"` : "No such blocking relationship.",
+					"info",
+				);
 			}
 		},
 	});

package/extension/src/discussion-detail-view.ts

@@ -8,18 +8,19 @@
  * underlying reason (task-detail-view.ts) -- this mirrors that scrolling-viewport idiom rather
  * than inventing a new one.
  */
-import type { ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
-import { matchesKey, truncateToWidth, type TUI } from "@earendil-works/pi-tui";
+
 import {
 	ARTIFACT_DETAIL_MAX_VISIBLE_LINES,
 	ARTIFACT_DETAIL_MIN_VISIBLE_LINES,
 	ARTIFACT_DETAIL_RESERVED_ROWS,
-	readDiscussionExtra,
 	type Artifact,
 	type DiscussionRound,
+	readDiscussionExtra,
 } from "@danypops/papyrus";
-import { renderMarkdownBody, type ActiveTheme } from "./markdown.ts";
+import type { ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
+import { matchesKey, type TUI, truncateToWidth } from "@earendil-works/pi-tui";
 import { DISCUSSION_STATE_PRESENTATION } from "./artifact-status-presentation.ts";
+import { type ActiveTheme, renderMarkdownBody } from "./markdown.ts";
 
 interface TranscriptLine {
 	text: string;
@@ -61,7 +62,9 @@ class DiscussionTranscriptViewport {
 		);
 	}
 
-	invalidate(): void { this.renderedWidth = 0; }
+	invalidate(): void {
+		this.renderedWidth = 0;
+	}
 
 	render(width: number): string[] {
 		const contentWidth = Math.max(1, width - 2);
@@ -70,10 +73,9 @@ class DiscussionTranscriptViewport {
 		const end = Math.min(this.lines.length, this.offsetY + this.visibleLines);
 		const theme = this.activeTheme();
 		const border = theme.fg("borderMuted", "─".repeat(Math.max(1, width)));
-		const footer = [
-			this.lines.length > this.visibleLines ? `↑/↓ scroll · ${this.offsetY + 1}-${end}/${this.lines.length}` : "",
-			"Esc back",
-		].filter(Boolean).join(" · ");
+		const footer = [this.lines.length > this.visibleLines ? `↑/↓ scroll · ${this.offsetY + 1}-${end}/${this.lines.length}` : "", "Esc back"]
+			.filter(Boolean)
+			.join(" · ");
 		return [
 			border,
 			truncateToWidth(theme.fg("accent", theme.bold("Discussion transcript")), width, ""),
@@ -85,10 +87,14 @@ class DiscussionTranscriptViewport {
 	}
 
 	handleInput(data: string): void {
-		if (matchesKey(data, "escape") || matchesKey(data, "ctrl+c")) { this.close(); return; }
+		if (matchesKey(data, "escape") || matchesKey(data, "ctrl+c")) {
+			this.close();
+			return;
+		}
 		if (matchesKey(data, "up")) this.offsetY = Math.max(0, this.offsetY - 1);
 		else if (matchesKey(data, "down")) this.offsetY = Math.min(Math.max(0, this.lines.length - this.visibleLines), this.offsetY + 1);
-		else if (matchesKey(data, "pageDown")) this.offsetY = Math.min(Math.max(0, this.lines.length - this.visibleLines), this.offsetY + this.visibleLines);
+		else if (matchesKey(data, "pageDown"))
+			this.offsetY = Math.min(Math.max(0, this.lines.length - this.visibleLines), this.offsetY + this.visibleLines);
 		else if (matchesKey(data, "pageUp")) this.offsetY = Math.max(0, this.offsetY - this.visibleLines);
 		else return;
 		this.tui.requestRender();
@@ -98,7 +104,13 @@ class DiscussionTranscriptViewport {
 		if (this.renderedWidth === width) return;
 		this.renderedWidth = width;
 		const theme = this.activeTheme();
-		const extra = (() => { try { return readDiscussionExtra(this.discussion.extra); } catch { return undefined; } })();
+		const extra = (() => {
+			try {
+				return readDiscussionExtra(this.discussion.extra);
+			} catch {
+				return undefined;
+			}
+		})();
 		const presentation = extra ? DISCUSSION_STATE_PRESENTATION[extra.state] : undefined;
 		const stateLine = presentation
 			? theme.fg(presentation.color, `${presentation.glyph} ${presentation.label}`)
@@ -111,14 +123,19 @@ class DiscussionTranscriptViewport {
 			{ text: "" },
 		];
 		const transcript: TranscriptLine[] = this.rounds.flatMap((round, index) => {
-			const roundHeader = theme.fg("accent", `[round ${round.roundNumber}] `) + theme.bold(round.actor) + theme.fg("dim", ` · ${round.occurredAt}`);
+			const roundHeader =
+				theme.fg("accent", `[round ${round.roundNumber}] `) + theme.bold(round.actor) + theme.fg("dim", ` · ${round.occurredAt}`);
 			const body = renderMarkdownBody(round.content, width - 2, this.activeTheme).map((line) => ({ text: `  ${line}` }));
-			const posed = round.options && round.options.length > 0
-				? [{ text: `  ${theme.fg("muted", `Posed (${round.optionsMode === "multi" ? "pick several" : "pick one"}): ${round.options.join(", ")}`)}` }]
-				: [];
-			const picked = round.selected && round.selected.length > 0
-				? [{ text: `  ${theme.fg("success", `Selected: ${round.selected.join(", ")}`)}` }]
-				: [];
+			const posed =
+				round.options && round.options.length > 0
+					? [
+							{
+								text: `  ${theme.fg("muted", `Posed (${round.optionsMode === "multi" ? "pick several" : "pick one"}): ${round.options.join(", ")}`)}`,
+							},
+						]
+					: [];
+			const picked =
+				round.selected && round.selected.length > 0 ? [{ text: `  ${theme.fg("success", `Selected: ${round.selected.join(", ")}`)}` }] : [];
 			return [{ text: roundHeader }, ...body, ...posed, ...picked, ...(index < this.rounds.length - 1 ? [{ text: "" }] : [])];
 		});
 		this.lines = [...header, ...(transcript.length > 0 ? transcript : [{ text: theme.fg("muted", "No rounds recorded.") }])];
@@ -126,12 +143,17 @@ class DiscussionTranscriptViewport {
 	}
 }
 
-export async function showDiscussionDetailView(ctx: ExtensionCommandContext, discussion: Artifact, rounds: DiscussionRound[]): Promise<void> {
+export async function showDiscussionDetailView(
+	ctx: ExtensionCommandContext,
+	discussion: Artifact,
+	rounds: DiscussionRound[],
+): Promise<void> {
 	if (ctx.mode !== "tui") {
 		const lines = rounds.map((round) => `[round ${round.roundNumber}] ${round.actor}: ${round.content}`);
 		ctx.ui.notify([discussion.title, ...lines].join("\n"), "info");
 		return;
 	}
-	await ctx.ui.custom<void>((tui, theme, _keybindings, done) =>
-		new DiscussionTranscriptViewport(tui, () => ctx.ui.theme ?? theme, discussion, rounds, done));
+	await ctx.ui.custom<void>(
+		(tui, theme, _keybindings, done) => new DiscussionTranscriptViewport(tui, () => ctx.ui.theme ?? theme, discussion, rounds, done),
+	);
 }

package/extension/src/docs.ts

@@ -1,5 +1,5 @@
-import type { ExtensionCommandContext, Theme } from "@earendil-works/pi-coding-agent";
 import type { Artifact } from "@danypops/papyrus";
+import type { ExtensionCommandContext, Theme } from "@earendil-works/pi-coding-agent";
 import { showArtifactBrowser, showArtifactDetails } from "./artifact-browser.ts";
 import { DOC_STATUS_PRESENTATION } from "./artifact-status-presentation.ts";
 import { callService } from "./service-client.ts";
@@ -48,7 +48,8 @@ export async function showDocs(ctx: ExtensionCommandContext): Promise<void> {
 				commandCtx.ui.notify(`Linked "${document.title}" via ${relation}`, "info");
 				return;
 			}
-			const operation = choice === "Activate" ? "docs.activate" : choice === "Archive" ? "docs.archive" : choice === "Reopen" ? "docs.reopen" : undefined;
+			const operation =
+				choice === "Activate" ? "docs.activate" : choice === "Archive" ? "docs.archive" : choice === "Reopen" ? "docs.reopen" : undefined;
 			if (operation) {
 				const updated = await callService<Record<string, unknown>, Artifact>(operation, { id: document.id });
 				commandCtx.ui.notify(`${updated.title} → [${updated.status}]`, "info");

package/extension/src/domain-tools.ts

@@ -1,12 +1,6 @@
+import { type Artifact, type DiscussionAndRounds, type DiscussionRound, type OperationName, readDiscussionExtra } from "@danypops/papyrus";
 import type { AgentToolUpdateCallback, ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
 import { Type } from "typebox";
-import {
-	readDiscussionExtra,
-	type Artifact,
-	type DiscussionAndRounds,
-	type DiscussionRound,
-	type OperationName,
-} from "@danypops/papyrus";
 import { askQuestion } from "./discuss-ask-view.ts";
 import { callService } from "./service-client.ts";
 import { renderPapyrusToolCall, renderPapyrusToolResult } from "./tool-rendering/index.ts";
@@ -43,7 +37,11 @@ function normalizeDiscussOptions(params: Record<string, unknown>): void {
 	const titles: string[] = [];
 	const descriptions: string[] = [];
 	for (const entry of raw) {
-		if (typeof entry === "string") { titles.push(entry); descriptions.push(""); continue; }
+		if (typeof entry === "string") {
+			titles.push(entry);
+			descriptions.push("");
+			continue;
+		}
 		if (entry && typeof entry === "object" && typeof (entry as Record<string, unknown>).title === "string") {
 			const record = entry as Record<string, unknown>;
 			titles.push(record.title as string);
@@ -59,9 +57,21 @@ function normalizeDiscussOptions(params: Record<string, unknown>): void {
 	if (anyDescription) params.option_descriptions = descriptions;
 }
 
-async function liveAnswer(ctx: ExtensionContext, discussion: Artifact, latestContent: string | undefined, onUpdate: AgentToolUpdateCallback | undefined, signal: AbortSignal | undefined): Promise<{ content: string; selected?: string[] } | undefined> {
+async function liveAnswer(
+	ctx: ExtensionContext,
+	discussion: Artifact,
+	latestContent: string | undefined,
+	onUpdate: AgentToolUpdateCallback | undefined,
+	signal: AbortSignal | undefined,
+): Promise<{ content: string; selected?: string[] } | undefined> {
 	if (!ctx.hasUI) return undefined;
-	const pending = (() => { try { return readDiscussionExtra(discussion.extra); } catch { return undefined; } })();
+	const pending = (() => {
+		try {
+			return readDiscussionExtra(discussion.extra);
+		} catch {
+			return undefined;
+		}
+	})();
 	// The just-recorded round's own content IS the real question -- a generic "Reply to <title>:"
 	// wrapper as the primary question, with the real content demoted to "Context:", left a human
 	// staring at a labeled-backwards prompt (live-observed). The wrapper is now only a fallback for
@@ -72,7 +82,10 @@ async function liveAnswer(ctx: ExtensionContext, discussion: Artifact, latestCon
 		return askQuestion(ctx, {
 			question,
 			subtitle,
-			options: pending.pendingOptions.map((title, index) => ({ title, description: pending.pendingOptionDescriptions?.[index] || undefined })),
+			options: pending.pendingOptions.map((title, index) => ({
+				title,
+				description: pending.pendingOptionDescriptions?.[index] || undefined,
+			})),
 			allowMultiple: pending.pendingOptionsMode === "multi",
 			onUpdate,
 			signal,
@@ -96,7 +109,9 @@ export function artifactLine(artifact: Artifact): string {
 export function artifactLines(artifacts: Artifact[]): string[] {
 	const titleCounts = new Map<string, number>();
 	for (const artifact of artifacts) titleCounts.set(artifact.title, (titleCounts.get(artifact.title) ?? 0) + 1);
-	return artifacts.map((artifact) => (titleCounts.get(artifact.title)! > 1 ? `${artifactLine(artifact)} (${artifact.id})` : artifactLine(artifact)));
+	return artifacts.map((artifact) =>
+		titleCounts.get(artifact.title)! > 1 ? `${artifactLine(artifact)} (${artifact.id})` : artifactLine(artifact),
+	);
 }
 
 /**
@@ -110,7 +125,9 @@ export function matchArtifactByName(candidates: Artifact[], name: string): strin
 	const matches = candidates.filter((artifact) => artifact.title.trim().toLowerCase() === needle);
 	if (matches.length === 0) throw new Error(`no artifact named "${name}" found in this scope`);
 	if (matches.length > 1) {
-		throw new Error(`${matches.length} artifacts are named "${name}": ${matches.map((artifact) => `${artifact.title} (${artifact.id})`).join(", ")} -- use id to disambiguate`);
+		throw new Error(
+			`${matches.length} artifacts are named "${name}": ${matches.map((artifact) => `${artifact.title} (${artifact.id})`).join(", ")} -- use id to disambiguate`,
+		);
 	}
 	return matches[0]!.id;
 }
@@ -126,9 +143,7 @@ const SCOPE_AWARE_LIST_OPERATIONS = new Set<OperationName>(["tasks.list"]);
 
 /** The widened-scope request tried once when a name isn't found under the caller's current scope. */
 function widenedRequest(listOperation: OperationName, baseRequest: Record<string, unknown>): Record<string, unknown> {
-	return SCOPE_AWARE_LIST_OPERATIONS.has(listOperation)
-		? { ...baseRequest, scope: "all" }
-		: { ...baseRequest, project_root: undefined };
+	return SCOPE_AWARE_LIST_OPERATIONS.has(listOperation) ? { ...baseRequest, scope: "all" } : { ...baseRequest, project_root: undefined };
 }
 
 /**
@@ -147,13 +162,21 @@ function widenedRequest(listOperation: OperationName, baseRequest: Record<string
  * widening, so the caller can surface that a search went wider than the caller's default scope
  * rather than resolving silently.
  */
-async function resolveArtifactIdByName(listOperation: OperationName, baseRequest: Record<string, unknown>, name: string, notes?: string[]): Promise<string> {
+async function resolveArtifactIdByName(
+	listOperation: OperationName,
+	baseRequest: Record<string, unknown>,
+	name: string,
+	notes?: string[],
+): Promise<string> {
 	const candidates = await callService<Record<string, unknown>, Artifact[]>(listOperation, { ...baseRequest, text: name });
 	try {
 		return matchArtifactByName(candidates, name);
 	} catch (error) {
-		if (!(error instanceof Error) || !error.message.startsWith("no artifact named") || baseRequest["scope"] !== undefined) throw error;
-		const widenedCandidates = await callService<Record<string, unknown>, Artifact[]>(listOperation, { ...widenedRequest(listOperation, baseRequest), text: name });
+		if (!(error instanceof Error) || !error.message.startsWith("no artifact named") || baseRequest.scope !== undefined) throw error;
+		const widenedCandidates = await callService<Record<string, unknown>, Artifact[]>(listOperation, {
+			...widenedRequest(listOperation, baseRequest),
+			text: name,
+		});
 		const id = matchArtifactByName(widenedCandidates, name);
 		notes?.push(`"${name}" was not found in the current project scope; resolved across all projects instead.`);
 		return id;
@@ -201,13 +224,13 @@ async function resolveNameArrayField(
  * over the same two operations rather than reinventing trash semantics four times.
  * Returns null when action is neither, so callers fall through to their own dispatch.
  */
-async function handleArtifactRemoveRestore(action: unknown, params: Record<string, unknown>): Promise<ReturnType<typeof text> | null> {
+async function _handleArtifactRemoveRestore(action: unknown, params: Record<string, unknown>): Promise<ReturnType<typeof text> | null> {
 	// Trashed/restored artifacts stay directly showable, so known identities render by title on
 	// either side of the action. An unresolved explicit id stays in structured/error channels;
 	// normal model text does not turn that backend key into the artifact's public name.
 	const titleOf = async (): Promise<string> => {
 		try {
-			const artifact = await callService<Record<string, unknown>, Artifact | null>("artifact.show", { id: params["id"] });
+			const artifact = await callService<Record<string, unknown>, Artifact | null>("artifact.show", { id: params.id });
 			return artifact ? `"${artifact.title}"` : "unknown artifact";
 		} catch {
 			return "unknown artifact";
@@ -215,7 +238,10 @@ async function handleArtifactRemoveRestore(action: unknown, params: Record<strin
 	};
 	if (action === "remove") {
 		const label = await titleOf();
-		const record = await callService<Record<string, unknown>, { artifactId: string; trashedAt: string; purgeAfter: string; reason?: string }>("artifact.remove", params);
+		const record = await callService<
+			Record<string, unknown>,
+			{ artifactId: string; trashedAt: string; purgeAfter: string; reason?: string }
+		>("artifact.remove", params);
 		const message = `Trashed ${label}, eligible for purge at ${record.purgeAfter}.`;
 		return text(message, createPreviewDetails("artifact.remove", "Trashed", record.artifactId));
 	}
@@ -242,7 +268,8 @@ export function registerDiscussTool(pi: ExtensionAPI): void {
 	pi.registerTool({
 		name: "discuss",
 		label: "Discuss",
-		description: "Native Papyrus deliberation with a real lifecycle -- distinct from a one-shot ask: a Discussion persists, takes multiple rounds, and can genuinely block a Task's completion until settled or deferred. ACTIONS: open, reply, defer, resume, settle, block, unblock, show, rounds, list. open starts round 1 and optionally blocks_task_ids immediately. reply is refused once deferred or settled -- resume first. defer is explicitly non-blocking (paused, resumable); settle is terminal and archives the discussion. block/unblock manage the blocking relationship to a task independently of open. A task's completion is refused while any active Discussion blocks it. open/reply can pose a structured choice via options (2-10 entries) + options_mode ('single' mutually exclusive, 'multi' allows several); reply answers a currently pending choice via selected, validated against it. Each option is either a bare string or {title, description}; description is optional for exactly 2 options (a self-evident yes/no) but REQUIRED and non-empty for every option once there are 3 or more -- rejected otherwise. One line: the real pro/con/risk/consequence, never padding that just restates the title. Pass live:true on open or reply to get the human's answer synchronously in this same call, via an interactive prompt (the pending choice's picker if one was posed, otherwise a freeform question) -- covers a completely open question with no artifact (open with no prior discussion) and a question tied to a specific existing artifact (reply, addressed by name) alike. Only takes effect with an interactive UI available; otherwise degrades silently to the normal async round. The live picker docks in the input area itself (falls back to a plain text prompt if unsupported in the current UI mode). PREFER `name` (the discussion's exact title) over `id`, `task_name`/`blocks_task_names` over `task_id`/`blocks_task_ids` -- all are backend implementation details, resolved from name automatically.",
+		description:
+			"Native Papyrus deliberation with a real lifecycle -- distinct from a one-shot ask: a Discussion persists, takes multiple rounds, and can genuinely block a Task's completion until settled or deferred. ACTIONS: open, reply, defer, resume, settle, block, unblock, show, rounds, list. open starts round 1 and optionally blocks_task_ids immediately. reply is refused once deferred or settled -- resume first. defer is explicitly non-blocking (paused, resumable); settle is terminal and archives the discussion. block/unblock manage the blocking relationship to a task independently of open. A task's completion is refused while any active Discussion blocks it. open/reply can pose a structured choice via options (2-10 entries) + options_mode ('single' mutually exclusive, 'multi' allows several); reply answers a currently pending choice via selected, validated against it. Each option is either a bare string or {title, description}; description is optional for exactly 2 options (a self-evident yes/no) but REQUIRED and non-empty for every option once there are 3 or more -- rejected otherwise. One line: the real pro/con/risk/consequence, never padding that just restates the title. Pass live:true on open or reply to get the human's answer synchronously in this same call, via an interactive prompt (the pending choice's picker if one was posed, otherwise a freeform question) -- covers a completely open question with no artifact (open with no prior discussion) and a question tied to a specific existing artifact (reply, addressed by name) alike. Only takes effect with an interactive UI available; otherwise degrades silently to the normal async round. The live picker docks in the input area itself (falls back to a plain text prompt if unsupported in the current UI mode). PREFER `name` (the discussion's exact title) over `id`, `task_name`/`blocks_task_names` over `task_id`/`blocks_task_ids` -- all are backend implementation details, resolved from name automatically.",
 		parameters: Type.Object({
 			action: Type.String(),
 			id: Type.Optional(Type.String()),
@@ -261,7 +288,9 @@ export function registerDiscussTool(pi: ExtensionAPI): void {
 			state: Type.Optional(Type.String()),
 			after_round: Type.Optional(Type.Number()),
 			limit: Type.Optional(Type.Number()),
-			options: Type.Optional(Type.Array(Type.Union([Type.String(), Type.Object({ title: Type.String(), description: Type.Optional(Type.String()) })]))),
+			options: Type.Optional(
+				Type.Array(Type.Union([Type.String(), Type.Object({ title: Type.String(), description: Type.Optional(Type.String()) })])),
+			),
 			options_mode: Type.Optional(Type.String()),
 			selected: Type.Optional(Type.Array(Type.String())),
 			live: Type.Optional(Type.Boolean()),
@@ -270,8 +299,12 @@ export function registerDiscussTool(pi: ExtensionAPI): void {
 		// back, same reasoning as pi-ask-user's own tool: the model must not batch a live ask with
 		// bash/edit/write and let those run before the human sees the prompt.
 		executionMode: "sequential",
-		renderCall(args, theme) { return renderPapyrusToolCall("Discuss", args, theme); },
-		renderResult(result, options, theme, context) { return renderPapyrusToolResult(result, options, theme, context); },
+		renderCall(args, theme) {
+			return renderPapyrusToolCall("Discuss", args, theme);
+		},
+		renderResult(result, options, theme, context) {
+			return renderPapyrusToolResult(result, options, theme, context);
+		},
 		async execute(_id, rawParams, signal, onUpdate, ctx) {
 			try {
 				const params: Record<string, unknown> = { ...rawParams };
@@ -290,14 +323,22 @@ export function registerDiscussTool(pi: ExtensionAPI): void {
 					if (typeof params.actor !== "string" || params.actor.length === 0) params.actor = "agent";
 					const operation = action === "open" ? "discuss.open" : "discuss.reply";
 					const result = await callService<Record<string, unknown>, DiscussionAndRounds>(operation, params);
-					const fallback = action === "open"
-						? text(`Opened discussion ${artifactLine(result.discussion)}`, createArtifactDetails("discuss.open", result.discussion))
-						: text(`Round ${result.rounds[0]?.roundNumber} added to "${result.discussion.title}"`, createArtifactDetails("discuss.reply", result.discussion));
+					const fallback =
+						action === "open"
+							? text(`Opened discussion ${artifactLine(result.discussion)}`, createArtifactDetails("discuss.open", result.discussion))
+							: text(
+									`Round ${result.rounds[0]?.roundNumber} added to "${result.discussion.title}"`,
+									createArtifactDetails("discuss.reply", result.discussion),
+								);
 					if (params.live !== true) return fallback;
 					const answer = await liveAnswer(ctx, result.discussion, result.rounds[0]?.content, onUpdate, signal);
 					if (!answer) return fallback;
 					const answered = await callService<Record<string, unknown>, DiscussionAndRounds>("discuss.reply", {
-						id: result.discussion.id, actor: "human", content: answer.content, ...(answer.selected ? { selected: answer.selected } : {}), source: "discuss-live",
+						id: result.discussion.id,
+						actor: "human",
+						content: answer.content,
+						...(answer.selected ? { selected: answer.selected } : {}),
+						source: "discuss-live",
 					});
 					return text(`"${answered.discussion.title}": ${answer.content}`, createArtifactDetails("discuss.reply", answered.discussion));
 				}
@@ -309,9 +350,10 @@ export function registerDiscussTool(pi: ExtensionAPI): void {
 						callService<Record<string, unknown>, Artifact>("tasks.show", { id: params.task_id }),
 					]);
 					const discussion = discussionAndRounds.discussion;
-					const message = action === "unblock" && !outcome.unblocked
-						? "No such blocking relationship."
-						: `"${discussion.title}" ${action === "block" ? "now blocks" : "no longer blocks"} "${task.title}"`;
+					const message =
+						action === "unblock" && !outcome.unblocked
+							? "No such blocking relationship."
+							: `"${discussion.title}" ${action === "block" ? "now blocks" : "no longer blocks"} "${task.title}"`;
 					return text(message, createPreviewDetails(operation, action === "block" ? "Blocked" : "Unblocked", message));
 				}
 				if (action === "show") {
@@ -326,7 +368,10 @@ export function registerDiscussTool(pi: ExtensionAPI): void {
 				}
 				if (action === "list") {
 					const rows = await callService<Record<string, unknown>, Artifact[]>("discuss.list", params);
-					return text(rows.length ? artifactLines(rows).join("\n") : "No discussions found.", createArtifactListDetails("discuss.list", rows));
+					return text(
+						rows.length ? artifactLines(rows).join("\n") : "No discussions found.",
+						createArtifactListDetails("discuss.list", rows),
+					);
 				}
 				const operations = { defer: "discuss.defer", resume: "discuss.resume", settle: "discuss.settle" } as const;
 				const operation = operations[action as keyof typeof operations];