@hheei/pi-ext-ui 0.8.1 → 0.8.3

package/dist/extension.js

@@ -14,6 +14,9 @@ export default function (pi) {
         key: "@hheei/pi-ext-ui",
         async start(context) {
             let patch;
+            // The host owns this setting; a message view that draws its own shell applies the same
+            // value the host applies to its own rows, including the default when it is unset.
+            const outputPad = () => (context.pi.getSettings().outputPad === 0 ? 0 : 1);
             const apply = (enabled) => {
                 if (context.extension.mode !== "tui")
                     return;
@@ -26,7 +29,7 @@ export default function (pi) {
                 if (patch !== undefined)
                     return;
                 try {
-                    patch = installHostToolGroups(context.extension, () => runtime.backgrounds);
+                    patch = installHostToolGroups(context.extension, () => runtime.backgrounds, outputPad);
                     redrawGroups = patch.requestRender;
                 }
                 catch (error) {
@@ -70,7 +73,7 @@ export default function (pi) {
             });
             const toolRenderers = registerExtensionHook(pi, toolRendererPoint, createOptionalToolRendererFactory(pi, runtime));
             context.resources.add("tool-renderer", () => toolRenderers.dispose());
-            const handle = registerExtensionHook(context.pi, messageViewPoint, createMessageView);
+            const handle = registerExtensionHook(context.pi, messageViewPoint, (spec) => createMessageView({ ...spec, outputPad: spec.outputPad ?? outputPad() }));
             context.resources.add("message-view", () => handle.dispose());
             try {
                 const state = await provider.storage.load({

package/dist/host-transcript-patch.d.ts

@@ -1,6 +1,6 @@
 import { type ExtensionContext } from "@earendil-works/pi-coding-agent";
 /** Private to pi-ext-ui: two process hooks, scoped by the actual host UI facade. */
-export declare function installHostToolGroups(context: ExtensionContext, background: () => boolean): {
+export declare function installHostToolGroups(context: ExtensionContext, background: () => boolean, outputPad: () => number): {
     dispose(): void;
     requestRender(): void;
     beginRun(): void;

package/dist/host-transcript-patch.js

@@ -1,4 +1,4 @@
-import { AssistantMessageComponent, InteractiveMode, ToolExecutionComponent, VERSION, } from "@earendil-works/pi-coding-agent";
+import { AssistantMessageComponent, InteractiveMode, ToolExecutionComponent, } from "@earendil-works/pi-coding-agent";
 import { Container, stripTerminalSequences, Text } from "@earendil-works/pi-tui";
 import { MessageView } from "./message-view.js";
 import { memberParts, TailUnit, ToolGroupView, } from "./tool-group-view.js";
@@ -7,8 +7,8 @@ const SEAMS = ["getRegisteredToolDefinition", "showExtensionNotify"];
 function record(value) {
     return typeof value === "object" && value !== null;
 }
-// Audited Pi >=1.0.1 compiles these TS-private members to ordinary JS properties.
-// All host-private access stays here; VERSION comes from the running host virtual module.
+// Audited Pi >=1.5.0 compiles these TS-private members to ordinary JS properties.
+// All host-private access stays here.
 function readHost(value) {
     if (!record(value))
         throw new Error("Invalid InteractiveMode instance");
@@ -147,12 +147,14 @@ function noticeMember(component) {
  * semantic color. The group draws these rows once, so a plain notice has no expansion of its own.
  */
 class NotifyView {
-    #text = new Text("", 1, 0);
+    #text = new Text("", 0, 0);
     #ui;
+    #outputPad;
     #message;
     #type;
-    constructor(message, type, ui) {
+    constructor(message, type, ui, outputPad) {
         this.#ui = ui;
+        this.#outputPad = outputPad;
         this.#message = message;
         this.#type = type;
     }
@@ -166,6 +168,8 @@ class NotifyView {
     }
     /** The notice body as it appears both in the transcript and inside the activity group. */
     rows(width) {
+        // The pad setting can change while a notice is on screen, so it is read per render.
+        this.#text.setPaddingX(this.#outputPad());
         // info is the terminal's own readable text; a warning or error keeps its status token and
         // its written prefix, so the level never depends on color alone.
         this.#text.setText(this.#ui.theme.fg(this.#type === "info" ? "text" : this.#type, this.#content));
@@ -182,7 +186,7 @@ class NotifyView {
         this.#text.invalidate();
     }
 }
-function attach(host, run, background) {
+function attach(host, run, background, outputPad) {
     const chat = host.chat;
     const original = {
         render: chat.render,
@@ -399,7 +403,7 @@ function attach(host, run, background) {
         notify(message, type) {
             if (!active)
                 return false;
-            chat.addChild(new NotifyView(message, type, host.ui));
+            chat.addChild(new NotifyView(message, type, host.ui, outputPad));
             host.requestRender();
             return true;
         },
@@ -513,13 +517,7 @@ function patchState() {
     return state;
 }
 /** Private to pi-ext-ui: two process hooks, scoped by the actual host UI facade. */
-export function installHostToolGroups(context, background) {
-    const parts = VERSION.split(".").map(Number);
-    const [major = 0, minor = 0, patch = 0] = parts;
-    if (parts.length !== 3 ||
-        !parts.every(Number.isInteger) ||
-        !(major > 1 || (major === 1 && (minor > 0 || (minor === 0 && patch >= 1)))))
-        throw new Error(`Tool grouping requires Pi >=1.0.1; running ${VERSION}`);
+export function installHostToolGroups(context, background, outputPad) {
     const state = patchState();
     const ui = context.ui;
     if (state.owners.has(ui))
@@ -545,7 +543,7 @@ export function installHostToolGroups(context, background) {
         ui,
         hosts: new Map(),
         failed: new WeakSet(),
-        attach: (host) => attach(host, run, background),
+        attach: (host) => attach(host, run, background, outputPad),
     };
     state.owners.set(ui, owner);
     // Reload rebuilds history before session_start; reconnect already discovered instances.

package/dist/native-tools.js

@@ -3,8 +3,9 @@ import { resolve } from "node:path";
 import { pathToFileURL } from "node:url";
 import { getLanguageFromPath, } from "@earendil-works/pi-coding-agent";
 import { getCapabilities, hyperlink, stripTerminalSequences } from "@earendil-works/pi-tui";
+import { isRecord } from "@hheei/pi-ext-core";
 import { diffStats } from "./diff-view.js";
-import { formatTimeoutFact, isRecord, parseCallArgs, } from "./payload-parser.js";
+import { formatTimeoutFact, parseCallArgs } from "./payload-parser.js";
 /**
  * The single owner of Pi's own tool vocabulary in this UI. Concrete tool packages state their own
  * semantics through ext-core's presentation point; this module is the explicit adapter for the tools

package/dist/nested-calls-view.d.ts

@@ -1,6 +1,6 @@
 import { type Theme, type ToolRenderers } from "@earendil-works/pi-coding-agent";
 import { type Component, type TuiMouseEvent, type TuiMouseEventResult } from "@earendil-works/pi-tui";
-import type { ToolPresentationProvider } from "@hheei/pi-ext-core";
+import { type ToolPresentationProvider } from "@hheei/pi-ext-core";
 import type { NestedCallRecord, ToolRenderContext } from "./renderer.js";
 import { type Collapsible } from "./tool-view.js";
 import type { CapturedToolCall } from "./tracker.js";

package/dist/nested-calls-view.js

@@ -1,13 +1,12 @@
 import { copyToClipboard as copyToSystemClipboard, getLanguageFromPath, highlightCode, } from "@earendil-works/pi-coding-agent";
 import { stripTerminalSequences, truncateToWidth, visibleWidth, wrapTextWithAnsi, } from "@earendil-works/pi-tui";
+import { isRecord } from "@hheei/pi-ext-core";
 import { extractChildToolCopyText, inlinePresentation, nativeRequestCopy } from "./native-tools.js";
+import { formatDurationMs } from "./payload-parser.js";
 import { formatToolHeaderText, TOOL_SPINNER_INTERVAL_MS, ToolView, toContentRows, toolSpinnerFrame, transitionFoldLevel, } from "./tool-view.js";
 function truncate(text, width) {
     return truncateToWidth(text, Math.max(1, width));
 }
-function isRecord(value) {
-    return typeof value === "object" && value !== null && !Array.isArray(value);
-}
 /**
  * A call without a final result is still running: its row carries the live spinner frame. A persisted
  * record states its own status, so a recovered board never animates a call that already ended.
@@ -27,11 +26,6 @@ function copyText(text, request) {
     }
     return request + stripTerminalSequences(text.slice(request.length));
 }
-function formatDuration(ms) {
-    if (ms === undefined || !Number.isFinite(ms))
-        return "";
-    return ms < 1000 ? `${String(Math.round(ms))}ms` : `${(ms / 1000).toFixed(1)}s`;
-}
 export const NESTED_CALLS_EXPANDED_KEY = "__pi_ext_ui_nested_expanded__";
 export const NESTED_CALLS_COPIED_KEY = "__pi_ext_ui_nested_copied__";
 export class NestedCallsView {
@@ -140,6 +134,9 @@ export class NestedCallsView {
             lastComponent: undefined,
             args: "args" in call ? call.args : undefined,
             toolCallId,
+            // The container frame already indented these rows by the output pad; a child body that
+            // applied it again would count the same setting twice.
+            outputPad: 0,
             durationMs: call.durationMs,
             isError: Boolean(call.isError || ("status" in call && call.status === "error")),
             isPartial: false,
@@ -261,7 +258,7 @@ export class NestedCallsView {
                 title: toolName,
                 summary: childSummary,
                 ...(ownSummary === undefined || ownSummary === "" ? {} : { summaryStyled: true }),
-                meta: formatDuration(call.durationMs),
+                meta: formatDurationMs(call.durationMs),
             }, theme);
             const isCopied = this.#copiedIndex === i;
             const copyIcon = this.#failedIndex === i

package/dist/payload-parser.d.ts

@@ -20,7 +20,6 @@ export interface NestedCallRecord {
     readonly cost?: number | undefined;
     readonly error?: string | undefined;
 }
-export declare function isRecord(value: unknown): value is Record<string, unknown>;
 export declare function estimateStreamingTokens(args: unknown): number;
 export declare function formatDurationMs(ms: number | undefined): string;
 export declare function formatCostUsd(cost: number | undefined): string;

package/dist/payload-parser.js

@@ -1,7 +1,5 @@
+import { isRecord } from "@hheei/pi-ext-core";
 export const RESOLVED_CALLS_KEY = "__piExtUiResolvedCalls";
-export function isRecord(value) {
-    return typeof value === "object" && value !== null && !Array.isArray(value);
-}
 export function estimateStreamingTokens(args) {
     if (args === undefined || args === null)
         return 0;

package/dist/renderer.d.ts

@@ -1,10 +1,7 @@
 import { type ExtensionAPI, type ToolRendererResolver, type ToolRenderers } from "@earendil-works/pi-coding-agent";
 import type { ToolUiRuntime } from "./runtime.js";
-/** Optional host metadata is absent on early supported Pi versions. */
-export type ToolRenderContext = Parameters<NonNullable<ToolRenderers["renderCall"]>>[2] & {
-    readonly durationMs?: number | undefined;
-    readonly outputPad?: number | undefined;
-};
+/** Everything the host hands a tool renderer for one call. */
+export type ToolRenderContext = Parameters<NonNullable<ToolRenderers["renderCall"]>>[2];
 /**
  * Upstream tools whose call and result views pi-ext-ui still frames itself. Concrete tool packages
  * own their own presentation through ext-core's optional renderer point, so their names never

package/dist/renderer.js

@@ -1,6 +1,6 @@
 import { highlightCode, } from "@earendil-works/pi-coding-agent";
 import { stripTerminalSequences } from "@earendil-works/pi-tui";
-import { agentResultText, getRegisteredToolPresentationProvider, getRegisteredToolRenderers, getToolPresentationProvider, hasOptionalToolRenderers, } from "@hheei/pi-ext-core";
+import { agentResultText, getRegisteredToolPresentationProvider, getRegisteredToolRenderers, getToolPresentationProvider, hasOptionalToolRenderers, isRecord, } from "@hheei/pi-ext-core";
 import { DiffView } from "./diff-view.js";
 import { NestedCallsView } from "./nested-calls-view.js";
 import { formatToolHeaderText, ToolView, } from "./tool-view.js";
@@ -11,7 +11,7 @@ import { formatToolHeaderText, ToolView, } from "./tool-view.js";
  */
 export const SPECIALIZED_TOOL_RENDERERS = ["codemode"];
 import { inlinePresentation, nativeHeaderSummary, nativeHeavyRequest, nativeOutputLanguage, nativeRequestBlock, nativeResultText, nativeTimeoutFact, } from "./native-tools.js";
-import { callStatsParts, estimateStreamingTokens, extractNestedCalls, formatCostUsd, formatDurationMs, isRecord, RESOLVED_CALLS_KEY, resolveContainerStatus, } from "./payload-parser.js";
+import { callStatsParts, estimateStreamingTokens, extractNestedCalls, formatCostUsd, formatDurationMs, RESOLVED_CALLS_KEY, resolveContainerStatus, } from "./payload-parser.js";
 export { extractChildToolCopyText, inlinePresentation } from "./native-tools.js";
 export { estimateStreamingTokens, } from "./payload-parser.js";
 const SPECIALIZED_TOOL_SET = new Set(SPECIALIZED_TOOL_RENDERERS);
@@ -415,6 +415,9 @@ function renderGenericResult(params) {
                 const upstreamContext = {
                     ...context,
                     lastComponent: lastUpstreamComponent,
+                    // This frame indents the rows it embeds, so the renderer it wraps must not indent
+                    // them again: the pad is applied once, by the outermost shell.
+                    outputPad: 0,
                 };
                 const comp = originalRenderResult(result, effectiveOptions, theme, upstreamContext);
                 lastUpstreamComponent = comp;
@@ -486,19 +489,34 @@ function renderOwnedPresentation(params) {
     const isError = context.isError || result?.isError === true;
     const pending = result === undefined || context.isPartial;
     const isDiff = !isError && !pending && presentation.kind === "diff";
+    // A container's children are facts: the provider states the ones it finished, and the tracker
+    // already holds the one still running, so the row can show work the tool has not reported yet.
+    const captured = runtime.tracker.getCaptured(context.toolCallId) ??
+        (result !== undefined &&
+            isRecord(result.details) &&
+            Array.isArray(result.details.__piExtUiCapturedCalls)
+            ? result.details.__piExtUiCapturedCalls
+            : undefined);
+    const ownCalls = presentation.children === undefined
+        ? extractNestedCalls(result)
+        : presentationChildCalls(presentation.children);
+    const childCalls = withSessionArgs(captured ?? ownCalls, runtime);
     // The tool owns its status: a container says "none" for a child's failure and a partial patch
     // says "warn" even though the host marks the result failed. Only a provider without an opinion
     // falls back to the frame's own reading of the execution.
     const status = presentation.status ??
         (isError
             ? "error"
-            : pending
-                ? context.argsComplete
-                    ? "running"
-                    : "streaming"
-                : isDiff
+            : // Child rows carry the work, so the container's own glyph is the fold pin only.
+                childCalls.length > 0
                     ? "none"
-                    : "ok");
+                    : pending
+                        ? context.argsComplete
+                            ? "running"
+                            : "streaming"
+                        : isDiff
+                            ? "none"
+                            : "ok");
     const railToken = isError ? "error" : pending ? "accent" : "success";
     const output = isError && result !== undefined
         ? agentResultText(result)
@@ -563,18 +581,6 @@ function renderOwnedPresentation(params) {
             if (presentation.collapsedRows !== undefined) {
                 return presentation.collapsedRows;
             }
-            const captured = result !== undefined
-                ? (runtime.tracker.getCaptured(context.toolCallId) ??
-                    (isRecord(result.details) && Array.isArray(result.details.__piExtUiCapturedCalls)
-                        ? result.details.__piExtUiCapturedCalls
-                        : undefined))
-                : undefined;
-            const nestedCalls = extractNestedCalls(result);
-            // The owner's own child facts win over parsing another package's details.
-            const ownCalls = presentation.children === undefined
-                ? nestedCalls
-                : presentationChildCalls(presentation.children);
-            const childCalls = withSessionArgs(captured ?? ownCalls, runtime);
             if (childCalls.length > 0) {
                 return new NestedCallsView({
                     calls: childCalls,
@@ -659,6 +665,7 @@ function createRendererResolver(runtime, options, resolvePresentation, resolveNa
         return {
             renderShell: "self",
             renderCall(args, theme, context) {
+                runtime.rememberRow(context.toolCallId, context.invalidate);
                 if (provider !== undefined) {
                     const callContext = { ...context, args };
                     const view = renderOwnedPresentation({
@@ -722,6 +729,7 @@ function createRendererResolver(runtime, options, resolvePresentation, resolveNa
                 return view;
             },
             renderResult(result, renderOptions, theme, context) {
+                runtime.rememberRow(context.toolCallId, context.invalidate);
                 const resultContext = {
                     ...context,
                     expanded: renderOptions.expanded,
@@ -787,9 +795,16 @@ export function registerCollapsibleToolRenderer(pi, runtime, options) {
     const disposers = [
         pi.on("tool_execution_start", (event) => {
             runtime.tracker.recordStart(event);
+            // A nested call has no row of its own until the container redraws, so the container is
+            // rebuilt the moment the call starts rather than when its result arrives.
+            runtime.repaintParent(event.parentToolCallId);
         }),
         pi.on("tool_execution_end", (event) => {
             runtime.tracker.recordEnd(event);
+            if (event.parentToolCallId === undefined)
+                runtime.forgetRow(event.toolCallId);
+            else
+                runtime.repaintParent(event.parentToolCallId);
         }),
         pi.on("tool_call", (event) => {
             runtime.tracker.recordResolved(event);

package/dist/runtime.d.ts

@@ -5,9 +5,14 @@ import { NestedCallArgsIndex, ToolExecutionTracker } from "./tracker.js";
  * restart, cleanup or settings change in one cannot reach the other.
  */
 export declare class ToolUiRuntime {
+    #private;
     readonly tracker: ToolExecutionTracker;
     readonly nestedCalls: NestedCallArgsIndex;
     /** Pi-native per-tool backgrounds are opt-in; the default is the quiet transparent row. */
     backgrounds: boolean;
+    rememberRow(toolCallId: string, invalidate: () => void): void;
+    /** The host rebuilds the row from fresh facts, so its child board shows the call that started. */
+    repaintParent(parentToolCallId: string | undefined): void;
+    forgetRow(toolCallId: string): void;
     clear(): void;
 }

package/dist/runtime.js

@@ -1,4 +1,6 @@
 import { NestedCallArgsIndex, ToolExecutionTracker } from "./tracker.js";
+/** Bounded like the capture table: only rows the session can still repaint are worth keeping. */
+const MAX_LIVE_ROWS = 200;
 /**
  * The mutable state one extension runtime owns: nested-call captures, the session-branch argument
  * index, and the tool background preference. Two Pi runtimes in one process never share it, so a
@@ -7,10 +9,30 @@ import { NestedCallArgsIndex, ToolExecutionTracker } from "./tracker.js";
 export class ToolUiRuntime {
     tracker = new ToolExecutionTracker();
     nestedCalls = new NestedCallArgsIndex();
+    /** A container row's own invalidator, so a live nested call can repaint the row that made it. */
+    #invalidators = new Map();
     /** Pi-native per-tool backgrounds are opt-in; the default is the quiet transparent row. */
     backgrounds = false;
+    rememberRow(toolCallId, invalidate) {
+        if (!this.#invalidators.has(toolCallId) && this.#invalidators.size >= MAX_LIVE_ROWS) {
+            const oldest = this.#invalidators.keys().next().value;
+            if (oldest !== undefined)
+                this.#invalidators.delete(oldest);
+        }
+        this.#invalidators.set(toolCallId, invalidate);
+    }
+    /** The host rebuilds the row from fresh facts, so its child board shows the call that started. */
+    repaintParent(parentToolCallId) {
+        if (parentToolCallId === undefined)
+            return;
+        this.#invalidators.get(parentToolCallId)?.();
+    }
+    forgetRow(toolCallId) {
+        this.#invalidators.delete(toolCallId);
+    }
     clear() {
         this.tracker.clear();
         this.nestedCalls.setSource(undefined);
+        this.#invalidators.clear();
     }
 }

package/dist/tool-view.js

@@ -1,6 +1,7 @@
 import { Buffer } from "node:buffer";
 import { copyToClipboard as copyToSystemClipboard, } from "@earendil-works/pi-coding-agent";
 import { truncateToWidth, visibleWidth, wrapTextWithAnsi, } from "@earendil-works/pi-tui";
+import { isRecord } from "@hheei/pi-ext-core";
 import { surfaceBlock } from "./background.js";
 /** Pi native tool surfaces: a failure outranks work in progress, which outranks a settled tool. */
 function nativeSurface(status) {
@@ -28,9 +29,6 @@ const DEFAULT_MAX_SECTION_LINES = 9;
 export const MAX_LEVEL2_SECTION_LINES = 500;
 export const MAX_LEVEL2_BYTES = 64 * 1024;
 const DEFAULT_COPY_FEEDBACK_MS = 1200;
-function isRecord(value) {
-    return typeof value === "object" && value !== null && !Array.isArray(value);
-}
 export function transitionFoldLevel(current, ctrlKey) {
     if (current === 0) {
         return ctrlKey ? 2 : 1;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hheei/pi-ext-ui",
-  "version": "0.8.1",
+  "version": "0.8.3",
   "description": "Collapsible tool rendering frame and tool renderer pilot for Pi.",
   "type": "module",
   "main": "dist/index.js",
@@ -35,15 +35,15 @@
     "access": "public"
   },
   "dependencies": {
-    "@hheei/pi-ext-core": "^0.8.0"
+    "@hheei/pi-ext-core": "^0.8.1"
   },
   "peerDependencies": {
-    "@earendil-works/pi-coding-agent": ">=1.0.1",
-    "@earendil-works/pi-tui": ">=1.0.1"
+    "@earendil-works/pi-coding-agent": ">=1.5.0",
+    "@earendil-works/pi-tui": ">=1.5.0"
   },
   "devDependencies": {
-    "@earendil-works/pi-coding-agent": "^1.4.0",
-    "@earendil-works/pi-tui": "^1.4.0"
+    "@earendil-works/pi-coding-agent": "^1.5.0",
+    "@earendil-works/pi-tui": "^1.5.0"
   },
   "scripts": {
     "build": "node -e \"import { rmSync } from 'node:fs'; rmSync('dist', { recursive: true, force: true })\" && tsc -p tsconfig.build.json"