@quandev104/pi-style 0.1.4 → 0.1.6

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

@@ -5,7 +5,8 @@ export type CompatibilitySubtype =
 	| "native-skill-message"
 	| "native-custom-message"
 	| "tool-call-renderer"
-	| "tool-result-renderer";
+	| "tool-result-renderer"
+	| "native-bash-execution";
 
 type CompatibilityShape = "supported" | "unsupported" | "conflict" | "installed" | "skipped";
 
@@ -171,6 +172,13 @@ function validateInstall(
 	if (!options.shape) return skippedResult(options, "unsupported", options.diagnostic ?? "unsupported shape", current);
 	const conflict = existing && activeConflict(existing, current, options.generation);
 	if (conflict) return conflict;
+	if (options.kind === "add-method") {
+		// Additive install: the slot must be unowned (the method is inherited); the
+		// delegate receives the inherited native function as its fallback identity.
+		if (current !== undefined || ownDescriptor !== undefined)
+			return skippedResult(options, "conflict", "target already owns the additive method", current);
+		return undefined;
+	}
 	if (typeof current !== "function")
 		return skippedResult(
 			options,
@@ -225,6 +233,8 @@ export function installDelegatingPatch(options: {
 	expectedIdentity?: unknown;
 	hasExpectedIdentity?: boolean;
 	diagnostic?: string | undefined;
+	/** "add-method" installs a new own method on the prototype (nothing may already own the slot); the delegate receives the inherited native function as the fallback. */
+	kind?: "method" | "add-method";
 	delegate: (original: unknown, thisArg: object, args: unknown[]) => unknown;
 }): InstallResult {
 	const { target, method } = options;
@@ -235,7 +245,9 @@ export function installDelegatingPatch(options: {
 	const validation = validateInstall(options, current, records.get(method), inspection.descriptor);
 	if (validation) return validation;
 	const originalDescriptor = Object.getOwnPropertyDescriptor(target, method);
-	const originalIdentity = current;
+	// Additive installs have no current owner; the captured inherited native
+	// function (expectedIdentity) is both the fallback and the delegate's original.
+	const originalIdentity = options.kind === "add-method" ? options.expectedIdentity : current;
 	let active = true;
 	const installed = function (this: object, ...args: unknown[]): unknown {
 		if (!active) return Reflect.apply(originalIdentity as (...values: unknown[]) => unknown, this, args);
@@ -260,7 +272,11 @@ export function installDelegatingPatch(options: {
 	};
 	try {
 		Object.defineProperty(installed, "__piStyleCompatibilityRecord", { value: record, configurable: false });
-		const descriptor = { ...originalDescriptor, value: installed };
+		// Additive installs have no original descriptor; write a fresh writable,
+		// configurable, non-enumerable own method so the slot stays reversible.
+		const descriptor = originalDescriptor
+			? { ...originalDescriptor, value: installed }
+			: { value: installed, writable: true, enumerable: false, configurable: true };
 		const wrote = Reflect.defineProperty(target, method, descriptor);
 		registryTestHooks.afterWrite?.();
 		const currentDescriptor = Object.getOwnPropertyDescriptor(target, method);

package/extension-src/pi-style/pi/index.ts

@@ -40,8 +40,9 @@ export function __setCompatibilityTestHooks(hooks: CompatibilityTestHooks): () =
 
 /** Thin Pi adapter: register flags, commands, and forward lifecycle events. */
 export default function piStyleExtension(pi: ExtensionAPI): void {
-	// The core/message/tool surfaces are default-on (safe: fingerprint-certified against
-	// exact Pi 0.83.0, fail-closed elsewhere, conflict-preserving). The OFF switch is the
+	// The core/message/tool surfaces are default-on (identity-certified per surface by
+	// name/arity/source fingerprint, graceful native fallback for any surface whose
+	// runtime identity is not recorded, conflict-preserving). The OFF switch is the
 	// product gate `compatibility.allowCorePatches: false` (or `enabled: false`) in config.
 	for (const [name, description] of [
 		["pi-style-core-patches", "Enable pi-style message/tool core patches"],
@@ -66,7 +67,24 @@ export default function piStyleExtension(pi: ExtensionAPI): void {
 		await coordinator.start(event, ctx);
 	});
 	pi.on("agent_start", () => coordinator.app.runtime.current?.dismissStartup());
-	pi.on("input", () => coordinator.app.runtime.current?.dismissStartup());
+	pi.on("input", (event, _ctx) => {
+		coordinator.app.runtime.current?.dismissStartup();
+		// Bare `!`/`!!` submit guard: Pi treats `!`-prefixed input as a direct bash
+		// command but falls through to normal message submission when the bang has
+		// no command after it — sending a literal `!` to the agent. Drop those
+		// accidental submits instead; Pi's submit path already cleared the editor
+		// (onChange("") resets isBashMode), so the input returns to the normal
+		// prompt without sending anything. Only the interactive input box is
+		// guarded; rpc/extension sources keep sending text verbatim.
+		if (event.source === "interactive") {
+			const trimmed = event.text.trimStart();
+			if (trimmed.startsWith("!")) {
+				const bangLength = trimmed.startsWith("!!") ? 2 : 1;
+				if (trimmed.slice(bangLength).trim() === "") return { action: "handled" };
+			}
+		}
+		return undefined;
+	});
 	pi.on("tool_execution_start", () => coordinator.app.runtime.current?.dismissStartup());
 	pi.on("model_select", (event) =>
 		coordinator.app.update(

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

@@ -4,6 +4,7 @@ import type { ConfigFilePort } from "../app/config-storage.js";
 import { createPiStyleApp, type PiStyleApp } from "../app/index.js";
 import { resolveTheme } from "../domain/theme.js";
 import { setSpecialBlockTheme } from "../features/messages/special-blocks.js";
+import { setBashExecutionTheme } from "../features/tools/bash-execution.js";
 import { resetBashTreeRegistry } from "../features/tools/boxed/bash.js";
 import { resetBatchRegistry } from "../features/tools/boxed/batch.js";
 import { resetGrepRegistry } from "../features/tools/boxed/grep.js";
@@ -94,6 +95,26 @@ export function createPiStyleSessionCoordinator(pi: ExtensionAPI, hooks: Compati
 	const applyMessagesConfig = (config: import("../domain/config-types.js").NormalizedPiStyleConfig) => {
 		sessionUi?.setHiddenThinkingLabel?.(config.messages.hideThinkingLabel ? "" : undefined);
 	};
+	/**
+	 * Auto-apply the configured pi-style theme (default "titanium") once per TUI
+	 * session before any surface captures the active theme, so a fresh install
+	 * renders with the intended palette. Failure-safe: an unresolvable target is
+	 * never passed to Pi (its setTheme falls back to the dark theme on load
+	 * error, which would clobber the user's theme), and "off" disables the
+	 * surface for users who keep their own theme.
+	 */
+	const applyAutoTheme = (
+		config: import("../domain/config-types.js").NormalizedPiStyleConfig,
+		ctx: ExtensionContext,
+	) => {
+		const target = config.theme.autoApply;
+		if (ctx.mode !== "tui" || !target || target === "off") return;
+		const ui = ctx.ui;
+		if (ui?.theme?.name === target) return;
+		// Resolve before switching (see failure-safe note above).
+		if (!ui?.getTheme?.(target)) return;
+		ui.setTheme?.(target);
+	};
 	const app: PiStyleApp = createPiStyleApp(
 		undefined,
 		{
@@ -162,6 +183,8 @@ export function createPiStyleSessionCoordinator(pi: ExtensionAPI, hooks: Compati
 			productGate = app.productPolicy.corePatchGate;
 			active = true;
 			compatibility.install(app.config, ctx.mode === "tui", productGate);
+			// Auto-apply the configured theme before surfaces capture the active one.
+			applyAutoTheme(app.config, ctx);
 			// Session-scoped render configuration for the boxed tool/message surfaces.
 			// Populated once per session (never inside render).
 			sessionTheme = ctx.ui?.theme as never;
@@ -169,6 +192,7 @@ export function createPiStyleSessionCoordinator(pi: ExtensionAPI, hooks: Compati
 			applyToolsRenderConfig(app.config);
 			applyMessagesConfig(app.config);
 			if (ctx.ui?.theme) setSpecialBlockTheme(ctx.ui.theme as never);
+			if (ctx.ui?.theme) setBashExecutionTheme(ctx.ui.theme as never);
 			const toolDetails = collectToolDetails(pi.getActiveTools?.(), pi.getAllTools?.());
 			app.sessionStart(
 				{

package/extension-src/pi-style/shared/box.ts

@@ -319,8 +319,6 @@ export function formatToolParamLines(args: unknown, theme?: BoxTheme): string[]
 	return lines;
 }
 
-const RESET_INTENSITY = "\x1b[22m";
-
 function colorFromExtra(theme: BoxTheme, extraKey: string, fallbackColor: string, text: string): string {
 	const color = getThemeExtra(theme, extraKey);
 	if (color) {
@@ -380,11 +378,17 @@ export function formatBoxedRunningStatus(theme: BoxTheme, elapsedMs: number | un
 	return `${theme.fg("dim", "◌ Running")}${elapsed}`;
 }
 
+/** Structural line — box frame, tree branch, divider, gutter — wrapped in
+ * dim terminal-default intensity. Visible in every theme; matches omp. */
+export function dimLine(text: string): string {
+	return `\x1b[2m${text}\x1b[22m`;
+}
+
 function boxText(theme: BoxTheme, text: string): string {
-	return `${RESET_INTENSITY}${theme.fg("borderMuted", text)}`;
+	return dimLine(text);
 }
 function boxFrameText(theme: BoxTheme, text: string): string {
-	return `${RESET_INTENSITY}${theme.fg("border", text)}`;
+	return dimLine(text);
 }
 
 export function boxedToolBgName(isError?: boolean, isPartial?: boolean): string {

package/extension-src/pi-style/shared/split-diff.ts

@@ -7,6 +7,7 @@ import { highlightCode } from "@earendil-works/pi-coding-agent";
 import type { Component } from "@earendil-works/pi-tui";
 
 import { stripAnsi } from "./ansi.js";
+import { dimLine } from "./box.js";
 import { safeTruncateToWidth, safeVisibleWidth } from "./render-budget.js";
 
 // ── Types ──────────────────────────────────────────────────────────
@@ -761,7 +762,7 @@ class SplitDiffRenderer {
 			lineKind === "add" ? "toolDiffAdded" : lineKind === "remove" ? "toolDiffRemoved" : "borderMuted";
 		const marker = this.ctx.fg(markerColor, markerChar);
 		const lineNumber = this.ctx.fg("dim", " ".repeat(this.ctx.lineNumberWidth));
-		const divider = this.ctx.fg("borderMuted", " │ ");
+		const divider = dimLine(" │ ");
 		const prefix = `${marker} ${lineNumber}${divider}`;
 		const prefixPlain = `${markerChar} ${" ".repeat(this.ctx.lineNumberWidth)} │ `;
 		const tailWidth = Math.max(0, columnWidth - safeVisibleWidth(prefixPlain));
@@ -791,14 +792,14 @@ class SplitDiffRenderer {
 			this.ctx.fg(markerColor, markerChar) +
 			" " +
 			this.ctx.fg(this.getNumberColor(lineKind), lineNumber) +
-			this.ctx.fg("borderMuted", " │ ");
+			dimLine(" │ ");
 		const firstPrefixPlain = `${markerChar} ${lineNumber} │ `;
 
 		const contPrefixAnsi =
 			this.ctx.fg(markerColor, markerChar) +
 			" " +
 			this.ctx.fg("dim", " ".repeat(this.ctx.lineNumberWidth)) +
-			this.ctx.fg("borderMuted", " │ ");
+			dimLine(" │ ");
 		const contPrefixPlain = `${markerChar} ${" ".repeat(this.ctx.lineNumberWidth)} │ `;
 
 		const codeWidth = Math.max(1, columnWidth - safeVisibleWidth(firstPrefixPlain));
@@ -865,7 +866,7 @@ class SplitDiffRenderer {
 
 	render(width: number): string[] {
 		const safeWidth = Math.max(20, width);
-		const columnSeparator = this.ctx.fg("borderMuted", " │ ");
+		const columnSeparator = dimLine(" │ ");
 		const separatorWidth = safeVisibleWidth(stripAnsi(columnSeparator));
 		const leftWidth = Math.max(20, Math.floor((safeWidth - separatorWidth) / 2));
 		const rightWidth = Math.max(20, safeWidth - separatorWidth - leftWidth);
@@ -877,15 +878,14 @@ class SplitDiffRenderer {
 			if (dividerIndex >= 0 && dividerIndex < chars.length) {
 				chars[dividerIndex] = junction;
 			}
-			return this.ctx.fg("borderMuted", chars.join(""));
+			return dimLine(chars.join(""));
 		};
 
 		const formatHeaderCell = (label: string, columnWidth: number): string => {
 			// Keep marker+space columns, then place label inside the line-number column.
 			const markerPad = "  ";
 			const lineNumberLabel = fitToWidth(label, this.ctx.lineNumberWidth);
-			const prefixAnsi =
-				this.ctx.fg("borderMuted", markerPad) + this.ctx.fg("dim", lineNumberLabel) + this.ctx.fg("borderMuted", " │ ");
+			const prefixAnsi = dimLine(markerPad) + this.ctx.fg("dim", lineNumberLabel) + dimLine(" │ ");
 			const prefixPlain = `${markerPad}${stripAnsi(lineNumberLabel)} │ `;
 			const codeWidth = Math.max(0, columnWidth - safeVisibleWidth(prefixPlain));
 			return padRenderedLineWidth(prefixAnsi + " ".repeat(codeWidth), columnWidth);
@@ -894,7 +894,7 @@ class SplitDiffRenderer {
 		const lines: string[] = [];
 		lines.push(
 			padRenderedLineWidth(
-				formatBorderCell(leftWidth, "┬") + this.ctx.fg("borderMuted", "─┬─") + formatBorderCell(rightWidth, "┬"),
+				formatBorderCell(leftWidth, "┬") + dimLine("─┬─") + formatBorderCell(rightWidth, "┬"),
 				safeWidth,
 			),
 		);
@@ -931,7 +931,7 @@ class SplitDiffRenderer {
 
 		lines.push(
 			padRenderedLineWidth(
-				formatBorderCell(leftWidth, "┴") + this.ctx.fg("borderMuted", "─┴─") + formatBorderCell(rightWidth, "┴"),
+				formatBorderCell(leftWidth, "┴") + dimLine("─┴─") + formatBorderCell(rightWidth, "┴"),
 				safeWidth,
 			),
 		);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quandev104/pi-style",
-  "version": "0.1.4",
+  "version": "0.1.6",
   "description": "A native-layout, cohesive visual style package for Pi.",
   "license": "MIT",
   "type": "module",
@@ -42,18 +42,18 @@
     "check": "npm run typecheck && npm run lint && npm run depcruise && npm run test && npm run build && npm run package:smoke"
   },
   "peerDependencies": {
-    "@earendil-works/pi-agent-core": ">=0.83.0 <0.84.0",
-    "@earendil-works/pi-ai": ">=0.83.0 <0.84.0",
-    "@earendil-works/pi-coding-agent": ">=0.83.0 <0.84.0",
-    "@earendil-works/pi-tui": ">=0.83.0 <0.84.0",
+    "@earendil-works/pi-agent-core": ">=0.83.0 <0.85.0",
+    "@earendil-works/pi-ai": ">=0.83.0 <0.85.0",
+    "@earendil-works/pi-coding-agent": ">=0.83.0 <0.85.0",
+    "@earendil-works/pi-tui": ">=0.83.0 <0.85.0",
     "typebox": ">=1.3.9 <2.0.0"
   },
   "devDependencies": {
     "@biomejs/biome": "^2.5.0",
-    "@earendil-works/pi-agent-core": "^0.83.0",
-    "@earendil-works/pi-ai": "^0.83.0",
-    "@earendil-works/pi-coding-agent": "^0.83.0",
-    "@earendil-works/pi-tui": "^0.83.0",
+    "@earendil-works/pi-agent-core": "^0.84.0",
+    "@earendil-works/pi-ai": "^0.84.0",
+    "@earendil-works/pi-coding-agent": "^0.84.0",
+    "@earendil-works/pi-tui": "^0.84.0",
     "@types/node": "^24.0.0",
     "dependency-cruiser": "^17.4.3",
     "git-cliff": "^2.13.1",

package/themes/titanium-light.json

@@ -0,0 +1,82 @@
+{
+  "$schema": "https://raw.githubusercontent.com/earendil-works/pi/main/packages/coding-agent/src/modes/interactive/theme/theme-schema.json",
+  "name": "titanium-light",
+  "vars": {
+    "teal": "#5a8080",
+    "blue": "#547da7",
+    "green": "#588458",
+    "red": "#aa5555",
+    "yellow": "#9a7326",
+    "mediumGray": "#6c6c6c",
+    "dimGray": "#767676",
+    "lightGray": "#b0b0b0",
+    "selectedBg": "#d0d0e0",
+    "userMsgBg": "#e8e8e8",
+    "toolPendingBg": "#e8e8f0",
+    "toolSuccessBg": "#e8f0e8",
+    "toolErrorBg": "#f0e8e8",
+    "customMsgBg": "#ede7f6"
+  },
+  "colors": {
+    "accent": "teal",
+    "border": "blue",
+    "borderAccent": "teal",
+    "borderMuted": "lightGray",
+    "success": "green",
+    "error": "red",
+    "warning": "yellow",
+    "muted": "mediumGray",
+    "dim": "dimGray",
+    "text": "",
+    "thinkingText": "mediumGray",
+
+    "selectedBg": "selectedBg",
+    "userMessageBg": "userMsgBg",
+    "userMessageText": "",
+    "customMessageBg": "customMsgBg",
+    "customMessageText": "",
+    "customMessageLabel": "#7e57c2",
+    "toolPendingBg": "toolPendingBg",
+    "toolSuccessBg": "toolSuccessBg",
+    "toolErrorBg": "toolErrorBg",
+    "toolTitle": "",
+    "toolOutput": "mediumGray",
+
+    "mdHeading": "yellow",
+    "mdLink": "blue",
+    "mdLinkUrl": "dimGray",
+    "mdCode": "teal",
+    "mdCodeBlock": "green",
+    "mdCodeBlockBorder": "mediumGray",
+    "mdQuote": "mediumGray",
+    "mdQuoteBorder": "mediumGray",
+    "mdHr": "mediumGray",
+    "mdListBullet": "green",
+
+    "toolDiffAdded": "green",
+    "toolDiffRemoved": "red",
+    "toolDiffContext": "mediumGray",
+    "syntaxComment": "#008000",
+    "syntaxKeyword": "#0000FF",
+    "syntaxFunction": "#795E26",
+    "syntaxVariable": "#001080",
+    "syntaxString": "#A31515",
+    "syntaxNumber": "#098658",
+    "syntaxType": "#267F99",
+    "syntaxOperator": "#000000",
+    "syntaxPunctuation": "#000000",
+
+    "thinkingOff": "lightGray",
+    "thinkingMinimal": "#767676",
+    "thinkingLow": "blue",
+    "thinkingMedium": "teal",
+    "thinkingHigh": "#875f87",
+    "thinkingXhigh": "#8b008b",
+    "bashMode": "green"
+  },
+  "export": {
+    "pageBg": "#f8f8f8",
+    "cardBg": "#ffffff",
+    "infoBg": "#fffae6"
+  }
+}

package/themes/titanium.json

@@ -0,0 +1,79 @@
+{
+  "$schema": "https://raw.githubusercontent.com/earendil-works/pi/main/packages/coding-agent/src/modes/interactive/theme/theme-schema.json",
+  "name": "titanium",
+  "vars": {
+    "brushedTitanium": "#151820",
+    "darkTitanium": "#0f1216",
+    "electricBlue": "#00b4ff",
+    "deepBlue": "#0082b3",
+    "titaniumGold": "#d4c090",
+    "brightAluminum": "#e8ecf4",
+    "dimAluminum": "#9ca3b0",
+    "warningAmber": "#ffb347",
+    "readoutGreen": "#00ff88",
+    "alertRed": "#ff4757",
+    "subtleGray": "#2a3038"
+  },
+  "colors": {
+    "accent": "electricBlue",
+    "border": "subtleGray",
+    "borderAccent": "electricBlue",
+    "borderMuted": "#1f252d",
+    "success": "readoutGreen",
+    "error": "alertRed",
+    "warning": "warningAmber",
+    "muted": "dimAluminum",
+    "dim": "#6b7280",
+    "text": "",
+    "thinkingText": "dimAluminum",
+
+    "selectedBg": "deepBlue",
+    "userMessageBg": "darkTitanium",
+    "userMessageText": "",
+    "customMessageBg": "subtleGray",
+    "customMessageText": "",
+    "customMessageLabel": "titaniumGold",
+    "toolPendingBg": "darkTitanium",
+    "toolSuccessBg": "darkTitanium",
+    "toolErrorBg": "#1a0f10",
+    "toolTitle": "",
+    "toolOutput": "dimAluminum",
+
+    "mdHeading": "electricBlue",
+    "mdLink": "electricBlue",
+    "mdLinkUrl": "deepBlue",
+    "mdCode": "readoutGreen",
+    "mdCodeBlock": "dimAluminum",
+    "mdCodeBlockBorder": "subtleGray",
+    "mdQuote": "dimAluminum",
+    "mdQuoteBorder": "subtleGray",
+    "mdHr": "subtleGray",
+    "mdListBullet": "electricBlue",
+
+    "toolDiffAdded": "readoutGreen",
+    "toolDiffRemoved": "alertRed",
+    "toolDiffContext": "dimAluminum",
+    "syntaxComment": "#6b7280",
+    "syntaxKeyword": "electricBlue",
+    "syntaxFunction": "readoutGreen",
+    "syntaxVariable": "brightAluminum",
+    "syntaxString": "titaniumGold",
+    "syntaxNumber": "warningAmber",
+    "syntaxType": "electricBlue",
+    "syntaxOperator": "electricBlue",
+    "syntaxPunctuation": "dimAluminum",
+
+    "thinkingOff": "#4a5058",
+    "thinkingMinimal": "#5a6068",
+    "thinkingLow": "#6a7078",
+    "thinkingMedium": "dimAluminum",
+    "thinkingHigh": "electricBlue",
+    "thinkingXhigh": "titaniumGold",
+    "bashMode": "readoutGreen"
+  },
+  "export": {
+    "pageBg": "brushedTitanium",
+    "cardBg": "darkTitanium",
+    "infoBg": "subtleGray"
+  }
+}