@danypops/papyrus 0.21.3 → 0.21.4

package/extension/src/domain-tools.ts

@@ -304,10 +304,13 @@ export function registerDomainTools(pi: ExtensionAPI): void {
 					return text(output, createPreviewDetails("tasks.complete", "Task completion", output));
 				}
 				if (action === "run_gates") {
-					const gates = await callService<Record<string, unknown>, GateResult[]>("tasks.run_gates", request);
+					const [gates, task] = await Promise.all([
+						callService<Record<string, unknown>, GateResult[]>("tasks.run_gates", request),
+						callService<Record<string, unknown>, Artifact>("tasks.show", { id: params.id }),
+					]);
 					return text(
 						gates.map((gate) => `${gate.passed ? "✓" : "✗"} ${gate.gate.type}: ${gate.gate.target} — ${gate.output}`).join("\n") || "No gates configured.",
-						createGateRunDetails("tasks.run_gates", (params.id as string | undefined) ?? "", gates.map((gate) => ({
+						createGateRunDetails("tasks.run_gates", (params.id as string | undefined) ?? "", task.title, gates.map((gate) => ({
 							passed: gate.passed, type: gate.gate.type, target: gate.gate.target, output: gate.output,
 						}))),
 					);

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

@@ -16,7 +16,9 @@ export interface PapyrusToolRenderContext {
 }
 
 function primaryArgument(args: Record<string, unknown>): string | undefined {
-	for (const key of ["id", "title", "text", "query", "kind", "template_id"]) {
+	// 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);
 	}
@@ -45,11 +47,11 @@ function textContent(result: AgentToolResult<unknown>): string {
 function simpleDetailsText(details: Exclude<PapyrusToolDetails, { kind: "artifact" | "artifact-list" | "graph" }>): string {
 	switch (details.kind) {
 		case "transition":
-			return `✓ ${details.artifact.id}  ${details.fromStatus} → ${details.toStatus}\n${details.artifact.title}`;
+			return `✓ ${details.fromStatus} → ${details.toStatus}\n${details.artifact.title}`;
 		case "gate-run": {
 			const passed = details.gates.filter((gate) => gate.passed).length;
 			return [
-				`${passed}/${details.gates.length} gates passed for ${details.artifactId}`,
+				`${passed}/${details.gates.length} gates passed for "${details.artifactTitle}"`,
 				...details.gates.map((gate) => `${gate.passed ? "✓" : "✗"} ${gate.type}: ${gate.target}${gate.output ? ` — ${gate.output}` : ""}`),
 			].join("\n");
 		}

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

@@ -81,6 +81,7 @@ export interface ToolGateRow {
 export interface GateRunToolDetails extends ToolDetailsBase {
 	kind: "gate-run";
 	artifactId: string;
+	artifactTitle: string;
 	gates: ToolGateRow[];
 	completeness: ResultCompleteness;
 }
@@ -218,6 +219,7 @@ export function createGraphDetails(
 export function createGateRunDetails(
 	operation: string,
 	artifactId: string,
+	artifactTitle: string,
 	gates: readonly ToolGateRow[],
 ): GateRunToolDetails {
 	const boundedGates = gates.slice(0, TOOL_DETAILS_MAX_ITEMS).map((gate) => ({
@@ -229,6 +231,7 @@ export function createGateRunDetails(
 		kind: "gate-run",
 		operation,
 		artifactId,
+		artifactTitle,
 		gates: boundedGates,
 		completeness: completeness(gates.length, boundedGates.length),
 	};
@@ -383,6 +386,7 @@ export function parsePapyrusToolDetails(value: unknown): PapyrusToolDetails | un
 				? value as unknown as GraphToolDetails : undefined;
 		case "gate-run":
 			return isBoundedString(value.artifactId)
+				&& isBoundedString(value.artifactTitle)
 				&& isBoundedArray(value.gates, TOOL_DETAILS_MAX_ITEMS, isGateRow)
 				&& isCompleteness(value.completeness)
 				? value as unknown as GateRunToolDetails : undefined;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@danypops/papyrus",
-  "version": "0.21.3",
+  "version": "0.21.4",
   "description": "Daemon-backed graph artifacts, evidence-bearing tasks, rules, skills, and native TUI workflows for Pi",
   "type": "module",
   "keywords": ["pi-package"],