@danypops/pi-papyrus 0.43.3 → 0.43.4

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

@@ -32,6 +32,17 @@ function statusColor(status: string): SemanticColor {
 	return "muted";
 }
 
+function focusLine(focus: ArtifactToolDetails["focus"], theme: Theme): string {
+	if (!focus) return "";
+	// Focus's own active/paused dimension is separate from the artifact's
+	// lifecycle status shown in the header above -- never merge the two.
+	if (focus.status === "paused") {
+		const reason = focus.pauseReason ? ` — ${focus.pauseReason}` : "";
+		return theme.fg("warning", `‖ focus paused${reason}`);
+	}
+	return theme.fg("accent", `▶ focus ${focus.status}`);
+}
+
 export function kindGlyph(kind: string): string {
 	return KIND_GLYPHS[kind] ?? "•";
 }
@@ -91,6 +102,10 @@ export class ArtifactCard implements Component {
 		const lines = [truncateToWidth(header, safeWidth)];
 		lines.push(truncateToWidth(this.theme.fg("text", artifact.title), safeWidth));
 
+		if (this.details.focus) {
+			lines.push(truncateToWidth(focusLine(this.details.focus, this.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));

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

@@ -37,10 +37,22 @@ interface ToolDetailsBase {
 	kind: string;
 }
 
+/** Distinct from the artifact's own lifecycle status (todo/in-progress/done/...) --
+ * this is Task Focus's own separate active/paused dimension, carried by
+ * tasks.focused/tasks.pause/tasks.unpause's {artifact, status, updatedAt}
+ * wrapper shape. Never conflate the two: an artifact can be "in-progress"
+ * while its focus is "paused". */
+export interface ArtifactFocusAnnotation {
+	status: string;
+	updatedAt: string;
+	pauseReason?: string;
+}
+
 export interface ArtifactToolDetails extends ToolDetailsBase {
 	kind: "artifact";
 	artifact: ToolArtifact;
 	completeness: ResultCompleteness;
+	focus?: ArtifactFocusAnnotation;
 }
 
 export interface ArtifactListToolDetails extends ToolDetailsBase {
@@ -150,7 +162,7 @@ function artifactSummary(artifact: Artifact): ToolArtifactSummary {
 	};
 }
 
-export function createArtifactDetails(operation: string, artifact: Artifact): ArtifactToolDetails {
+export function createArtifactDetails(operation: string, artifact: Artifact, focus?: ArtifactFocusAnnotation): ArtifactToolDetails {
 	const body = boundedText(artifact.body, TOOL_DETAILS_BODY_MAX_CHARACTERS);
 	return {
 		schemaVersion: PAPYRUS_TOOL_DETAILS_SCHEMA,
@@ -163,6 +175,7 @@ export function createArtifactDetails(operation: string, artifact: Artifact): Ar
 			updatedAt: artifact.updated_at,
 		},
 		completeness: body.completeness,
+		...(focus ? { focus } : {}),
 	};
 }
 
@@ -334,6 +347,15 @@ function isToolArtifact(value: unknown): value is ToolArtifact {
 	);
 }
 
+function isFocusAnnotation(value: unknown): value is ArtifactFocusAnnotation {
+	return (
+		isRecord(value) &&
+		isBoundedString(value.status) &&
+		isBoundedString(value.updatedAt) &&
+		(value.pauseReason === undefined || isBoundedString(value.pauseReason))
+	);
+}
+
 function isGraphEdge(value: unknown): value is ToolGraphEdge {
 	return isRecord(value) && isBoundedString(value.from) && isBoundedString(value.relation) && isBoundedString(value.to);
 }
@@ -371,7 +393,11 @@ export function parsePapyrusToolDetails(value: unknown): PapyrusToolDetails | un
 
 	switch (value.kind) {
 		case "artifact":
-			return isToolArtifact(value.artifact) && isCompleteness(value.completeness) ? (value as unknown as ArtifactToolDetails) : undefined;
+			return isToolArtifact(value.artifact) &&
+				isCompleteness(value.completeness) &&
+				(value.focus === undefined || isFocusAnnotation(value.focus))
+				? (value as unknown as ArtifactToolDetails)
+				: undefined;
 		case "artifact-list":
 			return isBoundedArray(value.rows, TOOL_DETAILS_MAX_ITEMS, isArtifactSummary) &&
 				Number.isSafeInteger(value.total) &&

package/extension/src/vehicle-artifact-renderers.ts

@@ -16,9 +16,11 @@ import type { Artifact } from "@danypops/papyrus";
 import type { VehicleToolRenderers } from "@danypops/vehicle-client-pi";
 import { renderVehicleResult } from "@danypops/vehicle-client-pi/vehicle-render";
 import type { VehicleOperationDescriptor } from "@danypops/vehicle-core";
+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 { createArtifactDetails, createArtifactListDetails } from "./tool-rendering/render-model.ts";
+import { type ArtifactFocusAnnotation, createArtifactDetails, createArtifactListDetails } from "./tool-rendering/render-model.ts";
 
 function isArtifact(value: unknown): value is Artifact {
 	if (typeof value !== "object" || value === null) return false;
@@ -40,6 +42,35 @@ function isArtifactArray(value: unknown): value is Artifact[] {
 	return Array.isArray(value) && value.every(isArtifact);
 }
 
+/** tasks.focused/tasks.pause/tasks.unpause's own wrapper shape -- an Artifact
+ * plus Task Focus's separate active/paused dimension. Detected the same
+ * name-independent, shape-based way as isArtifact/isArtifactArray above. */
+interface TaskFocusOutput {
+	artifact: Artifact;
+	status: string;
+	updatedAt: string;
+	pauseReason?: string;
+}
+
+function isTaskFocus(value: unknown): value is TaskFocusOutput {
+	if (typeof value !== "object" || value === null) return false;
+	const row = value as Record<string, unknown>;
+	return (
+		isArtifact(row.artifact) &&
+		typeof row.status === "string" &&
+		typeof row.updatedAt === "string" &&
+		(row.pauseReason === undefined || typeof row.pauseReason === "string")
+	);
+}
+
+function focusAnnotation(output: TaskFocusOutput): ArtifactFocusAnnotation {
+	return { status: output.status, updatedAt: output.updatedAt, ...(output.pauseReason ? { pauseReason: output.pauseReason } : {}) };
+}
+
+function renderNoFocusedTask(theme: Theme): Component {
+	return new Text(theme.fg("dim", "No focused task."), 0, 0);
+}
+
 export function papyrusVehicleRenderers(descriptor: VehicleOperationDescriptor): VehicleToolRenderers {
 	return {
 		renderResult(result, options, theme, context) {
@@ -51,6 +82,19 @@ export function papyrusVehicleRenderers(descriptor: VehicleOperationDescriptor):
 				if (isArtifact(output)) {
 					return new ArtifactCard(createArtifactDetails(descriptor.name, output), theme, options.expanded);
 				}
+				if (isTaskFocus(output)) {
+					return new ArtifactCard(
+						createArtifactDetails(descriptor.name, output.artifact, focusAnnotation(output)),
+						theme,
+						options.expanded,
+					);
+				}
+				// tasks.focused specifically returns null for "nothing focused" --
+				// scoped to this one operation so an unrelated null-output operation
+				// (e.g. a not-found lookup) is never mislabeled as a focus state.
+				if (output === null && descriptor.name === "tasks.focused") {
+					return renderNoFocusedTask(theme);
+				}
 			}
 			return renderVehicleResult(descriptor, result, options, theme, context);
 		},

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@danypops/pi-papyrus",
-  "version": "0.43.3",
+  "version": "0.43.4",
   "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,7 +20,7 @@
     "@danypops/jittor": "^0.14.0",
     "@danypops/papyrus": "^0.42.0",
     "@danypops/vehicle-client": "^0.2.0",
-    "@danypops/vehicle-client-pi": "^0.3.3",
+    "@danypops/vehicle-client-pi": "^0.5.1",
     "@danypops/vehicle-core": "^0.3.0",
     "@danypops/vehicle-server": "^0.4.1",
     "beautiful-mermaid": "1.1.3",