@hheei/pi-ext-ui 0.2.0 → 0.2.1

package/dist/payload-parser.js

@@ -1,14 +1,8 @@
+import { formatReadLineRange } from "@earendil-works/pi-coding-agent";
 import { stripTerminalSequences } from "@earendil-works/pi-tui";
-import { previewV4aOperations } from "./apply-patch-preview.js";
-import { applyPatchFacts } from "./apply-patch-view.js";
 import { diffStats } from "./diff-view.js";
-export const APPLY_PATCH_TOOL_NAME = "apply_patch";
-export const PYTHON_EVAL_TOOL_NAME = "python_eval";
-export const PYTHON_EVAL_DETAILS_FORMAT = "pi-ext-tools-python-eval";
 export const EDIT_TOOL_NAME = "edit";
 export const WRITE_TOOL_NAME = "write";
-export const EDIT_VIEW_KEY = "__piExtToolsEditView";
-export const WRITE_VIEW_KEY = "__piExtToolsWriteView";
 export const CODEMODE_OPTIONS_PREFIX = "// @options:";
 export const CODEMODE_SCRIPT_HEADER = /^Script (?:completed|failed)\r?\nWall time ([\d.]+) seconds\r?\nOutput:\r?\n/;
 export function isRecord(value) {
@@ -257,108 +251,17 @@ export function toolPath(args) {
     const val = parsed.path ?? parsed.file_path;
     return typeof val === "string" ? val.trim() : "";
 }
-export function editPath(args) {
-    return toolPath(args);
-}
-export function numericField(value, key) {
-    return isRecord(value) && typeof value[key] === "number" ? value[key] : undefined;
-}
-export function writeStoredDiff(lines) {
-    const hunks = [];
-    let hunk = [];
-    const finish = () => {
-        if (hunk.length > 0)
-            hunks.push(hunk);
-        hunk = [];
-    };
-    for (const entry of lines) {
-        if (!isRecord(entry))
-            return undefined;
-        if (entry.type === "sep") {
-            finish();
-            continue;
-        }
-        if (entry.type !== "add" && entry.type !== "del" && entry.type !== "ctx")
-            return undefined;
-        if (typeof entry.content !== "string")
-            return undefined;
-        const line = entry.type === "del" ? entry.oldNum : entry.newNum;
-        if (typeof line !== "number")
-            return undefined;
-        hunk.push({ kind: entry.type, line, text: entry.content });
-    }
-    finish();
-    return hunks.length === 0
-        ? undefined
-        : { kind: "hunks", hunks: hunks.map((lines) => ({ lines })) };
-}
-/** Uses only the baseline and content already stored by the write tool. */
-export function writePresentation(args, details) {
-    const view = isRecord(details) ? details[WRITE_VIEW_KEY] : undefined;
-    if (!isRecord(view))
-        return undefined;
-    if (view.kind === "noChange")
-        return { kind: "noChange" };
-    if (view.kind === "diff") {
-        const stored = Array.isArray(view.lines) ? writeStoredDiff(view.lines) : undefined;
-        if (stored === undefined)
-            return undefined;
-        const source = stored;
-        const stats = diffStats(source);
-        return stats === undefined
-            ? undefined
-            : {
-                kind: "diff",
-                source,
-                added: numericField(view, "added") ?? stats.added,
-                removed: numericField(view, "removed") ?? stats.removed,
-            };
-    }
-    if (view.kind !== "new" && view.kind !== "replace")
-        return undefined;
-    const content = typeof view.content === "string"
-        ? view.content
-        : isRecord(args) && typeof args.content === "string"
-            ? args.content
-            : undefined;
-    if (content === undefined)
-        return undefined;
-    const lines = numericField(view, "lines") ?? content.split("\n").length;
-    return {
-        kind: "content",
-        label: view.kind === "new" ? `new file (${String(lines)} lines)` : `wrote (${String(lines)} lines)`,
-        content,
-    };
-}
-export function fileToolPresentation(toolName, args, details, isError) {
+export function fileToolPresentation(toolName, args, _details, isError) {
     const path = toolPath(args);
     if (toolName === "read") {
         if (path === "")
             return undefined;
         const parsed = parseCallArgs(args);
-        let range = "";
-        if (typeof parsed?.offset === "number" || typeof parsed?.limit === "number") {
-            const start = typeof parsed.offset === "number" ? parsed.offset : 1;
-            const end = typeof parsed.limit === "number" ? start + parsed.limit - 1 : "";
-            range = `:${String(start)}${end ? `-${String(end)}` : ""}`;
-        }
+        const range = formatReadLineRange(parsed?.offset, parsed?.limit);
         return { summary: `${path}${range}`, status: isError ? "error" : "ok" };
     }
     if (toolName === WRITE_TOOL_NAME) {
-        const presentation = isError ? undefined : writePresentation(args, details);
-        const delta = presentation?.kind === "diff"
-            ? ` (+${String(presentation.added)} -${String(presentation.removed)})`
-            : presentation?.kind === "content"
-                ? ` (${presentation.label})`
-                : presentation?.kind === "noChange"
-                    ? " (no changes)"
-                    : "";
-        if (path === "" && delta === "")
-            return undefined;
-        return {
-            summary: `${path}${delta}`.trim(),
-            status: isError ? "error" : presentation?.kind === "noChange" ? "none" : "ok",
-        };
+        return path === "" ? undefined : { summary: path, status: isError ? "error" : "ok" };
     }
     return undefined;
 }
@@ -372,7 +275,7 @@ export function editDiffSource(details) {
     if (typeof details.diff === "string" && details.diff.trim() !== "") {
         return displayDiffSource(details.diff);
     }
-    return editViewSource(details[EDIT_VIEW_KEY]);
+    return undefined;
 }
 export function displayDiffSource(diff) {
     const hunks = [];
@@ -401,128 +304,6 @@ export function displayDiffSource(diff) {
     hunks.push(current);
     return { kind: "hunks", hunks: hunks.map((lines) => ({ lines })) };
 }
-export function editViewSource(view) {
-    const hunks = editViewHunks(view);
-    return hunks.length === 0 ? undefined : { kind: "hunks", hunks };
-}
-export function editViewHunks(view) {
-    if (!isRecord(view))
-        return [];
-    const ops = view.kind === "single" && isRecord(view.op)
-        ? [view.op]
-        : view.kind === "multi" && Array.isArray(view.ops)
-            ? view.ops.filter(isRecord)
-            : [];
-    return ops.flatMap((op) => hunksForEditOp(op));
-}
-export function hunksForEditOp(op) {
-    if (typeof op.oldContent !== "string" || typeof op.newContent !== "string")
-        return [];
-    const startLine = typeof op.startLine === "number" && op.startLine > 0 ? op.startLine : 1;
-    return lineHunks(op.oldContent.replace(/\r\n/g, "\n"), op.newContent.replace(/\r\n/g, "\n"), startLine);
-}
-export function splitFragment(text) {
-    if (text === "")
-        return [];
-    const lines = text.split("\n");
-    if (lines.at(-1) === "")
-        lines.pop();
-    return lines;
-}
-/** Line LCS of one stored before/after fragment. No file context is invented. */
-export function lineHunks(before, after, startLine) {
-    const oldLines = splitFragment(before);
-    const newLines = splitFragment(after);
-    const oldCount = oldLines.length;
-    const newCount = newLines.length;
-    const lengths = Array.from({ length: oldCount + 1 }, () => Array(newCount + 1).fill(0));
-    for (let oldIndex = oldCount - 1; oldIndex >= 0; oldIndex -= 1) {
-        const row = lengths[oldIndex];
-        const next = lengths[oldIndex + 1];
-        if (row === undefined || next === undefined)
-            continue;
-        for (let newIndex = newCount - 1; newIndex >= 0; newIndex -= 1) {
-            row[newIndex] =
-                oldLines[oldIndex] === newLines[newIndex]
-                    ? (next[newIndex + 1] ?? 0) + 1
-                    : Math.max(next[newIndex] ?? 0, row[newIndex + 1] ?? 0);
-        }
-    }
-    const lines = [];
-    let oldIndex = 0;
-    let newIndex = 0;
-    let oldLine = startLine;
-    let newLine = startLine;
-    while (oldIndex < oldCount || newIndex < newCount) {
-        const row = lengths[oldIndex];
-        const next = lengths[oldIndex + 1];
-        if (oldIndex < oldCount && newIndex < newCount && oldLines[oldIndex] === newLines[newIndex]) {
-            lines.push({ kind: "ctx", line: newLine, text: oldLines[oldIndex] ?? "" });
-            oldIndex += 1;
-            newIndex += 1;
-            oldLine += 1;
-            newLine += 1;
-        }
-        else if (newIndex < newCount &&
-            (oldIndex === oldCount || (row?.[newIndex + 1] ?? 0) > (next?.[newIndex] ?? 0))) {
-            lines.push({ kind: "add", line: newLine, text: newLines[newIndex] ?? "" });
-            newIndex += 1;
-            newLine += 1;
-        }
-        else {
-            lines.push({ kind: "del", line: oldLine, text: oldLines[oldIndex] ?? "" });
-            oldIndex += 1;
-            oldLine += 1;
-        }
-    }
-    const changed = lines.some((line) => line.kind !== "ctx");
-    return changed ? [{ lines }] : [];
-}
-export function patchTextForCopy(details) {
-    if (isRecord(details) && typeof details.patch === "string")
-        return details.patch;
-    const hunks = editViewHunks(isRecord(details) ? details[EDIT_VIEW_KEY] : undefined);
-    if (hunks.length === 0)
-        return undefined;
-    return (hunks
-        .map((hunk) => {
-        let oldStart = 0;
-        let newStart = 0;
-        let oldCount = 0;
-        let newCount = 0;
-        const body = hunk.lines.map((line) => {
-            if (line.kind !== "add") {
-                oldCount += 1;
-                if (oldStart === 0)
-                    oldStart = line.line;
-            }
-            if (line.kind !== "del") {
-                newCount += 1;
-                if (newStart === 0)
-                    newStart = line.line;
-            }
-            const sign = line.kind === "add" ? "+" : line.kind === "del" ? "-" : " ";
-            return sign + line.text;
-        });
-        if (body.length === 0)
-            return "";
-        const header = "@@ -" +
-            String(oldStart || 1) +
-            "," +
-            String(oldCount) +
-            " +" +
-            String(newStart || 1) +
-            "," +
-            String(newCount) +
-            " @@";
-        return [header, ...body].join("\n");
-    })
-        .filter((hunk) => hunk !== "")
-        .join("\n") || undefined);
-}
-export function patchSummary(files, added, removed) {
-    return `modify ${files}${files === 1 ? " file" : " files"} (+${added} -${removed})`;
-}
 /**
  * The name-free part of a summary for tools whose call arguments read well on their own. The tool
  * name is the row's title, so repeating it here would print it twice.
@@ -564,22 +345,6 @@ export function inlinePresentation(toolName, args, details, isError) {
     if (fileTool !== undefined)
         return fileTool;
     const parsed = parseCallArgs(args);
-    if (toolName === APPLY_PATCH_TOOL_NAME) {
-        const facts = applyPatchFacts(details);
-        if (facts !== undefined) {
-            return {
-                summary: patchSummary(new Set(facts.changedPaths).size, facts.addedLines, facts.removedLines),
-                status: isError ? "error" : "ok",
-            };
-        }
-        if (parsed === undefined || typeof parsed.patch !== "string")
-            return undefined;
-        const operations = previewV4aOperations(parsed.patch, true, undefined);
-        return {
-            summary: patchSummary(new Set(operations.map((operation) => operation.path)).size, operations.reduce((sum, operation) => sum + operation.addedLines, 0), operations.reduce((sum, operation) => sum + operation.removedLines, 0)),
-            status: isError ? "error" : "ok",
-        };
-    }
     if (parsed === undefined)
         return undefined;
     if (toolName === EDIT_TOOL_NAME) {
@@ -603,42 +368,3 @@ export function inlinePresentation(toolName, args, details, isError) {
         status: isError ? "error" : "ok",
     };
 }
-/** Reads only the persisted `pi-ext-tools-eval` details; other shapes degrade to plain output. */
-export function pythonEvalFacts(details) {
-    if (!isRecord(details) || details.format !== PYTHON_EVAL_DETAILS_FORMAT)
-        return undefined;
-    const rawRows = Array.isArray(details.rows) ? details.rows : [];
-    const rows = [];
-    for (const entry of rawRows) {
-        if (!isRecord(entry))
-            continue;
-        if (entry.kind === "text" && typeof entry.text === "string") {
-            rows.push({ kind: "text", text: entry.text });
-        }
-        else if (entry.kind === "display" && typeof entry.text === "string") {
-            rows.push({ kind: "display", text: entry.text });
-        }
-        else if (entry.kind === "result" && typeof entry.text === "string") {
-            rows.push({ kind: "result", text: entry.text });
-        }
-        else if (entry.kind === "state" && typeof entry.text === "string") {
-            rows.push({ kind: "state", text: entry.text });
-        }
-        else if (entry.kind === "tool" && isRecord(entry.trace)) {
-            const trace = entry.trace;
-            if (typeof trace.name !== "string")
-                continue;
-            rows.push({
-                kind: "tool",
-                name: trace.name,
-                args: trace.args,
-                durationMs: typeof trace.durationMs === "number" ? trace.durationMs : undefined,
-                error: typeof trace.error === "string" ? trace.error : undefined,
-            });
-        }
-    }
-    return {
-        rows,
-        error: typeof details.error === "string" ? details.error : undefined,
-    };
-}

package/dist/renderer.d.ts

@@ -1,7 +1,11 @@
 import { type ExtensionAPI, type ToolRendererResolver, type ToolRenderers } from "@earendil-works/pi-coding-agent";
 export type ToolRenderContext = Parameters<NonNullable<ToolRenderers["renderCall"]>>[2];
-/** Tools with a pi-ext-ui-owned renderer. Other tools stay on Pi's renderer. */
-export declare const SPECIALIZED_TOOL_RENDERERS: readonly ["edit", "write", "apply_patch", "python_eval", "codemode"];
+/**
+ * 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
+ * appear here.
+ */
+export declare const SPECIALIZED_TOOL_RENDERERS: readonly ["codemode"];
 export type SpecializedToolRendererName = (typeof SPECIALIZED_TOOL_RENDERERS)[number];
 /** Web-access tools use the shared folding shell while retaining their native body renderer. */
 export declare const WEB_ACCESS_TOOLS: readonly ["web_search", "web_fetch", "web_open"];
@@ -15,4 +19,6 @@ export interface CollapsibleToolRendererOptions {
     readonly copyToClipboard?: ((text: string) => Promise<void>) | undefined;
 }
 export declare function createCollapsibleToolRendererResolver(options?: CollapsibleToolRendererOptions): ToolRendererResolver;
+/** UI factory consumed by package-owned direct renderers, independently of host resolver precedence. */
+export declare function createOptionalToolRendererFactory(pi: ExtensionAPI, options?: CollapsibleToolRendererOptions): import("@hheei/pi-ext-core").ToolRendererFactory;
 export declare function registerCollapsibleToolRenderer(pi: ExtensionAPI, options?: CollapsibleToolRendererOptions): void;