pi-compact-tools 0.4.2 → 0.5.6

package/LICENSE

@@ -1,21 +1,21 @@
-MIT License
-
-Copyright (c) 2026 Dong Hyeong Lee
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.
+MIT License
+
+Copyright (c) 2026 Dong Hyeong Lee
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/README.md

@@ -22,7 +22,7 @@ The spinner animates during execution, then changes to a duration-aware completi
 
 ### Initially visible edit diffs
 
-Completed `edit` rows show their full diff initially by default. The per-tool `auto_compact` setting controls which tools start in their compact summary state without disabling click or `Ctrl+O` interaction.
+Completed `edit` rows show their full diff initially by default. The per-tool `auto_compact` setting controls which tools start with their result hidden without disabling click or `Ctrl+O` interaction.
 
 ![Edit diff](https://raw.githubusercontent.com/nedleeds/pi-compact-tools/main/assets/edit-diff.gif)
 
@@ -31,9 +31,10 @@ Completed `edit` rows show their full diff initially by default. The per-tool `a
 - Compact rendering for `read`, `write`, `edit`, `bash`, `powershell`, `grep`, `find`, and `ls`
 - Animated configurable spinner in fullscreen TUI mode
 - Millisecond-precision duration and configurable completion icons
-- Per-tool initial compaction with `auto_compact: true` or `false`
-- Click or `Ctrl+O` always cycles through meaningful detail levels
-- Short previews, full output, and initially visible edit diffs by default
+- Per-tool initial compaction and global-toggle participation with `auto_compact: true` or `false`
+- Tool invocation details are always fully visible
+- Click toggles one result between hidden and fully visible; `Ctrl+O` toggles all participating rows
+- Full output on expansion and initially visible edit diffs by default
 - Dark `│` and `└─` visual grouping
 - Distinct cool-blue thinking text
 - Windows, Unix, and classic Mac line-ending support
@@ -86,7 +87,6 @@ Default configuration:
     "edit": false,
     "bash": true
   },
-  "previewLines": 10,
   "spinner": {
     "frames": ["◐", "◓", "◑", "◒"],
     "intervalMs": 120
@@ -104,13 +104,13 @@ These defaults use emoji and standard Unicode, so they work without a Nerd Font.
 
 For a minimal blinking spinner that reuses the successful-tool circle, copy [`examples/compact-tools-2.json`](examples/compact-tools-2.json). Its frames use `● ● (blank) ● ●` at 120 ms intervals. The renderer's built-in frame dimming turns this into a fade-out/fade-in animation without Nerd Font glyphs. Duration indicators use a small `•` whose color changes by elapsed time; the fast tier uses the theme's warning color value `#E0A052` directly. Rename it to `compact-tools.json` or copy its contents to the active configuration path.
 
-`auto_compact` controls each tool's initial state independently:
+`auto_compact` controls each tool's initial state and whether it participates in the global `Ctrl+O` toggle:
 
-- `true` starts at the compact summary.
-- `false` starts at the most detailed available level.
-- Click or `Ctrl+O` continues to cycle through available levels in either case.
+- `true` starts with the result hidden and participates in `Ctrl+O`.
+- `false` starts with the full result visible and is unaffected by `Ctrl+O`.
+- Clicking any compact-rendered row still toggles that row's result.
 
-Unspecified entries inherit the previous configuration layer. By default, `read`, `write`, and `bash` start compact, while `edit` starts expanded so code diffs are immediately visible.
+Unspecified entries inherit the previous configuration layer. By default, `read`, `write`, and `bash` start compact and participate in the global toggle, while `edit` starts expanded and remains unaffected so code diffs stay visible.
 
 `durationIndicators` must have ascending `underMs` values, with a final fallback entry that omits `underMs`. `color` is optional and accepts a supported theme color or six-digit hex value. Existing icon-only configurations remain compatible.
 
@@ -132,7 +132,6 @@ If your terminal uses a [Nerd Font](https://www.nerdfonts.com/), this preset ena
 {
   "$schema": "https://raw.githubusercontent.com/nedleeds/pi-compact-tools/main/schemas/compact-tools.schema.json",
   "tools": ["read", "write", "edit", "bash", "powershell", "grep", "find", "ls"],
-  "previewLines": 10,
   "spinner": {
     "frames": ["✽", "✻", "✲", "✢", "✲", "✻"],
     "intervalMs": 80
@@ -150,13 +149,21 @@ The escaped code points remain readable on GitHub and are decoded to Nerd Font i
 
 ## Controls
 
-`Ctrl+O` or a mouse click cycles only through available content:
+Every tool's complete invocation is always visible, including paths, patterns, shell commands, and auxiliary arguments. Large payloads represented as results, such as write content and edit diffs, follow the result toggle.
+
+A mouse click toggles one row's result:
 
 ```text
-summary → arguments → output preview → full output → summary
+result hidden ↔ full result
 ```
 
-The `auto_compact` boolean only chooses where that cycle starts. With the default `edit: false`, the diff starts visible and the next click collapses it.
+`Ctrl+O` operates on all rows whose tool has `auto_compact: true`:
+
+- If any participating row is not fully expanded, it expands all participating rows.
+- If every participating row is fully expanded, it collapses all participating rows.
+- Rows whose tool has `auto_compact: false` are left unchanged.
+
+The extension handles and consumes `Ctrl+O` directly, so no Pi keybinding changes are required. With the default `edit: false`, edit diffs start visible and are excluded from the global toggle, while clicking an edit row still collapses or expands it.
 
 ## Notes
 

package/examples/compact-tools-2.json

@@ -7,7 +7,6 @@
     "edit": false,
     "bash": true
   },
-  "previewLines": 10,
   "spinner": {
     "frames": ["●", "●", " ", "●", "●"],
     "intervalMs": 120

package/examples/compact-tools.json

@@ -7,7 +7,6 @@
     "edit": false,
     "bash": true
   },
-  "previewLines": 10,
   "spinner": {
     "frames": ["◐", "◓", "◑", "◒"],
     "intervalMs": 120

package/extensions/compact-tools-config.ts

@@ -0,0 +1,163 @@
+import { existsSync, readFileSync } from "node:fs";
+import { join } from "node:path";
+import { CONFIG_DIR_NAME, getAgentDir } from "@earendil-works/pi-coding-agent";
+import { parseDurationIndicators } from "./compact-tools-core.ts";
+import {
+	SUPPORTED_TOOL_SET,
+	type CompactToolName,
+	type CompactToolsConfig,
+} from "./compact-tools-types.ts";
+
+const CONFIG_FILE = "compact-tools.json";
+type JsonObject = Record<string, unknown>;
+
+export const DEFAULT_CONFIG: CompactToolsConfig = {
+	tools: ["read", "write", "edit", "bash"],
+	auto_compact: {
+		read: true,
+		write: true,
+		edit: false,
+		bash: true,
+		powershell: true,
+		grep: true,
+		find: true,
+		ls: true,
+	},
+	spinner: {
+		frames: ["◐", "◓", "◑", "◒"],
+		intervalMs: 120,
+	},
+	durationIndicators: [
+		{ underMs: 1_000, icon: "⚡️", color: "warning" },
+		{ underMs: 10_000, icon: "🔥", color: "#D95C3F" },
+		{ underMs: 30_000, icon: "○", color: "#79C0FF" },
+		{ icon: "⏳", color: "#D2A8FF" },
+	],
+};
+
+function isObject(value: unknown): value is JsonObject {
+	return typeof value === "object" && value !== null && !Array.isArray(value);
+}
+
+function warn(path: string, message: string): void {
+	console.error(`[compact-tools] ${path}: ${message}`);
+}
+
+function isIntegerInRange(value: unknown, minimum: number, maximum: number): value is number {
+	return typeof value === "number" && Number.isInteger(value) && value >= minimum && value <= maximum;
+}
+
+function isNonEmptyStringArray(value: unknown): value is string[] {
+	return Array.isArray(value) && value.length > 0 && value.every((item) => typeof item === "string" && item.length > 0);
+}
+
+function parseTools(value: unknown, path: string): CompactToolName[] | undefined {
+	if (!Array.isArray(value)) {
+		warn(path, "tools must be an array; using previous values");
+		return undefined;
+	}
+	const tools: CompactToolName[] = [];
+	const invalid: unknown[] = [];
+	for (const item of value) {
+		if (typeof item === "string" && SUPPORTED_TOOL_SET.has(item)) tools.push(item as CompactToolName);
+		else invalid.push(item);
+	}
+	if (invalid.length > 0) warn(path, `ignoring unsupported tools: ${invalid.map(String).join(", ")}`);
+	return [...new Set(tools)];
+}
+
+function parseAutoCompact(
+	base: CompactToolsConfig["auto_compact"],
+	value: unknown,
+	path: string,
+): CompactToolsConfig["auto_compact"] {
+	if (value === undefined) return base;
+	if (!isObject(value)) {
+		warn(path, "auto_compact must be an object; using previous values");
+		return base;
+	}
+	const result = { ...base };
+	for (const [name, enabled] of Object.entries(value)) {
+		if (!SUPPORTED_TOOL_SET.has(name)) {
+			warn(path, `ignoring unsupported auto_compact tool: ${name}`);
+		} else if (typeof enabled !== "boolean") {
+			warn(path, `auto_compact.${name} must be true or false; using previous value`);
+		} else {
+			result[name as CompactToolName] = enabled;
+		}
+	}
+	return result;
+}
+
+function parseSpinner(
+	base: CompactToolsConfig["spinner"],
+	value: unknown,
+	path: string,
+): CompactToolsConfig["spinner"] {
+	if (value === undefined) return base;
+	if (!isObject(value)) {
+		warn(path, "spinner must be an object; using previous values");
+		return base;
+	}
+	const frames = value.frames;
+	const intervalMs = value.intervalMs;
+	const validFrames = isNonEmptyStringArray(frames);
+	const validInterval = isIntegerInRange(intervalMs, 40, 5_000);
+	if (frames !== undefined && !validFrames) warn(path, "spinner.frames must be a non-empty string array");
+	if (intervalMs !== undefined && !validInterval) warn(path, "spinner.intervalMs must be 40–5000");
+	return {
+		frames: validFrames ? frames : base.frames,
+		intervalMs: validInterval ? intervalMs : base.intervalMs,
+	};
+}
+
+export function mergeConfig(base: CompactToolsConfig, value: unknown, path: string): CompactToolsConfig {
+	if (value === undefined) return base;
+	if (!isObject(value)) {
+		warn(path, "expected a JSON object; using previous values");
+		return base;
+	}
+	const tools = value.tools === undefined ? base.tools : (parseTools(value.tools, path) ?? base.tools);
+	const auto_compact = parseAutoCompact(base.auto_compact, value.auto_compact, path);
+	const spinner = parseSpinner(base.spinner, value.spinner, path);
+	const parsedIndicators = value.durationIndicators === undefined
+		? undefined
+		: parseDurationIndicators(value.durationIndicators);
+	if (value.durationIndicators !== undefined && !parsedIndicators) {
+		warn(path, "invalid durationIndicators; using previous values");
+	}
+	return {
+		tools,
+		auto_compact,
+		spinner,
+		durationIndicators: parsedIndicators ?? base.durationIndicators,
+	};
+}
+
+function readJson(path: string): unknown {
+	if (!existsSync(path)) return undefined;
+	try {
+		return JSON.parse(readFileSync(path, "utf8"));
+	} catch (error) {
+		warn(path, error instanceof Error ? error.message : String(error));
+		return undefined;
+	}
+}
+
+export function loadConfig(cwd?: string, projectTrusted = false): CompactToolsConfig {
+	const globalPath = join(getAgentDir(), CONFIG_FILE);
+	const globalConfig = mergeConfig(DEFAULT_CONFIG, readJson(globalPath), globalPath);
+	if (!cwd || !projectTrusted) return globalConfig;
+	const projectPath = join(cwd, CONFIG_DIR_NAME, CONFIG_FILE);
+	return mergeConfig(globalConfig, readJson(projectPath), projectPath);
+}
+
+export function isFullscreenMode(argv = process.argv): boolean {
+	for (let index = 0; index < argv.length; index++) {
+		const argument = argv[index];
+		if (argument === "--tui-mode") return argv[index + 1] === "fullscreen";
+		if (argument?.startsWith("--tui-mode=")) return argument.slice("--tui-mode=".length) === "fullscreen";
+	}
+	const settings = readJson(join(getAgentDir(), "settings.json"));
+	return isObject(settings) && settings.tuiMode === "fullscreen";
+}

package/extensions/compact-tools-core.ts

@@ -13,7 +13,6 @@ export type DurationIndicatorConfig = {
 	color?: DurationIndicatorColor;
 };
 export type RowStatus = "pending" | "running" | "success" | "error";
-
 const DURATION_INDICATOR_COLOR_SET = new Set<string>(DURATION_INDICATOR_COLORS);
 export const HEX_COLOR_PATTERN = /^#[0-9a-f]{6}$/i;
 

package/extensions/compact-tools-input.ts

@@ -0,0 +1,8 @@
+import { isKeyRelease, Key, matchesKey } from "@earendil-works/pi-tui";
+
+export type ToggleInput = "toggle" | "release";
+
+export function classifyToggleInput(data: string): ToggleInput | undefined {
+	if (!matchesKey(data, Key.ctrl("o"))) return undefined;
+	return isKeyRelease(data) ? "release" : "toggle";
+}

package/extensions/compact-tools-invocation.ts

@@ -0,0 +1,59 @@
+import type { AgentToolResult } from "@earendil-works/pi-coding-agent";
+import { normalizeLineEndings } from "./compact-tools-core.ts";
+import type { ToolArgs } from "./compact-tools-types.ts";
+
+function pathFrom(args: ToolArgs): string {
+	const value = args.path ?? args.file_path;
+	return typeof value === "string" ? normalizeLineEndings(value) : "";
+}
+
+export function formatReadCallDetails(path: string, args: ToolArgs): string {
+	const options: string[] = [];
+	if (typeof args.offset === "number") options.push(`offset: ${args.offset}`);
+	if (typeof args.limit === "number") options.push(`limit: ${args.limit}`);
+	return options.length === 0 ? path : `${path} (${options.join(", ")})`;
+}
+
+export function getCallDetails(name: string, args: ToolArgs): string {
+	const path = pathFrom(args) || (name === "grep" || name === "find" || name === "ls" ? "." : "");
+	if (name === "read") return formatReadCallDetails(path, args);
+	if (name !== "grep" && name !== "find") return path;
+	const pattern = typeof args.pattern === "string" ? normalizeLineEndings(args.pattern) : "…";
+	return name === "grep" ? `/${pattern}/ in ${path}` : `${pattern} in ${path}`;
+}
+
+function omitArgument(name: string, key: string): boolean {
+	if (key === "path" || key === "file_path") return true;
+	if (name === "read" && (key === "offset" || key === "limit")) return true;
+	if (name === "write" && key === "content") return true;
+	return (name === "grep" || name === "find") && key === "pattern";
+}
+
+export function getArgumentDetails(name: string, args: ToolArgs): ToolArgs {
+	if (name === "edit") return {};
+	const details: ToolArgs = {};
+	for (const [key, value] of Object.entries(args)) {
+		if (value !== undefined && !omitArgument(name, key)) details[key] = value;
+	}
+	return details;
+}
+
+export function getTextResult(result: AgentToolResult<unknown>): string {
+	const parts: string[] = [];
+	for (const item of result.content) {
+		if (item.type === "text") parts.push(item.text);
+	}
+	return normalizeLineEndings(parts.join("\n")).trimEnd();
+}
+
+export function getFileOutput(
+	name: string,
+	args: ToolArgs,
+	result: AgentToolResult<unknown>,
+	isError: boolean,
+): string {
+	if (isError || name === "read" || name === "grep" || name === "find" || name === "ls") {
+		return getTextResult(result);
+	}
+	return name === "write" ? String(args.content ?? "") : "";
+}

package/extensions/compact-tools-layout.ts

@@ -0,0 +1,115 @@
+import type { Theme } from "@earendil-works/pi-coding-agent";
+import type { Component } from "@earendil-works/pi-tui";
+import {
+	sliceByColumn,
+	stripTerminalSequences,
+	visibleWidth,
+	wrapTextWithAnsi,
+} from "@earendil-works/pi-tui";
+import { normalizeLineEndings } from "./compact-tools-core.ts";
+import type { ToolArgs } from "./compact-tools-types.ts";
+
+class CachedComponent implements Component {
+	private cachedWidth?: number;
+	private cachedLines?: string[];
+
+	constructor(private readonly renderLines: (width: number) => string[]) {}
+
+	render(width: number): string[] {
+		if (this.cachedWidth !== width || !this.cachedLines) {
+			this.cachedWidth = width;
+			this.cachedLines = this.renderLines(width);
+		}
+		return this.cachedLines;
+	}
+
+	invalidate(): void {
+		this.cachedWidth = undefined;
+		this.cachedLines = undefined;
+	}
+}
+
+export function prefixedText(text: string, firstPrefix: string, continuationPrefix = firstPrefix): Component {
+	const prefixWidth = Math.max(visibleWidth(firstPrefix), visibleWidth(continuationPrefix));
+	const normalized = text.replace(/\t/g, "   ");
+	return new CachedComponent((width) => {
+		const lines = wrapTextWithAnsi(normalized, Math.max(1, width - prefixWidth));
+		return lines.map((line, index) => `${index === 0 ? firstPrefix : continuationPrefix}${line}`);
+	});
+}
+
+export function hardWrapTextWithAnsi(text: string, width: number): string[] {
+	const safeWidth = Math.max(1, width);
+	const wrapped: string[] = [];
+	for (const logicalLine of text.replace(/\t/g, "   ").split("\n")) {
+		const lineWidth = visibleWidth(logicalLine);
+		if (lineWidth === 0) {
+			wrapped.push("");
+			continue;
+		}
+		for (let offset = 0; offset < lineWidth; offset += safeWidth) {
+			wrapped.push(sliceByColumn(logicalLine, offset, safeWidth, true));
+		}
+	}
+	return wrapped;
+}
+
+export function renderToolCall(title: string, details: string | undefined, theme: Theme): Component {
+	const text = details ? `${title} ${details}` : title;
+	const firstPrefix = " ";
+	const continuationPrefix = theme.fg("border", " │ ");
+	const prefixWidth = visibleWidth(continuationPrefix);
+	return new CachedComponent((width) => {
+		const lines = hardWrapTextWithAnsi(text, width - prefixWidth);
+		return lines.map((line, index) => `${index === 0 ? firstPrefix : continuationPrefix}${line}`);
+	});
+}
+
+export function wrapEditResult(component: Component, theme: Theme): Component {
+	return {
+		render(width: number) {
+			const lines = component.render(Math.max(1, width - 3));
+			const first = lines.findIndex((line) => visibleWidth(line.trim()) > 0);
+			if (first < 0) return [];
+			const contentLines = lines.slice(first);
+			const commonIndent = Math.min(
+				...contentLines
+					.filter((line) => visibleWidth(line.trim()) > 0)
+					.map((line) => stripTerminalSequences(line).match(/^ */)?.[0].length ?? 0),
+			);
+			const prefix = theme.fg("border", " │ ");
+			return contentLines.map((line) => {
+				const content = sliceByColumn(line, commonIndent, visibleWidth(line) - commonIndent, true);
+				return `${prefix}${content}`;
+			});
+		},
+		invalidate() {
+			component.invalidate?.();
+		},
+	};
+}
+
+export function styleMultiline(text: string, style: (line: string) => string): string {
+	return normalizeLineEndings(text).split("\n").map(style).join("\n");
+}
+
+function styleOutputLine(text: string, theme: Theme, isError: boolean): string {
+	const color = isError ? "error" : "toolOutput";
+	return text
+		.split(/(⚡️?)/u)
+		.map((part) => theme.fg(!isError && part.startsWith("⚡") ? "warning" : color, part))
+		.join("");
+}
+
+export function renderOutput(output: string, theme: Theme, isError: boolean): Component | undefined {
+	const normalized = normalizeLineEndings(output).trimEnd();
+	if (!normalized) return undefined;
+	const styled = normalized.split("\n").map((line) => styleOutputLine(line, theme, isError)).join("\n");
+	return prefixedText(styled, theme.fg("border", " │ "));
+}
+
+export function renderArguments(args: ToolArgs, theme: Theme): Component {
+	const json = JSON.stringify(args, null, 2) ?? "{}";
+	const styled = styleMultiline(json, (line) => theme.fg("toolOutput", line));
+	return prefixedText(styled, theme.fg("border", " │ "));
+}