@asterxsk/kiln 0.3.0 → 0.4.2

package/agent/extensions/pi-web-access/promise-try.d.ts

@@ -1,7 +1,7 @@
-declare module "promise.try" {
-	function promiseTry<T>(callback: () => T | PromiseLike<T>): Promise<T>;
-	namespace promiseTry {
-		function shim(): void;
-	}
-	export default promiseTry;
-}
+declare module "promise.try" {
+	function promiseTry<T>(callback: () => T | PromiseLike<T>): Promise<T>;
+	namespace promiseTry {
+		function shim(): void;
+	}
+	export default promiseTry;
+}

package/agent/extensions/pi-web-access/query-rewrite.ts

@@ -1,51 +1,51 @@
-import { complete, type Api, type Model, type ProviderHeaders } from "@earendil-works/pi-ai/compat";
-type SummaryGenerationContext = Pick<import("@earendil-works/pi-coding-agent").ExtensionContext, "model" | "modelRegistry" | "cwd" | "isProjectTrusted">;
-import { findModelWithProviderRouting, loadEnabledModelPatterns, modelMatchesEnabledPatterns } from "./summary-model-scope.ts";
-
-async function resolveFirstAvailableModel(
-	ctx: SummaryGenerationContext,
-	candidates: Array<{ provider: string; id: string }>,
-): Promise<{ model: Model<Api>; apiKey: string; headers?: ProviderHeaders }> {
-	const enabledModelPatterns = loadEnabledModelPatterns(ctx);
-	for (const { provider, id } of candidates) {
-		const model = findModelWithProviderRouting(ctx.modelRegistry, provider, id);
-		if (!model || !modelMatchesEnabledPatterns(model, enabledModelPatterns)) continue;
-		const auth = await ctx.modelRegistry.getApiKeyAndHeaders(model);
-		if (auth.ok && auth.apiKey) return { model, apiKey: auth.apiKey, headers: auth.headers };
-	}
-	throw new Error(`No enabled model available: ${candidates.map(candidate => `${candidate.provider}/${candidate.id}`).join(", ")}`);
-}
-
-export async function rewriteSearchQuery(
-	query: string,
-	ctx: SummaryGenerationContext,
-	signal: AbortSignal,
-): Promise<string> {
-	const { model, apiKey, headers } = await resolveFirstAvailableModel(ctx, [
-		{ provider: "anthropic", id: "claude-haiku-4-5" },
-		{ provider: "google", id: "gemini-3.6-flash" },
-		{ provider: "openai", id: "gpt-5-mini" },
-	]);
-	const registry = ctx.modelRegistry as typeof ctx.modelRegistry & { complete?: typeof complete };
-	const usesRegistryComplete = typeof registry.complete === "function";
-	const completeFn = usesRegistryComplete ? registry.complete!.bind(registry) : complete;
-	const response = await completeFn(
-		model,
-		{
-			messages: [{
-				role: "user",
-				content: [{ type: "text", text: `Rewrite this web search query to get better, more specific results. Add relevant year qualifiers, precise technical terms, and specificity. Return ONLY the improved query text, nothing else.\n\nQuery: ${query}` }],
-				timestamp: Date.now(),
-			}],
-		},
-		usesRegistryComplete ? { signal } : { apiKey, headers, signal },
-	);
-	if (response.stopReason === "aborted") throw new Error("Aborted");
-	const contentParts = Array.isArray(response.content) ? response.content : [];
-	const text = contentParts
-		.map((part: unknown) => part && typeof part === "object" && (part as { type?: unknown }).type === "text" && typeof (part as { text?: unknown }).text === "string" ? (part as { text: string }).text : "")
-		.join("")
-		.trim();
-	if (!text) throw new Error("Rewrite returned empty response");
-	return text;
-}
+import { complete, type Api, type Model, type ProviderHeaders } from "@earendil-works/pi-ai/compat";
+type SummaryGenerationContext = Pick<import("@earendil-works/pi-coding-agent").ExtensionContext, "model" | "modelRegistry" | "cwd" | "isProjectTrusted">;
+import { findModelWithProviderRouting, loadEnabledModelPatterns, modelMatchesEnabledPatterns } from "./summary-model-scope.ts";
+
+async function resolveFirstAvailableModel(
+	ctx: SummaryGenerationContext,
+	candidates: Array<{ provider: string; id: string }>,
+): Promise<{ model: Model<Api>; apiKey: string; headers?: ProviderHeaders }> {
+	const enabledModelPatterns = loadEnabledModelPatterns(ctx);
+	for (const { provider, id } of candidates) {
+		const model = findModelWithProviderRouting(ctx.modelRegistry, provider, id);
+		if (!model || !modelMatchesEnabledPatterns(model, enabledModelPatterns)) continue;
+		const auth = await ctx.modelRegistry.getApiKeyAndHeaders(model);
+		if (auth.ok && auth.apiKey) return { model, apiKey: auth.apiKey, headers: auth.headers };
+	}
+	throw new Error(`No enabled model available: ${candidates.map(candidate => `${candidate.provider}/${candidate.id}`).join(", ")}`);
+}
+
+export async function rewriteSearchQuery(
+	query: string,
+	ctx: SummaryGenerationContext,
+	signal: AbortSignal,
+): Promise<string> {
+	const { model, apiKey, headers } = await resolveFirstAvailableModel(ctx, [
+		{ provider: "anthropic", id: "claude-haiku-4-5" },
+		{ provider: "google", id: "gemini-3.6-flash" },
+		{ provider: "openai", id: "gpt-5-mini" },
+	]);
+	const registry = ctx.modelRegistry as typeof ctx.modelRegistry & { complete?: typeof complete };
+	const usesRegistryComplete = typeof registry.complete === "function";
+	const completeFn = usesRegistryComplete ? registry.complete!.bind(registry) : complete;
+	const response = await completeFn(
+		model,
+		{
+			messages: [{
+				role: "user",
+				content: [{ type: "text", text: `Rewrite this web search query to get better, more specific results. Add relevant year qualifiers, precise technical terms, and specificity. Return ONLY the improved query text, nothing else.\n\nQuery: ${query}` }],
+				timestamp: Date.now(),
+			}],
+		},
+		usesRegistryComplete ? { signal } : { apiKey, headers, signal },
+	);
+	if (response.stopReason === "aborted") throw new Error("Aborted");
+	const contentParts = Array.isArray(response.content) ? response.content : [];
+	const text = contentParts
+		.map((part: unknown) => part && typeof part === "object" && (part as { type?: unknown }).type === "text" && typeof (part as { text?: unknown }).text === "string" ? (part as { text: string }).text : "")
+		.join("")
+		.trim();
+	if (!text) throw new Error("Rewrite returned empty response");
+	return text;
+}

package/agent/extensions/pi-web-access/render-search-error.ts

@@ -1,170 +1,170 @@
-/**
- * Pure, dependency-free renderer for web_search error / cancel results.
- *
- * WHY THIS EXISTS: the upstream web_search `renderResult` (index.ts) early-returns
- * a SINGLE line on the error/cancel path, before the collapsed/expanded branch.
- * That makes Ctrl+O (app.tools.expand) flip `expanded` with zero visible effect —
- * a dead-end with no diagnostics. It also discards the partial search results
- * gathered before the cancel, so even an expandable view would have nothing to show.
- *
- * This module is the single source of truth for the error/cancel render PLAN
- * (plain strings, no theme/ANSI), so it is unit-testable without pi's runtime
- * (@mariozechner/pi-tui Text/Box). index.ts.renderResult delegates to it and only
- * applies theme colors + creates Text/Box components.
- *
- * Contract for callers:
- *   const plan = buildSearchErrorPlan(details);
- *   if (plan === null) ... // not an error/cancel result; use the normal renderer
- *   // collapsed: [statusLine, ...plan.collapsed, plan.expandHint].filter(Boolean)
- *   // expanded:  plan.expanded
- */
-
-export interface CancelledQueryDetail {
-	query: string;
-	provider: string | null;
-	error: string | null; // null = completed ok; string = per-query error
-	resultCount: number;
-}
-
-export interface SearchErrorDetails {
-	/** The headline error/cancel message, e.g. "Search cancelled (stale)." */
-	error?: string;
-	cancelled?: boolean;
-	cancelReason?: string;
-	/** Did the search ever establish a connection / make progress? */
-	browserConnected?: boolean;
-	/** Age (ms) of the last progress heartbeat at cancel time, if known. */
-	lastHeartbeatAgeMs?: number | null;
-	/** Total queries the user requested. */
-	queryCount?: number;
-	/** Partial per-query results gathered before the cancel/error. */
-	cancelledQueries?: CancelledQueryDetail[];
-	/** Arbitrary extra diagnostic lines (e.g. URLs, response id) for non-cancel errors
-	 * like fetch_content / get_search_content. Shown in the expanded view. */
-	extraLines?: string[];
-}
-
-export interface SearchErrorPlan {
-	/** Full diagnostic block, shown when expanded (Ctrl+O). */
-	expanded: string[];
-	/** Short preview lines, shown under the headline when collapsed. */
-	collapsed: string[];
-	/** The "... (N more lines, ctrl+o to expand)" hint, or null if nothing is hidden. */
-	expandHint: string | null;
-}
-
-function truncate(text: string, max: number): string {
-	return text.length > max ? text.slice(0, max - 1) + "\u2026" : text;
-}
-
-/**
- * Build the error/cancel render plan. Returns null when `details` carries no
- * error/cancel signal (so the caller falls through to the normal success renderer).
- */
-export function buildSearchErrorPlan(details: SearchErrorDetails | undefined | null): SearchErrorPlan | null {
-	if (!details || (!details.error && !details.cancelled)) {
-		return null;
-	}
-
-	const headline = details.error ?? "Search cancelled.";
-	const queries = details.cancelledQueries ?? [];
-	const queryCount = typeof details.queryCount === "number" && details.queryCount > 0
-		? details.queryCount
-		: queries.length;
-	const done = queries.length;
-	const errored = queries.filter(q => q.error).length;
-
-	// Rich diagnostics only make sense when there is something to diagnose: a
-	// cancelled result with partial data, OR a non-cancel error that carries
-	// extra detail (urls/response-id for fetch_content, the failed query for
-	// get_search_content). A bare argument error (e.g. "No URL
-	// provided") stays a clean single line -- no noise.
-	const extras = details.extraLines ?? [];
-	const rich = details.cancelled === true || queries.length > 0 || extras.length > 0;
-	if (!rich) {
-		return { expanded: [headline], collapsed: [], expandHint: null };
-	}
-
-	// --- diagnostics block (expanded): cancel-specific only. For non-cancel
-	// errors (fetch_content/get_search_content) there is no browser or
-	// query state to report beyond the queries themselves, so we skip this block and show only Details. ---
-	const expanded: string[] = [headline, ""];
-
-	if (details.cancelled === true || queries.length > 0) {
-		const diag: string[] = [];
-		if (details.cancelled) {
-			diag.push(`cancel reason   : ${details.cancelReason ?? "unknown"}`);
-		}
-		// Browser connection state — the most common stale cause (page never opened).
-		const browserLabel = details.browserConnected === undefined
-			? "unknown"
-			: details.browserConnected
-				? "connected"
-				: "never connected";
-		diag.push(`browser         : ${browserLabel}`);
-		if (typeof details.lastHeartbeatAgeMs === "number" && Number.isFinite(details.lastHeartbeatAgeMs)) {
-			diag.push(`last heartbeat  : ${Math.round(details.lastHeartbeatAgeMs / 1000)}s ago`);
-		}
-		if (queryCount > 0) {
-			diag.push(`queries started : ${queryCount}`);
-			diag.push(`queries done    : ${done}`);
-			if (errored > 0) diag.push(`queries errored : ${errored}`);
-		}
-		expanded.push("Diagnostics:");
-		for (const line of diag) expanded.push(`  ${line}`);
-	}
-
-	// --- per-query results gathered before cancel ---
-	if (queries.length > 0) {
-		expanded.push("");
-		expanded.push("Per-query results (gathered before cancel):");
-		for (const q of queries) {
-			const dq = truncate(q.query, 52);
-			const tag = q.error ? "[err] " : "[ok]  ";
-			const provider = q.provider ? ` (${q.provider})` : "";
-			const tail = q.error
-				? `\u2014 ${truncate(q.error, 60)}`
-				: `\u2014 ${q.resultCount} source${q.resultCount === 1 ? "" : "s"}`;
-			expanded.push(`  ${tag}"${dq}"${provider} ${tail}`);
-		}
-	}
-
-	// --- arbitrary extra detail (non-cancel errors) ---
-	if (extras.length > 0) {
-		expanded.push("");
-		expanded.push("Details:");
-		for (const e of extras) expanded.push(`  ${e}`);
-	}
-
-	// --- collapsed preview ---
-	const collapsed: string[] = [];
-	const parts: string[] = [];
-	if (queryCount > 0) {
-		parts.push(`${done}/${queryCount} queries completed`);
-	}
-	if (errored > 0) {
-		parts.push(`${errored} errored`);
-	}
-	if (details.browserConnected === false) {
-		parts.push("browser never connected");
-	} else if (details.cancelReason) {
-		parts.push(`reason: ${details.cancelReason}`);
-	}
-	if (parts.length > 0) {
-		collapsed.push(parts.join("; ") + ".");
-	}
-	// For non-cancel errors with extra detail, preview the first detail line.
-	if (collapsed.length === 0 && extras.length > 0) {
-		for (const e of extras.slice(0, 2)) {
-			collapsed.push(truncate(e, 100));
-		}
-	}
-
-	// --- expand hint ---
-	const hiddenLines = Math.max(0, expanded.length - (1 + collapsed.length)); // headline + preview shown when collapsed
-	const expandHint = hiddenLines > 0
-		? `... (${hiddenLines} more lines, ${expanded.length} total, ctrl+o to expand)`
-		: null;
-
-	return { expanded, collapsed, expandHint };
-}
+/**
+ * Pure, dependency-free renderer for web_search error / cancel results.
+ *
+ * WHY THIS EXISTS: the upstream web_search `renderResult` (index.ts) early-returns
+ * a SINGLE line on the error/cancel path, before the collapsed/expanded branch.
+ * That makes Ctrl+O (app.tools.expand) flip `expanded` with zero visible effect —
+ * a dead-end with no diagnostics. It also discards the partial search results
+ * gathered before the cancel, so even an expandable view would have nothing to show.
+ *
+ * This module is the single source of truth for the error/cancel render PLAN
+ * (plain strings, no theme/ANSI), so it is unit-testable without pi's runtime
+ * (@mariozechner/pi-tui Text/Box). index.ts.renderResult delegates to it and only
+ * applies theme colors + creates Text/Box components.
+ *
+ * Contract for callers:
+ *   const plan = buildSearchErrorPlan(details);
+ *   if (plan === null) ... // not an error/cancel result; use the normal renderer
+ *   // collapsed: [statusLine, ...plan.collapsed, plan.expandHint].filter(Boolean)
+ *   // expanded:  plan.expanded
+ */
+
+export interface CancelledQueryDetail {
+	query: string;
+	provider: string | null;
+	error: string | null; // null = completed ok; string = per-query error
+	resultCount: number;
+}
+
+export interface SearchErrorDetails {
+	/** The headline error/cancel message, e.g. "Search cancelled (stale)." */
+	error?: string;
+	cancelled?: boolean;
+	cancelReason?: string;
+	/** Did the search ever establish a connection / make progress? */
+	browserConnected?: boolean;
+	/** Age (ms) of the last progress heartbeat at cancel time, if known. */
+	lastHeartbeatAgeMs?: number | null;
+	/** Total queries the user requested. */
+	queryCount?: number;
+	/** Partial per-query results gathered before the cancel/error. */
+	cancelledQueries?: CancelledQueryDetail[];
+	/** Arbitrary extra diagnostic lines (e.g. URLs, response id) for non-cancel errors
+	 * like fetch_content / get_search_content. Shown in the expanded view. */
+	extraLines?: string[];
+}
+
+export interface SearchErrorPlan {
+	/** Full diagnostic block, shown when expanded (Ctrl+O). */
+	expanded: string[];
+	/** Short preview lines, shown under the headline when collapsed. */
+	collapsed: string[];
+	/** The "... (N more lines, ctrl+o to expand)" hint, or null if nothing is hidden. */
+	expandHint: string | null;
+}
+
+function truncate(text: string, max: number): string {
+	return text.length > max ? text.slice(0, max - 1) + "\u2026" : text;
+}
+
+/**
+ * Build the error/cancel render plan. Returns null when `details` carries no
+ * error/cancel signal (so the caller falls through to the normal success renderer).
+ */
+export function buildSearchErrorPlan(details: SearchErrorDetails | undefined | null): SearchErrorPlan | null {
+	if (!details || (!details.error && !details.cancelled)) {
+		return null;
+	}
+
+	const headline = details.error ?? "Search cancelled.";
+	const queries = details.cancelledQueries ?? [];
+	const queryCount = typeof details.queryCount === "number" && details.queryCount > 0
+		? details.queryCount
+		: queries.length;
+	const done = queries.length;
+	const errored = queries.filter(q => q.error).length;
+
+	// Rich diagnostics only make sense when there is something to diagnose: a
+	// cancelled result with partial data, OR a non-cancel error that carries
+	// extra detail (urls/response-id for fetch_content, the failed query for
+	// get_search_content). A bare argument error (e.g. "No URL
+	// provided") stays a clean single line -- no noise.
+	const extras = details.extraLines ?? [];
+	const rich = details.cancelled === true || queries.length > 0 || extras.length > 0;
+	if (!rich) {
+		return { expanded: [headline], collapsed: [], expandHint: null };
+	}
+
+	// --- diagnostics block (expanded): cancel-specific only. For non-cancel
+	// errors (fetch_content/get_search_content) there is no browser or
+	// query state to report beyond the queries themselves, so we skip this block and show only Details. ---
+	const expanded: string[] = [headline, ""];
+
+	if (details.cancelled === true || queries.length > 0) {
+		const diag: string[] = [];
+		if (details.cancelled) {
+			diag.push(`cancel reason   : ${details.cancelReason ?? "unknown"}`);
+		}
+		// Browser connection state — the most common stale cause (page never opened).
+		const browserLabel = details.browserConnected === undefined
+			? "unknown"
+			: details.browserConnected
+				? "connected"
+				: "never connected";
+		diag.push(`browser         : ${browserLabel}`);
+		if (typeof details.lastHeartbeatAgeMs === "number" && Number.isFinite(details.lastHeartbeatAgeMs)) {
+			diag.push(`last heartbeat  : ${Math.round(details.lastHeartbeatAgeMs / 1000)}s ago`);
+		}
+		if (queryCount > 0) {
+			diag.push(`queries started : ${queryCount}`);
+			diag.push(`queries done    : ${done}`);
+			if (errored > 0) diag.push(`queries errored : ${errored}`);
+		}
+		expanded.push("Diagnostics:");
+		for (const line of diag) expanded.push(`  ${line}`);
+	}
+
+	// --- per-query results gathered before cancel ---
+	if (queries.length > 0) {
+		expanded.push("");
+		expanded.push("Per-query results (gathered before cancel):");
+		for (const q of queries) {
+			const dq = truncate(q.query, 52);
+			const tag = q.error ? "[err] " : "[ok]  ";
+			const provider = q.provider ? ` (${q.provider})` : "";
+			const tail = q.error
+				? `\u2014 ${truncate(q.error, 60)}`
+				: `\u2014 ${q.resultCount} source${q.resultCount === 1 ? "" : "s"}`;
+			expanded.push(`  ${tag}"${dq}"${provider} ${tail}`);
+		}
+	}
+
+	// --- arbitrary extra detail (non-cancel errors) ---
+	if (extras.length > 0) {
+		expanded.push("");
+		expanded.push("Details:");
+		for (const e of extras) expanded.push(`  ${e}`);
+	}
+
+	// --- collapsed preview ---
+	const collapsed: string[] = [];
+	const parts: string[] = [];
+	if (queryCount > 0) {
+		parts.push(`${done}/${queryCount} queries completed`);
+	}
+	if (errored > 0) {
+		parts.push(`${errored} errored`);
+	}
+	if (details.browserConnected === false) {
+		parts.push("browser never connected");
+	} else if (details.cancelReason) {
+		parts.push(`reason: ${details.cancelReason}`);
+	}
+	if (parts.length > 0) {
+		collapsed.push(parts.join("; ") + ".");
+	}
+	// For non-cancel errors with extra detail, preview the first detail line.
+	if (collapsed.length === 0 && extras.length > 0) {
+		for (const e of extras.slice(0, 2)) {
+			collapsed.push(truncate(e, 100));
+		}
+	}
+
+	// --- expand hint ---
+	const hiddenLines = Math.max(0, expanded.length - (1 + collapsed.length)); // headline + preview shown when collapsed
+	const expandHint = hiddenLines > 0
+		? `... (${hiddenLines} more lines, ${expanded.length} total, ctrl+o to expand)`
+		: null;
+
+	return { expanded, collapsed, expandHint };
+}