@quandev104/pi-style 0.1.4 → 0.1.6

package/extension-src/pi-style/features/tools/boxed/grep.ts

@@ -13,7 +13,7 @@
 
 import type { Component } from "@earendil-works/pi-tui";
 import { stripAnsi } from "../../../shared/ansi.js";
-import { type BoxTheme, getTextOutput, shortenPath } from "../../../shared/box.js";
+import { type BoxTheme, dimLine, getTextOutput, shortenPath } from "../../../shared/box.js";
 import { safeTruncateToWidth } from "../../../shared/render-budget.js";
 import {
 	type GrepMatch,
@@ -111,7 +111,7 @@ function renderErrorLines(theme: BoxTheme, errorText: string, width: number): st
 		.map((line) => line.trim())
 		.filter((line) => line.length > 0);
 	if (raw.length === 0) return [];
-	const prefix = `${TREE_INDENT}${theme.fg("borderMuted", "└─")} `;
+	const prefix = `${TREE_INDENT}${dimLine("└─")} `;
 	const out = raw
 		.slice(0, GREP_ERROR_LINES)
 		.map((line) => safeTruncateToWidth(`${prefix}${theme.fg("error", line)}`, Math.max(1, width), "…"));

package/extension-src/pi-style/features/tools/boxed/output-tree.ts

@@ -20,6 +20,7 @@
 //   as one visual family.
 
 import type { BoxTheme } from "../../../shared/box.js";
+import { dimLine } from "../../../shared/box.js";
 import { safeTruncateToWidth } from "../../../shared/render-budget.js";
 
 /** Indent for top-level tree rows; matches the quiet-tool batch panel. */
@@ -251,11 +252,11 @@ export function renderOutputTree(
 	const lastIndex = visible.length - 1;
 	for (let i = 0; i < visible.length; i++) {
 		const branch = i < lastIndex || more > 0 ? "├─" : "└─";
-		const line = `${indent}${theme.fg("borderMuted", branch)} ${theme.fg(entryColor, label(visible[i] ?? ""))}`;
+		const line = `${indent}${dimLine(branch)} ${theme.fg(entryColor, label(visible[i] ?? ""))}`;
 		out.push(safeTruncateToWidth(line, safeWidth, "…"));
 	}
 	if (more > 0) {
-		const line = `${indent}${theme.fg("borderMuted", "└─")} ${theme.fg("dim", `… ${more} more ${pluralForm(moreUnit, more)}`)}`;
+		const line = `${indent}${dimLine("└─")} ${theme.fg("dim", `… ${more} more ${pluralForm(moreUnit, more)}`)}`;
 		out.push(safeTruncateToWidth(line, safeWidth, "…"));
 	}
 	return out;
@@ -274,7 +275,7 @@ function formatMatchRow(theme: BoxTheme, match: GrepMatch): string {
 	// Match rows render in the output text color (not primary) so they read like
 	// the matched code; only the file nodes carry the primary color.
 	const label = theme.fg("toolOutput", `*${match.line}`);
-	const sep = theme.fg("borderMuted", "│");
+	const sep = dimLine("│");
 	return `${label}${sep} ${theme.fg("toolOutput", match.content)}`;
 }
 
@@ -313,7 +314,7 @@ export function renderGrepTree(
 	if (singleFile) {
 		budget.forEach((match, index) => {
 			const isLast = index === totalVisible - 1 && !truncated;
-			push(`${indent}${theme.fg("borderMuted", isLast ? "└─" : "├─")} ${formatMatchRow(theme, match)}`);
+			push(`${indent}${dimLine(isLast ? "└─" : "├─")} ${formatMatchRow(theme, match)}`);
 		});
 	} else {
 		// Walk the budget, tracking position within each file group so the file
@@ -323,7 +324,7 @@ export function renderGrepTree(
 			const group = groups[gi];
 			if (!group) continue;
 			const isLastGroup = gi === groups.length - 1;
-			const trunk = isLastGroup ? " " : theme.fg("borderMuted", "│");
+			const trunk = isLastGroup ? " " : dimLine("│");
 
 			const visibleHere: GrepMatch[] = [];
 			for (const match of group.matches) {
@@ -336,22 +337,20 @@ export function renderGrepTree(
 			const groupIsLastRendered = shown >= totalVisible && !truncated;
 			const fileLabel = options.withIcons ? `${fileIcon(group.file)} ${group.file}` : group.file;
 			// File nodes use the primary (accent) color, matching read/ls/find paths.
-			push(`${indent}${theme.fg("borderMuted", groupIsLastRendered ? "└─" : "├─")} ${theme.fg("accent", fileLabel)}`);
+			push(`${indent}${dimLine(groupIsLastRendered ? "└─" : "├─")} ${theme.fg("accent", fileLabel)}`);
 
 			visibleHere.forEach((match, index) => {
 				const isLastInGroup = index === visibleHere.length - 1;
 				const isLastOverall = groupIsLastRendered && isLastInGroup;
 				push(
-					`${indent}${trunk}${TREE_CHILD_INDENT}${theme.fg("borderMuted", isLastOverall ? "└─" : "├─")} ${formatMatchRow(theme, match)}`,
+					`${indent}${trunk}${TREE_CHILD_INDENT}${dimLine(isLastOverall ? "└─" : "├─")} ${formatMatchRow(theme, match)}`,
 				);
 			});
 		}
 	}
 
 	if (truncated) {
-		push(
-			`${indent}${theme.fg("borderMuted", "└─")} ${theme.fg("dim", `… ${remaining} more ${pluralForm("match", remaining)}`)}`,
-		);
+		push(`${indent}${dimLine("└─")} ${theme.fg("dim", `… ${remaining} more ${pluralForm("match", remaining)}`)}`);
 	}
 	return out;
 }

package/extension-src/pi-style/features/tools/boxed/read.ts

@@ -1,9 +1,9 @@
 // Boxed read tool renderer
 // (renderCall/renderResult only; no tool re-registration).
 //
-// Read calls render as a boxless tree panel — a lone read is a batch of one,
-// consecutive reads group into one panel (see batch.ts). There is no boxed
-// single-call special case.
+// Read calls render boxless: a lone read is a single inline line
+// (`➔ Read <path>`), consecutive reads group into one tree panel (see
+// batch.ts).
 
 import { stripAnsi } from "../../../shared/ansi.js";
 import { getTextOutput } from "../../../shared/box.js";

package/extension-src/pi-style/features/tools/boxed/write.ts

@@ -15,6 +15,7 @@ import { stripAnsi } from "../../../shared/ansi.js";
 import {
 	type BoxTheme,
 	boxedToolWidthKey,
+	dimLine,
 	getTextOutput,
 	renderBoxedToolResult,
 	renderCompactBoxedToolCall,
@@ -62,7 +63,7 @@ function numberedPreviewLines(content: string): NumberedLine[] {
 
 /** One boxed preview row: dim gutter + toolOutput content. */
 function formatNumberedLine(theme: BoxTheme, line: NumberedLine): string {
-	return `${theme.fg("borderMuted", `${line.number} `)}${theme.fg("toolOutput", line.content)}`;
+	return `${dimLine(`${line.number} `)}${theme.fg("toolOutput", line.content)}`;
 }
 
 /** Compact write box: path header, numbered content preview, metrics footer. */

package/extension-src/pi-style/pi/compatibility-coordinator.ts

@@ -6,6 +6,7 @@ import {
 	detectPiVersion,
 	disposePiCompatibilityProbe,
 	probePiCompatibility,
+	SUPPORTED_VERSION_RANGE,
 } from "./compatibility-probe.js";
 
 export interface CompatibilityCoordinator {
@@ -60,8 +61,10 @@ export function createCompatibilityCoordinator(dispose = disposePiCompatibilityP
 					report?.unsupported.filter((item) =>
 						feature === "messages" ? item.subtype.includes("message") : item.subtype.includes("tool"),
 					) ?? [];
-				const failed = records.some((item) => /identity|failed|unsupported shape/i.test(item.reason));
-				const fallback = records.some((item) => /fallback|authorization|disabled/i.test(item.reason));
+				// Identity drift degrades only the affected surface to native (graceful);
+				// a feature is only "failed" when an install/shape error occurred.
+				const failed = records.some((item) => /failed|rejected|rolled back|shape is not/i.test(item.reason));
+				const fallback = records.some((item) => /fallback|authorization|disabled|identity/i.test(item.reason));
 				const authorized = Boolean(authorization?.core && surfaceAuthorized);
 				const installedRecord = report?.recordSnapshots.some(
 					(item) => item.feature === feature && !item.disposed && (subtype === undefined || item.subtype === subtype),
@@ -69,7 +72,7 @@ export function createCompatibilityCoordinator(dispose = disposePiCompatibilityP
 				return {
 					configured,
 					authorized,
-					installed: Boolean(installedRecord && !failed && authorized && configured),
+					installed: Boolean(installedRecord && authorized && configured),
 					conflicted: records.some((item) => item.reason.includes("owner")),
 					failed,
 					cleanupPending,
@@ -81,12 +84,13 @@ export function createCompatibilityCoordinator(dispose = disposePiCompatibilityP
 				configured: messagesConfigured || toolsConfigured,
 				authorized: authorization?.core ?? false,
 				installed: report !== undefined,
-				conflicted: report?.unsupported.some((item) => item.reason.includes("identity")) ?? false,
-				failed: report?.unsupported.some((item) => item.reason.includes("failed")) ?? false,
+				conflicted: report?.unsupported.some((item) => item.reason.includes("owner")) ?? false,
+				failed: report?.unsupported.some((item) => /failed|rejected|rolled back/i.test(item.reason)) ?? false,
 				cleanupPending,
-				nativeFallbacks: report?.unsupported.filter((item) => item.reason.includes("fallback")).length ?? 0,
+				nativeFallbacks: report?.unsupported.filter((item) => /fallback|identity/i.test(item.reason)).length ?? 0,
 				piVersion: version.version ?? report?.piVersion ?? "unknown",
-				versionRange: report?.versionRange ?? ">=0.83.0 <0.84.0",
+				versionRange: report?.versionRange ?? SUPPORTED_VERSION_RANGE,
+				supportedVersions: report?.supportedVersions ?? [],
 				assistantMessage: surface(
 					"messages",
 					config.enabled && config.messages.enabled && config.messages.assistantPrefix,
@@ -105,11 +109,13 @@ export function createCompatibilityCoordinator(dispose = disposePiCompatibilityP
 			const productDenied = productGate === "deny";
 			if (cleanupPending && !report) cleanupPending = false;
 			if (cleanupPending || !tui || !config.enabled || !authorization?.core || productDenied) return undefined;
-			const certifiedHost = detectPiVersion().version === "0.83.0";
+			// Certification is per-surface identity (fingerprint) based, never pinned to
+			// a Pi version: authorization only needs the session flags + config. Version
+			// drift that preserves identities keeps working; changed identities degrade
+			// per-surface inside the probe.
 			const assistantEnabled =
 				authorization.assistant &&
 				isTierCAuthorized({
-					certifiedHost,
 					coreFlag: authorization.core,
 					surfaceFlag: true,
 					surface: "assistantMessage",
@@ -118,16 +124,29 @@ export function createCompatibilityCoordinator(dispose = disposePiCompatibilityP
 			const specialBlocksEnabled =
 				authorization.specialBlocks &&
 				isTierCAuthorized({
-					certifiedHost,
 					coreFlag: authorization.core,
 					surfaceFlag: true,
 					surface: "specialBlocks",
 					config,
 				});
 			const messagesEnabled = (assistantEnabled || specialBlocksEnabled) && config.messages.enabled;
+			// The hidden-thinking collapse is an assistant-message surface patch: it needs
+			// the assistant flag and `messages.hideThinkingLabel`, independent of the
+			// assistant prefix feature.
+			const thinkingCollapseEnabled = Boolean(
+				authorization.assistant &&
+					config.messages.enabled &&
+					config.messages.hideThinkingLabel &&
+					isTierCAuthorized({
+						coreFlag: authorization.core,
+						surfaceFlag: true,
+						surface: "messages",
+						config,
+					}),
+			);
 			const toolsEnabled =
 				authorization.tools &&
-				isTierCAuthorized({ certifiedHost, coreFlag: authorization.core, surfaceFlag: true, surface: "tools", config });
+				isTierCAuthorized({ coreFlag: authorization.core, surfaceFlag: true, surface: "tools", config });
 			if (!messagesEnabled && !toolsEnabled) return undefined;
 			const detected = detectPiVersion();
 			report = probePiCompatibility(detected.version, {
@@ -137,6 +156,7 @@ export function createCompatibilityCoordinator(dispose = disposePiCompatibilityP
 						...config.messages,
 						enabled: messagesEnabled,
 						assistantPrefix: assistantEnabled,
+						hideThinkingLabel: thinkingCollapseEnabled,
 						specialBlocks: messagesEnabled && config.messages.specialBlocks && specialBlocksEnabled,
 					},
 					tools: {
@@ -147,6 +167,7 @@ export function createCompatibilityCoordinator(dispose = disposePiCompatibilityP
 				messageSnapshot: {
 					assistantPrefix: authorization.ascii ? "[assistant] " : "│ ",
 					assistantEnabled,
+					collapseHiddenThinking: thinkingCollapseEnabled,
 				},
 				toolSnapshot: {
 					callMarker: authorization.ascii ? "[tool] " : "[tool] ",