pi-compact-tools 0.5.0 → 0.5.7

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)
 
@@ -32,8 +32,9 @@ Completed `edit` rows show their full diff initially by default. The per-tool `a
 - Animated configurable spinner in fullscreen TUI mode
 - Millisecond-precision duration and configurable completion icons
 - Per-tool initial compaction and global-toggle participation with `auto_compact: true` or `false`
-- Click cycles one row through meaningful detail levels; `Ctrl+O` expands or collapses all participating rows
-- Short previews, full output, and initially visible edit diffs by default
+- 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
@@ -106,9 +106,9 @@ For a minimal blinking spinner that reuses the successful-tool circle, copy [`ex
 
 `auto_compact` controls each tool's initial state and whether it participates in the global `Ctrl+O` toggle:
 
-- `true` starts at the compact summary and participates in `Ctrl+O`.
-- `false` starts at the most detailed available level and is unaffected by `Ctrl+O`.
-- Clicking any compact-rendered row still cycles that row through its available levels.
+- `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 and participate in the global toggle, while `edit` starts expanded and remains unaffected so code diffs stay visible.
 
@@ -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,10 +149,12 @@ The escaped code points remain readable on GitHub and are decoded to Nerd Font i
 
 ## Controls
 
-A mouse click cycles one row through its 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
 ```
 
 `Ctrl+O` operates on all rows whose tool has `auto_compact: true`:

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,15 +13,6 @@ export type DurationIndicatorConfig = {
 	color?: DurationIndicatorColor;
 };
 export type RowStatus = "pending" | "running" | "success" | "error";
-export type ExpandableLevelState = {
-	level?: number;
-	levels: readonly number[];
-};
-
-export function shouldExpandAll(rows: readonly ExpandableLevelState[]): boolean {
-	return rows.some(({ level, levels }) => level !== levels.at(-1));
-}
-
 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,123 @@
+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;
+		}
+		let offset = 0;
+		while (offset < lineWidth) {
+			if (offset > 0) {
+				const remainder = sliceByColumn(logicalLine, offset, lineWidth - offset, true);
+				const whitespace = stripTerminalSequences(remainder).match(/^\s+/u)?.[0] ?? "";
+				offset += visibleWidth(whitespace);
+				if (offset >= lineWidth) break;
+			}
+			wrapped.push(sliceByColumn(logicalLine, offset, safeWidth, true));
+			offset += safeWidth;
+		}
+	}
+	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", " │ "));
+}