@danypops/pi-papyrus 0.46.0 → 0.46.3

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

@@ -8,12 +8,15 @@ import {
 } from "@danypops/papyrus";
 import type { ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
 import { matchesKey, sliceByColumn, type TUI, truncateToWidth, visibleWidth, wrapTextWithAnsi } from "@earendil-works/pi-tui";
-import { buildDetailLines, type DetailField, type DetailSection } from "malevich-tui-components";
+import { buildDetailLines, type DetailField, type DetailSection, type TextMeasure } from "malevich-tui-components";
 import { BeautifulMermaidRenderer } from "../beautiful-mermaid-renderer.ts";
 import { type ActiveTheme, renderMarkdownBody } from "../markdown.ts";
 import { type ArtifactDetailContent, artifactDetailContent, artifactDetailsText } from "./artifact-detail-format.ts";
 import { buildArtifactRelationshipLines } from "./artifact-relationship-lines.ts";
 
+/** Real ANSI-aware measure for buildDetailLines -- without it, wrapped themed text loses color on every line but the first/last. */
+const measure: TextMeasure = { visibleWidth, truncateToWidth, wrapTextWithAnsi };
+
 interface ArtifactDetailLine {
 	text: string;
 	wide: boolean;
@@ -117,6 +120,7 @@ class ArtifactDetailViewport {
 							body: (s) => theme.fg("text", s),
 							line: (s) => theme.fg("dim", s),
 						},
+						measure,
 					}).map((text) => ({ text, wide: false }))
 				: [];
 		// buildDetailLines' fields/sections don't insert a leading blank before the

package/extension/src/rules/rules.ts

@@ -36,8 +36,12 @@ export async function showRules(ctx: ExtensionCommandContext): Promise<void> {
 				const updated = await callService<Record<string, unknown>, Artifact>("rules.update", { id: rule.id, title, body });
 				commandCtx.ui.notify(`Updated "${updated.title}"`, "info");
 			} else if (choice === "Preview injection") {
-				const preview = await callService<Record<string, unknown>, string>("rules.preview", { id: rule.id });
-				commandCtx.ui.notify(preview, "info");
+				const result = await callService<Record<string, unknown>, { preview: string; combinedLength: number; warning?: string }>(
+					"rules.preview",
+					{ id: rule.id },
+				);
+				const text = result.warning === undefined ? result.preview : `${result.preview}\n\n⚠ ${result.warning}`;
+				commandCtx.ui.notify(text, result.warning === undefined ? "info" : "warning");
 			} else if (choice === "Link gated task") {
 				const taskId = await commandCtx.ui.input("Task artifact id:", "");
 				if (taskId) await callService("rules.gate", { id: rule.id, task_id: taskId });

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

@@ -11,12 +11,15 @@ import {
 } from "@danypops/papyrus";
 import type { ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
 import { matchesKey, sliceByColumn, type TUI, truncateToWidth, visibleWidth, wrapTextWithAnsi } from "@earendil-works/pi-tui";
-import { buildDetailLines, type DetailField, type DetailSection } from "malevich-tui-components";
+import { buildDetailLines, type DetailField, type DetailSection, type TextMeasure } from "malevich-tui-components";
 import { BeautifulMermaidRenderer } from "../beautiful-mermaid-renderer.ts";
 import { type ActiveTheme, renderMarkdownBody } from "../markdown.ts";
 import { type TaskDetailContent, taskDetailContent, taskDetailsText } from "./task-detail-format.ts";
 import { TASK_STATUS_PRESENTATION } from "./task-presentation.ts";
 
+/** Real ANSI-aware measure for buildDetailLines -- without it, wrapped themed text loses color on every line but the first/last. */
+const measure: TextMeasure = { visibleWidth, truncateToWidth, wrapTextWithAnsi };
+
 interface DetailLine {
 	text: string;
 	graph: boolean;
@@ -122,12 +125,15 @@ class TaskDetailViewport {
 		const fields: DetailField[] = this.content.labels.length > 0 ? [{ label: "Labels", value: this.content.labels.join(", ") }] : [];
 		const labels =
 			fields.length > 0
-				? [...buildDetailLines(width, { fields, theme: detailTheme }).map((text) => ({ text, graph: false })), { text: "", graph: false }]
+				? [
+						...buildDetailLines(width, { fields, theme: detailTheme, measure }).map((text) => ({ text, graph: false })),
+						{ text: "", graph: false },
+					]
 				: [{ text: "", graph: false }];
 		const body = renderMarkdownBody(this.content.body, width, this.activeTheme).map((text) => ({ text, graph: false }));
 		const sections: DetailSection[] = this.content.sections.map((section) => ({ heading: section[0], lines: section.slice(1) }));
 		const sectionLines =
-			sections.length > 0 ? buildDetailLines(width, { sections, theme: detailTheme }).map((text) => ({ text, graph: false })) : [];
+			sections.length > 0 ? buildDetailLines(width, { sections, theme: detailTheme, measure }).map((text) => ({ text, graph: false })) : [];
 		const relationshipHeader =
 			this.graphLines.length > 0
 				? [{ text: "", graph: false }, ...wrap("Relationships:", "muted"), ...wrap("  Dependencies point prerequisite → dependent.", "dim")]

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

@@ -13,7 +13,7 @@ import type { ArtifactToolDetails } from "./render-model.ts";
  * terminal's default color. wrapTextWithAnsi re-injects the active codes on every wrapped
  * line instead.
  */
-const measure: TextMeasure = { visibleWidth, truncateToWidth, wrapTextWithAnsi };
+export const measure: TextMeasure = { visibleWidth, truncateToWidth, wrapTextWithAnsi };
 
 /** Shared by every buildDetailLines caller in this extension (ArtifactCard, and
  * tools/vehicle-artifact-renderers.ts's discuss/tasks.complete renderers) -- one Theme -> DetailViewTheme

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

@@ -19,8 +19,16 @@ import { renderVehicleResult } from "@danypops/vehicle-client-pi/vehicle-render"
 import type { VehicleOperationDescriptor } from "@danypops/vehicle-core";
 import type { Theme } from "@earendil-works/pi-coding-agent";
 import { type Component, Text, truncateToWidth } from "@earendil-works/pi-tui";
-import { buildDetailLines, type DagEdge, type DagNode, DagView, type DetailField, type DetailSection } from "malevich-tui-components";
-import { ArtifactCard, detailViewTheme, expandHint, statusColor, statusGlyph } from "../tool-rendering/artifact-card.ts";
+import {
+	buildDetailLines,
+	type DagEdge,
+	type DagNode,
+	DagView,
+	type DetailField,
+	type DetailSection,
+	statelessComponent,
+} from "malevich-tui-components";
+import { ArtifactCard, detailViewTheme, expandHint, measure, statusColor, statusGlyph } from "../tool-rendering/artifact-card.ts";
 import { ArtifactListCard } from "../tool-rendering/artifact-list.ts";
 import { type ArtifactFocusAnnotation, createArtifactDetails, createArtifactListDetails } from "../tool-rendering/render-model.ts";
 
@@ -268,12 +276,6 @@ function roundsSection(rounds: readonly DiscussionRoundOutput[]): DetailSection
 	};
 }
 
-/** A one-shot render function with nothing to invalidate -- every buildDetailLines-based
- * renderer in this file (unlike ArtifactCard/DagView) has no cache to clear. */
-function statelessComponent(render: (width: number) => string[]): Component {
-	return { render, invalidate: () => {} };
-}
-
 function renderDiscussionAndRounds(output: DiscussionAndRoundsOutput, theme: Theme, expanded: boolean): Component {
 	const discussion = output.discussion;
 	return statelessComponent((width) => {
@@ -283,7 +285,7 @@ function renderDiscussionAndRounds(output: DiscussionAndRoundsOutput, theme: The
 			{ label: "Status", value: theme.fg(statusColor(discussion.status), `${statusGlyph(discussion.status)} ${discussion.status}`) },
 		];
 		const sections: DetailSection[] = expanded && output.rounds.length > 0 ? [roundsSection(output.rounds)] : [];
-		const lines = buildDetailLines(safeWidth, { fields, sections, alignFields: true, theme: detailViewTheme(theme) });
+		const lines = buildDetailLines(safeWidth, { fields, sections, alignFields: true, theme: detailViewTheme(theme), measure });
 		if (!expanded && output.rounds.length > 0) {
 			const count = output.rounds.length;
 			lines.push(truncateToWidth(theme.fg("dim", `${count} round${count === 1 ? "" : "s"} · ${expandHint()}`), safeWidth));
@@ -295,7 +297,7 @@ function renderDiscussionAndRounds(output: DiscussionAndRoundsOutput, theme: The
 function renderDiscussionRoundsOnly(output: DiscussionRoundsOnlyOutput, theme: Theme): Component {
 	return statelessComponent((width) => {
 		const sections: DetailSection[] = output.rounds.length > 0 ? [roundsSection(output.rounds)] : [{ lines: ["No rounds."] }];
-		return buildDetailLines(Math.max(1, width), { sections, theme: detailViewTheme(theme) });
+		return buildDetailLines(Math.max(1, width), { sections, theme: detailViewTheme(theme), measure });
 	});
 }
 
@@ -373,7 +375,7 @@ function renderTaskCompletion(result: TaskCompletionOutput, theme: Theme, expand
 				lines: result.blocked.map((entry) => theme.fg("warning", `◼ ${entry.artifact.title}`)),
 			});
 		}
-		const lines = buildDetailLines(safeWidth, { fields, sections, alignFields: true, theme: detailViewTheme(theme) });
+		const lines = buildDetailLines(safeWidth, { fields, sections, alignFields: true, theme: detailViewTheme(theme), measure });
 		if (result.focused && expanded) {
 			lines.push(truncateToWidth(theme.fg("accent", `▶ focus ${result.focused.title}`), safeWidth));
 		}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@danypops/pi-papyrus",
-  "version": "0.46.0",
+  "version": "0.46.3",
   "description": "Pi host extension for Papyrus: native tools, TUI panels, and context injection over the daemon-backed graph store",
   "type": "module",
   "keywords": ["pi-package"],
@@ -18,13 +18,13 @@
   },
   "dependencies": {
     "@danypops/jittor": "^0.14.0",
-    "@danypops/papyrus": "^0.44.0",
+    "@danypops/papyrus": "^0.45.2",
     "@danypops/vehicle-client": "^0.5.0",
-    "@danypops/vehicle-client-pi": "^0.17.1",
+    "@danypops/vehicle-client-pi": "^0.18.4",
     "@danypops/vehicle-core": "^0.12.3",
     "@danypops/vehicle-server": "^0.18.1",
     "beautiful-mermaid": "1.1.3",
-    "malevich-tui-components": "^0.21.2"
+    "malevich-tui-components": "^0.24.0"
   },
   "devDependencies": {
     "@danypops/pi-tui-harness": "^0.0.2",