@oh-my-pi/pi-coding-agent 15.9.5 → 15.10.0

package/src/cli/gallery-fixtures/search.ts

@@ -0,0 +1,213 @@
+/** Gallery fixtures for the search tools (search, search_tool_bm25, ast_grep). */
+import type { GalleryFixture } from "./types";
+
+export const searchFixtures: Record<string, GalleryFixture> = {
+	search: {
+		label: "Search",
+		streamingArgs: {
+			pattern: "useState",
+		},
+		args: {
+			pattern: "useState",
+			paths: ["packages/tui/src"],
+		},
+		result: {
+			content: [
+				{
+					type: "text",
+					text: [
+						"# packages/tui/src/components/",
+						"## SearchBox.tsx",
+						'18:  const [query, setQuery] = useState("");',
+						"19:  const [results, setResults] = useState<Match[]>([]);",
+						"## StatusBar.tsx",
+						"27:  const [expanded, setExpanded] = useState(false);",
+						"",
+						"# packages/tui/src/hooks/",
+						"## useDebounced.ts",
+						"9:  const [value, setValue] = useState(initial);",
+						"10:  const [pending, setPending] = useState(false);",
+					].join("\n"),
+				},
+			],
+			details: {
+				scopePath: "packages/tui/src",
+				searchPath: "/Users/dev/Projects/pi/packages/tui/src",
+				matchCount: 5,
+				fileCount: 3,
+				files: [
+					"packages/tui/src/components/SearchBox.tsx",
+					"packages/tui/src/components/StatusBar.tsx",
+					"packages/tui/src/hooks/useDebounced.ts",
+				],
+				fileMatches: [
+					{ path: "packages/tui/src/components/SearchBox.tsx", count: 2 },
+					{ path: "packages/tui/src/components/StatusBar.tsx", count: 1 },
+					{ path: "packages/tui/src/hooks/useDebounced.ts", count: 2 },
+				],
+				truncated: false,
+				displayContent: [
+					"# packages/tui/src/components/",
+					"## SearchBox.tsx",
+					'*18│  const [query, setQuery] = useState("");',
+					"*19│  const [results, setResults] = useState<Match[]>([]);",
+					"## StatusBar.tsx",
+					"*27│  const [expanded, setExpanded] = useState(false);",
+					"",
+					"# packages/tui/src/hooks/",
+					"## useDebounced.ts",
+					" *9│  const [value, setValue] = useState(initial);",
+					"*10│  const [pending, setPending] = useState(false);",
+				].join("\n"),
+			},
+		},
+		errorResult: {
+			content: [
+				{
+					type: "text",
+					text: "Invalid regex pattern: unclosed group near index 8",
+				},
+			],
+			isError: true,
+			details: {
+				error: "Invalid regex pattern: unclosed group near index 8",
+			},
+		},
+	},
+
+	search_tool_bm25: {
+		label: "SearchTools",
+		streamingArgs: {
+			query: "read pdf and ext",
+		},
+		args: {
+			query: "read pdf and extract tables",
+			limit: 5,
+		},
+		result: {
+			content: [
+				{
+					type: "text",
+					text: JSON.stringify({
+						query: "read pdf and extract tables",
+						activated_tools: ["docling_extract_tables", "docling_convert", "pdf_read_text"],
+						match_count: 4,
+						total_tools: 142,
+					}),
+				},
+			],
+			details: {
+				query: "read pdf and extract tables",
+				limit: 5,
+				total_tools: 142,
+				activated_tools: ["docling_extract_tables", "docling_convert", "pdf_read_text"],
+				active_selected_tools: ["read", "search", "edit", "bash"],
+				tools: [
+					{
+						name: "docling_extract_tables",
+						label: "Extract Tables",
+						description: "Extract tabular data from PDF documents into CSV or JSON rows.",
+						server_name: "docling",
+						mcp_tool_name: "extract_tables",
+						schema_keys: ["path", "pages", "format"],
+						score: 9.412037,
+					},
+					{
+						name: "docling_convert",
+						label: "Convert Document",
+						description: "Convert PDF, DOCX, or PPTX into structured Markdown with layout preserved.",
+						server_name: "docling",
+						mcp_tool_name: "convert",
+						schema_keys: ["path", "target", "ocr"],
+						score: 6.83102,
+					},
+					{
+						name: "pdf_read_text",
+						label: "Read PDF Text",
+						description: "Read raw text from a PDF, optionally scoped to a page range.",
+						server_name: "pdf-tools",
+						mcp_tool_name: "read_text",
+						schema_keys: ["path", "page_start", "page_end"],
+						score: 5.207884,
+					},
+					{
+						name: "tabula_scan",
+						label: "Scan Tables",
+						description: "Detect table bounding boxes on scanned PDF pages before extraction.",
+						server_name: "pdf-tools",
+						mcp_tool_name: "scan",
+						schema_keys: ["path", "dpi"],
+						score: 3.119556,
+					},
+				],
+			},
+		},
+		errorResult: {
+			content: [
+				{
+					type: "text",
+					text: "Tool discovery is disabled. Enable tools.discoveryMode or mcp.discoveryMode to use search_tool_bm25.",
+				},
+			],
+			isError: true,
+		},
+	},
+
+	ast_grep: {
+		label: "AST Grep",
+		streamingArgs: {
+			pat: "useState(",
+		},
+		args: {
+			pat: "useState($A)",
+			paths: ["packages/tui/src/components"],
+		},
+		result: {
+			content: [
+				{
+					type: "text",
+					text: [
+						"# packages/tui/src/components/",
+						"## SearchBox.tsx",
+						'18:  const [query, setQuery] = useState("");',
+						'  meta: $A=""',
+						"## StatusBar.tsx",
+						"27:  const [expanded, setExpanded] = useState(false);",
+						"  meta: $A=false",
+					].join("\n"),
+				},
+			],
+			details: {
+				matchCount: 2,
+				fileCount: 2,
+				filesSearched: 14,
+				limitReached: false,
+				scopePath: "packages/tui/src/components",
+				searchPath: "/Users/dev/Projects/pi/packages/tui/src/components",
+				files: ["packages/tui/src/components/SearchBox.tsx", "packages/tui/src/components/StatusBar.tsx"],
+				fileMatches: [
+					{ path: "packages/tui/src/components/SearchBox.tsx", count: 1 },
+					{ path: "packages/tui/src/components/StatusBar.tsx", count: 1 },
+				],
+				displayContent: [
+					"# packages/tui/src/components/",
+					"## SearchBox.tsx",
+					'*18│  const [query, setQuery] = useState("");',
+					'  meta: $A=""',
+					"## StatusBar.tsx",
+					"*27│  const [expanded, setExpanded] = useState(false);",
+					"  meta: $A=false",
+				].join("\n"),
+			},
+		},
+		errorResult: {
+			content: [
+				{
+					type: "text",
+					text: "Pattern parse error: incomplete node `useState(` — expected a closing `)`",
+				},
+			],
+			isError: true,
+		},
+	},
+};

package/src/cli/gallery-fixtures/shell.ts

@@ -0,0 +1,167 @@
+/** Gallery fixtures for the shell tools (bash, eval). */
+import type { GalleryFixture } from "./types";
+
+export const shellFixtures: Record<string, GalleryFixture> = {
+	bash: {
+		label: "Bash",
+		streamingArgs: {
+			command: "git status --short && git log --on",
+		},
+		args: {
+			command: "git status --short && git log --oneline -5",
+			cwd: "packages/coding-agent",
+			timeout: 30,
+		},
+		result: {
+			content: [
+				{
+					type: "text",
+					text: [
+						" M src/cli/gallery-cli.ts",
+						" M src/tools/bash.ts",
+						"?? src/cli/gallery-fixtures/shell.ts",
+						"a1b2c3d Wire gallery command into CLI dispatch",
+						"9f8e7d6 Add ToolExecutionComponent lifecycle states",
+						"4c5b6a7 Extract createShellRenderer from bashToolRenderer",
+						"2d3e4f5 Strip LLM-facing notices before TUI render",
+						"7a8b9c0 Cap preview lines in pending command block",
+					].join("\n"),
+				},
+			],
+			details: {
+				exitCode: 0,
+				wallTimeMs: 184,
+				timeoutSeconds: 30,
+			},
+		},
+		errorResult: {
+			content: [
+				{
+					type: "text",
+					text: [
+						"src/tools/bash.ts:1142:34 - error TS2339: Property 'requestedTimeoutSeconds' does not exist on type 'BashToolDetails'.",
+						"",
+						"1142   const requestedTimeoutSeconds = details?.requestedTimeoutSeconds;",
+						"                                            ~~~~~~~~~~~~~~~~~~~~~~~~",
+						"Found 1 error in src/tools/bash.ts:1142",
+					].join("\n"),
+				},
+			],
+			isError: true,
+			details: {
+				exitCode: 2,
+				wallTimeMs: 5120,
+				timeoutSeconds: 30,
+			},
+		},
+	},
+
+	eval: {
+		label: "Eval",
+		streamingArgs: {
+			cells: [
+				{
+					language: "py",
+					code: 'import json\nfrom pathlib import Path\n\ndata = json.loads(Path("package.js',
+					title: "load config",
+				},
+			],
+		},
+		args: {
+			cells: [
+				{
+					language: "py",
+					title: "load config",
+					code: [
+						"import json",
+						"from pathlib import Path",
+						"",
+						'data = json.loads(Path("package.json").read_text())',
+						'deps = data.get("dependencies", {})',
+						'print(f"{data[\\"name\\"]} v{data[\\"version\\"]}")',
+						'print(f"{len(deps)} dependencies")',
+						"display(sorted(deps)[:3])",
+					].join("\n"),
+				},
+			],
+		},
+		result: {
+			content: [
+				{
+					type: "text",
+					text: ["@oh-my-pi/coding-agent v0.42.0", "37 dependencies"].join("\n"),
+				},
+			],
+			details: {
+				language: "python",
+				languages: ["python"],
+				jsonOutputs: [["@ai-sdk/anthropic", "@oh-my-pi/pi-ai", "@oh-my-pi/pi-tui"]],
+				cells: [
+					{
+						index: 0,
+						title: "load config",
+						language: "python",
+						code: [
+							"import json",
+							"from pathlib import Path",
+							"",
+							'data = json.loads(Path("package.json").read_text())',
+							'deps = data.get("dependencies", {})',
+							'print(f"{data[\\"name\\"]} v{data[\\"version\\"]}")',
+							'print(f"{len(deps)} dependencies")',
+							"display(sorted(deps)[:3])",
+						].join("\n"),
+						output: ["@oh-my-pi/coding-agent v0.42.0", "37 dependencies"].join("\n"),
+						status: "complete",
+						durationMs: 64,
+						exitCode: 0,
+					},
+				],
+			},
+		},
+		errorResult: {
+			content: [
+				{
+					type: "text",
+					text: [
+						"Traceback (most recent call last):",
+						'  File "<cell 0>", line 4, in <module>',
+						'    data = json.loads(Path("package.json").read_text())',
+						"          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^",
+						"json.decoder.JSONDecodeError: Expecting ',' delimiter: line 12 column 3 (char 318)",
+					].join("\n"),
+				},
+			],
+			isError: true,
+			details: {
+				language: "python",
+				languages: ["python"],
+				isError: true,
+				cells: [
+					{
+						index: 0,
+						title: "load config",
+						language: "python",
+						code: [
+							"import json",
+							"from pathlib import Path",
+							"",
+							'data = json.loads(Path("package.json").read_text())',
+							'deps = data.get("dependencies", {})',
+							'print(f"{data[\\"name\\"]} v{data[\\"version\\"]}")',
+						].join("\n"),
+						output: [
+							"Traceback (most recent call last):",
+							'  File "<cell 0>", line 4, in <module>',
+							'    data = json.loads(Path("package.json").read_text())',
+							"json.decoder.JSONDecodeError: Expecting ',' delimiter: line 12 column 3 (char 318)",
+						].join("\n"),
+						status: "error",
+						durationMs: 41,
+						exitCode: 1,
+					},
+				],
+			},
+		},
+	},
+};

package/src/cli/gallery-fixtures/types.ts

@@ -0,0 +1,41 @@
+/**
+ * Types for `omp gallery` sample data. See {@link ./index} for the aggregated
+ * fixture registry and the contract each fixture must satisfy.
+ */
+import type { EditMode } from "../../edit";
+
+/** A tool result snapshot, matching the shape `ToolExecutionComponent` consumes. */
+export interface GalleryResult {
+	content: Array<{ type: string; text?: string; data?: string; mimeType?: string }>;
+	details?: unknown;
+	isError?: boolean;
+}
+
+export interface GalleryFixture {
+	/** Display label for the tool header (defaults to the tool name). */
+	label?: string;
+	/** Edit mode for edit-like tools so the streaming preview dispatches correctly. */
+	editMode?: EditMode;
+	/**
+	 * Set for tools whose real `AgentTool` attaches `renderCall`/`renderResult`
+	 * directly on the instance (e.g. `lsp`, `task`). The harness then attaches
+	 * the registry renderer onto the fake tool so the component routes through
+	 * the custom-tool branch — the same path production takes — instead of the
+	 * built-in registry branch. The two branches can diverge, so exercising the
+	 * real one keeps the gallery honest for these tools.
+	 */
+	customRendered?: boolean;
+	/**
+	 * Arguments shown during the streaming state — a partial view of {@link args}
+	 * as if the tool-call JSON were still arriving. May include `__partialJson`
+	 * for renderers (bash, edit) that surface fields before the object closes.
+	 * Defaults to {@link args} when omitted.
+	 */
+	streamingArgs?: unknown;
+	/** Complete arguments shown for the in-progress, success, and error states. */
+	args: unknown;
+	/** Successful result. */
+	result: GalleryResult;
+	/** Failed result. Falls back to a generic error when omitted. */
+	errorResult?: GalleryResult;
+}

package/src/cli/gallery-fixtures/web.ts

@@ -0,0 +1,158 @@
+// Gallery fixtures for the web tools (web_search, browser).
+import type { GalleryFixture } from "./types";
+
+export const webFixtures: Record<string, GalleryFixture> = {
+	web_search: {
+		label: "Web Search",
+		// Streaming: query still being typed, no recency/limit yet.
+		streamingArgs: { query: "bun vs node performance" },
+		args: {
+			query: "Bun vs Node.js performance benchmarks 2026",
+			recency: "month",
+			limit: 4,
+		},
+		result: {
+			content: [
+				{
+					type: "text",
+					text: [
+						"Bun continues to outperform Node.js on raw HTTP throughput and cold-start",
+						"time thanks to its JavaScriptCore engine and native-Zig runtime, while",
+						"Node.js retains an edge in ecosystem maturity and long-term stability.",
+						"For script-heavy workflows Bun's faster startup is the decisive factor.",
+					].join("\n"),
+				},
+			],
+			details: {
+				response: {
+					provider: "perplexity",
+					model: "sonar-pro",
+					authMode: "api_key",
+					requestId: "req_a1b2c3d4e5f6",
+					answer: [
+						"Bun continues to outperform Node.js on raw HTTP throughput and cold-start",
+						"time thanks to its JavaScriptCore engine and native-Zig runtime, while",
+						"Node.js retains an edge in ecosystem maturity and long-term stability.",
+						"For script-heavy workflows Bun's faster startup is the decisive factor.",
+					].join("\n"),
+					searchQueries: ["bun vs node.js performance benchmarks 2026", "bun http throughput vs node"],
+					sources: [
+						{
+							title: "Bun 1.2 Benchmarks: HTTP, SQLite, and Startup Time",
+							url: "https://bun.sh/blog/bun-v1.2-benchmarks",
+							snippet:
+								"Bun serves roughly 2.5x the requests per second of Node.js on a simple HTTP server and starts in under 10ms.",
+							ageSeconds: 86400 * 12,
+							author: "The Bun Team",
+						},
+						{
+							title: "Node.js vs Bun: A 2026 Performance Deep Dive",
+							url: "https://blog.platformatic.dev/nodejs-vs-bun-2026",
+							snippet:
+								"Across CPU-bound workloads the gap narrows, but Bun's faster module resolution keeps cold starts ahead.",
+							ageSeconds: 86400 * 3,
+							author: "Matteo Collina",
+						},
+						{
+							title: "Real-world API latency: Bun, Deno, and Node compared",
+							url: "https://www.theregister.com/2026/05/18/js_runtime_latency/",
+							snippet:
+								"Under sustained load p99 latencies converge, suggesting runtime choice matters less for steady-state services.",
+							ageSeconds: 86400 * 19,
+						},
+						{
+							title: "Why we migrated our CLI tooling from Node to Bun",
+							url: "https://engineering.example.com/posts/bun-cli-migration",
+							snippet:
+								"Startup dropped from 180ms to 22ms, shaving seconds off every developer command invocation.",
+							ageSeconds: 86400 * 27,
+							author: "Dana Whitfield",
+						},
+					],
+					citations: [
+						{
+							url: "https://bun.sh/blog/bun-v1.2-benchmarks",
+							title: "Bun 1.2 Benchmarks",
+							citedText: "Bun serves roughly 2.5x the requests per second of Node.js",
+						},
+					],
+					usage: {
+						inputTokens: 312,
+						outputTokens: 248,
+						totalTokens: 560,
+						searchRequests: 2,
+					},
+				},
+			},
+		},
+		errorResult: {
+			isError: true,
+			content: [{ type: "text", text: "Web search failed: provider returned HTTP 429 (rate limited)." }],
+			details: {
+				response: {
+					provider: "perplexity",
+					sources: [],
+				},
+				error: "Provider returned HTTP 429 (rate limited). Retry after 30s.",
+			},
+		},
+	},
+
+	browser: {
+		label: "Browser",
+		// Streaming: code body still arriving for a `run` action.
+		streamingArgs: {
+			action: "run",
+			name: "docs",
+			code: "const obs = await tab.observe();\n",
+		},
+		args: {
+			action: "run",
+			name: "docs",
+			code: [
+				"const obs = await tab.observe();",
+				"const heading = obs.elements.find(e => e.role === 'heading');",
+				"display({ url: obs.url, title: obs.title, headings: obs.elements.filter(e => e.role === 'heading').length });",
+				"return heading?.name ?? 'no heading found';",
+			].join("\n"),
+		},
+		result: {
+			content: [
+				{
+					type: "text",
+					text: [
+						'{ url: "https://bun.sh/docs", title: "Bun Documentation", headings: 14 }',
+						'"Get started with Bun"',
+					].join("\n"),
+				},
+			],
+			details: {
+				action: "run",
+				name: "docs",
+				url: "https://bun.sh/docs",
+				browser: "headless",
+				viewport: { width: 1280, height: 800, deviceScaleFactor: 1 },
+				result: '"Get started with Bun"',
+			},
+		},
+		errorResult: {
+			isError: true,
+			content: [
+				{
+					type: "text",
+					text: [
+						"TimeoutError: waiting for selector `aria/Sign in` failed: timeout 30000ms exceeded",
+						"    at Tab.waitFor (browser/tab.ts:212:13)",
+						"    at run (eval:3:7)",
+					].join("\n"),
+				},
+			],
+			details: {
+				action: "run",
+				name: "docs",
+				url: "https://bun.sh/docs",
+				browser: "headless",
+			},
+		},
+	},
+};