@oh-my-pi/pi-coding-agent 6.8.4 → 6.9.0

package/src/core/tools/git.ts

@@ -1,213 +0,0 @@
-import type { AgentTool, AgentToolContext, AgentToolResult, AgentToolUpdateCallback } from "@oh-my-pi/pi-agent-core";
-import { StringEnum } from "@oh-my-pi/pi-ai";
-import { type GitParams, gitTool as gitToolCore, type ToolResponse } from "@oh-my-pi/pi-git-tool";
-import { type Static, Type } from "@sinclair/typebox";
-import gitDescription from "../../prompts/tools/git.md" with { type: "text" };
-import { renderPromptTemplate } from "../prompt-templates";
-import type { ToolSession } from "./index";
-
-const gitSchema = Type.Object({
-	operation: StringEnum([
-		"status",
-		"diff",
-		"log",
-		"show",
-		"blame",
-		"branch",
-		"add",
-		"restore",
-		"commit",
-		"checkout",
-		"merge",
-		"rebase",
-		"stash",
-		"cherry-pick",
-		"fetch",
-		"pull",
-		"push",
-		"tag",
-		"pr",
-		"issue",
-		"ci",
-		"release",
-	]),
-
-	// Status
-	only: Type.Optional(StringEnum(["branch", "modified", "staged", "untracked", "conflicts", "sync"])),
-	ignored: Type.Optional(Type.Boolean()),
-
-	// Diff
-	target: Type.Optional(
-		Type.Union([
-			Type.Object({
-				from: Type.String(),
-				to: Type.Optional(Type.String()),
-			}),
-			Type.String(),
-		]),
-	),
-	paths: Type.Optional(Type.Array(Type.String())),
-	stat_only: Type.Optional(Type.Boolean()),
-	name_only: Type.Optional(Type.Boolean()),
-	context: Type.Optional(Type.Number()),
-	max_lines: Type.Optional(Type.Number()),
-	ignore_whitespace: Type.Optional(Type.Boolean()),
-
-	// Log
-	limit: Type.Optional(Type.Number()),
-	ref: Type.Optional(Type.String()),
-	author: Type.Optional(Type.String()),
-	since: Type.Optional(Type.String()),
-	until: Type.Optional(Type.String()),
-	grep: Type.Optional(Type.String()),
-	format: Type.Optional(StringEnum(["oneline", "short", "full"])),
-	stat: Type.Optional(Type.Boolean()),
-	merges: Type.Optional(Type.Boolean()),
-	first_parent: Type.Optional(Type.Boolean()),
-
-	// Show
-	path: Type.Optional(Type.String()),
-	diff: Type.Optional(Type.Boolean()),
-	lines: Type.Optional(
-		Type.Object({
-			start: Type.Number(),
-			end: Type.Number(),
-		}),
-	),
-
-	// Blame
-	root: Type.Optional(Type.Boolean()),
-
-	// Branch
-	action: Type.Optional(StringEnum(["list", "create", "delete", "rename", "current"])),
-	name: Type.Optional(Type.String()),
-	newName: Type.Optional(Type.String()),
-	startPoint: Type.Optional(Type.String()),
-	remotes: Type.Optional(Type.Boolean()),
-	force: Type.Optional(Type.Boolean()),
-
-	// Add/Restore
-	update: Type.Optional(Type.Boolean()),
-	all: Type.Optional(Type.Boolean()),
-	dry_run: Type.Optional(Type.Boolean()),
-	staged: Type.Optional(Type.Boolean()),
-	worktree: Type.Optional(Type.Boolean()),
-	source: Type.Optional(Type.String()),
-
-	// Commit
-	message: Type.Optional(Type.String()),
-	allow_empty: Type.Optional(Type.Boolean()),
-	sign: Type.Optional(Type.Boolean()),
-	no_verify: Type.Optional(Type.Boolean()),
-	amend: Type.Optional(Type.Boolean()),
-
-	// Checkout
-	create: Type.Optional(Type.Boolean()),
-
-	// Merge
-	no_ff: Type.Optional(Type.Boolean()),
-	ff_only: Type.Optional(Type.Boolean()),
-	squash: Type.Optional(Type.Boolean()),
-	abort: Type.Optional(Type.Boolean()),
-	continue: Type.Optional(Type.Boolean()),
-
-	// Rebase
-	onto: Type.Optional(Type.String()),
-	upstream: Type.Optional(Type.String()),
-	skip: Type.Optional(Type.Boolean()),
-
-	// Stash
-	include_untracked: Type.Optional(Type.Boolean()),
-	index: Type.Optional(Type.Number()),
-	keep_index: Type.Optional(Type.Boolean()),
-
-	// Cherry-pick
-	commits: Type.Optional(Type.Array(Type.String())),
-	no_commit: Type.Optional(Type.Boolean()),
-
-	// Fetch/Pull/Push/Tag
-	remote: Type.Optional(Type.String()),
-	branch: Type.Optional(Type.String()),
-	prune: Type.Optional(Type.Boolean()),
-	tags: Type.Optional(Type.Boolean()),
-	rebase: Type.Optional(Type.Boolean()),
-	set_upstream: Type.Optional(Type.Boolean()),
-	force_with_lease: Type.Optional(Type.Boolean()),
-	delete: Type.Optional(Type.Boolean()),
-	force_override: Type.Optional(Type.Boolean()),
-
-	// Tag
-	// (name/message/ref already covered)
-
-	// PR
-	number: Type.Optional(Type.Number()),
-	title: Type.Optional(Type.String()),
-	body: Type.Optional(Type.String()),
-	base: Type.Optional(Type.String()),
-	head: Type.Optional(Type.String()),
-	draft: Type.Optional(Type.Boolean()),
-	state: Type.Optional(StringEnum(["open", "closed", "merged", "all"])),
-	merge_method: Type.Optional(StringEnum(["merge", "squash", "rebase"])),
-	review_action: Type.Optional(StringEnum(["approve", "request-changes", "comment"])),
-	review_body: Type.Optional(Type.String()),
-
-	// Issue
-	labels: Type.Optional(Type.Array(Type.String())),
-	assignee: Type.Optional(Type.String()),
-	comment_body: Type.Optional(Type.String()),
-
-	// CI
-	workflow: Type.Optional(Type.String()),
-	run_id: Type.Optional(Type.Number()),
-	inputs: Type.Optional(Type.Record(Type.String(), Type.String())),
-	logs_failed: Type.Optional(Type.Boolean()),
-
-	// Release
-	notes: Type.Optional(Type.String()),
-	generate_notes: Type.Optional(Type.Boolean()),
-	prerelease: Type.Optional(Type.Boolean()),
-	assets: Type.Optional(Type.Array(Type.String())),
-});
-
-export type GitToolDetails = ToolResponse<unknown>;
-
-export class GitTool implements AgentTool<typeof gitSchema, GitToolDetails> {
-	public readonly name = "git";
-	public readonly label = "Git";
-	public readonly description: string;
-	public readonly parameters = gitSchema;
-
-	private readonly session: ToolSession;
-
-	constructor(session: ToolSession) {
-		this.session = session;
-		this.description = renderPromptTemplate(gitDescription);
-	}
-
-	static createIf(session: ToolSession): GitTool | null {
-		return session.settings?.getGitToolEnabled() === false ? null : new GitTool(session);
-	}
-
-	public async execute(
-		_toolCallId: string,
-		params: Static<typeof gitSchema>,
-		_signal?: AbortSignal,
-		_onUpdate?: AgentToolUpdateCallback<GitToolDetails>,
-		_context?: AgentToolContext,
-	): Promise<AgentToolResult<GitToolDetails>> {
-		if (params.operation === "commit" && !params.message) {
-			throw new Error("Git commit requires a message to avoid an interactive editor. Provide `message`.");
-		}
-
-		const result = await gitToolCore(params as GitParams, this.session.cwd);
-		if ("error" in result) {
-			const message = result._rendered ?? result.error;
-			return { content: [{ type: "text", text: message }], details: result };
-		}
-		if ("confirm" in result) {
-			const message = result._rendered ?? result.confirm;
-			return { content: [{ type: "text", text: message }], details: result };
-		}
-		return { content: [{ type: "text", text: result._rendered }], details: result };
-	}
-}

package/src/core/voice-controller.ts

@@ -1,135 +0,0 @@
-import { logger } from "@oh-my-pi/pi-utils";
-import { Agent, run, setDefaultOpenAIKey } from "@openai/agents";
-import { z } from "zod";
-import type { ModelRegistry } from "./model-registry";
-
-const DEFAULT_CONTROLLER_MODEL = process.env.OMP_VOICE_CONTROLLER_MODEL ?? "gpt-4o-mini";
-const DEFAULT_SUMMARY_MODEL = process.env.OMP_VOICE_SUMMARY_MODEL ?? DEFAULT_CONTROLLER_MODEL;
-const MAX_INPUT_CHARS = 8000;
-
-export type VoiceSteeringDecision = { action: "pass" | "ask"; text: string };
-export type VoicePresentationDecision = { action: "skip" | "speak"; text?: string };
-type VoiceSummaryOutput = { text: string };
-
-const steeringSchema: z.ZodType<VoiceSteeringDecision> = z.object({
-	action: z.enum(["pass", "ask"]),
-	text: z.string().min(1),
-});
-
-const presentationSchema: z.ZodType<VoicePresentationDecision> = z.object({
-	action: z.enum(["skip", "speak"]),
-	text: z.string().min(1).optional(),
-});
-
-const summarySchema: z.ZodType<VoiceSummaryOutput> = z.object({
-	text: z.string().min(1),
-});
-
-function normalizeText(text: string): string {
-	return text.replace(/\s+/g, " ").trim();
-}
-
-function truncateText(text: string, maxChars: number): string {
-	if (text.length <= maxChars) return text;
-	return `${text.slice(0, maxChars)}...`;
-}
-
-export class VoiceController {
-	private lastApiKey: string | undefined;
-
-	constructor(private registry: ModelRegistry) {}
-
-	private async ensureApiKey(): Promise<string | null> {
-		const apiKey = await this.registry.getApiKeyForProvider("openai");
-		if (!apiKey) {
-			logger.debug("voice-controller: no OpenAI API key available");
-			return null;
-		}
-		if (apiKey !== this.lastApiKey) {
-			setDefaultOpenAIKey(apiKey);
-			this.lastApiKey = apiKey;
-		}
-		return apiKey;
-	}
-
-	async steerUserInput(text: string): Promise<VoiceSteeringDecision | null> {
-		if (!(await this.ensureApiKey())) return null;
-
-		const normalized = truncateText(normalizeText(text), MAX_INPUT_CHARS);
-		const agent = new Agent({
-			name: "Voice Input Steering",
-			instructions:
-				"You are a voice-input controller for a coding agent. " +
-				"Given a user's speech transcript, decide if it is clear enough to send to the agent. " +
-				"If unclear or missing key details, ask exactly one short question. " +
-				"If clear, rewrite it as a concise instruction for the agent. " +
-				"Keep it short and preserve intent.",
-			model: DEFAULT_CONTROLLER_MODEL,
-			outputType: steeringSchema,
-		});
-
-		try {
-			const result = await run(agent, normalized);
-			return result.finalOutput ?? null;
-		} catch (error) {
-			logger.debug("voice-controller: steering error", {
-				error: error instanceof Error ? error.message : String(error),
-			});
-			return null;
-		}
-	}
-
-	async decidePresentation(text: string): Promise<VoicePresentationDecision | null> {
-		if (!(await this.ensureApiKey())) return null;
-
-		const normalized = truncateText(normalizeText(text), MAX_INPUT_CHARS);
-		const agent = new Agent({
-			name: "Voice Presentation Gate",
-			instructions:
-				"You are a voice presentation gate for a coding agent. " +
-				"Decide whether to speak the assistant response to the user. " +
-				"Speak when there is a decision, summary, or a question for the user. " +
-				"Skip if it is mostly tool output, verbose logs, or not useful to speak. " +
-				"When speaking, respond in 1-3 short sentences (<=45 words) in a casual, concise tone. " +
-				"If user input is needed, ask exactly one short question.",
-			model: DEFAULT_CONTROLLER_MODEL,
-			outputType: presentationSchema,
-		});
-
-		try {
-			const result = await run(agent, normalized);
-			return result.finalOutput ?? null;
-		} catch (error) {
-			logger.debug("voice-controller: presentation error", {
-				error: error instanceof Error ? error.message : String(error),
-			});
-			return null;
-		}
-	}
-
-	async summarizeForVoice(text: string): Promise<string | null> {
-		if (!(await this.ensureApiKey())) return null;
-
-		const normalized = truncateText(normalizeText(text), MAX_INPUT_CHARS);
-		const agent = new Agent({
-			name: "Voice Summary",
-			instructions:
-				"Summarize the assistant response for voice playback. " +
-				"Use 1-2 short sentences. " +
-				"If a question is required from the user, ask one short question.",
-			model: DEFAULT_SUMMARY_MODEL,
-			outputType: summarySchema,
-		});
-
-		try {
-			const result = await run(agent, normalized);
-			const output = result.finalOutput?.text ?? "";
-			return output.trim() || null;
-		} catch (error) {
-			logger.debug("voice-controller: summary error", {
-				error: error instanceof Error ? error.message : String(error),
-			});
-			return null;
-		}
-	}
-}