@hheei/pi-ext-ui 0.8.2 → 0.8.4

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");
@@ -287,7 +287,8 @@ function attach(host, run, background, outputPad) {
                     weight += 0.5;
             }
             if (first !== undefined && weight >= 2) {
-                const group = groups.get(first) ?? new ToolGroupView(host.ui, host.requestRender, background);
+                const group = groups.get(first) ??
+                    new ToolGroupView(host.ui, host.requestRender, background, outputPad);
                 const boundaryTime = ended && boundary !== undefined
                     ? (run.observed.get(boundary)?.at ?? performance.now())
                     : undefined;
@@ -518,12 +519,6 @@ function patchState() {
 }
 /** Private to pi-ext-ui: two process hooks, scoped by the actual host UI facade. */
 export function installHostToolGroups(context, background, outputPad) {
-    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}`);
     const state = patchState();
     const ui = context.ui;
     if (state.owners.has(ui))

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 {
@@ -264,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);

package/dist/tool-group-view.d.ts

@@ -70,7 +70,7 @@ export declare class TailUnit implements Component {
  */
 export declare class ToolGroupView extends Container {
     #private;
-    constructor(ui: ExtensionUIContext, redraw: () => void, background: () => boolean);
+    constructor(ui: ExtensionUIContext, redraw: () => void, background: () => boolean, outputPad: () => number);
     /**
      * Release the block's own lifetime. The projection drops a block when it is evicted, when
      * grouping is disabled, or when the patch unloads; an idempotent dispose keeps that path honest,

package/dist/tool-group-view.js

@@ -157,6 +157,8 @@ export class ToolGroupView extends Container {
     #redraw;
     /** Pi-native block backgrounds are opt-in; while they are on, the fill replaces the rails. */
     #background;
+    /** Pi's own output padding, so the frame lines up with the tool rows it wraps. */
+    #outputPad;
     /** The stable clock for this block; restored history stays unknown until live content joins. */
     #clock;
     /** The block's own members, drawn inside the group. */
@@ -179,11 +181,12 @@ export class ToolGroupView extends Container {
     // One extra colored row above and below: the block needs breathing room inside its background.
     #box = new Box(0, 1);
     #spacer = new Spacer(1);
-    constructor(ui, redraw, background) {
+    constructor(ui, redraw, background, outputPad) {
         super();
         this.#ui = ui;
         this.#redraw = redraw;
         this.#background = background;
+        this.#outputPad = outputPad;
         this.#heading = new MouseRegion(this.#header, (event) => {
             if (event.type !== "click" || event.button !== "left")
                 return undefined;
@@ -387,8 +390,13 @@ export class ToolGroupView extends Container {
         const theme = this.#ui.theme;
         const lines = [];
         const hits = [];
+        // The frame is the block's own row; a tool row inside it pads itself, so only these rows take
+        // Pi's output padding.
+        const pad = Math.max(0, this.#outputPad());
+        const lead = " ".repeat(pad);
+        const inner = Math.max(0, width - pad * 2);
         const push = (row, action = { kind: "none" }) => {
-            lines.push(fitRow(row, Math.max(0, width), "…"));
+            lines.push(fitRow(`${lead}${row}`, Math.max(0, width), "…"));
             hits.push({ height: 1, action });
         };
         /** Rows that belong to one component keep one hit so its own geometry stays correct. */
@@ -404,7 +412,7 @@ export class ToolGroupView extends Container {
             if (member.kind === "tool")
                 tools.push(index);
         }
-        push(theme.fg("borderMuted", "─".repeat(Math.max(0, width))));
+        push(theme.fg("borderMuted", "─".repeat(inner)));
         const notices = this.#noticeItems();
         if (notices.length > 0) {
             const kinds = new Map();
@@ -418,7 +426,7 @@ export class ToolGroupView extends Container {
                 category: "notices",
             });
             if (this.#category === "notices")
-                this.#renderNotices(notices, width, push);
+                this.#renderNotices(notices, inner, push);
         }
         const thinking = this.#thinkingItems();
         if (thinking.length > 0) {
@@ -428,7 +436,7 @@ export class ToolGroupView extends Container {
                 category: "thinking",
             });
             if (this.#category === "thinking")
-                this.#renderThinking(thinking, width, push);
+                this.#renderThinking(thinking, inner, push);
         }
         // Names and header calls share one eligibility: only completed calls are counted. A tool that
         // is still streaming or running stays listed and expandable, but adds no number.
@@ -461,7 +469,7 @@ export class ToolGroupView extends Container {
                 this.#renderTools(tools, width, push, pushBlock);
         }
         // The rails bracket the header, categories, and any opened detail; the footer sits below them.
-        push(theme.fg("borderMuted", "─".repeat(Math.max(0, width))));
+        push(theme.fg("borderMuted", "─".repeat(inner)));
         // The footer is always the block's request and token totals: one row, never a tool status.
         push(this.#footerRow());
         this.#hits = hits;
@@ -698,7 +706,8 @@ export class ToolGroupView extends Container {
             theme.fg("dim", elapsed),
             theme.fg("dim", `${calls} ${calls === 1 ? "call" : "calls"}`),
         ].join(theme.fg("dim", " · "));
-        this.#header.setText(fitRow(`${theme.fg("dim", this.#renderedExpanded ? "▾" : "▸")} ${details}`, Math.max(0, width), "…"));
+        const pad = Math.max(0, this.#outputPad());
+        this.#header.setText(fitRow(`${" ".repeat(pad)}${theme.fg("dim", this.#renderedExpanded ? "▾" : "▸")} ${details}`, Math.max(0, width - pad * 2), "…"));
         return super.render(width);
     }
     /**

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.2",
+  "version": "0.8.4",
   "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"