@oh-my-pi/pi-coding-agent 15.0.0 → 15.0.1

package/src/eval/js/shared/rewrite-imports.ts

@@ -180,7 +180,7 @@ export function rewriteImports(code: string): string {
  * Nested declarations (inside functions, blocks, classes) are left alone \u2014 they're
  * scoped to their enclosing function/block regardless of `var` vs `let`/`const`.
  */
-export function demoteTopLevelLexicals(code: string): string {
+function demoteTopLevelLexicals(code: string): string {
 	if (!/\b(?:const|let|class)\b/.test(code)) return code;
 
 	const ast = parseProgram(code);
@@ -248,7 +248,7 @@ function returnFinalExpression(code: string): { source: string; returned: boolea
  * common case avoids an extra transpile pass. We detect "looks like TS" with a cheap regex
  * before invoking the transpiler.
  */
-export function stripTypeScript(code: string): string {
+function stripTypeScript(code: string): string {
 	if (!LOOKS_LIKE_TS.test(code)) return code;
 	try {
 		return new Bun.Transpiler({ loader: "ts" }).transformSync(code);

package/src/eval/py/executor.ts

@@ -1,6 +1,8 @@
 import { getProjectDir, logger } from "@oh-my-pi/pi-utils";
+import { Settings } from "../../config/settings";
 import { OutputSink } from "../../session/streaming-output";
 import type { ToolSession } from "../../tools";
+import { resolveOutputMaxColumns, resolveOutputSinkHeadBytes } from "../../tools/output-meta";
 import type { JsStatusEvent } from "../js/shared/types";
 import type { KernelDisplayOutput } from "./display";
 import {
@@ -815,10 +817,13 @@ async function executeWithKernel(
 	code: string,
 	options: PythonExecutorOptions | undefined,
 ): Promise<PythonResult> {
+	const settings = await Settings.init();
 	const sink = new OutputSink({
 		onChunk: options?.onChunk,
 		artifactPath: options?.artifactPath,
 		artifactId: options?.artifactId,
+		headBytes: resolveOutputSinkHeadBytes(settings),
+		maxColumns: resolveOutputMaxColumns(settings),
 	});
 	const displayOutputs: KernelDisplayOutput[] = [];
 	const deadlineMs = getExecutionDeadlineMs(options);

package/src/exa/factory.ts

@@ -3,7 +3,7 @@
  */
 import type { TObject, TProperties } from "@sinclair/typebox";
 import type { CustomTool } from "../extensibility/custom-tools/types";
-import { callExaTool, findApiKey, formatSearchResults, isSearchResponse } from "./mcp-client";
+import { callExaTool, findApiKey, formatGenericResponse, formatSearchResults, isSearchResponse } from "./mcp-client";
 import type { ExaRenderDetails } from "./types";
 
 /** Creates an Exa tool with standardized API key handling, error wrapping, and optional search response formatting. */
@@ -44,7 +44,7 @@ export function createExaTool(
 				}
 
 				return {
-					content: [{ type: "text" as const, text: JSON.stringify(response, null, 2) }],
+					content: [{ type: "text" as const, text: formatGenericResponse(response) }],
 					details: { raw: response, toolName: name },
 				};
 			} catch (error) {

package/src/exa/mcp-client.ts

@@ -174,6 +174,79 @@ export function formatSearchResults(data: ExaSearchResponse): string {
 
 	return output.trim();
 }
+/**
+ * Format a non-search MCP response as human-readable text.
+ * Handles objects, arrays, primitives, and common MCP response shapes.
+ */
+export function formatGenericResponse(data: unknown): string {
+	if (data === null || data === undefined) return "No result.";
+	if (typeof data === "string") return data;
+	if (typeof data === "number" || typeof data === "boolean") return String(data);
+
+	if (Array.isArray(data)) {
+		if (data.length === 0) return "(empty)";
+		const parts: string[] = [];
+		for (let i = 0; i < data.length; i++) {
+			const item = data[i];
+			if (typeof item === "object" && item !== null) {
+				const record = item as Record<string, unknown>;
+				const title = (record.title ?? record.name ?? record.id ?? `Item ${i + 1}`) as string;
+				parts.push(`\n### ${title}`);
+				for (const [k, v] of Object.entries(record)) {
+					if (["title", "name", "id"].includes(k)) continue;
+					parts.push(`- **${k}:** ${formatValue(v)}`);
+				}
+			} else {
+				parts.push(`- ${formatValue(item)}`);
+			}
+		}
+		return parts.join("\n");
+	}
+
+	if (typeof data === "object") {
+		const record = data as Record<string, unknown>;
+		if (record.content && Array.isArray(record.content)) {
+			// MCP-style content array — extract text blocks
+			const texts = record.content
+				.filter(
+					(c: unknown): c is { type: string; text?: string } =>
+						typeof c === "object" && c !== null && (c as Record<string, unknown>)?.type === "text",
+				)
+				.map(c => c.text ?? "")
+				.filter(Boolean);
+			if (texts.length > 0) return texts.join("\n");
+		}
+
+		const lines: string[] = [];
+		for (const [k, v] of Object.entries(record)) {
+			if (k === "content") continue; // handled above
+			if (v === null || v === undefined) continue;
+			if (typeof v === "object") {
+				const formatted = formatGenericResponse(v);
+				if (formatted) lines.push(`- **${k}:**\n${indent(formatted, 2)}`);
+			} else {
+				lines.push(`- **${k}:** ${formatValue(v)}`);
+			}
+		}
+		return lines.join("\n") || "(empty)";
+	}
+
+	return String(data);
+}
+
+function formatValue(v: unknown): string {
+	if (v === null || v === undefined) return "—";
+	if (typeof v === "object") return JSON.stringify(v);
+	return String(v);
+}
+
+function indent(text: string, spaces: number): string {
+	const pad = " ".repeat(spaces);
+	return text
+		.split("\n")
+		.map(line => pad + line)
+		.join("\n");
+}
 
 /** Check if result is a search response */
 export function isSearchResponse(data: unknown): data is ExaSearchResponse {
@@ -260,7 +333,7 @@ export class MCPWrappedTool implements CustomTool<TSchema, ExaRenderDetails> {
 			}
 
 			return {
-				content: [{ type: "text" as const, text: JSON.stringify(response, null, 2) }],
+				content: [{ type: "text" as const, text: formatGenericResponse(response) }],
 				details: { raw: response, toolName: this.config.name },
 			};
 		} catch (error) {

package/src/exec/bash-executor.ts

@@ -7,6 +7,7 @@ import * as fs from "node:fs/promises";
 import { executeShell, type MinimizerOptions, Shell } from "@oh-my-pi/pi-natives";
 import { Settings, type ShellMinimizerSettings } from "../config/settings";
 import { OutputSink } from "../session/streaming-output";
+import { resolveOutputMaxColumns, resolveOutputSinkHeadBytes } from "../tools/output-meta";
 import { getOrCreateSnapshot } from "../utils/shell-snapshot";
 import { NON_INTERACTIVE_ENV } from "./non-interactive-env";
 
@@ -64,7 +65,8 @@ async function resolveShellCwd(cwd: string | undefined): Promise<string | undefi
 	}
 }
 
-function buildMinimizerOptions(group: ShellMinimizerSettings): MinimizerOptions | undefined {
+/** Translate `ShellMinimizerSettings` into native `MinimizerOptions`, or `undefined` when disabled. */
+export function buildMinimizerOptions(group: ShellMinimizerSettings): MinimizerOptions | undefined {
 	if (!group.enabled) return undefined;
 	return {
 		enabled: true,
@@ -94,6 +96,8 @@ export async function executeBash(command: string, options?: BashExecutorOptions
 		onChunk: options?.onChunk,
 		artifactPath: options?.artifactPath,
 		artifactId: options?.artifactId,
+		headBytes: resolveOutputSinkHeadBytes(settings),
+		maxColumns: resolveOutputMaxColumns(settings),
 		// Throttle the streaming preview callback to avoid saturating the
 		// event loop when commands produce massive output (e.g. seq 1 50M).
 		chunkThrottleMs: options?.onChunk ? 50 : 0,