@gavin-hu/my-pi-agent 0.17.0 → 0.19.0

package/extensions/team/types.ts

@@ -0,0 +1,48 @@
+/**
+ * The shapes `team` registers and reports.
+ *
+ * A `team` result is one card plus what happened to the handoff, so the details say
+ * both: `assigned` is who the card went to, `delivered` is the labels the bus wrote
+ * a message file for, and `messageError` is the send that never landed — a card is
+ * never left in a state the report does not describe.
+ */
+
+import type { Card } from "../../lib/board/card.ts";
+import type { CardOwner } from "../../lib/board/card.ts";
+
+/** The tool's name. `board` owns the cards; `team` owns the round. */
+export const TEAM_TOOL = "team";
+
+/** The three actions: the lead's whole surface — hand out, offer, take back. */
+export const TEAM_ACTIONS = ["assign", "offer", "reclaim"] as const;
+
+/** One action. */
+export type TeamAction = (typeof TEAM_ACTIONS)[number];
+
+/** What one call did. */
+export interface TeamDetails {
+	action: TeamAction;
+	/** The card after the action, or the one a refusal left alone. */
+	card?: Card;
+	/** Who the card was assigned to, when the assignment happened. */
+	assigned?: CardOwner;
+	/**
+	 * The labels a message file was written for, when the send landed: the one peer for
+	 * `assign`, every other live session for `offer`.
+	 */
+	delivered?: string[];
+	/**
+	 * The peers the message reached whose record says they are not listening, so it waits
+	 * for their next turn instead of being raised as one. Absent when every peer listens.
+	 */
+	notListening?: string[];
+	/** Why the handoff was not sent; the card is untouched when this is set. */
+	messageError?: string;
+	/** A refusal's reason, which the renderer draws instead of the details. */
+	error?: string;
+}
+
+/** Whether a value is one of the actions, so an unknown one is a refusal and not a crash. */
+export function normalizeTeamAction(raw: unknown): TeamAction | undefined {
+	return typeof raw === "string" && TEAM_ACTIONS.includes(raw as TeamAction) ? (raw as TeamAction) : undefined;
+}

package/extensions/watch/index.ts

@@ -4,8 +4,9 @@
  * Registers the `watch` tool and `/watch` command. Watchers read a file or a
  * command's merged output, match each new line against a regular expression,
  * and alert the user; matching lines are reported to the model — and shown as a
- * bytefly notice — at the next turn, or wake a turn immediately when the
- * watcher opted into `wake`. Watchers are session-only and stopped on shutdown.
+ * bytefly notice — at the next turn, or wake a turn as soon as the agent is idle
+ * when the watcher opted into `wake`, including right after a running turn
+ * settles. Watchers are session-only and stopped on shutdown.
  *
  * Load with:  pi --extension ./extensions/watch
  */
@@ -14,7 +15,7 @@ import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { isExtensionEnabled } from "../../lib/env.ts";
 import { WATCH_MATCH_CHANNEL, type WatchMatchEvent } from "../../lib/events.ts";
 import { sanitizeLine } from "../../lib/format.ts";
-import { noticeMessage, type NoticeMessage } from "../../lib/notice.ts";
+import { burstText, noticeMessage, type NoticeMessage } from "../../lib/notice.ts";
 import { createPendingReporter, registerPendingChannel } from "../../lib/report.ts";
 import { registerCommands } from "./commands.ts";
 import { clipLabel } from "../../lib/tui/columns.ts";
@@ -32,10 +33,14 @@ const MAX_CONTEXT_MATCHES = 20;
 /** Build the model-facing text for the pending-match notice. */
 export function buildWatchContext(matches: PendingMatch[]): string {
 	const shown = matches.slice(0, MAX_CONTEXT_MATCHES);
-	const lines = shown.map(formatPendingMatch);
 	const more = matches.length - shown.length;
-	const suffix = more > 0 ? `\n… ${more} more match${more === 1 ? "" : "es"}` : "";
-	return `${WATCH_CONTEXT_MARKER}\n${matches.length} watched line${matches.length === 1 ? "" : "s"} matched:\n${lines.join("\n")}${suffix}\nRead more with \`watch matches <id>\`.`;
+	return burstText({
+		marker: WATCH_CONTEXT_MARKER,
+		headline: `${matches.length} watched line${matches.length === 1 ? "" : "s"} matched`,
+		lines: shown.map((match) => formatPendingMatch(match)),
+		...(more > 0 ? { more: `… ${more} more match${more === 1 ? "" : "es"}` } : {}),
+		hint: "Read more with `watch matches <id>`.",
+	});
 }
 
 /** The shared notice for pending matches: model text plus renderer details. */
@@ -64,15 +69,16 @@ export default function watch(pi: ExtensionAPI, deps: WatchDeps = {}): void {
 	if (!isExtensionEnabled("watch")) return;
 	const runtime = deps.runtime ?? createWatchRuntime();
 
-	// Opt-in wake: a `wake` watcher matched while the agent is idle. The reporter
-	// reports *all* pending matches, not just the waking one, so a non-wake match
-	// is not silently dropped.
+	// Opt-in wake: a `wake` watcher matched. The reporter reports *all* pending
+	// matches, not just the waking one, so a non-wake match is not silently
+	// dropped, and it retries a burst that arrived mid-turn once the run settles.
 	const reporter = createPendingReporter<PendingMatch>({
 		wake: (notice) => pi.sendMessage(notice, { triggerTurn: true }),
 		pending: () => runtime.pendingMatches(),
 		markReported: (matches) => runtime.markReported(matches),
 		buildNotice: watchNotice,
 		source: "watch",
+		wakeRequested: () => runtime.pendingMatches().some((match) => runtime.get(match.watcherId)?.wake === true),
 		// A wake means the user is away from the terminal, so the alert also leaves
 		// the process. bytefly's notify capability owns delivery; this publishes the
 		// fact and never learns how it is sent. The newest match is the one an alert

package/extensions/web-access/fetch/page.ts

@@ -11,6 +11,7 @@
 
 import type { WebFetchConfig } from "./config.ts";
 import { extractReadable } from "./extract.ts";
+import { renderStructured } from "./structured.ts";
 import { extractPdfText, type PdfExtractor } from "../../../lib/pdf.ts";
 import { createFetchRunner, type HttpRunner, type HttpResponse } from "../http.ts";
 import { decodeHtml } from "../charset.ts";
@@ -157,7 +158,9 @@ export async function runFetch(
 		title = page.title;
 		text = page.text;
 	} else if (isTextualType(response.contentType)) {
-		text = response.body;
+		// Rendered here, once per fetch, because the result is what the cache stores:
+		// paging and `find` then read one stable string instead of a fresh rendering.
+		text = renderStructured(response.body, response.contentType) ?? response.body;
 	} else {
 		text = `(binary content: ${response.contentType || "unknown"}, ${response.sizeBytes} bytes; web_fetch returns text only)`;
 	}

package/extensions/web-access/fetch/structured.ts

@@ -0,0 +1,447 @@
+/**
+ * Render a structured body — JSON, CSV, or XML — as readable text.
+ *
+ * `web_fetch` used to hand these back as the blob it downloaded: a page a reader
+ * has to parse in their head. This turns one into the shape a model reads best —
+ * a table for records, indented keys for an object, an indented tree for XML — and
+ * declines anything it cannot read, so the caller keeps the raw body and nothing
+ * usable is lost.
+ *
+ * A source cap keeps an oversized response out: past it the raw body, which pages,
+ * reads better than a rendered one. A rendered-character cap keeps one answer from
+ * doubling the cache, and the note says where it stopped. Nothing here throws —
+ * every failure is `undefined`, which `page.ts` reads as "return the body
+ * unchanged".
+ *
+ * The declared content type decides first. A body that declares nothing usable is
+ * sniffed only when it has a shape — an opening brace, a uniform comma count, a
+ * root tag — because a plain-text page is the common case and a mangled table is
+ * worse than a blob.
+ */
+
+import { asRecord } from "../../../lib/config.ts";
+import { decodeEntities } from "../html.ts";
+import { charLength, sliceChars } from "../text.ts";
+
+/** Past this, the raw body — which pages — is the better answer. */
+const MAX_SOURCE_CHARS = 1_048_576;
+
+/** Rendered characters before the cut; the cached text pays this too. */
+const MAX_RENDERED_CHARS = 60_000;
+
+/** One level of nesting, and the gap between two table columns. */
+const INDENT = "  ";
+
+/** A table cell is one line, so a nested value is compacted to about this width. */
+const CELL_CHARS = 40;
+
+/** A list short enough to sit on one line with its key. */
+const INLINE_LIST_CHARS = 80;
+
+/**
+ * Render a body as text, or `undefined` when it is not one this module reads.
+ */
+export function renderStructured(body: string, contentType: string): string | undefined {
+	if (body.length > MAX_SOURCE_CHARS) return undefined;
+	const declared = contentType.toLowerCase();
+	if (isJsonType(declared)) return renderJson(body);
+	if (isCsvType(declared)) return renderCsv(body);
+	if (isXmlType(declared)) return renderXml(body);
+
+	if (looksLikeJson(body)) return renderJson(body);
+	if (looksLikeCsv(body)) return renderCsv(body);
+	if (looksLikeXml(body)) return renderXml(body);
+	return undefined;
+}
+
+function isJsonType(contentType: string): boolean {
+	return /json/.test(contentType);
+}
+
+function isCsvType(contentType: string): boolean {
+	return /csv/.test(contentType);
+}
+
+function isXmlType(contentType: string): boolean {
+	return /xml/.test(contentType);
+}
+
+function looksLikeJson(body: string): boolean {
+	return /^\s*[[{]/.test(body);
+}
+
+/** Uniformly comma-separated lines, which is the shape a plain-text export has. */
+function looksLikeCsv(body: string): boolean {
+	const lines = body.split(/\r?\n/).filter((line) => line.trim() !== "");
+	if (lines.length < 2) return false;
+	const commas = lines[0].split(",").length - 1;
+	return commas > 0 && lines.every((line) => line.split(",").length - 1 === commas);
+}
+
+function looksLikeXml(body: string): boolean {
+	return /^\s*<(\?xml\b|[A-Za-z_])/.test(body) && body.includes("</");
+}
+
+/** The rendered lines, what they cost, and whether the cap cut anything. */
+interface Output {
+	lines: string[];
+	used: number;
+	cut: boolean;
+}
+
+function createOutput(): Output {
+	return { lines: [], used: 0, cut: false };
+}
+
+/** Append one line within the cap; `false` means the caller should stop. */
+function emit(out: Output, line: string): boolean {
+	const cost = line.length + 1;
+	if (out.used + cost > MAX_RENDERED_CHARS) {
+		out.cut = true;
+		return false;
+	}
+	out.lines.push(line);
+	out.used += cost;
+	return true;
+}
+
+function emitAt(out: Output, text: string, depth: number): boolean {
+	return emit(out, `${INDENT.repeat(depth)}${text}`);
+}
+
+/** The rendered text, with one note when the cap stopped it early. */
+function finish(out: Output, body: string): string {
+	const text = out.lines.join("\n");
+	if (!out.cut) return text;
+	return `${text}\n\n… (rendered ${out.used} characters of a ${body.length}-character body; the rest was cut)`;
+}
+
+/** One line for a value: a string's own line breaks become spaces. */
+function oneLine(text: string): string {
+	return text.replace(/\s*\n\s*/g, " ");
+}
+
+/** Collapse a text node or an attribute to one line. */
+function collapse(text: string): string {
+	return text.replace(/\s+/g, " ").trim();
+}
+
+/** A value JSON stores as one atom: no nesting to walk. */
+function isScalar(value: unknown): boolean {
+	return value === null || typeof value !== "object";
+}
+
+function isScalarList(value: readonly unknown[]): boolean {
+	return value.every(isScalar);
+}
+
+function isRecordList(value: readonly unknown[]): value is readonly Record<string, unknown>[] {
+	return value.every((item) => {
+		const record = asRecord(item);
+		return record !== undefined && Object.keys(record).length > 0;
+	});
+}
+
+/** One line of text for a scalar; a string's own line breaks become spaces. */
+function scalarText(value: unknown): string {
+	if (value === null) return "null";
+	if (typeof value === "string") return oneLine(value);
+	return String(value);
+}
+
+/** A table cell: a nested value is compacted and clipped. */
+function cellText(value: unknown): string {
+	if (value === undefined) return "";
+	if (isScalar(value)) return scalarText(value);
+	const compact = JSON.stringify(value) ?? "";
+	return charLength(compact) <= CELL_CHARS ? compact : `${sliceChars(compact, 0, CELL_CHARS)}…`;
+}
+
+/**
+ * Lines as a table: each column padded to its widest cell, two spaces between
+ * columns, no padding after the last one. Ragged rows are padded with empty cells,
+ * so a short record still lines up instead of shifting the columns after it.
+ */
+function renderGrid(rows: readonly (readonly string[])[], depth: number, out: Output): void {
+	const columns = rows.reduce((widest, row) => Math.max(widest, row.length), 0);
+	const widths = Array.from({ length: columns }, (_unused, column) =>
+		rows.reduce((widest, row) => Math.max(widest, charLength(row[column] ?? "")), 0),
+	);
+
+	for (const row of rows) {
+		const line: string[] = [];
+		for (let column = 0; column < columns; column++) {
+			const value = oneLine(row[column] ?? "");
+			line.push(column === columns - 1 ? value : value + " ".repeat(widths[column] - charLength(value)));
+		}
+		if (!emitAt(out, line.join(INDENT).trimEnd(), depth)) return;
+	}
+}
+
+/** Records as a table: the union of their keys, so one record's extra key shows. */
+function renderTable(rows: readonly Record<string, unknown>[], depth: number, out: Output): void {
+	const header: string[] = [];
+	for (const row of rows) {
+		for (const key of Object.keys(row)) if (!header.includes(key)) header.push(key);
+	}
+	const cells = rows.map((row) => header.map((key) => cellText(row[key])));
+	renderGrid([header, ...cells], depth, out);
+}
+
+/**
+ * The one-line form of a value, when it has one: a scalar, an empty container, or
+ * a short list. `undefined` means it needs its own block.
+ */
+function inlineValue(value: unknown): string | undefined {
+	if (isScalar(value)) return scalarText(value);
+	if (Array.isArray(value)) {
+		if (value.length === 0) return "[]";
+		if (!isScalarList(value)) return undefined;
+		const text = `[${value.map(scalarText).join(", ")}]`;
+		return charLength(text) <= INLINE_LIST_CHARS ? text : undefined;
+	}
+	// Only an empty object has a one-line form; anything else needs a block.
+	const record = asRecord(value);
+	return record !== undefined && Object.keys(record).length === 0 ? "{}" : undefined;
+}
+
+function renderValue(value: unknown, depth: number, out: Output): void {
+	if (isScalar(value)) {
+		emitAt(out, scalarText(value), depth);
+		return;
+	}
+	if (Array.isArray(value)) {
+		renderArray(value, depth, out);
+		return;
+	}
+	const record = asRecord(value);
+	if (record) {
+		renderObject(record, depth, out);
+	}
+}
+
+function renderObject(value: Record<string, unknown>, depth: number, out: Output): void {
+	const keys = Object.keys(value);
+	if (keys.length === 0) {
+		emitAt(out, "{}", depth);
+		return;
+	}
+	for (const key of keys) {
+		const child = value[key];
+		const inline = inlineValue(child);
+		if (inline !== undefined) {
+			if (!emitAt(out, `${key}: ${inline}`, depth)) return;
+			continue;
+		}
+		if (!emitAt(out, key, depth)) return;
+		renderValue(child, depth + 1, out);
+		if (out.cut) return;
+	}
+}
+
+function renderArray(value: readonly unknown[], depth: number, out: Output): void {
+	if (value.length === 0) {
+		emitAt(out, "[]", depth);
+		return;
+	}
+	if (isRecordList(value)) {
+		renderTable(value, depth, out);
+		return;
+	}
+	if (isScalarList(value)) {
+		for (const item of value) if (!emitAt(out, `- ${scalarText(item)}`, depth)) return;
+		return;
+	}
+	// Mixed shapes keep their order and their nesting rather than pretending to be
+	// a table they are not.
+	for (const item of value) {
+		if (!emitAt(out, "-", depth)) return;
+		renderValue(item, depth + 1, out);
+		if (out.cut) return;
+	}
+}
+
+function renderJson(body: string): string | undefined {
+	let value: unknown;
+	try {
+		value = JSON.parse(body);
+	} catch {
+		return undefined;
+	}
+	const out = createOutput();
+	renderValue(value, 0, out);
+	return finish(out, body);
+}
+
+/**
+ * RFC 4180: commas between fields, a quoted field may hold commas and line breaks,
+ * and a doubled quote inside one is a literal quote. An unterminated quote means
+ * this is not CSV we can read, so the caller keeps the raw body.
+ */
+function parseCsv(body: string): string[][] | undefined {
+	const rows: string[][] = [];
+	let row: string[] = [];
+	let field = "";
+	let quoted = false;
+
+	for (let i = 0; i < body.length; i++) {
+		const char = body[i];
+		if (quoted) {
+			if (char !== '"') field += char;
+			else if (body[i + 1] === '"') {
+				field += '"';
+				i++;
+			} else quoted = false;
+			continue;
+		}
+		if (char === '"') quoted = true;
+		else if (char === ",") {
+			row.push(field);
+			field = "";
+		} else if (char === "\n" || char === "\r") {
+			if (char === "\r" && body[i + 1] === "\n") i++;
+			row.push(field);
+			rows.push(row);
+			row = [];
+			field = "";
+		} else field += char;
+	}
+	if (quoted) return undefined;
+	if (field !== "" || row.length > 0) {
+		row.push(field);
+		rows.push(row);
+	}
+
+	const filled = rows.filter((cells) => cells.some((cell) => cell.trim() !== ""));
+	const columns = filled.reduce((widest, cells) => Math.max(widest, cells.length), 0);
+	return filled.length >= 2 && columns >= 2 ? filled : undefined;
+}
+
+function renderCsv(body: string): string | undefined {
+	const rows = parseCsv(body);
+	if (!rows) return undefined;
+	const out = createOutput();
+	renderGrid(rows, 0, out);
+	return finish(out, body);
+}
+
+/** One element: its attributes, its children, and the text directly inside it. */
+interface XmlNode {
+	name: string;
+	attributes: Array<readonly [string, string]>;
+	children: XmlNode[];
+	text: string;
+}
+
+/** Attributes of one tag; a bare attribute carries no value to show, so it is left out. */
+function parseAttributes(source: string): Array<readonly [string, string]> {
+	const attributes: Array<readonly [string, string]> = [];
+	for (const match of source.matchAll(/([^\s=]+)\s*=\s*("([^"]*)"|'([^']*)')/g)) {
+		attributes.push([match[1], decodeEntities(match[3] ?? match[4] ?? "")]);
+	}
+	return attributes;
+}
+
+/** Index just past `end` where it closes a construct opened at `from`, or -1. */
+function after(body: string, from: number, end: string): number {
+	const at = body.indexOf(end, from + 1);
+	return at < 0 ? -1 : at + end.length;
+}
+
+/**
+ * The smallest XML reader that answers "what is the tree": tags, attributes, text,
+ * comments, CDATA, and the declaration. It returns `undefined` on anything it
+ * cannot balance — two roots, a stray close, an unterminated quote or comment — so
+ * a body that is not really XML keeps its raw form.
+ */
+function parseXml(body: string): XmlNode | undefined {
+	const stack: XmlNode[] = [];
+	let root: XmlNode | undefined;
+	let index = 0;
+
+	const appendText = (text: string): void => {
+		const node = stack[stack.length - 1];
+		if (node) node.text += text;
+	};
+
+	while (index < body.length) {
+		const open = body.indexOf("<", index);
+		if (open < 0) {
+			appendText(decodeEntities(body.slice(index)));
+			break;
+		}
+		if (open > index) appendText(decodeEntities(body.slice(index, open)));
+
+		if (body.startsWith("<!--", open)) {
+			const next = after(body, open, "-->");
+			if (next < 0) return undefined;
+			index = next;
+			continue;
+		}
+		if (body.startsWith("<![CDATA[", open)) {
+			const at = body.indexOf("]]>", open);
+			if (at < 0) return undefined;
+			appendText(body.slice(open + "<![CDATA[".length, at));
+			index = at + "]]>".length;
+			continue;
+		}
+		if (body.startsWith("<?", open) || body.startsWith("<!")) {
+			const next = after(body, open, body.startsWith("<?", open) ? "?>" : ">");
+			if (next < 0) return undefined;
+			index = next;
+			continue;
+		}
+
+		const close = body.indexOf(">", open + 1);
+		if (close < 0) return undefined;
+		const tag = body.slice(open + 1, close).trim();
+		if (tag.startsWith("/")) {
+			const node = stack.pop();
+			if (!node || node.name !== tag.slice(1).trim()) return undefined;
+			index = close + 1;
+			continue;
+		}
+
+		const selfClosing = tag.endsWith("/");
+		const inner = selfClosing ? tag.slice(0, -1).trim() : tag;
+		const name = /^[^\s/>]+/.exec(inner)?.[0];
+		if (!name) return undefined;
+		const node: XmlNode = { name, attributes: parseAttributes(inner.slice(name.length)), children: [], text: "" };
+		const parent = stack[stack.length - 1];
+		if (parent) parent.children.push(node);
+		else if (root) return undefined;
+		else root = node;
+		if (!selfClosing) stack.push(node);
+		index = close + 1;
+	}
+
+	if (stack.length > 0 || !root) return undefined;
+	return root;
+}
+
+/**
+ * One element as an indented block: a leaf with only text reads `name: text`,
+ * anything with attributes or children gets a line of its own and indented detail.
+ */
+function renderXmlNode(node: XmlNode, depth: number, out: Output): void {
+	const text = collapse(node.text);
+	if (node.children.length === 0 && node.attributes.length === 0) {
+		emitAt(out, text ? `${node.name}: ${text}` : node.name, depth);
+		return;
+	}
+	if (!emitAt(out, node.name, depth)) return;
+	for (const [name, value] of node.attributes) if (!emitAt(out, `@${name}: ${value}`, depth + 1)) return;
+	for (const child of node.children) {
+		renderXmlNode(child, depth + 1, out);
+		if (out.cut) return;
+	}
+	if (text) emitAt(out, text, depth + 1);
+}
+
+function renderXml(body: string): string | undefined {
+	const root = parseXml(body);
+	if (!root) return undefined;
+	const out = createOutput();
+	renderXmlNode(root, 0, out);
+	return finish(out, body);
+}

package/extensions/web-access/fetch/tool.ts

@@ -11,6 +11,7 @@
 
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { formatTokens } from "../../../lib/format.ts";
+import { WEB_FETCH_TOOL } from "../../../lib/tool-names.ts";
 import { reuseText } from "../../../lib/tui/chrome.ts";
 import {
 	errorLine,
@@ -34,7 +35,7 @@ import { resolveRequest, WebFetchOutput, WebFetchParams, type FetchResponse, typ
 import type { FetchRequest } from "./types.ts";
 import { loadFetchConfig } from "../config.ts";
 
-export const TOOL_NAME = "web_fetch";
+export const TOOL_NAME = WEB_FETCH_TOOL;
 
 /** The host shown while a fetch is in flight, or `page` for an unparseable URL. */
 function hostOf(raw: string | undefined): string {

package/extensions/web-access/search/tool.ts

@@ -9,6 +9,7 @@
 
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { WEB_SEARCH_PROVIDER_CHANNEL } from "../../../lib/events.ts";
+import { WEB_SEARCH_TOOL } from "../../../lib/tool-names.ts";
 import { loadSearchConfig } from "../config.ts";
 import { oneLine } from "../transcript.ts";
 import { reuseText } from "../../../lib/tui/chrome.ts";
@@ -27,7 +28,7 @@ import { formatResults } from "./format.ts";
 import { createThrottle, runSearch, type SearchDeps } from "./search.ts";
 import { resolveRequest, WebSearchOutput, WebSearchParams, type SearchResponse, type WebSearchArgs } from "./schema.ts";
 
-export const TOOL_NAME = "web_search";
+export const TOOL_NAME = WEB_SEARCH_TOOL;
 
 export function registerSearchTool(pi: ExtensionAPI, deps: SearchDeps = {}): void {
 	// One throttle per extension instance, so repeated searches stay polite