@oh-my-pi/pi-coding-agent 12.2.1 → 12.4.0

package/src/modes/controllers/command-controller.ts

@@ -17,6 +17,7 @@ import { reset as resetCapabilities } from "../../capability";
 import { loadCustomShare } from "../../export/custom-share";
 import type { CompactOptions } from "../../extensibility/extensions/types";
 import { getGatewayStatus } from "../../ipy/gateway-coordinator";
+import { buildMemoryToolDeveloperInstructions, clearMemoryData, enqueueMemoryConsolidation } from "../../memories";
 import { BashExecutionComponent } from "../../modes/components/bash-execution";
 import { BorderedLoader } from "../../modes/components/bordered-loader";
 import { DynamicBorder } from "../../modes/components/dynamic-border";
@@ -408,6 +409,51 @@ export class CommandController {
 		this.ctx.ui.requestRender();
 	}
 
+	async handleMemoryCommand(text: string): Promise<void> {
+		const argumentText = text.slice(7).trim();
+		const action = argumentText.split(/\s+/, 1)[0]?.toLowerCase() || "view";
+		const agentDir = this.ctx.settings.getAgentDir();
+
+		if (action === "view") {
+			const payload = await buildMemoryToolDeveloperInstructions(agentDir, this.ctx.settings);
+			if (!payload) {
+				this.ctx.showWarning("Memory payload is empty (memories disabled or no memory summary found).");
+				return;
+			}
+			this.ctx.chatContainer.addChild(new Spacer(1));
+			this.ctx.chatContainer.addChild(new DynamicBorder());
+			this.ctx.chatContainer.addChild(new Text(theme.bold(theme.fg("accent", "Memory Injection Payload")), 1, 0));
+			this.ctx.chatContainer.addChild(new Spacer(1));
+			this.ctx.chatContainer.addChild(new Markdown(payload, 1, 1, getMarkdownTheme()));
+			this.ctx.chatContainer.addChild(new DynamicBorder());
+			this.ctx.ui.requestRender();
+			return;
+		}
+
+		if (action === "reset" || action === "clear") {
+			try {
+				await clearMemoryData(agentDir, this.ctx.sessionManager.getCwd());
+				await this.ctx.session.refreshBaseSystemPrompt();
+				this.ctx.showStatus("Memory data cleared and system prompt refreshed.");
+			} catch (error) {
+				this.ctx.showError(`Memory clear failed: ${error instanceof Error ? error.message : String(error)}`);
+			}
+			return;
+		}
+
+		if (action === "enqueue" || action === "rebuild") {
+			try {
+				enqueueMemoryConsolidation(agentDir);
+				this.ctx.showStatus("Memory consolidation enqueued.");
+			} catch (error) {
+				this.ctx.showError(`Memory enqueue failed: ${error instanceof Error ? error.message : String(error)}`);
+			}
+			return;
+		}
+
+		this.ctx.showError("Usage: /memory <view|clear|reset|enqueue|rebuild>");
+	}
+
 	async handleClearCommand(): Promise<void> {
 		if (this.ctx.loadingAnimation) {
 			this.ctx.loadingAnimation.stop();

package/src/modes/controllers/input-controller.ts

@@ -333,6 +333,11 @@ export class InputController {
 				this.ctx.editor.setText("");
 				return;
 			}
+			if (text === "/memory" || text.startsWith("/memory ")) {
+				this.ctx.editor.setText("");
+				await this.ctx.handleMemoryCommand(text);
+				return;
+			}
 			if (text === "/resume") {
 				this.ctx.showSessionSelector();
 				this.ctx.editor.setText("");

package/src/modes/interactive-mode.ts

@@ -919,6 +919,10 @@ export class InteractiveMode implements InteractiveModeContext {
 		return this.#commandController.handleMoveCommand(targetPath);
 	}
 
+	handleMemoryCommand(text: string): Promise<void> {
+		return this.#commandController.handleMemoryCommand(text);
+	}
+
 	showDebugSelector(): void {
 		this.#selectorController.showDebugSelector();
 	}

package/src/modes/types.ts

@@ -151,6 +151,7 @@ export interface InteractiveModeContext {
 	handleCompactCommand(customInstructions?: string): Promise<void>;
 	handleHandoffCommand(customInstructions?: string): Promise<void>;
 	handleMoveCommand(targetPath: string): Promise<void>;
+	handleMemoryCommand(text: string): Promise<void>;
 	executeCompaction(customInstructionsOrOptions?: string | CompactOptions, isAuto?: boolean): Promise<void>;
 	openInBrowser(urlOrPath: string): void;
 	refreshSlashCommandState(cwd?: string): Promise<void>;

package/src/prompts/memories/consolidation.md

@@ -0,0 +1,30 @@
+You are the memory consolidation agent.
+Memory root: {{memory_root}}
+Input corpus (raw memories):
+{{raw_memories}}
+Input corpus (rollout summaries):
+{{rollout_summaries}}
+Produce strict JSON only with this schema:
+{
+  "memory_md": "string",
+  "memory_summary": "string",
+  "skills": [
+    {
+      "name": "string",
+      "content": "string",
+      "scripts": [{ "path": "string", "content": "string" }],
+      "templates": [{ "path": "string", "content": "string" }],
+      "examples": [{ "path": "string", "content": "string" }]
+    }
+  ]
+}
+Requirements:
+- memory_md: full long-term memory document, curated and readable.
+- memory_summary: compact prompt-time memory guidance.
+- skills: reusable procedural playbooks. Empty array allowed.
+- Each skill.name maps to skills/<name>/.
+- Each skill.content maps to skills/<name>/SKILL.md.
+- scripts/templates/examples are optional. When present, each entry writes to skills/<name>/<bucket>/<path>.
+- Only include files worth keeping long-term; omit stale assets so they are pruned.
+- Preserve useful prior themes; remove stale or contradictory guidance.
+- Keep memory advisory: current repository state wins.

package/src/prompts/memories/read_path.md

@@ -0,0 +1,11 @@
+# Memory Guidance
+Memory root: {{base_path}}
+Operational rules:
+1) Read `{{base_path}}/memory_summary.md` first.
+2) If needed, inspect `{{base_path}}/MEMORY.md` and `{{base_path}}/skills/*/SKILL.md`.
+3) Decision boundary: trust memory for heuristics/process context; trust current repo files, runtime output, and user instruction for factual state and final decisions.
+4) Citation policy: when memory changes your plan, cite the memory artifact path you used (for example `memories/skills/<name>/SKILL.md`) and pair it with current-repo evidence before acting.
+5) Conflict workflow: if memory disagrees with repo state or user instruction, prefer repo/user, treat memory as stale, proceed with corrected behavior, then update/regenerate memory artifacts through normal execution.
+6) Escalate confidence only after repository verification; memory alone is never sufficient proof.
+Memory summary:
+{{memory_summary}}

package/src/prompts/memories/stage_one_input.md

@@ -0,0 +1,7 @@
+rollout_path: {{rollout_path}}
+cwd: {{cwd}}
+
+Persistable response items (JSON):
+{{response_items_json}}
+
+Extract durable memory now.

package/src/prompts/memories/stage_one_system.md

@@ -0,0 +1,21 @@
+You are memory-stage-one extractor.
+
+Return strict JSON only, no markdown, no commentary.
+
+Extraction goals:
+- Distill reusable durable knowledge from rollout history.
+- Keep concrete technical signal (constraints, decisions, workflows, pitfalls, resolved failures).
+- Exclude transient chatter and low-signal noise.
+
+Output contract (required keys):
+{
+  "rollout_summary": "string",
+  "rollout_slug": "string | null",
+  "raw_memory": "string"
+}
+
+Rules:
+- rollout_summary: compact synopsis of what future runs should remember.
+- rollout_slug: short lowercase slug (letters/numbers/_), or null.
+- raw_memory: detailed durable memory blocks with enough context to reuse.
+- If no durable signal exists, return empty strings for rollout_summary/raw_memory and null rollout_slug.

package/src/sdk.ts

@@ -46,6 +46,7 @@ import {
 } from "./internal-urls";
 import { disposeAllKernelSessions } from "./ipy/executor";
 import { discoverAndLoadMCPTools, type MCPManager, type MCPToolsLoadResult } from "./mcp";
+import { buildMemoryToolDeveloperInstructions, startMemoryStartupTask } from "./memories";
 import { AgentSession } from "./session/agent-session";
 import { AuthStorage } from "./session/auth-storage";
 import { convertToLlm } from "./session/messages";
@@ -914,6 +915,7 @@ export async function createAgentSession(options: CreateAgentSessionOptions = {}
 
 	const rebuildSystemPrompt = async (toolNames: string[], tools: Map<string, AgentTool>): Promise<string> => {
 		toolContextStore.setToolNames(toolNames);
+		const memoryInstructions = await buildMemoryToolDeveloperInstructions(agentDir, settings);
 		const defaultPrompt = await buildSystemPromptInternal({
 			cwd,
 			skills,
@@ -923,6 +925,7 @@ export async function createAgentSession(options: CreateAgentSessionOptions = {}
 			toolNames,
 			rules: rulebookRules,
 			skillsSettings: settings.getGroup("skills") as SkillsSettings,
+			appendSystemPrompt: memoryInstructions,
 		});
 
 		if (options.systemPrompt === undefined) {
@@ -939,6 +942,7 @@ export async function createAgentSession(options: CreateAgentSessionOptions = {}
 				rules: rulebookRules,
 				skillsSettings: settings.getGroup("skills") as SkillsSettings,
 				customPrompt: options.systemPrompt,
+				appendSystemPrompt: memoryInstructions,
 			});
 		}
 		return options.systemPrompt(defaultPrompt);
@@ -1133,6 +1137,14 @@ export async function createAgentSession(options: CreateAgentSessionOptions = {}
 		}
 	}
 
+	startMemoryStartupTask({
+		session,
+		settings,
+		modelRegistry,
+		agentDir,
+		taskDepth,
+	});
+
 	debugStartup("sdk:return");
 	return {
 		session,

package/src/session/agent-session.ts

@@ -998,6 +998,14 @@ export class AgentSession {
 		}
 	}
 
+	/** Rebuild the base system prompt using the current active tool set. */
+	async refreshBaseSystemPrompt(): Promise<void> {
+		if (!this.#rebuildSystemPrompt) return;
+		const activeToolNames = this.getActiveToolNames();
+		this.#baseSystemPrompt = await this.#rebuildSystemPrompt(activeToolNames, this.#toolRegistry);
+		this.agent.setSystemPrompt(this.#baseSystemPrompt);
+	}
+
 	/**
 	 * Replace MCP tools in the registry and activate the latest MCP tool set immediately.
 	 * This allows /mcp add/remove/reauth to take effect without restarting the session.

package/src/session/streaming-output.ts

@@ -1,4 +1,4 @@
-import { sanitizeText } from "@oh-my-pi/pi-utils";
+import { sanitizeText } from "@oh-my-pi/pi-natives";
 import { DEFAULT_MAX_BYTES } from "../tools/truncate";
 
 export interface OutputSummary {

package/src/system-prompt.ts

@@ -3,7 +3,7 @@
  */
 import * as os from "node:os";
 import { getSystemInfo as getNativeSystemInfo, type SystemInfo } from "@oh-my-pi/pi-natives";
-import { $env, logger } from "@oh-my-pi/pi-utils";
+import { $env, hasFsCode, isEnoent, logger } from "@oh-my-pi/pi-utils";
 import { getGpuCachePath, getProjectDir } from "@oh-my-pi/pi-utils/dirs";
 import { $ } from "bun";
 import { contextFileCapability } from "./capability/context-file";
@@ -346,19 +346,18 @@ async function getEnvironmentInfo(): Promise<Array<{ label: string; value: strin
 export async function resolvePromptInput(input: string | undefined, description: string): Promise<string | undefined> {
 	if (!input) {
 		return undefined;
+	} else if (input.includes("\n")) {
+		return input;
 	}
 
-	const file = Bun.file(input);
-	if (await file.exists()) {
-		try {
-			return await file.text();
-		} catch (error) {
+	try {
+		return await Bun.file(input).text();
+	} catch (error) {
+		if (!hasFsCode(error, "ENAMETOOLONG") && !isEnoent(error)) {
 			logger.warn(`Could not read ${description} file`, { path: input, error: String(error) });
-			return input;
 		}
+		return input;
 	}
-
-	return input;
 }
 
 export interface LoadContextFilesOptions {

package/src/tools/bash-interactive.ts

@@ -1,5 +1,5 @@
 import type { AgentToolContext } from "@oh-my-pi/pi-agent-core";
-import { type PtyRunResult, PtySession } from "@oh-my-pi/pi-natives";
+import { type PtyRunResult, PtySession, sanitizeText } from "@oh-my-pi/pi-natives";
 import {
 	type Component,
 	matchesKey,
@@ -23,9 +23,8 @@ export interface BashInteractiveResult extends OutputSummary {
 }
 
 function normalizeCaptureChunk(chunk: string): string {
-	const noAnsi = Bun.stripANSI(chunk);
-	const normalized = noAnsi.replace(/\r\n/gu, "\n").replace(/\r/gu, "\n");
-	return normalized.replace(/[\x00-\x08\x0B-\x1F\x7F]/gu, "");
+	const normalized = chunk.replace(/\r\n/gu, "\n").replace(/\r/gu, "\n");
+	return sanitizeText(normalized);
 }
 
 const XtermTerminal = xterm.Terminal;
@@ -168,7 +167,7 @@ class BashInteractiveOverlayComponent implements Component {
 		const visibleLines: string[] = [];
 		for (let i = 0; i < maxContentRows; i++) {
 			const line = buffer.getLine(viewportY + i)?.translateToString(true) ?? "";
-			visibleLines.push(truncateToWidth(replaceTabs(line), innerWidth));
+			visibleLines.push(truncateToWidth(replaceTabs(sanitizeText(line)), innerWidth));
 		}
 		return visibleLines;
 	}
@@ -350,7 +349,12 @@ export async function runInteractiveBashPty(
 					},
 					(err, chunk) => {
 						if (err || !chunk) return;
-						component.appendOutput(chunk);
+						try {
+							component.appendOutput(chunk);
+						} catch {
+							const normalizedChunk = normalizeCaptureChunk(chunk);
+							component.appendOutput(normalizedChunk);
+						}
 						const normalizedChunk = normalizeCaptureChunk(chunk);
 						pendingChunks = pendingChunks.then(() => sink.push(normalizedChunk)).catch(() => {});
 						tui.requestRender();

package/src/tools/fetch.ts

@@ -241,7 +241,7 @@ async function tryContentNegotiation(
 	if (!result.ok) return null;
 
 	const mime = normalizeMime(result.contentType);
-	if (mime.includes("markdown") || mime === "text/plain") {
+	if ((mime.includes("markdown") || mime === "text/plain") && !looksLikeHtml(result.content)) {
 		return { content: result.content, type: result.contentType };
 	}
 

package/src/tools/output-meta.ts

@@ -21,6 +21,7 @@ export interface TruncationMeta {
 	totalBytes: number;
 	outputLines: number;
 	outputBytes: number;
+	maxBytes?: number;
 	/** Line range shown (1-indexed, inclusive) */
 	shownRange?: { start: number; end: number };
 	/** Artifact ID if full output was saved */
@@ -128,6 +129,7 @@ export class OutputMetaBuilder {
 			totalBytes: result.totalBytes,
 			outputLines: result.outputLines,
 			outputBytes: result.outputBytes,
+			maxBytes: result.maxBytes,
 			shownRange: { start: shownStart, end: shownEnd },
 			artifactId,
 			nextOffset: direction === "head" ? shownEnd + 1 : undefined,
@@ -212,6 +214,7 @@ export class OutputMetaBuilder {
 			totalBytes,
 			outputLines,
 			outputBytes,
+			maxBytes: options.maxBytes,
 			shownRange: { start: shownStart, end: shownEnd },
 			nextOffset: options.direction === "head" ? shownEnd + 1 : undefined,
 		};
@@ -319,14 +322,15 @@ export function formatOutputNotice(meta: OutputMeta | undefined): string {
 		const range = t.shownRange;
 		let notice: string;
 
-		if (range) {
+		if (range && range.end >= range.start) {
 			notice = `Showing lines ${range.start}-${range.end} of ${t.totalLines}`;
 		} else {
 			notice = `Showing ${t.outputLines} of ${t.totalLines} lines`;
 		}
 
 		if (t.truncatedBy === "bytes") {
-			notice += ` (${formatSize(t.outputBytes)} limit)`;
+			const maxBytes = t.maxBytes ?? t.outputBytes;
+			notice += ` (${formatSize(maxBytes)} limit)`;
 		}
 
 		if (t.nextOffset != null) {

package/src/web/scrapers/types.ts

@@ -92,6 +92,7 @@ export async function loadPage(url: string, options: LoadPageOptions = {}): Prom
 					"User-Agent": userAgent,
 					Accept: "text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8",
 					"Accept-Language": "en-US,en;q=0.5",
+					"Accept-Encoding": "identity", // Cloudflare Markdown-for-Agents returns corrupted bytes when compression is negotiated
 					...headers,
 				},
 				redirect: "follow",