@danypops/pi-papyrus 0.59.2 → 0.59.4

package/extension/src/index.ts

@@ -924,16 +924,17 @@ export default function (pi: ExtensionAPI) {
 	// agent_settled is intentionally later than agent_end: Pi guarantees that
 	// retry, compaction retry, and queued follow-up processing have finished.
 
-	pi.on("input", async (event, ctx) => {
+	pi.on("input", (event, ctx) => {
 		if (event.source === "extension") return;
 		taskContinuation.onHumanInput();
-		try {
-			const sessionId = ctx.sessionManager.getSessionId();
-			const focus = await callService<Record<string, unknown>, { artifact: Artifact; status: string; pauseReason?: string } | null>(
-				"tasks.focused",
-				{ session_id: sessionId },
-			);
-			if (focus && shouldResumeFocusOnHumanInput(focus.status, focus.pauseReason)) {
+		const sessionId = ctx.sessionManager.getSessionId();
+		// Input handlers are on Pi's prompt-submission path. Resume bookkeeping is
+		// opportunistic and must never hold the user's message behind daemon I/O.
+		void callService<Record<string, unknown>, { artifact: Artifact; status: string; pauseReason?: string } | null>("tasks.focused", {
+			session_id: sessionId,
+		})
+			.then(async (focus) => {
+				if (!focus || !shouldResumeFocusOnHumanInput(focus.status, focus.pauseReason)) return;
 				await callService("tasks.unpause", {
 					actor: "system",
 					source: "task-continuation",
@@ -947,10 +948,10 @@ export default function (pi: ExtensionAPI) {
 					status: "unpaused",
 					effort: extractDeclaredEffort(focus.artifact.extra),
 				});
-			}
-		} catch {
-			// The daemon may be unavailable during startup, reload, or shutdown.
-		}
+			})
+			.catch(() => {
+				// The daemon may be unavailable during startup, reload, or shutdown.
+			});
 	});
 	pi.on("agent_start", () => {
 		taskContinuation.onAgentStart();

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

@@ -53,6 +53,8 @@ function simpleDetailsText(details: Exclude<PapyrusToolDetails, { kind: "artifac
 			].join("\n");
 		case "preview":
 			return `${details.title}\n${details.content}${details.completeness.truncated ? `\n[truncated ${details.completeness.omitted} characters]` : ""}`;
+		case "semantic-text":
+			return details.text;
 		case "error":
 			return `${details.code}: ${details.message}`;
 		case "execution-plan":

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

@@ -18,6 +18,7 @@ import { type GraphToolDetails, isGraphEdge } from "./graph.ts";
 import type { LeaseToolDetails } from "./lease.ts";
 import type { ErrorToolDetails, NoFocusToolDetails, PreviewToolDetails } from "./misc.ts";
 import type { InvocationToolDetails, PlaybookInvocationToolDetails, PlaybookMissingArgumentsToolDetails } from "./playbook.ts";
+import type { SemanticTextToolDetails } from "./semantic-text.ts";
 import {
 	isArtifactSummary,
 	isBoundedArray,
@@ -51,7 +52,8 @@ export type PapyrusToolDetails =
 	| DiscussionToolDetails
 	| TaskCompletionToolDetails
 	| NoFocusToolDetails
-	| LeaseToolDetails;
+	| LeaseToolDetails
+	| SemanticTextToolDetails;
 
 /** Validate renderer details restored from session history before using them as typed presentation state. */
 export function parsePapyrusToolDetails(value: unknown): PapyrusToolDetails | undefined {
@@ -176,6 +178,10 @@ export function parsePapyrusToolDetails(value: unknown): PapyrusToolDetails | un
 				(value.note === undefined || isBoundedString(value.note))
 				? (value as unknown as LeaseToolDetails)
 				: undefined;
+		case "semantic-text":
+			return isBoundedString(value.text, TOOL_DETAILS_BODY_MAX_CHARACTERS) && isCompleteness(value.completeness)
+				? (value as unknown as SemanticTextToolDetails)
+				: undefined;
 		default:
 			return undefined;
 	}
@@ -212,6 +218,7 @@ export {
 	type PlaybookMissingArgumentsToolDetails,
 	type ToolInvocationCreated,
 } from "./playbook.ts";
+export { createSemanticTextDetails, type SemanticTextToolDetails } from "./semantic-text.ts";
 export {
 	type ArtifactFocusAnnotation,
 	createModelContent,

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

@@ -0,0 +1,20 @@
+import { TOOL_DETAILS_BODY_MAX_CHARACTERS } from "@danypops/papyrus";
+import { boundedText, PAPYRUS_TOOL_DETAILS_SCHEMA, type ResultCompleteness, type ToolDetailsBase } from "./shared.ts";
+
+/** Carries an operation's bounded semantic text channel for human presentation. */
+export interface SemanticTextToolDetails extends ToolDetailsBase {
+	kind: "semantic-text";
+	text: string;
+	completeness: ResultCompleteness;
+}
+
+export function createSemanticTextDetails(operation: string, text: string): SemanticTextToolDetails {
+	const bounded = boundedText(text, TOOL_DETAILS_BODY_MAX_CHARACTERS);
+	return {
+		schemaVersion: PAPYRUS_TOOL_DETAILS_SCHEMA,
+		kind: "semantic-text",
+		operation,
+		text: bounded.value,
+		completeness: bounded.completeness,
+	};
+}

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

@@ -33,8 +33,9 @@ import {
 	createNoFocusDetails,
 	createPlaybookInvocationDetails,
 	createPlaybookMissingArgumentsDetails,
-	createPreviewDetails,
+	createSemanticTextDetails,
 	createTaskCompletionDetails,
+	type PapyrusToolDetails,
 	parsePapyrusToolDetails,
 } from "../../tool-rendering/render-model.ts";
 import { recordRenderDiagnostic, shapeFingerprint } from "../render-diagnostics.ts";
@@ -52,7 +53,16 @@ import {
 	renderPlaybookInvocationResult,
 	renderPlaybookMissingArguments,
 } from "./playbook.ts";
-import { boundedJsonPreview, focusAnnotation, isArtifact, isArtifactArray, isTaskFocus, renderNoFocusedTask } from "./shared.ts";
+import {
+	boundedJsonPreview,
+	focusAnnotation,
+	isArtifact,
+	isArtifactArray,
+	isSemanticTextOutput,
+	isTaskFocus,
+	renderNoFocusedTask,
+	semanticText,
+} from "./shared.ts";
 import { isTaskCompletion, renderTaskCompletion } from "./task-completion.ts";
 import { isTaskExecutionPlan, renderTaskExecutionPlan } from "./task-execution.ts";
 
@@ -133,17 +143,16 @@ export function papyrusVehicleRenderers(descriptor: VehicleOperationDescriptor):
  * before Pi ever persists it -- the seam papyrusVehicleRenderers' own renderResult never had
  * (it only converts shape at render time, from whatever the legacy {vehicle, output} path
  * already persisted verbatim, lease tokens and all). Every branch here mirrors the same
- * shape-detection papyrusVehicleRenderers's own renderResult uses, so the two stay in lockstep;
- * anything genuinely unmatched still becomes a real, bounded PreviewToolDetails rather than an
- * unprojected raw passthrough -- the one requirement this whole seam exists to satisfy.
+ * shape-detection papyrusVehicleRenderers's own renderResult uses, plus the server's explicit
+ * semantic text channel. An output matching no legal discriminated-union variant fails closed
+ * instead of silently persisting and rendering raw JSON.
  */
-function projectPapyrusPresentation(descriptor: VehicleOperationDescriptor, output: unknown): JsonValue {
-	if (isArtifactArray(output)) return createArtifactListDetails(descriptor.name, output) as unknown as JsonValue;
-	if (isArtifact(output)) return createArtifactDetails(descriptor.name, output) as unknown as JsonValue;
-	if (isTaskFocus(output)) return createArtifactDetails(descriptor.name, output.artifact, focusAnnotation(output)) as unknown as JsonValue;
-	if (output === null && descriptor.name === "tasks.focused") return createNoFocusDetails(descriptor.name) as unknown as JsonValue;
-	if (isTaskExecutionPlan(output))
-		return createExecutionPlanDetails(descriptor.name, output.nodes, output.layers, output.cycleIds) as unknown as JsonValue;
+function projectPapyrusPresentation(descriptor: VehicleOperationDescriptor, output: unknown): PapyrusToolDetails {
+	if (isArtifactArray(output)) return createArtifactListDetails(descriptor.name, output);
+	if (isArtifact(output)) return createArtifactDetails(descriptor.name, output);
+	if (isTaskFocus(output)) return createArtifactDetails(descriptor.name, output.artifact, focusAnnotation(output));
+	if (output === null && descriptor.name === "tasks.focused") return createNoFocusDetails(descriptor.name);
+	if (isTaskExecutionPlan(output)) return createExecutionPlanDetails(descriptor.name, output.nodes, output.layers, output.cycleIds);
 	if (isPlaybookInvocationResult(output)) {
 		return createPlaybookInvocationDetails(descriptor.name, {
 			playbookId: output.playbookId,
@@ -152,18 +161,18 @@ function projectPapyrusPresentation(descriptor: VehicleOperationDescriptor, outp
 			rootTaskIds: output.rootTaskIds,
 			entryTaskId: output.entryTaskId,
 			execution: output.execution,
-		}) as unknown as JsonValue;
+		});
 	}
 	if (isPlaybookMissingArguments(output)) {
-		return createPlaybookMissingArgumentsDetails(descriptor.name, output.playbookId, output.missingArguments) as unknown as JsonValue;
+		return createPlaybookMissingArgumentsDetails(descriptor.name, output.playbookId, output.missingArguments);
 	}
-	if (isDiscussionAndRounds(output))
-		return createDiscussionDetails(descriptor.name, output.rounds, output.discussion) as unknown as JsonValue;
-	if (isDiscussionRoundsOnly(output)) return createDiscussionDetails(descriptor.name, output.rounds) as unknown as JsonValue;
-	if (isDiscussionListOutput(output)) return createArtifactListDetails(descriptor.name, output.discussions) as unknown as JsonValue;
-	if (isTaskCompletion(output)) return createTaskCompletionDetails(descriptor.name, output) as unknown as JsonValue;
-	if (isTaskLeaseView(output)) return createLeaseDetails(descriptor.name, output) as unknown as JsonValue;
-	return createPreviewDetails(descriptor.name, descriptor.name, boundedJsonPreview(output)) as unknown as JsonValue;
+	if (isDiscussionAndRounds(output)) return createDiscussionDetails(descriptor.name, output.rounds, output.discussion);
+	if (isDiscussionRoundsOnly(output)) return createDiscussionDetails(descriptor.name, output.rounds);
+	if (isDiscussionListOutput(output)) return createArtifactListDetails(descriptor.name, output.discussions);
+	if (isTaskCompletion(output)) return createTaskCompletionDetails(descriptor.name, output);
+	if (isTaskLeaseView(output)) return createLeaseDetails(descriptor.name, output);
+	if (isSemanticTextOutput(output)) return createSemanticTextDetails(descriptor.name, semanticText(output));
+	throw new Error(`${descriptor.name} produced no legal presentation variant`);
 }
 
 function renderFromPapyrusPresentation(
@@ -194,15 +203,19 @@ function renderFromPapyrusPresentation(
 			return renderLease(presentation, theme);
 		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":
 		case "graph":
 		case "gate-run":
 		case "invocation":
 		case "error":
-			// Reachable only if a future caller starts producing these kinds through this seam
-			// (today's Papyrus Vehicle outputs never do) -- a bounded JSON preview is still a
-			// real, safe rendering rather than a crash.
+			// These variants are produced by native tools and remain valid when replayed through this shared renderer.
 			return new Text(theme.fg("toolOutput", boundedJsonPreview(presentation)), 0, 0);
+		default: {
+			const exhaustive: never = presentation;
+			return exhaustive;
+		}
 	}
 }
 
@@ -218,7 +231,8 @@ export function papyrusVehiclePresentations(descriptor: VehicleOperationDescript
 	return {
 		projector: {
 			maxBytes: TOOL_DETAILS_MAX_SERIALIZED_CHARACTERS,
-			project: (output: unknown, _request: PiVehicleInvocationRequest) => projectPapyrusPresentation(descriptor, output),
+			project: (output: unknown, _request: PiVehicleInvocationRequest) =>
+				projectPapyrusPresentation(descriptor, output) as unknown as JsonValue,
 		},
 		renderResult(result, options, theme, context) {
 			if (!options.isPartial && !context.isError) {

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

@@ -72,6 +72,31 @@ export function renderNoFocusedTask(theme: Theme): Component {
 	return new Text(theme.fg("dim", "No focused task."), 0, 0);
 }
 
+export interface SemanticTextOutput {
+	content: Array<{ type: "text"; text: string }>;
+}
+
+/** Recognizes the server's explicit model-facing semantic channel without projecting sibling payload fields. */
+export function isSemanticTextOutput(value: unknown): value is SemanticTextOutput {
+	if (typeof value !== "object" || value === null) return false;
+	const content = (value as Record<string, unknown>).content;
+	return (
+		Array.isArray(content) &&
+		content.length > 0 &&
+		content.every(
+			(block) =>
+				typeof block === "object" &&
+				block !== null &&
+				(block as Record<string, unknown>).type === "text" &&
+				typeof (block as Record<string, unknown>).text === "string",
+		)
+	);
+}
+
+export function semanticText(output: SemanticTextOutput): string {
+	return output.content.map((block) => block.text).join("\n");
+}
+
 /** What renderDiscussionAndRounds/renderTaskCompletion actually read -- satisfied by both a raw
  * Artifact (the live duck-typed output path) and the leaner projected ToolArtifactSummary (the
  * typed-DTO path), with no cast needed at either call site. */

package/extension/src/tools/vehicle-notes-client.ts

@@ -23,7 +23,6 @@
  * had died.
  */
 
-import { createAgentNotifier } from "@danypops/vehicle-client-pi/agent-poll-ticker";
 import { createReconnectingVehicleClient, daemonInstanceIdentity } from "@danypops/vehicle-client/daemon-client";
 import { RemoteVehicleClient } from "@danypops/vehicle-client/http";
 import {
@@ -32,6 +31,7 @@ import {
 	registerVehicleToolsWhenReady,
 	type VehicleReadyEvent,
 } from "@danypops/vehicle-client-pi";
+import { createAgentNotifier } from "@danypops/vehicle-client-pi/agent-poll-ticker";
 import { VehicleApprovalOutcomePoll } from "@danypops/vehicle-client-pi/vehicle-approval-outcome-poll";
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { discussLiveFollowUp } from "../discuss/discuss-live-follow-up.ts";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@danypops/pi-papyrus",
-  "version": "0.59.2",
+  "version": "0.59.4",
   "description": "Pi host extension for Papyrus: native tools, TUI panels, and context injection over the daemon-backed graph store",
   "license": "MIT",
   "type": "module",