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

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

@@ -0,0 +1,153 @@
+// biome-ignore-all lint/suspicious/noTemplateCurlyInString: sample source-code strings (read fixtures) intentionally contain literal ${...}.
+// Gallery fixtures for the filesystem tools (read, write, find).
+import type { GalleryFixture } from "./types";
+
+const readSnippet = [
+	"export const findToolRenderer = {",
+	"\tinline: true,",
+	"\trenderCall(args: FindRenderArgs, _options: RenderResultOptions, uiTheme: Theme): Component {",
+	"\t\tconst meta: string[] = [];",
+	"\t\tif (args.limit !== undefined) meta.push(`limit:${args.limit}`);",
+	"",
+	"\t\tconst text = renderStatusLine(",
+	'\t\t\t{ icon: "pending", title: "Find", description: formatFindRenderPaths(args.paths) || "*", meta },',
+	"\t\t\tuiTheme,",
+	"\t\t);",
+	"\t\treturn new Text(text, 0, 0);",
+	"\t},",
+].join("\n");
+
+const writtenContent = [
+	'import { describe, expect, it } from "bun:test";',
+	'import { parseSel } from "../src/tools/read";',
+	"",
+	'describe("parseSel", () => {',
+	'\tit("parses a single line range", () => {',
+	'\t\texpect(parseSel("42-58")).toEqual({',
+	'\t\t\tkind: "lines",',
+	"\t\t\tranges: [{ startLine: 42, endLine: 58 }],",
+	"\t\t});",
+	"\t});",
+	"",
+	'\tit("treats raw as a verbatim selector", () => {',
+	'\t\texpect(parseSel("raw")).toEqual({ kind: "raw" });',
+	"\t});",
+	"});",
+	"",
+].join("\n");
+
+export const fsFixtures: Record<string, GalleryFixture> = {
+	read: {
+		label: "Read",
+		// Streaming: path still being typed, selector not yet appended.
+		streamingArgs: { path: "packages/coding-agent/src/tools/find" },
+		args: { path: "packages/coding-agent/src/tools/find.ts:437-448" },
+		result: {
+			content: [
+				{
+					type: "text",
+					text: [
+						"[packages/coding-agent/src/tools/find.ts#E48E]",
+						"437:export const findToolRenderer = {",
+						"438:\tinline: true,",
+						"439:\trenderCall(args: FindRenderArgs, _options: RenderResultOptions, uiTheme: Theme): Component {",
+						"440:\t\tconst meta: string[] = [];",
+						"441:\t\tif (args.limit !== undefined) meta.push(`limit:${args.limit}`);",
+						"442:",
+						"443:\t\tconst text = renderStatusLine(",
+						'444:\t\t\t{ icon: "pending", title: "Find", description: formatFindRenderPaths(args.paths) || "*", meta },',
+						"445:\t\t\tuiTheme,",
+						"446:\t\t);",
+						"447:\t\treturn new Text(text, 0, 0);",
+						"448:\t},",
+					].join("\n"),
+				},
+			],
+			details: {
+				kind: "file",
+				resolvedPath: "/Users/dev/Projects/pi/packages/coding-agent/src/tools/find.ts",
+				contentType: "text/typescript",
+				displayContent: { text: readSnippet, startLine: 437 },
+			},
+		},
+		errorResult: {
+			isError: true,
+			content: [
+				{
+					type: "text",
+					text: "Error: ENOENT: no such file or directory, open 'packages/coding-agent/src/tools/find.ts'",
+				},
+			],
+		},
+	},
+
+	write: {
+		label: "Write",
+		// Streaming: path known, content still arriving (only the imports so far).
+		streamingArgs: {
+			path: "packages/coding-agent/test/parse-sel.test.ts",
+			content: 'import { describe, expect, it } from "bun:test";\nimport { parseSel } from "../src/tools/read";\n',
+		},
+		args: {
+			path: "packages/coding-agent/test/parse-sel.test.ts",
+			content: writtenContent,
+		},
+		result: {
+			content: [
+				{
+					type: "text",
+					text: "Created packages/coding-agent/test/parse-sel.test.ts (17 lines, 412 bytes).",
+				},
+			],
+			details: {},
+		},
+		errorResult: {
+			isError: true,
+			content: [
+				{
+					type: "text",
+					text: "Error: EACCES: permission denied, open 'packages/coding-agent/test/parse-sel.test.ts'",
+				},
+			],
+		},
+	},
+
+	find: {
+		label: "Find",
+		// Streaming: glob half-typed, no limit yet.
+		streamingArgs: { paths: ["packages/coding-agent/src/tools/*-render"] },
+		args: { paths: ["packages/coding-agent/src/**/*.test.ts"], limit: 50 },
+		result: {
+			content: [
+				{
+					type: "text",
+					text: [
+						"packages/coding-agent/src/tools/read.test.ts",
+						"packages/coding-agent/src/tools/write.test.ts",
+						"packages/coding-agent/src/tools/find.test.ts",
+						"packages/coding-agent/src/cli/gallery-cli.test.ts",
+						"packages/coding-agent/src/edit/edit.test.ts",
+					].join("\n"),
+				},
+			],
+			details: {
+				scopePath: "packages/coding-agent/src",
+				cwd: "/Users/dev/Projects/pi",
+				fileCount: 5,
+				truncated: false,
+				files: [
+					"packages/coding-agent/src/cli/gallery-cli.test.ts",
+					"packages/coding-agent/src/edit/edit.test.ts",
+					"packages/coding-agent/src/tools/find.test.ts",
+					"packages/coding-agent/src/tools/read.test.ts",
+					"packages/coding-agent/src/tools/write.test.ts",
+				],
+			},
+		},
+		errorResult: {
+			isError: true,
+			content: [{ type: "text", text: "Find failed: invalid glob pattern '[unclosed'." }],
+			details: { error: "invalid glob pattern '[unclosed'" },
+		},
+	},
+};

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

@@ -0,0 +1,40 @@
+/**
+ * Aggregated sample data for the `omp gallery` command.
+ *
+ * Each fixture drives one tool's renderer through the four lifecycle states the
+ * gallery showcases: arguments streaming in, arguments complete but awaiting a
+ * result, a successful result, and a failed result. The data is intentionally
+ * hand-written (rather than schema-derived) so the gallery reflects what a real
+ * tool call looks like — the whole point is visual QA of the renderers.
+ *
+ * Fixtures are grouped by subsystem into sibling modules and merged here.
+ * Adding a tool to one of those groups is enough for the gallery to render it.
+ * Tools present in the renderer registry but missing here fall back to a
+ * generic fixture (see `gallery-cli.ts`), so the gallery never crashes on a
+ * newly added tool — it just looks plain until a fixture is supplied.
+ */
+import { agenticFixtures } from "./agentic";
+import { codeintelFixtures } from "./codeintel";
+import { editFixtures } from "./edit";
+import { fsFixtures } from "./fs";
+import { interactionFixtures } from "./interaction";
+import { memoryFixtures } from "./memory";
+import { miscFixtures } from "./misc";
+import { searchFixtures } from "./search";
+import { shellFixtures } from "./shell";
+import { webFixtures } from "./web";
+
+export * from "./types";
+
+export const galleryFixtures = {
+	...interactionFixtures,
+	...shellFixtures,
+	...fsFixtures,
+	...searchFixtures,
+	...editFixtures,
+	...agenticFixtures,
+	...memoryFixtures,
+	...webFixtures,
+	...codeintelFixtures,
+	...miscFixtures,
+};

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

@@ -0,0 +1,49 @@
+/** Gallery fixtures for the todo / ask / resolve interaction tools. */
+import type { GalleryFixture } from "./types";
+
+export const interactionFixtures: Record<string, GalleryFixture> = {
+	todo: {
+		label: "Todo",
+		streamingArgs: {
+			ops: [{ op: "init", list: [{ phase: "Foundation", items: ["Scaffold crate"] }] }],
+		},
+		args: {
+			ops: [
+				{
+					op: "init",
+					list: [
+						{ phase: "Foundation", items: ["Scaffold crate", "Wire workspace"] },
+						{ phase: "Auth", items: ["Port credential store", "Wire OAuth providers"] },
+					],
+				},
+			],
+		},
+		result: {
+			content: [{ type: "text", text: "Initialized 4 tasks across 2 phases" }],
+			details: {
+				storage: "session",
+				phases: [
+					{
+						name: "Foundation",
+						tasks: [
+							{ content: "Scaffold crate", status: "done" },
+							{ content: "Wire workspace", status: "in_progress" },
+						],
+					},
+					{
+						name: "Auth",
+						tasks: [
+							{ content: "Port credential store", status: "pending" },
+							{ content: "Wire OAuth providers", status: "pending" },
+						],
+					},
+				],
+				completedTasks: [{ phase: "Foundation", content: "Scaffold crate" }],
+			},
+		},
+		errorResult: {
+			content: [{ type: "text", text: "Unknown phase 'Auth' — initialize the list first" }],
+			isError: true,
+		},
+	},
+};

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

@@ -0,0 +1,81 @@
+// Gallery fixtures for the long-term memory tools (retain, recall, reflect).
+import type { GalleryFixture } from "./types";
+
+export const memoryFixtures: Record<string, GalleryFixture> = {
+	retain: {
+		label: "Retain",
+		// Streaming: first item complete, second still arriving without a context.
+		streamingArgs: {
+			items: [{ content: "User prefers Bun over Node for all new scripts in this repo." }],
+		},
+		args: {
+			items: [
+				{
+					content: "User prefers Bun over Node for all new scripts in this repo.",
+					context: "Established while wiring up the gallery command tooling.",
+				},
+				{
+					content: "The TUI renderers live in packages/coding-agent/src/tools/*-render.ts.",
+					context: "Discovered during the gallery-fixtures task.",
+				},
+			],
+		},
+		result: {
+			content: [{ type: "text", text: "2 memories stored." }],
+			details: { count: 2 },
+		},
+		errorResult: {
+			isError: true,
+			content: [{ type: "text", text: "Retain failed: memory store is not initialized." }],
+		},
+	},
+
+	recall: {
+		label: "Recall",
+		// Streaming: query partially typed.
+		streamingArgs: { query: "bun vs node" },
+		args: { query: "Which runtime does the user prefer for scripts?" },
+		result: {
+			content: [
+				{
+					type: "text",
+					text: [
+						"Found 2 relevant memories:",
+						"",
+						"1. [0.92] User prefers Bun over Node for all new scripts in this repo.",
+						"   (Established while wiring up the gallery command tooling.)",
+						"2. [0.78] The TUI renderers live in packages/coding-agent/src/tools/*-render.ts.",
+						"   (Discovered during the gallery-fixtures task.)",
+					].join("\n"),
+				},
+			],
+		},
+		errorResult: {
+			isError: true,
+			content: [{ type: "text", text: "Recall failed: vector index unavailable." }],
+		},
+	},
+
+	reflect: {
+		label: "Reflect",
+		streamingArgs: { query: "what have we learned about the user's" },
+		args: { query: "What have we learned about the user's tooling preferences?" },
+		result: {
+			content: [
+				{
+					type: "text",
+					text: [
+						"The user consistently favors Bun as the runtime for scripts in this",
+						"repository, avoiding Node where possible. They also track the location",
+						"of TUI renderers under packages/coding-agent/src/tools, suggesting an",
+						"interest in keeping rendering logic discoverable and well-organized.",
+					].join("\n"),
+				},
+			],
+		},
+		errorResult: {
+			isError: true,
+			content: [{ type: "text", text: "Reflect failed: no memories matched the query." }],
+		},
+	},
+};

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

@@ -0,0 +1,221 @@
+/** Gallery fixtures for the ask / resolve / ssh / github / inspect_image tools. */
+import type { GalleryFixture } from "./types";
+
+export const miscFixtures: Record<string, GalleryFixture> = {
+	ask: {
+		label: "Ask",
+		streamingArgs: {
+			questions: [
+				{
+					id: "db",
+					question: "Which database should the new service use?",
+					options: [{ label: "Postgres" }],
+				},
+			],
+		},
+		args: {
+			questions: [
+				{
+					id: "db",
+					question: "Which database should the new service use?",
+					options: [
+						{ label: "Postgres", description: "Relational, strong consistency, JSONB support" },
+						{ label: "SQLite", description: "Embedded, zero-ops, great for single-node" },
+						{ label: "MongoDB", description: "Document store, flexible schema" },
+					],
+					recommended: 0,
+				},
+				{
+					id: "features",
+					question: "Which auth flows should ship in v1?",
+					options: [
+						{ label: "Email + password" },
+						{ label: "OAuth (Google, GitHub)" },
+						{ label: "Magic links" },
+						{ label: "SAML SSO", description: "Enterprise; can be deferred" },
+					],
+					multi: true,
+				},
+			],
+		},
+		result: {
+			content: [
+				{
+					type: "text",
+					text: "db: Postgres\nfeatures: Email + password, OAuth (Google, GitHub)",
+				},
+			],
+			details: {
+				results: [
+					{
+						id: "db",
+						question: "Which database should the new service use?",
+						options: ["Postgres", "SQLite", "MongoDB"],
+						multi: false,
+						selectedOptions: ["Postgres"],
+					},
+					{
+						id: "features",
+						question: "Which auth flows should ship in v1?",
+						options: ["Email + password", "OAuth (Google, GitHub)", "Magic links", "SAML SSO"],
+						multi: true,
+						selectedOptions: ["Email + password", "OAuth (Google, GitHub)"],
+					},
+				],
+			},
+		},
+		errorResult: {
+			content: [{ type: "text", text: "Prompt cancelled by user before any answer was given" }],
+			isError: true,
+		},
+	},
+
+	resolve: {
+		label: "Resolve",
+		streamingArgs: {
+			action: "apply",
+		},
+		args: {
+			action: "apply",
+			reason: "Rename is mechanical and the staged diff matches the intended refactor.",
+			extra: { title: "rename-usecredentials-hook" },
+		},
+		result: {
+			content: [{ type: "text", text: "Applied pending ast_edit: 7 replacements across 3 files" }],
+			details: {
+				action: "apply",
+				reason: "Rename is mechanical and the staged diff matches the intended refactor.",
+				extra: { title: "rename-usecredentials-hook" },
+				sourceToolName: "ast_edit",
+				label: "ast_edit: 7 replacements across 3 files",
+			},
+		},
+		errorResult: {
+			content: [{ type: "text", text: "No pending action to resolve" }],
+			isError: true,
+			details: {
+				action: "apply",
+				reason: "Rename is mechanical and the staged diff matches the intended refactor.",
+				sourceToolName: "ast_edit",
+				label: "ast_edit: 7 replacements across 3 files",
+			},
+		},
+	},
+
+	ssh: {
+		label: "SSH",
+		streamingArgs: {
+			host: "deploy@web-01",
+			command: "systemctl status",
+		},
+		args: {
+			host: "deploy@web-01",
+			command: "systemctl status omp-api --no-pager | head -n 12",
+			cwd: "/srv/omp",
+			timeout: 60,
+		},
+		result: {
+			content: [
+				{
+					type: "text",
+					text: [
+						"● omp-api.service - Oh My Pi API",
+						"     Loaded: loaded (/etc/systemd/system/omp-api.service; enabled)",
+						"     Active: active (running) since Sat 2026-06-06 09:14:02 UTC; 3h 21min ago",
+						"   Main PID: 4812 (bun)",
+						"      Tasks: 17 (limit: 4915)",
+						"     Memory: 142.6M",
+						"        CPU: 38.214s",
+						"     CGroup: /system.slice/omp-api.service",
+						"             └─4812 /usr/local/bin/bun run dist/server.js",
+					].join("\n"),
+				},
+			],
+		},
+		errorResult: {
+			content: [
+				{
+					type: "text",
+					text: "ssh: connect to host web-01 port 22: Connection timed out",
+				},
+			],
+			isError: true,
+		},
+	},
+
+	github: {
+		label: "GitHub",
+		streamingArgs: {
+			op: "search_prs",
+			query: "is:open author:@me",
+		},
+		args: {
+			op: "search_prs",
+			query: "is:open review-requested:@me sort:updated",
+			repo: "oh-my-pi/pi",
+		},
+		result: {
+			content: [
+				{
+					type: "text",
+					text: [
+						"#1842  feat(tui): virtualized scrollback for tool output     openyou · 2h ago   +312 -47",
+						"#1839  fix(agent): retry stream on transient 529             dvir   · 5h ago   +18 -4",
+						"#1830  refactor(edit): unify hashline + ast_edit previews    mira   · 1d ago   +540 -210",
+						"#1817  docs: document gallery fixtures contract             leo    · 2d ago   +96 -0",
+						"",
+						"4 open pull requests requesting your review",
+					].join("\n"),
+				},
+			],
+		},
+		errorResult: {
+			content: [
+				{
+					type: "text",
+					text: "gh: Could not resolve to a Repository with the name 'oh-my-pi/pi'. (HTTP 404)",
+				},
+			],
+			isError: true,
+		},
+	},
+
+	inspect_image: {
+		label: "Inspect Image",
+		streamingArgs: {
+			path: "docs/assets/dashboard-mock.png",
+		},
+		args: {
+			path: "docs/assets/dashboard-mock.png",
+			question: "What chart types are shown and roughly what layout does the dashboard use?",
+		},
+		result: {
+			content: [
+				{
+					type: "text",
+					text: [
+						"The dashboard uses a two-column layout on a dark background.",
+						"Top row: four KPI cards (Revenue, Active Users, Churn, MRR) with sparklines.",
+						"Left column: a stacked area chart of weekly sessions over ~3 months.",
+						"Right column: a horizontal bar chart ranking the top 6 referrers.",
+						"Bottom: a paginated table of recent transactions with status pills.",
+					].join("\n"),
+				},
+			],
+			details: {
+				model: "claude-opus-4",
+				imagePath: "docs/assets/dashboard-mock.png",
+				mimeType: "image/png",
+			},
+		},
+		errorResult: {
+			content: [{ type: "text", text: "Image not found: docs/assets/dashboard-mock.png" }],
+			isError: true,
+			details: {
+				model: "claude-opus-4",
+				imagePath: "docs/assets/dashboard-mock.png",
+				mimeType: "image/png",
+			},
+		},
+	},
+};