@danypops/pi-papyrus 0.52.2 → 0.52.4

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

@@ -1,4 +1,5 @@
-import { keyHint, type Theme } from "@earendil-works/pi-coding-agent";
+import { expandHint } from "@danypops/vehicle-client-pi/expand-hint";
+import type { Theme } from "@earendil-works/pi-coding-agent";
 import { type Component, truncateToWidth, visibleWidth, wrapTextWithAnsi } from "@earendil-works/pi-tui";
 import { buildDetailLines, type DetailField, type DetailSection, type DetailViewTheme, type TextMeasure } from "malevich-tui-components";
 import type { ArtifactToolDetails } from "./render-model.ts";
@@ -93,13 +94,6 @@ export function emptyState(noun: string): string {
 	return `No ${noun}.`;
 }
 
-/** Real, possibly user-remapped hotkey (defaults to ctrl+o, Pi's own "app.tools.expand"
- * binding) -- matches every one of Pi's own built-in tools (read/write/bash/find/grep/ls),
- * which all show their own truncation hint the same way instead of a hardcoded string. */
-export function expandHint(): string {
-	return keyHint("app.tools.expand", "expand for details");
-}
-
 /** Reusable width-safe artifact card for native tool result rows. */
 export class ArtifactCard implements Component {
 	private details: ArtifactToolDetails;

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

@@ -1,8 +1,9 @@
 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, truncateToWidth } from "@earendil-works/pi-tui";
 import { type TreeNode, TreeView } from "malevich-tui-components";
-import { countSummary, expandHint, kindGlyph, statusGlyph } from "./artifact-card.ts";
+import { countSummary, kindGlyph, statusGlyph } from "./artifact-card.ts";
 import type { ArtifactListToolDetails, GraphToolDetails, ToolArtifactSummary } from "./render-model.ts";
 
 function pluralKind(rows: readonly ToolArtifactSummary[]): string {

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

@@ -15,6 +15,7 @@
 import type { Artifact } from "@danypops/papyrus";
 import { TOOL_COLLAPSED_ROW_LIMIT, TOOL_DETAILS_MAX_SERIALIZED_CHARACTERS } from "@danypops/papyrus";
 import type { PiVehicleInvocationRequest, PiVehiclePresentationContract, VehicleToolRenderers } from "@danypops/vehicle-client-pi";
+import { expandHint } from "@danypops/vehicle-client-pi/expand-hint";
 import { renderVehicleCall, renderVehicleResult } from "@danypops/vehicle-client-pi/vehicle-render";
 import type { JsonValue, VehicleOperationDescriptor } from "@danypops/vehicle-core";
 import type { Theme } from "@earendil-works/pi-coding-agent";
@@ -28,7 +29,7 @@ import {
 	type DetailSection,
 	statelessComponent,
 } from "malevich-tui-components";
-import { ArtifactCard, detailViewTheme, expandHint, measure, statusColor, statusGlyph } from "../tool-rendering/artifact-card.ts";
+import { ArtifactCard, detailViewTheme, measure, statusColor, statusGlyph } from "../tool-rendering/artifact-card.ts";
 import { ArtifactListCard } from "../tool-rendering/artifact-list.ts";
 import {
 	type ArtifactFocusAnnotation,
@@ -46,7 +47,9 @@ import {
 } from "../tool-rendering/render-model.ts";
 import { recordRenderDiagnostic, shapeFingerprint } from "./render-diagnostics.ts";
 
-function isArtifact(value: unknown): value is Artifact {
+/** Every field an Artifact and its lean list-default ArtifactSummary (tasks.list/docs.list/
+ * rules.list/playbooks.list without full:true -- see summarizeArtifact()) both always carry. */
+function hasArtifactCoreFields(value: unknown): value is Omit<Artifact, "body" | "extra"> {
 	if (typeof value !== "object" || value === null) return false;
 	const row = value as Record<string, unknown>;
 	return (
@@ -55,15 +58,27 @@ function isArtifact(value: unknown): value is Artifact {
 		typeof row.title === "string" &&
 		typeof row.status === "string" &&
 		typeof row.subtype === "string" &&
-		typeof row.body === "string" &&
 		Array.isArray(row.labels) &&
 		typeof row.created_at === "string" &&
 		typeof row.updated_at === "string"
 	);
 }
 
+function isArtifact(value: unknown): value is Artifact {
+	return hasArtifactCoreFields(value) && typeof (value as Record<string, unknown>).body === "string";
+}
+
+/**
+ * Regression (real, live-observed): tasks.list's own documented default ("Returns a lean
+ * summary (no body/extra) unless full: true is passed") returns ArtifactSummary rows, which
+ * omit body entirely. createArtifactListDetails/artifactSummary (render-model.ts) never read
+ * .body for list rendering -- only single-artifact createArtifactDetails does -- so requiring
+ * body here (matching isArtifact) silently fell every default (lean, the common case) list
+ * call for tasks.list/docs.list/rules.list/playbooks.list through to the generic raw Vehicle
+ * table renderer instead of the curated ArtifactListCard.
+ */
 function isArtifactArray(value: unknown): value is Artifact[] {
-	return Array.isArray(value) && value.every(isArtifact);
+	return Array.isArray(value) && value.every(hasArtifactCoreFields);
 }
 
 /** tasks.focused/tasks.pause/tasks.unpause's own wrapper shape -- an Artifact

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@danypops/pi-papyrus",
-  "version": "0.52.2",
+  "version": "0.52.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.52.0",
     "@danypops/vehicle-client": "^0.7.0",
-    "@danypops/vehicle-client-pi": "^0.27.1",
+    "@danypops/vehicle-client-pi": "^0.30.0",
     "@danypops/vehicle-core": "^0.12.3",
     "@danypops/vehicle-server": "^0.18.1",
     "beautiful-mermaid": "1.1.3",