@danypops/pi-papyrus 0.45.2 → 0.45.4

package/extension/src/discuss-live-follow-up.ts

@@ -16,6 +16,8 @@ import { readDiscussionExtra } from "@danypops/papyrus";
 import type { PiVehicleInteractiveFollowUp } from "@danypops/vehicle-client-pi";
 import { askQuestion } from "./discuss-ask-view.ts";
 
+const LIVE_REPLY_PERMISSIONS = ["discuss:read", "discuss:write"] as const;
+
 export const discussLiveFollowUp: PiVehicleInteractiveFollowUp = async (request, output, client) => {
 	const live = (request.input as { live?: unknown } | undefined)?.live === true;
 	if (!live || !request.context.hasUI) return undefined;
@@ -52,13 +54,22 @@ export const discussLiveFollowUp: PiVehicleInteractiveFollowUp = async (request,
 	});
 	if (!answer) return undefined; // canceled, or no UI capable of answering -- default content/details stand
 
-	const answered = (await client.invoke("discuss.reply", 1, {
-		id: result.discussion.id,
-		actor: "human",
-		content: answer.content,
-		...(answer.selected ? { selected: answer.selected } : {}),
-		source: "discuss-live",
-	})) as DiscussionAndRounds;
+	const answered = (await client.invoke(
+		"discuss.reply",
+		1,
+		{
+			id: result.discussion.id,
+			actor: "human",
+			content: answer.content,
+			...(answer.selected ? { selected: answer.selected } : {}),
+			source: "discuss-live",
+		},
+		{
+			permissions: LIVE_REPLY_PERMISSIONS,
+			principal: { id: "pi-papyrus" },
+			signal: request.signal,
+		},
+	)) as DiscussionAndRounds;
 	return {
 		content: [{ type: "text", text: `"${answered.discussion.title}": ${answer.content}` }],
 		output: answered,

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.2",
+  "version": "0.45.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.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",