pi-compact-tools 0.6.0 → 0.6.1

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

@@ -30,7 +30,7 @@ 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
-- Three-state `Ctrl+T` thinking view: summary, detail, then hidden
+- Four-step `Ctrl+T` thinking cycle: summary, detail, summary, then hidden
 - Millisecond-precision duration with neutral status text
 - Per-tool initial compaction and global-toggle participation with `auto_compact: true` or `false`
 - Tool invocation details are always fully visible
@@ -140,12 +140,12 @@ Remove `"powershell"` on systems where it is unavailable.
 `Ctrl+T` cycles every thinking block through:
 
 ```text
-one-line summary + controls → summary + │-indented detail + controls → Thinking...
+one-line summary → summary + │-indented detail → one-line summary → Thinking...
 ```
 
 Standalone bold lines and Markdown headings split one provider thinking run into logical sections. Each section gets its own summary, optional `│`-indented detail, and control row; `Ctrl+T` cycles all sections together. While thinking streams, a text-color highlight sweeps across summaries using streaming updates; no spinner glyph or title circle is shown. The transformation is display-only; complete thinking remains unchanged in the session and model context.
 
-Relevant invocation details stay visible, including paths, patterns, shell commands, and auxiliary arguments. Read calls omit offset and limit noise, then report the number of lines read after completion. Large payloads represented as results, such as write content and edit diffs, follow the result toggle.
+Relevant invocation details stay visible, including paths, patterns, shell commands, and auxiliary arguments. After `grep`, `find`, or `ls` completes, auxiliary arguments appear in parentheses beside the `Done in …s` status instead of in the invocation line or a JSON block. Read calls omit offset and limit noise, then report the number of lines read after completion. Large payloads represented as results, such as write content and edit diffs, follow the result toggle.
 
 A mouse click toggles one row's result:
 

package/extensions/compact-tools-invocation.ts

@@ -7,13 +7,7 @@ function pathFrom(args: ToolArgs): string {
 	return typeof value === "string" ? normalizeLineEndings(value) : "";
 }
 
-export function getCallDetails(name: string, args: ToolArgs): string {
-	const path = pathFrom(args) || (name === "grep" || name === "find" || name === "ls" ? "." : "");
-	if (name === "read") return path;
-	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}`;
-}
+const INLINE_ARGUMENT_TOOLS = new Set(["grep", "find", "ls"]);
 
 function omitArgument(name: string, key: string): boolean {
 	if (key === "path" || key === "file_path") return true;
@@ -22,8 +16,7 @@ function omitArgument(name: string, key: string): boolean {
 	return (name === "grep" || name === "find") && key === "pattern";
 }
 
-export function getArgumentDetails(name: string, args: ToolArgs): ToolArgs {
-	if (name === "edit") return {};
+function collectArgumentDetails(name: string, args: ToolArgs): ToolArgs {
 	const details: ToolArgs = {};
 	for (const [key, value] of Object.entries(args)) {
 		if (value !== undefined && !omitArgument(name, key)) details[key] = value;
@@ -31,6 +24,37 @@ export function getArgumentDetails(name: string, args: ToolArgs): ToolArgs {
 	return details;
 }
 
+function formatInlineValue(value: unknown): string {
+	if (typeof value === "string") {
+		const compact = normalizeLineEndings(value).replace(/\s+/gu, " ").trim();
+		return compact || '""';
+	}
+	if (typeof value === "number" || typeof value === "boolean" || typeof value === "bigint") {
+		return String(value);
+	}
+	return JSON.stringify(value) ?? String(value);
+}
+
+export function formatArgumentSummary(name: string, args: ToolArgs): string | undefined {
+	if (!INLINE_ARGUMENT_TOOLS.has(name)) return undefined;
+	const arguments_ = Object.entries(collectArgumentDetails(name, args));
+	if (arguments_.length === 0) return undefined;
+	return arguments_.map(([key, value]) => `${key} ${formatInlineValue(value)}`).join(" · ");
+}
+
+export function getCallDetails(name: string, args: ToolArgs): string {
+	const path = pathFrom(args) || (INLINE_ARGUMENT_TOOLS.has(name) ? "." : "");
+	if (name === "read") return path;
+	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}`;
+}
+
+export function getArgumentDetails(name: string, args: ToolArgs): ToolArgs {
+	if (name === "edit" || INLINE_ARGUMENT_TOOLS.has(name)) return {};
+	return collectArgumentDetails(name, args);
+}
+
 export function getTextResult(result: AgentToolResult<unknown>): string {
 	const parts: string[] = [];
 	for (const item of result.content) {

package/extensions/compact-tools-thinking.ts

@@ -3,6 +3,7 @@ import { readFileSync } from "node:fs";
 import { join } from "node:path";
 import {
 	isKeyRelease,
+	isKeyRepeat,
 	Key,
 	matchesKey,
 	stripTerminalSequences,
@@ -11,6 +12,7 @@ import {
 } from "@earendil-works/pi-tui";
 
 export type ThinkingView = "summary" | "detail" | "hidden";
+export type ThinkingPhase = "summary" | "detail" | "summary-after-detail" | "hidden";
 
 type ThinkingRenderOptions = {
 	styleSummary?: (summary: string, sectionIndex: number) => string;
@@ -19,12 +21,29 @@ type ThinkingRenderOptions = {
 	styleControlPrefix?: (prefix: string) => string;
 };
 
-const NEXT_VIEW: Record<ThinkingView, ThinkingView> = {
+const NEXT_PHASE: Record<ThinkingPhase, ThinkingPhase> = {
 	summary: "detail",
-	detail: "hidden",
+	detail: "summary-after-detail",
+	"summary-after-detail": "hidden",
 	hidden: "summary",
 };
 
+export function thinkingViewForPhase(phase: ThinkingPhase): ThinkingView {
+	return phase === "summary-after-detail" ? "summary" : phase;
+}
+
+export function nextThinkingPhase(phase: ThinkingPhase): ThinkingPhase {
+	return NEXT_PHASE[phase];
+}
+
+export type ThinkingToggleInput = "toggle" | "repeat" | "release";
+
+export function classifyThinkingToggleInput(data: string): ThinkingToggleInput | undefined {
+	if (!matchesKey(data, Key.ctrl("t"))) return undefined;
+	if (isKeyRelease(data)) return "release";
+	return isKeyRepeat(data) ? "repeat" : "toggle";
+}
+
 function plainSummary(line: string): string {
 	return line
 		.replace(/^\s{0,3}#{1,6}\s+/u, "")
@@ -145,7 +164,7 @@ export function renderThinkingView(
 		.join("\n\n");
 }
 
-function getInitialThinkingView(): ThinkingView {
+function getInitialThinkingPhase(): ThinkingPhase {
 	try {
 		const settings = JSON.parse(readFileSync(join(getAgentDir(), "settings.json"), "utf8")) as {
 			hideThinkingBlock?: unknown;
@@ -160,6 +179,8 @@ export function isThinkingStreamEvent(eventType: string): boolean {
 	return eventType === "thinking_start" || eventType === "thinking_delta";
 }
 
+const SHIMMER_FRAME_INTERVAL_MS = 80;
+
 function shimmer(summary: string, theme: Theme, startedAt: number): string {
 	const characters = [...summary];
 	if (characters.length === 0) return summary;
@@ -168,7 +189,7 @@ function shimmer(summary: string, theme: Theme, startedAt: number): string {
 	// enters at its first character instead of inheriting another section's
 	// wall-clock phase.
 	const elapsed = Math.max(0, Date.now() - startedAt);
-	const center = Math.floor(elapsed / 80) % (characters.length + sweepTail);
+	const center = Math.floor(elapsed / SHIMMER_FRAME_INTERVAL_MS) % (characters.length + sweepTail);
 	let currentColor: Parameters<Theme["fg"]>[0] | undefined;
 	let chunk = "";
 	let output = "";
@@ -194,17 +215,15 @@ function shimmer(summary: string, theme: Theme, startedAt: number): string {
 }
 
 export class ThinkingCycleController {
-	private view: ThinkingView = "summary";
+	private phase: ThinkingPhase = "summary";
 	private unsubscribe: (() => void) | undefined;
 	private theme: Theme | undefined;
 	private thinkingStreamActive = false;
+	private refreshThinking: (() => void) | undefined;
 	private readonly summarySweepStartedAt = new Map<number, number>();
 
 	constructor(pi: ExtensionAPI) {
-		pi.on("message_start", () => {
-			this.thinkingStreamActive = false;
-			this.summarySweepStartedAt.clear();
-		});
+		pi.on("message_start", () => this.resetSweep());
 		pi.on("message_update", (event) => {
 			// `context.isStreaming` describes the whole assistant message. Tool-call
 			// argument streaming therefore remains true after thinking has ended.
@@ -213,20 +232,14 @@ export class ThinkingCycleController {
 			if (eventType === "thinking_start") this.summarySweepStartedAt.clear();
 			this.thinkingStreamActive = isThinkingStreamEvent(eventType);
 		});
-		pi.on("message_end", () => {
-			this.thinkingStreamActive = false;
-			this.summarySweepStartedAt.clear();
-		});
-		pi.on("tool_call", () => {
-			this.thinkingStreamActive = false;
-			this.summarySweepStartedAt.clear();
-		});
+		pi.on("message_end", () => this.resetSweep());
+		pi.on("tool_call", () => this.resetSweep());
 
 		pi.registerMarkdownTransformer((markdown, context) => {
 			if (context.messageType !== "assistant-thinking") return markdown;
 			const theme = this.theme;
 			const detailTextPrefix = theme?.getFgAnsi("thinkingText");
-			return renderThinkingView(markdown, this.view, context.availableWidth, {
+			return renderThinkingView(markdown, thinkingViewForPhase(this.phase), context.availableWidth, {
 				styleSummary: theme
 					? context.isStreaming && this.thinkingStreamActive
 						? (summary, sectionIndex) => {
@@ -254,6 +267,11 @@ export class ThinkingCycleController {
 		});
 	}
 
+	private resetSweep(): void {
+		this.thinkingStreamActive = false;
+		this.summarySweepStartedAt.clear();
+	}
+
 	bind(ctx: ExtensionContext): void {
 		this.unsubscribe?.();
 		this.theme = ctx.ui.theme;
@@ -263,21 +281,29 @@ export class ThinkingCycleController {
 		// host is hidden, Markdown transformers are not invoked at all. Pi wraps
 		// this label in thinkingText, so an inner thinkingMax span is required to
 		// keep it consistent with completed summary titles.
-		this.view = getInitialThinkingView();
+		this.phase = getInitialThinkingPhase();
 		const hiddenLabel = this.theme.fg("thinkingMax", this.theme.bold("Thinking..."));
-		ctx.ui.setHiddenThinkingLabel(hiddenLabel);
+		// Pi does not currently expose transcript invalidation directly. Updating
+		// this label rebuilds assistant Markdown and requests a render without
+		// changing the message or its hidden/visible state.
+		this.refreshThinking = () => ctx.ui.setHiddenThinkingLabel(hiddenLabel);
+		this.refreshThinking();
 		this.unsubscribe = ctx.ui.onTerminalInput((data) => {
-			if (!matchesKey(data, Key.ctrl("t"))) return undefined;
-			if (isKeyRelease(data)) return { consume: true };
-			this.view = NEXT_VIEW[this.view];
-			if (this.view === "detail") {
-				// summary -> detail stays host-visible; rebuild to apply the transformer.
-				ctx.ui.setHiddenThinkingLabel(hiddenLabel);
+			const input = classifyThinkingToggleInput(data);
+			if (input === undefined) return undefined;
+			// Kitty-capable macOS terminals report held keys as repeat events. Treating
+			// those as presses can skip detail before the user sees it.
+			if (input === "release" || input === "repeat") return { consume: true };
+			this.phase = nextThinkingPhase(this.phase);
+			if (this.phase === "detail" || this.phase === "summary-after-detail") {
+				// Visible-to-visible transitions are owned by this extension. Rebuild the
+				// Markdown and prevent Pi's built-in two-state toggle from hiding it.
+				this.refreshThinking?.();
 				return { consume: true };
 			}
-			// detail -> hidden and hidden -> summary intentionally reach Pi's existing
-			// two-state handler. This keeps host visibility/settings synchronized and
-			// clears built-in per-block click overrides without modifying Pi itself.
+			// summary-after-detail -> hidden and hidden -> summary intentionally reach
+			// Pi's existing two-state handler. This keeps visibility/settings in sync
+			// and clears built-in per-block click overrides.
 			return undefined;
 		});
 	}
@@ -285,6 +311,7 @@ export class ThinkingCycleController {
 	dispose(): void {
 		this.unsubscribe?.();
 		this.unsubscribe = undefined;
+		this.refreshThinking = undefined;
 		this.thinkingStreamActive = false;
 		this.summarySweepStartedAt.clear();
 		this.theme = undefined;

package/extensions/compact-tools.ts

@@ -28,6 +28,7 @@ import {
 import { isFullscreenMode, loadConfig } from "./compact-tools-config.ts";
 import { classifyToggleInput } from "./compact-tools-input.ts";
 import {
+	formatArgumentSummary,
 	formatReadResultSummary,
 	getArgumentDetails,
 	getCallDetails,
@@ -100,12 +101,14 @@ function renderControls(
 	running: boolean,
 	isError: boolean,
 	summary?: string,
+	argumentSummary?: string,
 ): Component {
 	const duration = running && !runtime.animatesRows ? undefined : formatDuration(state);
 	const status = running
 		? (duration ?? "Running")
 		: `${isError ? "Failed" : "Done"}${duration ? ` in ${duration}` : ""}`;
 	let details = theme.fg("borderAccent", status);
+	if (argumentSummary && !running) details += theme.fg("borderAccent", ` (${argumentSummary})`);
 	if (summary) details += theme.fg("borderAccent", ` • ${summary}`);
 	if (state.hasResult) {
 		const clickAction = state.expanded ? "to hide" : "for result";
@@ -192,7 +195,8 @@ function renderFileResult(
 	const container = new CachedContainer();
 	appendFileResult(container, definition, state, output, theme, ctx.isError);
 	const summary = name === "read" && !ctx.isError ? formatReadResultSummary(result) : undefined;
-	container.addChild(renderControls(theme, state, options.isPartial, ctx.isError, summary));
+	const argumentSummary = formatArgumentSummary(name, ctx.args);
+	container.addChild(renderControls(theme, state, options.isPartial, ctx.isError, summary, argumentSummary));
 	return container;
 }
 

package/package.json

@@ -1,64 +1,64 @@
-{
-  "name": "pi-compact-tools",
-  "version": "0.6.0",
-  "description": "Compact, expandable UI for pi's built-in tools, bundled with a polished GitHub Dark theme.",
-  "type": "module",
-  "license": "MIT",
-  "author": "Dong Hyeong Lee",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/nedleeds/pi-compact-tools.git"
-  },
-  "homepage": "https://github.com/nedleeds/pi-compact-tools#readme",
-  "bugs": {
-    "url": "https://github.com/nedleeds/pi-compact-tools/issues"
-  },
-  "keywords": [
-    "pi-package",
-    "pi-coding-agent",
-    "pi-extension",
-    "terminal-ui",
-    "theme"
-  ],
-  "files": [
-    "extensions",
-    "themes",
-    "examples",
-    "scripts",
-    "schemas",
-    "README.md",
-    "LICENSE"
-  ],
-  "scripts": {
-    "test": "tsx --test tests/*.test.ts",
-    "check": "tsc --noEmit && npm test && npm run check:resources",
-    "check:resources": "node scripts/check-resources.mjs",
-    "prepublishOnly": "npm run check"
-  },
-  "pi": {
-    "extensions": [
-      "./extensions/compact-tools.ts"
-    ],
-    "themes": [
-      "./themes/github-dark-pro.json"
-    ],
-    "image": "https://raw.githubusercontent.com/nedleeds/pi-compact-tools/main/assets/nerd-font-workflow-optimized.gif"
-  },
-  "peerDependencies": {
-    "@earendil-works/pi-coding-agent": "*",
-    "@earendil-works/pi-tui": "*"
-  },
-  "devDependencies": {
-    "@earendil-works/pi-coding-agent": "0.85.1",
-    "@earendil-works/pi-tui": "0.85.1",
-    "@types/node": "^24.0.0",
-    "tsx": "^4.23.13",
-    "typescript": "^5.9.0"
-  },
-  "engines": {
-    "node": ">=22.19.0"
-  },
-  "publishConfig": {
-    "access": "public"
-  }
-}
+{
+  "name": "pi-compact-tools",
+  "version": "0.6.1",
+  "description": "Compact, expandable UI for pi's built-in tools, bundled with a polished GitHub Dark theme.",
+  "type": "module",
+  "license": "MIT",
+  "author": "Dong Hyeong Lee",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/nedleeds/pi-compact-tools.git"
+  },
+  "homepage": "https://github.com/nedleeds/pi-compact-tools#readme",
+  "bugs": {
+    "url": "https://github.com/nedleeds/pi-compact-tools/issues"
+  },
+  "keywords": [
+    "pi-package",
+    "pi-coding-agent",
+    "pi-extension",
+    "terminal-ui",
+    "theme"
+  ],
+  "files": [
+    "extensions",
+    "themes",
+    "examples",
+    "scripts",
+    "schemas",
+    "README.md",
+    "LICENSE"
+  ],
+  "scripts": {
+    "test": "tsx --test tests/*.test.ts",
+    "check": "tsc --noEmit && npm test && npm run check:resources",
+    "check:resources": "node scripts/check-resources.mjs",
+    "prepublishOnly": "npm run check"
+  },
+  "pi": {
+    "extensions": [
+      "./extensions/compact-tools.ts"
+    ],
+    "themes": [
+      "./themes/github-dark-pro.json"
+    ],
+    "image": "https://raw.githubusercontent.com/nedleeds/pi-compact-tools/main/assets/nerd-font-workflow-optimized.gif"
+  },
+  "peerDependencies": {
+    "@earendil-works/pi-coding-agent": "*",
+    "@earendil-works/pi-tui": "*"
+  },
+  "devDependencies": {
+    "@earendil-works/pi-coding-agent": "0.85.1",
+    "@earendil-works/pi-tui": "0.85.1",
+    "@types/node": "^24.0.0",
+    "tsx": "^4.23.13",
+    "typescript": "^5.9.0"
+  },
+  "engines": {
+    "node": ">=22.19.0"
+  },
+  "publishConfig": {
+    "access": "public"
+  }
+}