@danypops/papyrus 0.34.2 → 0.34.3

package/extension/src/artifact-detail-format.ts

@@ -10,19 +10,26 @@ export interface ArtifactDetailContent {
 	relationships: string[];
 }
 
-export function artifactDetailContent(artifact: Artifact): ArtifactDetailContent {
+export function artifactDetailContent(artifact: Artifact, relationshipLines: string[] = []): ArtifactDetailContent {
 	return {
 		title: artifact.title,
 		identity: `${artifact.id} [${artifact.kind}|${artifact.status}]${artifact.subtype ? ` · ${artifact.subtype}` : ""}`,
 		body: artifact.body || "(no body)",
 		labels: [...artifact.labels],
 		metadata: Object.keys(artifact.extra).length > 0 ? formatMetadata(artifact.extra) : [],
-		relationships: (artifact.edges ?? []).map((edge) => `${edge.from} --${edge.relation}--> ${edge.to}`),
+		relationships: relationshipLines,
 	};
 }
 
-export function artifactDetailsText(artifact: Artifact): string {
-	const content = artifactDetailContent(artifact);
+/**
+ * Deliberately plain, unlike ArtifactDetailViewport's TUI body (renderMarkdownBody): this path
+ * feeds ctx.ui.notify(), used outside interactive mode (RPC, piped/non-terminal callers) where
+ * ANSI escape codes are noise or corruption, not formatting. Raw Markdown source stays readable
+ * as plain text either way. relationshipLines is still upgraded (a real graph or a resolved
+ * arrow list, never raw ids) since that needs no color to read.
+ */
+export function artifactDetailsText(artifact: Artifact, relationshipLines: string[] = []): string {
+	const content = artifactDetailContent(artifact, relationshipLines);
 	let output = `${content.title}\n${content.identity}\n\n${content.body}`;
 	if (content.labels.length > 0) output += `\n\nLabels: ${content.labels.join(", ")}`;
 	if (content.metadata.length > 0) output += `\n\nMetadata:\n${content.metadata.map((line) => `  ${line}`).join("\n")}`;

package/extension/src/artifact-detail-view.ts

@@ -7,7 +7,10 @@ import {
 	ARTIFACT_DETAIL_RESERVED_ROWS,
 } from "../../src/constants.ts";
 import type { Artifact } from "../../src/domain/artifact.ts";
+import type { GraphRenderer } from "../../src/ports/graph-renderer.ts";
 import { artifactDetailContent, artifactDetailsText, type ArtifactDetailContent } from "./artifact-detail-format.ts";
+import { buildArtifactRelationshipLines } from "./artifact-relationship-lines.ts";
+import { BeautifulMermaidRenderer } from "./beautiful-mermaid-renderer.ts";
 import { renderMarkdownBody, type ActiveTheme } from "./markdown.ts";
 
 interface ArtifactDetailLine {
@@ -27,13 +30,14 @@ class ArtifactDetailViewport {
 		private readonly tui: TUI,
 		private readonly activeTheme: ActiveTheme,
 		artifact: Artifact,
+		relationshipLines: string[],
 		private readonly close: () => void,
 	) {
 		this.visibleLines = Math.max(
 			ARTIFACT_DETAIL_MIN_VISIBLE_LINES,
 			Math.min(ARTIFACT_DETAIL_MAX_VISIBLE_LINES, tui.terminal.rows - ARTIFACT_DETAIL_RESERVED_ROWS),
 		);
-		this.content = artifactDetailContent(artifact);
+		this.content = artifactDetailContent(artifact, relationshipLines);
 	}
 
 	invalidate(): void { this.renderedWidth = 0; }
@@ -104,9 +108,14 @@ class ArtifactDetailViewport {
 	}
 }
 
-export async function showArtifactDetailView(ctx: ExtensionCommandContext, artifact: Artifact): Promise<void> {
-	const output = artifactDetailsText(artifact);
+export async function showArtifactDetailView(
+	ctx: ExtensionCommandContext,
+	artifact: Artifact,
+	renderer: GraphRenderer = new BeautifulMermaidRenderer(),
+): Promise<void> {
+	const relationshipLines = buildArtifactRelationshipLines(artifact, renderer);
+	const output = artifactDetailsText(artifact, relationshipLines);
 	if (ctx.mode !== "tui") { ctx.ui.notify(output, "info"); return; }
 	await ctx.ui.custom<void>((tui, theme, _keybindings, done) =>
-		new ArtifactDetailViewport(tui, () => ctx.ui.theme ?? theme, artifact, done));
+		new ArtifactDetailViewport(tui, () => ctx.ui.theme ?? theme, artifact, relationshipLines, done));
 }

package/extension/src/artifact-relationship-lines.ts

@@ -0,0 +1,24 @@
+import { GRAPH_RENDER_MAX_ROUTED_EDGES, GRAPH_RENDER_MAX_ROUTED_NODES } from "../../src/constants.ts";
+import { projectArtifactRelationships } from "../../src/artifact-relationship-view.ts";
+import type { Artifact } from "../../src/domain/artifact.ts";
+import type { GraphRenderer } from "../../src/ports/graph-renderer.ts";
+
+/**
+ * Renders an artifact's direct relationships as a small graph when the neighbor set is real
+ * (more than one node) and within the same bound Task graph rendering already enforces;
+ * otherwise falls back to a plain, still name-resolved arrow list -- never the renderer's own
+ * separate box-style fallback, so this view has exactly two shapes, not three.
+ */
+export function buildArtifactRelationshipLines(artifact: Artifact, renderer: GraphRenderer): string[] {
+	const graph = projectArtifactRelationships(artifact);
+	if (graph.edges.length === 0) return [];
+	const withinBounds = graph.nodes.length > 1
+		&& graph.nodes.length <= GRAPH_RENDER_MAX_ROUTED_NODES
+		&& graph.edges.length <= GRAPH_RENDER_MAX_ROUTED_EDGES;
+	if (withinBounds) {
+		const rendered = renderer.render(graph);
+		if (rendered.lines.length > 0) return rendered.lines;
+	}
+	const labelById = new Map(graph.nodes.map((node) => [node.id, node.label]));
+	return graph.edges.map((edge) => `${labelById.get(edge.from) ?? edge.from} --${edge.label}--> ${labelById.get(edge.to) ?? edge.to}`);
+}

package/package.json

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

package/src/artifact-relationship-view.ts

@@ -0,0 +1,23 @@
+import type { Artifact } from "./domain/artifact.ts";
+import type { DisplayGraph, DisplayGraphEdge, DisplayGraphNode } from "./domain/display-graph.ts";
+import { fallbackLabel } from "./task-relationship-view.ts";
+
+/**
+ * The generic-artifact counterpart to projectTaskRelationships: unlike a Task, which already
+ * has its containing TaskGraph's real titles in memory, a generic Doc/Rule/Skill/Playbook only
+ * has raw edge id pairs (artifact.show's tree fetch never resolves neighbor titles -- see
+ * ops.ts getArtifact). Reuses the same fallbackLabel heuristic rather than adding a network
+ * round-trip per neighbor, which would turn a rendering concern into a new daemon-adjacent one.
+ */
+export function projectArtifactRelationships(artifact: Artifact): DisplayGraph {
+	const edges: DisplayGraphEdge[] = (artifact.edges ?? []).map((edge) => ({ from: edge.from, to: edge.to, label: edge.relation }));
+	const nodeIds = new Set<string>();
+	for (const edge of edges) {
+		nodeIds.add(edge.from);
+		nodeIds.add(edge.to);
+	}
+	const nodes: DisplayGraphNode[] = [...nodeIds].map((id) => id === artifact.id
+		? { id, label: artifact.title, status: artifact.status }
+		: { id, label: fallbackLabel(id) });
+	return { direction: "LR", nodes, edges };
+}

package/src/task-relationship-view.ts

@@ -9,7 +9,8 @@ function normalizeEdge(edge: ArtifactEdge): DisplayGraphEdge {
 	return { from: edge.from, to: edge.to, label: edge.relation };
 }
 
-function fallbackLabel(id: string): string {
+/** Best-effort readable label from a bare id when no real title is known -- shared with artifact-relationship-view.ts's generic version of this same problem. */
+export function fallbackLabel(id: string): string {
 	return id.replace(/-[a-z0-9]{4}$/i, "").replaceAll("-", " ");
 }