@danypops/papyrus 0.34.2 → 0.35.0

package/extension/src/discuss.ts

@@ -1,134 +0,0 @@
-/**
- * discuss.ts — /discuss interactive panel.
- * Reuses the generic artifact browser (artifact-browser.ts), same as docs.ts/rules.ts/notes.ts:
- * a Discussion is a `doc` artifact, so the browser's list/filter/refresh/select-action loop
- * applies unchanged. The one real wrinkle is that Discuss's meaningful lifecycle state
- * (active/deferred/settled) lives in extra.discussion, not the shared doc status column the
- * browser colors its row glyph by (see artifact-status-presentation.ts's DISCUSSION_STATE_PRESENTATION
- * comment) -- so the real state is surfaced in rowMeta text instead, the same way rules.ts
- * surfaces severity and notes.ts surfaces history count, both also not the row glyph.
- *
- * Creating a new Discussion is left to the agent (the discuss tool), matching docs.ts/rules.ts/
- * skills.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 { ExtensionCommandContext, Theme } from "@earendil-works/pi-coding-agent";
-import type { Artifact } from "../../src/domain/artifact.ts";
-import type { DiscussionAndRounds } from "../../src/discussion-service.ts";
-import { readDiscussionExtra } from "../../src/domain/discussion.ts";
-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 { discussionRoundCountOf, discussionStateOf, showDiscussionDetailView } from "./discussion-detail-view.ts";
-import { callService } from "./service-client.ts";
-
-const SOURCE = "discuss-tui";
-const ACTOR = "human";
-
-/** Tasks a human could plausibly want to block on -- excludes terminal ones, since blocking already-finished or canceled work is meaningless. */
-export async function openTaskChoices(cwd: string): Promise<Artifact[]> {
-	const rows = await callService<Record<string, unknown>, Artifact[]>("tasks.list", { project_root: cwd });
-	return rows.filter((task) => task.status !== "done" && task.status !== "canceled");
-}
-
-/** Resolves the task ids a Discussion currently has a `blocks` edge to, by title -- so "Unblock" only ever offers tasks actually blocked by this one, never the whole task list. */
-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)));
-	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 (!label) return undefined;
-	const index = tasks.map((task) => `${task.title} [${task.status}]`).indexOf(label);
-	return index === -1 ? undefined : tasks[index];
-}
-
-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 rounds = discussionRoundCountOf(discussion);
-	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}`;
-}
-
-function discussionActions(discussion: Artifact): string[] {
-	const state = discussionStateOf(discussion);
-	if (state === "active") return ["Show transcript", "Reply", "Defer", "Settle", "Block a task", "Unblock a task"];
-	if (state === "deferred") return ["Show transcript", "Resume", "Settle"];
-	return ["Show transcript"]; // settled, or an unrecognized/corrupt state -- read-only either way
-}
-
-export async function showDiscussions(ctx: ExtensionCommandContext): Promise<void> {
-	await showArtifactBrowser(ctx, {
-		kind: "doc",
-		title: "Discussions",
-		listOperation: "discuss.list",
-		statusOrder: ["draft", "active", "archived"],
-		presentation: DOC_STATUS_PRESENTATION,
-		rowMeta: discussionRowMeta,
-		actions: discussionActions,
-		handleAction: async (choice, discussion, commandCtx) => {
-			if (choice === "Show transcript") {
-				const result = await callService<Record<string, unknown>, DiscussionAndRounds>("discuss.show", { id: discussion.id });
-				await showDiscussionDetailView(commandCtx, result.discussion, result.rounds);
-				return;
-			}
-			if (choice === "Reply") {
-				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 });
-				if (!answer) return; // canceled
-				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;
-			}
-			if (choice === "Defer") {
-				const reason = await commandCtx.ui.input("Defer reason (optional):", "");
-				await callService("discuss.defer", { id: discussion.id, ...(reason ? { reason } : {}), actor: ACTOR, source: SOURCE });
-				commandCtx.ui.notify("Deferred.", "info");
-				return;
-			}
-			if (choice === "Resume") {
-				await callService("discuss.resume", { id: discussion.id, actor: ACTOR, source: SOURCE });
-				commandCtx.ui.notify("Resumed.", "info");
-				return;
-			}
-			if (choice === "Settle") {
-				const settlement = await commandCtx.ui.input("Settlement:", "");
-				if (!settlement) return;
-				await callService("discuss.settle", { id: discussion.id, settlement, actor: ACTOR, source: SOURCE });
-				commandCtx.ui.notify("Settled.", "info");
-				return;
-			}
-			if (choice === "Block a task") {
-				const target = await pickTaskByName(commandCtx, "Block which task?", await openTaskChoices(commandCtx.cwd));
-				if (!target) return;
-				await callService("discuss.block", { id: discussion.id, task_id: target.id, actor: ACTOR, source: SOURCE });
-				commandCtx.ui.notify(`"${discussion.title}" now blocks "${target.title}"`, "info");
-				return;
-			}
-			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; }
-				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");
-			}
-		},
-	});
-}

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

@@ -1,136 +0,0 @@
-/**
- * discussion-detail-view.ts — the transcript view for a single Discussion.
- *
- * The generic artifact detail view (artifact-detail-view.ts) formats an artifact's own
- * fields (title, body, extra as JSON, edges); it has no way to show a Discussion's rounds,
- * since those live in a dedicated child table fetched separately (discuss.show / discuss.rounds),
- * not in the artifact row itself. Tasks needed the same kind of dedicated view for the same
- * 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,
-} from "../../src/constants.ts";
-import type { Artifact } from "../../src/domain/artifact.ts";
-import { readDiscussionExtra, type DiscussionRound } from "../../src/domain/discussion.ts";
-import { renderMarkdownBody, type ActiveTheme } from "./markdown.ts";
-import { DISCUSSION_STATE_PRESENTATION } from "./artifact-status-presentation.ts";
-
-interface TranscriptLine {
-	text: string;
-}
-
-/** Reads state defensively for display -- a corrupt/foreign extra.discussion shape shows as "unknown" rather than crashing the whole panel over one bad row. */
-export function discussionStateOf(discussion: Artifact): string {
-	try {
-		return readDiscussionExtra(discussion.extra).state;
-	} catch {
-		return "unknown";
-	}
-}
-
-export function discussionRoundCountOf(discussion: Artifact): number {
-	try {
-		return readDiscussionExtra(discussion.extra).roundCount;
-	} catch {
-		return 0;
-	}
-}
-
-class DiscussionTranscriptViewport {
-	private offsetY = 0;
-	private renderedWidth = 0;
-	private lines: TranscriptLine[] = [];
-	private readonly visibleLines: number;
-
-	constructor(
-		private readonly tui: TUI,
-		private readonly activeTheme: ActiveTheme,
-		private readonly discussion: Artifact,
-		private readonly rounds: DiscussionRound[],
-		private readonly close: () => void,
-	) {
-		this.visibleLines = Math.max(
-			ARTIFACT_DETAIL_MIN_VISIBLE_LINES,
-			Math.min(ARTIFACT_DETAIL_MAX_VISIBLE_LINES, tui.terminal.rows - ARTIFACT_DETAIL_RESERVED_ROWS),
-		);
-	}
-
-	invalidate(): void { this.renderedWidth = 0; }
-
-	render(width: number): string[] {
-		const contentWidth = Math.max(1, width - 2);
-		this.buildLines(contentWidth);
-		this.offsetY = Math.min(this.offsetY, Math.max(0, this.lines.length - this.visibleLines));
-		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(" · ");
-		return [
-			border,
-			truncateToWidth(theme.fg("accent", theme.bold("Discussion transcript")), width, ""),
-			border,
-			...this.lines.slice(this.offsetY, end).map((line) => truncateToWidth(` ${line.text}`, width, "")),
-			truncateToWidth(theme.fg("dim", footer), width, ""),
-			border,
-		];
-	}
-
-	handleInput(data: string): void {
-		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, "pageUp")) this.offsetY = Math.max(0, this.offsetY - this.visibleLines);
-		else return;
-		this.tui.requestRender();
-	}
-
-	private buildLines(width: number): void {
-		if (this.renderedWidth === width) return;
-		this.renderedWidth = width;
-		const theme = this.activeTheme();
-		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}`)
-			: theme.fg("muted", "state unknown");
-		const header: TranscriptLine[] = [
-			{ text: theme.bold(this.discussion.title) },
-			{ text: `${stateLine}${theme.fg("dim", ` · ${this.discussion.id}`)}` },
-			...(extra?.deferredReason ? [{ text: theme.fg("muted", `Deferred: ${extra.deferredReason}`) }] : []),
-			...(extra?.settlement ? [{ text: theme.fg("success", `Settled: ${extra.settlement}`) }] : []),
-			{ 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 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(", ")}`)}` }]
-				: [];
-			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.") }])];
-		this.offsetY = Math.min(this.offsetY, Math.max(0, this.lines.length - this.visibleLines));
-	}
-}
-
-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));
-}

package/extension/src/docs.ts

@@ -1,58 +0,0 @@
-import type { ExtensionCommandContext, Theme } from "@earendil-works/pi-coding-agent";
-import type { Artifact } from "../../src/domain/artifact.ts";
-import { showArtifactBrowser, showArtifactDetails } from "./artifact-browser.ts";
-import { DOC_STATUS_PRESENTATION } from "./artifact-status-presentation.ts";
-import { callService } from "./service-client.ts";
-
-const DOC_ACTIONS: Record<string, string[]> = {
-	draft: ["Activate", "Archive"],
-	active: ["Archive"],
-	archived: ["Reopen"],
-};
-const DOC_RELATIONS = ["references", "documents", "supersedes", "relates_to", "contains", "part_of"];
-
-export function documentRowMeta(document: Artifact, theme: Theme): string {
-	const subtype = document.subtype ? theme.fg("accent", document.subtype) : "";
-	return [subtype, document.labels.join(", ")].filter(Boolean).join(" · ");
-}
-
-export async function showDocs(ctx: ExtensionCommandContext): Promise<void> {
-	await showArtifactBrowser(ctx, {
-		kind: "doc",
-		title: "Documents",
-		listOperation: "docs.list",
-		statusOrder: ["draft", "active", "archived"],
-		presentation: DOC_STATUS_PRESENTATION,
-		rowMeta: documentRowMeta,
-		actions: (document) => ["Show details", "Edit", "Link artifact", ...(DOC_ACTIONS[document.status] ?? [])],
-		handleAction: async (choice, document, commandCtx) => {
-			if (choice === "Show details") {
-				await showArtifactDetails(commandCtx, document.id, "docs.show");
-				return;
-			}
-			if (choice === "Edit") {
-				const title = await commandCtx.ui.input("Title:", document.title);
-				if (title === undefined) return; // canceled
-				const body = await commandCtx.ui.input("Body:", document.body);
-				if (body === undefined) return; // canceled
-				const updated = await callService<Record<string, unknown>, Artifact>("docs.update", { id: document.id, title, body });
-				commandCtx.ui.notify(`Updated "${updated.title}"`, "info");
-				return;
-			}
-			if (choice === "Link artifact") {
-				const targetId = await commandCtx.ui.input("Target artifact id:", "");
-				if (!targetId) return;
-				const relation = await commandCtx.ui.select("Relation", DOC_RELATIONS);
-				if (!relation) return;
-				await callService("docs.link", { id: document.id, relation, target_id: targetId });
-				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;
-			if (operation) {
-				const updated = await callService<Record<string, unknown>, Artifact>(operation, { id: document.id });
-				commandCtx.ui.notify(`${updated.title} → [${updated.status}]`, "info");
-			}
-		},
-	});
-}