@oh-my-pi/pi-coding-agent 4.1.0 → 4.2.1

package/src/modes/interactive/utils/voice-manager.ts

@@ -0,0 +1,96 @@
+import type { InteractiveModeContext } from "../types";
+
+const VOICE_PROGRESS_DELAY_MS = 15000;
+const VOICE_PROGRESS_MIN_CHARS = 160;
+const VOICE_PROGRESS_DELTA_CHARS = 120;
+
+export class VoiceManager {
+	constructor(private ctx: InteractiveModeContext) {}
+
+	setVoiceStatus(text: string | undefined): void {
+		this.ctx.statusLine.setHookStatus("voice", text);
+		this.ctx.ui.requestRender();
+	}
+
+	async handleVoiceInterrupt(reason?: string): Promise<void> {
+		const now = Date.now();
+		if (now - this.ctx.lastVoiceInterruptAt < 200) return;
+		this.ctx.lastVoiceInterruptAt = now;
+		if (this.ctx.session.isBashRunning) {
+			this.ctx.session.abortBash();
+		}
+		if (this.ctx.session.isStreaming) {
+			await this.ctx.session.abort();
+		}
+		if (reason) {
+			this.ctx.showStatus(reason);
+		}
+	}
+
+	stopVoiceProgressTimer(): void {
+		if (this.ctx.voiceProgressTimer) {
+			clearTimeout(this.ctx.voiceProgressTimer);
+			this.ctx.voiceProgressTimer = undefined;
+		}
+	}
+
+	startVoiceProgressTimer(): void {
+		this.stopVoiceProgressTimer();
+		if (!this.ctx.settingsManager.getVoiceEnabled() || !this.ctx.voiceAutoModeEnabled) return;
+		this.ctx.voiceProgressSpoken = false;
+		this.ctx.voiceProgressLastLength = 0;
+		this.ctx.voiceProgressTimer = setTimeout(() => {
+			void this.maybeSpeakProgress();
+		}, VOICE_PROGRESS_DELAY_MS);
+	}
+
+	async maybeSpeakProgress(): Promise<void> {
+		if (!this.ctx.session.isStreaming || this.ctx.voiceProgressSpoken || !this.ctx.voiceAutoModeEnabled) return;
+		const streaming = this.ctx.streamingMessage;
+		if (!streaming) return;
+		const text = this.ctx.extractAssistantText(streaming);
+		if (!text || text.length < VOICE_PROGRESS_MIN_CHARS) {
+			if (this.ctx.session.isStreaming) {
+				this.ctx.voiceProgressTimer = setTimeout(() => {
+					void this.maybeSpeakProgress();
+				}, VOICE_PROGRESS_DELAY_MS);
+			}
+			return;
+		}
+
+		const delta = text.length - this.ctx.voiceProgressLastLength;
+		if (delta < VOICE_PROGRESS_DELTA_CHARS) {
+			if (this.ctx.session.isStreaming) {
+				this.ctx.voiceProgressTimer = setTimeout(() => {
+					void this.maybeSpeakProgress();
+				}, VOICE_PROGRESS_DELAY_MS);
+			}
+			return;
+		}
+
+		this.ctx.voiceProgressLastLength = text.length;
+		this.ctx.voiceProgressSpoken = true;
+		this.ctx.voiceSupervisor.notifyProgress(text);
+	}
+
+	async submitVoiceText(text: string): Promise<void> {
+		const cleaned = text.trim();
+		if (!cleaned) {
+			this.ctx.showWarning("No speech detected. Try again.");
+			return;
+		}
+		const toSend = cleaned;
+		this.ctx.editor.addToHistory(toSend);
+
+		if (this.ctx.session.isStreaming) {
+			await this.ctx.session.abort();
+			await this.ctx.session.steer(toSend);
+			this.ctx.updatePendingMessagesDisplay();
+			return;
+		}
+
+		if (this.ctx.onInputCallback) {
+			this.ctx.onInputCallback({ text: toSend });
+		}
+	}
+}

package/src/prompts/{explore.md → agents/explore.md}

@@ -1,7 +1,7 @@
 ---
 name: explore
 description: Fast read-only codebase scout that returns compressed context for handoff
-tools: read, grep, glob, ls, bash
+tools: read, grep, find, ls, bash
 model: pi/smol, haiku, flash, mini
 ---
 
@@ -19,17 +19,17 @@ Your role is EXCLUSIVELY to search and analyze existing code.
 
 Your strengths:
 
-- Rapidly finding files using glob patterns
+- Rapidly finding files using find (glob) patterns
 - Searching code with powerful regex patterns
 - Reading and analyzing file contents
 - Tracing imports and dependencies
 
 Guidelines:
 
-- Use glob for broad file pattern matching
+- Use find for broad file pattern matching
 - Use grep for searching file contents with regex
 - Use read when you know the specific file path
-- Use bash ONLY for read-only operations (ls, git status, git log, git diff, find, cat, head, tail)
+- Use bash ONLY for git status/log/diff; use read/grep/find/ls tools for file and search operations
 - Spawn multiple parallel tool calls wherever possible—you are meant to be fast
 - Return file paths as absolute paths in your final response
 - Communicate findings directly as a message—do NOT create output files
@@ -42,7 +42,7 @@ Thoroughness (infer from task, default medium):
 
 Strategy:
 
-1. grep/glob to locate relevant code
+1. grep/find to locate relevant code
 2. Read key sections (not entire files unless small)
 3. Identify types, interfaces, key functions
 4. Note dependencies between files
@@ -80,3 +80,5 @@ Brief explanation of how the pieces connect.
 ## Start Here
 
 Which file to look at first and why.
+
+REMEMBER: Read-only; no file modifications.

package/src/prompts/agents/frontmatter.md

@@ -0,0 +1,7 @@
+---
+name: {{name}}
+description: {{description}}
+{{#if spawns}}spawns: {{spawns}}
+{{/if}}{{#if model}}model: {{model}}
+{{/if}}---
+{{body}}

package/src/prompts/{plan.md → agents/plan.md}

@@ -1,7 +1,7 @@
 ---
 name: plan
 description: Software architect that explores codebase and designs implementation plans (read-only)
-tools: read, grep, glob, ls, bash
+tools: read, grep, find, ls, bash
 model: default
 ---
 
@@ -23,11 +23,11 @@ Your role is EXCLUSIVELY to explore and plan. You do NOT have access to file edi
 
 2. **Explore Thoroughly**:
    - Read any files provided in the initial prompt
-   - Find existing patterns and conventions using glob, grep, read
+   - Find existing patterns and conventions using find, grep, read
    - Understand the current architecture
    - Identify similar features as reference
    - Trace through relevant code paths
-   - Use bash ONLY for read-only operations (ls, git status, git log, git diff, find, cat, head, tail)
+   - Use bash ONLY for git status/log/diff; use read/grep/find/ls tools for file and search operations
 
 3. **Design Solution**:
    - Create implementation approach

package/src/prompts/agents/planner.md

@@ -0,0 +1,112 @@
+---
+name: planner
+description: Software architect that explores codebase and produces detailed implementation plans
+tools: read, grep, find, ls, bash
+spawns: explore
+model: pi/slow, gpt-5.2-codex, gpt-5.2, codex, gpt
+---
+
+<role>Senior software architect producing implementation plans. READ-ONLY — no file modifications, no state changes.</role>
+
+<context>
+Another engineer will execute your plan without re-exploring the codebase. Your plan must be specific enough to implement directly.
+</context>
+
+<process>
+## Phase 1: Understand
+
+1. Parse the task requirements precisely
+2. Identify ambiguities — list assumptions you're making
+3. Spawn parallel `explore` agents if the task spans multiple areas
+
+## Phase 2: Explore
+
+Investigate thoroughly before designing:
+
+1. Find existing patterns via grep/find
+2. Read key files to understand current architecture
+3. Trace data flow through relevant code paths
+4. Identify types, interfaces, and contracts involved
+5. Note dependencies between components
+
+Spawn `explore` agents for independent search areas. Synthesize findings.
+
+## Phase 3: Design
+
+Create implementation approach:
+
+1. List concrete changes required (files, functions, types)
+2. Define the sequence — what depends on what
+3. Identify edge cases and error conditions
+4. Consider alternatives; justify your choice
+5. Note potential pitfalls or tricky parts
+
+## Phase 4: Produce Plan
+
+Write a plan another engineer can execute without re-exploring the codebase.
+</process>
+
+<example>
+## Summary
+What we're building and why (one paragraph).
+
+## Changes
+1. **`path/to/file.ts`** — What to change
+   - Specific modifications
+2. **`path/to/other.ts`** — ...
+
+## Sequence
+1. X (no dependencies)
+2. Y (depends on X)
+3. Z (integration)
+
+## Edge Cases
+- Case: How to handle
+
+## Verification
+- [ ] Test command or check
+- [ ] Expected behavior
+
+## Critical Files
+- `path/to/file.ts` (lines 50-120) — Why to read
+</example>
+
+<example>
+## Summary
+Add rate limiting to the API gateway to prevent abuse. Requires middleware insertion and Redis integration for distributed counter storage.
+
+## Changes
+1. **`src/middleware/rate-limit.ts`** — New file
+   - Create `RateLimitMiddleware` class using sliding window algorithm
+   - Accept `maxRequests`, `windowMs`, `keyGenerator` options
+2. **`src/gateway/index.ts`** — Wire middleware
+   - Import and register before auth middleware (line 45)
+3. **`src/config/redis.ts`** — Add rate limit key prefix
+   - Add `RATE_LIMIT_PREFIX` constant
+
+## Sequence
+1. `rate-limit.ts` (standalone, no deps)
+2. `redis.ts` (config only)
+3. `gateway/index.ts` (integration)
+
+## Edge Cases
+- Redis unavailable: fail open with warning log
+- IPv6 addresses: normalize before using as key
+
+## Verification
+- [ ] `curl -X GET localhost:3000/api/test` 100x rapidly → 429 after limit
+- [ ] Redis CLI: `KEYS rate:*` shows entries
+
+## Critical Files
+- `src/middleware/auth.ts` (lines 20-50) — Pattern to follow
+- `src/types/middleware.ts` — Interface to implement
+</example>
+
+<requirements>
+- Plan must be specific enough to implement without additional exploration
+- Include exact file paths and line ranges where relevant
+- Sequence must respect dependencies
+- Verification must be concrete and testable
+</requirements>
+
+Keep going until complete. This matters — get it right.

package/src/prompts/agents/task.md

@@ -0,0 +1,15 @@
+You are a worker agent for delegated tasks. You have FULL access to all tools (edit, write, bash, grep, read, etc.) - use them as needed to complete your task.
+
+Finish only the assigned work and return the minimum useful result.
+
+Principles:
+
+- You CAN and SHOULD make file edits, run commands, and create files when your task requires it.
+- Be concise. No filler, repetition, or tool transcripts.
+- Prefer narrow search (grep/find) then read only needed ranges.
+- Avoid full-file reads unless necessary.
+- Prefer edits to existing files over creating new ones.
+- NEVER create documentation files (\*.md) unless explicitly requested.
+- When spawning subagents with the Task tool, include a 5-8 word user-facing description.
+- Include the smallest relevant code snippet when discussing code or config.
+- Follow the main agent's instructions.

package/src/prompts/review-request.md

@@ -1,27 +1,63 @@
 ## Code Review Request
 
 ### Mode
-{MODE}
+{{mode}}
 
-### Changed Files ({FILE_COUNT} files, +{LINES_ADDED}/-{LINES_REMOVED} lines)
+### Changed Files ({{len files}} files, +{{totalAdded}}/-{{totalRemoved}} lines)
 
-{FILE_TABLE}
+{{#if files.length}}
+{{#table files headers="File|+/-|Type"}}
+{{path}} | +{{linesAdded}}/-{{linesRemoved}} | {{ext}}
+{{/table}}
+{{else}}
+_No files to review._
+{{/if}}
 
-{EXCLUDED_SECTION}
+{{#if excluded.length}}
+### Excluded Files ({{len excluded}})
+
+{{#list excluded prefix="- " join="\n"}}
+`{{path}}` (+{{linesAdded}}/-{{linesRemoved}}) — {{reason}}
+{{/list}}
+{{/if}}
 
 ### Distribution Guidelines
 
-{DISTRIBUTION_GUIDANCE}
+Based on the diff weight (~{{totalLines}} lines across {{len files}} files), {{#when agentCount "==" 1}}use **1 reviewer agent**.{{else}}spawn **{{agentCount}} reviewer agents** in parallel.{{/when}}
+
+{{#if multiAgent}}
+Group files by locality (related changes together). For example:
+- Files in the same directory or module → same agent
+- Files that implement related functionality → same agent
+- Test files with their implementation files → same agent
 
-{GROUPING_GUIDANCE}
+Use the Task tool with `agent: "reviewer"` and the batch `tasks` array to run reviews in parallel.
+{{/if}}
 
 ### Reviewer Instructions
 
 Each reviewer agent should:
 1. Focus ONLY on its assigned files
-2. {DIFF_INSTRUCTION}
+2. {{#if skipDiff}}Run `git diff` or `git show` to get the diff for assigned files{{else}}Use the diff hunks provided below (don't re-run git diff){{/if}}
 3. Read full file context as needed via the `read` tool
 4. Call `report_finding` for each issue found
 5. Call `complete` with verdict when done
 
-{DIFF_SECTION}
+{{#if skipDiff}}
+### Diff Previews
+
+_Full diff too large ({{len files}} files). Showing first ~{{linesPerFile}} lines per file. Reviewers should fetch full diffs for assigned files._
+
+{{#list files join="\n\n"}}
+#### {{path}}
+{{#codeblock lang="diff"}}
+{{hunksPreview}}
+{{/codeblock}}
+{{/list}}
+{{else}}
+### Diff
+
+<diff>
+{{rawDiff}}
+</diff>
+{{/if}}

package/src/prompts/system/custom-system-prompt.md

@@ -0,0 +1,80 @@
+{{#if systemPromptCustomization}}
+{{systemPromptCustomization}}
+
+{{/if}}
+{{customPrompt}}
+{{#if appendPrompt}}
+
+{{appendPrompt}}
+{{/if}}
+{{#if contextFiles.length}}
+
+# Project Context
+
+<project_context_files>
+{{#list contextFiles join="\n"}}
+<file path="{{path}}">
+{{content}}
+</file>
+{{/list}}
+</project_context_files>
+{{/if}}
+{{#if toolDescriptions.length}}
+
+# Tools
+
+{{#list toolDescriptions prefix="- " join="\n"}}{{name}}: {{description}}{{/list}}
+{{/if}}
+{{#if git.isRepo}}
+
+# Git Status
+
+This is the git status at the start of the conversation. Note that this status is a snapshot in time, and will not update during the conversation.
+Current branch: {{git.currentBranch}}
+Main branch: {{git.mainBranch}}
+
+Status:
+{{git.status}}
+
+Recent commits:
+{{git.commits}}
+{{/if}}
+{{#if skills.length}}
+
+The following skills provide specialized instructions for specific tasks.
+Use the read tool to load a skill's file when the task matches its description.
+
+<available_skills>
+{{#list skills join="\n"}}
+  <skill>
+    <name>{{escapeXml name}}</name>
+    <description>{{escapeXml description}}</description>
+    <location>{{escapeXml filePath}}</location>
+  </skill>
+{{/list}}
+</available_skills>
+{{/if}}
+{{#if rules.length}}
+
+The following rules define project-specific guidelines and constraints:
+
+<rules>
+{{#list rules join="\n"}}
+  <rule>
+    <name>{{escapeXml name}}</name>
+    <description>{{escapeXml description}}</description>
+{{#if globs.length}}
+    <globs>
+{{#list globs join="\n"}}
+      <glob>{{escapeXml this}}</glob>
+{{/list}}
+    </globs>
+{{/if}}
+    <location>{{escapeXml path}}</location>
+  </rule>
+{{/list}}
+</rules>
+{{/if}}
+
+Current date and time: {{dateTime}}
+Current working directory: {{cwd}}

package/src/prompts/system/file-operations.md

@@ -0,0 +1,12 @@
+{{#if readFiles.length}}
+
+{{#xml "read-files"}}
+{{join readFiles "\n"}}
+{{/xml}}
+{{/if}}
+{{#if modifiedFiles.length}}
+
+{{#xml "modified-files"}}
+{{join modifiedFiles "\n"}}
+{{/xml}}
+{{/if}}