@xynogen/pix-9router 0.3.7 → 0.3.9

package/package.json

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

package/src/fetch.ts

@@ -6,15 +6,12 @@
  *   ROUTER_API_KEY   — bearer token for the router
  */
 
-import { type ExecFileException, execFile } from "node:child_process";
 import { StringEnum } from "@earendil-works/pi-ai";
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { Type } from "typebox";
-import { routerBaseUrl } from "./data.js";
+import { apiPost, curl, isCancelled } from "./http.js";
 import { makeRenderCall, makeRenderResult } from "./render.js";
 
-const REQUEST_TIMEOUT_MS = 30_000;
-
 type RouterOutcome = "running" | "fallback" | "success" | "cancelled" | "error";
 type FetchFormat = "markdown" | "text" | "html";
 
@@ -45,62 +42,6 @@ interface FetchDependencies {
 	curl: typeof curl;
 }
 
-function auth(): string | undefined {
-	return process.env.ROUTER_API_KEY;
-}
-
-async function apiPost(
-	path: string,
-	body: Record<string, unknown>,
-	signal?: AbortSignal,
-): Promise<string> {
-	const url = `${routerBaseUrl()}${path}`;
-	const key = auth();
-	const controller = new AbortController();
-	const timeout = setTimeout(() => controller.abort(), REQUEST_TIMEOUT_MS);
-	const s = signal ? AbortSignal.any([signal, controller.signal]) : controller.signal;
-
-	try {
-		const res = await fetch(url, {
-			method: "POST",
-			headers: {
-				"Content-Type": "application/json",
-				...(key ? { Authorization: `Bearer ${key}` } : {}),
-			},
-			body: JSON.stringify(body),
-			signal: s,
-		});
-		if (!res.ok) {
-			const errText = await res.text().catch(() => "");
-			throw new Error(`API ${res.status}: ${errText.slice(0, 500)}`);
-		}
-		return await res.text();
-	} finally {
-		clearTimeout(timeout);
-	}
-}
-
-function curl(args: string[]): Promise<string> {
-	return new Promise((resolve, reject) => {
-		execFile(
-			"curl",
-			["-sS", "--connect-timeout", "10", "--max-time", "25", ...args],
-			{ maxBuffer: 10 * 1024 * 1024, timeout: 30_000 },
-			(err, stdout, stderr) => {
-				if (err) {
-					const e = err as ExecFileException;
-					const msg = e.killed
-						? "curl timed out"
-						: `curl exit ${e.code ?? "??"}: ${stderr.slice(0, 300)}`;
-					reject(new Error(msg));
-					return;
-				}
-				resolve(stdout);
-			},
-		);
-	});
-}
-
 /**
  * Extract readable page content from the exa fetch JSON envelope.
  * Shape: { provider, url, title, content: { format, text }, ... }
@@ -131,10 +72,6 @@ export function formatFetchResult(raw: string): string {
 	return header.length > 0 ? `${header.join("\n")}\n\n${text}` : text;
 }
 
-function isCancelled(error: unknown, signal: AbortSignal | undefined): boolean {
-	return signal?.aborted === true || (error instanceof DOMException && error.name === "AbortError");
-}
-
 export async function executeFetch(
 	params: FetchParams,
 	signal?: AbortSignal,

package/src/http.ts

@@ -0,0 +1,91 @@
+/**
+ * http.ts — shared 9Router HTTP plumbing.
+ *
+ * auth / apiPost / curl / isCancelled were duplicated across fetch.ts,
+ * search.ts and transcribe.ts. Extracted here; the only real differences were
+ * timeout values (audio needs longer) and curl stdin (search pipes a body),
+ * so both are parameters with the old per-file defaults.
+ */
+
+import { type ExecFileException, execFile } from "node:child_process";
+import { routerBaseUrl } from "./data.js";
+
+/** Default request timeout — audio transcription overrides with a longer one. */
+export const REQUEST_TIMEOUT_MS = 30_000;
+
+export function auth(): string | undefined {
+	return process.env.ROUTER_API_KEY;
+}
+
+/** POST a JSON body to a router path, returning the raw response text. */
+export async function apiPost(
+	path: string,
+	body: Record<string, unknown>,
+	signal?: AbortSignal,
+	timeoutMs: number = REQUEST_TIMEOUT_MS,
+): Promise<string> {
+	const url = `${routerBaseUrl()}${path}`;
+	const key = auth();
+	const controller = new AbortController();
+	const timeout = setTimeout(() => controller.abort(), timeoutMs);
+	const s = signal ? AbortSignal.any([signal, controller.signal]) : controller.signal;
+
+	try {
+		const res = await fetch(url, {
+			method: "POST",
+			headers: {
+				"Content-Type": "application/json",
+				...(key ? { Authorization: `Bearer ${key}` } : {}),
+			},
+			body: JSON.stringify(body),
+			signal: s,
+		});
+		if (!res.ok) {
+			const errText = await res.text().catch(() => "");
+			throw new Error(`API ${res.status}: ${errText.slice(0, 500)}`);
+		}
+		return await res.text();
+	} finally {
+		clearTimeout(timeout);
+	}
+}
+
+export interface CurlOptions {
+	/** curl --max-time in seconds (default 25). */
+	maxTime?: number;
+	/** child-process kill timeout in ms (default 30_000). */
+	timeoutMs?: number;
+	/** Optional stdin body piped into curl. */
+	stdin?: string;
+}
+
+/** Run curl with sane defaults, rejecting on non-zero exit or timeout. */
+export function curl(args: string[], opts: CurlOptions = {}): Promise<string> {
+	const { maxTime = 25, timeoutMs = 30_000, stdin } = opts;
+	return new Promise((resolve, reject) => {
+		const child = execFile(
+			"curl",
+			["-sS", "--connect-timeout", "10", "--max-time", String(maxTime), ...args],
+			{ maxBuffer: 10 * 1024 * 1024, timeout: timeoutMs },
+			(err, stdout, stderr) => {
+				if (err) {
+					const e = err as ExecFileException;
+					const msg = e.killed
+						? "curl timed out"
+						: `curl exit ${e.code ?? "??"}: ${stderr.slice(0, 300)}`;
+					reject(new Error(msg));
+					return;
+				}
+				resolve(stdout);
+			},
+		);
+		if (stdin && child.stdin) {
+			child.stdin.write(stdin);
+			child.stdin.end();
+		}
+	});
+}
+
+export function isCancelled(error: unknown, signal: AbortSignal | undefined): boolean {
+	return signal?.aborted === true || (error instanceof DOMException && error.name === "AbortError");
+}

package/src/render.ts

@@ -16,12 +16,12 @@ import type {
 	ToolRenderResultOptions,
 } from "@earendil-works/pi-coding-agent";
 import type { Component } from "@earendil-works/pi-tui";
-import { type CollapseState, tickCollapse } from "@xynogen/pix-data/collapse";
 import {
 	type CollapsedToolStatus,
 	formatCollapsedToolRow,
 	hideCollapsedToolCall,
 } from "@xynogen/pix-pretty/utils";
+import { type CollapseState, tickCollapse } from "@xynogen/pix-runtime/collapse";
 
 interface TextLike extends Component {
 	setText(text: string): void;

package/src/search.ts

@@ -6,15 +6,13 @@
  *   ROUTER_API_KEY   — bearer token for the router
  */
 
-import { type ExecFileException, execFile } from "node:child_process";
 import { StringEnum } from "@earendil-works/pi-ai";
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { Type } from "typebox";
 import { routerBaseUrl } from "./data.js";
+import { apiPost, auth, curl, isCancelled } from "./http.js";
 import { makeRenderCall, makeRenderResult } from "./render.js";
 
-const REQUEST_TIMEOUT_MS = 30_000;
-
 type RouterOutcome = "running" | "fallback" | "success" | "cancelled" | "error";
 type SearchType = "web" | "news";
 
@@ -44,66 +42,6 @@ interface SearchDependencies {
 	curl: typeof curl;
 }
 
-function auth(): string | undefined {
-	return process.env.ROUTER_API_KEY;
-}
-
-async function apiPost(
-	path: string,
-	body: Record<string, unknown>,
-	signal?: AbortSignal,
-): Promise<string> {
-	const url = `${routerBaseUrl()}${path}`;
-	const key = auth();
-	const controller = new AbortController();
-	const timeout = setTimeout(() => controller.abort(), REQUEST_TIMEOUT_MS);
-	const s = signal ? AbortSignal.any([signal, controller.signal]) : controller.signal;
-
-	try {
-		const res = await fetch(url, {
-			method: "POST",
-			headers: {
-				"Content-Type": "application/json",
-				...(key ? { Authorization: `Bearer ${key}` } : {}),
-			},
-			body: JSON.stringify(body),
-			signal: s,
-		});
-		if (!res.ok) {
-			const errText = await res.text().catch(() => "");
-			throw new Error(`API ${res.status}: ${errText.slice(0, 500)}`);
-		}
-		return await res.text();
-	} finally {
-		clearTimeout(timeout);
-	}
-}
-
-function curl(args: string[], stdin?: string): Promise<string> {
-	return new Promise((resolve, reject) => {
-		const child = execFile(
-			"curl",
-			["-sS", "--connect-timeout", "10", "--max-time", "25", ...args],
-			{ maxBuffer: 10 * 1024 * 1024, timeout: 30_000 },
-			(err, stdout, stderr) => {
-				if (err) {
-					const e = err as ExecFileException;
-					const msg = e.killed
-						? "curl timed out"
-						: `curl exit ${e.code ?? "??"}: ${stderr.slice(0, 300)}`;
-					reject(new Error(msg));
-					return;
-				}
-				resolve(stdout);
-			},
-		);
-		if (stdin && child.stdin) {
-			child.stdin.write(stdin);
-			child.stdin.end();
-		}
-	});
-}
-
 interface SearchResultItem {
 	title?: string | null;
 	url?: string | null;
@@ -156,10 +94,6 @@ export function formatSearchResults(raw: string): string {
 	return parseSearchResults(raw).text;
 }
 
-function isCancelled(error: unknown, signal: AbortSignal | undefined): boolean {
-	return signal?.aborted === true || (error instanceof DOMException && error.name === "AbortError");
-}
-
 export async function executeSearch(
 	params: SearchParams,
 	signal?: AbortSignal,

package/src/transcribe.ts

@@ -11,7 +11,6 @@
  *   ROUTER_API_KEY   — bearer token for the router
  */
 
-import { type ExecFileException, execFile } from "node:child_process";
 import { constants as fsConstants } from "node:fs";
 import { access, lstat, mkdir, readFile, writeFile } from "node:fs/promises";
 import { homedir } from "node:os";
@@ -19,6 +18,7 @@ import { basename, dirname, extname, isAbsolute, resolve } from "node:path";
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { Type } from "typebox";
 import { routerBaseUrl } from "./data.js";
+import { auth, curl } from "./http.js";
 import { makeRenderCall, makeRenderResult } from "./render.js";
 
 const REQUEST_TIMEOUT_MS = 120_000; // audio transcription can take longer
@@ -46,10 +46,6 @@ interface TranscribeResult {
 	isError?: boolean;
 }
 
-function auth(): string | undefined {
-	return process.env.ROUTER_API_KEY;
-}
-
 /** Map file extension to MIME type for common audio formats. */
 export function mimeType(filePath: string): string {
 	const ext = extname(filePath).toLowerCase();
@@ -106,27 +102,6 @@ async function apiMultipart(
 	}
 }
 
-function curl(args: string[]): Promise<string> {
-	return new Promise((resolve, reject) => {
-		execFile(
-			"curl",
-			["-sS", "--connect-timeout", "10", "--max-time", "120", ...args],
-			{ maxBuffer: 10 * 1024 * 1024, timeout: REQUEST_TIMEOUT_MS },
-			(err, stdout, stderr) => {
-				if (err) {
-					const e = err as ExecFileException;
-					const msg = e.killed
-						? "curl timed out"
-						: `curl exit ${e.code ?? "??"}: ${stderr.slice(0, 300)}`;
-					reject(new Error(msg));
-					return;
-				}
-				resolve(stdout);
-			},
-		);
-	});
-}
-
 /** Extract the `text` field from a JSON envelope, or return the raw string. */
 export function parseTranscriptionResponse(raw: string): string {
 	try {
@@ -473,7 +448,7 @@ export default function registerTranscribe(pi: ExtensionAPI): void {
 					`${routerBaseUrl()}/audio/transcriptions`,
 				];
 
-				const raw = await curl(curlArgs);
+				const raw = await curl(curlArgs, { maxTime: 120, timeoutMs: REQUEST_TIMEOUT_MS });
 				return await run("curl-fallback", raw);
 			} catch (curlErr: unknown) {
 				const curlMsg = curlErr instanceof Error ? curlErr.message : String(curlErr);