@oh-my-pi/pi-coding-agent 9.2.5 → 9.3.1

package/CHANGELOG.md

@@ -2,6 +2,22 @@
 
 ## [Unreleased]
 
+## [9.3.1] - 2026-01-31
+### Added
+
+- Added `getCompactContext()` API to retrieve parent conversation context for subagents, excluding system prompts and tool results
+- Added automatic `submit_result` tool injection for subagents with explicit tool lists
+- Added `contextFile` parameter to pass parent conversation context to subagent sessions
+
+### Changed
+
+- Updated subagent system prompt to reference parent conversation context file when available
+- Enhanced subagent system prompt formatting with clearer backtick notation for tool and parameter names
+
+### Removed
+
+- Removed schema override notification from task summary prompt
+
 ## [9.2.5] - 2026-01-31
 ### Changed
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "9.2.5",
+	"version": "9.3.1",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"ompConfig": {
@@ -79,12 +79,12 @@
 		"test": "bun test"
 	},
 	"dependencies": {
-		"@oh-my-pi/omp-stats": "9.2.5",
-		"@oh-my-pi/pi-agent-core": "9.2.5",
-		"@oh-my-pi/pi-ai": "9.2.5",
-		"@oh-my-pi/pi-natives": "9.2.5",
-		"@oh-my-pi/pi-tui": "9.2.5",
-		"@oh-my-pi/pi-utils": "9.2.5",
+		"@oh-my-pi/omp-stats": "9.3.1",
+		"@oh-my-pi/pi-agent-core": "9.3.1",
+		"@oh-my-pi/pi-ai": "9.3.1",
+		"@oh-my-pi/pi-natives": "9.3.1",
+		"@oh-my-pi/pi-tui": "9.3.1",
+		"@oh-my-pi/pi-utils": "9.3.1",
 		"@openai/agents": "^0.4.4",
 		"@sinclair/typebox": "^0.34.48",
 		"ajv": "^8.17.1",

package/src/discovery/helpers.ts

@@ -189,7 +189,12 @@ export function parseAgentFields(frontmatter: Record<string, unknown>): ParsedAg
 		return null;
 	}
 
-	const tools = parseArrayOrCSV(frontmatter.tools);
+	let tools = parseArrayOrCSV(frontmatter.tools);
+
+	// Subagents with explicit tool lists always need submit_result
+	if (tools && !tools.includes("submit_result")) {
+		tools = [...tools, "submit_result"];
+	}
 
 	// Parse spawns field (array, "*", or CSV)
 	let spawns: string[] | "*" | undefined;

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

@@ -1,24 +1,31 @@
 {{base}}
------------------------------------
+
+====================================================
 
 {{agent}}
 
+{{#if contextFile}}
+<context>
+If you need additional context about the parent conversation, check {{contextFile}} (e.g., `tail -100` or `grep` for relevant terms).
+</context>
+{{/if}}
+
 <critical>
 {{#if worktree}}
 - You MUST work under this working tree: {{worktree}}. Do not modify anything under the original repository.
 {{/if}}
-- You MUST call the submit_result tool exactly once when finished. Do not output JSON in text. Do not end with a plain-text summary. Call submit_result with your result as the data parameter.
+- You MUST call the `submit_result` tool exactly once when finished. Do not output JSON in text. Do not end with a plain-text summary. Call `submit_result` with your result as the `data` parameter.
 {{#if outputSchema}}
-- If you cannot complete the task, call submit_result with status="aborted" and an error message. Do not provide a success result or pretend completion.
+- If you cannot complete the task, call `submit_result` with `status="aborted"` and an error message. Do not provide a success result or pretend completion.
 {{else}}
-- If you cannot complete the task, call submit_result with status="aborted" and an error message. Do not claim success.
+- If you cannot complete the task, call `submit_result` with `status="aborted"` and an error message. Do not claim success.
 {{/if}}
 {{#if outputSchema}}
-- The data parameter MUST be valid JSON matching this TypeScript interface:
-```typescript
+- The `data` parameter MUST be valid JSON matching this TypeScript interface:
+```ts
 {{jtdToTypeScript outputSchema}}
 ```
 {{/if}}
-- If you cannot complete the task, call submit_result exactly once with a result that explicitly indicates failure or abort status (use a failure/notes field if available). Do not claim success.
+- If you cannot complete the task, call `submit_result` exactly once with a result that explicitly indicates failure or abort status (use a failure/notes field if available). Do not claim success.
 - Keep going until request is fully fulfilled. This matters.
 </critical>

package/src/prompts/tools/task-summary.md

@@ -20,13 +20,6 @@
 {{/unless}}
 {{/each}}
 
-{{#if schemaOverridden}}
-<schema-note>
-Note: Agent '{{agentName}}' has a fixed output schema:
-{{requiredSchema}}
-</schema-note>
-{{/if}}
-
 {{#if patchApplySummary}}
 <patch-summary>
 {{patchApplySummary}}

package/src/sdk.ts

@@ -760,6 +760,7 @@ export async function createAgentSession(options: CreateAgentSessionOptions = {}
 			return undefined;
 		},
 		getPlanModeState: () => session.getPlanModeState(),
+		getCompactContext: () => session.formatCompactContext(),
 		settings: settingsManager,
 		settingsManager,
 		authStorage,

package/src/session/agent-session.ts

@@ -3916,6 +3916,69 @@ Be thorough - include exact file paths, function names, error messages, and tech
 		return lines.join("\n").trim();
 	}
 
+	/**
+	 * Format the conversation as compact context for subagents.
+	 * Includes only user messages and assistant text responses.
+	 * Excludes: system prompt, tool definitions, tool calls/results, thinking blocks.
+	 */
+	formatCompactContext(): string {
+		const lines: string[] = [];
+		lines.push("# Conversation Context");
+		lines.push("");
+		lines.push(
+			"This is a summary of the parent conversation. Read this if you need additional context about what was discussed or decided.",
+		);
+		lines.push("");
+
+		for (const msg of this.messages) {
+			if (msg.role === "user") {
+				lines.push("## User");
+				lines.push("");
+				if (typeof msg.content === "string") {
+					lines.push(msg.content);
+				} else {
+					for (const c of msg.content) {
+						if (c.type === "text") {
+							lines.push(c.text);
+						} else if (c.type === "image") {
+							lines.push("[Image attached]");
+						}
+					}
+				}
+				lines.push("");
+			} else if (msg.role === "assistant") {
+				const assistantMsg = msg as AssistantMessage;
+				// Only include text content, skip tool calls and thinking
+				const textParts: string[] = [];
+				for (const c of assistantMsg.content) {
+					if (c.type === "text" && c.text.trim()) {
+						textParts.push(c.text);
+					}
+				}
+				if (textParts.length > 0) {
+					lines.push("## Assistant");
+					lines.push("");
+					lines.push(textParts.join("\n\n"));
+					lines.push("");
+				}
+			} else if (msg.role === "fileMention") {
+				const fileMsg = msg as FileMentionMessage;
+				const paths = fileMsg.files.map(f => f.path).join(", ");
+				lines.push(`[Files referenced: ${paths}]`);
+				lines.push("");
+			} else if (msg.role === "compactionSummary") {
+				const compactMsg = msg as CompactionSummaryMessage;
+				lines.push("## Earlier Context (Summarized)");
+				lines.push("");
+				lines.push(compactMsg.summary);
+				lines.push("");
+			}
+			// Skip: toolResult, bashExecution, pythonExecution, branchSummary, custom, hookMessage
+		}
+
+		return lines.join("\n").trim();
+	}
+
 	// =========================================================================
 	// Extension System
 	// =========================================================================

package/src/system-prompt.ts

@@ -11,7 +11,7 @@ import chalk from "chalk";
 import { contextFileCapability } from "./capability/context-file";
 import { systemPromptCapability } from "./capability/system-prompt";
 import { renderPromptTemplate } from "./config/prompt-templates";
-import type { SkillsSettings } from "./config/settings-manager";
+import { SettingsManager, type SkillsSettings } from "./config/settings-manager";
 import { type ContextFile, loadCapability, type SystemPrompt as SystemPromptFile } from "./discovery";
 import { loadSkills, type Skill } from "./extensibility/skills";
 import customSystemPromptTemplate from "./prompts/system/custom-system-prompt.md" with { type: "text" };
@@ -573,11 +573,6 @@ async function getGpuModel(): Promise<string | null> {
 	}
 }
 
-function getShellName(): string {
-	const shell = firstNonEmpty([process.env.SHELL, process.env.ComSpec]);
-	return shell ?? "unknown";
-}
-
 function getTerminalName(): string {
 	const termProgram = process.env.TERM_PROGRAM;
 	const termProgramVersion = process.env.TERM_PROGRAM_VERSION;
@@ -765,6 +760,9 @@ async function getEnvironmentInfo(): Promise<Array<{ label: string; value: strin
 		await saveSystemInfoCache(sysInfo);
 	}
 
+	// Get the actual shell used for command execution (not $SHELL)
+	const shellConfig = await SettingsManager.getGlobalShellConfig();
+
 	return [
 		{ label: "OS", value: sysInfo.os },
 		{ label: "Distro", value: sysInfo.distro },
@@ -773,7 +771,7 @@ async function getEnvironmentInfo(): Promise<Array<{ label: string; value: strin
 		{ label: "CPU", value: sysInfo.cpu },
 		{ label: "GPU", value: sysInfo.gpu },
 		{ label: "Disk", value: sysInfo.disk },
-		{ label: "Shell", value: getShellName() },
+		{ label: "Shell", value: shellConfig.shell },
 		{ label: "Terminal", value: getTerminalName() },
 		{ label: "DE", value: getDesktopEnvironment() },
 		{ label: "WM", value: getWindowManager() },

package/src/task/executor.ts

@@ -196,6 +196,8 @@ export interface ExecutorOptions {
 	sessionFile?: string | null;
 	persistArtifacts?: boolean;
 	artifactsDir?: string;
+	/** Path to parent conversation context file */
+	contextFile?: string;
 	eventBus?: EventBus;
 	contextFiles?: ContextFileEntry[];
 	skills?: Skill[];
@@ -948,6 +950,7 @@ export async function runSubprocess(options: ExecutorOptions): Promise<SingleRes
 						agent: agent.systemPrompt,
 						worktree: worktree ?? "",
 						outputSchema: normalizedOutputSchema,
+						contextFile: options.contextFile,
 					}),
 				sessionManager,
 				hasUI: false,

package/src/task/index.ts

@@ -211,7 +211,6 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 		const thinkingLevelOverride = effectiveAgent.thinkingLevel;
 
 		// Output schema priority: agent frontmatter > params > inherited from parent session
-		const schemaOverridden = outputSchema !== undefined && effectiveAgent.output !== undefined;
 		const effectiveOutputSchema = effectiveAgent.output ?? outputSchema ?? this.session.outputSchema;
 
 		// Handle empty or missing tasks
@@ -382,6 +381,15 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 				};
 			}
 
+			// Write parent conversation context for subagents
+			await fs.mkdir(effectiveArtifactsDir, { recursive: true });
+			const compactContext = this.session.getCompactContext?.();
+			let contextFilePath: string | undefined;
+			if (compactContext) {
+				contextFilePath = path.join(effectiveArtifactsDir, "context.md");
+				await Bun.write(contextFilePath, compactContext);
+			}
+
 			// Build full prompts with context prepended
 			// Allocate unique IDs across the session to prevent artifact collisions
 			const outputManager =
@@ -478,6 +486,7 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 						sessionFile,
 						persistArtifacts: !!artifactsDir,
 						artifactsDir: effectiveArtifactsDir,
+						contextFile: contextFilePath,
 						enableLsp: false,
 						signal,
 						eventBus: undefined,
@@ -522,6 +531,7 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 						sessionFile,
 						persistArtifacts: !!artifactsDir,
 						artifactsDir: effectiveArtifactsDir,
+						contextFile: contextFilePath,
 						enableLsp: false,
 						signal,
 						eventBus: undefined,
@@ -735,9 +745,7 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 				duration: formatDuration(totalDuration),
 				summaries,
 				outputIds,
-				schemaOverridden,
 				agentName,
-				requiredSchema: agent.output ? JSON.stringify(agent.output) : "",
 				patchApplySummary,
 			});
 

package/src/tools/index.ts

@@ -173,6 +173,8 @@ export interface ToolSession {
 	};
 	/** Plan mode state (if active) */
 	getPlanModeState?: () => PlanModeState | undefined;
+	/** Get compact conversation context for subagents (excludes tool results, system prompts) */
+	getCompactContext?: () => string;
 }
 
 type ToolFactory = (session: ToolSession) => Tool | null | Promise<Tool | null>;