@bismawy/pi-arnative 0.3.6 → 0.3.8

package/CHANGELOG.md

@@ -1,5 +1,27 @@
 # Changelog
 
+## [0.3.8] - 2026-10-08
+
+### Added
+- New footer preset **Arnative (Minimal)**, selectable in `/arnative` → Footers. It shows a single head row (cwd, clock, diff, git, model) with a dim `▲` at the end of each side. Each head's own text is the click target — anywhere on the cwd/git string toggles the left side (extension statuses), anywhere on the model string toggles the right side (speed/cache/token) — while the blank gap between them is inert, so a stray click on empty space cannot expand anything. Opened sides show `▼` and their rows; the two sides open independently. Ctrl+alt+M (alt+M on Windows/WSL) toggles both, so it also works in `tuiMode: "regular"` where the terminal owns the mouse. Zone columns are measured from the row this module places itself rather than mirrored from `pairRow`, because the ellipsis on a truncated head shifts every column after it (`extensions/footer.ts`, `lib/shortcuts.ts`).
+- The footer shows the working tree's diff next to the session clock only while there is one: `\uf4d2 +500 -100`, with the same green/red the diff body uses (`toolDiffAdded`/`toolDiffRemoved`). It comes from `git diff HEAD --numstat` in the existing `refreshGit` pass (one extra spawn, nothing new on the per-frame path) and is hidden when the tree is clean or git cannot answer (not a repo, no binary) — no fake `+0 -0`, no dangling icon. A repo with an unborn `HEAD` (no commit yet) falls back to diffing against the empty tree, so a staged first commit still counts. Note the count is the whole uncommitted change, not just this session's edits: it includes anything done before or outside the session. Untracked files are not part of a diff and stay counted by the `~N` worktree segment (`extensions/footer.ts`).
+
+### Fixed
+- The footer's session clock no longer restarts when you switch sessions with `/resume`. The reset came from `session_start`, which set the start to `Date.now()` for every reason but `"reload"`. It now reads the session header's `timestamp` (the moment the session was created) for `"resume"`/`"startup"`, so the duration reflects the session's real age; `"new"`/`"fork"` write a fresh header and still start at zero. A missing or unparsable header falls back to now (`extensions/footer.ts`).
+- Every third-party tool that ships its own `renderCall`/`renderResult` now renders in the arnative box, not just the ones on a hardcoded name list. The predicate was an allowlist (`ls`, `powershell`, `ffgrep`, `fffind`, `find`, `grep`, `BOXED_TOOLS`, MCP), so any other extension's tool — present or future — fell through to pi's `contentBox`: a padded background block with no border. It is now shape-based: boxed unless the tool declares `renderShell: "self"` (arnative's own boxes) or is in `NOT_BOXED` (`extensions/tools.ts`).
+- The inter-box gap is now stripped for the generic box path too. The gap-strip patch keyed on a name list (`OWN_BOX`), so a tool boxed by the widened predicate but not registered through `minimal()`/`BOXED_TOOLS` kept the leading `Spacer` and floated a blank row above its box (`extensions/tools.ts`).
+
+## [0.3.7] - 2026-10-05
+
+### Fixed
+- The `powershell` and `ls` tools now render in the arnative box like every other call. pi draws a tool that ships its own `renderCall`/`renderResult` but sets no `renderShell` through its own `contentBox` — a background block with no border. Both tools match that shape (their definitions spread the renderer functions onto the tool object), so they now go through `minimal()` like `bash`/`grep`/`read`. `powershell` is Windows-only: `getPowerShellConfig()` throws elsewhere, and re-registering it would activate a tool the builtin deliberately leaves off (`extensions/tools.ts`).
+- pi-fff's default-mode tools `ffgrep` and `fffind` are boxed too. The box path only recognized the override names `find`/`grep`, which pi-fff uses when it replaces pi's builtins, so a session running it in default mode showed those calls as bare blocks (`extensions/tools.ts`).
+- pi's `[Extension issues]` diagnostic now draws in the arnative box, stripped to two content rows: the title (warning glyph + section name, brackets dropped, `[ctrl+o to expand]` hint) over the extension's source path. pi's extra diagnostic lines are dropped — the expand hint stands in for them (`extensions/section-headers.ts`).
+- The package-update box title lost the double space after its glyph (`extensions/ui-render-tweaks.ts`).
+
+### Changed
+- The `section-headers.ts` self-check moved out of the extension into `test/section-headers.test.ts` and runs under `node --test`, so `npm test` lists it as a test instead of a script that prints `OK`. This is a pilot for the other modules; the remaining in-file self-checks follow the same layout once proven (`extensions/section-headers.ts`, `test/section-headers.test.ts`, `package.json`).
+
 ## [0.3.6] - 2026-10-05
 
 ### Fixed

package/README.md

@@ -98,6 +98,8 @@ npm test        # Run self-checks & theme generator check
 npm run themes  # Rebuild all theme variants
 ```
 
+**CI:** `.github/workflows/ci.yml` runs `npm install` + `npm test` on every push to `main` and every pull request, against Node 22.x (the floor the peer packages require) and 24.x (current LTS). No lockfile is committed, so the workflow installs without a cache key. The suite must pass in a clean checkout — a test that reads `process.cwd()` or a machine-local path will fail there.
+
 **Releasing:** `scripts/release-check.mjs` is a read-only preflight (never writes, tags, pushes or publishes) with one stage per step of the release:
 
 | Stage | Run before | Checks |

package/extensions/arnative.ts

@@ -358,6 +358,8 @@ if (isMain(import.meta.url)) {
 	const footerLines = footerPicker.render(50);
 	assert(footerLines.some((l) => l.includes("Arnative · Footers")), "footers picker title");
 	assert(footerLines.some((l) => l.includes("● Pi (system)")), "footers picker marks the active preset");
+	assert(footerLines.some((l) => l.includes("Arnative (Minimal)")), "footers picker lists the minimal preset");
+	assert(footerLines.filter((l) => l.includes("Arnative (")).length === 2, "both arnative footer presets are selectable");
 
 	console.log("arnative.ts self-check OK");
 }

package/extensions/footer.ts

@@ -11,7 +11,7 @@
  */
 import type { AssistantMessage } from "@earendil-works/pi-ai";
 import { CustomEditor, FooterComponent, type ExtensionAPI, type ExtensionContext } from "@earendil-works/pi-coding-agent";
-import { truncateToWidth, visibleWidth } from "@earendil-works/pi-tui";
+import { truncateToWidth, visibleWidth, type TuiMouseEvent } from "@earendil-works/pi-tui";
 import { execFile } from "node:child_process";
 import { mkdirSync, mkdtempSync, readFileSync, rmSync, statSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
@@ -19,7 +19,7 @@ import { dirname, join, resolve } from "node:path";
 import { stripAnsi } from "../lib/ansi.ts";
 import { assert, isMain } from "../lib/check.ts";
 import { capitalize, markedLabels, modelDisplayParts } from "../lib/format.ts";
-import { IS_WINDOWS_LIKE, SHORTCUT_RELOAD } from "../lib/shortcuts.ts";
+import { IS_WINDOWS_LIKE, SHORTCUT_RELOAD, SHORTCUT_TOGGLE_FOOTER } from "../lib/shortcuts.ts";
 import { formatTokens } from "../lib/usage-store.ts";
 import { FOOTER_PRESET_KEY as FOOTER_PRESET_CONFIG_KEY, loadChoice, resetPresetCache, saveChoice } from "../lib/preset-store.ts";
 
@@ -32,9 +32,21 @@ import { FOOTER_PRESET_KEY as FOOTER_PRESET_CONFIG_KEY, loadChoice, resetPresetC
 // in both — pi's `setEditorComponent()` would close the picker mid-preview). The picker
 // lives in arnative.ts, so the choice is shared through globalThis like the header preset.
 // Listed built-in-first: the `(Full)` suffix is rendered dim, so the plain row reads cleaner on top.
-export const FOOTER_PRESETS = ["Pi (system)", "Arnative (Full)"] as const;
+export const FOOTER_PRESETS = ["Pi (system)", "Arnative (Full)", "Arnative (Minimal)"] as const;
 export type FooterPreset = (typeof FOOTER_PRESETS)[number];
 export const DEFAULT_FOOTER_PRESET: FooterPreset = "Arnative (Full)";
+/** Presets that this extension's registrar draws (everything but pi's built-in footer). */
+const ARNATIVE_PRESETS: readonly FooterPreset[] = ["Arnative (Full)", "Arnative (Minimal)"];
+/** Collapse state of the minimal footer: one flag per side, so a click on the left
+ *  head does not open the right metrics. Session-scoped, not persisted. */
+const MINIMAL_EXPAND_KEY = Symbol.for("pi-arnative.minimalFooter.expanded");
+export function minimalExpanded(): { left: boolean; right: boolean } {
+	const slot = (globalThis as Record<symbol, { left: boolean; right: boolean } | undefined>)[MINIMAL_EXPAND_KEY];
+	if (slot) return slot;
+	const fresh = { left: false, right: false };
+	(globalThis as Record<symbol, { left: boolean; right: boolean }>)[MINIMAL_EXPAND_KEY] = fresh;
+	return fresh;
+}
 const FOOTER_PRESET_KEY = Symbol.for("pi-arnative.footerPreset");
 const FOOTER_REGISTRAR_KEY = Symbol.for("pi-arnative.footerRegistrar");
 
@@ -56,7 +68,6 @@ export function applyFooterPreset(ctx: ExtensionContext): void {
 	const register = (globalThis as Record<symbol, ((ctx: ExtensionContext) => void) | undefined>)[FOOTER_REGISTRAR_KEY];
 	register?.(ctx);
 }
-
 /** Select a preset and apply it immediately (the footer swaps on the spot). */
 export function setFooterPreset(name: FooterPreset, ctx: ExtensionContext): void {
 	(globalThis as Record<symbol, unknown>)[FOOTER_PRESET_KEY] = name;
@@ -84,6 +95,10 @@ type GitInfo = {
 	/** Absent (null) when git could not answer — never a false "-"/"clean". */
 	tag: string | null;
 	uncommitted: number | null;
+	/** Working tree vs HEAD, tracked files only (an untracked file is not a diff).
+	 *  Null when git could not answer. */
+	added: number | null;
+	removed: number | null;
 	ahead: number;
 	behind: number;
 } | null;
@@ -94,6 +109,9 @@ let currentThinkingLevel: string | undefined = undefined;
 let currentModel: { id: string; name?: string; provider?: string } | undefined = undefined;
 let rerender: (() => void) | null = null;
 let lastPokeMs = 0;
+/** Column ranges of the minimal footer's disclosure markers, recorded per render
+ *  (mouse coordinates are component-local, so they must come from the same frame). */
+let minimalRanges: { left: [number, number]; right: [number, number] } | null = null;
 
 const SESSION_START_KEY = Symbol.for("pi-arnative.sessionStartMs");
 let sessionStartMs: number = (globalThis as Record<symbol, number>)[SESSION_START_KEY] || Date.now();
@@ -176,7 +194,11 @@ let gitRefreshQueuedCwd: string | null = null;
  * (no binary, or a repo the user does not own) are omitted, so a fresh user still sees
  * the branch but never a fabricated "-" or "clean".
  */
-export function gitParts(info: GitInfo, acc: (t: string) => string, soft: (t: string) => string): string[] {
+export function gitParts(
+	info: GitInfo,
+	acc: (t: string) => string,
+	soft: (t: string) => string,
+): string[] {
 	if (!info) return [];
 	const branch = `${acc("\uf126")} ${soft(info.branch)}${info.ahead > 0 ? ` ${soft(`↑${info.ahead}`)}` : ""}${info.behind > 0 ? ` ${soft(`↓${info.behind}`)}` : ""}`;
 	const parts = [branch];
@@ -185,6 +207,23 @@ export function gitParts(info: GitInfo, acc: (t: string) => string, soft: (t: st
 	return parts;
 }
 
+/**
+ * Working-tree diff vs HEAD as `+500 -100`, or `""` when there is nothing to show:
+ * a clean tree, or git cannot answer (not a repo, no binary, unreadable HEAD). This is
+ * the whole uncommitted change, not just what this session touched — a snapshot at
+ * session start would be needed for that, which is not worth the cost here.
+ */
+export function formatDiffStat(
+	info: GitInfo,
+	acc: (t: string) => string,
+	pos: (t: string) => string,
+	neg: (t: string) => string,
+): string {
+	if (!info || info.added === null || info.removed === null) return "";
+	if (info.added + info.removed === 0) return "";
+	return `${acc("\uf4d2")} ${pos(`+${info.added}`)} ${neg(`-${info.removed}`)}`;
+}
+
 async function refreshGit(cwd: string): Promise<void> {
 	if (gitRefreshInFlight) {
 		gitRefreshQueuedCwd = cwd;
@@ -199,19 +238,26 @@ async function refreshGit(cwd: string): Promise<void> {
 			(globalThis as Record<symbol, any>)[GIT_KEY] = null;
 			return;
 		}
-		const [tag, status, sync] = await Promise.all([
+		const [tag, status, sync, numstat] = await Promise.all([
 			run("git", ["describe", "--tags", "--abbrev=0"], cwd),
 			run("git", ["status", "--porcelain"], cwd),
 			run("git", ["rev-list", "--left-right", "--count", "@{u}...HEAD"], cwd),
+			run("git", ["diff", "HEAD", "--numstat"], cwd),
 		]);
 		const uncommitted = status.ok ? status.out.split("\n").filter((l) => l.trim().length > 0).length : null;
 		const parts = sync.ok && sync.out !== "" ? sync.out.split(/\s+/).map(Number) : [];
 		const behind = parts.length > 0 && parts[0] > 0 ? parts[0] : 0;
 		const ahead = parts.length > 1 && parts[1] > 0 ? parts[1] : 0;
+		// An unborn HEAD (a repo with no commit yet) makes `diff HEAD` fail; compare the
+		// working tree against the canonical empty tree so a staged first commit still counts.
+		const numstat2 = numstat.ok ? numstat : await run("git", ["diff", "--numstat", "4b825dc642cb6eb9a060e54bf8d69288fbee4904"], cwd);
+		const { added, removed } = sumNumstat(numstat2.out);
 		git = {
 			branch,
 			tag: tag.ok && tag.out ? tag.out : null,
 			uncommitted,
+			added: numstat2.ok ? added : null,
+			removed: numstat2.ok ? removed : null,
 			ahead,
 			behind,
 		};
@@ -226,6 +272,43 @@ async function refreshGit(cwd: string): Promise<void> {
 	}
 }
 
+/**
+ * Head row for the minimal footer, plus the exact click zones (0-based, inclusive
+ * columns) of its two heads. Placing the row here instead of guessing at `pairRow`'s
+ * arithmetic is deliberate: `pairRow` truncates with an ellipsis, which shifts columns,
+ * and a mirror of that logic silently drifts at narrow widths — the bug where a click on
+ * blank space toggled a side. Measuring the placed cells cannot drift. The blank gap and
+ * any padding stay inert; a head the row cannot fit gets `[-1, -1]`.
+ */
+export function placeMinimalHead(
+	left: string,
+	right: string,
+	width: number,
+): { row: string; left: [number, number]; right: [number, number] } {
+	const lw = visibleWidth(left);
+	const rw = visibleWidth(right);
+	if (lw + 1 + rw <= width) {
+		const rightStart = width - rw;
+		return {
+			row: left + " ".repeat(rightStart - lw) + right,
+			left: lw > 0 ? [0, lw - 1] : [-1, -1],
+			right: rw > 0 ? [rightStart, width - 1] : [-1, -1],
+		};
+	}
+	// Too wide: the left head is cut, the right one keeps its edge column like pairRow.
+	const room = Math.max(1, width - Math.min(rw, width) - 1);
+	const lCut = truncateToWidth(left, room);
+	const rCut = truncateToWidth(right, width - room - 1);
+	const lCutW = visibleWidth(lCut);
+	const rCutW = visibleWidth(rCut);
+	const rightStart = lCutW + 1;
+	return {
+		row: lCut + " " + rCut,
+		left: lCutW > 0 ? [0, lCutW - 1] : [-1, -1],
+		right: rCutW > 0 ? [rightStart, rightStart + rCutW - 1] : [-1, -1],
+	};
+}
+
 function formatModelName(
 	model: { id: string; name?: string; provider?: string } | undefined,
 	thinkingLevel: string | undefined,
@@ -237,7 +320,7 @@ function formatModelName(
 	const { name, provider } = modelDisplayParts({ id: model.id, name: model.name, provider: model.provider });
 	const provStr = provider ? ` ${dim(`(${provider})`)}` : "";
 	const capThinking = thinkingLevel && thinkingLevel !== "off" ? capitalize(thinkingLevel) : "";
-	const thinkStr = capThinking ? `${acc("\udb80\udf35")} ${soft(capThinking)} ${dim("·")} ` : "";
+	const thinkStr = capThinking ? `${acc("\uee9c")} ${soft(capThinking)} ${dim("·")} ` : "";
 	return `${thinkStr}${acc(name)}${provStr}`;
 }
 
@@ -254,6 +337,30 @@ export function shortenCwd(raw: string): string {
 	return head + parts.slice(-2).join("/");
 }
 
+// Session age must survive `/resume`: the header holds the moment the session was
+// created, so reopening it shows the real age instead of restarting at 0. `new` and
+// `fork` write a fresh header, so the same read yields "just now" there. `/reload`
+// keeps the in-memory value (the header would be identical anyway).
+export function resolveSessionStartMs(reason: string, headerTimestamp: string | undefined, now: number, prev: number): number {
+	if (reason === "reload") return prev;
+	const parsed = headerTimestamp ? Date.parse(headerTimestamp) : NaN;
+	return Number.isFinite(parsed) && parsed > 0 && parsed <= now ? parsed : now;
+}
+
+/** Sum `git diff --numstat` output: `added<TAB>removed<TAB>path` per line. A `-`
+ * marks a binary file — no line count, so it is skipped, not counted as 0. */
+export function sumNumstat(out: string): { added: number; removed: number } {
+	let added = 0;
+	let removed = 0;
+	for (const line of out.split("\n")) {
+		const [a, r] = line.split("\t");
+		if (!/^\d+$/.test(a ?? "") || !/^\d+$/.test(r ?? "")) continue;
+		added += Number(a);
+		removed += Number(r);
+	}
+	return { added, removed };
+}
+
 export function formatDuration(ms: number): string {
 	if (!ms || ms <= 0) return "-";
 	const totalSec = Math.floor(ms / 1000);
@@ -516,6 +623,20 @@ export function getToolWorkingMessage(toolName: string, args?: any): string {
 export default function (pi: ExtensionAPI) {
 	let runtimeGen = 0;
 
+	// Keyboard path for the minimal footer's disclosure: mouse clicks only exist in
+	// fullscreen mode, so pi's own rule ("always design a keyboard path") needs this.
+	// One key toggles both sides: expand all unless everything is already open.
+	const toggleMinimal = () => {
+		const state = minimalExpanded();
+		const allOpen = state.left && state.right;
+		state.left = !allOpen;
+		state.right = !allOpen;
+		poke();
+	};
+	for (const key of SHORTCUT_TOGGLE_FOOTER.keys) {
+		pi.registerShortcut(key, { description: "Expand/collapse the minimal footer", handler: toggleMinimal });
+	}
+
 	// Assistant speed timer (user message timestamps come from extensions/timestamps.ts)
 	pi.on("message_start", async (event) => {
 		if (event.message.role === "assistant") {
@@ -529,7 +650,7 @@ export default function (pi: ExtensionAPI) {
 		// ctx.ui closures) may still be alive; clear it before starting a new one.
 		stopWorkingClock();
 		if (event.reason !== "reload") {
-			sessionStartMs = Date.now();
+			sessionStartMs = resolveSessionStartMs(event.reason, ctx.sessionManager.getHeader()?.timestamp, Date.now(), sessionStartMs);
 			(globalThis as Record<symbol, number>)[SESSION_START_KEY] = sessionStartMs;
 			currentModel = ctx.model;
 			currentThinkingLevel = ctx.thinkingLevel;
@@ -578,16 +699,32 @@ export default function (pi: ExtensionAPI) {
 								// fallback
 							}
 							const soft = (text: string) => fgAny(softName, text);
+							// Diff colors: the same theme slots pi uses for a diff body, with the
+							// plain soft color as fallback for a theme that omits them.
+							const colorOr = (name: string, fallback: (t: string) => string) => (t: string) => {
+								try {
+									return fgAny(name, t);
+								} catch {
+									return fallback(t);
+								}
+							};
+							const diffAdd = colorOr("toolDiffAdded", soft);
+							const diffDel = colorOr("toolDiffRemoved", soft);
 							const sep = dim(" | ");
 
 							const durationStr = formatDuration(Date.now() - sessionStartMs);
-							const pDuration = `${acc("\uf017")} ${soft(durationStr)}`;
+							const diffStat = formatDiffStat(git, acc, diffAdd, diffDel);
+							const pDuration = `${acc("\uf017")} ${soft(durationStr)}${diffStat ? `${sep}${diffStat}` : ""}`;
 							const cwdShort = shortenCwd(cwd);
+							const isMinimal = activeFooterPreset() === "Arnative (Minimal)";
+							const open = minimalExpanded();
+							const arrow = (expanded: boolean) => dim(expanded ? " ▼" : " ▲");
 							let left1 = `${acc("\uf07b")} ${dim(cwdShort)}${sep}${pDuration}`;
 							if (git) {
 								left1 = `${acc("\uf07b")} ${dim(cwdShort)}${sep}${pDuration}${gitParts(git, acc, soft).map((p) => `${sep}${p}`).join("")}`;
 							}
-							const right1 = formatModelName(currentModel, currentThinkingLevel, acc, soft, dim);
+							if (isMinimal) left1 += arrow(open.left);
+							const right1 = formatModelName(currentModel, currentThinkingLevel, acc, soft, dim) + (isMinimal ? arrow(open.right) : "");
 
 							const statuses: ReadonlyMap<string, string> = (() => {
 								try {
@@ -613,7 +750,7 @@ export default function (pi: ExtensionAPI) {
 									if (/LITE/i.test(s)) mode = "LITE";
 									else if (/ULTRA/i.test(s)) mode = "ULTRA";
 									else if (/FULL/i.test(s)) mode = "FULL";
-									return `${acc("\uef04")}  ${soft("ponytail:")} ${bullet} ${soft(mode)}`;
+									return `${acc("\uef51")} ${soft("ponytail:")} ${bullet} ${soft(mode)}`;
 								}
 								if (s.includes("jev-eye")) {
 									const isOff = s.includes("OFF") || s.includes("○");
@@ -652,8 +789,36 @@ export default function (pi: ExtensionAPI) {
 							const metricCells = [speedStr, opt ? `${acc("\udb80\udf5b")} ${soft(opt)}` : "", usageStr].filter(Boolean);
 
 							// Fixed 3-row grid: head (cwd/model), then statuses and metrics each
-							// wrapped inside their half of the width — no click toggle.
-							return layoutFooterGrid(left1, right1, groups, metricCells, sep, ` ${dim("·")} `, width);
+							// wrapped inside their half of the width. Arnative (Minimal) starts collapsed
+							// and reveals the same rows on click/keypress, one side at a time.
+							const leftRows = isMinimal && !open.left ? [] : groups;
+							const rightRows = isMinimal && !open.right ? [] : metricCells;
+							if (isMinimal) {
+								// Own head row: it reports the zones measured from what it actually placed.
+								const placed = placeMinimalHead(left1, right1, width);
+								minimalRanges = { left: placed.left, right: placed.right };
+								const rest = layoutFooterGrid("", "", leftRows, rightRows, sep, ` ${dim("·")} `, width).slice(1);
+								return [placed.row, ...rest];
+							}
+							minimalRanges = null;
+							return layoutFooterGrid(left1, right1, leftRows, rightRows, sep, ` ${dim("·")} `, width);
+						},
+						handleMouse(event: TuiMouseEvent) {
+							// Clicks only reach here in fullscreen mode; the keyboard shortcut covers
+							// regular mode. Return handled on press so pi records the target — without
+							// that it never dispatches the follow-up click.
+							if (event.button !== "left" || event.y !== 0) return undefined;
+							const ranges = minimalRanges;
+							if (!ranges) return undefined;
+							const inside = (r: [number, number]) => event.x >= r[0] && event.x <= r[1];
+							const hit = inside(ranges.left) ? "left" : inside(ranges.right) ? "right" : null;
+							if (!hit) return undefined;
+							if (event.type === "press") return { handled: true, render: false };
+							if (event.type !== "click") return undefined;
+							const state = minimalExpanded();
+							state[hit] = !state[hit];
+							poke();
+							return { handled: true, render: true };
 						},
 					};
 				});
@@ -969,16 +1134,53 @@ if (isMain(import.meta.url)) {
 	// rather than the old false "-"/"clean".
 	const A = (t: string) => t;
 	const S = (t: string) => t;
-	const unknownGit = gitParts({ branch: "main", tag: null, uncommitted: null, ahead: 0, behind: 0 }, A, S);
+	const unknownGit = gitParts({ branch: "main", tag: null, uncommitted: null, added: null, removed: null, ahead: 0, behind: 0 }, A, S);
 	assert(unknownGit.length === 1 && unknownGit[0]!.endsWith("main"), "unreadable git: branch only, no fake '-'/'clean'");
-	const fullGit = gitParts({ branch: "main", tag: "v1.2.3", uncommitted: 2, ahead: 1, behind: 0 }, A, S);
+	const fullGit = gitParts({ branch: "main", tag: "v1.2.3", uncommitted: 2, added: 500, removed: 100, ahead: 1, behind: 0 }, A, S);
 	assert(fullGit.length === 3 && fullGit[1]!.endsWith("v1.2.3") && fullGit[2]!.includes("~2"), "readable git: tag + dirty count present");
 	assert(gitParts(null, A, S).length === 0, "not a repo: no git segment at all");
+	// Diff stat is clock-adjacent, so it renders with or without a repo.
+	const dirty = { branch: "main", tag: null, uncommitted: 2, added: 500, removed: 100, ahead: 0, behind: 0 };
+	assert(formatDiffStat(dirty, A, (t) => `<A>${t}</A>`, (t) => `<D>${t}</D>`).includes("<A>+500</A> <D>-100</D>"), "+/- use the diff colors");
+	assert(formatDiffStat({ ...dirty, added: 0, removed: 0 }, A, A, A) === "", "a clean tree hides the diff segment");
+	assert(formatDiffStat(null, A, A, A) === "", "no repo: the diff segment is hidden, never a fake 0");
+	assert(formatDiffStat({ ...dirty, added: null, removed: null }, A, A, A) === "", "git cannot answer: the diff segment is hidden");
+	assert(sumNumstat("12\t3\tsrc/a.ts\n-\t-\timg.png\n4\t0\tb.ts\n").added === 16, "numstat sums added lines and skips binary");
+	assert(sumNumstat("12\t3\tsrc/a.ts\n-\t-\timg.png\n4\t0\tb.ts\n").removed === 3, "numstat sums removed lines and skips binary");
+	assert(sumNumstat("").added === 0 && sumNumstat("").removed === 0, "empty numstat is zero, not NaN")
 
 	// Footer presets: Arnative (Full) is the default; Pi (system) hands the footer slot back to pi (the editor stays arnative).
-	assert(DEFAULT_FOOTER_PRESET === "Arnative (Full)" && FOOTER_PRESETS.length === 2, "Arnative (Full) is the default footer preset");
-	assert(FOOTER_PRESETS.join(" | ") === "Pi (system) | Arnative (Full)", "built-in preset is listed first");
-	assert(markedLabels(FOOTER_PRESETS, "Pi (system)").join(" | ") === "● Pi (system) |   Arnative (Full)", "footer preset labels mark the active one");
+	assert(DEFAULT_FOOTER_PRESET === "Arnative (Full)" && FOOTER_PRESETS.length === 3, "Arnative (Full) is the default footer preset");
+	assert(FOOTER_PRESETS.join(" | ") === "Pi (system) | Arnative (Full) | Arnative (Minimal)", "built-in preset is listed first, minimal last");
+	assert(markedLabels(FOOTER_PRESETS, "Pi (system)").join(" | ") === "● Pi (system) |   Arnative (Full) |   Arnative (Minimal)", "footer preset labels mark the active one");
+	assert(ARNATIVE_PRESETS.join(" | ") === "Arnative (Full) | Arnative (Minimal)", "the arnative-drawn presets exclude pi's built-in footer");
+	// Minimal disclosure: both sides start closed, then toggle independently.
+	delete (globalThis as Record<symbol, unknown>)[MINIMAL_EXPAND_KEY];
+	const expand0 = minimalExpanded();
+	assert(expand0.left === false && expand0.right === false, "the minimal footer starts fully collapsed");
+	expand0.left = true;
+	assert(minimalExpanded().left === true && minimalExpanded().right === false, "one side opens without opening the other");
+	delete (globalThis as Record<symbol, unknown>)[MINIMAL_EXPAND_KEY];
+	// Disclosure arrows: the left zone ends the left head, the right one the terminal edge.
+	const normal = placeMinimalHead("L".repeat(20), "R".repeat(10), 80);
+	assert(visibleWidth(normal.row) === 80, "the head row fills the terminal width");
+	assert(normal.left[0] === 0 && normal.left[1] === 19, "the left zone is exactly the left head text");
+	assert(normal.right[0] === 70 && normal.right[1] === 79, "the right zone is exactly the right head text");
+	assert(50 > normal.left[1] && 50 < normal.right[0], "the gap between the heads is inert");
+	const clipped = placeMinimalHead("L".repeat(70), "R".repeat(20), 80);
+	assert(visibleWidth(clipped.row) <= 80, "a cut head row never exceeds the width");
+	assert(clipped.left[1] < clipped.right[0], "cut zones stay ordered and disjoint");
+	assert(clipped.row.slice(clipped.left[0], clipped.left[1] + 1) === clipped.row.slice(0, clipped.left[1] + 1), "the left zone matches the placed text");
+	// Squeezed: the left head keeps at least one column, the right one is dropped once
+	// nothing is left for it. Both outcomes are measured from the placed row.
+	const squeezed = placeMinimalHead("L".repeat(30), "R".repeat(30), 12);
+	assert(visibleWidth(squeezed.row) <= 12, "a squeezed head row never exceeds the width");
+	assert(squeezed.left[0] === 0, "the left head keeps its columns");
+	assert(squeezed.right[0] === -1 || squeezed.right[0] > squeezed.left[1], "zones stay disjoint when squeezed");
+	const noRight = placeMinimalHead("L", "R".repeat(40), 2);
+	assert(noRight.right[0] === -1, "a right head with no room gets no zone");
+	const onlyLeft = placeMinimalHead("L".repeat(5), "", 20);
+	assert(onlyLeft.left[0] === 0 && onlyLeft.left[1] === 4 && onlyLeft.right[0] === -1, "an empty head has no zone");
 	let footerCleared = 0;
 	let editorTouched = 0;
 	const presetCtx: any = {
@@ -1010,5 +1212,18 @@ if (isMain(import.meta.url)) {
 	else process.env.PI_CODING_AGENT_DIR = prevAgentDir;
 	resetPresetCache();
 
+	// Self-check: formatModelName thinking icon
+	const fmnRes = formatModelName({ id: "claude-3-7-sonnet", name: "Sonnet" }, "low", (s) => s, (s) => s, (s) => s);
+	assert(fmnRes.includes("\uee9c Low"), "formatModelName includes \\uee9c Low thinking indicator");
+
+	// Self-check: session age survives /resume (header timestamp wins over now).
+	const NOW = Date.parse("2026-10-08T12:00:00.000Z");
+	assert(resolveSessionStartMs("resume", "2026-10-08T09:00:00.000Z", NOW, NOW) === Date.parse("2026-10-08T09:00:00.000Z"), "resume restores the session age from the header");
+	assert(resolveSessionStartMs("startup", "2026-10-08T11:59:59.000Z", NOW, NOW) === Date.parse("2026-10-08T11:59:59.000Z"), "startup reads the freshly written header");
+	assert(resolveSessionStartMs("reload", undefined, NOW, 1234) === 1234, "reload keeps the in-memory start");
+	assert(resolveSessionStartMs("resume", undefined, NOW, 1234) === NOW, "a missing header falls back to now");
+	assert(resolveSessionStartMs("resume", "not-a-date", NOW, 1234) === NOW, "an unparsable header falls back to now");
+	assert(resolveSessionStartMs("resume", "2026-10-08T13:00:00.000Z", NOW, 1234) === NOW, "a future header is clamped to now, never a negative age");
+
 	console.log("footer.ts self-check OK");
 }

package/extensions/section-headers.ts

@@ -11,9 +11,9 @@
  * - Box divider '├────┤'. Active tab content is left-aligned, wrapped, soft.
  * - Hides pi's built-in stacked list in loadedResourcesContainer.
  */
-import { existsSync, mkdirSync, mkdtempSync, readFileSync, readdirSync, rmSync, writeFileSync } from "node:fs";
+import { readFileSync, readdirSync } from "node:fs";
 import { fileURLToPath } from "node:url";
-import { tmpdir, userInfo } from "node:os";
+import { userInfo } from "node:os";
 import { join } from "node:path";
 import {
 	UserMessageComponent,
@@ -30,12 +30,12 @@ import {
 	type TuiMouseEvent,
 	type TuiMouseEventResult,
 } from "@earendil-works/pi-tui";
+import type { BoxTheme } from "../lib/box.ts";
 import { renderBoxLines } from "../lib/box.ts";
 import { getActiveTheme, setActiveTheme, stripAnsi, themeOf } from "../lib/ansi.ts";
-import { assert, isMain } from "../lib/check.ts";
-import { capitalize, markedLabels, modelDisplayParts } from "../lib/format.ts";
+import { capitalize, expandKeyName, modelDisplayParts } from "../lib/format.ts";
 import { SHORTCUT_NEW_SESSION, SHORTCUT_NEXT_TAB, SHORTCUT_RELOAD } from "../lib/shortcuts.ts";
-import { HEADER_PRESET_KEY as HEADER_PRESET_CONFIG_KEY, loadChoice, resetPresetCache, saveChoice } from "../lib/preset-store.ts";
+import { HEADER_PRESET_KEY as HEADER_PRESET_CONFIG_KEY, loadChoice, saveChoice } from "../lib/preset-store.ts";
 import { formatTokens, getModelAllTimeUsage } from "../lib/usage-store.ts";
 
 export type TabKey = "Model" | "Directory" | "Context" | "Skills" | "Extensions" | "Shortcut";
@@ -52,6 +52,9 @@ export const SECTION_ICONS: Record<TabKey, string> = {
 	Shortcut: "\uf11c", // nf-fa-keyboard_o
 };
 
+// nf-cod-warning, width 1 — prefixes a pi diagnostic block's compact header.
+const DIAGNOSTIC_ICON = "\uea6c";
+
 export type Themeish = { fg?(color: string, text: string): string; bg?(color: string, text: string): string } | null;
 
 const MODEL_SNAPSHOT_KEY = Symbol.for("pi-arnative.currentModel");
@@ -644,6 +647,30 @@ export class ArnativeHeader implements Component {
 	}
 }
 
+// Arnative box for pi's `[Extension issues]` diagnostic: exactly two rows, the title
+// (warning icon + section name, brackets dropped, + expand hint) over the extension's
+// source path. pi's extra diagnostic lines are dropped — the expand hint stands in for
+// them. Uses only `warning` + `dim`, both defined by every pi theme, so it draws the
+// same under any theme; `rows` is pi's own render output, row 0 being the title.
+export function extensionIssuesLines(th: BoxTheme | null, width: number, rows: string[], hint: string): string[] {
+	const paint = (color: string, text: string) => {
+		try {
+			return th?.fg ? th.fg(color, text) : text;
+		} catch {
+			return text;
+		}
+	};
+	// Rows are plain text: ANSI inside would break the width math of renderBoxLines.
+	const title = stripAnsi(rows[0] ?? "").trim().replace(/^\[(.+)\]$/, "$1");
+	const hintPlain = stripAnsi(hint).trim();
+	const detail = stripAnsi(rows[1] ?? "").trim();
+	const header = `${paint("warning", `${DIAGNOSTIC_ICON} ${title}`)}${hintPlain ? `  ${paint("dim", hintPlain)}` : ""}`;
+	return renderBoxLines(th, width, [header, detail], undefined, "warning");
+}
+
+// Hooks the shared `app.tools.expand` key: `ctrl+o to expand` -> `[ctrl+o to expand]`.
+const wrapHint = (key: string): string => `[${key} to expand]`;
+
 // Intercept loadedResourcesContainer: capture data for the tabs and hide pi's own display
 const SECTION_HEADER_KEY = Symbol.for("pi-arnative.sectionHeadersRewritten");
 if (UserMessageComponent?.prototype && !(globalThis as Record<symbol, boolean>)[SECTION_HEADER_KEY]) {
@@ -687,7 +714,11 @@ if (UserMessageComponent?.prototype && !(globalThis as Record<symbol, boolean>)[
 					const text = themedTextOf(child);
 					if (text.includes("[Extension issues]") && typeof child.render === "function") {
 						const origRender = child.render.bind(child);
-						child.render = (w: number) => renderBoxLines(getActiveTheme<Themeish>(), w, origRender(w), undefined, "warning");
+						// Shortcut hint only when the shortcut registry is reachable; otherwise just the
+						// two-line compact block.
+						const rawHint = wrapHint(expandKeyName());
+						child.render = (w: number) =>
+							extensionIssuesLines(getActiveTheme<BoxTheme>(), w, origRender(w), rawHint);
 						return origAddChild.call(this, child);
 					}
 					// Any other element inside loadedResourcesContainer (e.g. Spacer) is hidden too
@@ -748,346 +779,3 @@ export default function (pi: ExtensionAPI) {
 		}
 	});
 }
-
-// Self-check: `node extensions/section-headers.ts`
-if (isMain(import.meta.url)) {
-
-	assert(sectionNameOf("\x1b[33m[Skills]\x1b[39m\n  a, b") === "Skills", "header [Skills] detected");
-	assert(sectionNameOf("pi v0.87.1") === null, "non-section text is not detected");
-
-	// Header presets: Arnative (Full) is the default; Pi (system) hands the header slot back to pi.
-	assert(DEFAULT_HEADER_PRESET === "Arnative (Full)" && HEADER_PRESETS.length === 2, "Arnative (Full) is the default header preset");
-	assert(HEADER_PRESETS.join(" | ") === "Pi (system) | Arnative (Full)", "built-in preset is listed first");
-	assert(markedLabels(HEADER_PRESETS, "Arnative (Full)").join(" | ") === "  Pi (system) | ● Arnative (Full)", "preset labels mark the active one");
-	// Persisting a preset writes a config file: point it at a fixture so the real agent dir stays clean.
-	const presetFixture = mkdtempSync(join(tmpdir(), "arnative-hdr-"));
-	const prevAgentDir = process.env.PI_CODING_AGENT_DIR;
-	process.env.PI_CODING_AGENT_DIR = presetFixture;
-	resetPresetCache();
-	let installed: unknown = "untouched";
-	const presetCtx: any = { ui: { setHeader: (factory: unknown) => { installed = factory; } } };
-	setHeaderPreset("Pi (system)", presetCtx);
-	assert(installed === undefined && activeHeaderPreset() === "Pi (system)", "Pi (system) clears the custom header");
-	setHeaderPreset("Arnative (Full)", presetCtx);
-	assert(typeof installed === "function" && activeHeaderPreset() === "Arnative (Full)", "Arnative (Full) installs the header factory");
-	// Persist a NON-default value: asserting the default survives proves nothing.
-	setHeaderPreset("Pi (system)", presetCtx);
-	assert(loadChoice(HEADER_PRESET_CONFIG_KEY, HEADER_PRESETS) === "Pi (system)", "the header preset is persisted to disk");
-	delete (globalThis as Record<symbol, unknown>)[HEADER_PRESET_KEY];
-	resetPresetCache();
-	assert(activeHeaderPreset() === "Pi (system)", "a restart reads the non-default preset back from disk");
-	rmSync(presetFixture, { recursive: true, force: true });
-	if (prevAgentDir === undefined) delete process.env.PI_CODING_AGENT_DIR;
-	else process.env.PI_CODING_AGENT_DIR = prevAgentDir;
-	resetPresetCache();
-
-	// Parsing & wrapping
-	assert(extractItemsFromBody("  a, b, c").length === 3, "extractItemsFromBody splits 3 comma items");
-	assert(extractItemsFromBody("  x\n  y\n  z").length === 3, "extractItemsFromBody returns 3 lines");
-
-	// Directory path wrapping: never truncated, always the full path
-	const wp = wrapPath("/run/media/bisma/DATA/Pi/pi-arnative", 20);
-	assert(wp.join("") === "/run/media/bisma/DATA/Pi/pi-arnative", "wrapPath drops no segment");
-	assert(wp.every((l) => visibleWidth(l) <= 20), "wrapPath respects the width");
-	assert(wrapPath("D:\\Pi\\pi-arnative", 80).join("") === "D:/Pi/pi-arnative", "wrapPath normalizes backslashes");
-	assert(wrapPath("/a/very-long-segment-name", 6).every((l) => visibleWidth(l) <= 6), "long segment is sliced");
-
-	const wrapTest = wrapCommaItems(["item1", "item2", "item3", "item4"], 15);
-	assert(wrapTest.length >= 2, "wrapCommaItems wraps into lines");
-	assert(visibleWidth(wrapTest[0]!) <= 15, "wrapped line width within the limit");
-
-	const cLine = centerLine("test", 20);
-	assert(visibleWidth(cLine) === 20, "centerLine is exactly 20 columns");
-	assert(cLine.startsWith("│") && cLine.endsWith("│"), "centerLine starts and ends with │");
-
-	// Model name formatting
-	assert(
-		formatModelDisplayName({ name: "Gemini 3.8 Flash", provider: "antigravity" }) ===
-			"Gemini 3.8 Flash (Antigravity)",
-		"formatModelDisplayName with name and provider",
-	);
-	assert(
-		formatModelDisplayName({ id: "deepseek-v4.1-flash", provider: "yarz" }) ===
-			"Deepseek V4.1 Flash (Yarz)",
-		"formatModelDisplayName converts id to titlecase",
-	);
-
-	// Package & version resolver
-	assert(packageNameOf("@bismawy/pi-agentrouter") === "@bismawy/pi-agentrouter", "plain npm label");
-	assert(packageNameOf("footer.ts") === "footer.ts", "local label");
-
-	const gitTmp = mkdtempSync(join(tmpdir(), "pi-ver-"));
-	mkdirSync(join(gitTmp, "git", "github.com", "test/arnative"), { recursive: true });
-	writeFileSync(join(gitTmp, "git", "github.com", "test/arnative", "package.json"), JSON.stringify({ version: "9.9.9" }));
-	assert(installedVersion("test/arnative", join(gitTmp, "npm"), join(gitTmp, "git")) === "9.9.9", "version from the git clone");
-	rmSync(gitTmp, { recursive: true, force: true });
-
-	// ArnativeHeader component & mouse tab click
-	tabStore.set("Context", ["AGENTS.md"]);
-	tabStore.set("Skills", ["agents-sdk", "cloudflare"]);
-	tabStore.set("Extensions", ["@bismawy/pi-agentrouter@1.6.1", "footer.ts"]);
-
-	let renderRequested = false;
-	const fakeTui: any = {
-		requestRender: () => {
-			renderRequested = true;
-		},
-	};
-	const fakeCtx: any = {
-		model: { name: "Gemini 3.8 Flash", provider: "antigravity" },
-		thinkingLevel: "low",
-		cwd: "/run/media/bisma/DATA/Pi/pi-arnative",
-	};
-
-	// Only "dim" is colored, so the box line colors can be asserted.
-	const ansiTheme = { fg: (c: string, t: string) => (c === "dim" ? `\x1b[2m${t}\x1b[22m` : t) };
-
-	// Counts come from pi's loaded-resource listing. With `quietStartup: true | "header"`
-	// pi skips that listing, so the counts are unknown — shown as "—", never a misleading 0.
-	resourcesListingShown(false);
-	tabStore.set("Context", []);
-	tabStore.set("Skills", []);
-	tabStore.set("Extensions", []);
-	const hdrQuiet = new ArnativeHeader(fakeTui, ansiTheme, fakeCtx);
-	assert(hdrQuiet.getTabsData(120).map((t) => t.label).join("|").includes("Skills [—]"), "hidden listing: counts render as —, not 0");
-	hdrQuiet.activeTab = "Skills";
-	assert(
-		hdrQuiet.render(120).some((l) => stripAnsi(l).includes("(not tracked")),
-		"hidden listing: empty tab explains itself instead of a bare (empty)",
-	);
-	resourcesListingShown(true);
-	assert(new ArnativeHeader(fakeTui, ansiTheme, fakeCtx).getTabsData(120).some((t) => t.label === "Skills [0]"), "shown listing: 0 is a real count again");
-	tabStore.set("Context", ["AGENTS.md"]);
-	tabStore.set("Skills", ["agents-sdk", "cloudflare"]);
-	tabStore.set("Extensions", ["@bismawy/pi-agentrouter@1.6.1", "footer.ts"]);
-	const hdr = new ArnativeHeader(fakeTui, ansiTheme, fakeCtx);
-	assert(hdr.activeTab === "Directory", "default tab is Directory");
-
-	// Default view: the working directory path, shown on open
-	const linesDefault = hdr.render(100);
-	assert(stripAnsi(linesDefault[5]!).includes("/run/media/bisma/DATA/Pi/pi-arnative"), "directory path shown as soon as the first tab opens");
-
-	hdr.activeTab = "Model";
-	const lines100 = hdr.render(100);
-	assert(lines100.length === 7, "render 100 = border + 5 rows + border (left: mark + gap + pi version; info: brand line + tab box + data)");
-	assert(lines100.every((l) => visibleWidth(l) === 100), "every line at width 100 is exactly 100 columns");
-	assert(lines100[0]!.includes("╭") && lines100[0]!.includes("╮"), "round top border");
-	assert(lines100[6]!.includes("╰") && lines100[6]!.includes("╯"), "round bottom border");
-	// The column divider joins the frame: full-height split, no gap at the borders.
-	const jx = stripAnsi(lines100[0]!).indexOf("┬");
-	assert(jx > 0 && stripAnsi(lines100[6]!).indexOf("┴") === jx, "┬/┴ junctions sit on the border at the divider column");
-	assert(
-		lines100.slice(1, 6).every((l) => stripAnsi(l)[jx] === "│"),
-		"the divider sits in the same column on every row",
-	);
-	const logoPlain = buildLogoLines(null);
-	assert(logoPlain.length === 4 && visibleWidth(logoPlain[0]!) === 8, "pi block mark intact (4 lines x 8 columns)");
-	// P (columns 0-2) accent, i (the right bar) soft — the old wordmark's two-tone.
-	const logoTagged = buildLogoLines({
-		fg: (c: string, t: string) => (c === "accent" ? `<A>${t}</A>` : c === "accentSoft" ? `<T>${t}</T>` : t),
-	} as any);
-	assert(logoTagged[0] === "<A>██████</A>  ", "mark top row: P in the theme accent");
-	assert(logoTagged[2] === "<A>████</A>  <T>██</T>", "mark lower rows: P accent + i (right bar) soft");
-	assert(
-		lines100.slice(1, 5).every((l, i) => stripAnsi(l).slice(2, 10) === logoPlain[i]),
-		"pi block mark in the left column, 4 lines",
-	);
-	assert(
-		/Welcome back, \S/.test(stripAnsi(lines100[1]!)) && !stripAnsi(lines100[1]!).includes("pi v"),
-		"line 1: greeting uses the device username",
-	);
-	// Left column: the pi version sits directly under the mark (no gap).
-	assert(stripAnsi(lines100[5]!).slice(2, 12).includes("pi v"), "pi version sits under the mark in the left column");
-	// Info column: brand + greeting, then the tab box straight away (no blank line).
-	assert(/Arnative v\d+\.\d+\.\d+ · Welcome back, \S/.test(stripAnsi(lines100[1]!)), "info line 1: Arnative version · Welcome back, user");
-	assert(!stripAnsi(lines100[2]!).includes("Menu"), "tab box border carries no label");
-	// Tab box separators join its borders: ┬ above and ┴ below each "│". Compare by display
-	// column, not string index: the tab icons are astral (2 code units, 1 column).
-	const atColumn = (line: string, col: number): string | undefined => {
-		let c = 0;
-		for (const ch of stripAnsi(line)) {
-			const w = visibleWidth(ch);
-			if (col >= c && col < c + w) return ch;
-			c += w;
-		}
-		return undefined;
-	};
-	const tabSepCols = [...stripAnsi(lines100[2]!).matchAll(/┬/g)].map((m) => visibleWidth(stripAnsi(lines100[2]!).slice(0, m.index)));
-	assert(tabSepCols.length >= 2, "tab box top border has ┬ at each column divider");
-	assert(
-		tabSepCols.every((c) => atColumn(lines100[3]!, c) === "│" && atColumn(lines100[4]!, c) === "┴"),
-		"each tab divider is full height (┬ / │ / ┴ in the same column)",
-	);
-	assert(!stripAnsi(lines100[1]!).includes("Interrupt"), "shortcut legend no longer on the top line");
-	// Both brand names use the theme accent, not only "pi".
-	const accentTheme = {
-		fg: (c: string, t: string) => (c === "accent" ? `<A>${t}</A>` : c === "dim" ? `<D>${t}</D>` : t),
-	};
-	const accentHeader = new ArnativeHeader(fakeTui, accentTheme as any, fakeCtx);
-	// The marker theme returns literal "<A>" text (not ANSI), so the measured width is
-	// inflated — render wide enough that the brand/greeting line is not truncated.
-	const accentLines = accentHeader.render(140);
-	assert(accentLines[5]!.includes("<A>\x1b[1mpi"), "the pi name uses the theme accent");
-	assert(accentLines[1]!.includes("<A>\x1b[1mArnative"), "the Arnative name uses the same theme accent");
-	const greetLine = accentHeader.render(140)[1]!;
-	assert(greetLine.includes("<D>Welcome back,"), "greeting text uses the dim color");
-	assert(greetLine.includes(`<A>\x1b[1m${capitalize(deviceUser())}`), "device username uses the theme accent, first letter capitalized");
-	assert(stripAnsi(lines100[3]!).includes("Model [Gemini 3.8 Flash]"), "Model tab in the menu (Model [name])");
-	// Directory tab: name only in the menu, whole path in the content line below
-	hdr.activeTab = "Directory";
-	hdr.render(100);
-	assert(stripAnsi(lines100[3]!).includes("Directory"), "Directory tab shown in the menu");
-	assert(!stripAnsi(lines100[3]!).includes("Directory: /"), "Directory menu entry has no path (detail only on click)");
-	assert(stripAnsi(lines100[3]!).indexOf("Directory") < stripAnsi(lines100[3]!).indexOf("Model ["), "Directory tab is leftmost");
-	hdr.activeTab = "Model";
-	hdr.render(100);
-	assert(stripAnsi(lines100[5]!).includes("Gemini 3.8 Flash (Antigravity)"), "active Model tab data on the last info line");
-	// Directory tab content: full path, wrapped, nothing lost
-	hdr.activeTab = "Directory";
-	const linesDir = hdr.render(100);
-	const dirContent = stripAnsi(linesDir[5]!);
-	assert(dirContent.includes("/run/media/bisma"), "full Directory tab path (start) shown");
-	assert(dirContent.includes("pi-arnative"), "full Directory tab path (end) shown — no truncation");
-	hdr.activeTab = "Model";
-	hdr.render(100);
-	// Tab box borders are a single fully dimmed block (the "─"/"┬"/"┴" runs are colored too).
-	assert((lines100[4]!.match(/\x1b\[/g) ?? []).length === 8, "bottom box line (row 4) = one fully dimmed block");
-	assert(
-		lines100[2]!.includes("\x1b[2m╭─") && lines100[2]!.includes("┬") && lines100[2]!.includes("╮\x1b[22m"),
-		"tab box top border = one dim block carrying the ┬ junctions",
-	);
-
-	// Tab regions: Model, Context, Skills, Extensions
-	const tabY = (hdr as any).renderedTabLineY;
-	assert(tabY > 0, "tab line Y position recorded");
-	const tabRegions = (hdr as any).renderedTabRegions as Array<{ key: TabKey; startX: number; endX: number }>;
-	assert(tabRegions.length >= 3, "tab regions filled (right tabs may be cut at 100 columns)");
-	assert(tabRegions.some((r) => r.key === "Directory"), "Directory tab present in the menu (width 100)");
-
-	// Click the Extensions tab (when truncated at 100 columns, use the rightmost tab)
-	const extRegion = tabRegions.find((r) => r.key === "Extensions") ?? tabRegions[tabRegions.length - 1]!;
-	const clickExt = hdr.handleMouse({
-		type: "click",
-		button: "left",
-		x: Math.floor((extRegion.startX + extRegion.endX) / 2),
-		y: tabY,
-	} as any);
-	assert(clickExt?.handled === true, "mouse click on the tab is handled");
-	assert(hdr.activeTab === extRegion.key, "activeTab switches to the clicked tab");
-	assert(renderRequested === true, "requestRender called when the tab switches");
-
-	const linesExt = hdr.render(100);
-	assert(stripAnsi(linesExt[5]!).includes(extRegion.key === "Extensions" ? "@bismawy/pi-agentrouter@1.6.1" : ""), "active tab data on the last info line");
-
-	// Click the Skills tab (measured at a width where every tab fits)
-	const wide = hdr.render(140);
-	const wideRegions = (hdr as any).renderedTabRegions as Array<{ key: TabKey; startX: number; endX: number }>;
-	const skillsRegion = wideRegions.find((r) => r.key === "Skills") ?? wideRegions[wideRegions.length - 1]!;
-	hdr.activeTab = "Model";
-	hdr.render(140);
-	const skillsY = (hdr as any).renderedTabLineY;
-	hdr.handleMouse({
-		type: "click",
-		button: "left",
-		x: Math.floor((skillsRegion.startX + skillsRegion.endX) / 2),
-		y: skillsY,
-	} as any);
-	assert(hdr.activeTab === skillsRegion.key, `activeTab switches to ${skillsRegion.key} after the click`);
-
-	const linesSkills = hdr.render(140);
-	assert(wide.every((l) => visibleWidth(l) === 140), "lines at 140 columns stay 140");
-	assert(wideRegions.length === 6, "width 140: all 6 tabs shown (Shortcut included)");
-	if (skillsRegion.key === "Skills") {
-		assert(linesSkills[5]!.includes("agents-sdk, cloudflare"), "Skills tab data on the last info line");
-		assert(!linesSkills[5]!.includes("Gemini 3.8 Flash (Antigravity)"), "Model data not shown in the Skills tab content");
-		assert(linesSkills[3]!.includes("Model [Gemini 3.8 Flash]"), "tab menu still shown while another tab is active");
-	}
-
-	// The Y of a tab must point at the tab menu line, not the line above
-	const rowsPlain = lines100.map((l) => stripAnsi(l));
-	assert(rowsPlain[tabY]!.includes("Model ["), "renderedTabLineY points at the tab menu line");
-	assert(tabRegions.every((r) => rowsPlain[tabY]!.slice(r.startX, r.endX + 1).includes(r.key === "Model" ? "Model [" : r.key)), "tab regions align with the text on that line");
-
-	// Shortcut tab: the key cheatsheet lives in the tab content, not in a header line
-	hdr.activeTab = "Shortcut";
-	const linesShortcut = hdr.render(170);
-	assert(
-		((hdr as any).renderedTabRegions as Array<{ key: TabKey }>).some((r) => r.key === "Shortcut"),
-		"width 170: Shortcut tab shown (not dropped)",
-	);
-	const scContent = stripAnsi(linesShortcut.slice(5).join(" "));
-	assert(scContent.includes("[Esc] Interrupt"), "Shortcut tab: legend moved into the content");
-	assert(
-		scContent.includes(`[${SHORTCUT_RELOAD.label}] Reload`),
-		`Shortcut tab: reload shortcut present in the content (${SHORTCUT_RELOAD.label})`,
-	);
-	assert(stripAnsi(linesShortcut[1]!).includes("Arnative v"), "the Arnative brand stays on line 1 while the Shortcut tab is active");
-	hdr.activeTab = "Model";
-	hdr.render(140);
-
-	// Narrow screen: brand lines + tabs + data stay intact, rightmost tabs are dropped
-	const narrow = hdr.render(60);
-	assert(narrow.every((l) => visibleWidth(l) === 60), "lines at 60 columns stay 60");
-	assert(stripAnsi(narrow[1]!).includes("Welcome back,"), "greeting stays intact at 60 columns");
-	assert(stripAnsi(narrow[1]!).includes("Arnative v"), "Arnative brand intact at 60 columns");
-	assert(stripAnsi(narrow[1]!).includes("Interrupt") === false, "legend does not come back to the top line");
-	assert(
-		!((hdr as any).renderedTabRegions as Array<{ key: TabKey }>).some((r) => r.key === "Shortcut"),
-		"rightmost tab dropped at 60 columns",
-	);
-	assert(stripAnsi(narrow[3]!).includes("Model ["), "Model tab still shown at 60 columns");
-
-	// Intercept addChild: capture the data and suppress the child
-	const proto = Object.getPrototypeOf(UserMessageComponent.prototype) as { addChild?: unknown };
-	const container: any = { children: [] as any[], addChild: proto.addChild as (c: any) => unknown };
-	const fakeSection = {
-		getCollapsedText: () => "[Skills]\n  new-skill-1, new-skill-2",
-		getExpandedText: () => "[Skills]\n  new-skill-1, new-skill-2",
-		setText: () => {},
-		render: (_w: number) => ["should be hidden"],
-	};
-	container.addChild(fakeSection);
-	assert(fakeSection.render(80).length === 0, "child of loadedResourcesContainer suppressed (render = [])");
-	assert(tabStore.get("Skills")?.includes("new-skill-1") === true, "new data lands in tabStore");
-
-	// pi 0.99 ExpandableText shape: `build` callback + `state` object, no getters.
-	const modernSection = {
-		state: { expanded: false },
-		build: () => "[Context]\n  AGENTS.md, MEMORY.md",
-		setText: () => {},
-		render: (_w: number) => ["should be hidden"],
-	};
-	container.addChild(modernSection);
-	assert(modernSection.render(80).length === 0, "pi 0.99 section suppressed (build + state shape)");
-	assert(tabStore.get("Context")?.includes("MEMORY.md") === true, "pi 0.99 section data lands in tabStore");
-
-	// Startup help block is not a resource section and must survive untouched.
-	const helpBlock = { build: () => "\u2580\u2580\u2588  v0.99.1\n\u2588\u2580 \u2588 escape interrupt", state: { expanded: false }, render: () => ["help"] };
-	container.addChild(helpBlock);
-	assert(sectionBodyOf(helpBlock) === "\u2580\u2580\u2588  v0.99.1\n\u2588\u2580 \u2588 escape interrupt", "sectionBodyOf reads the build shape");
-	assert(sectionNameOf(sectionBodyOf(helpBlock)!) === null, "startup help block is not a resource section");
-
-	// pi's "[Extension issues]" diagnostic renders while the installing module's theme is
-	// still null (a `/reload` copy never saw session_start). The box must draw uncolored,
-	// not deref null — that crash was reported in the wild.
-	setActiveTheme(null);
-	const issuesChild = {
-		build: () => "[Extension issues]\n  extension \"x\" failed",
-		render: (_w: number) => ["[Extension issues]", '  extension "x" failed'],
-	};
-	container.addChild(issuesChild);
-	const issuesLines = issuesChild.render(80);
-	assert(issuesLines[0]!.startsWith("\u256d") && issuesLines.at(-1)!.startsWith("\u2570"), "extension-issues box renders without a theme");
-
-	// The theme proxy is globalThis-backed, so the copy that owns the addChild patch still
-	// paints once the other copy's session_start has stored the theme.
-	setActiveTheme({ fg: (c: string, t: string) => `<${c}>${t}</${c}>` });
-	const coloredChild = {
-		build: () => "[Extension issues]\n  extension \"x\" failed",
-		render: (_w: number) => ["[Extension issues]", '  extension "x" failed'],
-	};
-	container.addChild(coloredChild);
-	assert(coloredChild.render(80)[0]!.includes("<warning>"), "shared theme proxy colors the box even in the patched copy");
-
-	console.log("section-headers.ts self-check OK");
-}

package/extensions/tools.ts

@@ -24,6 +24,10 @@ import {
 	createEditTool,
 	createFindTool,
 	createGrepTool,
+	createLsTool,
+	createLsToolDefinition,
+	createPowerShellTool,
+	createPowerShellToolDefinition,
 	createReadTool,
 	createWriteTool,
 	getMarkdownTheme,
@@ -206,9 +210,9 @@ export function stripBlankEdges(lines: string[]): string[] {
 }
 
 // The inter-box gap is the built-in Spacer(1) of ToolExecutionComponent, stripped
-// with a prototype render patch (same pattern as the footer patch). Only lines we
-// draw ourselves are stripped: third-party tools use their own "self" shell too, so
-// the marker is a name list (OWN_BOX, filled by minimal() + BOXED_TOOLS), not the shell.
+// with a prototype render patch (same pattern as the footer patch). Own boxes (our
+// minimal() tools + codemode) and every box forced by the generic predicate below
+// are stripped; a box-owning third party left untouched is not.
 const OWN_BOX = new Set<string>();
 const hasOwnRendererDef = (self: { toolDefinition?: { renderCall?: unknown; renderResult?: unknown } }): boolean =>
 	Boolean(self?.toolDefinition?.renderCall || self?.toolDefinition?.renderResult);
@@ -236,30 +240,28 @@ if (ToolExecutionComponent?.prototype?.render && !(globalThis as Record<symbol,
 	const origRender = ToolExecutionComponent.prototype.render;
 	ToolExecutionComponent.prototype.render = function (width: number): string[] {
 		const lines = origRender.call(this, width);
-		const name = (this as { toolName?: string }).toolName ?? "";
-		return OWN_BOX.has(name) || !hasOwnRendererDef(this) ? stripEdgesKeepingImages(lines) : lines;
+		return OWN_BOX.has(this.toolName) || boxTheirRenderers(this) || !hasOwnRendererDef(this)
+			? stripEdgesKeepingImages(lines)
+			: lines;
 	};
 }
 
 // --- Tools without their own renderer: same box, one-line summary ---
 // Without this pi renders them as a plain bg block (name + 10 output lines). Our
 // renderer is installed through the prototype (same pattern as the gap patch) only
-// for tools that define no renderCall/renderResult of their own (our own tools and
-// pi-web-access etc. are untouched).
+// for tools that define no renderCall/renderResult of their own; third-party
+// renderers are handled by the generic predicate below.
 //
-// BOXED_TOOLS exceptions: the pi-web-access tools whose display names that package
-// hardcodes as "search "/"fetch "/"get_content " — their renderer content is used
-// as-is and only wrapped in our box. `source_check` is deliberately NOT boxed: its
-// partial phase (curator: URLs + approval state) would disappear because our partial
-// path returns nothing. The `todo` tool (@juicesharp/rpiv-todo) is boxed with a
-// check-square icon (\uf14a) and the toolPendingBg/toolSuccessBg backgrounds.
+// BOXED_TOOLS is now a display-name map (plus the todo/cdp special render paths);
+// boxing itself is generic (see boxTheirRenderers). The pi-web-access tools get a
+// friendlier title ("search "/"fetch "/"get_content ") and their renderer content is
+// reused inside our box. `todo` (@juicesharp/rpiv-todo) gets a check-square icon
+// (\uf14a) and the toolPendingBg/toolSuccessBg backgrounds.
 //
-// MCP tools (pi-mcp-adapter) already use the "self" shell, but their result renderer
-// prints the WHOLE output on error (unbounded, outside the box) and their call args
-// can run to dozens of lines: a 1-line summary + [ctrl+o to expand] is tighter while
-// the invoked tool stays visible. pi-fff "override" mode re-registers `find`/`grep`
-// with its own renderers, replacing our boxed versions -> box theirs too (ours carry
-// renderShell "self" = minimal).
+// MCP tools (pi-mcp-adapter) declare `renderShell: "default"`, so the generic
+// predicate boxes them, but their result renderer prints the WHOLE output on error
+// (unbounded) and their call args can run to dozens of lines: a 1-line summary +
+// [ctrl+o to expand] is tighter while the invoked tool stays visible.
 const BOXED_TOOLS = new Map([
 	["web_search", "search"],
 	["fetch_content", "fetch"],
@@ -290,11 +292,20 @@ for (const n of BOXED_TOOLS.keys()) OWN_BOX.add(n);
 
 const MCP_TOOL = (name: string): boolean => name === "mcp" || name === "mcpScript" || name.startsWith("mcp__");
 
-const FFF_OVERRIDE = ["find", "grep"];
-const usesTheirCallRenderer = (self: any): boolean =>
-	BOXED_TOOLS.has(self.toolName) ||
-	MCP_TOOL(self.toolName) ||
-	(FFF_OVERRIDE.includes(self.toolName) && self.toolDefinition?.renderShell !== "self");
+// Generic, not a name list: any tool that ships its own renderers is boxed, unless
+// it declares `renderShell: "self"` (our own `minimal()`/codemode boxes) or sits in
+// NOT_BOXED. That covers every third-party extension without a registration step:
+// pi-fff (`ffgrep`/`fffind`, and `find`/`grep` in its override mode), pi core's
+// `powershell`/`ls` (their definitions spread `renderCall`/`renderResult` onto the
+// tool object) and pi-mcp-adapter (`renderShell: "default"`) all fall in by shape.
+//
+// `source_check` is the single exception: its partial phase (curator URLs + approval
+// state) would disappear because our partial path returns nothing.
+const NOT_BOXED = new Set(["source_check"]);
+const boxTheirRenderers = (self: any): boolean =>
+	Boolean(self.toolDefinition) &&
+	self.toolDefinition.renderShell !== "self" &&
+	!NOT_BOXED.has(self.toolName);
 
 // The MCP box title names the invoked tool (full args when expanded).
 const mcpInfo = (name: string, args: any, th: Theme): string => {
@@ -352,7 +363,11 @@ if (ToolExecutionComponent?.prototype?.render && !(globalThis as Record<symbol,
 	const origCall = proto.getCallRenderer;
 	const origResult = proto.getResultRenderer;
 	const origShell = proto.getRenderShell;
-	const hasOwnRenderer = (self: any): boolean => Boolean(origCall.call(self) || origResult.call(self));
+	const hasOwnRenderer = (self: any): boolean =>
+		!boxTheirRenderers(self) && Boolean(origCall.call(self) || origResult.call(self));
+	// pi-fff splits title (renderCall) and result (renderResult) — both merge into one box.
+	// Its two naming modes are covered: `find`/`grep` (override) and `ffgrep`/`fffind` (default).
+	const FFF_TOOLS = ["find", "grep", "ffgrep", "fffind"];
 
 	// A tool call with no registered definition (e.g. an unknown tool name that
 	// errored with "Tool X not found") makes hasRendererDefinition() false, so the
@@ -366,13 +381,13 @@ if (ToolExecutionComponent?.prototype?.render && !(globalThis as Record<symbol,
 	};
 
 	proto.getRenderShell = function (): string {
-		if (usesTheirCallRenderer(this)) return "self";
+		if (boxTheirRenderers(this)) return "self";
 		return hasOwnRenderer(this) ? origShell.call(this) : "self";
 	};
 
 	proto.getCallRenderer = function () {
 		const name: string = this.toolName;
-		const mine = usesTheirCallRenderer(this);
+		const mine = boxTheirRenderers(this);
 		const own = origCall.call(this);
 		// Chrome DevTools' own renderCall is a bare line; ours replaces it (icon + name).
 		if (own && !mine && !CDP_NAME.has(name)) return own;
@@ -394,10 +409,9 @@ if (ToolExecutionComponent?.prototype?.render && !(globalThis as Record<symbol,
 		const name: string = this.toolName;
 		const own = origResult.call(this);
 		// pi-fff splits title (renderCall) and result (renderResult); both go in one box.
-		const fff = FFF_OVERRIDE.includes(name) && this.toolDefinition?.renderShell !== "self";
+		const fff = FFF_TOOLS.includes(name) && this.toolDefinition?.renderShell !== "self";
 		// MCP: their result renderer is deliberately skipped (unbounded error dump).
-		if (own && !usesTheirCallRenderer(this)) return own;
-		const boxed = BOXED_TOOLS.has(name);
+		if (own && !boxTheirRenderers(this)) return own;
 		// Chrome DevTools are in BOXED_TOOLS for the title/icon only: their renderResult
 		// (raw text, empty when collapsed) would drop our title row, so it is ignored.
 		const cdp = CDP_NAME.has(name);
@@ -408,10 +422,12 @@ if (ToolExecutionComponent?.prototype?.render && !(globalThis as Record<symbol,
 			// pi-web-access reports failure via `details.error` without throwing, so
 			// ctx.isError alone is not enough: without this the error box shows ✓.
 			const isErr = Boolean(ctx.isError || result.isError || (result.details as { error?: unknown } | undefined)?.error);
-			// Errors use our own path: pi-web-access' own error box is a box inside a
-			// box. Their renderer content is only used on success.
+			// Their renderer content is reused inside our box unless it is unusable:
+			// MCP dumps the whole output on error (1-line summary instead), Chrome
+			// DevTools is a bare line, and errors take our own path (pi-web-access'
+			// own error box would nest a box inside a box).
 			const theirs =
-				(boxed || fff) && !isErr && !cdp
+				own && !isErr && !cdp && !MCP_TOOL(name)
 					? safeCall(() => own.call(this, { content: result.content, details: result.details }, opts, th, ctx))
 					: null;
 			return new Lines((width) => {
@@ -660,6 +676,31 @@ function arnativeTools(pi: ExtensionAPI) {
 
 	// Gap is only stripped for our own boxes (see OWN_BOX above).
 
+	// powershell: pi core registers it as a separate builtin (only on Windows); without
+	// our own registration it keeps pi's "default" shell and renders as a bg block, not a
+	// box (the same reason `bash` is re-registered below). Windows only: on any other
+	// platform `getPowerShellConfig()` throws, and an unregistered tool would stay off
+	// while our registration is active on load.
+	if (process.platform === "win32")
+		minimal(
+			pi,
+			createPowerShellTool(cwd),
+			(th) => th.fg("accent", "PS>"),
+			(a, _th, soft, expanded) => {
+				const cmd = (a.command ?? "").replace(/\r?\n/g, " ").trim();
+				return paintLinks(expanded ? cmd : smartTitle(cmd), soft);
+			},
+		);
+
+	// ls: no renderShell of its own, so pi's default shell turns it into a bg block.
+	minimal(
+		pi,
+		createLsTool(cwd),
+		(th) => th.fg("accent", "ls"),
+		(a, _th, soft) => soft(shortPath(a.path ?? ".")),
+		numRes((n) => `${n} entries`),
+	);
+
 	// bash: `󰔟 $ <6 words>…` / `✓ $ <full title when expanded>  0.1s`
 	minimal(
 		pi,
@@ -874,11 +915,18 @@ if (isMain(import.meta.url)) {
 			noDefOut.some((l: string) => l.includes('[ctrl+o to expand]')),
 		"tool without a definition: title + summary + expand hint",
 	);
+	// Generic predicate: any tool that ships its own renderers and no `renderShell: "self"`
+	// is boxed — no name list to maintain for new extensions.
 	const withRenderer = { toolName: "bash", toolDefinition: { renderResult: () => undefined } };
 	assert(ToolExecutionComponent.prototype.getRenderShell.call(noRenderer) === "self", "plain tool -> self shell");
 	assert(
-		ToolExecutionComponent.prototype.getRenderShell.call(withRenderer) === "default",
-		"tool with a renderer -> its own shell (untouched)",
+		ToolExecutionComponent.prototype.getRenderShell.call(withRenderer) === "self",
+		"tool with its own renderer -> boxed too (generic predicate, no name list)",
+	);
+	const extTool = { toolName: "some_ext_tool", toolDefinition: { renderCall: () => undefined } };
+	assert(
+		ToolExecutionComponent.prototype.getRenderShell.call(extTool) === "self",
+		"unknown third-party tool with its own renderer -> boxed",
 	);
 	assert(
 		toolSummary("memory_write", "Appended to MEMORY.md\n\nExisting MEMORY.md preview (1 lines)", {
@@ -997,6 +1045,38 @@ if (isMain(import.meta.url)) {
 	assert(fffOut.some((l: string) => l.includes("*.ts in D:/Pi/x")), "pi-fff find: renderCall title goes in the box");
 	assert(fffOut.some((l: string) => l.includes("6 more lines")), "pi-fff find: renderResult summary goes in the box");
 
+	// pi-fff default mode (`ffgrep`/`fffind`): same renderer shape, different names.
+	for (const n of ["ffgrep", "fffind"]) {
+		const t = { toolName: n, toolDefinition: { renderCall: () => fffCall, renderResult: () => fffRes } };
+		assert(
+			ToolExecutionComponent.prototype.getRenderShell.call(t) === "self",
+			`pi-fff ${n}: shell forced to self (default mode is boxed too)`,
+		);
+		const out = (ToolExecutionComponent.prototype.getResultRenderer.call(t) as any)(
+			{ content: [{ type: "text", text: "src/a.ts\nsrc/b.ts" }] },
+			{ expanded: false, isPartial: false },
+			th,
+			{ args: { pattern: "*.ts" }, state: {} },
+		).render(60);
+		assert(
+			out[0].startsWith("╭") && out.some((l: string) => l.includes("6 more lines")),
+			`pi-fff ${n}: result + summary merged into one box`,
+		);
+	}
+
+	// pi core `powershell` / `ls` spread their renderers into the tool object and set no
+	// renderShell, so pi's "default" shell used to render them as a bare bg block (see the
+	// OWN_BOX block below for the registration half of this fix).
+	for (const [n, t] of [
+		["powershell", createPowerShellToolDefinition(cwd)],
+		["ls", createLsToolDefinition(cwd)],
+	] as const) {
+		assert(
+			ToolExecutionComponent.prototype.getRenderShell.call({ toolName: n, toolDefinition: t }) === "self",
+			`pi core ${n}: shell forced to self (joins our box, not a bg block)`,
+		);
+	}
+
 	const oursFind = { toolName: "find", toolDefinition: { renderShell: "self", renderCall: () => fffCall, renderResult: () => fffRes } };
 	assert(
 		(ToolExecutionComponent.prototype.getCallRenderer.call(oursFind) as any)() === fffCall &&
@@ -1067,7 +1147,35 @@ if (isMain(import.meta.url)) {
 		"OWN_BOX: our own tools + boxed tools registered",
 	);
 	assert(!OWN_BOX.has("mcp") && !OWN_BOX.has("source_check"), "OWN_BOX: third-party renderers not tightened");
-	assert(hasOwnRendererDef(mcpTool) && !hasOwnRendererDef(noRenderer), "tightened benchmark: tool renderer definition");
+	// pi core `powershell`/`ls` spread their renderer functions onto the tool object and
+	// set no renderShell; that is exactly the shape that used to fall into pi's "default"
+	// shell and render as a bare bg block. Registering them through minimal() boxes them.
+	const coreReg: Record<string, any> = {};
+	arnativeTools({ registerTool: (t: any) => { coreReg[t.name] = t; }, registerToolRenderer: () => {}, on: () => {} } as never);
+	for (const [n, t] of [
+		["powershell", createPowerShellToolDefinition(cwd)],
+		["ls", createLsToolDefinition(cwd)],
+	] as const) {
+		const def = t as any;
+		assert(
+			Boolean(def.renderCall) && Boolean(def.renderResult),
+			`pi core ${n}: the definition pi runs spreads its renderers onto the object`,
+		);
+		assert(
+			ToolExecutionComponent.prototype.getRenderShell.call({ toolName: n, toolDefinition: def }) === "self",
+			`pi core ${n}: shell forced to self (joins our box, not a bg block)`,
+		);
+	}
+	// The guard is load-bearing: `getPowerShellConfig()` throws off Windows, and our
+	// registration activates a tool the builtin deliberately leaves off.
+	assert(
+		coreReg.ls?.renderShell === "self",
+		"pi core ls: registered through minimal() (boxing is live)",
+	);
+	assert(
+		process.platform === "win32" ? coreReg.powershell?.renderShell === "self" : coreReg.powershell === undefined,
+		"pi core powershell: registered on Windows only (getPowerShellConfig throws elsewhere)",
+	);	assert(hasOwnRendererDef(mcpTool) && !hasOwnRendererDef(noRenderer), "tightened benchmark: tool renderer definition");
 	assert(
 		capped(th, ["a", "b", "c"], false).length === CALL_ROWS + 1 &&
 			capped(th, ["a", "b", "c"], true).length === 3 &&

package/extensions/ui-render-tweaks.ts

@@ -238,7 +238,7 @@ export class PackageUpdateBoxComponent implements Component {
 		const th = this.th;
 		const fg = (color: string, text: string) => (th ? th.fg(color, text) : text);
 		const rows = [
-			fg("warning", "\x1b[1m\uf449  Package Updates Available\x1b[22m"),
+			fg("warning", "\x1b[1m\uf449 Package Updates Available\x1b[22m"),
 			`${fg("muted", "Package updates are available. Run ")}${fg("accent", "pi update --extensions")}`,
 			fg("muted", "Packages:"),
 			...this.packages.map((pkg) => `- ${pkg}`),

package/lib/shortcuts.ts

@@ -42,6 +42,9 @@ function build(letter: string): Shortcut {
 export const SHORTCUT_NEXT_TAB = build("t");
 export const SHORTCUT_NEW_SESSION = build("n");
 export const SHORTCUT_RELOAD = build("r");
+/** Expand/collapse the minimal footer. Ctrl+alt+m on macOS/Linux, alt+m on Windows/WSL
+ *  (same platform split as the shortcuts above). */
+export const SHORTCUT_TOGGLE_FOOTER = build("m");
 
 if (isMain(import.meta.url)) {
 	assert(SHORTCUT_RELOAD.keys[0] === Key.ctrlAlt("r"), "ctrl+alt+r is registered on every platform");
@@ -63,5 +66,11 @@ if (isMain(import.meta.url)) {
 	assert(!SHORTCUT_RELOAD.matches("\x12"), "bare control char does not match ctrl+alt+r");
 	assert(!SHORTCUT_RELOAD.matches("r"), "bare letter does not match ctrl+alt+r");
 	assert(!SHORTCUT_NEXT_TAB.matches("\x1br"), "alt+r does not match the next-tab shortcut");
+	assert(SHORTCUT_TOGGLE_FOOTER.keys[0] === Key.ctrlAlt("m"), "ctrl+alt+m is registered on every platform");
+	assert(
+		SHORTCUT_TOGGLE_FOOTER.keys.includes(Key.alt("m")) === IS_WINDOWS_LIKE,
+		"alt+m is registered only where Ctrl+Alt is aliased away",
+	);
+	assert(!SHORTCUT_TOGGLE_FOOTER.matches("\x1b\x12"), "ctrl+alt+r does not match the footer toggle");
 	console.log("shortcuts.ts self-check OK");
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bismawy/pi-arnative",
-  "version": "0.3.6",
+  "version": "0.3.8",
   "description": "Refined aesthetics. Cohesive tools. Built for Pi. Warm themes, rounded tool boxes, tabbed header, compact footer and a /usage dashboard.",
   "keywords": [
     "pi-package",
@@ -41,7 +41,7 @@
   },
   "scripts": {
     "dev": "pi --extension ./extensions/tools.ts --extension ./extensions/section-headers.ts --extension ./extensions/footer.ts --extension ./extensions/timestamps.ts --extension ./extensions/ui-render-tweaks.ts --extension ./extensions/usage.ts --extension ./extensions/arnative.ts",
-    "test": "node lib/box.ts && node lib/ansi.ts && node lib/color.ts && node lib/shortcuts.ts && node lib/preset-store.ts && node themes/gen-themes.mjs --check && node extensions/tools.ts && node extensions/footer.ts && node extensions/timestamps.ts && node extensions/section-headers.ts && node extensions/ui-render-tweaks.ts && node extensions/usage.ts && node extensions/arnative.ts",
+    "test": "node lib/box.ts && node lib/ansi.ts && node lib/color.ts && node lib/shortcuts.ts && node lib/preset-store.ts && node themes/gen-themes.mjs --check && node extensions/tools.ts && node extensions/footer.ts && node extensions/timestamps.ts && node extensions/ui-render-tweaks.ts && node extensions/usage.ts && node extensions/arnative.ts && node --test",
     "themes": "node themes/gen-themes.mjs",
     "release:check": "node scripts/release-check.mjs pre-tag",
     "prepublishOnly": "npm test && node scripts/release-check.mjs pre-publish"