@oh-my-pi/pi-coding-agent 15.1.2 → 15.1.4

package/src/tools/gh.ts

@@ -213,58 +213,34 @@ const githubSchema = z
 				"run_watch",
 			] as const)
 			.describe("github operation"),
-		repo: z.string().describe("owner/repo (any op)").optional(),
-		branch: z.string().describe("branch (repo_view, pr_push local branch, run_watch)").optional(),
+		repo: z.string().describe("owner/repo").optional(),
+		branch: z.string().describe("branch").optional(),
 		pr: z
 			.union([z.string(), z.array(z.string())])
-			.describe(
-				"pr number, url, or branch (pr_checkout); pass an array to batch-process multiple pull requests in one call",
-			)
-			.optional(),
-		force: z.boolean().describe("reset existing local branch (pr_checkout)").optional(),
-		forceWithLease: z.boolean().describe("force-with-lease push (pr_push)").optional(),
-		title: z.string().describe("PR title (pr_create)").optional(),
-		body: z.string().describe("PR body markdown (pr_create); mutually exclusive with fill").optional(),
-		base: z.string().describe("PR base branch (pr_create); defaults to repo default branch").optional(),
-		head: z.string().describe("PR head branch (pr_create); defaults to current branch").optional(),
-		draft: z.boolean().describe("open PR as draft (pr_create)").optional(),
-		fill: z
-			.boolean()
-			.describe("auto-fill PR title/body from commits (pr_create); mutually exclusive with title/body")
-			.optional(),
-		reviewer: z.array(z.string()).describe("reviewers to request (pr_create); accepts users or org/team").optional(),
-		assignee: z.array(z.string()).describe("assignees (pr_create); use @me for the authenticated user").optional(),
-		label: z.array(z.string()).describe("labels to apply (pr_create)").optional(),
-		query: z
-			.string()
-			.describe("search query (search_issues, search_prs, search_code, search_commits, search_repos)")
-			.optional(),
-		since: z
-			.string()
-			.describe(
-				"lower-bound date for search_issues/search_prs/search_commits/search_repos. Accepts a relative duration (`<n><unit>` with unit `m`/`h`/`d`/`w`/`mo`/`y`, e.g. `3d`, `12h`, `2w`) or an ISO date (`YYYY-MM-DD`) / datetime. Translated to a `created:>=…` (or `committer-date:`/`pushed:`) qualifier; not supported by search_code.",
-			)
-			.optional(),
-		until: z
-			.string()
-			.describe(
-				"upper-bound date in the same format as `since`. With both, builds a `field:since..until` range qualifier.",
-			)
+			.describe("pr number, url, or branch")
 			.optional(),
+		force: z.boolean().describe("reset existing local branch").optional(),
+		forceWithLease: z.boolean().describe("force-with-lease push").optional(),
+		title: z.string().describe("pr title").optional(),
+		body: z.string().describe("pr body markdown").optional(),
+		base: z.string().describe("pr base branch").optional(),
+		head: z.string().describe("pr head branch").optional(),
+		draft: z.boolean().describe("open pr as draft").optional(),
+		fill: z.boolean().describe("auto-fill pr title/body from commits").optional(),
+		reviewer: z.array(z.string()).describe("reviewers").optional(),
+		assignee: z.array(z.string()).describe("assignees").optional(),
+		label: z.array(z.string()).describe("labels").optional(),
+		query: z.string().describe("search query").optional(),
+		since: z.string().describe("lower-bound date filter").optional(),
+		until: z.string().describe("upper-bound date filter").optional(),
 		dateField: z
 			.enum(["created", "updated"] as const)
-			.describe(
-				"date field used by `since`/`until`. issues/prs: `created` (default) or `updated`. repos: `created` (default) or `updated` (mapped to GitHub's `pushed:`). commits: ignored — always uses `committer-date`.",
-			)
+			.describe("date field")
 			.default("created")
 			.optional(),
-		limit: z
-			.number()
-			.default(10)
-			.describe("max results (search_issues, search_prs, search_code, search_commits, search_repos)")
-			.optional(),
-		run: z.string().describe("actions run id or url (run_watch)").optional(),
-		tail: z.number().default(15).describe("log lines per failed job (run_watch)").optional(),
+		limit: z.number().default(10).describe("max results").optional(),
+		run: z.string().describe("actions run id or url").optional(),
+		tail: z.number().default(15).describe("log lines per failed job").optional(),
 	})
 	.strict();
 

package/src/tools/hindsight-recall.ts

@@ -6,7 +6,7 @@ import recallDescription from "../prompts/tools/recall.md" with { type: "text" }
 import type { ToolSession } from ".";
 
 const hindsightRecallSchema = z.object({
-	query: z.string().describe("Natural language search query. Be specific about what you need to know."),
+	query: z.string().describe("natural language search query"),
 });
 
 export type HindsightRecallParams = z.infer<typeof hindsightRecallSchema>;

package/src/tools/hindsight-reflect.ts

@@ -6,8 +6,8 @@ import reflectDescription from "../prompts/tools/reflect.md" with { type: "text"
 import type { ToolSession } from ".";
 
 const hindsightReflectSchema = z.object({
-	query: z.string().describe("The question to answer using long-term memory."),
-	context: z.string().describe("Optional additional context to guide the reflection.").optional(),
+	query: z.string().describe("question to answer"),
+	context: z.string().describe("optional context").optional(),
 });
 
 export type HindsightReflectParams = z.infer<typeof hindsightReflectSchema>;

package/src/tools/hindsight-retain.ts

@@ -7,16 +7,12 @@ const hindsightRetainSchema = z.object({
 	items: z
 		.array(
 			z.object({
-				content: z
-					.string()
-					.describe("The information to remember. Be specific and self-contained — include who, what, when, why."),
-				context: z.string().describe("Optional context describing where this information came from.").optional(),
+				content: z.string().describe("information to remember"),
+				context: z.string().describe("source context").optional(),
 			}),
 		)
 		.min(1)
-		.describe(
-			"One or more memories to retain. Batch related facts in a single call rather than calling retain repeatedly — they are deduplicated and consolidated together.",
-		),
+		.describe("memories to retain"),
 });
 
 export type HindsightRetainParams = z.infer<typeof hindsightRetainSchema>;

package/src/tools/index.ts

@@ -514,7 +514,14 @@ export async function createTools(session: ToolSession, toolNames?: string[]): P
 	// Injected unconditionally into every agent, regardless of requested tool list.
 	const autoQA = isAutoQaEnabled(session.settings);
 	if (autoQA && !tools.some(t => t.name === "report_tool_issue")) {
-		const qaTool = await HIDDEN_TOOLS.report_tool_issue(session);
+		// Build the enum from tools we just constructed via BUILTIN_TOOLS / HIDDEN_TOOLS.
+		// Extension overrides (e.g. a user's custom `bash`) get added later by
+		// other code paths, so they're absent here — exactly what we want; MCP /
+		// extension tools never end up in the report enum.
+		const activeBuiltinNames = tools
+			.map(t => t.name)
+			.filter(name => (name in BUILTIN_TOOLS || name in HIDDEN_TOOLS) && name !== "report_tool_issue");
+		const qaTool = createReportToolIssueTool(session, activeBuiltinNames);
 		if (qaTool) {
 			tools.push(wrapToolWithMetaNotice(qaTool));
 		}

package/src/tools/inspect-image.ts

@@ -1,4 +1,5 @@
 import type { AgentTool, AgentToolContext, AgentToolResult, AgentToolUpdateCallback } from "@oh-my-pi/pi-agent-core";
+import { instrumentedCompleteSimple, resolveTelemetry } from "@oh-my-pi/pi-agent-core";
 import { type Api, completeSimple, type Model } from "@oh-my-pi/pi-ai";
 import { prompt } from "@oh-my-pi/pi-utils";
 import * as z from "zod/v4";
@@ -118,7 +119,8 @@ export class InspectImageTool implements AgentTool<typeof inspectImageSchema, In
 			throw new ToolError("inspect_image only supports PNG, JPEG, GIF, and WEBP files detected by file content.");
 		}
 
-		const response = await this.completeImageRequest(
+		const telemetry = resolveTelemetry(this.session.getTelemetry?.(), this.session.getSessionId?.() ?? undefined);
+		const response = await instrumentedCompleteSimple(
 			model,
 			{
 				systemPrompt: [prompt.render(inspectImageSystemPromptTemplate)],
@@ -134,6 +136,7 @@ export class InspectImageTool implements AgentTool<typeof inspectImageSchema, In
 				],
 			},
 			{ apiKey, signal },
+			{ telemetry, oneshotKind: "inspect_image", completeImpl: this.completeImageRequest },
 		);
 
 		if (response.stopReason === "error") {

package/src/tools/irc.ts

@@ -26,18 +26,10 @@ import type { AgentRef, AgentRegistry } from "../registry/agent-registry";
 import type { ToolSession } from ".";
 
 const ircSchema = z.object({
-	op: z
-		.union([
-			z.literal("send").describe("Send a message to one peer or to all peers"),
-			z.literal("list").describe("List currently visible peers"),
-		])
-		.describe("IRC operation"),
-	to: z.string().optional().describe('Recipient agent id (e.g. "0-Main", "0-AuthLoader") or "all" to broadcast'),
-	message: z.string().optional().describe("Message body to deliver"),
-	awaitReply: z
-		.boolean()
-		.optional()
-		.describe("Wait for the recipient's prose reply (default: true for DM, false for broadcast)"),
+	op: z.enum(["send", "list"]).describe("irc operation"),
+	to: z.string().optional().describe('recipient agent id or "all"'),
+	message: z.string().optional().describe("message body"),
+	awaitReply: z.boolean().optional().describe("wait for prose reply"),
 });
 
 type IrcParams = z.infer<typeof ircSchema>;

package/src/tools/job.ts

@@ -23,17 +23,9 @@ import {
 import { ToolError } from "./tool-errors";
 
 const jobSchema = z.object({
-	poll: z
-		.array(z.string())
-		.optional()
-		.describe("background job ids to wait for; omit (with no `cancel`) to wait on all running jobs"),
-	cancel: z.array(z.string()).optional().describe("background job ids to cancel"),
-	list: z
-		.boolean()
-		.optional()
-		.describe(
-			"Return an immediate snapshot of every job spawned by this agent (running + completed within retention). Read-only \u2014 cannot be combined with `poll` or `cancel`.",
-		),
+	poll: z.array(z.string()).optional().describe("job ids to wait for"),
+	cancel: z.array(z.string()).optional().describe("job ids to cancel"),
+	list: z.boolean().optional().describe("snapshot all jobs"),
 });
 
 type JobParams = z.infer<typeof jobSchema>;