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

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

@@ -0,0 +1,292 @@
+// Gallery fixtures for the agentic orchestration tools (task, goal, job).
+import type { GalleryFixture } from "./types";
+
+export const agenticFixtures: Record<string, GalleryFixture> = {
+	task: {
+		label: "Task",
+		customRendered: true,
+		// Streaming: agent chosen, first task fully arrived, second still landing.
+		streamingArgs: {
+			agent: "task",
+			tasks: [
+				{
+					id: "AuthLoader",
+					description: "Load auth middleware",
+					assignment: "Read packages/server/src/auth/*.ts and summarize the session-cookie flow.",
+				},
+				{ id: "RateLimiter", description: "Audit rate limiter" },
+			],
+		},
+		args: {
+			agent: "task",
+			context: [
+				"# Goal",
+				"Harden the HTTP auth stack before the release cut.",
+				"# Constraints",
+				"Touch only files under packages/server/src/auth/. Do not run gates.",
+			].join("\n"),
+			tasks: [
+				{
+					id: "AuthLoader",
+					description: "Load auth middleware",
+					assignment:
+						"Read packages/server/src/auth/session.ts and middleware.ts, then document the session-cookie validation flow and any TODOs.",
+				},
+				{
+					id: "RateLimiter",
+					description: "Audit rate limiter",
+					assignment:
+						"Inspect packages/server/src/auth/rate-limit.ts. Confirm the 429 path sets Retry-After and report gaps.",
+				},
+				{
+					id: "TokenRotation",
+					description: "Check token rotation",
+					assignment:
+						"Trace refresh-token rotation in packages/server/src/auth/tokens.ts and flag any reuse window.",
+				},
+			],
+		},
+		result: {
+			content: [
+				{
+					type: "text",
+					text: "3 agents completed: AuthLoader, RateLimiter, TokenRotation.",
+				},
+			],
+			details: {
+				projectAgentsDir: null,
+				totalDurationMs: 48_200,
+				usage: { cost: { total: 0.34 } },
+				results: [
+					{
+						index: 0,
+						id: "AuthLoader",
+						agent: "task",
+						agentSource: "bundled",
+						description: "Load auth middleware",
+						task: "Read packages/server/src/auth/session.ts and middleware.ts",
+						assignment:
+							"Read packages/server/src/auth/session.ts and middleware.ts, then document the session-cookie validation flow and any TODOs.",
+						exitCode: 0,
+						output: [
+							"Session validation runs in middleware.ts:42 via verifySessionCookie().",
+							"Cookies are HMAC-signed (SHA-256) and checked against the session store.",
+							"TODO at session.ts:88 — sliding-expiration refresh is stubbed.",
+						].join("\n"),
+						stderr: "",
+						truncated: false,
+						durationMs: 41_900,
+						tokens: 61_400,
+						contextTokens: 23_100,
+						contextWindow: 200_000,
+						resolvedModel: "anthropic/claude-sonnet",
+						usage: { cost: { total: 0.12 } },
+						outputMeta: { lineCount: 3, charCount: 214 },
+					},
+					{
+						index: 1,
+						id: "RateLimiter",
+						agent: "task",
+						agentSource: "bundled",
+						description: "Audit rate limiter",
+						task: "Inspect packages/server/src/auth/rate-limit.ts",
+						assignment:
+							"Inspect packages/server/src/auth/rate-limit.ts. Confirm the 429 path sets Retry-After and report gaps.",
+						exitCode: 0,
+						output: [
+							"rate-limit.ts uses a fixed-window counter keyed by client IP.",
+							"429 responses set Retry-After (rate-limit.ts:57).",
+							"Gap: no per-account limit, so a botnet across IPs bypasses the cap.",
+						].join("\n"),
+						stderr: "",
+						truncated: false,
+						durationMs: 38_500,
+						tokens: 54_800,
+						contextTokens: 19_700,
+						contextWindow: 200_000,
+						resolvedModel: "anthropic/claude-sonnet",
+						usage: { cost: { total: 0.1 } },
+						outputMeta: { lineCount: 3, charCount: 198 },
+					},
+					{
+						index: 2,
+						id: "TokenRotation",
+						agent: "task",
+						agentSource: "bundled",
+						description: "Check token rotation",
+						task: "Trace refresh-token rotation in packages/server/src/auth/tokens.ts",
+						assignment:
+							"Trace refresh-token rotation in packages/server/src/auth/tokens.ts and flag any reuse window.",
+						exitCode: 0,
+						output: [
+							"Refresh tokens rotate on every use (tokens.ts:120) and the old jti is revoked.",
+							"Reuse of a rotated token triggers full-family revocation — no reuse window found.",
+						].join("\n"),
+						stderr: "",
+						truncated: false,
+						durationMs: 48_200,
+						tokens: 49_200,
+						contextTokens: 17_500,
+						contextWindow: 200_000,
+						resolvedModel: "anthropic/claude-sonnet",
+						usage: { cost: { total: 0.12 } },
+						outputMeta: { lineCount: 2, charCount: 160 },
+					},
+				],
+			},
+		},
+		errorResult: {
+			isError: true,
+			content: [
+				{
+					type: "text",
+					text: "1 of 3 agents failed: RateLimiter.",
+				},
+			],
+			details: {
+				projectAgentsDir: null,
+				totalDurationMs: 39_400,
+				usage: { cost: { total: 0.21 } },
+				results: [
+					{
+						index: 0,
+						id: "AuthLoader",
+						agent: "task",
+						agentSource: "bundled",
+						description: "Load auth middleware",
+						task: "Read packages/server/src/auth/session.ts and middleware.ts",
+						assignment:
+							"Read packages/server/src/auth/session.ts and middleware.ts, then document the session-cookie validation flow and any TODOs.",
+						exitCode: 0,
+						output: "Session validation runs in middleware.ts:42 via verifySessionCookie().",
+						stderr: "",
+						truncated: false,
+						durationMs: 31_200,
+						tokens: 58_100,
+						contextTokens: 21_900,
+						contextWindow: 200_000,
+						resolvedModel: "anthropic/claude-sonnet",
+						usage: { cost: { total: 0.11 } },
+						outputMeta: { lineCount: 1, charCount: 70 },
+					},
+					{
+						index: 1,
+						id: "RateLimiter",
+						agent: "task",
+						agentSource: "bundled",
+						description: "Audit rate limiter",
+						task: "Inspect packages/server/src/auth/rate-limit.ts",
+						assignment:
+							"Inspect packages/server/src/auth/rate-limit.ts. Confirm the 429 path sets Retry-After and report gaps.",
+						exitCode: 1,
+						output: "",
+						stderr: "ENOENT: packages/server/src/auth/rate-limit.ts",
+						truncated: false,
+						durationMs: 9_800,
+						tokens: 12_300,
+						contextTokens: 6_400,
+						contextWindow: 200_000,
+						resolvedModel: "anthropic/claude-sonnet",
+						usage: { cost: { total: 0.1 } },
+						error: "Subagent exited 1: target file packages/server/src/auth/rate-limit.ts does not exist.",
+						outputMeta: { lineCount: 0, charCount: 0 },
+					},
+				],
+			},
+		},
+	},
+
+	goal: {
+		label: "Goal",
+		// Streaming: op is "create"; objective text still being typed.
+		streamingArgs: { op: "create", objective: "Ship the auth hardening" },
+		args: {
+			op: "create",
+			objective: "Ship the auth hardening pass: per-account rate limits and sliding session expiry.",
+			token_budget: 500_000,
+		},
+		result: {
+			content: [
+				{
+					type: "text",
+					text: "Goal set. Working toward: Ship the auth hardening pass.",
+				},
+			],
+			details: {
+				op: "create",
+				remainingTokens: 451_800,
+				completionBudgetReport: null,
+				goal: {
+					id: "goal_8f2a",
+					objective: "Ship the auth hardening pass: per-account rate limits and sliding session expiry.",
+					status: "active",
+					tokenBudget: 500_000,
+					tokensUsed: 48_200,
+					timeUsedSeconds: 312,
+					createdAt: 1_749_200_000_000,
+					updatedAt: 1_749_200_312_000,
+				},
+			},
+		},
+		errorResult: {
+			isError: true,
+			content: [{ type: "text", text: "Goal tool failed: objective is required when op=create." }],
+			details: { op: "create" },
+		},
+	},
+
+	job: {
+		label: "Job",
+		// Streaming: polling a single job id; the second id is still arriving.
+		streamingArgs: { poll: ["job_a1"] },
+		args: { poll: ["job_a1", "job_b2", "job_c3"] },
+		result: {
+			content: [{ type: "text", text: "3 jobs settled." }],
+			details: {
+				jobs: [
+					{
+						id: "job_a1",
+						type: "bash",
+						status: "completed",
+						label: "bun test packages/server/test/auth.test.ts",
+						durationMs: 18_400,
+						resultText: "42 pass, 0 fail (18.4s)",
+					},
+					{
+						id: "job_b2",
+						type: "task",
+						status: "completed",
+						label: "Migrate rate limiter to a sliding window",
+						durationMs: 96_700,
+						resultText: "Rewrote rate-limit.ts to a token-bucket; added per-account keys.",
+					},
+					{
+						id: "job_c3",
+						type: "bash",
+						status: "failed",
+						label: "bunx biome check packages/server/src/auth",
+						durationMs: 4_100,
+						errorText: "biome: 2 errors in tokens.ts — noUnusedVariables, useConst",
+					},
+				],
+			},
+		},
+		errorResult: {
+			isError: true,
+			content: [{ type: "text", text: "Job cancelled by user." }],
+			details: {
+				jobs: [
+					{
+						id: "job_d4",
+						type: "task",
+						status: "cancelled",
+						label: "Refactor the session store to Redis",
+						durationMs: 52_300,
+						errorText: "Aborted: superseded by goal re-scope.",
+					},
+				],
+				cancelled: [{ id: "job_d4", status: "cancelled" }],
+			},
+		},
+	},
+};

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

@@ -0,0 +1,188 @@
+/** Gallery fixtures for the code-intelligence tools (lsp, debug). */
+import type { GalleryFixture } from "./types";
+
+export const codeintelFixtures: Record<string, GalleryFixture> = {
+	lsp: {
+		label: "LSP",
+		customRendered: true,
+		streamingArgs: {
+			action: "references",
+			file: "src/server/auth.ts",
+		},
+		args: {
+			action: "references",
+			file: "src/server/auth.ts",
+			line: 42,
+			symbol: "validateToken",
+		},
+		result: {
+			content: [
+				{
+					type: "text",
+					text: [
+						"Found 6 reference(s):",
+						"  src/server/auth.ts:42:14",
+						"    41: ",
+						"    42: export function validateToken(token: string): Claims {",
+						"    43:   const claims = verifyJwt(token);",
+						"  src/server/auth.ts:118:21",
+						'    117:   if (!header) throw new HttpError(401, "missing token");',
+						"    118:   const claims = validateToken(stripBearer(header));",
+						"    119:   return claims.sub;",
+						"  src/server/middleware/session.ts:57:18",
+						"    56:   const token = req.cookies.session;",
+						"    57:   const claims = validateToken(token);",
+						"    58:   req.userId = claims.sub;",
+						"  src/server/router.ts:153:20",
+						"    152: router.use(async (req, res, next) => {",
+						"    153:   req.claims = await validateToken(req.token);",
+						"    154:   next();",
+						"  test/auth.test.ts:24:9",
+						'    23: it("rejects expired tokens", () => {',
+						"    24:   expect(() => validateToken(expired)).toThrow(/expired/);",
+						"    25: });",
+						"  test/auth.test.ts:41:9",
+						'    40: it("accepts valid tokens", () => {',
+						"    41:   const claims = validateToken(signed);",
+						'    42:   expect(claims.sub).toBe("u_123");',
+					].join("\n"),
+				},
+			],
+			details: {
+				serverName: "typescript-language-server",
+				action: "references",
+				success: true,
+				request: {
+					action: "references",
+					file: "src/server/auth.ts",
+					line: 42,
+					symbol: "validateToken",
+				},
+			},
+		},
+		errorResult: {
+			content: [
+				{
+					type: "text",
+					text: "No language server found for this file",
+				},
+			],
+			isError: true,
+			details: {
+				serverName: "typescript-language-server",
+				action: "references",
+				success: false,
+				request: {
+					action: "references",
+					file: "src/server/auth.ts",
+					line: 42,
+					symbol: "validateToken",
+				},
+			},
+		},
+	},
+
+	debug: {
+		label: "Debug",
+		streamingArgs: {
+			action: "stack_trace",
+		},
+		args: {
+			action: "stack_trace",
+			levels: 20,
+		},
+		result: {
+			content: [
+				{
+					type: "text",
+					text: [
+						"Stack trace:",
+						"- #1000 validate_token @ app/server.py:42:14",
+						"- #1001 authenticate @ app/server.py:88:9",
+						"- #1002 handle_request @ app/router.py:153:20",
+						"- #1003 dispatch @ app/router.py:97:5",
+						"- #1004 <module> @ app/server.py:212:1",
+					].join("\n"),
+				},
+			],
+			details: {
+				action: "stack_trace",
+				success: true,
+				snapshot: {
+					id: "dbg-1",
+					adapter: "debugpy",
+					cwd: "/Users/dev/project",
+					program: "./app/server.py",
+					status: "stopped",
+					launchedAt: "2026-06-06T14:21:08.412Z",
+					lastUsedAt: "2026-06-06T14:22:55.901Z",
+					threadId: 1,
+					frameId: 1000,
+					stopReason: "breakpoint",
+					stopDescription: "breakpoint 2",
+					frameName: "validate_token",
+					instructionPointerReference: "0x00000001000034a8",
+					source: { name: "server.py", path: "app/server.py" },
+					line: 42,
+					column: 14,
+					breakpointFiles: 1,
+					breakpointCount: 2,
+					functionBreakpointCount: 0,
+					outputBytes: 248,
+					outputTruncated: false,
+					needsConfigurationDone: false,
+				},
+				stackFrames: [
+					{
+						id: 1000,
+						name: "validate_token",
+						source: { name: "server.py", path: "app/server.py" },
+						line: 42,
+						column: 14,
+					},
+					{
+						id: 1001,
+						name: "authenticate",
+						source: { name: "server.py", path: "app/server.py" },
+						line: 88,
+						column: 9,
+					},
+					{
+						id: 1002,
+						name: "handle_request",
+						source: { name: "router.py", path: "app/router.py" },
+						line: 153,
+						column: 20,
+					},
+					{
+						id: 1003,
+						name: "dispatch",
+						source: { name: "router.py", path: "app/router.py" },
+						line: 97,
+						column: 5,
+					},
+					{
+						id: 1004,
+						name: "<module>",
+						source: { name: "server.py", path: "app/server.py" },
+						line: 212,
+						column: 1,
+					},
+				],
+			},
+		},
+		errorResult: {
+			content: [
+				{
+					type: "text",
+					text: "No active debug session. Launch or attach first.",
+				},
+			],
+			isError: true,
+			details: {
+				action: "stack_trace",
+				success: false,
+			},
+		},
+	},
+};

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

@@ -0,0 +1,194 @@
+/** Gallery fixtures for the edit tools (edit, apply_patch, ast_edit). */
+import type { GalleryFixture } from "./types";
+
+export const editFixtures: Record<string, GalleryFixture> = {
+	edit: {
+		label: "Edit",
+		editMode: "replace",
+		// `previewDiff` is surfaced verbatim by the renderer's call preview, and the
+		// harness diff strategy skips `{ file_path, previewDiff }` (no `path`/`edits`),
+		// so the canned diff survives the streaming and progress states.
+		streamingArgs: {
+			file_path: "packages/coding-agent/src/tools/read.ts",
+			previewDiff: [
+				"@@ -88,3 +88,4 @@",
+				" 	const offset = args.offset ?? 1;",
+				"-	const limit = args.limit ?? 2000;",
+				"+	const limit = args.limit ?? 4000;",
+			].join("\n"),
+		},
+		args: {
+			file_path: "packages/coding-agent/src/tools/read.ts",
+			previewDiff: [
+				"@@ -88,5 +88,6 @@",
+				" 	const offset = args.offset ?? 1;",
+				"-	const limit = args.limit ?? 2000;",
+				"+	const limit = args.limit ?? 4000;",
+				" 	const raw = await Bun.file(path).text();",
+				"-	return raw.slice(offset, offset + limit);",
+				'+	return raw.split("\\n").slice(offset - 1, offset - 1 + limit).join("\\n");',
+			].join("\n"),
+		},
+		result: {
+			content: [{ type: "text", text: "Edited packages/coding-agent/src/tools/read.ts (1 hunk, +3 -2)" }],
+			details: {
+				path: "packages/coding-agent/src/tools/read.ts",
+				firstChangedLine: 89,
+				diff: [
+					"@@ -88,5 +88,6 @@",
+					" 	const offset = args.offset ?? 1;",
+					"-	const limit = args.limit ?? 2000;",
+					"+	const limit = args.limit ?? 4000;",
+					" 	const raw = await Bun.file(path).text();",
+					"-	return raw.slice(offset, offset + limit);",
+					'+	return raw.split("\\n").slice(offset - 1, offset - 1 + limit).join("\\n");',
+				].join("\n"),
+			},
+		},
+		errorResult: {
+			content: [
+				{
+					type: "text",
+					text: "Edit failed: the search text was not found in packages/coding-agent/src/tools/read.ts",
+				},
+			],
+			isError: true,
+			details: {
+				path: "packages/coding-agent/src/tools/read.ts",
+				diff: "",
+				errorText:
+					"No match for the search text. Expected `const limit = args.limit ?? 2000;` near line 89, but the file has `const limit = args.limit ?? 1000;`. Re-read the file and retry with the current contents.",
+			},
+		},
+	},
+
+	apply_patch: {
+		label: "Apply Patch",
+		editMode: "apply_patch",
+		streamingArgs: {
+			file_path: "packages/coding-agent/src/edit/renderer.ts",
+			previewDiff: [
+				"@@ -464,2 +464,2 @@",
+				"-		fileCount = countEditFiles(editArgs.edits);",
+				"+		fileCount = countDistinctFiles(editArgs.edits);",
+			].join("\n"),
+		},
+		args: {
+			file_path: "packages/coding-agent/src/edit/renderer.ts",
+			previewDiff: [
+				"@@ -177,4 +177,4 @@",
+				" /** Count distinct file paths in an edits array. */",
+				"-function countEditFiles(edits: EditRenderEntry[]): number {",
+				"+function countDistinctFiles(edits: EditRenderEntry[]): number {",
+				" 	return new Set(edits.map(edit => filePathFromEditEntry(edit.path)).filter(Boolean)).size;",
+				" }",
+				"@@ -467,2 +467,2 @@",
+				"-		fileCount = countEditFiles(editArgs.edits);",
+				"+		fileCount = countDistinctFiles(editArgs.edits);",
+			].join("\n"),
+		},
+		result: {
+			content: [
+				{ type: "text", text: "Applied patch to packages/coding-agent/src/edit/renderer.ts (2 hunks, +2 -2)" },
+			],
+			details: {
+				op: "update",
+				path: "packages/coding-agent/src/edit/renderer.ts",
+				firstChangedLine: 178,
+				diff: [
+					"@@ -177,4 +177,4 @@",
+					" /** Count distinct file paths in an edits array. */",
+					"-function countEditFiles(edits: EditRenderEntry[]): number {",
+					"+function countDistinctFiles(edits: EditRenderEntry[]): number {",
+					" 	return new Set(edits.map(edit => filePathFromEditEntry(edit.path)).filter(Boolean)).size;",
+					" }",
+					"@@ -467,2 +467,2 @@",
+					"-		fileCount = countEditFiles(editArgs.edits);",
+					"+		fileCount = countDistinctFiles(editArgs.edits);",
+				].join("\n"),
+			},
+		},
+		errorResult: {
+			content: [
+				{
+					type: "text",
+					text: "Apply patch failed: context does not match at line 177 of packages/coding-agent/src/edit/renderer.ts",
+				},
+			],
+			isError: true,
+			details: {
+				op: "update",
+				path: "packages/coding-agent/src/edit/renderer.ts",
+				diff: "",
+				errorText:
+					"Hunk @@ -177,4 +177,4 @@ failed to apply: the context line `function countEditFiles(edits: EditRenderEntry[]): number {` does not match the file. The file may have changed since it was read.",
+			},
+		},
+	},
+
+	ast_edit: {
+		label: "AST Edit",
+		streamingArgs: {
+			ops: [{ pat: "countEditFiles($$$ARGS)" }],
+			paths: ["packages/coding-agent/src/**/*.ts"],
+		},
+		args: {
+			ops: [{ pat: "countEditFiles($$$ARGS)", out: "countDistinctFiles($$$ARGS)" }],
+			paths: ["packages/coding-agent/src/**/*.ts"],
+		},
+		result: {
+			content: [
+				{
+					type: "text",
+					text: [
+						"# edit/renderer.ts (2 replacements)",
+						"-468:		fileCount = countEditFiles(editArgs.edits);",
+						"+468:		fileCount = countDistinctFiles(editArgs.edits);",
+						"-488:		const totalFiles = args?.edits ? countEditFiles(args.edits) : 0;",
+						"+488:		const totalFiles = args?.edits ? countDistinctFiles(args.edits) : 0;",
+						"",
+						"# tools/tool-result.ts (1 replacement)",
+						"-42:	return countEditFiles(files);",
+						"+42:	return countDistinctFiles(files);",
+					].join("\n"),
+				},
+			],
+			details: {
+				totalReplacements: 3,
+				filesTouched: 2,
+				filesSearched: 214,
+				applied: false,
+				limitReached: false,
+				scopePath: "packages/coding-agent/src",
+				searchPath: "/Users/dev/Projects/pi/packages/coding-agent/src",
+				files: ["edit/renderer.ts", "tools/tool-result.ts"],
+				fileReplacements: [
+					{ path: "edit/renderer.ts", count: 2 },
+					{ path: "tools/tool-result.ts", count: 1 },
+				],
+				displayContent: [
+					"# edit/",
+					"## renderer.ts (2 replacements)",
+					"-468│		fileCount = countEditFiles(editArgs.edits);",
+					"+468│		fileCount = countDistinctFiles(editArgs.edits);",
+					"-488│		const totalFiles = args?.edits ? countEditFiles(args.edits) : 0;",
+					"+488│		const totalFiles = args?.edits ? countDistinctFiles(args.edits) : 0;",
+					"",
+					"# tools/",
+					"## tool-result.ts (1 replacement)",
+					"-42│	return countEditFiles(files);",
+					"+42│	return countDistinctFiles(files);",
+				].join("\n"),
+			},
+		},
+		errorResult: {
+			content: [
+				{
+					type: "text",
+					text: "Pattern parse error in ops[0].pat: unbalanced parenthesis in `countEditFiles($$$ARGS`",
+				},
+			],
+			isError: true,
+		},
+	},
+};