@danypops/pi-papyrus 0.45.3 → 0.45.5

package/extension/src/tool-rendering/artifact-card.ts

@@ -1,5 +1,6 @@
 import type { Theme } from "@earendil-works/pi-coding-agent";
-import { type Component, truncateToWidth, wrapTextWithAnsi } from "@earendil-works/pi-tui";
+import { type Component, truncateToWidth } from "@earendil-works/pi-tui";
+import { buildDetailLines, type DetailField, type DetailSection } from "malevich-tui-components";
 import type { ArtifactToolDetails } from "./render-model.ts";
 
 const KIND_GLYPHS: Readonly<Record<string, string>> = {
@@ -92,33 +93,55 @@ export class ArtifactCard implements Component {
 		this.invalidate();
 	}
 
+	/** Every field as a labeled `Label: value` line (buildDetailLines), matching the same
+	 * convention pi-tickets' issue-detail-view.ts already uses -- not bare stacked values. */
+	private fields(): DetailField[] {
+		const artifact = this.details.artifact;
+		const status = this.theme.fg(statusColor(artifact.status), `${statusGlyph(artifact.status)} ${artifact.status}`);
+		return [
+			{ label: "Title", value: artifact.title },
+			{ label: "Kind", value: `${kindGlyph(artifact.kind)} ${artifact.kind}` },
+			{ label: "Status", value: status },
+			...(this.expanded ? [{ label: "ID", value: artifact.id }] : []),
+			...(this.expanded && artifact.subtype ? [{ label: "Subtype", value: artifact.subtype }] : []),
+			...(this.expanded && artifact.labels.length > 0 ? [{ label: "Labels", value: artifact.labels.join(", ") }] : []),
+		];
+	}
+
+	private sections(): DetailSection[] {
+		if (!this.expanded) return [];
+		const sections: DetailSection[] = [];
+		const artifact = this.details.artifact;
+		if (artifact.body) sections.push({ heading: "Body:", body: artifact.body });
+		if (this.details.completeness.truncated) {
+			sections.push({ lines: [`[truncated ${this.details.completeness.omitted} characters]`] });
+		}
+		return sections;
+	}
+
 	render(width: number): string[] {
 		const safeWidth = Math.max(1, width);
 		if (this.cachedLines && this.cachedWidth === safeWidth) return this.cachedLines;
 
-		const artifact = this.details.artifact;
-		const status = `${statusGlyph(artifact.status)} ${artifact.status}`;
-		const header = [
-			this.theme.fg("toolTitle", this.theme.bold(`${kindGlyph(artifact.kind)} ${artifact.kind.toUpperCase()}`)),
-			...(this.expanded ? [this.theme.fg("accent", artifact.id)] : []),
-			this.theme.fg(statusColor(artifact.status), status),
-		].join("  ");
-		const lines = [truncateToWidth(header, safeWidth)];
-		lines.push(truncateToWidth(this.theme.fg("text", artifact.title), safeWidth));
+		const theme = this.theme;
+		const lines = buildDetailLines(safeWidth, {
+			fields: this.fields(),
+			sections: this.sections(),
+			alignFields: true,
+			theme: {
+				field: (s) => theme.fg("text", s),
+				heading: (s) => theme.fg("toolTitle", theme.bold(s)),
+				byline: (s) => theme.fg("muted", s),
+				body: (s) => theme.fg("text", s),
+				line: (s) => theme.fg("warning", s),
+			},
+		});
 
 		if (this.details.focus) {
-			lines.push(truncateToWidth(focusLine(this.details.focus, this.theme), safeWidth));
+			lines.push(truncateToWidth(focusLine(this.details.focus, theme), safeWidth));
 		}
-
-		if (this.expanded) {
-			const metadata = [artifact.subtype, ...artifact.labels].filter(Boolean).join(" · ");
-			if (metadata) lines.push(truncateToWidth(this.theme.fg("muted", metadata), safeWidth));
-			if (artifact.body) lines.push(...wrapTextWithAnsi(artifact.body, safeWidth));
-			if (this.details.completeness.truncated) {
-				lines.push(truncateToWidth(this.theme.fg("warning", `[truncated ${this.details.completeness.omitted} characters]`), safeWidth));
-			}
-		} else if (artifact.body || artifact.labels.length > 0) {
-			lines.push(truncateToWidth(this.theme.fg("dim", expandHint()), safeWidth));
+		if (!this.expanded && (this.details.artifact.body || this.details.artifact.labels.length > 0)) {
+			lines.push(truncateToWidth(theme.fg("dim", expandHint()), safeWidth));
 		}
 
 		this.cachedWidth = safeWidth;

package/extension/src/tool-rendering/index.ts

@@ -1,3 +1,4 @@
+import { pickIdentityArgument } from "@danypops/vehicle-client-pi/vehicle-render";
 import type { AgentToolResult, Theme, ToolRenderResultOptions } from "@earendil-works/pi-coding-agent";
 import { type Component, Text } from "@earendil-works/pi-tui";
 import { ArtifactCard } from "./artifact-card.ts";
@@ -6,25 +7,18 @@ import { type PapyrusToolDetails, parsePapyrusToolDetails } from "./render-model
 
 const CALL_VALUE_MAX_CHARACTERS = 80;
 
+/** name/title before id: a caller that already knows the name shouldn't have the raw id echoed back at it. */
+const IDENTITY_ARG_KEYS = ["name", "title", "id", "text", "query", "kind", "template_id"];
+
 export interface PapyrusToolRenderContext {
 	lastComponent: Component | undefined;
 	isError: boolean;
 }
 
-function primaryArgument(args: Record<string, unknown>): string | undefined {
-	// name/title before id: a caller that already knows the name shouldn't have the raw id echoed
-	// back at it; id only surfaces here when it's genuinely the only identifying argument given.
-	for (const key of ["name", "title", "id", "text", "query", "kind", "template_id"]) {
-		const value = args[key];
-		if (typeof value === "string" && value.trim()) return value.slice(0, CALL_VALUE_MAX_CHARACTERS);
-	}
-	return undefined;
-}
-
 /** Compact native call header that never echoes bodies or structured payloads. */
 export function renderPapyrusToolCall(label: string, args: Record<string, unknown>, theme: Theme): Component {
 	const action = typeof args.action === "string" ? args.action : "call";
-	const primary = primaryArgument(args);
+	const primary = pickIdentityArgument(args, IDENTITY_ARG_KEYS, CALL_VALUE_MAX_CHARACTERS);
 	const text = [
 		theme.fg("toolTitle", theme.bold(label)),
 		theme.fg("muted", action),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@danypops/pi-papyrus",
-  "version": "0.45.3",
+  "version": "0.45.5",
   "description": "Pi host extension for Papyrus: native tools, TUI panels, and context injection over the daemon-backed graph store",
   "type": "module",
   "keywords": ["pi-package"],
@@ -20,11 +20,11 @@
     "@danypops/jittor": "^0.14.0",
     "@danypops/papyrus": "^0.44.0",
     "@danypops/vehicle-client": "^0.5.0",
-    "@danypops/vehicle-client-pi": "^0.16.1",
+    "@danypops/vehicle-client-pi": "^0.16.7",
     "@danypops/vehicle-core": "^0.10.0",
     "@danypops/vehicle-server": "^0.13.0",
     "beautiful-mermaid": "1.1.3",
-    "malevich-tui-components": "^0.20.3"
+    "malevich-tui-components": "^0.21.2"
   },
   "devDependencies": {
     "@danypops/pi-tui-harness": "^0.0.2",