pi-claude-code-ui 1.0.72 → 1.0.74

package/CHANGELOG.md

@@ -3,11 +3,24 @@
 > [!IMPORTANT]
 > **1.0.69 — package rename (permanent).** Canonical npm name is now [`pi-claude-code-ui`](https://www.npmjs.com/package/pi-claude-code-ui). `pi-claude-style-tools` is legacy and will not receive further releases. Install with `pi install npm:pi-claude-code-ui` or `npm i pi-claude-code-ui`.
 
+## 1.0.74 — 2026-07-18
+
+### Fixed
+
+- **Grouped tool boxes keep Agent breathe aligned** — group rows strip all breathe glyphs (including `·` and the blank off-phase) before re-prefixing a fresh light, so titles no longer walk sideways. Group header/child lights also follow the shared blink phase (and Agent breathe) instead of wall-clock `isBlinkOn()`.
+
+## 1.0.73 — 2026-07-18
+
+### Fixed
+
+- **Agent rows align with other tools again** — removed the extra leading indent that only applied to Agent-family tool rows.
+- **Agent breathe stays centered** — drop double-width `⬤` (it walked the baseline). Cycle is now single-cell glyphs `● → • → · → (invisible) → · → •` so the optical center never moves.
+
 ## 1.0.72 — 2026-07-18
 
 ### Changed
 
-- **Agent tools breathe** — `Agent` / subagent tools use a size cycle while pending (`⬤ → ● → • → (invisible) → • → ●`) instead of the ordinary on/off `●` blink, so agent work reads as a different kind of tool.
+- **Agent tools breathe** — `Agent` / subagent tools use a size cycle while pending instead of the ordinary on/off `●` blink, so agent work reads as a different kind of tool.
 
 ## 1.0.71 — 2026-07-18
 

package/extensions/index.ts

@@ -440,12 +440,6 @@ function isAgentFamilyToolName(name: unknown): boolean {
 	return typeof name === "string" && AGENT_FAMILY_TOOL_NAMES.has(name);
 }
 
-function shouldIndentToolExecution(value: unknown): boolean {
-	if (!value || typeof value !== "object") return false;
-	const toolName = (value as Record<string, unknown>).toolName;
-	return isAgentFamilyToolName(toolName);
-}
-
 function isTerminalImageLine(line: string): boolean {
 	return line.includes(KITTY_IMAGE_PREFIX) || line.includes(ITERM2_IMAGE_PREFIX);
 }
@@ -550,8 +544,11 @@ function getToolGroupOverallStatus(tools: any[]): ToolStatus {
 // spot for ordinary tools. Agent-family tools use a breathing size cycle.
 const STATUS_DOT_FILLED = "●";
 const STATUS_DOT_BOLD = "\x1b[1m";
-// big → medium → small → invisible → small → medium (then back to big)
-const AGENT_BREATHE_GLYPHS = ["⬤", "●", "•", " ", "•", "●"] as const;
+// Single-cell glyphs only (⬤ is often double-width and walks the baseline).
+// Optical sizes share the same cell so the center stays put while breathing:
+// big ● → medium • → small · → invisible → small · → medium •
+const AGENT_BREATHE_GLYPHS = ["●", "•", "·", " ", "·", "•"] as const;
+const AGENT_BREATHE_LEN = AGENT_BREATHE_GLYPHS.length;
 
 function paintStatusDot(colorAnsi: string): string {
 	return `${colorAnsi}${STATUS_DOT_BOLD}${STATUS_DOT_FILLED}${TRANSPARENT_RESET}`;
@@ -562,17 +559,33 @@ function themeStatusDot(theme: Theme, colorKey: "success" | "error" | "dim" | "m
 	return theme.fg(colorKey, `${STATUS_DOT_BOLD}${STATUS_DOT_FILLED}`);
 }
 
+function agentBreatheGlyphRaw(): string {
+	// Always exactly one display cell — matches ordinary tool dots, keeps titles aligned.
+	return AGENT_BREATHE_GLYPHS[_globalBlinkPhaseIndex % AGENT_BREATHE_LEN];
+}
+
+function paintAgentBreatheDot(colorAnsi: string = TOOL_STATUS_SUCCESS): string {
+	const glyph = agentBreatheGlyphRaw();
+	if (glyph === " ") return " ";
+	// Bold only on the largest frame so weight changes without shifting the cell.
+	const bold = glyph === "●" ? STATUS_DOT_BOLD : "";
+	return `${colorAnsi}${bold}${glyph}${TRANSPARENT_RESET}`;
+}
+
 function agentBreatheDot(theme: Theme): string {
-	const glyph = AGENT_BREATHE_GLYPHS[_globalBlinkPhaseIndex % AGENT_BREATHE_GLYPHS.length];
+	const glyph = agentBreatheGlyphRaw();
 	if (glyph === " ") return " ";
-	return theme.fg("success", `${STATUS_DOT_BOLD}${glyph}`);
+	const bold = glyph === "●" ? STATUS_DOT_BOLD : "";
+	return theme.fg("success", `${bold}${glyph}`);
 }
 
-function groupStatusLight(status: ToolStatus): string {
+function groupStatusLight(status: ToolStatus, options?: { agentBreathe?: boolean }): string {
 	const color = status === "success" ? TOOL_STATUS_SUCCESS : status === "error" ? TOOL_STATUS_ERROR : TOOL_STATUS_PENDING;
 	if (status === "pending") {
-		// On phase: solid green. Off phase: disappear completely (space keeps alignment).
-		return isBlinkOn() ? paintStatusDot(TOOL_STATUS_SUCCESS) : " ";
+		// Prefer the shared blink phase over wall-clock so group lights stay in sync
+		// with the global timer (and Agent breathe). Space keeps column alignment.
+		if (options?.agentBreathe) return paintAgentBreatheDot(TOOL_STATUS_SUCCESS);
+		return _globalBlinkPhase ? paintStatusDot(TOOL_STATUS_SUCCESS) : " ";
 	}
 	return paintStatusDot(color);
 }
@@ -610,7 +623,13 @@ function stripToolChrome(lines: string[]): string[] {
 }
 
 function stripLeadingToolStatus(line: string): string {
-	return line.replace(/^((?:\x1b\[[0-9;]*m|[ \t]|[├└│─])*)(?:\x1b\[[0-9;]*m)*[●○✗■⬤•](?:\x1b\[[0-9;]*m)*\s+/, "$1");
+	// Drop the single-cell status marker so group rows can re-prefix a fresh light.
+	// Include Agent breathe glyphs (·) and the blank off-phase (space) so the title
+	// never keeps a leftover marker that shifts when size changes.
+	return line.replace(
+		/^((?:\x1b\[[0-9;]*m|[ \t]|[├└│─])*)(?:\x1b\[[0-9;]*m)*(?:[●○✗■⬤•·]| )(?:\x1b\[[0-9;]*m)*\s+/,
+		"$1",
+	);
 }
 
 function trimAnsiLeft(text: string): string {
@@ -684,19 +703,30 @@ function branchContinuation(index: number, total: number, theme?: Theme): string
 	return index === total - 1 ? "   " : ` ${rule}│${TRANSPARENT_RESET} `;
 }
 
-function formatBranchedToolLines(lines: string[], index: number, total: number, width: number, status: ToolStatus): string[] {
+function formatBranchedToolLines(
+	lines: string[],
+	index: number,
+	total: number,
+	width: number,
+	status: ToolStatus,
+	options?: { agentBreathe?: boolean },
+): string[] {
 	const output: string[] = [];
 	const content = lines.filter((line) => isTerminalImageLine(line) || stripAnsi(line).trim().length > 0);
 	const safeContent = content.length > 0 ? content : [""];
-	const light = groupStatusLight(status);
+	const light = groupStatusLight(status, options);
 	for (let lineIndex = 0; lineIndex < safeContent.length; lineIndex++) {
 		const line = safeContent[lineIndex];
 		if (isTerminalImageLine(line)) {
 			output.push(line);
 			continue;
 		}
+		// Always strip any leftover status marker from the child call line before
+		// re-prefixing. Agent breathe used · which the old stripper missed, so the
+		// title walked sideways as size changed inside groups.
+		const body = lineIndex === 0 ? removeGroupedToolPrefix(line) : trimAnsiLeft(line);
 		const prefix = lineIndex === 0 ? `${branchPrefix(index, total)}${light} ` : `${branchContinuation(index, total)}  `;
-		output.push(clampLineWidth(`${prefix}${trimAnsiLeft(line)}`, width));
+		output.push(clampLineWidth(`${prefix}${body}`, width));
 	}
 	return output;
 }
@@ -818,7 +848,11 @@ class ToolGroupComponent extends Container {
 		const label = getToolGroupLabel(this.tools);
 		const names = groupedName ? "" : formatToolNameList(this.tools);
 		const overall: ToolStatus = status.error > 0 ? "error" : status.pending > 0 ? "pending" : "success";
-		const light = groupStatusLight(overall);
+		// Group header breathes only when every pending member is Agent-family;
+		// mixed groups keep the ordinary on/off light.
+		const pendingTools = this.tools.filter((tool) => getToolStatusForGroup(tool) === "pending");
+		const headerBreathe = pendingTools.length > 0 && pendingTools.every((tool) => isAgentFamilyToolName(getToolName(tool)));
+		const light = groupStatusLight(overall, { agentBreathe: headerBreathe });
 		const summaryLabel = `${label}:`;
 		const countParts: string[] = [];
 		if (status.pending) countParts.push(statusText("pending", status.pending));
@@ -835,7 +869,14 @@ class ToolGroupComponent extends Container {
 			const rawLines = this.expanded
 				? getExpandedToolGroupLines(tool, childWidth, groupedName ? label : undefined)
 				: [getCompactToolLine(tool, childWidth, groupedName ? label : undefined)];
-			const branched = formatBranchedToolLines(rawLines, index, total, safeWidth, getToolStatusForGroup(tool));
+			const branched = formatBranchedToolLines(
+				rawLines,
+				index,
+				total,
+				safeWidth,
+				getToolStatusForGroup(tool),
+				{ agentBreathe: isAgentFamilyToolName(getToolName(tool)) },
+			);
 			for (let i = 0; i < branched.length; i++) {
 				lines.push(clampLineWidth(branched[i], safeWidth));
 			}
@@ -1018,10 +1059,11 @@ function patchGlobalToolBorders(): void {
 			(this as any)[TOOL_RENDER_CACHE] = { width, mode: toolBackgroundMode, lines: rendered, ...branchCache };
 			return rendered;
 		}
-		const indentTool = shouldIndentToolExecution(this);
+		// Agent-family tools stay column-aligned with every other tool row — no extra
+		// leading indent (the old nested pad made Agent look offset from Read/Bash).
 		const core = textLines.map((line) => {
 			const normalized = normalizeLeadingCheckGlyph(line);
-			return clampLineWidth(stripOuterBackgroundAnsi(indentTool && normalized ? ` ${normalized}` : normalized), width);
+			return clampLineWidth(stripOuterBackgroundAnsi(normalized), width);
 		});
 		const spacerLine = " ".repeat(width);
 		let result: string[];
@@ -2681,7 +2723,7 @@ function _scheduleGlobalBlinkTimer(): void {
 			_clearAllBlinkContexts();
 			return;
 		}
-		_globalBlinkPhaseIndex = (_globalBlinkPhaseIndex + 1) % AGENT_BREATHE_GLYPHS.length;
+		_globalBlinkPhaseIndex = (_globalBlinkPhaseIndex + 1) % AGENT_BREATHE_LEN;
 		_globalBlinkPhase = _globalBlinkPhaseIndex % 2 === 0;
 		for (const entry of getBlinkingEntries()) {
 			try { entry.invalidate(); } catch { /* noop */ }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-claude-code-ui",
-  "version": "1.0.72",
+  "version": "1.0.74",
   "description": "Claude Code-style grouped tool rows for pi with Ctrl+O/Ctrl+Shift+O previews, math rendering, and consistent built-in, MCP, and custom tool rendering",
   "keywords": [
     "pi-package",