@danypops/pi-papyrus 0.60.0 → 0.60.2

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

@@ -76,6 +76,7 @@ export class ArtifactListCard implements Component {
 			const omitted = Math.max(0, this.details.total - display.length);
 			if (omitted > 0) lines.push(truncateToWidth(this.theme.fg("dim", `${omitted} more · ${expandHint()}`), safeWidth));
 		}
+		if (this.details.hasMore) lines.push(truncateToWidth(this.theme.fg("dim", "More pages available"), safeWidth));
 		this.cachedWidth = safeWidth;
 		this.cachedLines = lines;
 		return lines;
@@ -188,7 +189,20 @@ export class TaskHierarchyPreview implements Component {
 			safeWidth,
 		);
 		const tree = new TreeView({ nodes: toTreeNodes(this.details, this.theme, this.expanded) });
-		const lines = [header, ...tree.render(safeWidth)];
+		const rendered = tree.render(safeWidth);
+		const displayed = rendered.slice(0, this.expanded ? 200 : TOOL_COLLAPSED_ROW_LIMIT);
+		const lines = [header, ...displayed];
+		if (rendered.length > displayed.length) {
+			lines.push(
+				truncateToWidth(
+					this.theme.fg("dim", `${rendered.length - displayed.length} more lines · ${this.expanded ? "display limit" : expandHint()}`),
+					safeWidth,
+				),
+			);
+		}
+		if (this.details.nodeCompleteness.truncated || this.details.edgeCompleteness.truncated) {
+			lines.push(truncateToWidth(this.theme.fg("dim", "Partial graph · presentation limit"), safeWidth));
+		}
 		this.cachedWidth = safeWidth;
 		this.cachedLines = lines;
 		return lines;

package/extension/src/tool-rendering/render-model/artifact.ts

@@ -22,6 +22,7 @@ export interface ArtifactListToolDetails extends ToolDetailsBase {
 	kind: "artifact-list";
 	rows: ToolArtifactSummary[];
 	total: number;
+	hasMore?: boolean;
 	completeness: ResultCompleteness;
 }
 

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

@@ -83,6 +83,7 @@ export function parsePapyrusToolDetails(value: unknown): PapyrusToolDetails | un
 			return isBoundedArray(value.rows, TOOL_DETAILS_MAX_ITEMS, isArtifactSummary) &&
 				Number.isSafeInteger(value.total) &&
 				Number(value.total) >= value.rows.length &&
+				(value.hasMore === undefined || typeof value.hasMore === "boolean") &&
 				isCompleteness(value.completeness)
 				? (value as unknown as ArtifactListToolDetails)
 				: undefined;

package/extension/src/tool-rendering/render-model/semantic-text.ts

@@ -1,3 +1,4 @@
+import { stripVTControlCharacters } from "node:util";
 import { TOOL_DETAILS_BODY_MAX_CHARACTERS } from "@danypops/papyrus";
 import { boundedText, PAPYRUS_TOOL_DETAILS_SCHEMA, type ResultCompleteness, type ToolDetailsBase } from "./shared.ts";
 
@@ -8,8 +9,15 @@ export interface SemanticTextToolDetails extends ToolDetailsBase {
 	completeness: ResultCompleteness;
 }
 
+/** Preserves diagnostic text while removing terminal commands and normalizing line breaks and tabs. */
+export function plainTerminalText(text: string): string {
+	return stripVTControlCharacters(text)
+		.replace(/\r\n?/g, "\n")
+		.replace(/\p{Cc}/gu, (character) => (character === "\n" ? "\n" : character === "\t" ? "    " : ""));
+}
+
 export function createSemanticTextDetails(operation: string, text: string): SemanticTextToolDetails {
-	const bounded = boundedText(text, TOOL_DETAILS_BODY_MAX_CHARACTERS);
+	const bounded = boundedText(plainTerminalText(text), TOOL_DETAILS_BODY_MAX_CHARACTERS);
 	return {
 		schemaVersion: PAPYRUS_TOOL_DETAILS_SCHEMA,
 		kind: "semantic-text",

package/extension/src/tool-rendering/semantic-text.ts

@@ -0,0 +1,38 @@
+import { TOOL_COLLAPSED_ROW_LIMIT } from "@danypops/papyrus";
+import { expandHint } from "@danypops/vehicle-client-pi/expand-hint";
+import type { Theme } from "@earendil-works/pi-coding-agent";
+import { type Component, Text, truncateToWidth } from "@earendil-works/pi-tui";
+import { createSemanticTextDetails, type SemanticTextToolDetails } from "./render-model/semantic-text.ts";
+
+const MAX_EXPANDED_TEXT_ROWS = 200;
+
+/** Renders diagnostic text with host-owned styles and bounded collapsed or expanded rows. */
+export class SemanticTextCard implements Component {
+	private readonly details: SemanticTextToolDetails;
+
+	constructor(
+		details: SemanticTextToolDetails,
+		private readonly theme: Theme,
+		private readonly expanded: boolean,
+	) {
+		const normalized = createSemanticTextDetails(details.operation, details.text);
+		this.details = { ...normalized, completeness: details.completeness.truncated ? details.completeness : normalized.completeness };
+	}
+
+	render(width: number): string[] {
+		if (width < 1) return [];
+		const lines = new Text(this.details.text, 0, 0).render(width);
+		const limit = this.expanded ? MAX_EXPANDED_TEXT_ROWS : TOOL_COLLAPSED_ROW_LIMIT;
+		const visible = lines.slice(0, limit).map((line) => truncateToWidth(this.theme.fg("toolOutput", line), width));
+		const omitted = lines.length - visible.length;
+		if (omitted > 0) {
+			const hint = this.expanded ? `${omitted} more lines omitted · display limit` : `${omitted} more lines · ${expandHint()}`;
+			visible.push(truncateToWidth(this.theme.fg("dim", hint), width));
+		} else if (this.details.completeness.truncated) {
+			visible.push(truncateToWidth(this.theme.fg("dim", "Output truncated"), width));
+		}
+		return visible;
+	}
+
+	invalidate(): void {}
+}

package/extension/src/tools/renderers/index.ts

@@ -23,7 +23,7 @@ import type { JsonValue, VehicleOperationDescriptor } from "@danypops/vehicle-co
 import type { Theme } from "@earendil-works/pi-coding-agent";
 import { type Component, Text } from "@earendil-works/pi-tui";
 import { ArtifactCard } from "../../tool-rendering/artifact-card.ts";
-import { ArtifactListCard } from "../../tool-rendering/artifact-list.ts";
+import { ArtifactListCard, TaskHierarchyPreview } from "../../tool-rendering/artifact-list.ts";
 import {
 	createArtifactDetails,
 	createArtifactListDetails,
@@ -40,6 +40,7 @@ import {
 	type PapyrusToolDetails,
 	parsePapyrusToolDetails,
 } from "../../tool-rendering/render-model.ts";
+import { SemanticTextCard } from "../../tool-rendering/semantic-text.ts";
 import { recordRenderDiagnostic, shapeFingerprint } from "../render-diagnostics.ts";
 import { batchOutcomeSummary } from "./batch.ts";
 import {
@@ -66,6 +67,7 @@ import {
 	renderNoFocusedTask,
 	semanticText,
 } from "./shared.ts";
+import { taskGraphPresentation, taskPagePresentation } from "./task-collections.ts";
 import { isTaskCompletion, renderTaskCompletion } from "./task-completion.ts";
 import { isTaskExecutionPlan, renderTaskExecutionPlan } from "./task-execution.ts";
 
@@ -134,6 +136,9 @@ export function papyrusVehicleRenderers(descriptor: VehicleOperationDescriptor):
 				if (isTaskCompletion(output)) {
 					return renderTaskCompletion(output, theme, options.expanded);
 				}
+				if (isSemanticTextOutput(output)) {
+					return new SemanticTextCard(createSemanticTextDetails(descriptor.name, semanticText(output)), theme, options.expanded);
+				}
 				recordRenderDiagnostic({ event: "render-result-fell-through-to-generic", operation: descriptor.name });
 			}
 			return renderVehicleResult(descriptor, result, options, theme, context);
@@ -151,6 +156,11 @@ export function papyrusVehicleRenderers(descriptor: VehicleOperationDescriptor):
  * instead of silently persisting and rendering raw JSON.
  */
 function projectPapyrusPresentation(descriptor: VehicleOperationDescriptor, output: unknown): PapyrusToolDetails {
+	if (descriptor.name === "tasks.graph" || descriptor.name === "tasks.list_page") {
+		const presentation = descriptor.name === "tasks.graph" ? taskGraphPresentation(output) : taskPagePresentation(output);
+		if (presentation) return presentation;
+		throw new Error(`${descriptor.name} produced no legal presentation variant`);
+	}
 	if (descriptor.name === "batch.execute") {
 		const summary = batchOutcomeSummary(output);
 		if (summary !== undefined) return createSemanticTextDetails(descriptor.name, summary);
@@ -217,9 +227,10 @@ function renderFromPapyrusPresentation(
 		case "preview":
 			return new Text(theme.fg("toolOutput", presentation.content), 0, 0);
 		case "semantic-text":
-			return new Text(theme.fg("toolOutput", presentation.text), 0, 0);
-		case "transition":
+			return new SemanticTextCard(presentation, theme, expanded);
 		case "graph":
+			return new TaskHierarchyPreview(presentation, theme, expanded);
+		case "transition":
 		case "gate-run":
 		case "invocation":
 		case "error":

package/extension/src/tools/renderers/task-collections.ts

@@ -0,0 +1,64 @@
+import { TASK_EXECUTION_MAX_EDGES, TASK_EXECUTION_MAX_NODES, TASK_LIST_PAGE_MAX_LIMIT, type TaskNode } from "@danypops/papyrus";
+import { type ArtifactListToolDetails, createArtifactListDetails } from "../../tool-rendering/render-model/artifact.ts";
+import { createGraphDetails, type GraphToolDetails, type ToolGraphEdge } from "../../tool-rendering/render-model/graph.ts";
+import { isRecord } from "../../tool-rendering/render-model/shared.ts";
+import { isArtifact, isArtifactArray } from "./shared.ts";
+
+function identifiers(value: unknown): value is string[] {
+	return (
+		Array.isArray(value) &&
+		value.length <= TASK_EXECUTION_MAX_NODES &&
+		value.every((id) => typeof id === "string" && id.length > 0 && id.length <= 500)
+	);
+}
+
+function graphNode(value: unknown): value is TaskNode {
+	return (
+		isRecord(value) &&
+		isArtifact(value.task) &&
+		identifiers(value.parentIds) &&
+		identifiers(value.childIds) &&
+		identifiers(value.dependencyIds)
+	);
+}
+
+/** Projects one cursor page while keeping its continuation token in the operation result only. */
+export function taskPagePresentation(output: unknown): ArtifactListToolDetails | undefined {
+	if (!isRecord(output) || !Array.isArray(output.items) || output.items.length > TASK_LIST_PAGE_MAX_LIMIT || !isArtifactArray(output.items))
+		return undefined;
+	if (
+		output.nextCursor !== undefined &&
+		(typeof output.nextCursor !== "string" || output.nextCursor.length === 0 || output.nextCursor.length > 16384)
+	)
+		return undefined;
+	return { ...createArtifactListDetails("tasks.list_page", output.items), hasMore: output.nextCursor !== undefined };
+}
+
+/** Projects selected tasks and their typed relationships into a bounded hierarchy. */
+export function taskGraphPresentation(output: unknown): GraphToolDetails | undefined {
+	if (
+		!isRecord(output) ||
+		!Array.isArray(output.nodes) ||
+		output.nodes.length > TASK_EXECUTION_MAX_NODES ||
+		!identifiers(output.rootIds) ||
+		!output.nodes.every(graphNode)
+	)
+		return undefined;
+	const edges = new Map<string, ToolGraphEdge>();
+	let linkCount = 0;
+	for (const node of output.nodes) {
+		linkCount += node.parentIds.length + node.childIds.length + node.dependencyIds.length;
+		if (linkCount > TASK_EXECUTION_MAX_EDGES * 3) return undefined;
+		const add = (from: string, relation: string, to: string) => {
+			edges.set(JSON.stringify([from, relation, to]), { from, relation, to });
+		};
+		for (const parent of node.parentIds) add(parent, "contains", node.task.id);
+		for (const child of node.childIds) add(node.task.id, "contains", child);
+		for (const dependency of node.dependencyIds) add(dependency, "unlocks", node.task.id);
+	}
+	return createGraphDetails(
+		"tasks.graph",
+		output.nodes.map((node) => node.task),
+		[...edges.values()],
+	);
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@danypops/pi-papyrus",
-  "version": "0.60.0",
+  "version": "0.60.2",
   "description": "Pi host extension for Papyrus: native tools, TUI panels, and context injection over the daemon-backed graph store",
   "license": "MIT",
   "type": "module",
@@ -13,14 +13,14 @@
     "typecheck": "tsc --noEmit -p tsconfig.json"
   },
   "peerDependencies": {
-    "@danypops/vehicle-client-pi": "^0.46.0",
+    "@danypops/vehicle-client-pi": "^0.48.3",
     "@earendil-works/pi-coding-agent": "*",
     "@earendil-works/pi-tui": "*",
     "typebox": "*"
   },
   "dependencies": {
     "@danypops/jittor": "^0.19.2",
-    "@danypops/papyrus": "^0.60.12",
+    "@danypops/papyrus": "^0.60.13",
     "@danypops/vehicle-client": "^0.10.8",
     "@danypops/vehicle-core": "^0.19.1",
     "@danypops/vehicle-server": "^0.27.1",
@@ -30,7 +30,7 @@
   "devDependencies": {
     "@danypops/pi-extension-harness": "^0.8.3",
     "@danypops/pi-tui-harness": "^0.0.2",
-    "@danypops/vehicle-client-pi": "^0.46.0",
+    "@danypops/vehicle-client-pi": "^0.48.3",
     "@danypops/vehicle-conformance": "^0.3.0",
     "@earendil-works/pi-coding-agent": "^0.80.10",
     "bun-types": "latest",