@xynogen/pix-9router 0.4.0 → 0.4.3

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-9router",
-	"version": "0.4.0",
+	"version": "0.4.3",
 	"description": "Pi extension — 9Router provider + fetch/search tools via router API",
 	"type": "module",
 	"main": "src/index.ts",
@@ -43,7 +43,7 @@
 		"@earendil-works/pi-tui": "*"
 	},
 	"dependencies": {
-		"@xynogen/pix-pretty": "^1.11.2",
+		"@xynogen/pix-pretty": "^1.19.0",
 		"typebox": "^1.1.38",
 		"@xynogen/pix-runtime": "^0.8.0"
 	}

package/src/fetch.ts

@@ -206,9 +206,7 @@ export default function registerFetch(pi: ExtensionAPI): void {
 		promptSnippet:
 			"fetch(url, format, max_characters?) — format: 'markdown', 'text', or 'html'. Read page content via exa.",
 		promptGuidelines: [
-			"Use fetch when you need to read the full content of a specific URL.",
-			"Prefer format='markdown' for readable content, 'text' for plain text extraction, 'html' for raw source.",
-			"Set max_characters to cap response size (default 1000, 0 = unlimited). Use 5000-10000 for typical pages.",
+			"fetch: format='markdown' for readable content, 'text' for plain extraction, 'html' for raw source. max_characters caps size (default 1000, 0 = unlimited; 5000-10000 for typical pages).",
 		],
 		renderCall: makeRenderCall<unknown>("fetch", (args) =>
 			String((args as Partial<FetchParams>).url ?? ""),

package/src/provider.ts

@@ -11,6 +11,7 @@
 
 import type { RefreshModelsContext } from "@earendil-works/pi-ai";
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+import { ioTimeoutSignal } from "@xynogen/pix-runtime/io";
 import type { ModelsDevModel, RouterModel } from "./data.js";
 import { fetchModelsDevIndex, lookupInIndex, routerBaseUrl, routerModels } from "./data.js";
 
@@ -147,7 +148,7 @@ function providerConfig(
 		async refreshModels({ signal, allowNetwork }: RefreshModelsContext) {
 			if (!allowNetwork) return models.map(toModelConfig(devIndex));
 			const res = await fetch(`${routerBaseUrl()}/models`, {
-				signal: AbortSignal.any([AbortSignal.timeout(8_000), ...(signal ? [signal] : [])]),
+				signal: ioTimeoutSignal(signal),
 				headers: {
 					Authorization: `Bearer ${apiKey}`,
 					"User-Agent": "pi-coding-agent",

package/src/render.test.ts

@@ -13,7 +13,7 @@ function stubComponent() {
 		setText: (s: string) => {
 			captured = s;
 		},
-		render: () => [],
+		render: (_width?: number) => captured.split("\n"),
 		invalidate: () => {},
 	};
 	return { component, get: () => captured };
@@ -89,13 +89,14 @@ test("partial updates keep detail visible and do not schedule collapse", () => {
 		context(stub, { state }),
 	);
 	expect(stub.get()).toContain("streaming body");
+	expect(stub.component.render(20)).toEqual(["  [dim]streaming body"]);
 	expect(state.timer).toBeUndefined();
 });
 
 test("expanded mode restores full joined text", () => {
 	const stub = stubComponent();
 	const rr = makeRenderResult(config);
-	rr(
+	const component = rr(
 		{
 			content: [
 				{ type: "text", text: "first" },
@@ -109,6 +110,54 @@ test("expanded mode restores full joined text", () => {
 		context(stub, { state: { collapsed: true }, expanded: true }),
 	);
 	expect(stub.get()).toBe("  [dim]first\n  [dim]second");
+	expect(component.render(20)).toEqual([
+		"[success]────────────────────",
+		"  [dim]first",
+		"  [dim]second",
+		"[success]────────────────────",
+	]);
+});
+
+test("structured errors get red open frames but collapsed summaries stay unframed", () => {
+	const rr = makeRenderResult({ ...config, status: () => "error" as const });
+	const open = stubComponent();
+	const openComponent = rr(
+		{
+			content: [{ type: "text", text: "failed body" }],
+			details: { _type: "fetchResult", chars: 11 },
+		} as never,
+		{ expanded: false, isPartial: false },
+		theme,
+		context(open),
+	);
+	expect(openComponent.render(20)[0]).toBe("[error]────────────────────");
+
+	const collapsed = stubComponent();
+	const collapsedComponent = rr(
+		{
+			content: [{ type: "text", text: "failed body" }],
+			details: { _type: "fetchResult", chars: 11 },
+		} as never,
+		{ expanded: false, isPartial: false },
+		theme,
+		context(collapsed, { state: { collapsed: true } }),
+	);
+	expect(collapsedComponent.render(20).join("\n")).not.toContain("────");
+});
+
+test("terminal error body gets an error frame", () => {
+	const stub = stubComponent();
+	const component = makeRenderResult(config)(
+		{ content: [{ type: "text", text: "boom" }] } as never,
+		{ expanded: true, isPartial: false },
+		theme,
+		context(stub, { isError: true, expanded: true }),
+	);
+	expect(component.render(20)).toEqual([
+		"[error]────────────────────",
+		"  [error]boom",
+		"[error]────────────────────",
+	]);
 });
 
 test("renderResult caps the normal preview", () => {

package/src/render.ts

@@ -19,7 +19,9 @@ import type { Component } from "@earendil-works/pi-tui";
 import {
 	type CollapsedToolStatus,
 	formatCollapsedToolRow,
+	frameToolResult,
 	hideCollapsedToolCall,
+	unframeToolResult,
 } from "@xynogen/pix-pretty/utils";
 import { type CollapseState, tickCollapse } from "@xynogen/pix-runtime/collapse";
 
@@ -59,7 +61,8 @@ try {
 }
 
 function getText(lastComponent: Component | undefined): TextLike {
-	if (lastComponent && "setText" in lastComponent) return lastComponent as TextLike;
+	if (lastComponent && "setText" in lastComponent)
+		return unframeToolResult(lastComponent as TextLike);
 	// TextComponent is always present in the interactive TUI (pi-tui peer); the
 	// require() guard only matters for headless/test contexts where renderers
 	// are never invoked.
@@ -115,14 +118,15 @@ export function makeRenderResult<TDetails>(config?: CompactRendererConfig<TDetai
 
 		if (ctx.isError) {
 			text.setText(`  ${theme.fg("error", body || "Error")}`);
-			return text;
+			return opts.isPartial ? text : frameToolResult(text, theme, true);
 		}
 		if (!body.trim()) {
 			text.setText(`  ${theme.fg("muted", "(empty)")}`);
-			return text;
+			return opts.isPartial ? text : frameToolResult(text, theme, false);
 		}
 
 		const details = result.details as TDetails | undefined;
+		const isError = Boolean(details && config?.status?.(details) === "error");
 		if (
 			!opts.isPartial &&
 			config &&
@@ -142,6 +146,6 @@ export function makeRenderResult<TDetails>(config?: CompactRendererConfig<TDetai
 		}
 
 		text.setText(dimBody(body, theme, opts.expanded));
-		return text;
+		return opts.isPartial ? text : frameToolResult(text, theme, isError);
 	};
 }

package/src/search.ts

@@ -232,8 +232,7 @@ export default function registerSearch(pi: ExtensionAPI): void {
 		promptSnippet:
 			"search(query, search_type, max_results?) — search_type: 'web' or 'news'. Defaults to 5 results, max 10.",
 		promptGuidelines: [
-			"Use search when you need up-to-date information or facts from the web.",
-			"Set search_type='web' for general web results, search_type='news' for recent news articles.",
+			"search: search_type='web' for general web results, 'news' for recent news articles.",
 		],
 		renderCall: makeRenderCall<unknown>("search", (args) =>
 			String((args as Partial<SearchParams>).query ?? ""),

package/src/transcribe.ts

@@ -340,14 +340,7 @@ export default function registerTranscribe(pi: ExtensionAPI): void {
 				isError: context.isError && options.expanded,
 			}),
 		promptGuidelines: [
-			"Use transcribe when you need to convert speech/audio to text.",
-			"The file parameter should be an absolute or relative path to an audio file on disk.",
-			"Supports common audio formats: mp3, wav, flac, ogg, m4a, webm, mp4.",
-			"Default model is dg/nova-3 (Deepgram Nova 3). Override with model parameter if needed.",
-			"Optionally specify language as ISO 639-1 code (e.g. 'en', 'es', 'fr') for better accuracy.",
-			"Pass output_file to write the full transcription to disk — useful when the result is long, when it will be re-read or piped to another tool, or to keep chat context small. Relative paths are resolved against the current working directory.",
-			"output_file is pre-flight checked: paths under /etc, /proc, /sys, /boot, ~/.ssh, ~/.aws, ~/.gnupg, and ~/.config/gh are rejected; the target must not be a symlink or existing directory; the parent must exist and be writable; symlinks anywhere in the parent chain are rejected. On rejection, the transcription text is still returned inline so it is not lost.",
-			"Without output_file the transcribed text is returned inline (truncated to 50,000 chars).",
+			"transcribe: `language` as ISO 639-1 (e.g. 'en', 'es') improves accuracy. Pass `output_file` for long results — writes full text to disk (sensitive paths/symlinks rejected; text still returned inline on rejection) and returns a path summary. Without it, text returns inline truncated to 50,000 chars.",
 		],
 		parameters: Type.Object({
 			file: Type.String({